roast-my-design-system 4.2.4 → 4.2.6

package/bin/roast.mjs

@@ -12,7 +12,7 @@ import { spawnSync } from 'node:child_process';
 import { mkdtempSync, rmSync, readFileSync, existsSync, statSync } from 'node:fs';
 import { join, dirname, resolve } from 'node:path';
 import { tmpdir } from 'node:os';
-import { fileURLToPath } from 'node:url';
+import { fileURLToPath, pathToFileURL } from 'node:url';
 
 const HERE = dirname(fileURLToPath(import.meta.url));
 const SCRIPTS = join(HERE, '../skills/roast-my-design-system/scripts');
@@ -106,7 +106,7 @@ function run(script, args, env) {
 }
 const say = (s) => { if (!asJson) console.log(s); };
 
-say(`roast-my-design-system ${VERSION} · read-only scan, nothing leaves your machine\n`);
+say(`roast-my-design-system ${VERSION} · read-only scan, nothing leaves your machine`);
 // the harvest goes to a temp dir this wrapper deletes right after; tell the
 // script so it does not print a path that will be gone seconds later
 run('harvest/index.mjs', [target, '--out', harvestPath,
@@ -115,6 +115,19 @@ say('');
 run('diagnose/index.mjs', [harvestPath, '--out', outPath, '--theme', theme, '--summary', summaryPath,
   ...(commissionedBy ? ['--by', commissionedBy] : [])]);
 
+// The verdict leads, the evidence follows: harvest details print here, after
+// the diagnosis, rendered from harvest.json via the same lines the direct
+// harvest run uses.
+if (!asJson) {
+  try {
+    const harvestData = JSON.parse(readFileSync(harvestPath, 'utf8'));
+    const { detailLines } = await import(pathToFileURL(join(SCRIPTS, 'harvest/summary.mjs')).href);
+    console.log('');
+    for (const l of detailLines(harvestData)) console.log(l);
+    console.log(`\n  scanned in ${harvestData.tookMs}ms`);
+  } catch { /* details are garnish; the report exists either way */ }
+}
+
 const rulesPath = resolve(join(target, 'design-system-rules.md'));
 if (wantRules) {
   say('');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roast-my-design-system",
-  "version": "4.2.4",
+  "version": "4.2.6",
   "description": "Your AI can write the UI. This makes sure it writes your UI. A deterministic scanner counts every colour, spacing value and duplicate component, scores you 0-100 against 34 public repos, scopes the scan with .roastignore, injects agent rules with --apply.",
   "keywords": [
     "design-system",

package/skills/roast-my-design-system/scripts/diagnose/index.mjs

@@ -1324,5 +1324,5 @@ if (summaryPath) {
 
 console.log(`✓ Diagnosis for ${repoName}`);
 if (healthScore !== null) console.log(`  score: ${healthScore}/100`);
-console.log(`  verdict: ${verdict}`);
-console.log(`  → ${outPath}`);
+console.log(`\n  Verdict: ${verdict}`);
+console.log(`  Your free report is here → ${outPath}`);

package/skills/roast-my-design-system/scripts/harvest/index.mjs

@@ -15,6 +15,7 @@
 import { writeFileSync } from 'node:fs';
 import { resolve } from 'node:path';
 import { walkRepo, profileRepo } from './walk.mjs';
+import { headerLines, detailLines } from './summary.mjs';
 import { harvestComponents } from './components.mjs';
 import { harvestTokens } from './tokens.mjs';
 import { findDuplicates } from './duplicates.mjs';
@@ -157,47 +158,12 @@ harvest.tookMs = Date.now() - t0;
 writeFileSync(outPath, JSON.stringify(harvest, null, 2));
 
 // ---------- one-screen summary ----------
-const nonPage = components.filter((c) => !c.isPage);
-const hexColors = tokens.colors.filter((c) => c.value.startsWith('#'));
-// A truncated capture like "rgba(var(--ink-rgb)" reads as a glitch in the
-// terminal; the token reference inside it is the real story, so show that.
-const showVal = (v) => {
-  const m = /var\((--[A-Za-z0-9_-]+)/.exec(v);
-  const balanced = (v.match(/\(/g) ?? []).length === (v.match(/\)/g) ?? []).length;
-  return m && !balanced ? `var(${m[1]})` : v;
-};
-// A "top" list is only news when something repeats; all-×1 says nothing.
-const top = (list, n = 5) => list.some((e) => e.count > 1)
-  ? `   top: ${list.slice(0, n).map((e) => `${showVal(e.value)} ×${e.count}`).join(', ')}`
-  : (list.length ? ', none repeated' : '');
-
-console.log(`\nHarvest: ${profile.name ?? target}`);
-console.log(`  framework: ${profile.framework}${profile.typescript ? ' + TS' : ''}   design system: ${profile.designSystem.kind}${profile.designSystem.name ? ` (${profile.designSystem.name})` : ''}   styling: ${profile.stylingDeps.join(', ') || 'none detected'}`);
-console.log(`  files: ${files.code.length} code, ${files.styles.length} style`);
-if (exclusions.patterns.length) {
-  // Same voice as the report header: source named once, slashes on folders,
-  // and the total at the end because that is the number that lands.
-  const groups = [...new Set(exclusions.patterns.map((p) => p.source))].map((src) => {
-    const own = exclusions.patterns.filter((p) => p.source === src);
-    return `(${src}): ${own.map((p, i) => `${p.pattern}/ ${p.files}${i === 0 ? ' files' : ''}`).join(', ')}`;
-  });
-  const total = exclusions.patterns.reduce((sum, p) => sum + p.files, 0);
-  console.log(`  excluded by you ${groups.join(' · ')} · ${total} files kept out of this scan`);
-}
-console.log(`\n  components: ${components.length} defined (${nonPage.length} reusable, ${components.length - nonPage.length} pages)`);
-console.log(`  duplicates: ${duplicates.exactDuplicates.length} exact same-name, ${duplicates.families.length} name families`);
-for (const d of duplicates.exactDuplicates.slice(0, 3)) console.log(`    · ${d.name} defined in ${d.files.length} files`);
-for (const f of duplicates.families.slice(0, 3)) console.log(`    · ${f.root} family: ${f.members.map((m) => m.name).join(', ')}`);
-console.log(`\n  colours: ${tokens.colors.length} distinct (${hexColors.length} hex, of which ${tokens.greyCount} greys)${top(tokens.colors, 4)}`);
-console.log(`  spacing: ${tokens.spacing.length} distinct CSS values${top(tokens.spacing, 5)}`);
-console.log(`  radii: ${tokens.radii.length}   font sizes: ${tokens.fontSizes.length}   font families: ${tokens.fontFamilies.length}   shadows: ${tokens.shadows.length}`);
-console.log(`  tailwind: ${tokens.tailwind.colors.length} colour utils, ${tokens.tailwind.spacing.length} spacing utils, ${tokens.tailwind.textSizes.length} text sizes`);
-console.log(`  inline styles: ${tokens.inlineStyles.count} blocks`);
-console.log(`  context files: ${context.map((c) => c.file).join(', ') || 'none'}`);
-// The npx wrapper writes the harvest to a temp dir it deletes right after;
-// pointing the user at a path that will not exist is noise there.
-if (process.env.ROAST_EPHEMERAL_OUT === '1') {
-  console.log(`\n  scanned in ${harvest.tookMs}ms\n`);
-} else {
-  console.log(`\n  → ${outPath}   (${harvest.tookMs}ms)\n`);
+// The npx wrapper reorders the story (header, then the diagnosis, then these
+// details), so the lines live in summary.mjs and the wrapper prints details
+// itself from harvest.json. Run directly (the skill flow), everything prints
+// here in one go, with the real output path at the end.
+for (const l of headerLines(harvest)) console.log(l);
+if (process.env.ROAST_EPHEMERAL_OUT !== '1') {
+  for (const l of detailLines(harvest)) console.log(l);
+  console.log(`\n  → ${outPath}   (${harvest.tookMs}ms)`);
 }

package/skills/roast-my-design-system/scripts/harvest/summary.mjs

@@ -0,0 +1,61 @@
+/**
+ * Terminal summary lines for a harvest, split so the npx wrapper can change
+ * the running order: header first, diagnosis from step 2 next, details after.
+ * Running harvest/index.mjs directly (the skill flow) prints both in one go.
+ * Everything reads from the harvest object itself, so the wrapper can render
+ * details from harvest.json after the harvest process has exited.
+ */
+
+// A truncated capture like "rgba(var(--ink-rgb)" reads as a glitch in the
+// terminal; the token reference inside it is the real story, so show that.
+const showVal = (v) => {
+  const m = /var\((--[A-Za-z0-9_-]+)/.exec(v);
+  const balanced = (v.match(/\(/g) ?? []).length === (v.match(/\)/g) ?? []).length;
+  return m && !balanced ? `var(${m[1]})` : v;
+};
+
+// A "top" list is only news when something repeats; all-×1 says nothing.
+const top = (list, n = 5) => list.some((e) => e.count > 1)
+  ? `   top: ${list.slice(0, n).map((e) => `${showVal(e.value)} ×${e.count}`).join(', ')}`
+  : (list.length ? ', none repeated' : '');
+
+export function headerLines(h) {
+  const p = h.profile;
+  return [
+    `Harvest: ${p.name ?? h.repo}`,
+    `  → framework: ${p.framework}${p.typescript ? ' + TS' : ''}   design system: ${p.designSystem.kind}${p.designSystem.name ? ` (${p.designSystem.name})` : ''}   styling: ${p.stylingDeps.join(', ') || 'none detected'}`,
+  ];
+}
+
+export function detailLines(h) {
+  const lines = [];
+  const components = h.components ?? [];
+  const duplicates = h.duplicates ?? { exactDuplicates: [], families: [] };
+  const tokens = h.tokens;
+  const nonPage = components.filter((c) => !c.isPage);
+  const hexColors = tokens.colors.filter((c) => c.value.startsWith('#'));
+
+  lines.push(`  files: ${h.files.code} code, ${h.files.styles} style`);
+  if (h.exclusions?.patterns?.length) {
+    // Same voice as the report header: source named once, slashes on folders,
+    // and the total at the end because that is the number that lands.
+    const groups = [...new Set(h.exclusions.patterns.map((p) => p.source))].map((src) => {
+      const own = h.exclusions.patterns.filter((p) => p.source === src);
+      return `(${src}): ${own.map((p, i) => `${p.pattern}/ ${p.files}${i === 0 ? ' files' : ''}`).join(', ')}`;
+    });
+    lines.push(`  excluded by you ${groups.join(' · ')} · ${h.exclusions.filesExcluded} files kept out of this scan`);
+  }
+  lines.push('');
+  lines.push(`  components: ${components.length} defined (${nonPage.length} reusable, ${components.length - nonPage.length} pages)`);
+  lines.push(`  duplicates: ${duplicates.exactDuplicates.length} exact same-name, ${duplicates.families.length} name families`);
+  for (const d of duplicates.exactDuplicates.slice(0, 3)) lines.push(`    · ${d.name} defined in ${d.files.length} files`);
+  for (const f of duplicates.families.slice(0, 3)) lines.push(`    · ${f.root} family: ${f.members.map((m) => m.name).join(', ')}`);
+  lines.push('');
+  lines.push(`  colours: ${tokens.colors.length} distinct (${hexColors.length} hex, of which ${tokens.greyCount} greys)${top(tokens.colors, 4)}`);
+  lines.push(`  spacing: ${tokens.spacing.length} distinct CSS values${top(tokens.spacing, 5)}`);
+  lines.push(`  radii: ${tokens.radii.length}   font sizes: ${tokens.fontSizes.length}   font families: ${tokens.fontFamilies.length}   shadows: ${tokens.shadows.length}`);
+  lines.push(`  tailwind: ${tokens.tailwind.colors.length} colour utils, ${tokens.tailwind.spacing.length} spacing utils, ${tokens.tailwind.textSizes.length} text sizes`);
+  lines.push(`  inline styles: ${tokens.inlineStyles.count} blocks`);
+  lines.push(`  context files: ${(h.context ?? []).map((c) => c.file).join(', ') || 'none'}`);
+  return lines;
+}

package/skills/roast-my-design-system/scripts/lib/version.mjs

@@ -1,4 +1,4 @@
 // Single version constant for the engine — imported by diagnose (report
 // footer) and rules (generated-by line). This is the bump spot that used to
 // live as a const inside diagnose/index.mjs.
-export const VERSION = '4.2.4';
+export const VERSION = '4.2.6';