From 2497055f6315e3dd2af7422cba3dde51db5c7d82 Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Mon, 28 Sep 2026 12:58:42 +0200 Subject: Let arecord die with the assistant It was the only child started without PR_SET_PDEATHSIG. The assistant died with no log line and left arecord recording into the unlinked raw file for ~76 minutes. Co-Authored-By: Claude Opus 5.5 --- AGENTS.md | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'AGENTS.md') diff --git a/AGENTS.md b/AGENTS.md index 770be39..4aef988 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -303,6 +303,10 @@ drives `arecord` on/off, clip POSTed to `/inference`. `prctl(PR_SET_PDEATHSIG, SIGTERM)` via ctypes in `preexec_fn`: they die with the assistant even on SIGKILL or a kill during startup (tested). Python's default SIGTERM skips `finally`; a handler calls `sys.exit`. +- 2026-09-28: `arecord` lacked it. The assistant died with no log line + (last: "ready, mic is on", cause unknown) and left `arecord` recording + into the unlinked raw file for ~76 min. It now gets `die_with_parent` + too (not yet tested live); `assistant.sh status` lists a stray one. - Gemma confidently gave a wrong fact (a city-to-city distance off by ~10x). No tools yet, so no time/weather/facts lookup. - Recordings live in `$XDG_RUNTIME_DIR` and are deleted on exit. -- cgit v1.2.3