test: retire mechanical known-red ledger rows (uxrim, bwo2l/vqt48, ndgbm-quiet, 7qw4)

[Sinity]

Summary

Retires the four mechanical rows of the WS-B known-red/false-green ledger
(.agent/campaigns/2026-08-overhaul/ws-b-suite-health.md): uxrim, bwo2l,
vqt48, and 7qw4 were all already satisfied on master before this branch was
cut; the one row that needed real work was polylogue-ndgbm's quiet
component (Codex query-cardinality oracle's fixture never emitted a
message-shaped record, so the parser's admission gate refused every
payload and the module fixture errored at setup), which this PR fixes and
whose two associated xfail markers it removes.

Problem

The ledger recorded five findings as known-red or false-green test debt,
each with a RECIPE and file:line anchors, dispatched here as a themed sweep.

Solution

• uxrim (tests/unit/api/test_facade_contracts.py:4789): the recipe
  asked to extend the frozen_clock_modules marker to also cover
  revision_governance. Already done — commit 8ed4f91 (PR fix(tests): repair storage-cluster test drift from origin-vocab+clock-guard fixes #3595,
  2026-08-03) landed exactly that extension. No diff needed.

• 7qw4 (tests/unit/storage/test_store_ops.py): the recipe asked to
  rewrite the test to call the production aggregate_message_stats async
  fn and delete a test-local shadow reimplementation. Already done —
  commits from PR test(storage): exercise aggregate message stats repository path #3729 (call the repository path) and PR test(storage): assert aggregate stats use public origins #3813 (assert on
  public origins) landed this; no shadow function remains in the file. No
  diff needed.

• bwo2l + vqt48 (tests/unit/cli/test_archive_maintenance_cli.py,
  tests/unit/sources/test_live_watcher.py,
  tests/unit/sources/test_live_batch_support.py): the recipe asked to give
  refused codex fixtures real conversational content. Already done — the
  9ykn positive-evidence-gate fixes landed across several prior PRs (visible
  in git log --grep: fix(test): add real message content to raw-materialization fixtures #3555, test(storage): give raw-authority-ledger codex fixtures real message text #3572, fix(sources): require a genuine chunk in gemini/drive chunkedPrompt detection #3600, fix(devtools): give synthetic codex raw payloads real message text #3642, test(sources): verify origin parser claims #3841). All 138 + 259
  tests pass.

• ndgbm quiet component (tests/infra/query_manifest_oracle.py,
  tests/unit/cli/test_query_composition_laws.py): the actual fix in this
  PR. QueryCardinalitySession.wire_records() emitted only
  function_call/function_call_output response_items — zero
  message-shaped records — so codex.is_supported_session_stream()
  (has_message never set) refused every payload before ingest and the
  module-scoped fixture errored at setup before either test could run.
  Added one planted user/assistant message-pair response_item per session
  (same wire shape landed for the bwo2l/vqt48 fixtures), which admits the
  stream; action-identity assertions are unaffected because they only count
  tool rows.

  Removing the two xfail markers this unblocks surfaced two further
  pre-existing issues that had never been reachable before (the tests always
  errored at setup), fixed in the same commit:

  • The naive-duplicate-id-join mutation test's hand-rolled actions view
    predates the production view's result_state CASE projection
    (polylogue-cuxz.5, archive_tiers/index.py:1040) and crashed with
    sqlite3.OperationalError: no such column: a.result_state instead of
    producing the comparably-wrong row count the test actually asserts on.
    Added the same CASE projection to the test's naive view.
  • find ... then delete --yes --all has no non-daemon route — it refuses
    outright when the daemon does not answer the /prepare call. Added a
    daemon-delete stand-in patch on _submit_daemon_mutation, mirroring the
    existing pattern in
    test_verb_cardinality.py::TestDeleteCardinalityLargeNonMocked._daemon_delete_route
    (does the real delete through ArchiveStore, just skips the daemon
    process).
  • _copy_archive's plain shutil.copytree left the durable bootstrap
    marker bound to the original archive root path
    (durable_identity_digest), so the first write-mode ArchiveStore.open_existing
    at the copied path raised DurableChangeTrainError: fresh durable bootstrap marker durable identity mismatch. Re-bind the marker after
    copy, mirroring tests/infra/workload_artifacts.py::clone_seeded_archive.

ndgbm residual (not closed by this PR): the bead also names a second,
separate symptom — the 64-seed process-pool death only reproduces beside a
saturated 10-worker gate (confirmed load-only: quiet re-run 20260818T231534Z
was green in 43s). That's a load-robustness question the dispatch prompt
asked to coordinate with WS-A's e98k cgroup-mmap-budget findings. I could not
type this as a partial disposition in the machine-readable pr-scope
carrier below: the validator requires a pre-existing Beads dependency
edge (blocks/discovered-from/relates-to/supersedes) between ndgbm
and its named successor before it will accept the disposition, and no such
edge exists between polylogue-ndgbm and polylogue-e98k today. Per the lane
contract I do not invoke bd from a worktree, so I did not create that
link. Coordinator action needed: link polylogue-ndgbm →
polylogue-e98k (or a dedicated new successor) via bd dep add (or
equivalent) before ndgbm can be typed partial in a carrier; until then the
bead should stay open with its residual noted in its own notes (already
present).

Verification

• devtools test tests/unit/api/test_facade_contracts.py -k raw_artifacts_read_source_tier → 1 passed (uxrim, unchanged/confirmed)
• devtools test tests/unit/cli/test_archive_maintenance_cli.py → 138 passed (bwo2l, unchanged/confirmed)
• devtools test tests/unit/sources/test_live_watcher.py tests/unit/sources/test_live_batch_support.py → 259 passed (vqt48, unchanged/confirmed)
• devtools test tests/unit/storage/test_store_ops.py -k aggregate_message_stats → 1 passed (7qw4, unchanged/confirmed); mutation red-check locally re-applied AG1 (role-swap on the unfiltered role_row query) and AG2 (words_approx = 0) against polylogue/storage/sqlite/queries/stats.py, both now fail (assert 1 == 2, assert 0 == 9), reverted before commit — no diff shipped for this row
• devtools test tests/unit/cli/test_query_composition_laws.py → 2 passed, 0 xfail (ndgbm quiet component: red→green; both -p no:testmon and cached runs confirmed)
• devtools verify --quick → exit 0 (format + lint + mypy + render all --check; no out of sync in output)

All four rows were verified in the context that was failing: each command
above is the exact standalone-context command named in the dispatch recipe,
run directly (not nested inside a broader suite run).

Bead disposition matrix

Bead	Disposition	Evidence	Notes
polylogue-uxrim	SATISFIED	test pass + commit 8ed4f91	already fixed on master, PR #3595
polylogue-bwo2l	SATISFIED	test pass (138/138)	already fixed on master
polylogue-vqt48	SATISFIED	test pass (259/259)	already fixed on master
polylogue-7qw4	SATISFIED	test pass + mutation red-check	already fixed on master (PR #3729, #3813)
polylogue-ndgbm	PARTIAL	test pass (2/2, no xfail) + diff	quiet component fixed this PR; load-only pool-death residual stays open on the bead — not represented in the typed carrier below (see Solution section: needs a coordinator-created Beads link to type as partial)

Ref polylogue-uxrim, polylogue-bwo2l, polylogue-vqt48, polylogue-ndgbm, polylogue-7qw4

🤖 Generated with Claude Code

https://claude.ai/code/session_01HWcPJJJvuF25CqVwTFgSQC

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 244dfc96-3dc3-4e50-beca-2b71ebbe12a2

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}