Skip to content

fix(ci): disable 'Fix this' links in AI review comments - #6093

Merged
jam-jee merged 1 commit into
aws:masterfrom
jam-jee:disable-fix-links
Jul 23, 2026
Merged

jam-jee merged 1 commit into
aws:masterfrom
jam-jee:disable-fix-links

Conversation

@jam-jee

@jam-jee jam-jee commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

What

Sets include_fix_links: false on the claude-code-action step in the AI Code Review workflow.

Why

By default the action appends a "Fix this" deep-link to each review comment that opens Claude Code with the issue's context. For this repo that link is noise:

  • ~90% of PRs come from external forks — those contributors can't use a Claude Code deep-link into this setup.
  • It adds clutter to every inline comment.

include_fix_links is the action's official input for this (action.yml, described as "Include 'Fix this' links in PR code review feedback that open Claude Code with context to fix the issue"). It's independent of track_progress, so the live progress comment is unaffected.

Effect

Review comments will no longer contain the "Fix this" link. No other behavior changes; no infra changes.

Sets include_fix_links: false on claude-code-action. The 'Fix this' deep-links
open Claude Code with issue context — useless for external contributors (~90% of
PRs here) and just noise in review comments.
@jam-jee

jam-jee commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator Author

overriding and merging to fix review comments posted by AI code reviewer immediately.

@jam-jee
jam-jee merged commit 60165a8 into aws:master Jul 23, 2026
4 of 5 checks passed

This branch was previously deployed

1 inactive deployment
auto-approve — f31288c5 Deployed Jul 23, 2026 by jam-jee via wait-for-approval #1102
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