unsnooze 1.14.3 → 1.14.4

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 1.14.4 — 2026-08-08
+
+- **Windows: the daemon PATH check split on the wrong separator.** The check
+  added in 1.14.3 split `PATH` on `:`, which is right on macOS and Linux and
+  wrong on Windows — there the separator is `;` and every absolute path
+  contains a colon (`C:\...`), so entries were shredded into fragments. The
+  predicate now uses the host's own delimiter. No behavior changed on macOS or
+  Linux, and Windows has no autostart unit for the check to reach, so this is a
+  correctness and test-portability fix rather than a user-visible one.
+
 ## 1.14.3 — 2026-08-08
 
 - **Upgrades now reach running watchers.** A per-pane monitor lives as long as

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "unsnooze",
-  "version": "1.14.3",
+  "version": "1.14.4",
   "description": "Automatically resume Claude Code, Codex CLI, Grok, Qwen Code, Kimi CLI, OpenCode, and Antigravity sessions when 5-hour or weekly usage limits reset—across tmux, Zellij, VS Code, and desktop apps.",
   "keywords": [
     "claude",

package/src/install.js

@@ -11,7 +11,7 @@
 import { readFileSync, writeFileSync, renameSync, existsSync, copyFileSync, rmSync, mkdirSync, unlinkSync } from 'node:fs';
 import { execFileSync } from 'node:child_process';
 import { homedir, userInfo } from 'node:os';
-import { join, dirname } from 'node:path';
+import { join, dirname, delimiter } from 'node:path';
 import { CLAUDE_SETTINGS, STATE_DIR } from './config.js';
 import { getConfig, configFileExists } from './settings.js';
 import { xmlEscape } from './notify.js';
@@ -187,9 +187,13 @@ export function autostartUnitPath({ platform = process.platform, dir = null } =
 // cannot revive anything — every call dies with ENOENT. Pure and synchronous
 // on purpose: this decides whether to rewrite-and-reload a unit, and that
 // decision must be deterministic.
-export function pathResolves(pathStr, bin) {
+// `sep` defaults to the host's PATH delimiter, not a hardcoded ':'. On Windows
+// that is ';' AND every absolute path contains a colon (C:\...), so splitting
+// on ':' shreds the entries into nonsense. The units we inspect are always
+// POSIX, but this predicate has to give an honest answer wherever it runs.
+export function pathResolves(pathStr, bin, { sep = delimiter } = {}) {
   if (typeof pathStr !== 'string' || pathStr === '') return false;
-  return pathStr.split(':').some(d => d && existsSync(join(d, bin)));
+  return pathStr.split(sep).some(d => d && existsSync(join(d, bin)));
 }
 
 function xmlUnescape(s) {