[viztor]

Phase P1b of docs/improvement-plan.md. It is stacked on #106 (P1a), which is stacked on #105.

Hygiene only. It removes dead code and adds gates. There are no user-facing behaviour changes.

#	Change	How it's pinned
1	adapter-replacement.ts and its test are deleted. Nothing imported them. docs/engineering-notes.md gains a section on two points. First, deleting another plugin's registry entry is not a host API: registerAdapter is all-or-nothing, and only the owner's handle.replace moves routes. Second, the plugin keeps pi-ai from claiming the planes through the config waterfall instead (D13).	A new gate in architecture.test.ts fails the build if anything in src/'s import graph writes the llm registry map (adapters.set/delete/clear, including optional chaining).
2	The stream hook's dead prepared handling is gone. In the host's llm/src/index.ts, stream(options) passes the prepared dispatch as a separate argument, which only prepareCall's one-shot stream supplies. options.prepared is never read.	A green pin on the Go-Google redirect: it forwards the caller's request unchanged apart from the provider and the Google-shape history.
3	.freebuff/ is untracked and ignored. It held a per-machine agent workspace id.	none
4	The settings partition test walks Config.dict. It used Config({}), which drops keys with no default, and that is how debugFile went missing from it. debugFile is now registered as config-only, and described by the gate main chose (D24): naming it writes both halves whether or not debug is on, and debug governs the console. A new manifest test checks cordis.patch.yml against the schema and the field registry. That surfaced the phantom showUsagePrice, the wrong freeModelMarker: '' text and the incomplete field lists, all now fixed.	Red on the base, green now.
5	utils/retry.ts takes its timer and logger as parameters. The utils purity gate now also rejects console and global timers: called or passed as values, bare or through globalThis./window./global./self.. It walks utils/ recursively.	Verified by planting violations and watching the gate fail.
6	tool-fallback's unreachable Anthropic schemas are deleted. The plane rule is main's (D25): the request URL decides the plane, with a plane-blind lookup as the fallback.	The existing fallback cases, plus a case on the shipped shim whose premise is asserted: a model free on Zen and absent from the Go catalogue is still admitted on Go by the plane-blind fallback.
7	test/support/src.ts is the one door tests use into src/, so later module moves touch one file. This phase converts the files it touches, P2 converts the rest, and CONTRIBUTING.md states the rule (with the setup file as a named exception). The synthetic catalogue rows and the plane-divergent seed are one fixture, test/support/catalogs.ts, behind the door.	A gate requires each name in the door to be declared in its target module, not re-exported from elsewhere. Known answers are included.

Restacked onto main 1.8.0

The branch now carries main 1.8.0 through its parent (P1a), which is an ancestor of this head.

Scope after the restack. Deletes src/adapter-replacement.ts and its test (with an engineering note and a registry-write gate), the stream hook's dead prepared handling, and the tracked .freebuff/. The settings partition test walks Config.dict, and a manifest test holds cordis.patch.yml to the schema and the field registry, with debugFile described by D24. utils/retry.ts takes its timer and logger as parameters, and the utils purity gate also rejects console and global timers, walking recursively. tool-fallback loses its unreachable Anthropic schemas. test/support/src.ts is the one door into src.

Dropped because main now does it.

• The route-mapped tool-fallback plane (a route option on maybeInjectCoreTools, and a plane lookup by route with no plane-blind fallback). main's feat(test): a fixture harness for the worlds the vendor has not produced #138 rule supersedes it (D25).
• The earlier "Behaviour change" section, which said a model free only on Zen is no longer treated as free on Go. Under D25 the plane-blind fallback admits it, so that change and its test are gone.
• The fetch-patch case "the turn's route decides off a gateway URL". It is rewritten as "the plane comes from the URL's path, never the turn's route".
• The standalone test/support/catalog-fixture.ts, folded into test/support/catalogs.ts together with main's test/fixtures/catalogs.ts.

Decisions applied.

• D25: the tool fallback's plane is the request URL's, with the plane-blind fallback. P1b keeps only the deletion of the dead Anthropic branch.
• D24: debugFile is described by the gate main chose, in src/settings-fields.ts and cordis.patch.yml.
• D14'/D22': unchanged here. The question of verifying both host channels (rc and alpha) is recorded for P-T1a in docs/improvement-plan.md, for the owner to decide before P-T1a starts. The committed install and lockfile stay as they are.

