storyforge 0.4.5 → 0.4.6

package/dist/index.js

@@ -5,7 +5,7 @@ import * as fs2 from "fs";
 import * as os2 from "os";
 import * as path2 from "path";
 import * as http from "http";
-import { execFile, exec as execCb, execSync } from "child_process";
+import { execFile, exec as execCb, execSync, spawn } from "child_process";
 import { promisify } from "util";
 
 // src/utils/log.ts
@@ -88,13 +88,31 @@ ${args.referenceScript.slice(0, 8e3)}
 === END SAMPLE ===
 
 Use this ONLY to calibrate the channel's voice \u2014 sentence rhythm, tolerance for technical detail, willingness to name real things. Do NOT copy its structure. If the sample opens with historical examples and this topic works better with a counter-intuitive question or a single unfolding metaphor, choose the format that serves the topic. The best script for THIS topic may look structurally different from the sample, and that's correct.` : "";
+  const guidanceBlock = (args.additionalGuidance ?? "").trim() ? `
+
+\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550
+VIEWER'S SPECIFIC ANGLE (RULE -2 \u2014 overrides everything below)
+\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550
+
+The person commissioning this script has spelled out the lens they
+want. This is NOT a hint. It is the deciding constraint when you pick
+structural format, dominant metaphor, named characters, antagonist,
+and the open-loop. If the LLM's default instinct conflicts with this
+guidance, the guidance wins.
+
+>>> ${args.additionalGuidance.trim().replace(/\n/g, "\n>>> ")}
+
+Before you commit to a format / metaphor / payoff, re-read the lens
+above. Every chunk should serve it. Reject any structural choice that
+doesn't.
+` : "";
   return `You are a senior documentary scriptwriter for ${channel} (${handle}). The channel's promise: "${tagline}".
 
 Your job is to write a script that is UNPUTDOWNABLE \u2014 emotionally engaging, viscerally specific, and dense with surprise. Every chunk must earn the next click. "Technically accurate but academic" = FAILURE. "Comprehensive but boring" = FAILURE. The bar is: would a smart 22-year-old skip this video at second 30, or would they keep watching at minute 18?
 
 TOPIC: "${args.topic}"
 STYLE: ${args.style} \u2014 ${args.styleGuide}
-TARGET LENGTH: ${minMin}-${maxMin} minutes (write to the topic's natural length \u2014 pad nothing, rush nothing).
+TARGET LENGTH: ${minMin}-${maxMin} minutes (write to the topic's natural length \u2014 pad nothing, rush nothing).${guidanceBlock}
 
 \u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550
 ENGAGEMENT BAR (this overrides everything below \u2014 RULE -1)
@@ -501,6 +519,46 @@ Now write the script.`;
 // src/commands/dev.ts
 var exec = promisify(execCb);
 var PORT = 4444;
+function runCliPipingStdin(cmd, args, stdinData, opts = {}) {
+  const maxBytes = (opts.maxBufferMB ?? 16) * 1024 * 1024;
+  return new Promise((resolve2, reject) => {
+    const proc = spawn(cmd, args, { stdio: ["pipe", "pipe", "pipe"] });
+    let stdout = "";
+    let stderr = "";
+    let total = 0;
+    const timer = opts.timeoutMs ? setTimeout(() => {
+      try {
+        proc.kill("SIGKILL");
+      } catch {
+      }
+      reject(new Error(`${cmd} timed out after ${opts.timeoutMs}ms`));
+    }, opts.timeoutMs) : null;
+    proc.stdout.on("data", (d) => {
+      total += d.length;
+      if (total > maxBytes) {
+        try {
+          proc.kill("SIGKILL");
+        } catch {
+        }
+        reject(new Error(`${cmd} stdout exceeded ${opts.maxBufferMB ?? 16}MB`));
+        return;
+      }
+      stdout += d.toString("utf-8");
+    });
+    proc.stderr.on("data", (d) => {
+      stderr += d.toString("utf-8");
+    });
+    proc.on("error", (err) => {
+      if (timer) clearTimeout(timer);
+      reject(err);
+    });
+    proc.on("close", (code) => {
+      if (timer) clearTimeout(timer);
+      resolve2({ stdout, stderr, code });
+    });
+    proc.stdin.end(stdinData);
+  });
+}
 var WEB_URL = "https://forge.algo-thinker.com";
 function getApiConfig() {
   const creds = loadCredentials();
@@ -765,6 +823,14 @@ async function devCommand(options) {
     const url = new URL(req.url || "/", `http://localhost:${port}`);
     const pathname = url.pathname;
     if (pathname === "/api/health") {
+      const probeCli = (binary) => {
+        try {
+          const out = execSync(`which ${binary}`, { stdio: ["ignore", "pipe", "ignore"], timeout: 2e3 }).toString().trim();
+          return { available: !!out, path: out || null };
+        } catch {
+          return { available: false, path: null };
+        }
+      };
       res.writeHead(200, { ...CORS_HEADERS, "Content-Type": "application/json" });
       res.end(JSON.stringify({
         status: "ok",
@@ -772,7 +838,14 @@ async function devCommand(options) {
         channelId: meta.channelId,
         title: meta.title,
         channelSlug: meta.channelSlug,
-        dir
+        dir,
+        // CLI bridge surface — mirrors the production VPS bridge's /health
+        // shape so the web UI can use the same code path for both.
+        bridge: {
+          source: "local-forge-dev",
+          claude: probeCli("claude"),
+          codex: probeCli("codex")
+        }
       }));
       return;
     }
@@ -935,35 +1008,41 @@ async function devCommand(options) {
           forbiddenVisuals: body.forbiddenVisuals,
           referenceScript: body.referenceScript,
           targetMinutesMin: body.targetMinutesMin,
-          targetMinutesMax: body.targetMinutesMax
+          targetMinutesMax: body.targetMinutesMax,
+          additionalGuidance: body.additionalGuidance
         });
-        const tmpFile = path2.join(os2.tmpdir(), `forge-script-${Date.now()}.txt`);
-        fs2.writeFileSync(tmpFile, prompt2, "utf-8");
-        const cleanup = () => {
-          if (fs2.existsSync(tmpFile)) fs2.unlinkSync(tmpFile);
-        };
-        const candidates = ["claude-opus-4-7", "claude-sonnet-4-6"];
+        const cli = body.cli === "codex" ? "codex" : "claude";
+        const candidates = cli === "codex" ? ["gpt-5.5", "gpt-5.4", "gpt-5.4-mini"] : ["claude-opus-4-7", "claude-sonnet-4-6"];
         let raw = "";
         let modelUsed = "";
         let lastErr;
         for (const model of candidates) {
+          if (!/^[a-z0-9.\-]+$/i.test(model)) {
+            lastErr = `model alias "${model}" rejected by validator`;
+            continue;
+          }
           try {
-            log.info(`[script-gen] Generating via Claude CLI \xB7 model=${model}`);
-            const { stdout } = await exec(
-              `cat "${tmpFile}" | claude -p --model ${model} --no-session-persistence`,
-              { maxBuffer: 16 * 1024 * 1024, timeout: 3e5 }
-            );
+            log.info(`[script-gen] Generating via ${cli} CLI \xB7 model=${model}`);
+            const args = cli === "codex" ? ["exec", "-", "--model", model] : ["-p", "--model", model, "--no-session-persistence"];
+            const { stdout, code, stderr } = await runCliPipingStdin(cli, args, prompt2, {
+              timeoutMs: 3e5,
+              maxBufferMB: 16
+            });
+            if (code !== 0) {
+              lastErr = `${cli}/${model} exit ${code}: ${stderr.slice(-300)}`;
+              log.warn(`[script-gen] ${lastErr.slice(0, 120)}`);
+              continue;
+            }
             raw = (stdout ?? "").trim();
             if (raw) {
-              modelUsed = model;
+              modelUsed = `${cli}:${model}`;
               break;
             }
           } catch (err) {
             lastErr = err instanceof Error ? err.message : String(err);
-            log.warn(`[script-gen] ${model} failed: ${lastErr.slice(0, 120)}`);
+            log.warn(`[script-gen] ${cli}/${model} failed: ${lastErr.slice(0, 120)}`);
           }
         }
-        cleanup();
         if (!raw) {
           res.writeHead(502, { ...CORS_HEADERS, "Content-Type": "application/json" });
           res.end(JSON.stringify({
@@ -1003,8 +1082,10 @@ async function devCommand(options) {
         res.end(JSON.stringify({
           chunks,
           model: modelUsed,
-          provider: "claude-cli",
-          total_sec: totalSec
+          provider: cli === "codex" ? "codex-cli" : "claude-cli",
+          cli,
+          total_sec: totalSec,
+          renderedPrompt: prompt2
         }));
       });
       req.on("error", () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "storyforge",
-  "version": "0.4.5",
+  "version": "0.4.6",
   "description": "StoryForge — local bridge for the Forge video production web app. Zero runtime dependencies.",
   "type": "module",
   "bin": {