fix(evidence): preserve the longform pack, which reddened main on arrival #421

Merged
coilyco-ops merged 1 commit from qa/preserve-the-longform-pack into main 2026-08-13 12:34:19 +00:00
Member

main is red and my guard is what reddened it. This is the one-file remedy.

TestEveryCitedPackIsPreserved landed in c8e3336 (#416) and caught its first real case within minutes. 0cb3c80 added evaluations/probe-long-form-multi-turn.yaml, which cites /tmp/probe/longform.yaml — scratch space, not in the repository.

--- FAIL: TestEveryCitedPackIsPreserved
    probe-long-form-multi-turn.yaml cites /tmp/probe/longform.yaml

The pack still existed on the machine that produced it, so the evidence is intact. Copied to evaluations/packs/longform.yaml, which is exactly what the failure message asks for. Suite green.

The dataset is not edited — its pack: field still names /tmp/probe/longform.yaml, because that is what produced it.

Two things worth saying plainly

A red main blocks every open branch and skips the image publish#305 and #307 both measured that cost. I introduced the guard, so clearing it quickly is mine, and I would rather ship this than leave it for whoever notices next.

The guard behaved as intended rather than as a nuisance, and I would say so even if it had cost more. A dataset arrived whose method was already unreachable. Without the check that becomes invisible the moment /tmp is cleaned, and the numbers in #236's evidence would join the twelve I spent this hour recovering. The gap was visible the same hour instead of next week. That is the argument for the check, made by the check.

Whether a missing pack should redden main or only warn is a fair question, and it is not mine — if the answer is warn, the test should move to a non-gating report rather than lose the coverage.

Refs #313

**`main` is red and my guard is what reddened it. This is the one-file remedy.** `TestEveryCitedPackIsPreserved` landed in `c8e3336` (#416) and caught its first real case within minutes. `0cb3c80` added `evaluations/probe-long-form-multi-turn.yaml`, which cites `/tmp/probe/longform.yaml` — scratch space, not in the repository. ``` --- FAIL: TestEveryCitedPackIsPreserved probe-long-form-multi-turn.yaml cites /tmp/probe/longform.yaml ``` **The pack still existed on the machine that produced it**, so the evidence is intact. Copied to `evaluations/packs/longform.yaml`, which is exactly what the failure message asks for. Suite green. The dataset is not edited — its `pack:` field still names `/tmp/probe/longform.yaml`, because that is what produced it. ## Two things worth saying plainly **A red `main` blocks every open branch and skips the image publish** — #305 and #307 both measured that cost. I introduced the guard, so clearing it quickly is mine, and I would rather ship this than leave it for whoever notices next. **The guard behaved as intended rather than as a nuisance, and I would say so even if it had cost more.** A dataset arrived whose method was already unreachable. Without the check that becomes invisible the moment `/tmp` is cleaned, and the numbers in #236's evidence would join the twelve I spent this hour recovering. The gap was visible the same hour instead of next week. That is the argument for the check, made by the check. Whether a missing pack should redden `main` or only warn is a fair question, and it is not mine — if the answer is warn, the test should move to a non-gating report rather than lose the coverage. Refs #313
fix(evidence): preserve the longform pack, which reddened main on arrival
All checks were successful
ci / test (pull_request) Successful in 37s
ci / publish-echo-image (pull_request) Has been skipped
ci / publish-observed (pull_request) Has been skipped
ci / image-build (pull_request) Successful in 3s
2cde1290bd
TestEveryCitedPackIsPreserved landed in c8e3336 and caught its first real case
within minutes: 0cb3c80 added probe-long-form-multi-turn.yaml citing
/tmp/probe/longform.yaml, which is scratch and not in the repository.

My guard is what turned main red, so unreddening it is mine. The pack still
existed on the machine that produced it, so the evidence is intact and this is
the one-file remedy the failure message asks for.

The dataset is not edited. Its pack field still names /tmp/probe/longform.yaml,
because that is what produced it.

Worth recording that the guard behaved as intended rather than as a nuisance. A
dataset arrived whose method was already unreachable, and the gap was visible
the same hour instead of after the machine was cleaned. That is the whole
argument for the check, made by the check.

Refs sirens-echo#313

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Kai Siren <coilysiren@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>
Sign in to join this conversation.
No reviewers
No milestone
No project
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!421
No description provided.