Problem
The grounded Q&A commands (@gittensory ask, #4595's @gittensory chat) and the intent-classification router (#4596) are live, but nothing in the PR review comment itself tells a contributor these commands exist or how to use them. A user has to already know the exact command syntax to try it.
Proposal
Add one more collapsible <details> section to the bottom of the main review comment (after "Signal definitions" / "Test coverage", matching the existing collapsible list), titled something like "[BETA] Chat with Gittensory". Content: a couple of lines on what ask/chat do, the exact command syntax, and a link to the command reference docs.
Must follow the same convention as the existing "Test coverage" collapsible (testCoverageBody in src/signals/engine.ts): empty body ⇒ invisible, so repos where neither chatQa nor intentRouting is enabled never see a collapsible advertising a command that would just bounce.
Acceptance criteria
- New collapsible appended to
buildPublicSafeCollapsibles in src/signals/engine.ts, gated on settings.advisoryAiRouting?.chatQa or .intentRouting.
- Invisible (empty body) when neither flag is enabled for the repo.
- Links to the existing command-reference doc URL (reuse
commandReferenceUrl, not a new hardcoded link).
- Full branch-coverage tests.
Problem
The grounded Q&A commands (
@gittensory ask, #4595's@gittensory chat) and the intent-classification router (#4596) are live, but nothing in the PR review comment itself tells a contributor these commands exist or how to use them. A user has to already know the exact command syntax to try it.Proposal
Add one more collapsible
<details>section to the bottom of the main review comment (after "Signal definitions" / "Test coverage", matching the existing collapsible list), titled something like "[BETA] Chat with Gittensory". Content: a couple of lines on whatask/chatdo, the exact command syntax, and a link to the command reference docs.Must follow the same convention as the existing "Test coverage" collapsible (
testCoverageBodyinsrc/signals/engine.ts): empty body ⇒ invisible, so repos where neitherchatQanorintentRoutingis enabled never see a collapsible advertising a command that would just bounce.Acceptance criteria
buildPublicSafeCollapsiblesinsrc/signals/engine.ts, gated onsettings.advisoryAiRouting?.chatQaor.intentRouting.commandReferenceUrl, not a new hardcoded link).