chore: delete six dead automation lanes (append compactor, commit review, crawl-remote deploy, finalize-open-PRs, proof nudges, state compaction) - #964
Merged
Conversation
Three runs ever, zero successes (last: 2026-07-29 run stuck waiting). Removes the 3,056-line deploy-crawl-remote.yml workflow, the pinned Wrangler toolchain under .github/deploy, and the 3,030-line workflow shape test. The ci.yml toolchain job and checkout-pin test follow in the wiring commit.
The state-materializer workflow burned a runner every 20 minutes to report drained=0 acked=0 (latest: run 30551689153). The git projection it fed was deleted in #936 and no producer posts /internal/state/append anymore; the DO endpoints and tables are removed in the follow-up commit.
state-compaction.yml rewrote openclaw/clawsweeper-state main to a single-root commit once a month. The repo's written surface is nearly retired (records/ledger/assets are Cloudflare-owned) and dropping this writer moves the archive goal forward. state-repo-size.ts stays: the exact-review reconcile workflow still runs it.
Last success April 29; workflow_dispatch-only since. The finalizer module and its --write-report path stay because repair-publish-results runs them after every worker-result publish.
Proof nudges and bot-proof decisions contributed nothing to weekly review coverage. The clawsweeper.ts command surgery follows in the wiring commit; the proof-nudge comment marker constant survives because the stalled-unproven close policy dates historical proof asks with it.
Zero successes in the last 20 commit-review runs (11 failed, 9 cancelled). Removes the hosted per-commit review workflow, its classifier and Check Run publisher, the commit-finding repair intake, and the dispatcher doc. The offline local-review engine survives in commit-sweeper.ts (trimmed in the wiring commit); existing commit_finding repair jobs remain executable.
…bles
/internal/state/{append,drain,ack,dispose} had no remaining producer:
post-#929/#932 every record tuple goes through /records/tuples and the
only drain consumer was the deleted state-materializer workflow. The
Durable Object now drops the five state_append_* tables on upgrade, and
queue status no longer reports the empty window. The internal-route HMAC
test re-targets /internal/state/records/tuples.
Deletes proofNudgesCommand/botProofCommand, their eligibility, marker parsing, rendering, cursor, and candidate-selection helpers plus lane types and CLI dispatch (about 1,300 lines of clawsweeper.ts). The PROOF_NUDGE_MARKER_PREFIX constant stays: stalledUnprovenProofRequest BlockReason still dates historical proof-nudge comments with it, and hasDispatchableMantisScenario stays for review rendering.
…ne's budget wiring commit-sweeper.ts keeps only the offline local-review command plus the shared helpers the main sweeper's --local-range path imports. The commit_review worker lane leaves limits, workflow-utils, sweep.yml budget accounting, and the docs; docs/commit-sweeper.md now documents local-review only.
…angelog setup-state sites 25->19, git publishers 23->19, records-repo-slugs and non-fatal-bookkeeping lists shrink accordingly, and all seven deleted workflows join the stays-deleted list. package.json drops the proof-nudges/bot-proof/commit-review/commit-reports and repair:commit-finding-intake scripts. README and CHANGELOG record the six lane removals.
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.
Executes Stages 2-3 of the simplification proposal: delete six dead or dormant lanes outright, with the code, tests, docs, and workflow-shape invariants that existed solely for them. Net -15,820 lines (896+/16,716-) across 55 files.
Deleted lanes and evidence
drained=0 acked=0 errors=0(e.g. run 30551689153); the git projection it fed was deleted in #936, and no producer posts/internal/state/appendanywhere in the tree —state-append-client.tsonly speaks the canonical/records/tuples+/records/commitsendpointsstate-materializer.yml(a runner every 20 min),src/repair/state-materializer.ts, the four/internal/state/{append,drain,ack,dispose}Worker routes and DO endpoints, and the fivestate_append_*DO tables (dropped on upgrade) plus their tests and stats surfacecommit-review.yml,repair-commit-finding-intake.yml, the hosted commit-sweeper CLI commands,commit-classifier.ts,commit-checks.ts,commit-finding-intake.ts, thecommit_reviewworker lane in limits/scheduler/sweep budget accounting, and the dispatcher doc. The offlinepnpm local-reviewengine and its--local-rangereuse in the main sweeper are preserved; existingcommit_findingrepair jobs stay executablewaitingsince 2026-07-29)deploy-crawl-remote.yml(3,055 lines), the pinned Wrangler toolchain under.github/deploy/, the ci.yml integration job, and the 3,030-line workflow testrepair-finalize-open-prs.ymlonly — see skips below for why the module staysproof-nudges.yml,docs/proof-nudges.md, theproof-nudges/bot-proofCLI commands and their eligibility/marker/rendering/cursor machinery inclawsweeper.ts, and the 911-line policy test.PROOF_NUDGE_MARKER_PREFIXsurvives: the stalled-unproven close policy dates historical proof asks with it, and the comment-router/dispatcher keep ignoring old nudge commentsopenclaw/clawsweeper-state, whose written surface is nearly retired; removing this writer advances the archive goalstate-compaction.ymlandsrc/repair/state-compaction.ts+ test.state-repo-size.tsstays (still run byexact-review-reconcile.yml)Skips (live load-bearing consumers found)
src/repair/finalize-open-prs.tsmodule:repair-publish-results.yml(repair cluster, out of scope) runsrepair:finalize-open-prs --write-reportafter every worker-result publish, andpublish-result.tsreadsresults/finalize-open-prs.json. Only the dormant dispatch workflow was deleted.idea-archive-revival.yml+src/idea-archive-revival.ts) — skipped entirely: it is the live reversibility half of theunsponsored_feature_requestclose policy. The apply lane (out of scope) actively parks ideas whose close comments promise "@clawsweeper revive... will automatically reopen", the close policy imports its threshold/label/reaction helpers, and its run history is green every 6 hours. Deleting it would silently turn reversible parks into irreversible closes; pausing that close policy is a separate decision.Invariant updates
test/state-writer-workflow.test.ts: setup-state sites 25->19, git publishers 23->19, slug-scoped/undefined hydration lists and non-fatal-bookkeeping lists shrunk to match, and all seven deleted workflows joined the stays-deleted assertion. Sweep/clawsweeper workflow-shape tests, queue-pressure and worker-limit lane tests updated; the internal-route HMAC test now proves the signature gate on the canonical/records/tuplesroute.Proof
pnpm run checkgreen locally (build:all, lint, format:check, limits/active-surface/queue-boundary checks, full unit + coverage suites).test:repairgreen except the pre-existing Linux-only containment/Landlock failures on macOS (Landlock probe, procfs enumeration, containment runtime — unrelated to this change).