feat(commands): wire @gittensory review / re-review command (#2163)

[jimcody1995]

Summary

• Add maybeProcessReviewCommand for @gittensory review / re-review: shared classifyPrCommandRequest guards, authorizePrActionActor for the review command (including confirmed-miner detection), then dispatch to the existing reReviewStoredPullRequest path with force: true — no gate-disposition mutation (PR command surface — @gittensory review / pause / resume / resolve / help / configuration / explain #1960).
• Record github_app.review_command_{skipped,denied,completed} audit events and matching product-usage telemetry, mirroring the gate-override skip/denied/completed shape.
• Register the handler in the issue-comment dispatch chain immediately before gate-override.

Test plan

• Authorized maintainer @gittensory review dispatches re-review and records review_command_completed without changing gate_check_mode or emitting gate_overridden.
• @gittensory re-review alias resolves to the same handler.
• Unauthorized actor records review_command_denied.
• Bot author, edited comment, and missing PR target record review_command_skipped with classifier reasons.
• Missing cached PR row records cached_pr_missing skip.
• agentPaused / agentDryRun skip without dispatching re-review.
• Completed/skip metadata fallbacks (headSha, commentId, repoFullName) covered for 100% patch branch coverage.
• npm run typecheck
• Full npm run test:coverage (local)

Closes #2163

Made with Cursor

[loopover-orb]

Warning

🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨

⏸️ Gittensory review result - manual review recommended

_{Review updated: 2026-07-07 14:08:38 UTC}

2 files · 1 AI reviewer · no blockers · readiness 100/100 · CI pending · blocked

⏸️ Suggested Action - Manual Review

Review summary
This PR wires a new `@​gittensory review`/`re-review` slash command by reusing the existing `classifyPrCommandRequest` → `authorizePrActionActor` → `reReviewStoredPullRequest` pipeline already proven by the gate-override command, and it registers the handler ahead of gate-override in the issue-comment dispatch chain. The flow is correct: it forces a re-review via `reReviewStoredPullRequest(..., { force: true })` without touching `gate_check_mode` or emitting `gate_overridden`, which the new test explicitly asserts. Test coverage is thorough (authorized dispatch, alias resolution, denial, bot/edited/missing-PR classifier skips, missing cached row, paused/dry-run skip, and metadata fallbacks), and the PR is correctly scoped to and closes #2163.

Nits — 5 non-blocking

• src/queue/processors.ts: `recordReviewCommandSkip` records the audit event with `outcome: "completed"` while the paired product-usage event uses `outcome: "skipped"` for the same skip — confirm this mirrors gate-override's existing convention intentionally rather than being a copy-paste slip, since it reads oddly in the audit log.
• src/queue/processors.ts: `commandName: "review" as GittensoryMentionCommandName` uses a type cast — if `"review"` isn't already a member of that union, prefer widening the type properly instead of casting past it.
• test/unit/queue.test.ts: several inline string literals like `"installation-token"` trip generic-secret-assignment scanners; consider a shared test constant/fixture name that scanners won't flag, even though these are clearly fake tokens.
• External brief flags magic numbers `2163`/`1960` in a doc comment at src/queue/processors.ts:10363-10365 — these are just issue-number references in prose, not logic constants, so no action needed there.
• Add a one-line comment (or reuse an existing helper) clarifying why `getPullRequest` (cached) is fetched before authorization rather than after, to make the ordering intentional for future readers touching this shared pattern.

Signal	Result	Evidence
Code review	✅ No blockers	1 reviewer
Linked issue	✅ Linked	#2163
Related work	✅ No active overlap found	No same-issue or scoped active PR overlap found.
Change scope	✅ 20/20	Low review scope from cached public metadata (1 linked issue).
Validation posture	✅ 25/25	PR body includes validation/test evidence.
Contributor workload	✅ 10/10	Author activity: 153 registered-repo PR(s), 98 merged, 0 issue(s).
Contributor context	✅ Confirmed Gittensor contributor	jimcody1995; Gittensor profile; 153 PR(s), 0 issue(s).
Gate result	✅ Passing	No configured blocker found.

Review context

• Author: jimcody1995
• Role context: outside_contributor
• Public audience mode: oss maintainer
• Lane context: Repository is configured for direct PR review.
• Public profile languages: TypeScript, JavaScript
• Official Gittensor activity: 153 PR(s), 0 issue(s).
• PR-specific overlap: none found.

Contributor next steps

• Keep the PR focused and include validation evidence before maintainer review.

Signal definitions

• Related work = same linked issue, overlapping active PRs, or title/path similarity.
• Change scope = cached public metadata such as size labels, draft state, and review-burden hints.
• Validation posture = whether the PR provides enough public validation/test evidence for maintainer review.
• Contributor workload = public contributor activity and cleanup pressure, not a repo-wide quality failure.
• Contributor context = public GitHub/Gittensor identity context; non-Gittensor status is not a blocker.

_{🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed}

---

💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →.

Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.

• Re-run Gittensory review

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.57%. Comparing base (39cc958) to head (f10648c).
⚠️ Report is 103 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4050   +/-   ##
=======================================
  Coverage   93.56%   93.57%           
=======================================
  Files         378      378           
  Lines       35448    35485   +37     
  Branches    13000    13014   +14     
=======================================
+ Hits        33167    33204   +37     
  Misses       1618     1618           
  Partials      663      663           

Files with missing lines	Coverage Δ
src/queue/processors.ts	94.92% <100.00%> (+0.06%)	⬆️

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[superagent-security]

🚨 Contributor flagged. Click here for more info: Superagent Dashboard

[superagent-security]

Superagent didn't find any vulnerabilities or security issues in this PR.