From 3c68037de4745954055fe97ebb40e7b6be15e600 Mon Sep 17 00:00:00 2001 From: Kadir Mert Okumus Date: Sat, 24 Oct 2026 04:41:20 +0300 Subject: [PATCH] Record the cleanup closing checks (d576) The fresh-clone Full gate result, a correction to d573's branch proof, and the leftovers that have no roadmap item yet. Co-Authored-By: Claude Opus 5.5 --- docs/decisions.md | 1 + docs/development/roadmap.md | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/decisions.md b/docs/decisions.md index 6c455533..5a57474d 100644 --- a/docs/decisions.md +++ b/docs/decisions.md @@ -560,3 +560,4 @@ result files when the documentation was cleaned up on 2026-10-17. | d573 | 2026-10-24 | DECISION | process | The docs and code cleanup of 2026-10-23/24 is finished. Docs: the roadmap is trimmed 333 -> 252 lines (PR 226); README 370 -> 227 lines, with the reference guides `docs/reference/mcp-and-rest.md`, `docs/reference/cli.md`, `docs/reference/retrieval.md` and `docs/development/module-boundaries.md` (PR 228; the Rust help and OpenAPI strings that named moved README sections are repointed, PR 229); every statement a docs-vs-code audit found to disagree with the code is fixed, and `docs/architecture.md` 360 -> 238 lines, with `docs/lsp-channel.md` and `docs/traverse-flow.md` (PR 230). Process: the PR-closing workflow for the Copilot bot is removed (PR 227); 122 remote and 13 local branches are deleted, each proven merged first (a merged PR whose head SHA is the branch tip, and the tip an ancestor of main). Code, behaviour-preserving with new pins: PRs 231, 232 and 233 (d574). Each docs PR (226, 228, 230) passed a token-inventory compare (ids, numbers, dates, statuses, conditions, negations) and a separate loss review. | merges b642599 (PR 226), cff615f (227), 09da959 (228), 3c3092f (229), 4fd0e72 (230); d574 | | d574 | 2026-10-24 | DECISION | process | d487 amended: two of its "not taken" items were taken in the cleanup. The LSP `initialize_params` dedupe (PR 231): `initialize_params_for` with `InitializeVariant` builds the `initialize` request of python, go, clangd and java (`Base`), markdown (`DocumentSymbol`) and typescript (`PublishDiagnostics`); rust and csharp keep their own `initializationOptions`; an exact-JSON pin per adapter, written against the old code first. The SCIP map boilerplate (PR 233): `scip_ranked_hits` in `lci-maps/src/edge.rs` holds the `query` skeleton of `ImpactMap`, `ImplementsMap` and `TestedByMap`, with the note, rank, score, evidence, dedupe and truncation pinned per map. Also: `app::traverse::search` takes a `TraverseRequest` (PR 232) and loses its `too_many_arguments` allow. Still not taken: the multilingual acceptance table and a shared per-language acceptance runner, which only a live acceptance run proves (N127). Found in review, left for a later output change: N126. | merges bb3e071 (PR 231), ffdeeff (232), a60923c (233); d487 | | d575 | 2026-10-24 | FINDING | process | Rescued before the cleanup removed three local work items that had no decision id. An evaluation worktree at bfaff44 carried an uncommitted `.lciignore` variant that drops the `.agents/**` and `.github/instructions/**` lines (kept outside the repository as a diff that applies cleanly). An evaluation worktree at f1d401d was clean and is reproducible from that SHA. Branch `feat/live-grep-lockfiles` (tip 39aff96: the lockfile-skip default that d393 dropped) is superseded: main keeps the skip opt-in (`GrepOptions::skip_files`, empty by default); its patch is kept outside the repository. | cleanup S1c, S4 (work area, not in this repository) | +| d576 | 2026-10-24 | FINDING | process | Closing checks of the cleanup (d573). A fresh clone of e51341c, following the README build steps with no undocumented step, built and passed `cargo xtask test --suite Full` (2321 passed, 3 skipped); `cargo xtask setup` downloaded the official protoc 33.0 release. Correction to d573: 4 of the 13 deleted local branches (`feat/live-grep-lockfiles`, `feat/system-one-rescorer`, `feat/docs-lane-rerank-off`, `feat/k37c-probe-compile-grant`) were proven by their tip being an ancestor of main or by owner word, not by a merged PR whose head SHA is the tip. Left after the cleanup and not on the roadmap: `config.example.toml` comments that narrate measurements; whether spike files count toward the guide size limit (two are over 250 lines); user docs for `cargo xtask gold external` and `maps baseline`, which only this log describes; the README `[run_profile]` table cell that repeats `docs/reference/cli.md`; a second work-area pass for the held directories; six keep and eval tags that exist only locally. | fresh clone e51341c; cleanup sanity check 21 (work area, not in this repository) | diff --git a/docs/development/roadmap.md b/docs/development/roadmap.md index 9ac2882f..6f171bd8 100644 --- a/docs/development/roadmap.md +++ b/docs/development/roadmap.md @@ -48,7 +48,7 @@ that document and is not a roadmap id. ## Now and next -Cycle 1 implementation is complete (finish line d422): the last feature merges were K39b (d564) and K12a-2 (d533); M2 closed 2026-10-23 without K11b-6, whose release-build measurement failed, so per-language `places = "auto"` moves to cycle 2 with mixed-tree surfacing (d562). On main: the docs and code cleanup (PRs 226 onward, finished 2026-10-24, d573) on top of PR 225. Next: the cycle 1 report went to the owner on 2026-10-23; the owner's cut (asked 2026-10-23) and the answers to the report's questions follow, then cycle 2. Waiting on the cut: sanity 20 (d563) recommends K12b-2 and K31 move to cycle 2 and K12b-1 stay parked on its branch (nothing merges before the tsgo facts check and the lsp-cost pair, d537), and the placement of the K20c rust and K26 value windows (each needs a release quiet window with no cargo; the arbitration stays off until the K20c measurement passes), which sanity 20 proposes for cycle 2. K7b-3b and K11d-2b stay open (their rows), and ASYNC-2 C stays parked. Per the autonomy rule the coordinator waits for the owner's word after the report. Only docs PRs and the coordinator rewrite this section; a feature PR leaves it alone so parallel PRs do not conflict (d476). The live state is the coordinator's run checkpoint. +Cycle 1 implementation is complete (finish line d422): the last feature merges were K39b (d564) and K12a-2 (d533); M2 closed 2026-10-23 without K11b-6, whose release-build measurement failed, so per-language `places = "auto"` moves to cycle 2 with mixed-tree surfacing (d562). On main: the docs and code cleanup (PRs 226 onward, finished 2026-10-24, d573, d576) on top of PR 225. Next: the cycle 1 report went to the owner on 2026-10-23; the owner's cut (asked 2026-10-23) and the answers to the report's questions follow, then cycle 2. Waiting on the cut: sanity 20 (d563) recommends K12b-2 and K31 move to cycle 2 and K12b-1 stay parked on its branch (nothing merges before the tsgo facts check and the lsp-cost pair, d537), and the placement of the K20c rust and K26 value windows (each needs a release quiet window with no cargo; the arbitration stays off until the K20c measurement passes), which sanity 20 proposes for cycle 2. K7b-3b and K11d-2b stay open (their rows), and ASYNC-2 C stays parked. Per the autonomy rule the coordinator waits for the owner's word after the report. Only docs PRs and the coordinator rewrite this section; a feature PR leaves it alone so parallel PRs do not conflict (d476). The live state is the coordinator's run checkpoint. ## Cycle 1