Skip to content

fix: claude mentions#844

Merged
dknopik merged 1 commit intosigp:stablefrom
shane-moore:fix/claude-mentions
Feb 26, 2026
Merged

fix: claude mentions#844
dknopik merged 1 commit intosigp:stablefrom
shane-moore:fix/claude-mentions

Conversation

@shane-moore
Copy link
Member

Issue Addressed

@claude mentions on PRs stopped posting comments after PR #830

Proposed Changes

  • Remove prompt input from claude-mentions workflow to restore tag mode (the prompt input switched the action to agent mode, which doesn't manage comment lifecycle)
  • Inject review criteria via --append-system-prompt in claude_args instead, preserving tag mode while keeping review guidelines
  • Extract shared review criteria to .github/prompts/review.md as single source of truth

Additional Info

N/A

Copy link
Member

@dknopik dknopik left a comment

Choose a reason for hiding this comment

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

LGTM, lets see what happens

@dknopik dknopik merged commit d6c8606 into sigp:stable Feb 26, 2026
13 of 15 checks passed
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