Skip to content

Fix: Add safety improvements to GitHub workflows#5

Merged
andreasronge merged 1 commit intomainfrom
fix/workflow-safety-improvements
Dec 1, 2025
Merged

Fix: Add safety improvements to GitHub workflows#5
andreasronge merged 1 commit intomainfrom
fix/workflow-safety-improvements

Conversation

@andreasronge
Copy link
Copy Markdown
Owner

Summary

This PR adds safety improvements to prevent infinite loops and edge cases in the GitHub workflow automation:

  • Fix triage summary @claude trigger: Changed summary template to avoid @claude text that could trigger claude.yml
  • Standardize concurrency groups: PR-based workflows now use PR number consistently
  • Add PAT warnings: Workflows now warn if PAT_WORKFLOW_TRIGGER is not set
  • Fork PR handling: Detect and skip fork PRs with explanatory comment
  • PM open PR check: Prevent PM from creating parallel work when PRs are open
  • Self-triggering prevention: Added warning to claude.yml prompt about not posting @claude in comments

Test plan

  • Verify CI passes
  • Verify Claude Code Review runs and posts review
  • Verify triage runs after review completes
  • Check that PAT warning does NOT appear (since PAT is now configured)

🤖 Generated with Claude Code

- Fix triage summary containing @claude text that could trigger loops
- Standardize concurrency groups (PR number for claude.yml/code-review)
- Add PAT_WORKFLOW_TRIGGER availability warnings
- Add fork PR detection and skip with explanatory comment
- Add open PR check to PM workflow to prevent parallel work
- Add self-triggering prevention warning to claude.yml prompt

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@andreasronge andreasronge merged commit a6993c7 into main Dec 1, 2025
1 of 2 checks passed
@andreasronge andreasronge deleted the fix/workflow-safety-improvements branch December 1, 2025 15:53
@claude claude bot mentioned this pull request Jan 11, 2026
6 tasks
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