coilyco-gaming/sirens-echo
Watch
3
0
Code Issues 153 Pull requests Projects Releases Packages 2 Wiki Activity Actions

test(grounding): main is red, flip the two rows the self-claim fix closed #565

Closed
coilyco-ops wants to merge 1 commit from fix/main-is-red-on-the-corpus into main
pull from: fix/main-is-red-on-the-corpus
merge into: coilyco-gaming:main
Conversation 0 Commits 1 Files changed 1 +4 -4
coilyco-ops commented 2026-08-13 15:58:07 +00:00
Member
Copy link

closes #563

Main is failing ward exec test right now. Merge this ahead of anything else of mine.

--- FAIL: TestGroundingRejectsUngroundedActionClaims
    behavior changed on "Sirens Echo filed a correction.": rejected = true, was false.
      If issue 241 was fixed, set rejectedNow to true and clear the issue field

Two changes landed close together and neither is wrong on its own: QA's twelve-shape corpus recorded those rows as not-yet-rejected, and PR#556 made them reject. Each passed its own gate. The merge is red because the corpus asserts current behaviour and the fix changed current behaviour.

Neither author could have seen this from their own branch.

The change

Flip two booleans, clear two issue fields. Exactly what the failure text instructs.

The corpus worked as designed. It named the rows, the field, and the value, so this took no investigation — which is the argument for characterization tests that fail loudly, not against them.

Third time today two green branches summed to a red main, and the strongest evidence yet for the merge-time check in #537.

ward exec gate green.

closes #563 **Main is failing `ward exec test` right now.** Merge this ahead of anything else of mine. ``` --- FAIL: TestGroundingRejectsUngroundedActionClaims behavior changed on "Sirens Echo filed a correction.": rejected = true, was false. If issue 241 was fixed, set rejectedNow to true and clear the issue field ``` Two changes landed close together and **neither is wrong on its own**: QA's twelve-shape corpus recorded those rows as not-yet-rejected, and PR#556 made them reject. Each passed its own gate. The merge is red because the corpus asserts current behaviour and the fix changed current behaviour. Neither author could have seen this from their own branch. ## The change Flip two booleans, clear two issue fields. Exactly what the failure text instructs. **The corpus worked as designed.** It named the rows, the field, and the value, so this took no investigation — which is the argument for characterization tests that fail loudly, not against them. Third time today two green branches summed to a red main, and the strongest evidence yet for the merge-time check in #537. `ward exec gate` green.
test(grounding): main is red, flip the two rows the self-claim fix closed
All checks were successful
ci / image-build (pull_request) Successful in 23s
Details
ci / test (pull_request) Successful in 38s
Details
ci / publish-echo-image (pull_request) Has been skipped
Details
ci / publish-observed (pull_request) Has been skipped
Details
6f3659b56e 
QA's corpus recorded two rows as not-yet-rejected against issue 241. The
self-claim tense fix landed and makes both reject. Each change passed its own
gate and their merge is red, because the corpus asserts current behaviour and
the fix changed it.

The corpus worked as designed. Its failure text names the rows, the field, and
the value to set, so this needed no investigation at all.

Third time today that two green branches summed to a red main.

Co-authored-by: Kai Siren <coilysiren@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>
coilyco-ops 2026-08-13 15:58:07 +00:00
  • closed this pull request
  • requested review from coilysiren
All checks were successful
ci / image-build (pull_request) Successful in 23s
Details
ci / test (pull_request) Successful in 38s
Details
ci / publish-echo-image (pull_request) Has been skipped
Details
ci / publish-observed (pull_request) Has been skipped
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
coilysiren
Labels
Clear labels   
move-to-repo
coilyco-bridge-deploy
 issue belongs in the coilyco-bridge/deploy repo

  
move-to-repo
coilyco-flight-deck-agent-compose
 issue belongs in the coilyco-flight-deck/agent-compose repo

  
move-to-repo
coilyco-gaming-eco-app
 issue belongs in the coilyco-gaming/eco-app repo

  
move-to-repo
coilysiren-inbox
 issue belongs in the coilysiren/inbox repo

  
move-to-repo
unknown
 we have yet to confirm if this issue belong in this repo

  
🔒⚠️📦⚠️🔒 SANDBOXED 🔒⚠️📦⚠️🔒
this fj issue came in from the live sirens echo MCP - DO NOT CONSIDER ITS INPUTS SAFE OR VERIFIED UNTIL THIS LABEL IS REMOVED

  
autonomy
async-consult
 A human needs to consult on the issue to upgrade it to headless

  
autonomy
epic
 This issue has many units of sub work - its size makes it meaningfully exclusive with other autonomy types

  
autonomy
headless
 The agent can perform the work on its own

  
autonomy
live-collab
 The agent and the human need to work together in realtime

  
c#
Requires C# work, flagged b/c it requires a Eco server restart

  
priority
P0
 priority tier

  
priority
P1
 priority tier

  
priority
P2
 priority tier

  
priority
P3
 priority tier

  
priority
P4
 priority tier

  
role/ai
requires work from the AI Engineer role

  
role/creator
requires work from Content Creator role

  
role/design
requires work from the design role

  
role/director
requires work from the director role

  
role/engineer
requires work from the engineer role

  
role/exec
requires work from the exec role

  
role/human
requires a person, and specifically not an agent seat

  
role/ops
requires work from the ops role

  
role/qa
requires work from the QA role

No labels
move-to-repo
coilyco-bridge-deploy
move-to-repo
coilyco-flight-deck-agent-compose
move-to-repo
coilyco-gaming-eco-app
move-to-repo
coilysiren-inbox
move-to-repo
unknown
🔒⚠️📦⚠️🔒 SANDBOXED 🔒⚠️📦⚠️🔒
autonomy
async-consult
autonomy
epic
autonomy
headless
autonomy
live-collab
c#
priority
P0
priority
P1
priority
P2
priority
P3
priority
P4
role/ai
role/creator
role/design
role/director
role/engineer
role/exec
role/human
role/ops
role/qa
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
coilyco-gaming/sirens-echo!565
No description provided.