@koda-sl/baker-cli 0.290.2 → 0.290.3

package/README.md

@@ -5870,6 +5870,8 @@ baker landing critique spring-offer summer-offer claude  # score three landings
 
 - **Positioning integrity is scored too, and it is the one copy rule that blocks on a single sentence.** `competitor-concession` fires when the page ranks somebody else above the client, or says the client does not lead at what the page sells ("su constructor es mejor que el nuestro", "we're not the best"). Every other prose rule waits for a cluster because one AI tell proves nothing; this one is the whole defect on its own — the page is paid traffic, and it is arguing the visitor should leave. Two shapes that read identically are exempt by design: a concession about **price**, in either direction ("we are not the cheapest / the most expensive, and here is why" is premium positioning), and a superlative flipped by a negation ("ningún equipo trabaja más rápido que el nuestro"), which is the strongest claim *for* the client. Its warn-tier sibling `competitor-named-in-comparison` fires when a name from `src/content/competitors/` appears in a comparative sentence outside a `<table>` — a comparison table is a section the landing skill teaches, but naming a rival anywhere else on paid traffic is the client's call, not the build's. Spanish, Portuguese and English.
 
+- **Comments are not markup, and no rule reads one.** Astro expression comments (`{/* … */}`) and HTML comments (`<!-- … -->`) are blanked before detection, offsets preserved so `file:line` stays exact. A shared scaffold component explaining its `preconnect` in prose that spelled `<img>` twice was read as two broken images — block-tier — so every landing rendering a video failed the publish gate on a file the client cannot edit. Comment prose is also where a component's reasoning lives, em-dashes and buzzwords included, so reading it inflated the count-threshold tells too.
+
 - **Copy held in component frontmatter is read like markup copy.** Astro landings keep FAQ pairs, testimonial quotes, feature cards and pricing rows as a `const items = [{ q, a }]` array above the `---` fence and render them in a `.map()`, so a critic that blanked the whole block was blind to a large share of every page's actual words. Prose string literals in that block now feed every copy rule; imports, class strings, URLs and config do not, and the bar is deliberately high in that direction — a Tailwind class list in front of the copy rules is worse than one missed tell.
 
 Output is the standard envelope `{ ok, data, hints }` with `data = { advisory, slug, overall, counts, dimensions, findings }`. `dimensions` scores seven design families (typography, color, borders_depth, motion, spacing, copy, integrity) 0–1 (higher is better); `counts` is the block/warn/advisory tally.

package/dist/cli.js

@@ -48982,6 +48982,20 @@ function blankAstroFrontmatter(text2) {
   }
   return kept.join("") + text2.slice(cut);
 }
+function blankUnrenderedSource(text2) {
+  return blankTemplateComments(blankAstroFrontmatter(text2));
+}
+function blankTemplateComments(text2) {
+  const out = text2.split("");
+  const pattern = /\{\s*\/\*[\s\S]*?\*\/\s*\}|<!--[\s\S]*?-->/g;
+  for (const m of text2.matchAll(pattern)) {
+    const start = m.index ?? 0;
+    for (let i = start; i < start + m[0].length && i < out.length; i++) {
+      if (out[i] !== "\n") out[i] = " ";
+    }
+  }
+  return out.join("");
+}
 function frontmatterLength(text2) {
   if (!text2.startsWith("---")) return 0;
   const lines = text2.split("\n");
@@ -49355,7 +49369,7 @@ function distinctCompetitorNames(competitors) {
   return [...byFold.values()];
 }
 function mentionsInSource(source, names) {
-  const text2 = blankAstroFrontmatter(source.text);
+  const text2 = blankUnrenderedSource(source.text);
   const visible = blankHtmlKeepingOffsets(text2);
   const tables = tableRanges(text2);
   const out = [];
@@ -49395,7 +49409,7 @@ function escapeRegExp(value) {
 }
 function detectSource(source) {
   const file = source.path;
-  const text2 = blankAstroFrontmatter(source.text);
+  const text2 = blankUnrenderedSource(source.text);
   const findings = [];
   const lines = text2.split("\n");
   for (const matcher of LINE_MATCHERS) {
@@ -49413,7 +49427,7 @@ function detectSource(source) {
 }
 function detectPage(sources) {
   if (sources.length === 0) return [];
-  const combined = sources.map((s) => blankAstroFrontmatter(s.text)).join("\n");
+  const combined = sources.map((s) => blankUnrenderedSource(s.text)).join("\n");
   const pageFile = sources.find((s) => s.path === "index.astro" || s.path.endsWith("/index.astro"))?.path ?? sources[0]?.path ?? "index.astro";
   const findings = [];
   for (const analyzer of ANALYZERS) {
@@ -49493,7 +49507,7 @@ function describeCounts(findings) {
 // src/commands/landing/snapshot.ts
 import { mkdir as mkdir9, rename as rename2, writeFile as writeFile13 } from "fs/promises";
 import path31 from "path";
-var CRITIC_VERSION = "3";
+var CRITIC_VERSION = "4";
 function critiqueCacheDir(projectRoot) {
   return path31.join(projectRoot, ".cache", "landing-critique");
 }