@navipartner/demo-video 0.5.0 → 0.6.0

package/bin/pick.mjs

@@ -1,27 +1,23 @@
-import fs from "node:fs";
 import path from "node:path";
-import { fileURLToPath } from "node:url";
 
 import { listStories } from "./args.mjs";
 
 // `demo-video pick`: turns a `/demo-video …` request into one of the
-// already-made stories and a brand. No AI and nothing written: stories are a
-// developer's to make (locally, with the skill); a request only chooses.
+// already-made stories. No AI and nothing written: stories are a developer's
+// to make (locally, with the skill); a request only chooses. Every video is
+// in NaviPartner's brand: a customer's is researched by @Linear for one video
+// (the skill's linear.md), never kept in a repo.
 //
-//   /demo-video                                  → the primary story (one app)
-//   /demo-video help                             → the menu
-//   /demo-video checkout in acme branding   → story "checkout", brand "acme"
-//   /demo-video retail checkout                  → the app whose folder says retail
+//   /demo-video                    → the primary story (one app)
+//   /demo-video help               → the menu
+//   /demo-video checkout           → story "checkout"
+//   /demo-video checkout in danish → its translation "checkout-da"
+//   /demo-video retail checkout    → the app whose folder says retail
 //
 // Anything it can't settle on one story is answered with the menu.
 
-const engineDir = path.resolve(
-  path.dirname(fileURLToPath(import.meta.url)),
-  "..",
-);
-
 const STOP = new Set(
-  "a an and as at by for from in into make me of on or our please show story the this to video with branding brand marketing demo film render".split(
+  "a an and as at by for from in into make me of on or our please show story the this to video with branding brand marketing demo film render navipartner på".split(
     " ",
   ),
 );
@@ -31,44 +27,46 @@ const words = (text) =>
     (w) => [w, ...(w.includes("-") ? w.split("-") : [])],
   );
 
-const brandNames = (dir) =>
-  fs.existsSync(dir)
-    ? fs
-        .readdirSync(dir)
-        .filter((f) => f.endsWith(".json"))
-        .map((f) => f.slice(0, -".json".length))
-    : [];
-
-// The adapter may keep its brands elsewhere (`brands: "../brands"`). Read
-// from demo.config.ts's text: a literal path, as the field is documented.
-const adapterBrandsDir = (appDir) => {
-  const config = path.join(appDir, "demo", "demo.config.ts");
-  const text = fs.existsSync(config) ? fs.readFileSync(config, "utf8") : "";
-  const dir = /\bbrands:\s*["'`]([^"'`]+)["'`]/.exec(text)?.[1] ?? "./brands";
-  return path.resolve(appDir, "demo", dir);
+// A translation is a story named for its language (checkout-da): "danish"
+// and "dansk" say its "da" too.
+const languageNames = (part) => {
+  if (!/^[a-z]{2}$/.test(part)) {
+    return [];
+  }
+  try {
+    return [
+      new Intl.DisplayNames(["en"], { type: "language" }).of(part),
+      new Intl.DisplayNames([part], { type: "language" }).of(part),
+    ]
+      .filter((name) => name && name.toLowerCase() !== part)
+      .map((name) => name.toLowerCase());
+  } catch {
+    return [];
+  }
 };
 
-/** What an app offers: its stories, and its brands plus the engine's. */
+/** What an app offers: its stories. */
 export const appMenu = (appDir) => ({
   dir: appDir,
   name: path.basename(appDir),
   stories: listStories(path.join(appDir, "demo")),
-  brands: [
-    ...new Set([
-      ...brandNames(adapterBrandsDir(appDir)),
-      ...brandNames(path.join(engineDir, "brands")),
-    ]),
-  ],
 });
 
+const LINEAR =
+  "A customer's branding isn't kept in a repo: in Linear, ask @Linear for the video, and it researches their brand for that one video.";
+
 /**
- * Picks from `apps` (appMenu results). Returns { app, story, brand } (brand
- * null = the adapter's default) or { menu: true, reason }.
+ * Picks from `apps` (appMenu results). Returns { app, story } or
+ * { menu: true, reason }.
  */
 export const pick = (request, apps) => {
   // A bare /demo-video films the primary story; this asks what there is.
   if (/^\s*(help|menu|list)\s*$/i.test(request)) {
-    return { menu: true, reason: "The stories and brands:" };
+    return { menu: true, reason: "The stories:" };
+  }
+  const branded = /(?:^|\s)in\s+([\w-]+)\s+brand(?:ing)?\b/i.exec(request);
+  if (branded && branded[1].toLowerCase() !== "navipartner") {
+    return { menu: true, reason: LINEAR };
   }
   return pickStory(request, apps);
 };
@@ -76,22 +74,10 @@ export const pick = (request, apps) => {
 const pickStory = (request, apps) => {
   const said = words(request);
   const has = (word) => said.includes(word.toLowerCase());
+  const saysPart = (part) => has(part) || languageNames(part).some(has);
 
-  // The brand: any offered brand named in the request.
-  const allBrands = [...new Set(apps.flatMap((a) => a.brands))];
-  const brands = allBrands.filter(has);
-  if (brands.length > 1) {
-    return {
-      menu: true,
-      reason: `Name one brand, not ${brands.join(" and ")}.`,
-    };
-  }
-  const brand = brands[0] ?? null;
-
-  // What's left of the request once the brand and filler are gone.
-  const content = said.filter(
-    (w) => !STOP.has(w) && !brands.includes(w) && !/^\d+$/.test(w),
-  );
+  // What's left of the request once the filler is gone.
+  const content = said.filter((w) => !STOP.has(w) && !/^\d+$/.test(w));
 
   // The app: named by its folder, else by a part of it no other app's
   // folder has ("retail", not "self"), else the only one.
@@ -114,23 +100,18 @@ const pickStory = (request, apps) => {
     }
   }
 
-  // Score every story: its name (or a part of it) said outright counts most,
+  // Score every story: its name (or its parts) said outright counts most,
   // then words shared with its description.
-  // Variants aren't scored: the base story plus the brand finds them.
   const scored = candidates.flatMap((app) =>
-    app.stories
-      .filter((story) => !story.extends)
-      .map((story) => {
-        const nameParts = story.name.split("-");
-        // "checkout express" says the story checkout-express in full.
-        let score = has(story.name) || nameParts.every((p) => has(p)) ? 10 : 0;
-        score += nameParts.filter((p) => p.length > 2 && has(p)).length * 3;
-        const description = new Set(words(story.description ?? ""));
-        score += content.filter(
-          (w) => w.length > 2 && description.has(w),
-        ).length;
-        return { app, story, score };
-      }),
+    app.stories.map((story) => {
+      const nameParts = story.name.split("-");
+      // "checkout express" says the story checkout-express in full.
+      let score = has(story.name) || nameParts.every(saysPart) ? 10 : 0;
+      score += nameParts.filter(saysPart).length * 3;
+      const description = new Set(words(story.description ?? ""));
+      score += content.filter((w) => w.length > 2 && description.has(w)).length;
+      return { app, story, score };
+    }),
   );
   const best = Math.max(0, ...scored.map((s) => s.score));
   const top = scored.filter((s) => s.score === best);
@@ -147,7 +128,7 @@ const pickStory = (request, apps) => {
       .map((app) => ({ app, story: app.stories.find((s) => s.primary) }))
       .filter((p) => p.story);
     if (noFeature && candidates.length === 1 && primaries.length === 1) {
-      return withBrand(primaries[0].app, primaries[0].story.name, brand);
+      return { app: primaries[0].app.dir, story: primaries[0].story.name };
     }
     return {
       menu: true,
@@ -163,24 +144,7 @@ const pickStory = (request, apps) => {
       reason: `That matches ${top.map((t) => `\`${t.story.name}\``).join(", ")}. Pick one by name:`,
     };
   }
-  return withBrand(top[0].app, top[0].story.name, brand);
-};
-
-// The brand must be one the chosen app has: another app's brand would fail
-// at render time with a worse message. A story with a variant for that brand
-// (checkout + acme → checkout.acme) films the variant: its words are the
-// customer's.
-const withBrand = (app, story, brand) => {
-  if (brand && !app.brands.includes(brand)) {
-    return {
-      menu: true,
-      reason: `No brand \`${brand}\` in ${app.name}. Pick one:`,
-      apps: [app],
-    };
-  }
-  const variant =
-    brand && app.stories.find((s) => s.extends === story && s.brand === brand);
-  return { app: app.dir, story: variant ? variant.name : story, brand };
+  return { app: top[0].app.dir, story: top[0].story.name };
 };
 
 /** The menu as Markdown, for the PR reply. */
