@struq/skill 0.4.1 → 0.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@struq/skill",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Install the free Struq skill into Claude Code, Cursor, Codex and other AI coding tools: websites at the level of a design studio, built by your own AI.",
   "license": "MIT",
   "author": "Struq (https://struq.nl)",

package/skills/struq/references/tells.md

@@ -45,6 +45,9 @@ connected); the rest are judged by eye. The fix is beside each.
 - Emoji and the cliche icons (sparkles, rocket, lightning, bulb, wand). Fix: Struq Icons, functional only. (check)
 - Invented facts: testimonials, client logos, stats, awards, prices, addresses, hours, bios, quotes the brief did not
   give. Fix: leave them out and list them under Assumptions.
+- **Notes to the client on the page:** "het e-mailadres volgt zodra de studio het aanlevert", "coming soon", "[telefoon]",
+  "to be confirmed". Honest, but a visitor reads it as an unfinished site. Fix: leave the missing fact out of the page
+  (drop the line, the column or the link) and list it under Assumptions in STRUQ.md. (check)
 
 ## Judged by eye
 

package/skills/struq/scripts/check.mjs

@@ -154,6 +154,22 @@ const plan = read("STRUQ.md");
   }
   if (brokenPhotos.length) add(G, "FAIL", "These image files are not real images (an error page saved as .webp?). Delete them and run images.mjs again.", brokenPhotos.map(rel));
 
