Skip to content

Log a ~10s memory heartbeat during playback - #90

Merged
sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-playback-mem-heartbeat
Jul 18, 2026
Merged

sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-playback-mem-heartbeat

Conversation

@sanylax2

Copy link
Copy Markdown
Collaborator

What

The latest kill survived a full song of plain playback with zero breadcrumbs in the console — the stem pipeline never ran, so the remaining eater lives in a window no existing instrumentation covers. Player.tick() now logs os_proc_available_memory every ~10 s while playing (mem[playback] N MB headroom, category audio).

Why this matters

The next kill's console gives three numbers we currently lack: the headroom baseline right after play starts (reveals the actual iOS 26 per-process budget), the decay slope (MB lost per 10 s — a leak rate), and the floor at the last line before death. That discriminates the remaining suspects (per-tick animation churn in the leaves, artwork/URLCache growth, AVAudioEngine/timePitch internals, or a system-wide vm-pageshortage kill that isn't the app's fault at all — the JetsamEvent reason field cross-checks that last one).

Ask

After merging: reproduce once and paste all the mem[playback] lines plus the top ~15 lines of the newest JetsamEvent-*.ips (especially reason and the app's footprint).

🤖 Generated with Claude Code

https://claude.ai/code/session_013TJoWkqg8bzkGdzxWhWjWP


Generated by Claude Code

The latest jetsam took a full song of plain playback with zero ingest
activity — a window no existing breadcrumb covers. tick() now logs
os_proc_available_memory every ~10 s while playing, so the next kill's
console shows the decay slope (how fast, from what baseline) instead
of silence.
@sanylax0
sanylax0 merged commit 7ae851f into main Jul 18, 2026
1 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