@@ -191,36 +155,30 @@ export const menuMarkdown = (apps, reason) => {
     if (app.stories.length === 0) {
       lines.push("- no stories yet");
     }
-    for (const s of app.stories.filter((x) => !x.extends)) {
+    for (const s of app.stories) {
       lines.push(
         `- \`${s.name}\`${s.primary ? " (primary)" : ""}${s.description ? `: ${s.description}` : ""}`,
       );
-      for (const v of app.stories.filter((x) => x.extends === s.name)) {
-        lines.push(`  - in \`${v.brand}\` branding: \`${v.name}\``);
-      }
     }
-    lines.push(`- brands: ${app.brands.map((b) => `\`${b}\``).join(", ")}`, "");
+    lines.push("");
   }
-  const example = apps.find((a) => a.stories.some((s) => !s.extends));
+  const example = apps.find((a) => a.stories.length > 0);
   if (example) {
-    lines.push(
-      `For example: \`/demo-video ${example.stories.find((s) => !s.extends).name}${example.brands.length > 1 ? ` in ${example.brands.find((b) => b !== "navipartner") ?? example.brands[0]} branding` : ""}\``,
-    );
-    lines.push(
-      "",
-      "A customer without a brand here: ask @Linear for a pitch draft (see the demo-video README).",
-    );
+    lines.push(`For example: \`/demo-video ${example.stories[0].name}\``);
+  }
+  if (reason !== LINEAR) {
+    lines.push("", LINEAR);
   }
   return lines.join("\n");
 };
 
 /**
- * The form's version (Actions → Run workflow): app, story and brand by name,
- * no guessing. Empty story = the primary story, empty brand = the adapter's
- * default, empty app = the only app. A name that doesn't exist gets the menu,
- * so a typo never films something else. `apps` in a menu result narrows it.
+ * The form's version (Actions → Run workflow): app and story by name, no
+ * guessing. Empty story = the primary story, empty app = the only app. A
+ * name that doesn't exist gets the menu, so a typo never films something
+ * else. `apps` in a menu result narrows it.
  */
-export const pickExact = (apps, { app = "", story = "", brand = "" }) => {
+export const pickExact = (apps, { app = "", story = "" }) => {
   const slashes = (p) => p.split("\\").join("/");
   const wanted = slashes(app).replace(/\/+$/, "");
   const candidates = wanted
@@ -250,16 +208,5 @@ export const pickExact = (apps, { app = "", story = "", brand = "" }) => {
       apps: [chosen],
     };
   }
-  if (found.extends) {
-    // A variant is made for one brand.
-    if (brand && brand !== found.brand) {
-      return {
-        menu: true,
-        reason: `\`${found.name}\` is made for \`${found.brand}\`. Leave Brand empty, or pick \`${found.extends}\`:`,
-        apps: [chosen],
-      };
-    }
-    return { app: chosen.dir, story: found.name, brand: found.brand };
-  }
-  return withBrand(chosen, found.name, brand || null);
+  return { app: chosen.dir, story: found.name };
 };

package/bin/pitch.mjs

@@ -75,7 +75,7 @@ const proposalSchema = z
   })
   .strict();
 
