Skip to content

fix: use Hyper Gonk identity for git commits in release workflows#7482

Merged
paulbalaji merged 1 commit into
mainfrom
pb/fix-gonk-git-identity
Nov 28, 2025
Merged

fix: use Hyper Gonk identity for git commits in release workflows#7482
paulbalaji merged 1 commit into
mainfrom
pb/fix-gonk-git-identity

Conversation

@paulbalaji
Copy link
Copy Markdown
Collaborator

@paulbalaji paulbalaji commented Nov 27, 2025

Summary

The previous fix (PR #7472) used Hyper Gonk token for authentication but still used github-actions[bot] for git commit identity. This caused force pushes to the release branch to still show as github-actions bot in the UI.

Changes

rust-release.yml

Uses the app-slug output from create-github-app-token to properly configure git user name and email as the Hyper Gonk bot.

release.yml (npm releases)

  • Added setupGitUser: false to the changesets action (which defaults to using github-actions[bot])
  • Added explicit git config step before changesets action to use Hyper Gonk identity

The git identity format:

  • user.name: \${{ steps.generate-token.outputs.app-slug }}[bot]
  • user.email: \${{ secrets.HYPER_GONK_APP_ID }}+\${{ steps.generate-token.outputs.app-slug }}[bot]@users.noreply.github.com

Related

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Improved Git configuration in release workflows with enhanced commit attribution handling
    • Updated automated release process to use dynamic bot identification for better commit traceability

✏️ Tip: You can customize this high-level summary in your review settings.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Nov 27, 2025

⚠️ No Changeset found

Latest commit: 6e37985

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

The previous fix (PR #7472) used Hyper Gonk token for authentication
but still used github-actions[bot] for git commit identity. This caused
force pushes to the release branch to still show as github-actions bot.

Changes:
- rust-release.yml: Use app-slug output for git config
- release.yml: Add setupGitUser: false to changesets action and
  configure git identity manually before the action runs

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@paulbalaji paulbalaji force-pushed the pb/fix-gonk-git-identity branch from 96b30a3 to 6e37985 Compare November 27, 2025 16:07
@paulbalaji paulbalaji changed the title fix: use Hyper Gonk identity for git commits in rust-release fix: use Hyper Gonk identity for git commits in release workflows Nov 27, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Nov 27, 2025

📝 Walkthrough

Walkthrough

Two workflow files receive Git configuration updates to dynamically set commit author and email using a GitHub App's slug and ID instead of static bot identifiers. The release.yml file adds a dedicated Git configuration step before the PR creation, while rust-release.yml updates existing configuration across release and publish steps.

Changes

Cohort / File(s) Summary
Git Author Configuration Updates
.github/workflows/release.yml
Adds new "Configure Git for Hyper Gonk" step to set commit author and email using app slug and app ID before PR creation; disables automatic Git user setup in the changesets action
Git Author Configuration Updates
.github/workflows/rust-release.yml
Updates two Git config lines (Release PR and Publish Release steps) to use dynamic bot username and composite email based on GitHub App slug and ID, replacing static github-actions[bot] values

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Attention areas:
    • Verify that app slug and app ID are correctly referenced from prior workflow outputs/secrets in both files
    • Confirm the setupGitUser: false flag in release.yml properly disables conflicting auto-setup
    • Validate that the composite email format works correctly across both workflows
    • Ensure consistency between the manual Git configuration in release.yml and the inline updates in rust-release.yml

Possibly related PRs

Suggested reviewers

  • xeno097
  • Xaroz
  • Mo-Hussain
  • ameten

Poem

🧅 Layers get added, nice and steady—
Bot names shift from static to ready,
App slugs and IDs join the dance,
Git commits get a better chance,
Git knows who's really at the wheel,
This configuration's got real appeal!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: using Hyper Gonk identity for git commits in release workflows, which directly matches the changeset modifications.
Description check ✅ Passed The description is comprehensive, covering the problem, changes to both workflows, the git identity format, and related context, though the Testing section is missing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pb/fix-gonk-git-identity

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b87c5c9 and 6e37985.

📒 Files selected for processing (2)
  • .github/workflows/release.yml (1 hunks)
  • .github/workflows/rust-release.yml (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: paulbalaji
Repo: hyperlane-xyz/hyperlane-monorepo PR: 6943
File: rust/main/config/mainnet_config.json:965-965
Timestamp: 2025-08-26T13:46:37.695Z
Learning: In the repository hyperlane-xyz/hyperlane-monorepo, skip reviewing the file rust/main/config/mainnet_config.json in future code reviews as requested by paulbalaji.
Learnt from: paulbalaji
Repo: hyperlane-xyz/hyperlane-monorepo PR: 6943
File: rust/main/config/mainnet_config.json:965-965
Timestamp: 2025-08-26T13:46:37.695Z
Learning: In the repository hyperlane-xyz/hyperlane-monorepo, skip reviewing the file rust/main/config/testnet_config.json in future code reviews as requested by paulbalaji.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (50)
  • GitHub Check: cli-evm-e2e-matrix (warp-check-5)
  • GitHub Check: cli-evm-e2e-matrix (warp-send)
  • GitHub Check: cli-evm-e2e-matrix (warp-extend-config)
  • GitHub Check: cli-evm-e2e-matrix (warp-deploy-2)
  • GitHub Check: cli-evm-e2e-matrix (warp-rebalancer)
  • GitHub Check: cli-evm-e2e-matrix (warp-apply-1)
  • GitHub Check: cli-evm-e2e-matrix (warp-check-2)
  • GitHub Check: cli-evm-e2e-matrix (warp-extend-recovery)
  • GitHub Check: cli-evm-e2e-matrix (warp-extend-basic)
  • GitHub Check: cli-evm-e2e-matrix (warp-apply-submitters)
  • GitHub Check: cli-evm-e2e-matrix (warp-read)
  • GitHub Check: cli-evm-e2e-matrix (warp-init)
  • GitHub Check: cli-evm-e2e-matrix (warp-deploy-1)
  • GitHub Check: cli-evm-e2e-matrix (relay)
  • GitHub Check: cli-evm-e2e-matrix (core-apply)
  • GitHub Check: cli-evm-e2e-matrix (warp-check-1)
  • GitHub Check: cli-evm-e2e-matrix (core-check)
  • GitHub Check: cli-evm-e2e-matrix (warp-apply-2)
  • GitHub Check: cli-evm-e2e-matrix (warp-check-4)
  • GitHub Check: cli-evm-e2e-matrix (warp-bridge-1)
  • GitHub Check: env-test-matrix (mainnet3, inevm, igp)
  • GitHub Check: cli-evm-e2e-matrix (warp-check-3)
  • GitHub Check: env-test-matrix (mainnet3, inevm, core)
  • GitHub Check: env-test-matrix (mainnet3, optimism, core)
  • GitHub Check: cli-evm-e2e-matrix (warp-bridge-2)
  • GitHub Check: cli-evm-e2e-matrix (core-init)
  • GitHub Check: cli-evm-e2e-matrix (core-deploy)
  • GitHub Check: cli-evm-e2e-matrix (warp-apply-ism-updates)
  • GitHub Check: cli-radix-e2e-matrix (warp-apply-ownership-updates)
  • GitHub Check: env-test-matrix (mainnet3, arbitrum, core)
  • GitHub Check: env-test-matrix (mainnet3, optimism, igp)
  • GitHub Check: env-test-matrix (mainnet3, ethereum, core)
  • GitHub Check: env-test-matrix (mainnet3, ethereum, igp)
  • GitHub Check: env-test-matrix (testnet4, sepolia, core)
  • GitHub Check: cli-radix-e2e-matrix (warp-deploy)
  • GitHub Check: cosmos-sdk-e2e-run
  • GitHub Check: cli-radix-e2e-matrix (warp-apply-route-extension)
  • GitHub Check: cli-radix-e2e-matrix (core-deploy)
  • GitHub Check: cli-cross-chain-e2e-matrix (warp-deploy)
  • GitHub Check: cli-cosmos-e2e-matrix (warp-read)
  • GitHub Check: cli-cross-chain-e2e-matrix (warp-apply)
  • GitHub Check: cli-cosmos-e2e-matrix (core-deploy)
  • GitHub Check: cli-cosmos-e2e-matrix (warp-deploy)
  • GitHub Check: cli-cosmos-e2e-matrix (core-apply)
  • GitHub Check: cli-cosmos-e2e-matrix (core-read)
  • GitHub Check: cli-cosmos-e2e-matrix (core-check)
  • GitHub Check: e2e-matrix (evm)
  • GitHub Check: lander-coverage
  • GitHub Check: lint-rs
  • GitHub Check: test-rs
🔇 Additional comments (2)
.github/workflows/rust-release.yml (1)

186-187: Git identity configured correctly for release PR creation.

The new Hyper Gonk app-slug format looks solid here. The email format matches the GitHub bot convention nicely.

.github/workflows/release.yml (1)

65-76: Git configuration and changesets setup looks right.

Good move adding the dedicated "Configure Git for Hyper Gonk" step and setting setupGitUser: false on the changesets action. This prevents any conflicts and keeps the Gonk identity consistent throughout. The email format and app-slug usage matches what's in rust-release.yml—nice and coherent.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@paulbalaji paulbalaji added this pull request to the merge queue Nov 28, 2025
Merged via the queue into main with commit c32fa59 Nov 28, 2025
84 checks passed
@paulbalaji paulbalaji deleted the pb/fix-gonk-git-identity branch November 28, 2025 16:34
@github-project-automation github-project-automation Bot moved this from In Review to Done in Hyperlane Tasks Nov 28, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (b87c5c9) to head (6e37985).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff      @@
##   main   #7482   +/-   ##
============================
============================
Components Coverage Δ
core ∅ <ø> (∅)
hooks ∅ <ø> (∅)
isms ∅ <ø> (∅)
token ∅ <ø> (∅)
middlewares ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants