patchwork-os 0.2.0-alpha.33 → 0.2.0-alpha.35

package/dist/index.js

@@ -3,10 +3,15 @@
 // Uses Node 20.6+ native dotenv loader; falls back to manual parse for older Node.
 {
     const { fileURLToPath: _fileURLToPath } = await import("node:url");
-    const envPath = _fileURLToPath(new URL("../.env", import.meta.url));
     try {
         const { readFileSync, existsSync } = await import("node:fs");
-        if (existsSync(envPath)) {
+        // Try both "../.env" (compiled dist/) and ".env" (tsx src/ dev run)
+        const candidates = [
+            _fileURLToPath(new URL("../.env", import.meta.url)),
+            _fileURLToPath(new URL(".env", import.meta.url)),
+        ];
+        const envPath = candidates.find(existsSync);
+        if (envPath) {
             for (const line of readFileSync(envPath, "utf-8").split("\n")) {
                 const m = /^([A-Z_][A-Z0-9_]*)=(.*)$/.exec(line.trim());
                 if (m?.[1] && !process.env[m[1]]) {
@@ -87,6 +92,59 @@ async function downloadVsixFromOpenVsx() {
     writeFileSync(tmpPath, Buffer.from(buf));
     return tmpPath;
 }
+// Closes the race where bridge.start() began initialising in parallel with
+// a subcommand's async work — observed in the 2026-04-29 dogfood pass
+// where `recipe install` errors interleaved with bridge "Tools: full"
+// startup logs.
+//
+// Every subcommand `if`-block below dispatches via an `(async () => {...})()`
+// IIFE that ends with `process.exit`. The IIFE invocation returns
+// synchronously, so without this gate, control immediately falls through
+// to the bridge.start() block at end-of-file and starts initialising
+// alongside the subcommand's async work. process.exit fires *eventually*
+// after the await chain, but the bridge has already begun in parallel.
+// Two IIFEs (patchwork no-args dashboard, recipe watch) lack process.exit
+// entirely — without this gate they would run alongside the bridge
+// indefinitely.
+//
+// Single source of truth for "is this argv invoking a subcommand?" — the
+// same list is also used by the unknown-command suggester at L2570.
+const KNOWN_SUBCOMMANDS = [
+    "init",
+    "patchwork-init",
+    "start-all",
+    "install-extension",
+    "gen-claude-md",
+    "print-token",
+    "gen-plugin-stub",
+    "notify",
+    "install",
+    "marketplace",
+    "status",
+    "shim",
+    "recipe",
+    "dashboard",
+    "launchd",
+];
+const __invokedSubcommand = (() => {
+    const sub = process.argv[2];
+    if (!sub || sub.startsWith("-"))
+        return null;
+    // Treat KNOWN_SUBCOMMANDS as the dispatch source. The bare-binary
+    // dashboard launcher (no argv) is handled separately below.
+    return KNOWN_SUBCOMMANDS.includes(sub)
+        ? sub
+        : null;
+})();
+const __invokedBareBinaryDashboard = (() => {
+    if (process.argv[2])
+        return false;
+    const binName = path.basename(process.argv[1] ?? "");
+    return (binName === "patchwork-os" ||
+        binName === "patchwork" ||
+        binName === "patchwork.js");
+})();
+const __subcommandWillRun = __invokedSubcommand !== null || __invokedBareBinaryDashboard;
 // Handle --version flag — print package version and exit.
 if (process.argv[2] === "--version" || process.argv[2] === "-v") {
     console.log(`claude-ide-bridge ${PACKAGE_VERSION}`);
@@ -592,6 +650,68 @@ if (process.argv[2] === "recipe" && process.argv[3] === "list") {
         process.exit(0);
     })();
 }
+// Patchwork: `patchwork recipe enable <name>` / `recipe disable <name>` —
+// flip the disabled marker so scheduled triggers (cron/file-watch) take
+// effect (or stop). Manual `recipe run` is unaffected.
+if (process.argv[2] === "recipe" &&
+    (process.argv[3] === "enable" || process.argv[3] === "disable")) {
+    const subcommand = process.argv[3];
+    const name = process.argv[4];
+    if (!name) {
+        process.stderr.write(`Usage: patchwork recipe ${subcommand} <name>\n` +
+            `  See \`patchwork recipe list\` for installed recipe names.\n`);
+        process.exit(1);
+    }
+    (async () => {
+        try {
+            const { runRecipeEnable, runRecipeDisable } = await import("./commands/recipeInstall.js");
+            if (subcommand === "enable") {
+                const r = runRecipeEnable(name);
+                process.stdout.write(r.alreadyEnabled
+                    ? `  ℹ ${r.name} is already enabled\n`
+                    : `  ✓ enabled ${r.name}\n`);
+            }
+            else {
+                const r = runRecipeDisable(name);
+                process.stdout.write(r.alreadyDisabled
+                    ? `  ℹ ${r.name} is already disabled\n`
+                    : `  ✓ disabled ${r.name}\n`);
+            }
+            process.exit(0);
+        }
+        catch (err) {
+            process.stderr.write(`Error: ${err instanceof Error ? err.message : String(err)}\n`);
+            process.exit(1);
+        }
+    })();
+}
+// Patchwork: `patchwork recipe uninstall <name>` — remove an installed recipe
+// directory and all its files. Sister to `recipe install`. Idempotent on
+// success (subsequent uninstalls error with "no installed recipe").
+if (process.argv[2] === "recipe" && process.argv[3] === "uninstall") {
+    const name = process.argv[4];
+    if (!name) {
+        process.stderr.write("Usage: patchwork recipe uninstall <name>\n" +
+            "  See `patchwork recipe list` for installed recipe names.\n");
+        process.exit(1);
+    }
+    (async () => {
+        try {
+            const { runRecipeUninstall } = await import("./commands/recipeInstall.js");
+            const r = runRecipeUninstall(name);
+            if (!r.ok) {
+                process.stderr.write(`Error: ${r.error}\n`);
+                process.exit(1);
+            }
+            process.stdout.write(`  ✓ Uninstalled ${name} (${r.installDir})\n`);
+            process.exit(0);
+        }
+        catch (err) {
+            process.stderr.write(`Error: ${err instanceof Error ? err.message : String(err)}\n`);
+            process.exit(1);
+        }
+    })();
+}
 // Patchwork: `patchwork recipe run <name>` — runs a recipe locally or via
 // a running bridge's /recipes/run endpoint if one is available.
 if (process.argv[2] === "recipe" && process.argv[3] === "run") {
@@ -831,7 +951,10 @@ if (process.argv[2] === "recipe" && process.argv[3] === "new") {
     const args = process.argv.slice(4);
     const recipeName = args[0];
     if (!recipeName) {
-        process.stderr.write("Usage: patchwork recipe new <name> [--template <name>] [--desc <description>]\n");
+        process.stderr.write("Usage: patchwork recipe new <name> [--template <name>] [--desc <description>] [--out <dir>]\n" +
+            "  --out <dir>  Write the recipe to <dir>/<name>.yaml.\n" +
+            "               Defaults to ~/.patchwork/recipes/ — pass `--out .` to\n" +
+            "               write into the current directory instead.\n");
         process.stderr.write("\nTemplates:\n");
         (async () => {
             const { listTemplates } = await import("./commands/recipe.js");
@@ -850,10 +973,16 @@ if (process.argv[2] === "recipe" && process.argv[3] === "new") {
                 const descIdx = args.indexOf("--desc");
                 const description = (descIdx >= 0 ? args[descIdx + 1] : undefined) ??
                     `Recipe: ${recipeName}`;
+                const outIdx = args.indexOf("--out");
+                const outRaw = outIdx >= 0 ? args[outIdx + 1] : undefined;
+                // `--out .` is the common case for "scaffold in cwd" — resolve so
+                // the success message shows the absolute path the user can open.
+                const outputDir = outRaw ? path.resolve(outRaw) : undefined;
                 const result = runNew({
                     name: recipeName,
                     description,
                     ...(template ? { template } : {}),
+                    ...(outputDir ? { outputDir } : {}),
                 });
                 process.stdout.write(`  ✓ Created ${result.path}\n`);
                 process.exit(0);
@@ -1957,27 +2086,12 @@ if (process.argv[2] === "launchd") {
     }
 }
 {
-    const KNOWN_COMMANDS = [
-        "init",
-        "patchwork-init",
-        "start-all",
-        "install-extension",
-        "gen-claude-md",
-        "print-token",
-        "gen-plugin-stub",
-        "notify",
-        "install",
-        "marketplace",
-        "status",
-        "shim",
-        "recipe",
-        "dashboard",
-        "launchd",
-    ];
+    // Reuses the KNOWN_SUBCOMMANDS list from the top of this file as a single
+    // source of truth for "what subcommand argv tokens are recognized".
     const unknownSub = process.argv[2];
     if (unknownSub &&
         !unknownSub.startsWith("-") &&
-        !KNOWN_COMMANDS.includes(unknownSub)) {
+        !KNOWN_SUBCOMMANDS.includes(unknownSub)) {
         const lev = (a, b) => {
             const dp = Array.from({ length: a.length + 1 }, (_, i) => Array.from({ length: b.length + 1 }, (_, j) => i === 0 ? j : j === 0 ? i : 0));
             for (let i = 1; i <= a.length; i++)
@@ -1993,127 +2107,146 @@ if (process.argv[2] === "launchd") {
             // biome-ignore lint/style/noNonNullAssertion: dp is fully pre-allocated
             return dp[a.length][b.length];
         };
-        const closest = [...KNOWN_COMMANDS].sort((a, b) => lev(unknownSub, a) - lev(unknownSub, b))[0];
+        const closest = [...KNOWN_SUBCOMMANDS].sort((a, b) => lev(unknownSub, a) - lev(unknownSub, b))[0];
         console.error(`Unknown command: '${unknownSub}'. Did you mean: ${closest}?`);
         process.exit(1);
     }
 }
-const config = parseConfig(process.argv);
-// Patchwork: resolve --model flag (optional, non-invasive) — stashes the
-// configured adapter on globalThis for consumers that opt into the adapter
-// layer. Bridge subprocess driver still works when --model is absent.
-try {
-    const { resolveModel } = await import("./patchworkCli.js");
-    const resolved = resolveModel(process.argv);
-    if (resolved) {
-        globalThis.__patchworkAdapter =
-            resolved.adapter;
-        process.stderr.write(`[patchwork] model adapter initialized: ${resolved.adapter.name}\n`);
-    }
-}
-catch (err) {
-    process.stderr.write(`[patchwork] adapter init failed: ${err instanceof Error ? err.message : String(err)}\n`);
+// Skip the bridge-mode tail entirely when a subcommand IIFE will own the
+// process. `parseConfig` validates argv against the bridge's known-flag list
+// and raises "Unknown option" for subcommand-specific flags (e.g. `recipe
+// new --out .`); without this guard that throw kills the process before
+// the IIFE's microtask runs. The subcommand handles its own arg parsing.
+if (__subcommandWillRun) {
+    // Subcommand IIFE is in flight or about to fire; sit tight until it
+    // process.exits. Empty body — control naturally falls past end-of-file
+    // and Node keeps the process alive on the IIFE's pending microtask.
 }
-// If --analytics flag was passed, persist the preference immediately
-if (config.analyticsEnabled !== null) {
-    setAnalyticsPref(config.analyticsEnabled);
-}
-// Auto-tmux: if requested and not already inside tmux or screen, re-exec inside a tmux session
-if (config.autoTmux &&
-    !process.env.TMUX &&
-    !process.env.STY &&
-    !process.env.ZELLIJ &&
-    !process.env.ZELLIJ_SESSION_NAME) {
-    const ws = config.workspace.replace(/[^a-zA-Z0-9]/g, "").slice(-8);
-    const hash = crypto
-        .createHash("sha256")
-        .update(config.workspace)
-        .digest("hex")
-        .slice(0, 6);
-    const sessionName = `claude-bridge-${ws}${hash}`;
-    // Check if tmux is available
-    const tmuxCheck = spawnSync("which", ["tmux"], { stdio: "ignore" });
-    if (tmuxCheck.status !== 0) {
-        process.stderr.write("WARNING: --auto-tmux requested but tmux is not installed. Running without tmux.\n");
-    }
-    else {
-        // Strip --auto-tmux from argv to avoid infinite re-exec loop
-        const newArgv = process.argv.filter((a) => a !== "--auto-tmux");
-        // Pass each argv token as a separate tmux argument so paths with spaces work correctly
-        const result = spawnSync("tmux", ["new-session", "-d", "-s", sessionName, ...newArgv], { stdio: "inherit", timeout: 5000 });
-        if (result.status === 0) {
-            process.stderr.write(`Bridge launched in tmux session '${sessionName}'.\n`);
-            process.stderr.write(`  Attach with: tmux attach -t ${sessionName}\n`);
-            process.exit(0);
-        }
-        else {
-            // tmux session likely already exists — attach to it or fall through
-            process.stderr.write(`WARNING: Could not create tmux session '${sessionName}' (already exists?). Running without auto-tmux.\n`);
+else {
+    const config = parseConfig(process.argv);
+    // Patchwork: resolve --model flag (optional, non-invasive) — stashes the
+    // configured adapter on globalThis for consumers that opt into the adapter
+    // layer. Bridge subprocess driver still works when --model is absent.
+    try {
+        const { resolveModel } = await import("./patchworkCli.js");
+        const resolved = resolveModel(process.argv);
+        if (resolved) {
+            globalThis.__patchworkAdapter =
+                resolved.adapter;
+            process.stderr.write(`[patchwork] model adapter initialized: ${resolved.adapter.name}\n`);
         }
     }
-}
-// --watch: supervisor mode — spawn this binary as a child (without --watch) and restart on crash
-if (config.watch) {
-    const childArgv = process.argv.filter((a) => a !== "--watch");
-    const STABLE_THRESHOLD_MS = 60_000;
-    const BASE_DELAY_MS = 2_000;
-    const MAX_DELAY_MS = 30_000;
-    let delay = BASE_DELAY_MS;
-    let stopping = false;
-    function runChild() {
-        if (stopping)
-            return;
-        const startAt = Date.now();
-        process.stderr.write("[supervisor] starting bridge\n");
-        const [cmd, ...args] = childArgv;
-        if (!cmd)
-            return;
-        const child = spawn(cmd, args, {
-            stdio: "inherit",
-        });
-        for (const sig of ["SIGTERM", "SIGINT"]) {
-            process.once(sig, () => {
-                stopping = true;
-                child.kill(sig);
-            });
+    catch (err) {
+        process.stderr.write(`[patchwork] adapter init failed: ${err instanceof Error ? err.message : String(err)}\n`);
+    }
+    // If --analytics flag was passed, persist the preference immediately
+    if (config.analyticsEnabled !== null) {
+        setAnalyticsPref(config.analyticsEnabled);
+    }
+    // Auto-tmux: if requested and not already inside tmux or screen, re-exec inside a tmux session
+    if (config.autoTmux &&
+        !process.env.TMUX &&
+        !process.env.STY &&
+        !process.env.ZELLIJ &&
+        !process.env.ZELLIJ_SESSION_NAME) {
+        const ws = config.workspace.replace(/[^a-zA-Z0-9]/g, "").slice(-8);
+        const hash = crypto
+            .createHash("sha256")
+            .update(config.workspace)
+            .digest("hex")
+            .slice(0, 6);
+        const sessionName = `claude-bridge-${ws}${hash}`;
+        // Check if tmux is available
+        const tmuxCheck = spawnSync("which", ["tmux"], { stdio: "ignore" });
+        if (tmuxCheck.status !== 0) {
+            process.stderr.write("WARNING: --auto-tmux requested but tmux is not installed. Running without tmux.\n");
         }
-        child.on("exit", (code, signal) => {
-            if (stopping) {
-                process.stderr.write("[supervisor] bridge stopped\n");
+        else {
+            // Strip --auto-tmux from argv to avoid infinite re-exec loop
+            const newArgv = process.argv.filter((a) => a !== "--auto-tmux");
+            // Pass each argv token as a separate tmux argument so paths with spaces work correctly
+            const result = spawnSync("tmux", ["new-session", "-d", "-s", sessionName, ...newArgv], { stdio: "inherit", timeout: 5000 });
+            if (result.status === 0) {
+                process.stderr.write(`Bridge launched in tmux session '${sessionName}'.\n`);
+                process.stderr.write(`  Attach with: tmux attach -t ${sessionName}\n`);
                 process.exit(0);
             }
-            const uptime = Date.now() - startAt;
-            if (uptime >= STABLE_THRESHOLD_MS) {
-                delay = BASE_DELAY_MS; // reset backoff after a stable run
+            else {
+                // tmux session likely already exists — attach to it or fall through
+                process.stderr.write(`WARNING: Could not create tmux session '${sessionName}' (already exists?). Running without auto-tmux.\n`);
             }
-            process.stderr.write(`[supervisor] bridge exited (code=${code ?? signal}), restarting in ${delay / 1000}s\n`);
-            setTimeout(() => {
-                delay = Math.min(delay * 2, MAX_DELAY_MS);
-                runChild();
-            }, delay);
-        });
+        }
     }
-    runChild();
-}
-else {
-    const bridge = new Bridge(config);
-    bridge.start().catch((err) => {
-        const message = err instanceof Error ? err.message : String(err);
-        process.stderr.write(`Error: ${message}\n`);
-        process.exit(1);
-    });
-    // F5: Silent self-update nudge (fire-and-forget)
-    import("node:child_process")
-        .then(({ exec }) => {
-        exec("npm view claude-ide-bridge version", { timeout: 5000 }, (err, stdout) => {
-            if (err || !stdout)
+    // Skip bridge boot when a subcommand IIFE is doing the work — avoids the
+    // race where bridge.start() began initialising in parallel with the
+    // subcommand's async path. See the KNOWN_SUBCOMMANDS / __subcommandWillRun
+    // gate at the top of this file.
+    if (__subcommandWillRun) {
+        // intentionally empty — subcommand IIFE owns the process from here.
+    }
+    // --watch: supervisor mode — spawn this binary as a child (without --watch) and restart on crash
+    else if (config.watch) {
+        const childArgv = process.argv.filter((a) => a !== "--watch");
+        const STABLE_THRESHOLD_MS = 60_000;
+        const BASE_DELAY_MS = 2_000;
+        const MAX_DELAY_MS = 30_000;
+        let delay = BASE_DELAY_MS;
+        let stopping = false;
+        function runChild() {
+            if (stopping)
+                return;
+            const startAt = Date.now();
+            process.stderr.write("[supervisor] starting bridge\n");
+            const [cmd, ...args] = childArgv;
+            if (!cmd)
                 return;
-            const latest = stdout.trim();
-            if (latest && semverGt(latest, PACKAGE_VERSION)) {
-                console.log(`\n  Bridge v${latest} available — run: npm update -g claude-ide-bridge\n`);
+            const child = spawn(cmd, args, {
+                stdio: "inherit",
+            });
+            for (const sig of ["SIGTERM", "SIGINT"]) {
+                process.once(sig, () => {
+                    stopping = true;
+                    child.kill(sig);
+                });
             }
+            child.on("exit", (code, signal) => {
+                if (stopping) {
+                    process.stderr.write("[supervisor] bridge stopped\n");
+                    process.exit(0);
+                }
+                const uptime = Date.now() - startAt;
+                if (uptime >= STABLE_THRESHOLD_MS) {
+                    delay = BASE_DELAY_MS; // reset backoff after a stable run
+                }
+                process.stderr.write(`[supervisor] bridge exited (code=${code ?? signal}), restarting in ${delay / 1000}s\n`);
+                setTimeout(() => {
+                    delay = Math.min(delay * 2, MAX_DELAY_MS);
+                    runChild();
+                }, delay);
+            });
+        }
+        runChild();
+    }
+    else {
+        const bridge = new Bridge(config);
+        bridge.start().catch((err) => {
+            const message = err instanceof Error ? err.message : String(err);
+            process.stderr.write(`Error: ${message}\n`);
+            process.exit(1);
         });
-    })
-        .catch(() => { });
-}
+        // F5: Silent self-update nudge (fire-and-forget)
+        import("node:child_process")
+            .then(({ exec }) => {
+            exec("npm view claude-ide-bridge version", { timeout: 5000 }, (err, stdout) => {
+                if (err || !stdout)
+                    return;
+                const latest = stdout.trim();
+                if (latest && semverGt(latest, PACKAGE_VERSION)) {
+                    console.log(`\n  Bridge v${latest} available — run: npm update -g claude-ide-bridge\n`);
+                }
+            });
+        })
+            .catch(() => { });
+    }
+} // end of `else` for `if (__subcommandWillRun)` (bridge-mode block)
 //# sourceMappingURL=index.js.map