-/** "Hotel Royal Århus" → "pitch-hotel-royal-arhus". */
+/** "Northwind Århus" → "pitch-northwind-arhus". */
 export const pitchName = (customer) => {
   const slug = customer
     .toLowerCase()

package/dist/{adapter-D_iU2dh2.d.ts → adapter-CDOXdUZI.d.ts}

@@ -152,13 +152,6 @@ interface DemoAdapter {
     setup(page: Page, ctx: {
         brand: Brand;
     }): Promise<void>;
-    /**
-     * The app's own brands folder, relative to demo/ (default "./brands").
-     * Searched before the brands shipped with the engine (navipartner, ...).
-     */
-    brands?: string;
-    /** Default "navipartner". */
-    defaultBrand?: string;
     /**
      * What this app takes from a brand, each with what it is and where it
      * shows ("full-screen photo behind the menu, landscape"). Apps differ: one

package/dist/{chunk-BYAFDWPA.js → chunk-D7UYGNSF.js}

@@ -502,76 +502,15 @@ var parseStoryboard = (raw, adapter, source = "storyboard") => {
   throw new StoryboardError(`Invalid ${source}:
 ${lines.join("\n")}`);
 };
-var cardChange = z.object({
-  eyebrow: cardFields.eyebrow,
-  heading: cardFields.heading,
-  sub: cardFields.sub,
-  logo: cardFields.logo,
-  style: z.enum(["pale", "brand"]).optional(),
-  seconds: z.number().min(0.5).max(10).optional()
-}).strict();
-var variantSchema = z.object({
-  extends: z.string().regex(/^[\w-]+$/, "the base story's name"),
-  brand: z.string().regex(/^[\w-]+$/, "a brand name"),
-  description: z.string().min(1).max(280).optional(),
-  title: cardChange.optional(),
-  closing: cardChange.optional(),
-  cards: z.array(cardChange).optional()
-}).strict();
 var readJson = (file) => JSON.parse(fs.readFileSync(file, "utf8"));
-var isVariant = (raw) => typeof raw === "object" && raw !== null && "extends" in raw;
-var loadVariant = (raw, file, adapter) => {
-  const name = path.basename(file);
-  const result = variantSchema.safeParse(raw);
-  if (!result.success) {
-    const lines = result.error.issues.map(
-      (i) => `  ${i.path.join(".") || "(root)"}: ${i.message}`
-    );
-    throw new StoryboardError(`Invalid ${name}:
-${lines.join("\n")}`);
-  }
-  const variant = result.data;
-  const baseFile = path.join(path.dirname(file), `${variant.extends}.json`);
-  if (!fs.existsSync(baseFile)) {
-    throw new StoryboardError(
-      `${name} extends "${variant.extends}", but there is no ${variant.extends}.json beside it`
-    );
-  }
-  const base = readJson(baseFile);
-  if (isVariant(base)) {
-    throw new StoryboardError(
-      `${name} extends ${variant.extends}.json, which is a variant itself; extend the story it extends`
-    );
-  }
-  const steps = Array.isArray(base.steps) ? base.steps : [];
-  const cardAt = steps.flatMap((s, i) => s.do === "card" ? [i] : []);
-  if (variant.cards && variant.cards.length !== cardAt.length) {
+var loadStoryboard = (file, adapter) => {
+  const raw = readJson(file);
+  if (typeof raw === "object" && raw !== null && ("extends" in raw || "brand" in raw)) {
     throw new StoryboardError(
-      `${name} gives ${variant.cards.length} card${variant.cards.length === 1 ? "" : "s"}; ${variant.extends}.json has ${cardAt.length}. A variant gives every card of its story, in order.`
+      `${path.basename(file)} names a brand or extends a story: stories hold neither. A customer's brand is researched by @Linear for one video; a translation is a story of its own (checkout-da.json).`
     );
   }
-  const change = (card, over) => over ? { ...card, ...over } : card;
-  const merged = steps.map(
-    (step, i) => variant.cards ? change(step, variant.cards[cardAt.indexOf(i)]) : step
-  );
-  const rest = { ...base };
-  delete rest.primary;
-  const story = parseStoryboard(
-    {
-      ...rest,
-      description: variant.description ?? base.description,
-      title: change(base.title, variant.title),
-      closing: change(base.closing, variant.closing),
-      steps: merged
-    },
-    adapter,
-    `${name} (on ${variant.extends}.json)`
-  );
-  return { ...story, brand: variant.brand, variantOf: variant.extends };
-};
-var loadStoryboard = (file, adapter) => {
-  const raw = readJson(file);
-  return isVariant(raw) ? loadVariant(raw, file, adapter) : parseStoryboard(raw, adapter, path.basename(file));
+  return parseStoryboard(raw, adapter, path.basename(file));
 };
 var MIN_ZOOM = MIN_SCALE;
 var MAX_ZOOM = MAX_SCALE;

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { D as DemoAdapter } from './adapter-D_iU2dh2.js';
-export { B as Brand, C as CardSpec, a as DemoAction, b as DirectorApi, T as Target, V as VideoBrand, d as defineAction, c as defineDemo } from './adapter-D_iU2dh2.js';
+import { D as DemoAdapter } from './adapter-CDOXdUZI.js';
+export { B as Brand, C as CardSpec, a as DemoAction, b as DirectorApi, T as Target, V as VideoBrand, d as defineAction, c as defineDemo } from './adapter-CDOXdUZI.js';
 import { Page } from '@playwright/test';
 import { z } from 'zod';
 export { z } from 'zod';
@@ -236,10 +236,6 @@ declare const cardSchema: z.ZodEffects<z.ZodObject<{
 type Card = z.infer<typeof cardSchema>;
 interface Storyboard {
     app: string;
-    /** A variant's brand (see loadStoryboard): it is filmed with this one only. */
-    brand?: string;
-    /** The story a variant extends. */
-    variantOf?: string;
     /** What the story shows, for people choosing one (and the menu). */
     description?: string;
     /** The app's primary journey: the story filmed when none is named. One per app. */

package/dist/index.js

@@ -1,7 +1,7 @@
 import {
   StoryboardError,
   parseStoryboard
-} from "./chunk-BYAFDWPA.js";
+} from "./chunk-D7UYGNSF.js";
 
 // src/adapter.ts
 var defineDemo = (adapter) => adapter;

package/dist/runner/index.d.ts

@@ -1,5 +1,5 @@
 import { PlaywrightTestConfig, Page } from '@playwright/test';
-import { D as DemoAdapter } from '../adapter-D_iU2dh2.js';
+import { D as DemoAdapter } from '../adapter-CDOXdUZI.js';
 import 'zod';
 
 /**

package/dist/runner/index.js

@@ -30,7 +30,7 @@ import {
   unionRect,
   visibleRect,
   wideShot
-} from "../chunk-BYAFDWPA.js";
+} from "../chunk-D7UYGNSF.js";
 
 // src/runner/config.ts
 import path from "path";
@@ -135,10 +135,17 @@ var engineRoot = () => {
   }
 };
 var sharedBrandsDir = () => path2.join(engineRoot(), "brands");
-var brandDirs = (demoDir, brands = "./brands") => [
-  path2.resolve(demoDir, brands),
+var brandDirs = (pitchDir) => [
+  ...pitchDir ? [pitchDir] : [],
   sharedBrandsDir()
 ];
+var refuseRepoBrands = (demoDir) => {
+  if (fs.existsSync(path2.join(demoDir, "brands"))) {
+    throw new Error(
+      `${path2.join(demoDir, "brands")} isn't used: repos keep no brands. NaviPartner's ships with the engine; a customer's is researched by @Linear for one video and never saved. Delete the folder.`
+    );
+  }
+};
 var brandNames = (dir) => fs.existsSync(dir) ? fs.readdirSync(dir).filter((f) => f.endsWith(".json")).map((f) => f.slice(0, -".json".length)) : [];
 var loadBrand = (dirs, name) => {
   if (!NAME_RE.test(name)) {
@@ -1196,17 +1203,10 @@ var registerDemoTests = (test, adapter, demoDir) => {
         ].join("\n")
       );
     }
-    if (storyboard.brand && env.brand && env.brand !== storyboard.brand) {
-      throw new StoryboardError(
-        `${path4.basename(env.storyboardPath, ".json")} is ${storyboard.variantOf} for the brand ${storyboard.brand}. Render it without --brand, or render ${storyboard.variantOf} with --brand ${env.brand}.`
-      );
-    }
+    refuseRepoBrands(demoDir);
     const brand = loadBrand(
-      [
-        ...env.brandsDir ? [env.brandsDir] : [],
-        ...brandDirs(demoDir, adapter.brands)
-      ],
-      env.brand ?? storyboard.brand ?? adapter.defaultBrand ?? "navipartner"
+      brandDirs(env.brandsDir),
+      env.brand ?? "navipartner"
     );
     const missing = missingNeeds(brand, adapter.brandNeeds);
     if (missing.length > 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@navipartner/demo-video",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "Marketing demo videos of a React app, filmed by Playwright against the app's own e2e mocks.",
   "license": "UNLICENSED",
   "repository": {