scenri 0.3.3 → 0.3.5

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## [0.3.5](https://github.com/tonygorb/Scenri/compare/v0.3.4...v0.3.5) (2026-08-20)
+
+
+### Bug Fixes
+
+* keep untrusted text out of the Windows shell lines ([f564c35](https://github.com/tonygorb/Scenri/commit/f564c358dc018f2e7e52e00eae004122690dc2be))
+* make the Codex engine and updates work on Windows ([f94f420](https://github.com/tonygorb/Scenri/commit/f94f42069049b65920dfc95491a074cfd3617b8f))
+* make the Codex engine and updates work on Windows ([f21cb9d](https://github.com/tonygorb/Scenri/commit/f21cb9d513f291bfc64585f9f3a3f67ea5738bde))
+
+
+### Performance Improvements
+
+* cut the e2e wall clock roughly in half ([2cacbce](https://github.com/tonygorb/Scenri/commit/2cacbced34b6c58541c0b7942cd107aaa234f477))
+
+## [0.3.4](https://github.com/tonygorb/Scenri/compare/v0.3.3...v0.3.4) (2026-08-20)
+
+
+### Bug Fixes
+
+* presenter references are identity, not wardrobe ([dc372fd](https://github.com/tonygorb/Scenri/commit/dc372fd40d7e15d7d8dc72b5ec8f41993e078ae9))
+* presenter references are identity, not wardrobe ([c8bca43](https://github.com/tonygorb/Scenri/commit/c8bca431730a2d4c41d2783fb6eba2c564af5141))
+* stop custom presenter and scene pages freezing the whole app ([fcf88dd](https://github.com/tonygorb/Scenri/commit/fcf88ddbcd57d01354c833cf9f8456fa1b0b0434))
+* stop custom presenter and scene pages freezing the whole app ([8595078](https://github.com/tonygorb/Scenri/commit/859507893892cb23a7ff2f132f273db4dbd0a0f9))
+
 ## [0.3.3](https://github.com/tonygorb/Scenri/compare/v0.3.2...v0.3.3) (2026-08-20)
 
 

package/dist/{chunk-GA7P7K2M.js → chunk-3W2RHBZC.js}

@@ -1,6 +1,6 @@
 import { stagingDir, versionsDir, isValidVersionDir, pruneStaged, entryOf } from './chunk-4MAFHYAD.js';
 import { spawnSync, spawn } from 'child_process';
-import { rmSync, mkdirSync, readFileSync, renameSync } from 'fs';
+import { existsSync, rmSync, mkdirSync, readFileSync, renameSync } from 'fs';
 import { join, dirname } from 'path';
 
 // src/update/check.ts
@@ -97,6 +97,7 @@ function createUpdateChecker(deps) {
 }
 function findNpm(opts = {}) {
   const env = opts.env ?? process.env;
+  const platform = opts.platform ?? process.platform;
   const canRun = opts.canRun ?? ((argv) => {
     try {
       return spawnSync(argv[0], [...argv.slice(1), "--version"], { stdio: "ignore", timeout: 1e4 }).status === 0;
@@ -104,9 +105,26 @@ function findNpm(opts = {}) {
       return false;
     }
   });
-  if (canRun(["npm"])) return ["npm"];
-  const ep = env.npm_execpath;
+  if (platform !== "win32" && canRun(["npm"])) return ["npm"];
+  const ep = env.npm_execpath?.replace(/npx-cli\.js$/, "npm-cli.js");
   if (ep && /npm-cli\.js$/.test(ep) && canRun([process.execPath, ep])) return [process.execPath, ep];
+  if (platform === "win32") {
+    const cli = windowsNpmCli();
+    if (cli && canRun([process.execPath, cli])) return [process.execPath, cli];
+  }
+  return null;
+}
+function windowsNpmCli() {
+  try {
+    const out = spawnSync("where", ["npm"], { encoding: "utf8", timeout: 1e4 });
+    if (out.status !== 0 || !out.stdout) return null;
+    for (const line of out.stdout.split(/\r?\n/)) {
+      if (!/npm(\.cmd)?$/i.test(line.trim())) continue;
+      const cli = join(dirname(line.trim()), "node_modules", "npm", "bin", "npm-cli.js");
+      if (existsSync(cli)) return cli;
+    }
+  } catch {
+  }
   return null;
 }
 function run(argv) {
@@ -202,5 +220,5 @@ async function stageVersion(deps) {
 }
 
 export { classify, createUpdateChecker, fetchDistTagLatest, findNpm, stageVersion };
-//# sourceMappingURL=chunk-GA7P7K2M.js.map
-//# sourceMappingURL=chunk-GA7P7K2M.js.map
+//# sourceMappingURL=chunk-3W2RHBZC.js.map
+//# sourceMappingURL=chunk-3W2RHBZC.js.map

package/dist/{cli-GQLRYYGU.js → cli-HF5KPURA.js}

@@ -1,4 +1,4 @@
-import { stageVersion, fetchDistTagLatest, findNpm } from './chunk-GA7P7K2M.js';
+import { stageVersion, fetchDistTagLatest, findNpm } from './chunk-3W2RHBZC.js';
 import { readMeta } from './chunk-Y3ZPBPLP.js';
 import { defaultHome, newestStaged, compareSemver } from './chunk-4MAFHYAD.js';
 
@@ -43,7 +43,8 @@ async function runUpdateCommand(opts) {
   }
   if (!findNpm()) {
     console.error("  npm is not reachable from here, so nothing can be downloaded.");
-    console.error("  Install Node.js with npm, or run: npx scenri@latest");
+    console.error("  Run the newest version directly with: npx scenri@latest");
+    console.error("  (or install Node.js with npm and try again)");
     return 1;
   }
   console.log(`  downloading ${meta.name} ${latest}\u2026`);
@@ -62,5 +63,5 @@ async function runUpdateCommand(opts) {
 }
 
 export { runUpdateCommand };
-//# sourceMappingURL=cli-GQLRYYGU.js.map
-//# sourceMappingURL=cli-GQLRYYGU.js.map
+//# sourceMappingURL=cli-HF5KPURA.js.map
+//# sourceMappingURL=cli-HF5KPURA.js.map

package/dist/index.js

@@ -64,7 +64,7 @@ try {
       await (await import('./serve.js')).serve();
       break;
     case "update": {
-      const { runUpdateCommand } = await import('./cli-GQLRYYGU.js');
+      const { runUpdateCommand } = await import('./cli-HF5KPURA.js');
       process.exit(await runUpdateCommand(command));
       break;
     }

package/dist/serve.js

@@ -1,5 +1,5 @@
 import { portBusyLines } from './chunk-WGIZJXNE.js';
-import { createUpdateChecker, classify, stageVersion, findNpm } from './chunk-GA7P7K2M.js';
+import { createUpdateChecker, classify, stageVersion, findNpm } from './chunk-3W2RHBZC.js';
 import { readMeta, repoSlug } from './chunk-Y3ZPBPLP.js';
 import { newestStaged, compareSemver } from './chunk-4MAFHYAD.js';
 import { networkInterfaces, homedir, tmpdir } from 'os';
@@ -1271,7 +1271,7 @@ function createCatalogStore(db) {
 // ../core/src/engine.ts
 var REFERENCE_ROLE_DIRECTIVE = {
   product: "the exact product \u2014 preserve its label, shape, colors and design faithfully; do not redesign it",
-  character: "the exact person \u2014 preserve their face, hair and build faithfully; do not restyle them",
+  character: "the exact person \u2014 match their face, facial structure, skin, hair and build exactly; their clothing, pose and background are capture context, not styling to reproduce",
   brand: "the brand's own mark \u2014 if the direction calls for the mark to appear, reproduce it exactly as drawn, same colours, letterforms and proportions; otherwise take only its colour and treatment, and never its subject, geometry or composition",
   scene: "a reference for the environment and light only \u2014 take no subject, product or person from it",
   composition: "a reference for framing, camera angle and pose only \u2014 take no subject, color, material or branding from it",
@@ -1280,7 +1280,7 @@ var REFERENCE_ROLE_DIRECTIVE = {
 };
 var EDIT_REFERENCE_ROLE_DIRECTIVE = {
   product: "the exact product: keep or restore its label, shape and design faithfully",
-  character: "the exact person: keep their face, hair and build faithfully",
+  character: "the exact person: keep their face, facial structure, skin, hair and build faithfully; take no clothing, pose or background from this reference, and keep the source image's existing outfit unless the instruction changes it",
   brand: "the brand's own mark: reproduce it exactly as drawn wherever it appears, never redrawn or re-lettered",
   scene: "a reference for environment and light only",
   composition: "a reference for framing and pose only",
@@ -1857,11 +1857,17 @@ function execArgs(dir, promptText, effort = "low") {
 function createRunner(opts = {}) {
   const spawnImpl = opts.spawnImpl ?? spawn;
   const timeoutMs = opts.timeoutMs ?? DEFAULT_TIMEOUT_MS2;
+  const platform = opts.platform ?? process.platform;
+  const winArg = (a) => `"${a.replace(/[\r\n]+/g, " ").replace(/"/g, "'").replace(/%/g, " percent ")}"`;
+  const spawnCodex = (args) => platform === "win32" ? spawnImpl(["codex", ...args].map(winArg).join(" "), [], {
+    stdio: ["ignore", "pipe", "pipe"],
+    shell: true
+  }) : spawnImpl("codex", args, { stdio: ["ignore", "pipe", "pipe"] });
   function run2(args, signal) {
     return new Promise((resolve, reject) => {
       let child;
       try {
-        child = spawnImpl("codex", args, { stdio: ["ignore", "pipe", "pipe"] });
+        child = spawnCodex(args);
       } catch (err) {
         reject(new Error(`Failed to spawn codex: ${err.message}`));
         return;
@@ -1930,7 +1936,7 @@ function createRunner(opts = {}) {
       };
       let child;
       try {
-        child = spawnImpl("codex", args, { stdio: ["ignore", "pipe", "pipe"] });
+        child = spawnCodex(args);
       } catch {
         done(false);
         return;
@@ -1944,6 +1950,9 @@ function createRunner(opts = {}) {
     });
   }
   async function probe() {
+    if (process.env.SCENRI_NO_CODEX === "1") {
+      return { ok: false, reason: NOT_INSTALLED_REASON, code: "not-installed" };
+    }
     if (!await exitedZero(["--version"])) {
       return { ok: false, reason: NOT_INSTALLED_REASON, code: "not-installed" };
     }
@@ -2105,6 +2114,7 @@ function stateFrom(avail) {
 }
 function createCodexSetup(opts = {}) {
   const spawnImpl = opts.spawnImpl ?? spawn;
+  const platform = opts.platform ?? process.platform;
   const runner = createRunner(opts);
   const installTimeoutMs = opts.installTimeoutMs ?? DEFAULT_INSTALL_TIMEOUT_MS;
   function run2(cmd, args, timeoutMs) {
@@ -2123,7 +2133,7 @@ function createCodexSetup(opts = {}) {
         done({ code: null, stderr, spawnError: `${cmd} timed out after ${timeoutMs}ms` });
       }, timeoutMs);
       try {
-        child = spawnImpl(cmd, args, { stdio: ["ignore", "pipe", "pipe"], shell: process.platform === "win32" });
+        child = spawnImpl(cmd, args, { stdio: ["ignore", "pipe", "pipe"], shell: platform === "win32" });
       } catch (err) {
         done({ code: null, stderr, spawnError: err.message });
         return;
@@ -2156,7 +2166,7 @@ function createCodexSetup(opts = {}) {
         }
         return { ok: true };
       }
-      if (process.platform !== "win32" && /EACCES|permission denied/i.test(res.stderr)) {
+      if (platform !== "win32" && /EACCES|permission denied/i.test(res.stderr)) {
         return {
           ok: false,
           fallbackCommand: INSTALL_COMMAND_SUDO,
@@ -2487,7 +2497,14 @@ async function resolvePresenterImages(core, templatesRoot, presenter) {
     const hash = core.images.save(png);
     shots.push({ file: `asset:${hash}`, angle, locked: true });
   }
-  return shots.length ? { id: presenter.id, name: presenter.name, shots } : null;
+  if (!shots.length) return null;
+  return {
+    id: presenter.id,
+    name: presenter.name,
+    ...presenter.identityNotes ? { identityNotes: presenter.identityNotes } : {},
+    ...presenter.negativeConstraints?.length ? { negativeConstraints: presenter.negativeConstraints } : {},
+    shots
+  };
 }
 async function brandJsonWithResolvedPresenters(core, templatesRoot, presenters, brandJson, tokens) {
   const existing = brandJson?.characters ?? [];
@@ -2793,7 +2810,7 @@ function compileBrief(brief, ctx) {
             attachments.push({ role: "character", id: c.id, label: c.name, hash: chash, essential: i === 0 });
           });
           personDirectives.push(
-            "The attached person reference is the same person every time: hold their face, hair and build, and do not restyle them."
+            "The attached person reference is the same person every time: match their face, facial structure, skin, hair and build exactly. Their outfit, pose, background and lighting are neutral studio capture conditions, not styling direction: dress and style them for this shot, to a commercial standard, following any wardrobe the direction itself specifies."
           );
           if (c.identityNotes) personDirectives.push(String(c.identityNotes));
           if (c.negativeConstraints?.length)
@@ -2891,10 +2908,14 @@ function compileBrief(brief, ctx) {
   const sceneCamera = inlineTemplates[0]?.camera?.trim() || ctx.template?.camera?.trim() || "";
   const cameraDirectives = sceneCamera && !shotSpecifiesCamera(sentence) ? [`Camera for this shot: ${sceneCamera}`] : [];
   const guard = scene ? sceneGuardDirectives({ hasProduct: !!productId, hasPerson }) : [];
+  const pairDirectives = productId && hasPerson ? [
+    "If the attached product is something a person wears, the presenter wears that exact product, with the rest of the outfit styled around it; otherwise the presenter presents or uses the product naturally."
+  ] : [];
   const brandLines = brandRuleDirectives(ctx.brand);
   const allDirectives = [
     ...productDirectives,
     ...personDirectives,
+    ...pairDirectives,
     ...otherDirectives,
     ...cameraDirectives,
     ...brandLines,
@@ -6524,6 +6545,30 @@ function registerImageRoutes(app, deps) {
 
 // src/release/notes.data.ts
 var RELEASES = [
+  {
+    version: "0.3.5",
+    date: "2026-08-21",
+    sections: [
+      {
+        heading: "Fixes",
+        body: "Setting up the Codex engine on Windows no longer reports a successful install as missing, and updates can now find npm there. When npm truly is unreachable, the update command points to the one-line recovery instead of a dead end."
+      }
+    ]
+  },
+  {
+    version: "0.3.4",
+    date: "2026-08-21",
+    sections: [
+      {
+        heading: "Presenters",
+        body: "A presenter's reference photos now define who they are, not what they wear. The neutral studio outfit from their reference set no longer follows them into finished shots; wardrobe comes from the shot, the product and the scene instead, and refining an image keeps the outfit it already has."
+      },
+      {
+        heading: "Fixes",
+        body: "Opening a presenter or scene you built yourself no longer freezes the app."
+      }
+    ]
+  },
   {
     version: "0.3.3",
     date: "2026-08-20",
@@ -6867,7 +6912,7 @@ function registerSystemRoutes(app, deps) {
     }
     core.close();
     for (const name of ["scenri.db", "scenri.db-wal", "scenri.db-shm", "images", "backups"]) {
-      rmSync(join(core.home, name), { recursive: true, force: true });
+      rmSync(join(core.home, name), { recursive: true, force: true, maxRetries: 5, retryDelay: 100 });
     }
     return { ok: true, scope };
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scenri",
-  "version": "0.3.3",
+  "version": "0.3.5",
   "author": "Tony Gorb <hello@scenri.co>",
   "license": "AGPL-3.0-only",
   "type": "module",