Repository navigation
feat(futo-backups-bot): repeat investigations - #46
Conversation
[Medium risk] In agent.go, a staff retry with no extra prompt still appends "Additional staff request:" followed by nothing. The reported P2 touches this only as a missing test, and the review does not address the dangling label. The risk rating rests on ordinary application logic across the Discord bot and the Columbo worker, which a revert undoes, plus an internal staff-only endpoint guarded by role checks. Approved Findings
Earlier findings (1 resolved)SummaryAdds staff-triggered investigation retries with prompts and recent ticket context. SupportService routes staff commands and modal submissions into retry requests containing the original issue and bounded recent conversation. Columbo's worker bypasses triage for staff requests, while Runner.Investigate adds the retry context to its model message. The latest change catches staff-announcement failures in SupportService so they do not replace the queued acknowledgement with the generic interaction error. What's good
Review context · 0/6 commands
Reviews (2) · Last reviewed commit: "feat(futo-backups-bot): repeat investiga..." · openai/gpt-6.1-sol · $0.39 |
f0e75a3 to
d494ac5
Compare
Part 2/7, based on #45: add staff-only investigation retries with an optional prompt, original issue and recent conversation, bypass triage and preserve queue acknowledgement after announcement failures; validated with 155 bot tests and Columbo race tests.