@koda-sl/baker-cli 0.152.0 → 0.153.0

package/README.md

@@ -4668,10 +4668,12 @@ Without `--slug` the command behaves as before (one creative record per publishe
 Deterministic design-quality critic for landing pages — an offline, no-LLM detector for the well-known "AI slop" tells (gradient text, overused fonts like Inter, side-tab colored borders, cream/parchment grounds, purple-on-heading palettes, buzzword copy, broken images, and more). It scores a landing's `.astro` source and returns findings tiered **block / warn / advisory**.
 
 ```bash
-baker landing critique spring-offer      # score src/pages/spring-offer/
-baker landing critique spring-offer --full   # also print advisory-tier findings
+baker landing critique spring-offer                      # score src/pages/spring-offer/
+baker landing critique spring-offer --full               # also print advisory-tier findings
+baker landing critique spring-offer summer-offer claude  # score three landings in one call
 ```
 
+- **Takes several slugs at once.** `data.landings` is always an array with one entry per slug (`{ slug, overall, counts, dimensions, findings }`). For a single slug those fields are also spread at the top of `data`, so existing one-landing readers are unaffected. A wrong slug anywhere in the batch fails the whole call before any landing is scored.
 - **Advisory by design — findings never fail it.** A finding is guidance, not a gate; the command errors (exit 2) only on a wrong slug (`NOT_FOUND` / `INVALID_SLUG`, with `error.fix.availableSlugs`).
 - **BRAND.md is the allowlist.** A font, hue, or ground that the client's `src/brand/BRAND.md` (mirrored into `src/styles/global.css` `@theme`, hex or oklch) commits is a decision, not a tell — matched values downgrade to advisory `brandDrift` so the critic never fights a real brand.
 - **Records a freshness snapshot** to `.cache/landing-critique/<slug>.json` (the source hash it scored). The publish quality gate reads it to enforce that every changed landing was critiqued and ships no block-tier tell. Re-run after edits so the snapshot stays fresh.
@@ -4739,6 +4741,7 @@ This CLI is designed for AI agent consumption. Key patterns:
 - **0.119.0**: `draft amend`/`draft show` land on both `baker ads google` and `baker ads linkedin` — a generic JSON-merge-patch to update any staged op in place plus a full-payload receipt, replacing remove+recreate as the correction path. Google gains `assets update` and `asset-groups create|update` (Performance Max asset groups are now their own entity — `ads create --format performanceMaxAssetGroup` never worked and is gone); `ads create --format video` moves from a bare YouTube id to `--video-assets` refs (**breaking flag change** — stage the video as an asset first); `--format demandGen` gains `--image-assets`/`--square-image-assets`/`--logo-image-assets` and flag-building for headlines/descriptions. LinkedIn's `draft list` now renders a readable Campaign group ▸ Campaign ▸ Creative tree by default (`--json` for raw), `creatives update` gains `--campaign` (re-parent while staged), and `campaigns update` passes create-only fields (`--group`/`--type`/`--locale`/`--associated-entity`) through when amending a `li_temp_*` staged create instead of always stripping them.
 - **0.150.0**: `baker landing critique` gains an `agent-washing` rule in the `copy` family — an autonomy claim ("fully autonomous", "while you sleep", "no human intervention") with no signal anywhere on the page about who oversees the agent (approval, review, override, undo/rollback, audit log, escalation) scores a warn. Page-scope, so a hero may defer the trust story to a later section; generic privacy boilerplate does not clear it. `CRITIC_VERSION` bumps to `2`.
 - **0.152.0**: `baker ads linkedin campaigns url-params <ad-set-id>` stages an ad set's URL tracking parameters (LinkedIn's `adTrackingParameters`) — `--param key=value` for fixed values (repeatable or `&`-joined), `--dynamic key=PLACEHOLDER` for values LinkedIn fills in per ad (`CAMPAIGN_NAME`, `CREATIVE_ID`, …), `--clear` to remove them. LinkedIn appends these to the landing URL of every ad in the ad set, including ads already running, so this replaces stamping the same UTM onto each ad's `--landing-url` — which missed later ads and double-appended keys the ad set already set. Account-level parameters remain UI-only (LinkedIn exposes no API for them).
