Skip to content

Nudge users to enable the PR comment when github-token is missing - #163

Merged
reuvenharrison merged 1 commit into
mainfrom
fix/free-review-token-nudge
Jun 14, 2026
Merged

Nudge users to enable the PR comment when github-token is missing#163
reuvenharrison merged 1 commit into
mainfrom
fix/free-review-token-nudge

Conversation

@reuvenharrison

Copy link
Copy Markdown
Contributor

P0 launch-hardening (#1). After upgrading to v0.1.0 by bumping only the version (not adding github-token + permissions: pull-requests: write), the free breaking/changelog action posted the review link to the job summary only, silently — the default for every upgrader, landing on the low-traffic surface the whole encrypted-review effort was meant to escape.

Now, when a review link is produced on a pull request but no github-token is set, the action emits a ::notice:: explaining how to get the review as a PR comment (with a docs link). It fires only when there's a review link + PR context, so no-changes runs and non-PR events stay quiet, and fork PRs (which have a read-only token present) keep the existing 403-fallback notice.

🤖 Generated with Claude Code

The default after upgrading to v0.1.0 (version bumped, but github-token +
pull-requests: write not added) was a silent job-summary-only experience.
Now, when a review link is produced on a pull_request but no github-token
is set, emit a ::notice:: telling the user how to get it as a PR comment.
Fires only with a review link + PR context, so no-changes runs and non-PR
events stay quiet; fork PRs (read-only token present) keep the existing
403-fallback path.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@reuvenharrison
reuvenharrison merged commit dc68976 into main Jun 14, 2026
56 checks passed
@reuvenharrison
reuvenharrison deleted the fix/free-review-token-nudge branch June 14, 2026 11:45
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