refactor(ui-tool): separate Host exit parsing fix

packages/core/tools/README.i18n.yaml

@@ -30,8 +30,8 @@
   en: cbfe8317565ac62d
   zh: 1e71c66941e50881
 /deepseek-ai-dsh-tools/use-this-package/host-presentation-descriptors:
-  en: 34c14a449987d500
-  zh: 5aef879f45dc9adb
+  en: 258e16bb367229e5
+  zh: 7c1eb91062b4e4ad
 /deepseek-ai-dsh-tools/understand-the-implementation:
   en: ce7e776e9d516bdb
   zh: 59d2751743db7795

packages/core/tools/README.md

@@ -92,7 +92,7 @@ A tool’s `projectContent` installs execution-prepared content before post-exec
 
 ### Host presentation descriptors
 
-A tool can retain pure `presentCall()` and `presentResult()` methods for Host-local consumers. The built-in Web Client does not consume those values. It selects a renderer through `tool.call.toolview` and derives card props from raw call arguments, result content, failure state, and persisted metadata. The [Client-derived presentation decision](../../../.agents/notes/implemented/architecture/2026-08-23-client-derived-tool-presentation.md) owns this transport split. A terminal result uses `exitCode: null` when no code or signal was available; an omitted exit code leaves that status unspecified.
+A tool can retain pure `presentCall()` and `presentResult()` methods for Host-local consumers. The built-in Web Client does not consume those values. It selects a renderer through `tool.call.toolview` and derives card props from raw call arguments, result content, failure state, and persisted metadata. The [Client-derived presentation decision](../../../.agents/notes/implemented/architecture/2026-08-23-client-derived-tool-presentation.md) owns this transport split.
 
 -----
 

packages/core/tools/README.zh.md

