Fix verifier-fail routing that cannot reach green in implement map #1100
Labels
No labels
P0
P1
P2
P3
bug
category/code-change
category/cross-repo
category/eval-run
category/investigation
category/no-test-target
category/owner-ruling
category/prose-artifact
category/recurring-chore
category/structure
category/umbrella
crc-refactor
create
delete
enhancement
filed-by/agent
filed-by/user
frozen
lint-rule
needs-info
needs-triage
next
plugin/cc-architect
plugin/os
plugin/os-adr
plugin/os-aidd-lint
plugin/os-backlog
plugin/os-context
plugin/os-doc-hygiene
plugin/os-sdlc
plugin/os-vault
project/cc-os
ready-for-agent
ready-for-human
recurring
review
rule-proposal
update
waiting
wayfinder:grilling
wayfinder:map
wayfinder:map
wayfinder:research
wayfinder:task
wayfinder:task
wontfix
worklist/deviations
worklist/lint-rule
worklist/new-implement-build
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
jared/cc-os#1100
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Problem
A behavior-verifier fail on a green increment sends the run back to a red-first test-writer round. The test-reviewer round_tests rule needs each new test to be red. Tests for behavior that the green production code already has cannot be red. So the loop can only end with a bound escalation.
A second problem: gate briefs judged against more than the AC text. Behavior-verifier (542) and test-reviewer (546) asked for things that the AC text does not ask for, such as lib_root on Project and a new --ticket option with a TSV ticket column.
Observed
Implementation 63, dispatches 542-550, ended with escalation 35
bound_exhausted:test-reviewer/fail.Test-reviewer at 550 said "no test change can make the concept-brief tests red. The runner must decide."
Reproduce
n/a: the evidence is implementation 63 in the pipeline database.
Approach
Expected: a verifier fail on green work routes to a node that can add the missing behavior. Or round_tests leaves out tests for behavior that already exists. Gate briefs judge against the AC text only.
Required work:
Acceptance criteria
Origin
17d4c688), session 6f78b932-f117-4bd7-bbed-524776dbb719.