hooks/ponytail-runtime.js: setMode has no try/catch — disk full crashes hook

[nanaubusiness]

In hooks/ponytail-runtime.js lines 15-18:

function setMode(mode) {
  fs.mkdirSync(path.dirname(statePath), { recursive: true });
  fs.writeFileSync(statePath, mode);
}

No try/catch. If disk is full, permissions are wrong, or path is invalid, this throws and crashes the hook. Should gracefully degrade instead of propagating a crash.

[nanaubusiness]

Already fixed in PR #394.

[nanaubusiness]

Reopening — corresponding PR is under review.

[DietrichGebert]

Closing as already handled. Both call sites of setMode already wrap it in try/catch, so a disk-full / permission / bad-path failure degrades silently instead of crashing the hook:

• hooks/ponytail-activate.js: try { setMode(mode); } catch (e) { /* flag is best-effort, do not block the hook */ }
• hooks/ponytail-mode-tracker.js: setMode runs inside finish()`s try/catch (which also has to wrap JSON.parse anyway).

Verified live on current main by forcing the write to fail (pointing CLAUDE_CONFIG_DIR into a file so mkdirSync throws): both hooks exit 0 with no crash, and the mode message is still emitted. So the reported crash does not reproduce. Adding a third guard inside setMode would be redundant, so leaving it as is.