Skip to content

Fix reusable workflow secret name (PR_REVIEW_TRACKER_PROJECT_TOKEN)#2

Open
hatton wants to merge 1 commit into
mainfrom
fix-pr-review-tracker-secret-name
Open

Fix reusable workflow secret name (PR_REVIEW_TRACKER_PROJECT_TOKEN)#2
hatton wants to merge 1 commit into
mainfrom
fix-pr-review-tracker-secret-name

Conversation

@hatton

@hatton hatton commented Jun 15, 2026

Copy link
Copy Markdown
Member

[Claude Opus 4.8] The secret rename landed after #1 was merged, so main still declares the old PROJECT_TOKEN while caller repos now pass PR_REVIEW_TRACKER_PROJECT_TOKEN. That mismatch makes caller runs fail at startup (startup_failure). This aligns the reusable workflow with the callers.

🤖 Generated with Claude Code

The rename landed after PR #1 was merged, so main still declared the old PROJECT_TOKEN while callers now pass PR_REVIEW_TRACKER_PROJECT_TOKEN. The mismatch caused caller runs to fail at startup. This aligns main with the callers.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 15, 2026 23:20
@greptile-apps

greptile-apps Bot commented Jun 15, 2026

Copy link
Copy Markdown

Greptile Summary

Fixes a secret name mismatch between the reusable workflow and its callers. After the rename landed post-#1, the reusable workflow still declared PROJECT_TOKEN while callers were already passing PR_REVIEW_TRACKER_PROJECT_TOKEN, causing startup_failure on every caller run.

  • Renames the workflow_call secret declaration from PROJECT_TOKEN to PR_REVIEW_TRACKER_PROJECT_TOKEN and updates the single ${{ secrets.PR_REVIEW_TRACKER_PROJECT_TOKEN }} reference in the GH_TOKEN env var — the change is atomic and complete.
  • Updates the header comment to note that callers must map the secret explicitly and that BloomBooks now requires a fine-grained PAT instead of a classic PAT.

Important Files Changed

Filename Overview
.github/workflows/pr-review-tracker.yml Renames the workflow_call secret from PROJECT_TOKEN to PR_REVIEW_TRACKER_PROJECT_TOKEN in both the declaration and the single usage, and updates the header comment to reflect fine-grained PAT requirements; change is consistent and complete.

Reviews (1): Last reviewed commit: "Fix reusable workflow secret name to PR_..." | Re-trigger Greptile

Copilot AI 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.

Pull request overview

Updates the reusable PR Review Tracker workflow to expect the renamed secret (PR_REVIEW_TRACKER_PROJECT_TOKEN) so caller repositories can successfully start runs and authenticate to write to the BloomBooks org Projects (v2) board.

Changes:

  • Renamed the reusable workflow workflow_call required secret from PROJECT_TOKEN to PR_REVIEW_TRACKER_PROJECT_TOKEN.
  • Updated the GH_TOKEN environment variable wiring to use the new secret name.
  • Adjusted the header documentation to describe the new secret and token expectations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants