Skip to content

chore(CI): regenerate app token before late checkout in nightly_detect_failure#1658

Merged
kim-em merged 1 commit intomainfrom
fix-nightly-detect-failure-token
Feb 9, 2026
Merged

chore(CI): regenerate app token before late checkout in nightly_detect_failure#1658
kim-em merged 1 commit intomainfrom
fix-nightly-detect-failure-token

Conversation

@kim-em
Copy link
Copy Markdown
Collaborator

@kim-em kim-em commented Feb 6, 2026

This PR fixes a potential token expiration issue in nightly_detect_failure.yml.

In the handle_success job, the GitHub App token is generated at the start of the job but then used again much later for a second checkout and PR creation. Between these uses, there are:

  • Zulip API calls (Python scripts)
  • GitHub API pagination calls
  • Multiple github-script steps
  • Setup operations

This can exceed 1 hour under adverse conditions (slow network, API retries), causing the token to expire before the final checkout step.

Fix: Regenerate a fresh token (app-token-2) just before the second checkout.

🤖 Prepared with Claude Code

…t_failure

GitHub App installation tokens expire after 1 hour. In the
`handle_success` job, the token is generated at the start but then
used again much later (after Zulip API calls, GitHub API pagination,
Python scripts, etc.) for a second checkout and PR creation.

This can cause the token to expire before the final checkout step,
leading to authentication failures.

Fix by regenerating a fresh token just before the second checkout.
@github-actions github-actions bot added the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Feb 6, 2026
@kim-em kim-em added this pull request to the merge queue Feb 9, 2026
Merged via the queue into main with commit 100083c Feb 9, 2026
3 checks passed
@github-actions github-actions bot removed the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Feb 9, 2026
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