+- **0.153.0**: `baker landing critique` accepts several slugs in one call (`baker landing critique a b c`). `data.landings[]` always carries one entry per slug; a single slug additionally keeps the old top-level `data.slug`/`counts`/`dimensions`/`findings` shape, so existing readers are unaffected. An invalid or missing slug anywhere in the batch still fails the whole call up front. The scaffold's Stop-hook critique gate now emits one batched command instead of one line per stale landing.
 
 ## Publishing
 

package/dist/cli.js

@@ -28065,7 +28065,11 @@ registerSchema({
   command: "landing.critique",
   description: "Start here: `baker landing critique my-landing` after building or editing a landing. Deterministic design-quality critic (ADVISORY \u2014 findings never fail it). Flags the known AI 'slop' tells (gradient text, overused fonts, side-tab borders, cream palettes, buzzword copy, broken images\u2026) tiered block/warn/advisory, and records the critique the publish quality gate requires.",
   args: {
-    slug: { type: "string", description: "Landing slug (folder under src/pages/)", required: true },
+    slug: {
+      type: "string",
+      description: "Landing slug (folder under src/pages/). Pass several, space-separated, to critique them in one call.",
+      required: true
+    },
     full: {
       type: "boolean",
       description: "Also print advisory-tier findings (default: block + warn only)",
@@ -28074,6 +28078,11 @@ registerSchema({
   }
 });
 var SLUG_RE = /^[a-z0-9][a-z0-9._-]*$/i;
+function parseCritiqueSlugs(args) {
+  const rest = Array.isArray(args._) ? args._ : [];
+  const all = [args.slug, ...rest].filter((s) => typeof s === "string" && s.length > 0);
+  return [...new Set(all.map(String))];
+}
 function fail5(code, message, fix) {
   process.stderr.write(
     `${JSON.stringify({ ok: false, error: { code, message, ...fix ? { fix } : {} } }, null, 2)}
@@ -28087,63 +28096,53 @@ var critiqueCommand2 = defineCommand135({
     description: "Start here: `baker landing critique <slug>` after building or editing a landing. Deterministic design-quality critic (ADVISORY \u2014 findings never fail it). Flags the known AI design tells (gradient text, overused fonts, side-tab borders, cream palettes, buzzword copy, broken images) tiered block/warn/advisory, respecting the client's BRAND.md as the allowlist. Also records the critique that publishing requires \u2014 run it before finishing a landing."
   },
   args: {
-    slug: { type: "positional", required: true, description: "Landing slug (folder under src/pages/)" },
+    slug: {
+      type: "positional",
+      required: true,
+      description: "Landing slug (folder under src/pages/). Space-separate several to critique them in one call."
+    },
     full: { type: "boolean", description: "Also print advisory-tier findings", default: false }
   },
   async run({ args }) {
-    const slug = String(args.slug);
+    const slugs = parseCritiqueSlugs(args);
     const projectRoot = process.cwd();
-    if (!SLUG_RE.test(slug) || slug.includes("..")) {
-      fail5(
-        "INVALID_SLUG",
-        `"${slug}" is not a landing slug \u2014 use the folder name directly under src/pages/ (letters, digits, dashes; not a path, not a _-private folder).`,
-        { availableSlugs: await listLandingSlugs(projectRoot) }
-      );
-    }
-    const landingDir = path27.resolve(projectRoot, "src", "pages", slug);
-    if (!await isDir(landingDir)) {
-      fail5("NOT_FOUND", `No landing at src/pages/${slug}/`, {
-        availableSlugs: await listLandingSlugs(projectRoot)
-      });
-    }
-    const [sources, brand, sourceSha] = await Promise.all([
-      readLandingSources(landingDir),
-      loadBrandTokens(projectRoot),
-      computeLandingSourceSha(landingDir)
-    ]);
-    const report = critiqueLanding({ slug, sources, brand });
-    let snapshotFailed = false;
-    try {
-      await writeCritiqueSnapshot(projectRoot, {
-        slug,
-        sourceSha,
-        criticVersion: CRITIC_VERSION,
-        at: (/* @__PURE__ */ new Date()).toISOString(),
-        blockCount: report.counts.block,
-        warnCount: report.counts.warn
-      });
-    } catch {
-      snapshotFailed = true;
+    for (const slug of slugs) {
+      if (!SLUG_RE.test(slug) || slug.includes("..")) {
+        fail5(
+          "INVALID_SLUG",
+          `"${slug}" is not a landing slug \u2014 use the folder name directly under src/pages/ (letters, digits, dashes; not a path, not a _-private folder).`,
+          { availableSlugs: await listLandingSlugs(projectRoot) }
+        );
+      }
+      if (!await isDir(path27.resolve(projectRoot, "src", "pages", slug))) {
+        fail5("NOT_FOUND", `No landing at src/pages/${slug}/`, {
+          availableSlugs: await listLandingSlugs(projectRoot)
+        });
+      }
     }
-    const shown = report.findings.filter((f) => args.full ? true : f.severity !== "advisory");
+    const brand = await loadBrandTokens(projectRoot);
+    const results = await Promise.all(slugs.map((slug) => critiqueOne(projectRoot, slug, brand)));
+    const landings = results.map(({ slug, report }) => ({
+      slug,
+      overall: report.overall,
+      counts: report.counts,
+      dimensions: report.dimensions,
+      findings: report.findings.filter((f) => args.full ? true : f.severity !== "advisory").map(present)
+    }));
+    const blocked = results.filter((r) => r.report.counts.block > 0).map((r) => r.slug);
+    const snapshotFailures = results.filter((r) => r.snapshotFailed).map((r) => r.slug);
     const hints = [
-      report.counts.block > 0 ? "Block-tier tells are held at publish \u2014 fix them. Warn/advisory are guidance. Scores are 0\u20131 (higher is better)." : "No block-tier tells. Warn/advisory findings are guidance, not gates. Re-run after edits so the publish gate stays fresh.",
-      ...snapshotFailed ? [
-        "Could not record the critique snapshot (.cache/ not writable?) \u2014 the publish quality gate will still report this landing as un-critiqued."
+      blocked.length > 0 ? `Block-tier tells are held at publish \u2014 fix them (${blocked.join(", ")}). Warn/advisory are guidance. Scores are 0\u20131 (higher is better).` : "No block-tier tells. Warn/advisory findings are guidance, not gates. Re-run after edits so the publish gate stays fresh.",
+      ...snapshotFailures.length > 0 ? [
+        `Could not record the critique snapshot for ${snapshotFailures.join(", ")} (.cache/ not writable?) \u2014 the publish quality gate will still report ${snapshotFailures.length === 1 ? "this landing" : "these landings"} as un-critiqued.`
       ] : []
     ];
+    const single = landings.length === 1 ? landings[0] : null;
     process.stdout.write(
       `${JSON.stringify(
         {
           ok: true,
-          data: {
-            advisory: true,
-            slug,
-            overall: report.overall,
-            counts: report.counts,
-            dimensions: report.dimensions,
-            findings: shown.map(present)
-          },
+          data: { advisory: true, ...single ?? {}, landings },
           hints
         },
         null,
@@ -28153,6 +28152,25 @@ var critiqueCommand2 = defineCommand135({
     );
   }
 });
+async function critiqueOne(projectRoot, slug, brand) {
+  const landingDir = path27.resolve(projectRoot, "src", "pages", slug);
+  const [sources, sourceSha] = await Promise.all([readLandingSources(landingDir), computeLandingSourceSha(landingDir)]);
+  const report = critiqueLanding({ slug, sources, brand });
+  let snapshotFailed = false;
+  try {
+    await writeCritiqueSnapshot(projectRoot, {
+      slug,
+      sourceSha,
+      criticVersion: CRITIC_VERSION,
+      at: (/* @__PURE__ */ new Date()).toISOString(),
+      blockCount: report.counts.block,
+      warnCount: report.counts.warn
+    });
+  } catch {
+    snapshotFailed = true;
+  }
+  return { slug, report, snapshotFailed };
+}
 async function listLandingSlugs(projectRoot) {
   try {
     const entries = await readdir8(path27.join(projectRoot, "src", "pages"), { withFileTypes: true });