Skip to content

Remove "Generated by construct" watermark from git commits#5

Merged
mtsfoni merged 2 commits intomainfrom
copilot/remove-watermark-from-commits
Mar 12, 2026
Merged

Remove "Generated by construct" watermark from git commits#5
mtsfoni merged 2 commits intomainfrom
copilot/remove-watermark-from-commits

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

construct was silently appending a Generated by construct trailer to every commit made inside a session via a commit-msg hook injected at container startup. This removes that watermark entirely.

Changes

  • internal/runner/runner.go — drop the commit-msg hook block from generatedEntrypoint() (mkdir .githooks, hook script, chmod +x, git config core.hooksPath); clean up the related comment in buildRunArgs
  • internal/runner/runner_test.go — replace the two hook-specific tests with a single TestGeneratedEntrypoint_EndsWithExec
  • docs/spec/git-identity.md — remove section 3 ("Commit-msg hook") and update the files-changed table
  • docs/spec/image-build-layers.md — remove hook step from entrypoint description and hook row from the layer-responsibility table
  • docs/spec/construct-agents-md.md — remove hook reference from files-changed table
  • docs/threat-model.md — remove mention of the trailer from the attribution note
  • CHANGELOG.md — remove the commit-msg hook bullet from [Unreleased]
Original prompt

This section details on the original issue you should resolve

<issue_title>please remove the watermark we set in git commits</issue_title>
<issue_description>Also check if it is somewhere in the spec.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: mtsfoni <80639729+mtsfoni@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix removal of watermark in git commits Remove "Generated by construct" watermark from git commits Mar 12, 2026
@mtsfoni mtsfoni marked this pull request as ready for review March 12, 2026 07:15
@mtsfoni mtsfoni merged commit be4df30 into main Mar 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

please remove the watermark we set in git commits

2 participants