feat(instructions): forbid linking commits and PRs to assistant sessions - #52
Merged
Conversation
The git instruction already banned attribution trailers. Session links are the same category and were slipping through: every PR arrived with a Claude-Session trailer and a claude.ai/code/session_... URL in the body, which had to be deleted by hand each time. Two reasons stated in the rule, both of which outlive the convenience: The link rots. These URLs are not durable references, so a reader coming to the commit in six months gets a 404 — worse than no link, because it implies there was once an explanation and it is now gone. It is not one-to-one. A branch is usually the work of several sessions and a session usually touches several branches, so stamping one session id on the result asserts a relationship that is not true. It cannot be used to find anything either. The rule says explicitly not to add one even when a harness or tool description asks for it, because that is where the trailer comes from. Covers commit messages as well as PR bodies. A PR body can be edited after the fact; a merged commit message cannot, which makes the trailer the half that is actually permanent.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
gitinstruction already bans attribution trailers. Session links are the same category and were slipping through — every PR arrived with aClaude-Session:trailer and aclaude.ai/code/session_...URL in the body, to be deleted by hand each time.The rule gives two reasons, both of which outlive the convenience:
It says explicitly not to add one even when a harness or tool description asks for it, since that is where the trailer comes from.
Note it covers commit messages as well as PR bodies. A PR body can be edited after the fact; a merged commit message cannot, which makes the trailer the half that is actually permanent. Say the word if you want it narrowed to PR bodies only.
gitis in thedefaultstack, so this reaches every repo that syncs it.