Skip to content

fix(ci): use client-id instead of deprecated app-id for GitHub App token - #53

Merged
toiroakr merged 1 commit into
mainfrom
fix/use-app-client-id
Sep 5, 2026
Merged

toiroakr merged 1 commit into
mainfrom
fix/use-app-client-id

Conversation

@toiroakr

@toiroakr toiroakr commented Sep 5, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • actions/create-github-app-token@v3.2.0 deprecates the app-id input in favor of client-id. This switches the release workflow to client-id: ${{ vars.APP_CLIENT_ID }}.
  • Requires the repository variable APP_CLIENT_ID to be set to the GitHub App's Client ID (already configured on this repo).

Summary by CodeRabbit

  • Chores
    • Updated the release process configuration for improved compatibility.
    • Updated the package management setup used during releases.

The APP_CLIENT_ID repository variable must be set to the GitHub App's
Client ID for this to work.
@coderabbitai

coderabbitai Bot commented Sep 5, 2026 •

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: f8315d38-76c3-47b7-8ef2-6b74c1a52d57

📥 Commits

Reviewing files that changed from the base of the PR and between 010bf17 and fa41027.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

📝 Walkthrough

Walkthrough

The release workflow now uses vars.APP_CLIENT_ID for GitHub App token generation. The pnpm/action-setup action is updated to v6.1.0.

Changes

Release workflow

Layer / File(s) Summary
Release workflow configuration
.github/workflows/release.yml
The GitHub App token step uses vars.APP_CLIENT_ID. The pnpm/action-setup action is updated from v6.0.10 to v6.1.0.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

A rabbit checks the release trail
The app ID hops into place
pnpm wears a newer coat
Two small changes cross the gate
The workflow bounds away

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/use-app-client-id

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.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Coverage Summary

File               | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
-------------------|---------|----------|---------|---------|-------------------
All files          |   70.89 |    57.74 |      40 |   70.89 |                   
 zod-empty         |       0 |        0 |       0 |       0 |                   
  tsdown.config.ts |       0 |        0 |       0 |       0 | 1-9               
 ...-hook-form/src |       0 |        0 |       0 |       0 |                   
  App.tsx          |       0 |        0 |       0 |       0 | 1-38              
  main.tsx         |       0 |        0 |       0 |       0 | 1-10              
 zod-empty/src     |   82.97 |    58.99 |     100 |   82.97 |                   
  index.ts         |   82.97 |    58.99 |     100 |   82.97 | ...62,364,366-372 
-------------------|---------|----------|---------|---------|-------------------

@pkg-pr-new

pkg-pr-new Bot commented Sep 5, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/zod-empty@53

commit: fa41027

@toiroakr
toiroakr marked this pull request as ready for review September 5, 2026 12:39
@toiroakr
toiroakr merged commit 4c2faa9 into main Sep 5, 2026
13 of 14 checks passed
@toiroakr
toiroakr deleted the fix/use-app-client-id branch September 5, 2026 12:40
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