Skip to content

feat(ci): sign-artifacts opens an auto-merge signature PR (#341) - #655

Merged
enricopiovesan merged 1 commit into
mainfrom
claude/issue-341-signature-auto-pr
Nov 4, 2026
Merged

enricopiovesan merged 1 commit into
mainfrom
claude/issue-341-signature-auto-pr

Conversation

@enricopiovesan

Copy link
Copy Markdown
Contributor

Summary

Governing Spec

  • 001-registry-foundation

Project Item

Definition of Done

  • Decision recorded (entry 113 + new entry 136)
  • sign-artifacts updated to open the signature PR via a CI-triggering token
  • docs/artifact-signing.md step 3 updated
  • Owner provisions SIGNATURE_PR_TOKEN (cannot be done by an agent)
  • Verified on the next real new-capability publish (issue stays open until then)

Validation

  • Workflow YAML parses; the step's shell passes bash -n.
  • Ran the step script in a sandbox git repo with a local bare remote and a stub gh: no changes (exits clean), no token (warning only), token (branch pushed, PR created, auto-merge queued with the expected args).
  • Real end-to-end path is only observable on a real publish with the token present.

🤖 Generated with Claude Code

Implements decision-log entry 113 option 1: with SIGNATURE_PR_TOKEN set the
job pushes chore/signatures-<sha> and queues a squash auto-merge PR; unset,
it warns and the artifact fallback stays. Drops the always-rejected direct
push to main.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@enricopiovesan
enricopiovesan enabled auto-merge (squash) November 4, 2026 17:29
@enricopiovesan
enricopiovesan merged commit 3984b73 into main Nov 4, 2026
11 checks passed
@enricopiovesan
enricopiovesan deleted the claude/issue-341-signature-auto-pr branch November 4, 2026 17:52
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