`createChatMessage` and `promoteChatQueuedMessage` were awaiting
`invalidateChatDebugRuns` inside `onSuccess`, which made their
`mutateAsync` promises pend until the Debug panel's active refetch
completed. `sendMessage`/promote callers in `AgentChatPage` await
that promise before running send-side UI updates, so an open Debug
panel was adding avoidable latency to every send and promote.

Switch both handlers to fire-and-forget `void` invalidation, matching
the pattern already used in `editChatMessage.onSettled`.

Change-Id: I8edb16124cadf7eb984c0ab4d0ba299156cbe3eb
Signed-off-by: Thomas Kosiewski <tk@coder.com>

`tryDecodeBase64` used `atob` directly, which is lenient and happily
turned plain-text strings like `"test"` or `"abcd"` (valid base64
alphabet, length divisible by 4) into mojibake whenever the body was
not JSON.

Gate the decode with a strict base64 regex and a fatal UTF-8 decode
so only truly base64-encoded UTF-8 payloads are rewritten; everything
else falls through and the raw string is preserved. Add regression
tests in `normalizeAttempts` covering both the preservation path and
the happy-path decode of non-JSON base64 text.

Change-Id: I691dce0d2f68ab9d271a88dc295eb2f1eacd195f
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Streamed responses can carry a `tool_input` streaming delta followed by
a finalized `tool_call` summary for the same call ID. Because
`isToolCallPartType` accepts both variants, the content loop emitted
two rows for the same invocation in the Debug panel.

Collapse by call ID inside `coerceStepResponse`: track per-ID index and
whether we already have a finalized entry, prefer the finalized payload
when a partial was seen first, and ignore stray partials that arrive
after the finalized form. Entries without an ID fall through unchanged.

Adds regression tests covering the tool_input→tool_call upgrade path,
the tool_call→tool_input stability path, and multi-call ordering.

Change-Id: Ifb8fe32ce9f8fcec040c7dc2364c00ef8751dc0d
Signed-off-by: Thomas Kosiewski <tk@coder.com>

The regression test for the tool_input → tool_call upgrade path used
`'{"query":"fo'` as the partial streaming fixture. `typos` treats `fo`
as a suggestion for `of`/`for`/`do`/`go`/`to`, failing the repo-wide
typos lint step.

Shorten the partial value to `'{"query":"f'` — the test semantic is
unchanged (partial arguments get replaced by the finalized payload)
and the shorter fragment no longer trips the typos rule.

Change-Id: Ief69a7ae91680d69d513b992a12a62a1e0fba45f
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Review round 23 flagged en-dash characters (U+2013) in debug utils
comments and the knip whole-file exclusion with no tracking comment.
Switch those dashes to ASCII `--` and add a TODO noting that the
knip ignore is only needed until the Debug panel components land in
PRs 8/9.

Change-Id: I7fc85b275f9ea4659d2eb62a12d47eaab95aa90a
Signed-off-by: Thomas Kosiewski <tk@coder.com>

coerceRunSummary and coerceStepRequest's options/policy inputs can arrive as JSON-string wrappers after a json.RawMessage round-trip. Run them through deepParse before the isRecord check so the wrapped values are exposed instead of silently dropped.

Change-Id: I9dae8bc6afa8ba77b5337a38ad957397a3ee0884
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Go's zero value for string is "" and ChatStreamToolCall.tool_call_id
serializes without omitempty, so unset IDs arrive as "" on the wire.
Treating "" as a real dedup key collapsed distinct tool calls onto the
same Map slot, silently dropping the second call. Normalize at the two
extraction sites so the dedup guard and downstream React keys treat it
as "no id".

Change-Id: I43e38f584ec617223e3853d54b62ed9dfa792924
Signed-off-by: Thomas Kosiewski <tk@coder.com>

interruptChat was the only debug-run-producing mutation that didn't kick
the Debug panel query on success. The other four (createChatMessage,
editChatMessage, promoteChatQueuedMessage, regenerateChatTitle) all
invalidate chatDebugRunsKey. Without it, the panel shows stale
"running" for up to the 5s poll interval after the user interrupts.

Change-Id: I086ca5dde03439f6bc48458b91f9d0f0e81041fa
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Drop inline debug-run invalidation comments (now self-evident from
`invalidateChatDebugRuns` call), move debug-logging queries into
`chatDebugLogging.ts` so the @public JSDoc annotations can be dropped,
and re-export them from `chats.ts` so upstack imports continue to
resolve.

Change-Id: I461a27027797b579ee148be7b9b84bf48bbf6a87
Signed-off-by: Thomas Kosiewski <tk@coder.com>