chore(devtools): prune retired-workflow fossils (tranche 1) - #4005
Merged
Conversation
Removes CommandSpec entries for claim_vs_evidence, turso_probe, proof_world_real_slice, help_latency_probe, temporal_read_profile, and temporal_archive_aggregates ahead of deleting their implementations. resume_ranking_eval was never catalog-registered. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
The polylogue-sru campaign closed with terminal artifacts 2026-07-06; this CLI re-proved a claim that has already been made and recorded. PublicClaimProjection (the durable production primitive it wrote through) remains untouched in polylogue/insights/measurement/. Trims the finding doc's now-dead reproduction/regeneration sections to reflect the retired harness while keeping the finding's substantive text as a frozen historical record. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
No catalog entry, no importers beyond its own test, no hook/CI/doc references. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
The uplift campaign (jxe) it screened real-archive slices for is closed. No hook/CI/doc references. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
…arch) No dedicated test file, no hook/CI/doc references; the interactive-tier cold-CLI budget it checked is not enforced by any automated gate. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
…egates One-shot read-profiling research (July 2). No hook/CI/doc references. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
verify doc-commands flags any backtick-quoted 'devtools ...' span against the live Click tree; the retired command needed prose, not code, framing. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 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. Comment |
Cold adversarial review found the deletion rationale wrong: the ledger row cited the closed jxe uplift campaign, but the module and its test actually cite polylogue-212.11 -- OPEN, P3, whose notes name this tool as the harness half of the real-archive-data extension with a pending unpromoted deliverable. Same name-collision failure class as the dev_loop/mutmut_campaign exclusions caught before this PR was opened, just missed on this one row. Restores devtools/proof_world_real_slice.py, tests/unit/devtools/test_proof_world_real_slice.py, and the 'demo real-slice-screen' CommandSpec verbatim from before deletion (dc7333f^), then regenerates docs/devtools.md via devtools render devtools-reference. The other 5 deletions in this PR were independently verified safe and are untouched. Ref .agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Deletes 5 of the 8 devtools modules the WS-C pruning ledger's tranche-1 table
listed for deletion, plus their catalog entries and tests. Three named rows
(
dev_loop,mutmut_campaign/run_campaign,proof_world_real_slice)turned out to be live or contested on verification and are explicitly
excluded — see Problem below.
Problem
.agent/campaigns/2026-08-overhaul/ws-c-devtools-pruning.mdcatalogs ~139devtools modules toward a ~45k LOC / ~55 command target. Tranche 1 lists 8
"retired-workflow fossil" rows as high-confidence deletions, gated on a
per-row checklist (rule 5, "EVIDENCE BEFORE DELETION"): re-grep
.beads-hooks/,.circleci/config.yml,.claude/hooks/,scripts/, and.github/workflows/*.ymlfor module invocations before deleting, becausestring-keyed CLI dispatch and shell-invoked entrypoints don't show up in an
import-graph grep (this is exactly how
pre_push_gateandcoverage_gatewere wrongly flagged for deletion in the ledger's own 2026-08-18 correction
pass).
Re-running that checklist per row (the ledger's 2026-08-19 spot-check had
already run it, but omitted
.github/workflows/*.ymlfrom the grep targetlist) found two more wrong entries before this PR was first opened, and a
third was caught afterward by cold adversarial re-verification:
dev_loop(3,844 LOC + a cited product-side family indaemon/http.py,daemon/cli.py,route_contracts.py) is not dead. It's catalog-registeredas
workspace dev-loopwith a large flag surface(
--prepare/--launch-daemon/--capture-cli/--receiver-smoke/etc.), hasa 1,629-line test file, and a 757-line doc (
docs/dev-loop.md) linked fromdocs/README.md's nav — last touched by a real feature commit 5 days beforethis PR (feat(devloop): close remaining polylogue-5en gaps (stale-run detection, MCP recipe) #3333, "close remaining polylogue-5en gaps"). The ledger's citation
("devloop workflow explicitly retired, CLAUDE.md: do not resurrect") is a
name collision: CLAUDE.md's retirement note is about the unrelated bespoke
Beads/conductor-packet workflow, not this branch-local
daemon/browser-capture preflight tool.
mutmut_campaign+run_campaign(624 LOC) are two unrelated modulesthe ledger conflated by name similarity.
mutmut_campaign.py(catalog:bench mutation) is invoked weekly by.github/workflows/mutation-testing.yml(devtools bench mutation run "$campaign") — exactly the grep the spot-check missed by skipping.github/workflows/.run_campaign.py(catalog:bench synthetic) is aseparate, live-tested command
(
test_benchmark_campaigns.py::test_run_campaign_skips_seed_archive_for_daemon_live)driving the ledger's own NEEDS-EVIDENCE/default-keep
large_archive_generatorsynthetic_benchmark_runtimemodules; it has nothing to do with mutationtesting.
push, restored in a follow-up commit):
proof_world_real_slice(440 LOC +test). The ledger's row cited it as belonging to the closed
jxeupliftcampaign, but the module's own docstring and its test both cite
polylogue-212.11instead — an open, P3 bead whose notes name thistool as "the harness half of the real-archive-data extension" with a
pending, unpromoted deliverable. Same name-collision failure class as the
dev_loop/mutmut_campaigncatches above, just missed on this row duringthe original per-row pass. Restored verbatim (module, test, and the
demo real-slice-screenCommandSpec) from before deletion.All three exclusions were reported to the coordinator (the first two before
this PR was opened, the third as soon as cold review surfaced it).
Solution
Deletes the remaining 5 rows, each independently re-verified clean (no
hook/CI/doc/production references beyond the module's own catalog entry and
test file):
claim_vs_evidence.py+claim_vs_evidence_evidence.py(+ both testfiles) — the
polylogue-srucampaign closed with terminal artifacts2026-07-06; this CLI re-proved an already-recorded claim. The durable
production primitive it wrote through,
PublicClaimProjection(
polylogue/insights/measurement/public_claims.py), is untouched and livesindependently. Trimmed
docs/findings/claim-vs-evidence.md's now-dead"Reproduce"/"Regenerate" sections (they invoked the deleted command) while
keeping the finding's substantive text as a frozen historical record — the
2026-08-14 purge (refactor: remove self-attesting campaign bureaucracy #3950) had already retired "the closed claim-vs-evidence
campaign's private-report/calibration/publishing harness" in spirit; this
PR finishes that.
turso_probe.py(+ test) — Turso storage-backend research (research(storage): evaluate SQLite to Turso Database backend swap #2257/research(storage): benchmark Turso ops workload (#2257) #2281,June) concluded; direction settled on SQLite independence + Sinex mode.
resume_ranking_eval.py(+ test) — orphan: no catalog entry, no importersbeyond its own test.
help_latency_probe.py(no dedicated test file existed) — one-shotCLI-latency research; not wired into any gate.
temporal_read_profile.py+temporal_archive_aggregates.py(+ both testfiles) — one-shot read-profiling research (July 2).
Each deleted module's
CommandSpecwas removed fromdevtools/command_catalog.py, thendevtools render devtools-referenceregenerated
docs/devtools.md. No CLAUDE.md tooling-list lines named any ofthe six deleted commands, so none needed trimming there.
proof_world_real_slice'sCommandSpecwas restored to the same spot in thefollow-up commit and
devtools render devtools-referencere-run.Verification
devtools render all --check— exit 0, noout of syncsurfaces.devtools verify --quick—"status": "success","exit_code": 0(runagain after the
proof_world_real_slicerestore, still green).devtools verify(full, complete-corpus) —"status": "success","exit_code": 0,"terminal_green": true,"complete_corpus_covered": true,"non_green_count": 0,"release_baseline_allowed": true.devtools test tests/unit/devtools/test_proof_world_real_slice.py— 20/20passed, confirming the restored module's tests are intact.
python -m devtools.verify_doc_commands—67 doc files scanned, no stale commands(caught and fixed one accidental literal reference to thenow-deleted claim-vs-evidence command in the trimmed findings doc).
Anti-vacuity: pure deletion (plus one restoration); reachability evidence per
surviving row is in the ledger plus the re-verification above (all rows
independently re-grepped against
.beads-hooks/,.circleci/config.yml,.claude/hooks/,scripts/,.git/hooks/, and.github/workflows/*.yml,and cross-checked against
docs/and git log recency).Bead disposition matrix
This PR does not close, claim, or otherwise mutate any Beads issue — it is
pure code/doc deletion against the campaign's markdown ledger, not a Beads
tracker item.
mutated_beadsis empty; using theself_containedscopecarrier.
polylogue-212.11is cited as evidence for theproof_world_real_sliceexclusion but is not touched, claimed, or closed bythis PR.
Candidate follow-ups for the coordinator (not closed by this PR):
dev_loop,mutmut_campaign/run_campaign, andproof_world_real_sliceas confirmed-live/contested, plus record the.github/workflows/gap in the spot-check methodology so future tranches don't repeat it.Verification commands run