Skip to content

Commit 22d2ea0

Browse files
committed
docs(plans): lifecycle sweep — slim shipped inventory, fix fact errors
Collapse substrate-extraction shipped tiers (1–6, partial 9–12) to stubs with canonical pointers; slim github-marketplace (Slice 5 runbook) and perf-triangulation (Phase 3 open work). Fix audit-worktree API name, fts --no-fts claim, graph-CRAP motivator, apply-write-safety dupe table.
1 parent fe4209f commit 22d2ea0

8 files changed

Lines changed: 69 additions & 1419 deletions

docs/plans/apply-write-safety.md

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,7 @@ Cross-file rollback on phase-2 failure; BOM round-trip policy (Q2); adversarial
4444

4545
## Current state (shipped)
4646

47-
| Behavior | Status |
48-
| ------------------------------------------- | -------------------------------------- |
49-
| Phase-1 line match (`line content drifted`) ||
50-
| Per-file temp + `rename` ||
51-
| All-or-nothing on phase-1 conflicts ||
52-
| Symlink / path-escape guards ||
53-
| Content-hash recheck before write ||
54-
| `fsync` before `rename` ||
55-
| Mixed CRLF/LF skip ||
56-
| Cross-file rollback on phase-2 failure | ❌ (deferred per architecture § Apply) |
47+
See [`architecture.md` § Apply](../architecture.md#apply--input-modes-transport-and-policy) and the TOCTOU / EOL comments in [`apply-engine.ts`](../../src/application/apply-engine.ts). **Open (this plan):** content-hash recheck before write, `fsync` before `rename`, mixed CRLF/LF skip. Cross-file rollback stays deferred per architecture § Apply.
5748

5849
---
5950

docs/plans/audit-delta-attribution.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ Implement **`findingKey()`** + attribution on the **`deprecated`** delta first (
2727
```text
2828
audit --base <ref>
2929
→ git resolve ref → sha
30-
audit-worktree.ensureCachedWorktree(sha) → .codemap/audit-cache/<sha>/index.db
30+
lookupCacheEntry(sha) or populateWorktree({ sha, … }) → .codemap/audit-cache/<sha>/index.db
3131
→ runAuditFromRef: HEAD index + base cached index
3232
→ per delta: compute added rows → findingKey(row) vs base key Set → attribution
3333
→ envelope.deltas[<key>].added[].attribution (CLI / MCP / HTTP identical)

docs/plans/fts-default-on-evaluation.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
| --- | ----------------------------------------------------------------------------------------------------------- | ------------------------- |
1515
| L.1 | **Measure before flipping default** — benchmark 3 fixture sizes for index time + DB bytes with/without FTS. | Empirical gate |
1616
| L.2 | If adopted: change **`config.ts` default** `fts5: true` for new projects only; existing configs unchanged. | No silent behavior change |
17-
| L.3 | **`--with-fts` / `--no-fts`** CLI flags continue to override. | Explicit escape hatch |
17+
| L.3 | **`--with-fts`** CLI flag overrides config; config `fts5: false` opts out (no `--no-fts` flag today). | Explicit escape hatch |
1818
| L.4 | Document size tax in README + packaging.md. | Transparency |
1919

2020
---

docs/plans/github-marketplace-action.md

Lines changed: 22 additions & 202 deletions
Large diffs are not rendered by default.

docs/plans/graph-estimated-crap.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
> **Status:** open · **Priority:** P2 · **Effort:** M (~2 weeks)
44
>
5-
> **Motivator:** CRAP (Change Risk Analysis and Predictions) ranks functions that are **complex and undertested** — a standard refactor-risk metric. Codemap already has `symbols.complexity` and real `coverage` after `ingest-coverage`, but recipes like `high-complexity-untested` are empty or misleading when no coverage file exists (`COALESCE(coverage_pct, 0)` treats everything as 0%). Graph-estimated coverage uses **test-file reachability** over `dependencies` / `calls` / `test_suites` to infer coverage tiers when measured data is absent.
5+
> **Motivator:** CRAP ranks **complex and undertested** functions. Codemap has `symbols.complexity` + ingested `coverage`, but `high-complexity-untested` is **misleading without ingest** (`COALESCE(coverage_pct, 0)` treats missing as 0%). Graph-estimated tiers (85/40/0%) from test reachability when measured coverage is absent.
66
>
77
> **Roadmap:** [§ Recipe & audit enrichment](../roadmap.md#recipe--audit-enrichment)
88

docs/plans/mcp-tool-annotations.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ Smallest slice: add **`mcp-tool-annotations.ts`** map + thread into **one** `reg
2828
MCP_TOOL_ANNOTATIONS: Record<toolName, ToolAnnotations>
2929
→ mcp-server registerTool({ …, annotations })
3030
→ tools/list response (advisory hints only)
31-
→ GET /tools parity (F.5)
31+
→ GET /tools parity (architecture § HTTP wiring)
3232
```
3333

3434
Apply tools already gated by `--yes` / `yes: true` in handlers — annotations are client-side UX only.

docs/plans/perf-triangulation-rollout.md

Lines changed: 6 additions & 140 deletions
Large diffs are not rendered by default.

docs/plans/substrate-extraction.md

Lines changed: 36 additions & 1063 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)