[trissim]

What changed

Two commands broke the CLI family's Answer: CliResult bound and hid it with their own run overrides:

• WatchCliCommand.apply returned an async iterator.
• InvokeCliCommand.apply returned a dict.

Both wrote output by hand beside CliCommand.written. Writing now belongs to the result:

• CliResult.write(output) is the one writer. CliCommand.run and the three installer-facing execute overrides (read-back, store-reconcile, replace-route) call it. CliCommand.written is deleted.
• WatchStream(CliResult) owns the JSON-lines stream. WatchCliCommand declares CliCommand[WatchStream], and its run override is deleted.
• ToolReplyResult(CliResult) carries the tool protocol reply, unwrapped, as its wire form. InvokeCliCommand declares CliCommand[ToolReplyResult], and its run override is deleted.

The ratchet already exists

debt-src already runs AnyName and UnannotatedParameter in tools/debt_ratchet.py. Both are AST-based and counted per file, and Measure.blocks fails any growth in any touched file, so no new measure is needed.

Proof that AnyName tracks real edits, measured across #2762's merge (f6fad69^1 → f6fad69):

Measure	Before	After
AnyName	41	39
UnannotatedParameter	1642	1642

The whole −2 is command.py:17 (def apply(self, ctx: Any) -> Any), which is exactly what the diff removed. #2762's report of "42 → 38" counted import lines as uses.

Not drained here, with owners

• registry_document.py:508: mypy's replace check types each keyword against its field, so **reused only typechecks as Any. The section names come from FieldCodec.field_plans. The owner of dynamic field replacement is FieldCodec, which Give DM peers one typed encoding, fix stale tests, repin native build #2763 holds.
• child_process.py:1968: SpawnedProcess.launch(**options: Any) forwards to launching. Deriving its signature with a ParamSpec fails under mypy (launching in the class body has no __func__, and launch turns into Any). The real owner is a declared launch-options record, which touches 31 call sites; that's a slice of its own.

This PR leaves both measures unchanged: AnyName stays at 39 and UnannotatedParameter at 1642. It fixed two bound violations, not Any sites.

Checks run locally

• mypy and ruff on the touched files, head against main with the project config: no new mypy messages, one fewer error each in cli_commands.py and tools.py, ruff counts equal.
• Touched tests compared against main: see the comment below.

🤖 Generated with Claude Code

[trissim]

Touched tests (test_cli, test_watch_claims, test_tools, test_child_process, test_command_families, test_codec_capabilities, test_runtime, test_mcp_acceptance), -p no:xdist: head 7f0b38e 156 passed / 4 failed; origin/main 156 passed / 4 failed; identical failing set (test_command_families normal_history[DmHistoryPageResult-1], test_runtime new_session_metadata_has_a_display_title_before_first_switch, test_runtime subscription_replay_uses_requested_alias[True/False]). None new.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.