Skip to content

feat: strands review command#326

Merged
aidandaly24 merged 2 commits intomainfrom
feat/strands-review-command
Feb 18, 2026
Merged

feat: strands review command#326
aidandaly24 merged 2 commits intomainfrom
feat/strands-review-command

Conversation

@aidandaly24
Copy link
Contributor

Description

Adds a /strands review command that runs a comment-only PR review agent. When a user comments /strands review on a PR, the agent gathers context (diff, linked issue, repo docs), evaluates the change, and posts review comments — without modifying any code.

Two changes:

  • New task-reviewer.sop.md SOP that directs the agent through a 4-step review process (gather context → understand code → evaluate → post comments) with explicit forbidden actions (no file modifications, commits, or merges)
  • Updated process-inputs.cjs to detect review subcommand and route it to the new reviewer mode with its own SOP

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Integration test failures (10 files / 34 tests) are pre-existing on main — identical results on both branches.

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

@aidandaly24 aidandaly24 requested a review from a team February 17, 2026 21:57
@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 28.26% 1807 / 6393
🔵 Statements 27.59% 1875 / 6794
🔵 Functions 27.36% 356 / 1301
🔵 Branches 24.52% 918 / 3743
Generated in workflow #415 for commit c59eb4e by the Vitest Coverage Report Action

Copy link
Contributor

@notgitika notgitika left a comment

Choose a reason for hiding this comment

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

awesome

@aidandaly24 aidandaly24 force-pushed the feat/strands-review-command branch from 919de36 to c59eb4e Compare February 18, 2026 14:34
@aidandaly24 aidandaly24 merged commit 93ed835 into main Feb 18, 2026
15 checks passed
@aidandaly24 aidandaly24 deleted the feat/strands-review-command branch February 18, 2026 14:46
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

Comments