@koda-sl/baker-cli 0.148.0 → 0.149.0-dev.7b64ca6b5

package/dist/cli.js

@@ -34,7 +34,7 @@ import {
   toModelSafeImage,
   ulid,
   validateCanvasDeep
-} from "./chunk-2F5SCTNS.js";
+} from "./chunk-ZBRVPPJP.js";
 import {
   csvOrJson,
   daysAgoIso,
@@ -2544,6 +2544,13 @@ var imageDocSchema = z8.object({
   width: z8.number().optional(),
   height: z8.number().optional(),
   aspectRatio: z8.number().optional(),
+  /** Any non-opaque pixel in the decoded image. */
+  hasAlpha: z8.boolean().optional(),
+  /** Opaque pixels ÷ their bounding-box area, 0..1. Feed with `aspectRatio`
+   * into `classifyLogoShape` to tell a brand wordmark from an app-icon plate
+   * before placing it — the library keeps assets forever, so a bad ingest is
+   * otherwise indistinguishable from a good one months later. */
+  solidity: z8.number().optional(),
   dominantColor: z8.string().optional(),
   imagePalette: z8.array(z8.string()).optional(),
   thumbhashDataUri: z8.string().optional(),
@@ -2626,6 +2633,13 @@ var imageSearchResultSchema = z8.object({
   width: z8.number().optional(),
   height: z8.number().optional(),
   aspectRatio: z8.number().optional(),
+  /** Any non-opaque pixel in the decoded image. */
+  hasAlpha: z8.boolean().optional(),
+  /** Opaque pixels ÷ their bounding-box area, 0..1. Feed with `aspectRatio`
+   * into `classifyLogoShape` to tell a brand wordmark from an app-icon plate
+   * before placing it — the library keeps assets forever, so a bad ingest is
+   * otherwise indistinguishable from a good one months later. */
+  solidity: z8.number().optional(),
   dominantColor: z8.string().optional(),
   imagePalette: z8.array(z8.string()).optional(),
   source: z8.string(),
@@ -2749,10 +2763,13 @@ var imagesLogoRequestSchema = z8.object({
   descriptionContext: z8.string().optional()
 });
 var imagesLogoResponseSchema = providerHitsResponseSchema();
+var hexColorSchema = z8.string().transform((value) => value.trim().replace(/^%23/i, "#")).refine((value) => /^#?([0-9a-fA-F]{3}|[0-9a-fA-F]{6})$/.test(value), {
+  message: "Expected a hex color like #D7FFA4"
+}).transform((value) => value.startsWith("#") ? value : `#${value}`);
 var imagesIconRequestSchema = z8.object({
   name: z8.string().min(1),
   set: z8.string().optional(),
-  color: z8.string().optional(),
+  color: hexColorSchema.optional(),
   width: z8.coerce.number().int().positive().optional(),
   autoIngest: z8.coerce.number().int().min(0).max(20).optional(),
   descriptionContext: z8.string().optional()
@@ -2802,6 +2819,19 @@ var imagesIngestResponseSchema = z8.object({
   contentHash: z8.string()
 });
 
+// ../api/src/logoShape.ts
+var PLATE_SOLIDITY_THRESHOLD = 0.6;
+var WORDMARK_MIN_ASPECT = 1.8;
+function classifyLogoShape({
+  aspectRatio: aspectRatio2,
+  hasAlpha,
+  solidity
+}) {
+  if (solidity === void 0 || aspectRatio2 === void 0) return "unknown";
+  if (solidity >= PLATE_SOLIDITY_THRESHOLD || hasAlpha === false) return "plated-icon";
+  return aspectRatio2 >= WORDMARK_MIN_ASPECT ? "wordmark" : "symbol";
+}
+
 // ../api/src/tags.ts
 import { z as z9 } from "zod";
 var TAG_TYPES = [
@@ -5395,27 +5425,17 @@ var googleDraftStageRequestSchema = z13.object({
   chatId: z13.string(),
   op: googleDraftOpInputSchema
 });
-var googleDraftStageResponseSchema = z13.discriminatedUnion("staged", [
-  z13.object({
-    staged: z13.literal(true),
-    ref: z13.string(),
-    kind: googleDraftOpKindSchema,
-    mode: googleWriteModeSchema,
-    dependsOn: z13.array(z13.string()),
-    summary: z13.string(),
-    warnings: z13.array(z13.string()),
-    /** True when the op amended an already-staged op in place instead of appending a new one. */
-    amended: z13.boolean().optional()
-  }),
-  z13.object({
-    staged: z13.literal(false),
-    noop: z13.literal(true),
-    kind: googleDraftOpKindSchema,
-    mode: googleWriteModeSchema,
-    summary: z13.string(),
-    reason: z13.string()
-  })
-]);
+var googleDraftStageResponseSchema = z13.object({
+  staged: z13.literal(true),
+  ref: z13.string(),
+  kind: googleDraftOpKindSchema,
+  mode: googleWriteModeSchema,
+  dependsOn: z13.array(z13.string()),
+  summary: z13.string(),
+  warnings: z13.array(z13.string()),
+  /** True when the op amended an already-staged op in place instead of appending a new one. */
+  amended: z13.boolean().optional()
+});
 var googleDraftAmendRequestSchema = z13.object({
   chatId: z13.string(),
   ref: z13.string(),
@@ -5442,8 +5462,7 @@ var googleDraftStageBatchResponseSchema = z13.object({
       summary: z13.string(),
       warnings: z13.array(z13.string())
     })
-  ),
-  skipped: z13.array(z13.object({ kind: googleDraftOpKindSchema, summary: z13.string(), reason: z13.string() })).optional()
+  )
 });
 var googleDraftOpViewSchema = z13.object({
   ref: z13.string(),
@@ -5702,7 +5721,7 @@ function keywordEntries(args) {
   const seen = /* @__PURE__ */ new Set();
   for (const item of raw) {
     const entry = parseKeywordEntry(item, defaultMatch);
-    const key = JSON.stringify([entry.text, entry.matchType]);
+    const key = `${entry.text}\0${entry.matchType}`;
     if (!seen.has(key)) {
       seen.add(key);
       entries.push(entry);
@@ -5769,9 +5788,6 @@ function handleGoogleError(err) {
   });
   process.exit(1);
 }
-function noopHints(data) {
-  return data.staged ? [] : [`NOT STAGED \u2014 ${data.reason}. Nothing to publish for this change; say so rather than reporting it as done.`];
-}
 async function stageGoogleOp(raw, hints) {
   const preflight = googleDraftOpInputSchema.safeParse(raw);
   if (!preflight.success) {
@@ -5789,8 +5805,7 @@ async function stageGoogleOp(raw, hints) {
   try {
     const chatId = requireChatId();
     const response = await apiPost("/api/ads/google/draft/stage", { chatId, op: preflight.data });
-    const allHints = [...noopHints(response.data), ...hints ?? []];
-    writeJsonEnvelope(allHints.length > 0 ? { ...response, hints: allHints } : response);
+    writeJsonEnvelope(hints && hints.length > 0 ? { ...response, hints } : response);
   } catch (err) {
     handleGoogleError(err);
   }
@@ -5826,14 +5841,7 @@ async function stageGoogleOps(rawOps, hints) {
   try {
     const chatId = requireChatId();
     const response = await apiPost("/api/ads/google/draft/stage-batch", { chatId, ops });
-    const skipped = response.data.skipped ?? [];
-    const allHints = [
-      ...skipped.length > 0 ? [
-        `${skipped.length} of ${ops.length} op(s) NOT STAGED \u2014 already in the requested state: ${skipped.map((s) => s.reason).join("; ")}`
-      ] : [],
-      ...hints ?? []
-    ];
-    writeJsonEnvelope(allHints.length > 0 ? { ...response, hints: allHints } : response);
+    writeJsonEnvelope(hints && hints.length > 0 ? { ...response, hints } : response);
   } catch (err) {
     handleGoogleError(err);
   }
@@ -24492,6 +24500,31 @@ var ingestCommand = defineCommand119({
 
 // src/commands/images/library.ts
 import { defineCommand as defineCommand120 } from "citty";
+
+// src/commands/images/logoHints.ts
+var LOGO_QUERY_RE = /\blogos?\b|\bwordmark\b|\bbrand mark\b/i;
+function withLogoShape(row) {
+  return {
+    ...row,
+    logoShape: classifyLogoShape({
+      aspectRatio: row.aspectRatio,
+      hasAlpha: row.hasAlpha,
+      solidity: row.solidity
+    })
+  };
+}
+function buildLogoLibraryHints(query, rows) {
+  if (!LOGO_QUERY_RE.test(query)) return [];
+  const plates = rows.filter((row) => row.logoShape === "plated-icon");
+  if (plates.length === 0) return [];
+  const names = plates.map((row) => row.name).filter((name) => Boolean(name)).slice(0, 4);
+  const subject = names.length > 0 ? names.join(", ") : `${plates.length} result(s)`;
+  return [
+    `PLATED ICON: ${subject} \u2014 solid tile with the mark knocked out, not a wordmark. In a logo strip it renders as a filled box and verify blocks it (logo-plate). Re-source with 'baker images logo <domain> --variant logo', or strip the plate with 'baker images normalize <file> --remove-bg --shrink-to-content'. Only use a plate in slots under ~32px.`
+  ];
+}
+
+// src/commands/images/library.ts
 registerSchema({
   command: "images.library",
   description: "Search the company image library. Returns only ready images.",
@@ -24556,8 +24589,10 @@ var libraryCommand = defineCommand120({
       if (minScore !== void 0) {
         data = data.filter((r) => typeof r.score === "number" && r.score >= minScore);
       }
+      const shaped = data.map(withLogoShape);
+      const hints = buildLogoLibraryHints(query, shaped);
       writeOutput(
-        { ok: true, data },
+        { ok: true, data: shaped, ...hints.length > 0 ? { hints } : {} },
         args.output || "json",
         args.fields ? args.fields.split(",") : void 0,
         args.full
@@ -24575,11 +24610,16 @@ var libraryCommand = defineCommand120({
 
 // src/commands/images/logo.ts
 import { defineCommand as defineCommand121 } from "citty";
+var MAX_DOMAINS = 20;
 registerSchema({
   command: "images.logo",
   description: "Brand logo lookup via Brandfetch CDN (fallback/404). Auto-ingests by default.",
   args: {
-    domain: { type: "string", description: "Brand domain (e.g. stripe.com)", required: true },
+    domain: {
+      type: "string",
+      description: "Brand domain, or a comma-separated list for a whole strip (e.g. stripe.com,intercom.com)",
+      required: true
+    },
     variant: { type: "string", description: "icon | logo | symbol", required: false },
     "auto-ingest": {
       type: "number",
@@ -24602,10 +24642,10 @@ registerSchema({
 var logoCommand = defineCommand121({
   meta: {
     name: "logo",
-    description: "Brand logo via Brandfetch CDN. Returns up to 5 variants (icon, light/dark logo, light/dark symbol). Auto-ingests the first variant.\n\nExample: baker images logo stripe.com --variant logo"
+    description: "Brand logo via Brandfetch CDN. Returns up to 5 variants (icon, light/dark logo, light/dark symbol). Auto-ingests the first variant.\n\nSource a whole strip in one call by passing a comma-separated list \u2014 a text-rendered brand name is never the cheaper option.\n\nExamples:\n  baker images logo stripe.com --variant logo\n  baker images logo stripe.com,intercom.com,notion.so --variant logo"
   },
   args: {
-    domain: { type: "positional", description: "Brand domain", required: false },
+    domain: { type: "positional", description: "Brand domain, or a comma-separated list", required: false },
     variant: { type: "string", description: "icon|logo|symbol", required: false },
     "auto-ingest": { type: "string", description: "Ingest top N (0-20, default 1)", required: false },
     "no-auto-ingest": { type: "boolean", description: "Skip auto-ingest", required: false },
@@ -24613,18 +24653,50 @@ var logoCommand = defineCommand121({
   },
   run: async ({ args }) => {
     try {
-      const domain = args.domain;
-      if (!domain) {
+      const raw = args.domain;
+      const domains = [
+        ...new Set(
+          (raw ?? "").split(",").map((d) => d.trim()).filter(Boolean)
+        )
+      ];
+      if (domains.length === 0) {
         writeJson({ ok: false, error: { code: "VALIDATION_ERROR", message: "Domain is required" } });
         process.exit(1);
       }
-      const body = { domain };
-      if (args.variant) body.variant = args.variant;
-      if (args["auto-ingest"] !== void 0) body.autoIngest = Number(args["auto-ingest"]);
-      else if (args["no-auto-ingest"]) body.autoIngest = 0;
-      if (args.context) body.descriptionContext = args.context;
-      const data = await apiPost("/api/images/logo", body);
-      writeJson({ ok: true, data });
+      if (domains.length > MAX_DOMAINS) {
+        writeJson({
+          ok: false,
+          error: {
+            code: "VALIDATION_ERROR",
+            message: `Too many domains (${domains.length}); pass at most ${MAX_DOMAINS} per call`
+          }
+        });
+        process.exit(1);
+      }
+      const base = {};
+      if (args.variant) base.variant = args.variant;
+      if (args["auto-ingest"] !== void 0) base.autoIngest = Number(args["auto-ingest"]);
+      else if (args["no-auto-ingest"]) base.autoIngest = 0;
+      if (args.context) base.descriptionContext = args.context;
+      const fetchOne = (domain) => apiPost("/api/images/logo", { ...base, domain });
+      if (domains.length === 1) {
+        writeJson({ ok: true, data: await fetchOne(domains[0]) });
+        return;
+      }
+      const settled = await Promise.all(
+        domains.map(async (domain) => {
+          try {
+            return { domain, ...await fetchOne(domain) };
+          } catch (err) {
+            return { domain, hits: [], error: err instanceof ApiError ? err.message : "Lookup failed" };
+          }
+        })
+      );
+      const empty = settled.filter((r) => r.hits.length === 0).map((r) => r.domain);
+      const hints = empty.length > 0 ? [
+        `NO LOGO: ${empty.join(", ")} \u2014 Brandfetch doesn't know ${empty.length === 1 ? "this brand" : "these brands"}. Fall back per domain: 'baker images extract <domain> --auto-ingest 5', then 'baker images icon <brand> --set simple-icons'. Still nothing \u2192 'baker actions create' to acquire the file. Do not render the name as text.`
+      ] : [];
+      writeJson({ ok: true, data: { results: settled }, ...hints.length > 0 ? { hints } : {} });
     } catch (err) {
       if (err instanceof ApiError) {
         writeJson({ ok: false, error: { code: err.code, message: err.message } });
@@ -24668,6 +24740,7 @@ function getDominantEdgeColor(data, width, height) {
   const colorCount = {};
   function accumulateColor(i, j) {
     const idx = (i * width + j) * 4;
+    if ((data[idx + 3] ?? 0) < 10) return;
     const colorKey = `${data[idx]},${data[idx + 1]},${data[idx + 2]}`;
     colorCount[colorKey] = (colorCount[colorKey] ?? 0) + 1;
   }
@@ -24680,7 +24753,7 @@ function getDominantEdgeColor(data, width, height) {
     accumulateColor(i, width - 1);
   }
   let maxCount = 0;
-  let dominantColor = { r: 0, g: 0, b: 0 };
+  let dominantColor = null;
   for (const key in colorCount) {
     const count = colorCount[key];
     if (count > maxCount) {
@@ -24691,6 +24764,27 @@ function getDominantEdgeColor(data, width, height) {
   }
   return dominantColor;
 }
+function opaqueSolidity(data, width, height) {
+  let minX = width;
+  let minY = height;
+  let maxX = -1;
+  let maxY = -1;
+  let opaque = 0;
+  for (let y = 0; y < height; y++) {
+    for (let x = 0; x < width; x++) {
+      if ((data[(y * width + x) * 4 + 3] ?? 0) <= 200) continue;
+      opaque++;
+      if (x < minX) minX = x;
+      if (x > maxX) maxX = x;
+      if (y < minY) minY = y;
+      if (y > maxY) maxY = y;
+    }
+  }
+  if (maxX < 0) return 0;
+  const boxArea = (maxX - minX + 1) * (maxY - minY + 1);
+  return boxArea === 0 ? 0 : opaque / boxArea;
+}
+var PLATE_SOLIDITY_THRESHOLD2 = 0.6;
 function hasTransparency(data, threshold = 0.02) {
   let transparentPixels = 0;
   let totalPixels = 0;
@@ -24755,6 +24849,9 @@ function removeBackground(data, width, height, colorRangeThreshold = COLOR_RANGE
     return data;
   }
   const dominantEdgeColor = getDominantEdgeColor(data, width, height);
+  if (!dominantEdgeColor) {
+    return data;
+  }
   const isGradient = hasGradientColors(data);
   const result = Buffer.from(data);
   if (isGradient) {
@@ -25032,8 +25129,8 @@ async function processInternal(inputBuffer, isSVG, options) {
   const metadata = await sharp3(inputBuffer).metadata();
   let alreadyTransparent = false;
   if (metadata.hasAlpha) {
-    const { data: alphaData } = await sharp3(inputBuffer).raw().toBuffer({ resolveWithObject: true });
-    alreadyTransparent = hasTransparency(alphaData, 0.05);
+    const { data: alphaData, info: info2 } = await sharp3(inputBuffer).raw().toBuffer({ resolveWithObject: true });
+    alreadyTransparent = opaqueSolidity(alphaData, info2.width, info2.height) < PLATE_SOLIDITY_THRESHOLD2;
   }
   let { data: processedData, info } = await sharp3(inputBuffer).ensureAlpha().raw().toBuffer({ resolveWithObject: true });
   if (options.color) {