Skip to content

feat(ci): use GraphQL createCommitOnBranch for signed commits#286

Merged
guglez merged 2 commits into
mainfrom
feat/signed-commits-auto-deploy
May 27, 2026
Merged

feat(ci): use GraphQL createCommitOnBranch for signed commits#286
guglez merged 2 commits into
mainfrom
feat/signed-commits-auto-deploy

Conversation

@guglez
Copy link
Copy Markdown
Contributor

@guglez guglez commented May 27, 2026

Summary

Replaces git commit + git push with the GitHub GraphQL createCommitOnBranch mutation.

Why: The required_signatures ruleset on ChainSafe/infra-kubernetes requires all commits on main to be signed. Commits via git push are unsigned and block auto-merge. Commits via the GitHub GraphQL API are automatically signed by GitHub and show as Verified — no bypass actor or GPG key needed.

Changes:

  • Branch creation via REST API
  • Signed commit via GraphQL createCommitOnBranch
  • Raw file fetch via Accept: application/vnd.github.raw (no base64 decode)
  • Idempotency check now compares the branch (not main)
  • git commit / git push / git config removed entirely

Closes #285
Refs ChainSafe/infrastructure-general#1246

🤖 Generated with Claude Code

Replace git commit+push with GitHub GraphQL API to produce
Verified/signed commits — satisfies required_signatures ruleset
in infra-kubernetes without needing a bypass actor.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@guglez guglez enabled auto-merge (squash) May 27, 2026 17:34
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@05a1ba2). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #286   +/-   ##
=======================================
  Coverage        ?   31.28%           
=======================================
  Files           ?      131           
  Lines           ?    10179           
  Branches        ?        0           
=======================================
  Hits            ?     3185           
  Misses          ?     6724           
  Partials        ?      270           
Flag Coverage Δ
unittests 31.28% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@salindne salindne self-requested a review May 27, 2026 17:42
@guglez guglez merged commit 38ad918 into main May 27, 2026
2 checks passed
@guglez guglez deleted the feat/signed-commits-auto-deploy branch May 27, 2026 17:42
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.

3 participants