Skip to content

ci: use BOT_CLIENT_ID secret for the app token#202

Merged
joryirving merged 1 commit into
mainfrom
chore/bot-client-id
Jun 10, 2026
Merged

ci: use BOT_CLIENT_ID secret for the app token#202
joryirving merged 1 commit into
mainfrom
chore/bot-client-id

Conversation

@joryirving

Copy link
Copy Markdown
Collaborator

Completes the BOT_APP_ID → BOT_CLIENT_ID migration started in the v1.2.2 upgrade PR — this was the remaining reference outside the AI-review workflow.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@its-saffron its-saffron Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

AI Automated Review

Full PR review.

Analysis engine: review@https://litellm.jory.dev/v1 (openai)

Recommendation

Approved. This is a documentation update to align the AI review rules with the ongoing migration of GitHub App credentials.

Change-by-change findings

  • .github/ai-review-rules.md: Updated the documentation table to replace the deprecated BOT_APP_ID with BOT_CLIENT_ID. This correctly reflects the credential name used for the GitHub App client ID in the current CI configuration.

Standards Compliance

  • Repository Standards: The change is purely documentation-based and does not affect the game logic or core UX constraints defined in AGENTS.md.
  • CI/CD: The change aligns with the migration mentioned in the PR description, ensuring the documentation accurately describes the secrets required for the AI review workflow.

Linked Issue Fit

  • The PR body notes this completes a migration started in a previous PR (v1.2.2 upgrade), ensuring consistency across the repository's documentation and CI configuration.

@joryirving joryirving merged commit d7b3fa0 into main Jun 10, 2026
4 checks passed
@joryirving joryirving deleted the chore/bot-client-id branch June 10, 2026 18:27
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