Skip to content

Commit 21fc75b

Browse files
committed
post-step: drop broad @\w+ from isNeedsInfoComment regex
The @\w+ pattern matched @ignore, @test, @OverRide etc. in backticked code references inside plain-text comments. This caused the post-step to misclassify an agent's 'Deleted the file...' claim on #483 as agent/needs-info, because the comment text mentioned @ignore and @test when describing why the class was dead. Result: the issue got agent/needs-info, the orchestrator advanced past it (treating needs-info as a completion signal), and the master closed prematurely with no actual work merged. Tighten the regex to only match explicit needs-info markers: - 'needs more info' - '## \ud83e\udd16 Triage' (the agent's heading) - 'needs the following' The proper way for the agent to signal needs-info is the \ud83e\udd16 Triage heading; plain text is not reliable.
1 parent dcd49c9 commit 21fc75b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/scripts/opencode-post-step.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ function isRefusalComment(body) {
3030

3131
function isNeedsInfoComment(body) {
3232
if (!body) return false;
33-
return /needs more info|## 🤖 Triage|needs the following|@\w+/i.test(body);
33+
return /needs more info|## 🤖 Triage|needs the following/i.test(body);
3434
}
3535

3636
function isInvestigationReport(body) {

0 commit comments

Comments
 (0)