+  // A file whose size differs from the one images.mjs recorded is not the photo its alt and caption
+  // describe: a run of another pack, or a hand copy, left it under the same name.
+  const manifestFile = join(pub, "images", "struq-pack.json");
+  if (existsSync(manifestFile)) {
+    let recorded = [];
+    try {
+      recorded = JSON.parse(readFileSync(manifestFile, "utf8")).images ?? [];
+    } catch {
+      add(G, "WARN", "public/images/struq-pack.json is not readable JSON. Run images.mjs again.");
+    }
+    const mismatched = recorded
+      .filter((m) => typeof m.bytes === "number" && existsSync(join(root, m.file)) && statSync(join(root, m.file)).size !== m.bytes)
+      .map((m) => m.file);
+    if (mismatched.length) add(G, "FAIL", "These photos are not the ones their alt and caption describe (left by an earlier pack or copied by hand). Run images.mjs again with the pack you chose.", mismatched);
+  }
+
   // Each pack reference (images.card[2], images.card.slice(0, 5)) or own image tag counts once.
   const used = own.reduce((n, f) => n + (f.text.match(/images\.(hero|band|card|column|detail|backdrop)\b|<img\b|src=["'{`]\/images\//g) ?? []).length, 0);
   const total = Object.values(slotCount).reduce((a, b) => a + b, 0);
@@ -217,6 +233,11 @@ const plan = read("STRUQ.md");
   };
   const ownOnly = { scope: own };
 
+  // Copy
+  tell("Notes to the client on the page",
+    /volg(?:t|en) zodra|zodra (?:de studio|de klant|je|jullie|we)\b[^"'<>]{0,40}(?:aanlever|doorgeef|doorgeven|binnen)|nog aan te leveren|wordt nog aangevuld|lorem ipsum|\bcoming soon\b|to be confirmed|\bTBC\b|\[(?:naam|adres|telefoon|e-?mail|name|phone|address)\]/i,
+    "a line written to the client or the developer is not copy. Leave a missing fact out of the page and list it under Assumptions: in STRUQ.md.");
+
   // Type and labels
   tell("Eyebrows and uppercase labels", /\buppercase\b|text-transform:\s*uppercase|\b(eyebrow|kicker|overline)\b/i,
     "small uppercase labels above headings, uppercase marquees and tracked caps are AI tells. Delete the label; the heading, a hairline rule or a jump in scale does the wayfinding. A deliberate all-caps wordmark passes when its element carries data-wordmark.",

package/skills/struq/scripts/images.mjs

@@ -18,8 +18,10 @@
 // 4. If struq.nl cannot be reached, or a file fails its check, it writes a graded fallback plate
 //    in the site palette for that slot instead, so no frame is ever empty, and says so.
 //
-// Node 18 or later, no dependencies. Safe to run again: valid files are kept.
-import { existsSync, mkdirSync, readFileSync, statSync, writeFileSync } from "node:fs";
+// Node 18 or later, no dependencies. Safe to run again: a file is kept only when the last run
+// downloaded it from the same source (struq-pack.json records each one); a file from another pack
+// is replaced, and a slot file the new pack no longer fills is removed.
+import { existsSync, mkdirSync, readdirSync, readFileSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { join, resolve } from "node:path";
 import { appendLog } from "./log.mjs";
 import { fitImages, packForBrief, packReason } from "./packs.mjs";
@@ -201,6 +203,28 @@ ${body}
   return file;
 }
 
+/** file -> source from the last run's struq-pack.json; empty when there was no run or it is unreadable. */
+function readManifest(outDir) {
+  try {
+    const data = JSON.parse(readFileSync(join(outDir, "struq-pack.json"), "utf8"));
+    return new Map((data.images ?? []).map((i) => [i.file, i.source]));
+  } catch {
+    return new Map();
+  }
+}
+
+/**
+ * Removes the slot files (hero-1.webp, card-3.svg, ...) this run did not write, so a photo or plate
+ * from an earlier pack can never sit under this pack's caption. Other files in public/images stay.
+ */
+function removeStale(outDir, manifest) {
+  const written = new Set(manifest.map((m) => m.file.split("/").pop()));
+  const slotFile = new RegExp(`^(${SLOTS.join("|")})-\\d+\\.(webp|svg)$`);
+  for (const name of readdirSync(outDir)) {
+    if (slotFile.test(name) && !written.has(name)) unlinkSync(join(outDir, name));
+  }
+}
+
 async function list() {
   const index = await getJson(API);
   console.log(`${index.packCount} packs, ${index.totalImages} images, ${index.license}\n`);
@@ -220,6 +244,7 @@ async function main() {
   const outDir = join(root, "public", "images");
   mkdirSync(outDir, { recursive: true });
   const palette = readPalette();
+  const previous = readManifest(outDir);
 
   let pack = null;
   let droppedImages = [];
@@ -268,7 +293,9 @@ async function main() {
       while (cursor < jobs.length) {
         const job = jobs[cursor++];
         const target = join(outDir, `${job.name}.webp`);
-        const ok = fileIsImage(target) || (await download(imageUrl(job.img), target));
+        const source = imageUrl(job.img);
+        const kept = previous.get(`public/images/${job.name}.webp`) === source && fileIsImage(target);
+        const ok = kept || (await download(source, target));
         if (ok) {
           real++;
           entries.push({
@@ -282,7 +309,8 @@ async function main() {
             description: job.img.description ?? "",
             negativeSpace: job.img.negativeSpace ?? "",
           });
-          manifest.push({ file: `public/images/${job.name}.webp`, id: job.img.id, slot: job.img.slot, source: imageUrl(job.img) });
+          // bytes lets check.mjs tell this file from one an earlier pack left under the same name.
+          manifest.push({ file: `public/images/${job.name}.webp`, id: job.img.id, slot: job.img.slot, source, bytes: statSync(target).size });
         } else {
           plates++;
           const size = [job.img.width ?? FALLBACK_SIZE[job.img.slot][0], job.img.height ?? FALLBACK_SIZE[job.img.slot][1]];
@@ -326,6 +354,7 @@ async function main() {
   }
 
   entries.sort((a, b) => SLOTS.indexOf(a.slot) - SLOTS.indexOf(b.slot) || a.order - b.order);
+  removeStale(outDir, manifest);
   writeFileSync(join(outDir, "struq-pack.json"), JSON.stringify({ pack: slug, images: manifest }, null, 2) + "\n");
   const module = writeModule(entries, slug, !pack, imageFit);