Skip to content

feat(commands): @gittensory review / re-review command — parse + classify + dispatch to existing re-review action #2163

Description

@JSONbored

Wire the @gittensory review (alias re-review) PR-comment command: on an authorized maintainer comment, re-run the review by dispatching to the EXISTING reReviewStoredPullRequest path (src/queue/processors.ts:1737) — the action exists; this bounty wires the command to it, mirroring maybeProcessGateOverrideCommand (:5977). Hard constraint from #1960: this affects only auto-review, never the gate disposition. The actual real-permission auth call (authorizePrActionActor) is maintainer-owned; this bounty stops at building the classifier + skip/audit recording + the dispatch call and its tests.

Deliverables

  • Add maybeProcessReviewCommand handler using the shared classifier (or the inline preamble) + evaluateCommandAuthorization for command "review", then call the existing re-review path (reReviewStoredPullRequest / enqueue) — do NOT reimplement review
  • Record a github_app.review_command_* audit + product-usage event on skip/denied/completed, matching the gate-override recording shape (src/queue/processors.ts:6117)
  • Register the handler in the comment-dispatch chain next to maybeProcessGateOverrideCommand (src/queue/processors.ts:3044) behind the existing pattern
  • Assert the disposition/gate is untouched (the command re-reviews; it never flips advisory) — add a test asserting no gate-mode mutation
  • Unit tests: authorized→dispatch, unauthorized→denied audit, bot/action/missing-target skips, re-review alias resolves here

References

  • src/queue/processors.ts:1737 (reReviewStoredPullRequest)
  • src/queue/processors.ts:5977 (handler pattern)
  • src/queue/processors.ts:3044 (dispatch chain)
  • src/settings/command-authorization.ts:89 (evaluateCommandAuthorization)
  • #1960

Part of #1960.


size: S · held for later release (gittensor:feature once activated)

Metadata

Metadata

Assignees

No one assigned

    Labels

    gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.

    Projects

    Status
    In Progress

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions