Skip to content

ci(claude): mint the app token with the client id - #203

Merged
Troublor merged 1 commit into
mainfrom
troublor/ci/claude-client-id
Sep 8, 2026
Merged

Troublor merged 1 commit into
mainfrom
troublor/ci/claude-client-id

Conversation

@Troublor

@Troublor Troublor commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

create-github-app-token@v3 deprecates app-id in favour of client-id; the release workflows (#201) already use client-id: ${{ vars.MEGA_MAXWELL_CLIENT_ID }}. This moves claude.yml to the same input. Same app, same private key.

🤖 Generated with Claude Code

create-github-app-token@v3 deprecates app-id in favour of client-id; the
release workflows already use it. Same app, same private key.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013eLFMaDpEwgDzyDBzCQzLH
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-08T07:28:27.363561Z 93f2a3f PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@mega-maxwell

mega-maxwell Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Claude review status

Living comment — rewritten in place. The review workflow keeps this single comment up to date instead of posting a new one each round, so it always describes the latest reviewed commit and the earlier text is intentionally gone. No reply is needed here; reply to a finding in its own review thread, and answer an open question in a reply on this PR. The next review round reconciles your answer.

🛠️ Review did not finish

Attempted head 93f2a3f5 · updated 2026-09-08T07:26:31+00:00

This round did not publish: MODEL_NO_OUTPUT in phase compile. Anything listed below is from the last round that did. Re-run the workflow or push a new commit to try again.

@Troublor
Troublor merged commit 5a5ddc2 into main Sep 8, 2026
12 of 13 checks passed
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