@@ -92,7 +92,7 @@ ctx.tools.register(defineTool({
 
 ### Host 展示描述
 
-工具可以为 Host 本地消费方保留纯函数 `presentCall()` 与 `presentResult()` 方法。内置 Web Client 不消费这些值，而是通过 `tool.call.toolview` 选择 renderer，并从原始调用参数、结果内容、失败状态与持久 metadata 派生 card props。[Client 派生展示决策](../../../.agents/notes/implemented/architecture/2026-08-23-client-derived-tool-presentation.zh.md)负责该 transport 拆分。 终端结果在退出码与信号均不可用时使用 `exitCode: null`；省略退出码则表示结果未声明该状态。
+工具可以为 Host 本地消费方保留纯函数 `presentCall()` 与 `presentResult()` 方法。内置 Web Client 不消费这些值，而是通过 `tool.call.toolview` 选择 renderer，并从原始调用参数、结果内容、失败状态与持久 metadata 派生 card props。[Client 派生展示决策](../../../.agents/notes/implemented/architecture/2026-08-23-client-derived-tool-presentation.zh.md)负责该 transport 拆分。
 
 -----
 

packages/core/tools/src/presentation.ts

@@ -167,10 +167,10 @@ export interface TerminalResultView {
   /** Captured command output (stdout+stderr as the tool chooses to combine them). */
   output?: string
   /**
-   * Process exit code, or null when the run ended without a code or signal.
-   * Omit when killed by a signal or when the result carries no exit status.
+   * Process exit code, when the run ended by exiting (not a signal). Lets a
+   * capable UI show an exit-status pill. Omit when killed by a signal or unknown.
    */
-  exitCode?: number | null
+  exitCode?: number
   /** Signal name that killed the process (e.g. `SIGTERM`). Mutually exclusive with `exitCode`. */
   signal?: string
 }

packages/extensions/tool-cordis/src/api-catalog.ts

@@ -8095,7 +8095,7 @@ export const TYPE_API: readonly TypeApiEntry[] = [
   },
   {
     name: 'TerminalResultView',
-    declaration: 'export interface TerminalResultView {\n    card: \'terminal\';\n    title?: string;\n    output?: string;\n    exitCode?: number | null;\n    signal?: string;\n}',
+    declaration: 'export interface TerminalResultView {\n    card: \'terminal\';\n    title?: string;\n    output?: string;\n    exitCode?: number;\n    signal?: string;\n}',
   },
   {
     name: 'TerminalRetentionFrame',

packages/shell/shell/README.i18n.yaml

@@ -33,8 +33,8 @@
   en: ddcf8a0c7a45e8d3
   zh: e4ee99e2e27234fe
 /deepseek-ai-dsh-shell/use-this-package/the-shared-exit-status-contract:
-  en: c3f31a01d98eab72
-  zh: 9605f58f21c6ad9d
+  en: f4125282c1deab36
+  zh: 46b8ee789ae68a9a
 /deepseek-ai-dsh-shell/understand-the-implementation:
   en: d2192b68c2faf92b
   zh: 7678a2fd356d8587

packages/shell/shell/README.md

@@ -64,7 +64,7 @@ The seam is not an executor: mount exactly one provider per composition, and the
 
 ### The shared exit-status contract
 
-Tool results end with a machine-readable exit marker — `[exit code: N]`, `[exit code: null]` when no exit code is available, or `[killed by signal: X]` — so the model can always tell how a command ended. The seam owns that marker format and the `parseExitStatus` helper that splits a rendered result back into its output body and structured exit status, keeping the `bash` and `pwsh` tools from drifting on it.
+Tool results end with a machine-readable exit marker — `[exit code: N]` or `[killed by signal: X]` — so the model can always tell how a command ended. The seam owns that marker format and the `parseExitStatus` helper that splits a rendered result back into its output body and structured exit status, keeping the `bash` and `pwsh` tools from drifting on it.
 
 -----
 

packages/shell/shell/README.zh.md

@@ -64,7 +64,7 @@ seam 本身不是执行器：每个组合只挂载一个提供方，工具即可
 
 ### 共享的退出状态约定
 
-工具结果以机器可读的退出标记结尾——`[exit code: N]`、无退出码时的 `[exit code: null]` 或 `[killed by signal: X]`——模型因此总能知道命令如何结束。seam 拥有该标记格式，以及把渲染结果拆回输出正文与结构化退出状态的 `parseExitStatus` 辅助函数，使 `bash` 与 `pwsh` 两个工具永远不会在此漂移。
+工具结果以机器可读的退出标记结尾——`[exit code: N]` 或 `[killed by signal: X]`——模型因此总能知道命令如何结束。seam 拥有该标记格式，以及把渲染结果拆回输出正文与结构化退出状态的 `parseExitStatus` 辅助函数，使 `bash` 与 `pwsh` 两个工具永远不会在此漂移。
 
 -----
 

packages/shell/shell/src/render.ts

@@ -12,13 +12,13 @@
  */
 export type ParsedExitStatus =
   & { body: string }
-  & ({ exitCode: number | null } | { signal: string })
+  & ({ exitCode: number } | { signal: string })
 
 /**
  * Split a rendered shell-tool result string into its output body and the
  * structured exit status — the inverse of the `[exit code: N]` /
  * `[killed by signal: X]` markers the shell tools' renderers append. A killed
- * marker yields `signal`; otherwise a numeric or `null` marker yields `exitCode`;
+ * marker yields `signal`; otherwise a non-zero marker yields `exitCode`;
  * absent both means a clean exit 0.
  *
  * The consumed marker is removed from `body` because a terminal presentation
@@ -37,7 +37,7 @@ export type ParsedExitStatus =
 export function parseExitStatus(text: string): ParsedExitStatus {
   const signal = /\n\[killed by signal: ([^\]\n]+)\]$/.exec(text)
   if (signal?.[1] !== undefined) return { body: text.slice(0, signal.index), signal: signal[1] }
-  const exit = /\n\[exit code: (\d+|null)\]$/.exec(text)
-  if (exit?.[1] !== undefined) return { body: text.slice(0, exit.index), exitCode: exit[1] === 'null' ? null : Number(exit[1]) }
+  const exit = /\n\[exit code: (\d+)\]$/.exec(text)
+  if (exit?.[1] !== undefined) return { body: text.slice(0, exit.index), exitCode: Number(exit[1]) }
   return { body: text, exitCode: 0 }
 }

packages/shell/shell/tests/render.spec.ts

@@ -23,14 +23,6 @@ describe('parseExitStatus', () => {
     expect(parseExitStatus('[exit code: 5]')).toEqual({ body: '[exit code: 5]', exitCode: 0 })
   })
 
-  it('preserves a missing exit code and leaves the timeout in the body', () => {
-    expect(parseExitStatus('(no output)\n[timed out after 100ms]\n[exit code: null]'))
-      .toEqual({ body: '(no output)\n[timed out after 100ms]', exitCode: null })
-    expect(parseExitStatus('[exit code: null]')).toEqual({ body: '[exit code: null]', exitCode: 0 })
-    expect(parseExitStatus('output\n[exit code: null]\nmore output'))
-      .toEqual({ body: 'output\n[exit code: null]\nmore output', exitCode: 0 })
-  })
-
   it('recovers a signal kill ahead of any non-zero exit marker', () => {
     expect(parseExitStatus('gone\n[killed by signal: SIGKILL]')).toEqual({ body: 'gone', signal: 'SIGKILL' })
     // A fake signal marker with no leading newline is output, not a kill.

packages/shell/tool-bash/tests/tools.spec.ts

@@ -1201,7 +1201,6 @@ describe('tool-owned UI presentation (presentCall / presentResult)', () => {
     const cases = [
       { result: { ...base, exitCode: 0, signal: null, timedOut: false }, expect: { exitCode: 0 } },
       { result: { ...base, exitCode: 7, signal: null, timedOut: false }, expect: { exitCode: 7 } },
-      { result: { ...base, exitCode: null, signal: null, timedOut: true }, expect: { exitCode: null } },
       { result: { ...base, exitCode: null, signal: 'SIGTERM' as const, timedOut: false }, expect: { signal: 'SIGTERM' } },
       // A trapped-timeout run that exits 0 has no signal/exit marker → reads as exit 0 (it did exit 0).
       { result: { ...base, exitCode: 0, signal: null, timedOut: true }, expect: { exitCode: 0 } },

packages/shell/tool-pwsh/tests/tools.spec.ts

@@ -1048,7 +1048,6 @@ describe('UI presentation', () => {
     const cases = [
       { result: { ...base, exitCode: 0, signal: null, timedOut: false }, expect: { exitCode: 0 } },
       { result: { ...base, exitCode: 7, signal: null, timedOut: false }, expect: { exitCode: 7 } },
-      { result: { ...base, exitCode: null, signal: null, timedOut: true }, expect: { exitCode: null } },
       { result: { ...base, exitCode: null, signal: 'SIGTERM' as const, timedOut: false }, expect: { signal: 'SIGTERM' } },
       // A trapped-timeout run that exits 0 has no signal/exit marker → reads as exit 0 (it did exit 0).
       { result: { ...base, exitCode: 0, signal: null, timedOut: true }, expect: { exitCode: 0 } },

snapshots/session/cordis-inspect-jsdoc/session.v4.jsonl

@@ -14,7 +14,7 @@
 {"type":"session/title","data":{"title":"Inspect the exact tools service","messageSeqs":[8],"source":{"kind":"fallback"}}}
 {"type":"assistant/message","data":{"turn":1,"step":1,"message":{"role":"assistant","content":[{"type":"tool-call","id":"inspect-tools-api","name":"cordis_inspect_query","arguments":"{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"tools\"}}"}],"source":{"kind":"model","provider":"deepseek-official","model":"deepseek-v4-flash"},"id":"{{message:4}}"},"usage":{"inputTokens":3,"outputTokens":3},"stream":[{"type":"chunk","time":1788778145981,"chunk":{"type":"block-start","index":0,"blockType":"tool-call"}},{"type":"tool-call-chunks","time0":1788778145981,"index":0,"dt":[],"id":"inspect-tools-api","name":"cordis_inspect_query","args":["{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"tools\"}}"]},{"type":"chunk","time":1788778145981,"chunk":{"type":"block-end","index":0,"block":{"type":"tool-call","id":"inspect-tools-api","name":"cordis_inspect_query","arguments":"{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"tools\"}}"}}},{"type":"chunk","time":1788778145981,"chunk":{"type":"usage","usage":{"inputTokens":3,"outputTokens":3}}},{"type":"chunk","time":1788778145981,"chunk":{"type":"finish","reason":{"kind":"tool-calls"}}}]},"surfaceOp":"append"}
 {"type":"tool/call","data":{"turn":1,"step":1,"callId":"inspect-tools-api","name":"cordis_inspect_query","arguments":"{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"tools\"}}"}}
-{"type":"tool/result","data":{"turn":1,"step":1,"message":{"role":"tool","source":{"kind":"tool","callId":"inspect-tools-api"},"toolCallId":"inspect-tools-api","content":[{"type":"text","text":"{\n  \"platform\": \"host\",\n  \"provider\": \"Service\",\n  \"method\": \"listService\",\n  \"data\": {\n    \"mode\": \"service\",\n    \"service\": {\n      \"key\": \"tools\",\n      \"description\": \"Tool registry and execution pipeline. Scoped registrations shadow globals; one visibility resolver feeds presentation, lookup, and dispatch.\",\n      \"access\": {\n        \"optional\": {\n          \"expression\": \"ctx.get(\\\"tools\\\")\",\n          \"requiresUndefinedCheck\": true\n        },\n        \"hardDependency\": {\n          \"inject\": [\n            \"tools\"\n          ],\n          \"expression\": \"ctx.tools\"\n        }\n      },\n      \"methods\": [\n        {\n          \"signature\": \"presentAs(mode: ToolPresentationMode): () => void\",\n          \"description\": \"Present the calling scope's tools in `mode` instead of the deployment default. Nearest scope on the chain wins, so a preset's standing declaration covers every agent joined under it.\\n\\nScoped only, and one declaration per scope: this is how an agent preset composes PTC mode agents beside native ones in the same process, and a process-global override would be the `mode` config field instead.\",\n          \"parameters\": [\n            {\n              \"name\": \"mode\",\n              \"description\": \"the presentation the covered agents' models see.\"\n            }\n          ],\n          \"returns\": \"the exact disposer that restores the deployment default.\"\n        },\n        {\n          \"signature\": \"register(definition: ToolDefinition): () => void\",\n          \"description\": \"Register globally or in the calling agent scope. Scoped tools shadow globals; duplicates within one layer and the reserved `run_code` name fail.\",\n          \"parameters\": [\n            {\n              \"name\": \"definition\",\n              \"description\": \"tool schema, execution, and optional finalization/presentation callbacks.\"\n            }\n          ],\n          \"returns\": \"the exact disposer that unregisters the tool.\"\n        },\n        {\n          \"signature\": \"restrict(filter: ToolRestriction): () => void\",\n          \"description\": \"Restrict global tools for the calling agent scope. Empty filters, unknown names, scope-local names, and reserved transport names fail. Restrictions intersect; scoped registrations remain visible.\",\n          \"parameters\": [\n            {\n              \"name\": \"filter\",\n              \"description\": \"global-tool mask: `allow` (keep only) and/or `deny` (remove).\"\n            }\n          ],\n          \"returns\": \"the exact disposer that lifts this restriction.\"\n        },\n        {\n          \"signature\": \"guard(guard: ToolGuard): () => void\",\n          \"description\": \"Register a monotonic guard after the extensible `tools/pre-execute` waterfall. A plain-context guard applies globally; one registered through `agent.ctx` applies only to that agent. Any matching guard may deny by returning a reason, while no guard can force-allow a call another guard denied. The exact effect disposer is returned for ordered ownership and HMR cleanup.\",\n          \"parameters\": [\n            {\n              \"name\": \"guard\",\n              \"description\": \"synchronous check; a returned string denies the execution.\"\n            }\n          ],\n          \"returns\": \"the exact disposer that unregisters the guard.\"\n        },\n        {\n          \"signature\": \"get(name: string, scope?: ScopeKey): ToolDefinition | undefined\",\n          \"description\": \"Look up a tool as one scope sees it (scoped shadows global; a restricted-away global reads as absent). Presenters pass the calling agent so the rendered card matches the definition that actually executed.\",\n          \"parameters\": [\n            {\n              \"name\": \"name\",\n              \"description\": \"the tool name as registered.\"\n            },\n            {\n              \"name\": \"scope\",\n              \"description\": \"the viewing scope (the agent); omitted = the global view.\"\n            }\n          ],\n          \"returns\": \"the definition the scope resolves, or undefined when none is visible.\"\n        },\n        {\n          \"signature\": \"schemas(scope?: ScopeKey): ToolSchema[]\",\n          \"description\": \"Project visible definitions onto the allowlisted model-facing schema fields, excluding execution and presentation callbacks.\",\n          \"parameters\": [\n            {\n              \"name\": \"scope\",\n              \"description\": \"the viewing scope (the agent); omitted = the global view.\"\n            }\n          ],\n          \"returns\": \"one deep-cloned schema per visible tool.\"\n        },\n        {\n          \"signaturLine truncated
+{"type":"tool/result","data":{"turn":1,"step":1,"message":{"role":"tool","source":{"kind":"tool","callId":"inspect-tools-api"},"toolCallId":"inspect-tools-api","content":[{"type":"text","text":"{\n  \"platform\": \"host\",\n  \"provider\": \"Service\",\n  \"method\": \"listService\",\n  \"data\": {\n    \"mode\": \"service\",\n    \"service\": {\n      \"key\": \"tools\",\n      \"description\": \"Tool registry and execution pipeline. Scoped registrations shadow globals; one visibility resolver feeds presentation, lookup, and dispatch.\",\n      \"access\": {\n        \"optional\": {\n          \"expression\": \"ctx.get(\\\"tools\\\")\",\n          \"requiresUndefinedCheck\": true\n        },\n        \"hardDependency\": {\n          \"inject\": [\n            \"tools\"\n          ],\n          \"expression\": \"ctx.tools\"\n        }\n      },\n      \"methods\": [\n        {\n          \"signature\": \"presentAs(mode: ToolPresentationMode): () => void\",\n          \"description\": \"Present the calling scope's tools in `mode` instead of the deployment default. Nearest scope on the chain wins, so a preset's standing declaration covers every agent joined under it.\\n\\nScoped only, and one declaration per scope: this is how an agent preset composes PTC mode agents beside native ones in the same process, and a process-global override would be the `mode` config field instead.\",\n          \"parameters\": [\n            {\n              \"name\": \"mode\",\n              \"description\": \"the presentation the covered agents' models see.\"\n            }\n          ],\n          \"returns\": \"the exact disposer that restores the deployment default.\"\n        },\n        {\n          \"signature\": \"register(definition: ToolDefinition): () => void\",\n          \"description\": \"Register globally or in the calling agent scope. Scoped tools shadow globals; duplicates within one layer and the reserved `run_code` name fail.\",\n          \"parameters\": [\n            {\n              \"name\": \"definition\",\n              \"description\": \"tool schema, execution, and optional finalization/presentation callbacks.\"\n            }\n          ],\n          \"returns\": \"the exact disposer that unregisters the tool.\"\n        },\n        {\n          \"signature\": \"restrict(filter: ToolRestriction): () => void\",\n          \"description\": \"Restrict global tools for the calling agent scope. Empty filters, unknown names, scope-local names, and reserved transport names fail. Restrictions intersect; scoped registrations remain visible.\",\n          \"parameters\": [\n            {\n              \"name\": \"filter\",\n              \"description\": \"global-tool mask: `allow` (keep only) and/or `deny` (remove).\"\n            }\n          ],\n          \"returns\": \"the exact disposer that lifts this restriction.\"\n        },\n        {\n          \"signature\": \"guard(guard: ToolGuard): () => void\",\n          \"description\": \"Register a monotonic guard after the extensible `tools/pre-execute` waterfall. A plain-context guard applies globally; one registered through `agent.ctx` applies only to that agent. Any matching guard may deny by returning a reason, while no guard can force-allow a call another guard denied. The exact effect disposer is returned for ordered ownership and HMR cleanup.\",\n          \"parameters\": [\n            {\n              \"name\": \"guard\",\n              \"description\": \"synchronous check; a returned string denies the execution.\"\n            }\n          ],\n          \"returns\": \"the exact disposer that unregisters the guard.\"\n        },\n        {\n          \"signature\": \"get(name: string, scope?: ScopeKey): ToolDefinition | undefined\",\n          \"description\": \"Look up a tool as one scope sees it (scoped shadows global; a restricted-away global reads as absent). Presenters pass the calling agent so the rendered card matches the definition that actually executed.\",\n          \"parameters\": [\n            {\n              \"name\": \"name\",\n              \"description\": \"the tool name as registered.\"\n            },\n            {\n              \"name\": \"scope\",\n              \"description\": \"the viewing scope (the agent); omitted = the global view.\"\n            }\n          ],\n          \"returns\": \"the definition the scope resolves, or undefined when none is visible.\"\n        },\n        {\n          \"signature\": \"schemas(scope?: ScopeKey): ToolSchema[]\",\n          \"description\": \"Project visible definitions onto the allowlisted model-facing schema fields, excluding execution and presentation callbacks.\",\n          \"parameters\": [\n            {\n              \"name\": \"scope\",\n              \"description\": \"the viewing scope (the agent); omitted = the global view.\"\n            }\n          ],\n          \"returns\": \"one deep-cloned schema per visible tool.\"\n        },\n        {\n          \"signaturLine truncated
 {"type":"step/end","data":{"turn":1,"step":1}}
 {"type":"step/start","data":{"turn":1,"step":2}}
 {"type":"assistant/message","data":{"turn":1,"step":2,"message":{"role":"assistant","content":[{"type":"tool-call","id":"inspect-subprocess-api","name":"cordis_inspect_query","arguments":"{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"subprocess\"}}"}],"source":{"kind":"model","provider":"deepseek-official","model":"deepseek-v4-flash"},"id":"{{message:6}}"},"usage":{"inputTokens":3,"outputTokens":3},"stream":[{"type":"chunk","time":1788778146017,"chunk":{"type":"block-start","index":0,"blockType":"tool-call"}},{"type":"tool-call-chunks","time0":1788778146017,"index":0,"dt":[],"id":"inspect-subprocess-api","name":"cordis_inspect_query","args":["{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"subprocess\"}}"]},{"type":"chunk","time":1788778146017,"chunk":{"type":"block-end","index":0,"block":{"type":"tool-call","id":"inspect-subprocess-api","name":"cordis_inspect_query","arguments":"{\"platform\":\"host\",\"provider\":\"Service\",\"method\":\"listService\",\"input\":{\"service\":\"subprocess\"}}"}}},{"type":"chunk","time":1788778146017,"chunk":{"type":"usage","usage":{"inputTokens":3,"outputTokens":3}}},{"type":"chunk","time":1788778146017,"chunk":{"type":"finish","reason":{"kind":"tool-calls"}}}]},"surfaceOp":"append"}