Repository navigation
Live compaction synthesis can leak raw analysis and summary XML into the feed #345
Description
Problem
During /compact, the feed can render the raw synthesis response as ordinary assistant markdown, including <analysis> / <summary> style content, instead of showing the live compaction placeholder.
Evidence
Bundle 2026-06-22T12-42-54-045-c973322e:
feed-debug.jsonl:claude.compactioncondition is visible/running.proxy-semantic.json.currentTurn.textstarts with<analysis>.- Semantic
turn_startedevent has noisCompactionSynthesis. proxy-events.jsonl: request flow reportsrequest_shape.compaction_synthesis:false.
Bundle 2026-06-14T14-02-05-796-a8ad1ebb:
- Same shape: semantic current turn text starts with
<analysis>, text length around 15k, rendered as a normal semantic block.
Suspected Cause
The proxy compaction detector likely depends on a request signature that drifted. Because request shape explicitly reports compaction_synthesis:false, the adapter does not attach isCompactionSynthesis, and StreamingTurn renders the raw semantic text.
Related paths:
packages/claude-code-headless/src/proxy/mitmAddon.pypackages/claude-code-headless/src/proxy/ClaudeProxyAdapter.tssrc/renderer/src/features/feed/ui/semantic/StreamingTurn.tsx
Expected
When any reliable signal says compaction is running, the feed should never paint raw synthesis XML. It should show the compaction placeholder until compact boundary/summary entries land.
Activity
Juliusolsson05 commentedon Jun 22, 2026
Linking this back to #172. This issue has a provider-detection side, but it is also an ownership failure: raw compaction synthesis text is allowed to render even while another signal says compaction is active.
The detector may still need a targeted fix, but the #172 single-owner feed model should prevent raw synthesis content from painting whenever a compaction owner is active. Re-test after #172 to separate renderer ownership leftovers from proxy/request-shape detection leftovers.
Juliusolsson05 commentedon Aug 30, 2026
Closing as fixed and covered. This one is stronger than a supersede — the fix cites this issue by number in the source.
src/renderer/src/rendering/observations/semantic.ts:19-20:The guard is a first-class suppression in the ledger, not an ad-hoc filter:
compaction-synthesisis a member of the closedRenderReasonenum (rendering/model/types.ts:173), emitted atobservations/semantic.ts:189.RenderReasonmust have at least one fixture proving when it fires — so this suppression carries recorded evidence rather than resting on review judgement. Coverage lives inrendering/observations/semantic.test.tsandcommitted.test.ts.Raw
<analysis>/<summary>XML can therefore no longer reach the feed as assistant prose: the collector declines to emit a candidate at all, and the decision is recorded with its reason.Landed via PR #427 (semantic collector with model-level guards).