roast-my-design-system 4.2.3 → 4.2.5

package/bin/roast.mjs

@@ -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,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roast-my-design-system",
-  "version": "4.2.3",
+  "version": "4.2.5",
   "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

@@ -171,8 +171,8 @@ 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(`Harvest: ${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,
@@ -197,7 +197,7 @@ 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`);
+  console.log(`\n  scanned in ${harvest.tookMs}ms`);
 } else {
-  console.log(`\n  → ${outPath}   (${harvest.tookMs}ms)\n`);
+  console.log(`\n  → ${outPath}   (${harvest.tookMs}ms)`);
 }

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

@@ -160,10 +160,19 @@ export function profileRepo(root, files) {
   const knownLib = KNOWN_LIBRARIES.find((d) => deps[d.pkg]);
   const homegrown = files.code.filter((f) => /(^|\/)components\//.test(f) && !/\/components\/ui\//.test(f) && /\.(tsx|jsx)$/.test(f));
 
+  // A design system does not have to arrive through npm. A stylesheet defining
+  // a real set of CSS custom properties IS one — arguably the purest form —
+  // and calling it "none" undersells exactly the discipline the ideal asks for.
+  const tokenDefs = files.styles.slice(0, 8).reduce((sum, f) => {
+    const css = read(join(root, f));
+    return sum + (css ? (css.match(/--[A-Za-z0-9_-]+\s*:/g) ?? []).length : 0);
+  }, 0);
+
   let designSystem;
   if (isShadcn) designSystem = { kind: 'shadcn', name: 'shadcn/ui', confidence: 'high' };
   else if (knownLib) designSystem = { kind: 'library', name: knownLib.name, pkg: knownLib.pkg, confidence: 'high' };
   else if (homegrown.length >= 3) designSystem = { kind: 'custom', name: 'custom (unrecognized)', confidence: 'low' };
+  else if (tokenDefs >= 5) designSystem = { kind: 'custom', name: 'CSS tokens', confidence: 'medium' };
   else designSystem = { kind: 'none', confidence: 'high' };
 
   // Import alias from tsconfig paths or components.json
@@ -175,7 +184,10 @@ export function profileRepo(root, files) {
     if (key) alias = key.replace('/*', '');
   }
 
-  const styling = STYLING_DEPS.filter((s) => deps[s.pkg]).map((s) => s.label);
+  // No toolchain in package.json does not mean no styling: if the walk found
+  // real stylesheets, plain CSS is what is there — say so, not "none detected".
+  let styling = STYLING_DEPS.filter((s) => deps[s.pkg]).map((s) => s.label);
+  if (!styling.length && files.styles.length) styling = ['plain CSS'];
 
   // The git remote is a far better identity than package.json's name field
   // ("chatbot" vs "vercel/ai-chatbot"). Parsed from .git/config, no git exec.

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.3';
+export const VERSION = '4.2.5';