Skip to content

Migrate PR review to packaged cagent-action sub-actions#1543

Merged
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:implement-new-pr-review-action
Feb 2, 2026
Merged

Migrate PR review to packaged cagent-action sub-actions#1543
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:implement-new-pr-review-action

Conversation

@derekmisler
Copy link
Contributor

@derekmisler derekmisler commented Jan 30, 2026

Summary

Migrates the PR review workflow from inline cagent YAML definitions to the new packaged docker/cagent-action sub-actions, dramatically reducing workflow complexity and maintenance burden.

Changes

  • Delete inline agent configs: Removed .github/workflows/agents/pr-review.yaml and pr-review-feedback.yaml (221 lines of agent configuration)
  • Simplify main review job: Replaced ~100 lines of workflow steps with single docker/cagent-action/review-pr@latest action call
  • Simplify feedback learning job: Replaced ~80 lines with single docker/cagent-action/review-pr/learn@latest action call
  • Remove manual orchestration: Eliminated context building, memory caching, reaction handling, and metadata gathering (now handled by the action)

Before vs After

Aspect Before After
Workflow lines ~200 ~35
Agent config files 2 files (221 lines) 0 (packaged in action)
Memory management Manual cache/restore Handled by action
PR context building Manual bash scripts Handled by action

Test Plan (after merging to main, because that's how GHA works)

  • Trigger a review by commenting /review on a PR
  • Reply to an agent review comment to verify feedback learning works
  • Verify review comments still contain <!-- cagent-review --> marker
  • Check that APPROVE/REQUEST_CHANGES/COMMENT events are posted correctly

@derekmisler derekmisler self-assigned this Jan 30, 2026
@derekmisler derekmisler marked this pull request as ready for review January 30, 2026 20:00
@derekmisler derekmisler requested a review from a team as a code owner January 30, 2026 20:00
@derekmisler derekmisler merged commit 3033b7c into docker:main Feb 2, 2026
5 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.

3 participants