Facets-cloud/flow

Today flow do <task> calls liveClaudeSessions() to detect duplicate-tab races before spawning. If ps fails (sandboxed env, broken shim, etc.) the function returns (nil, err) and the live-session guard in internal/app/do.go:176 silently falls through to the spawn path: if live, err := liveClaudeSessions(); err == nil { // ... focus / refuse logic } The error is swallowed with no user-visible signal. In environments where ps reliably fails, users will silently see duplicate-spawn behavior with no indication that the safety check was skipped. Proposed change When liveClaudeSessions returns a non-nil error, print a single line to stderr before falling through: warning: live-session check skipped: <error> Then continue to the spawn path as today. Behavior on the happy path is unchanged. Files internal/app/do.go:175-197 — add the stderr line in the else branch of the err == nil check internal/app/do_test.go — extend an existing test (or add one) that stubs psRunner to return an err

The regex matching --session-id <uuid> / --resume <uuid> in a claude argv is duplicated byte-for-byte across 5 files: internal/app/sessions.go:19 (claudeSessionArgRe) internal/iterm/iterm.go:107 (claudeSessionRowRe) internal/terminal/terminal.go:195 (claudeSessionRowRe) internal/zellij/zellij.go:134 (claudeSessionRowRe) internal/kitty/kitty.go:163 (claudeSessionRowRe) All five compile the same pattern: `(?:--session-id|--resume)[ =]([0-9a-fA-F]{8}-[0-9a-fA-F]{4}-4[0-9a-fA-F]{3}-[89abAB][0-9a-fA-F]{3}-[0-9a-fA-F]{12})` Proposed change Create a new package internal/claudematch exporting: the compiled regex a small helper like func ExtractSessionUUID(line string) (string, bool) returning the lowercased UUID if a match is found Update the 5 call sites to import and use it. Behavior must be identical — drop the duplicate vars in each backend. Notes Pure refactor: no behavior change, no contract change, no test rewrites beyond import paths. Existing per-package tests should keep passi