Skip to content

fix(ci): switch release workflow to GitHub App token auth#162

Merged
iamfj merged 1 commit intonextfrom
fix/release-gh-app-token
Apr 23, 2026
Merged

fix(ci): switch release workflow to GitHub App token auth#162
iamfj merged 1 commit intonextfrom
fix/release-gh-app-token

Conversation

@iamfj
Copy link
Copy Markdown
Member

@iamfj iamfj commented Apr 23, 2026

What does this PR do?

Migrates release automation workflows from deploy-key/default workflow-token behavior to GitHub App installation token auth, so release-related pushes use the configured ruleset bypass actor.

Type of change

  • Bug fix
  • New feature
  • Refactor (no behavior change)
  • Documentation
  • Tests
  • Build / CI

Checklist

  • npm run check:ci passes (lint + format)
  • npx tsc --noEmit passes (type check)
  • npm test passes (unit tests)
  • New code has tests (happy path + primary error case)
  • Commit messages follow Conventional Commits

Testing

Executed in the PR worktree:

  • npm run check:ci
  • npx tsc --noEmit
  • npm test

Notes:

  • npm run check:ci reports an existing Biome schema-version info message, but exits successfully.
  • npm test outputs repeated Paste your Linear API token: prompt text from existing test behavior; suite still passes.

Notes for reviewers

.github/workflows/release-check.yml

  • Removes deploy-key usage.
  • Keeps actions/checkout with persist-credentials: false so checkout token is not used for pushes.
  • Creates short-lived GitHub App installation token via actions/create-github-app-token@v2.
  • Resolves app bot identity (<app-slug>[bot]) and configures git author email as <id>+<app-slug>[bot]@users.noreply.github.com.
  • Uses app token for semantic-release (GH_TOKEN and GITHUB_TOKEN).

.github/workflows/sync-main-release-back-to-next.yml

  • Removes deploy-key usage.
  • Keeps actions/checkout with persist-credentials: false.
  • Creates short-lived GitHub App installation token via actions/create-github-app-token@v2.
  • Resolves app bot identity and configures git author to app bot noreply identity.
  • Sets origin remote to authenticated HTTPS using app token for git push origin HEAD:next.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8a0ff64697

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/release-check.yml
Switch release-check and back-sync workflows to App token auth.

- create installation token in both workflows

- keep checkout persist-credentials disabled

- use App token for semantic-release and back-sync pushes

- set git author to App bot noreply identity
@iamfj iamfj force-pushed the fix/release-gh-app-token branch from 84d8194 to f0ba4e0 Compare April 23, 2026 06:49
@iamfj
Copy link
Copy Markdown
Member Author

iamfj commented Apr 23, 2026

@codex review

@iamfj iamfj merged commit fd0f266 into next Apr 23, 2026
10 checks passed
@iamfj iamfj deleted the fix/release-gh-app-token branch April 23, 2026 06:55
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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