Live re-measures CI must confirm. None are new: no change on this branch touches the wire, the routes or the credentials. CI's unit, shuffled and mock suites and the bundle gate cover the head. The scheduled live probe suites only need to stay green on it.

Review

The change went through three review lenses (correctness and host contracts, tests, repository rules). Two independent skeptics checked each finding. Over three rounds before the restack:

• 30 findings survived: 3 medium, 27 low.
• All of them were fixed. The registry-write gate was kept as item 1's executable form.

The restack had three more rounds of review. Their counts are in the restack comment on this pull request.

Verification

From the restacked head's gate:

Step	Result
pnpm run check	0 errors; 21 warnings
unit suite	85 files, 1346 tests pass
test:mock	3 files, 52 tests pass

The diff contains no local paths.

---

Generated by Claude Code

[viztor]

Restacked onto main 1.8.0. The branch now carries main 1.8.0 through its parent (P1a, #106), which is an ancestor of this head. Pushed head: 239d6352b4a04214e6b71e5693e434981cdc73f4.

The description was written before the restack and has now been rewritten. It no longer claims a behaviour change, item 6 follows D25, and the verification table holds this head's gate numbers.

Conflicts and how they were resolved

Tool fallback (D25)

• src/tool-fallback.ts: took main's structure via P1a (coreToolDecision, and isCoreToolModel with a plane taken from the URL and a plane-blind fallback). Ported only the deletion of the unreachable Anthropic schemas.
• src/fetch-patch.ts (merged cleanly, but wrong): removed P1b's route option on maybeInjectCoreTools. The file is now identical to P1a's.
• test/tool-fallback.test.ts (merged cleanly, but wrong): retargeted to main's signature, with the plane-blind fallback and the URL-path plane asserted.
• test/fetch-patch.test.ts (merged cleanly, but wrong): kept "classifies a free model by the plane the request is on". The route case is rewritten as "the plane comes from the URL's path, never the turn's route", covering relay paths and gateway URLs.

Tests and fixtures

• test/stream-hook.test.ts: kept P1b's door form of hookWith without the unused overrides parameter. main's new cases now go through ./support/src.ts. P1a's D24 cases are unchanged.
• test/fixtures/catalogs.ts and test/support/catalog-fixture.ts: merged into one fixture, test/support/catalogs.ts, behind the door. coreToolDecision was added to the door, and docs/testing.md was updated.
• test/retry.test.ts: changed to satisfy main's new promise(always-return) lint. The assertions are unchanged, and it is committed separately as style:.

Docs

• docs/improvement-plan.md: took P1a's restack status table. P1b's row now reads "merged P1a", and a restack note is added under P1b.
• src/settings-fields.ts and cordis.patch.yml (merged cleanly, but wrong under D24): debugFile is reworded to match main's gate.

Dropped

• P1b's route-mapped plane for the tool fallback, superseded by feat(test): a fixture harness for the worlds the vendor has not produced #138 (D25).
• P1b's "a Zen-only free model is no longer free on Go" change and its test. The plane-blind fallback admits that model.
• The fetch-patch case "the turn's route decides off a gateway URL", rewritten as described above.
• The standalone test/support/catalog-fixture.ts.
• The unused overrides parameter in test/stream-hook.test.ts.

Review rounds

• Round 1: 6 found, 6 upheld, 6 fixed. These covered stale D25 comments, an unasserted fixture premise, plan and AGENTS.md rows, and the setup-file wording.
• Round 2: 6 found, 3 upheld, 3 fixed. All three were about the door rule's exception list and the entry-point count.
• Round 3: 5 found, 5 upheld.
  • Three are about verifying both host channels. They are recorded for P-T1a as an owner decision, with no change to D14'/D22' on this branch.
  • Two are about this description. They are fixed by the rewrite above.

Live re-measures still owed

None from this branch. No change here touches the wire, the routes or the credentials. The scheduled live probe suites only need to stay green on this head.

All reviews ran on the dedicated reviewer. None fell back to the general model.

---

Generated by Claude Code