Skip to content

Conversation

@code-yeongyu
Copy link
Owner

Summary

Add automatic code review mode to the Sisyphus agent workflow, enabling it to proactively review PRs when:

  • Review is requested from sisyphus-dev-ai
  • PR is opened/reopened/marked ready for review by maintainers

Changes

Workflow (sisyphus-agent.yml)

  • New trigger: pull_request events (opened, ready_for_review, review_requested, reopened)
  • Concurrency control: One workflow run per PR/issue to prevent duplicates
  • Recent review skip: Avoids spam by skipping if reviewed within 10 minutes
  • Mode detection: Distinguishes between review mode (PR events) and agent mode (@mention)
  • PR context collection: GraphQL query fetches diff, changed files, and unresolved review threads
  • Maintainer-only restriction: All triggers require OWNER, MEMBER, or COLLABORATOR association

New Prompt Templates

  • .github/sisyphus/prompts/review.md - Instructions for PR review mode
  • .github/sisyphus/prompts/agent.md - Instructions for @mention agent mode

Testing

  • Request review from sisyphus-dev-ai on a test PR
  • Verify maintainer-only restriction works (non-maintainer PRs should be skipped)
  • Verify recent review skip works (re-requesting within 10 min should skip)

🤖 GENERATED WITH ASSISTANCE OF OhMyOpenCode

- Add pull_request trigger for opened, ready_for_review, review_requested, reopened events
- Implement review mode with PR context collection (diff, files, unresolved threads via GraphQL)
- Add concurrency control to prevent duplicate runs per PR/issue
- Add recent review skip (10 min threshold) to prevent spam reviews
- Create separate prompt templates for review and agent modes
- Restrict all triggers to maintainers only (OWNER, MEMBER, COLLABORATOR)

🤖 GENERATED WITH ASSISTANCE OF [OhMyOpenCode](https://github.com/code-yeongyu/oh-my-opencode)
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