Skip to content

fix: enforce attribution messages - #5

Merged
nothingnesses merged 1 commit into
mainfrom
impl/enforce-attribution-messages
Sep 2, 2026
Merged

fix: enforce attribution messages#5
nothingnesses merged 1 commit into
mainfrom
impl/enforce-attribution-messages

Conversation

@nothingnesses

Copy link
Copy Markdown
Owner

Summary

  • Apply one structural attribution policy to reachable commit messages and pull-request metadata.
  • Parse the GitHub event file without evaluating untrusted title or body text.
  • Re-run the required workflow after pull-request metadata edits.
  • Cover Claude, GPT or Codex, Gemini, Copilot, safe prose, malformed events, and existing history rules with 43 scratch cases.

Review

Independent review found a versioned GPT model-name gap and a carriage-return-only hardening case. Separate triage confirmed only the GPT gap under the accepted criteria. One scoped fix added direct commit and pull-request tests, and focused verification passed.

Verification

  • just ci
  • 429 unit tests and all integration tests
  • 43 attribution regression cases
  • Complete reachable-history attribution check

@nothingnesses
nothingnesses merged commit e9b2b61 into main Sep 2, 2026
1 check passed
@nothingnesses
nothingnesses deleted the impl/enforce-attribution-messages branch September 2, 2026 11:18
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.

1 participant