Skip to content

OOM diagnostics: silence-scan breadcrumbs, unified log subsystem, separation kill switch - #85

Merged
sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-oom-diagnostics
Jul 18, 2026
Merged

sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-oom-diagnostics

Conversation

@sanylax2

Copy link
Copy Markdown
Collaborator

What

PR 1 of the jetsam RCA plan — closes the diagnostic blind spots so one instrumented run pins the eater:

  • SilenceScan breadcrumbs: it can decode ~32 MB PCM buffers ×3 concurrently with zero logging; now bracketed with mem[silence scan begin/end].
  • Unified log subsystem: MemoryFootprint and StemSeparator logged under com.sanylax.continuity while every other logger uses com.continuity.app — a single Console filter would silently hide them. All on com.continuity.app now.
  • Separation kill switch: launch argument -debug.disableStemSeparation YES makes ensureStems a no-op, so one run can rule the whole ONNX pipeline in or out of the repro without code edits.

How to run the diagnosis (from the approved plan)

  1. Merge this, rebuild, launch fresh with the Xcode console open, press play, wait for the kill.
  2. Filter console for mem[ — the last breadcrumb before death names the stage: session load vs window 0 vs stair-stepping windows (→ separation), or separation done/no breadcrumbs with smoothly decaying headroom (→ UI render ramp).
  3. Grab the JetsamEvent-*.ips from Settings → Privacy & Security → Analytics Data for the footprint at kill.
  4. Optional cross-check: repeat the run with -debug.disableStemSeparation YES in the scheme's launch arguments — surviving that run confirms the separation branch.

🤖 Generated with Claude Code

https://claude.ai/code/session_013TJoWkqg8bzkGdzxWhWjWP


Generated by Claude Code

…aration kill switch

Closes the diagnostic gaps found in the jetsam RCA: SilenceScan could
decode ~32 MB buffers x3 with no breadcrumb; the mem breadcrumbs and
StemSeparator logged under a different subsystem than every other
logger (a single Console filter would hide them); and there was no way
to rule the separation pipeline in or out of a repro without editing
code. Adds silence-scan begin/end breadcrumbs, unifies all loggers on
com.continuity.app, and adds a -debug.disableStemSeparation launch
argument guard in ensureStems.
@sanylax0
sanylax0 merged commit 309bcb4 into main Jul 18, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants