roast-my-design-system 7.1.0 → 7.2.0

package/README.md

@@ -1,4 +1,4 @@
-<img src="https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/roaster_logo_300px.png?v=7.1.0" width="72" alt="roast-my-design-system">
+<img src="https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/roaster_logo_300px.png?v=7.2.0" width="72" alt="roast-my-design-system">
 
 # roast-my-design-system
 
@@ -10,6 +10,8 @@
 
 A free CLI tool (and Claude Code skill) that roasts your repo's design system with real data, then generates the rules that keep your AI agent on-system.
 
+> **New in 7.2: shadcn repos are read as shadcn repos.** The scanner finds the installed catalogue where it actually lives, reads the theme file `components.json` names, and prints how it decided under the report header. 2 new tiles appear on shadcn repos only, both from shadcn's own rules for agents: palette colours in your own code where a theme row exists, and kit components repainted through `className`. Counted per 100 of your own files, never inside the catalogue. The rules file gains a shadcn section. Scores on other repos do not move.
+
 > **New in 7.0.** The scoring code is now a separate file that other tools can import, so a CI check gets the same score as the report. Every `summary.json` now records its schema version and which benchmark it was scored against. Five counting bugs are fixed: a bracket spacing class like `p-[13px]` was counted twice, a hex colour inside a CSS comment was counted, and three smaller ones. Most repos lose a few counts. shadcn/ui goes from 325 colours to 323 and keeps its score of 65. Large monorepos scan about ten times faster. If you use the score as a CI threshold, run a fresh scan before comparing. If you read the JSON, a tile's `value` is now a number and the printed text is in `display`.
 
 > **New in 6.0: a dark mode is not sprawl, and the ruler has been rebuilt.** A system with a dark theme states most of its colours twice. One with a density switch states its spacing twice again. Every restatement was counted, so the systems doing the most work scored the worst. Now only a token's first statement counts. A factory-fresh shadcn install goes from 20 colours to 16, and Shoelace from 420 to 219. Two related faults went with it. `border-radius: var(--radius)` was counted as a radius value, so Polaris showed 49 radii of which 36 were references to its own tokens, and Telekom showed 52 font sizes of which 48 were. A new finding names one token holding two different colours in two packages of a monorepo, which fires 8 times across 19 real repos. The benchmark was rebuilt with all of it, so every score is measured against a ruler that counts the same way: cal.com 15 to 20, Cloudscape 35 to 40, MUI 50 to 55. If you have a score from 5.x, take a fresh one before you compare.
@@ -73,11 +75,11 @@ Not to be confused with each other: **"Why this matters"** is generic, ships wit
 
 The full report for vercel/ai-chatbot, top to bottom, including "What the numbers mean", Claude's read of the scan, embedded right under the verdict:
 
-![The full diagnosis report for vercel/ai-chatbot in dark mode: health score, the What the numbers mean analysis written by Claude, priced Where to start moves each with its copy-the-fix-prompt button, the wrapped present with the agent rules, an agent trap callout, 3-yardstick tiles, the adoption map treemap, palette forensics, spacing receipts, typography specimens, offenders, duplicates, and the component usage ledger](https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/report-full-dark.png?v=7.1.0)
+![The full diagnosis report for vercel/ai-chatbot in dark mode: health score, the line saying how the repo was read as a shadcn install, the What the numbers mean analysis written by Claude, priced Where to start moves each with its copy-the-fix-prompt button, the wrapped present with the agent rules, an agent trap callout, 3-yardstick tiles including the 2 shadcn tiles, the adoption map treemap, palette forensics, the shadcn theme variable by variable, spacing receipts, typography specimens, offenders, duplicates, and the component usage ledger](https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/report-full-dark.png?v=7.2.0)
 
 The same report in light mode (one file, built-in toggle):
 
-![The diagnosis report in light mode](https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/report-light-hero.png?v=7.1.0)
+![The diagnosis report in light mode](https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/report-light-hero.png?v=7.2.0)
 
 ## What makes the numbers trustworthy
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roast-my-design-system",
-  "version": "7.1.0",
+  "version": "7.2.0",
   "mcpName": "io.github.gregkozakiewicz/roast-my-design-system",
   "description": "Your AI can write the UI. This makes sure it writes your UI. A deterministic scanner scores your design system 0-100 against 34 public repos, reads React and web components (Stencil, Lit), hands you a copy-paste fix prompt for each top finding, writes rules for Claude, Cursor, Copilot and Windsurf with --apply, and runs as a local MCP server with --mcp.",
   "keywords": [

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

@@ -32,14 +32,11 @@ import { fixPrompt } from '../lib/fixprompt.mjs';
 import { WHY } from './why.mjs';
 import { parseColor, luminance, isGrey } from '../lib/color.mjs';
 import { loadBenchmark, benchHelpers, makeHealthOf, coreMetrics, tileHealths, scoreOfTiles, scorePackage as scorePackageOf, ZERO_IDEAL, WARN_TOLERANCE, SCORE_OF, SCHEMA_VERSION } from './score.mjs';
-import { profileOf } from '../profiles/index.mjs';
+import { profileOf, profileYardstick } from '../profiles/index.mjs';
 
 // The benchmark and every judgement made against it live in score.mjs; this
 // file only draws. The same numbers reach summary.json through scoreHarvest.
 const bench = loadBenchmark();
-const B = benchHelpers(bench);
-const { percentile, cleanerPct, ideal, median, displayAvg, refMedian } = B;
-const healthOf = makeHealthOf(B);
 
 function arg(name, fallback) {
   const i = process.argv.indexOf(`--${name}`);
@@ -50,6 +47,11 @@ if (!inPath) { console.error('Usage: node src/diagnose/index.mjs <harvest.json>
 const outPath = resolve(arg('out', 'diagnosis.html'));
 
 const h = JSON.parse(readFileSync(inPath, 'utf8'));
+// The yardstick: the general benchmark, plus whatever the repo's kind brings
+// for the tiles only it measures (the shadcn card owns 2).
+const B = benchHelpers(bench, profileYardstick(h));
+const { percentile, cleanerPct, ideal, median, displayAvg, refMedian } = B;
+const healthOf = makeHealthOf(B);
 const M = coreMetrics(h);
 
 // The GK mark (32px favicon, also gregkozakiewicz.com). Reused in the footer
@@ -336,6 +338,7 @@ const FALLBACK_TARGET = {
   colors: 'a system needs ~24', greys: 'a scale has up to 13', spacing: 'a dozen deliberate exceptions',
   exactDuplicates: 'should be 0', inlineStyles: 'invisible to any system', nearPairs: 'copy-paste, not decisions',
   important: 'the cascade admitting defeat', neverImported: 'the system nobody found', arbitrary: 'a handful of deliberate exceptions',
+  paintTin: 'the theme file already has a variable for it', doorOverrides: 'use a variant instead',
 };
 // Dress one judged tile for the page: formatted number, comparison rows.
 // Accepts a judged tile from score.mjs, or the positional form the
@@ -367,6 +370,9 @@ function tile(t, pLabel, pMetric, pFallback) {
       ? 'library: internal use only, downstream consumers invisible'
       : 'catalogue stock: installed by the shadcn CLI, not used yet', val: '', dir: '' });
   }
+  if (health === 'info' && metric === 'paintTin') {
+    rows.push({ label: 'utility-class mode: the palette is the theme by design, not counted', val: '', dir: '' });
+  }
   return { num: n(value), label, health, rows, metric, healthValue };
 }
 const judgedTiles = tileHealths(M, healthOf);
@@ -574,6 +580,42 @@ function spacingBars() {
     ${whyToggle('spacing')}${arb}</section>`;
 }
 
+// The shadcn sheet and the 2 paint checks: receipts, with the files and the
+// exact classes. Only a shadcn kitchen renders this.
+function sheetSection() {
+  if (!P.isShadcn) return '';
+  const sc = P.shadcn ?? {};
+  const sheet = sc.sheet, paint = sc.paint;
+  const parts = [];
+  if (sheet?.found) {
+    const bits = [`${n(sheet.lightRows)} variables for light, ${n(sheet.darkRows)} for dark in ${esc(sheet.file)}`];
+    if (sheet.hslEra) bits.push('written in the Tailwind 3 form (hsl triplets)');
+    if (sheet.shadcnMissing?.length) bits.push(`${sheet.shadcnMissing.length} of the ${SHADCN_ROW_COUNT} current shadcn variables not defined (${sheet.shadcnMissing.slice(0, 4).map((r) => `--${esc(r)}`).join(', ')}${sheet.shadcnMissing.length > 4 ? '…' : ''})${sheet.hslEra ? ', normal for an install from before the chart and sidebar rows existed' : ''}`);
+    if (sheet.missingDark?.length) bits.push(`${sheet.missingDark.length} variable${sheet.missingDark.length === 1 ? '' : 's'} with no dark value (${sheet.missingDark.slice(0, 4).map((r) => `--${esc(r)}`).join(', ')})`);
+    if (sheet.custom?.length) bits.push(`${sheet.custom.length} custom variable${sheet.custom.length === 1 ? '' : 's'} of your own (${sheet.custom.slice(0, 5).map((r) => `--${esc(r)}`).join(', ')}${sheet.custom.length > 5 ? '…' : ''})${sheet.customMissingDark?.length ? `, ${sheet.customMissingDark.length} of them light only` : ''}${sheet.customUnregistered?.length ? `, ${sheet.customUnregistered.length} never mapped in @theme inline` : ''}`);
+    if (sheet.tweakcnPresent >= 10) bits.push('tweakcn variables present: shadows and letter-spacing are themed');
+    if (sheet.spacingChanged) bits.push(`<b>--spacing is ${esc(sheet.spacing)}</b>, not the default 0.25rem: this resizes every gap in the app at once, which shadcn\'s own changelog says never to do`);
+    parts.push(`<div class="receipts">${eyebrow('the theme file, variable by variable')}<p class="sub">${bits.join(' · ')}.</p></div>`);
+  } else if (sheet) {
+    parts.push(`<div class="receipts">${eyebrow('the theme')}<p class="sub">components.json names ${esc(sheet.file)} as the theme file, and it was not found.</p></div>`);
+  }
+  if (paint) {
+    const tinChips = (paint.tin.samples ?? []).slice(0, 8).map((s) => `<span class="vchip bad">${esc(s.value)} ×${s.count}</span>`).join('');
+    const tinFiles = (paint.tin.top ?? []).slice(0, 5).map((f) => `<span class="vchip" title="${esc(f.file)}">${esc(basename(f.file))} ×${f.count}</span>`).join('');
+    parts.push(`<div class="receipts">${eyebrow(`${n(paint.tin.uses)} colours from outside the theme in ${n(paint.tin.files)} of ${n(paint.ownFiles)} own files · ${n(paint.tin.per100)} per 100 files`)}${paint.tin.uses ? `<div class="chips-row">${tinChips}</div><div class="chips-row">${tinFiles}</div>` : '<p class="sub">Your own code takes every colour from the theme file. This is what shadcn is designed for.</p>'}</div>`);
+    parts.push(whyToggle('paintTin'));
+    const doorChips = (paint.doors.samples ?? []).slice(0, 6).map((s) => `<span class="vchip bad">${esc(s.value)} ×${s.count}</span>`).join('');
+    parts.push(`<div class="receipts">${eyebrow(`${n(paint.doors.uses)} shadcn components restyled through className · ${n(paint.doors.per100)} per 100 files`)}${paint.doors.uses ? `<div class="chips-row">${doorChips}</div>` : '<p class="sub">No shadcn component is given a colour or a font through className. Variants are doing their job.</p>'}</div>`);
+    parts.push(whyToggle('doorOverrides'));
+  }
+  if (!parts.length) return '';
+  return `<section class="glass pad" id="shadcn" style="margin-top:16px">
+    ${sectionHead('The shadcn theme and the 2 shadcn checks', 'shadcn gives you a theme file of named variables and a set of components with variants. These receipts show where your own code went around both.')}
+    ${parts.join('')}
+  </section>`;
+}
+const SHADCN_ROW_COUNT = 33;
+
 function duplicatesSection() {
   if (!exactDupes.length && !families.length && !iconCollisions.length) return '';
   const iconCard = iconCollisions.length ? `
@@ -1020,6 +1062,22 @@ function whereToStartSection() {
       sub: `${neverImported.slice(0, 3).map((x) => `&lt;${esc(x.name)}&gt;`).join(', ')}${neverImported.length > 3 ? ' and others' : ''} sit in the system with no callers. Adopt them or delete them: either answer is better than a system with rooms nobody enters.` });
   }
 
+  if (P.isShadcn && P.shadcn?.paint) {
+    const pt = P.shadcn.paint;
+    if (pt.tin.uses >= 10) {
+      const s0 = pt.tin.samples[0], f0 = pt.tin.top[0];
+      c.push({ score: 20 + pt.tin.per100 / 4, metric: 'paintTin', after: 0,
+        title: `Repaint the ${n(pt.tin.uses)} colours from outside the theme`,
+        sub: `${esc(s0.value)} appears ${s0.count} times${f0 ? `, ${esc(basename(f0.file))} alone carries ${f0.count}` : ''}. The theme file already has a variable for each (a grey is <code>text-muted-foreground</code>, a status colour is a Badge variant or a variable you add). Swap the class, not the value.` });
+    }
+    if (pt.doors.uses >= 5) {
+      const s0 = pt.doors.samples[0];
+      c.push({ score: 15 + pt.doors.per100 / 4, metric: 'doorOverrides', after: 0,
+        title: `Stop restyling shadcn components through className`,
+        sub: `${n(pt.doors.uses)} shadcn components receive a colour or a font through className, like ${esc(s0.value)}. Use the variant that does it, or add one to the component you own. className on a shadcn component is for layout only.` });
+    }
+  }
+
   // What each move is actually worth, then rank by payoff for real.
   for (const item of c) {
     item.delta = 0; item.target = null;
@@ -1171,7 +1229,7 @@ function agentSection() {
 // quiet unrecognised chip, because not knowing IS a finding here.
 const ns = h.tokens.namespaces ?? null;
 const dsChip =
-  ds.kind === 'shadcn' ? (ds.cssVariables === false ? 'shadcn/ui (utility classes, no CSS variables)' : 'shadcn/ui')
+  ds.kind === 'shadcn' ? `shadcn/ui${ds.cssVariables === false ? ' (utility classes, no CSS variables)' : ''}${P.shadcn?.kit?.style ? ` · ${P.shadcn.kit.style}` : ''}`
   : ds.kind === 'library' ? ds.name
   : ns ? `custom design system (--${ns.primary}-*${ns.partner ? ` + --${ns.partner}-*` : ''})`
   // A tokenFile alone is a technicality (Lion's is one drawer style file);
@@ -1190,6 +1248,24 @@ const stack = [
 const dsUnrecognised = !dsChip && !noSystemLikely;
 const legacyChip = ns?.others?.length ? `also present: ${ns.others.map((l) => `--${l}-*`).join(', ')}` : null;
 
+// The recognition receipt: how the scanner decided this is a shadcn kitchen
+// and what kit it read. Printed under the chips, never hidden, so a wrong
+// guess is visible before any number is.
+function shadcnReceipt() {
+  if (!P.isShadcn) return '';
+  const k = P.shadcn?.kit ?? null;
+  const facts = [
+    k?.theme ? `accent ${k.theme}` : null,
+    k?.chartColor ? `charts ${k.chartColor}` : null,
+    k?.radius ? `radius ${k.radius}` : null,
+    k?.iconLibrary ? `icons ${k.iconLibrary}` : null,
+    k && k.cssVariables === false ? 'utility-class mode: the palette is the theme' : null,
+  ].filter(Boolean);
+  const unmatched = (k?.fellBack ?? []).filter((f) => f === 'theme' || f === 'chartColor');
+  const fell = unmatched.length ? ` · ${unmatched.length === 2 ? 'accent and chart colour' : unmatched[0] === 'theme' ? 'accent' : 'chart colour'} not matched to a named shadcn theme` : '';
+  return `<div class="excl">Read as a shadcn install (${esc(P.confidence ?? 'medium')} confidence): ${esc(P.evidence.join(' · '))}${facts.length ? ` · ${esc(facts.join(', '))}` : ''}${esc(fell)}</div>`;
+}
+
 // User exclusions are printed in the header, never hidden: a scoped scan must
 // say it is scoped, or the score could be quietly gamed. Grouped by source
 // (.roastignore vs --exclude), with the total number of files kept out.
@@ -1612,6 +1688,7 @@ const html = `<!doctype html>
     ${healthScore !== null ? `<div class="score${noSystemLikely ? ' muted' : ''}">${eyebrow('Health score')}<div class="val">${healthScore}<span class="slash">/</span><span class="of">100</span></div>${noSystemLikely ? '<div class="note">little here to score · see the note below</div>' : ''}</div>` : ''}
   </div>
   <div class="chips">${stack.map((s) => `<span class="chip">${esc(s)}</span>`).join('')}${dsUnrecognised ? '<span class="chip chip-dim">design system: unrecognised</span>' : ''}${legacyChip ? `<span class="chip chip-dim">${esc(legacyChip)}</span>` : ''}${agentFiles.map((c) => `<span class="chip chip-agent">${esc(c.file)}</span>`).join('')}</div>
+  ${shadcnReceipt()}
   ${exclusionsLine()}
   ${noSystemLikely ? `<div class="nods">${ICONS.warn}<span>There is most likely <b>no design system in this repo</b>: almost no colour or spacing values were found. Styling may live outside this codebase (CDN stylesheets, a parent repo, or generated output).</span></div>` : ''}
   <div class="glass verdict-card">
@@ -1638,6 +1715,7 @@ ${trapsBlock()}
 ${packagesSection()}
 
 <section style="margin-top:16px">${paletteSection()}</section>
+${sheetSection()}
 
 ${spacingBars()}
 ${typographySection()}
@@ -1769,7 +1847,9 @@ if (summaryPath) {
     score: healthScore,
     noSystemLikely,
     verdict,
-    role: P.kind,
+    role: P.role,
+    kind: P.kind,
+    ...(P.isShadcn && P.shadcn ? { shadcn: { confidence: P.confidence, evidence: P.evidence, style: P.shadcn.kit?.style ?? null, baseColor: P.shadcn.kit?.baseColor ?? null, tailwind: P.shadcn.kit?.tailwind ?? null, catalogues: P.uiDirs, ownFiles: P.shadcn.paint?.ownFiles ?? null } } : {}),
     componentsMeasured,
     metrics: (({ colors, colorTokens, colorStrays, greys, greyStrays, spacing, exactDuplicates, inlineStyles, nearPairs, important, neverImported, arbitrary, tokenLed }) =>
       ({ colors, colorTokens, colorStrays, greys, greyStrays, spacing, exactDuplicates, inlineStyles, nearPairs, important, neverImported, arbitrary, tokenLed }))(M),

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

@@ -26,7 +26,7 @@ import { isGrey } from '../lib/color.mjs';
 import { nearColorPairs } from '../lib/nearpairs.mjs';
 import { neverImportedComponents } from '../lib/neverimported.mjs';
 import { SCHEMA_VERSION } from '../lib/version.mjs';
-import { profileOf } from '../profiles/index.mjs';
+import { profileOf, profileYardstick } from '../profiles/index.mjs';
 
 export { SCHEMA_VERSION };
 
@@ -37,8 +37,20 @@ export function loadBenchmark() {
   return existsSync(BENCH_PATH) ? JSON.parse(readFileSync(BENCH_PATH, 'utf8')) : null;
 }
 
-/** The yardstick readers for one benchmark object (null-safe). */
-export function benchHelpers(bench) {
+/**
+ * The yardstick readers for one benchmark object (null-safe). A profile may
+ * bring its own ideals and fleet stats for the metrics only it measures
+ * (question 6 of the profile plan); they overlay the general benchmark and
+ * never replace an entry the benchmark already has.
+ */
+export function benchHelpers(bench, yardstick = null) {
+  if (yardstick && (Object.keys(yardstick.ideals ?? {}).length || Object.keys(yardstick.stats ?? {}).length)) {
+    bench = {
+      ...(bench ?? {}),
+      ideal2026: { ...(yardstick.ideals ?? {}), ...(bench?.ideal2026 ?? {}) },
+      stats: { ...(yardstick.stats ?? {}), ...(bench?.stats ?? {}) },
+    };
+  }
   // where does this value sit among the scanned fleet? ("more colours than 90%")
   const percentile = (metric, value) => {
     const vals = bench?.stats?.[metric]?.values;
@@ -89,6 +101,15 @@ export function makeHealthOf(b) {
   };
 }
 
+/** Tiles a kind of repo adds to the nine. Only the shadcn card has any today. */
+export const PROFILE_TILES = {
+  shadcn: [
+    ['paintTin', 'off-theme colours per 100 files'],
+    ['doorOverrides', 'components restyled from outside per 100 files'],
+  ],
+};
+export const tilesFor = (kind) => [...TILES, ...(PROFILE_TILES[kind] ?? [])];
+
 /** The tiles, in report order: metric key, the label the report prints. */
 export const TILES = [
   ['colors', 'distinct colours'],
@@ -140,6 +161,13 @@ export function coreMetrics(h) {
     componentsMeasured: profileOf(h).componentsMeasured,
     isLibrary: profileOf(h).isLibrary,
     vendoredUi: profileOf(h).vendoredUi,
+    // the kind, and the tiles only that kind measures (per 100 own-code files)
+    kind: profileOf(h).kind,
+    // utility-class mode (components.json cssVariables: false): the palette IS
+    // the theme by design, so a palette class is not paint from a tin
+    utilityPalette: profileOf(h).designSystem?.cssVariables === false,
+    paintTin: profileOf(h).shadcn?.paint?.tin?.per100 ?? 0,
+    doorOverrides: profileOf(h).shadcn?.paint?.doors?.per100 ?? 0,
   };
 }
 
@@ -152,15 +180,18 @@ export function tileHealths(m, healthOf) {
     colors: m.colors, greys: m.greys, spacing: m.spacing, exactDuplicates: m.exactDuplicates,
     inlineStyles: m.inlineStyles, nearPairs: m.nearPairs, important: m.important,
     neverImported: m.neverImported, arbitrary: m.arbitrary,
+    paintTin: m.paintTin ?? 0, doorOverrides: m.doorOverrides ?? 0,
   };
   const judged = { ...shown, colors: m.tokenLed ? m.colorStrays : m.colors, greys: m.tokenLed ? m.greyStrays : m.greys };
-  return TILES.map(([metric, label]) => {
+  return tilesFor(m.kind ?? 'product').map(([metric, label]) => {
     let health = healthOf(metric, judged[metric]);
     let value = shown[metric], healthValue = judged[metric];
     if (!m.componentsMeasured && (metric === 'exactDuplicates' || metric === 'neverImported')) {
       health = 'na'; value = null; healthValue = null;
     } else if (metric === 'neverImported' && (m.isLibrary || (m.vendoredUi && m.neverImported > 0))) {
       health = 'info';
+    } else if (metric === 'paintTin' && m.utilityPalette) {
+      health = 'info';
     }
     return { metric, label, value, healthValue, health };
   });
@@ -199,7 +230,7 @@ export function scorePackage(m, healthOf, b) {
 
 /** The whole judgement of one harvest, as data. */
 export function scoreHarvest(h, bench = loadBenchmark()) {
-  const b = benchHelpers(bench);
+  const b = benchHelpers(bench, profileYardstick(h));
   const healthOf = makeHealthOf(b);
   const metrics = coreMetrics(h);
   const tiles = tileHealths(metrics, healthOf);

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

@@ -9,6 +9,17 @@
 // and must move together with benchmark.json if the norms are ever re-tuned.
 
 export const WHY = {
+  // DRAFT for Greg's approval (2026-09-12): the 2 shadcn tiles.
+  paintTin: [
+    'shadcn gives every project a theme file. It holds one CSS variable per job: background, foreground, muted text, border, ring, and about 30 more. Each has a light value and a dark value. Components use them through classes like bg-background and text-muted-foreground.',
+    'A palette class such as text-gray-500 or bg-blue-100 skips that file. It works on the day. It stops working at the first theme change: the variables move, the palette colour stays, and the page shows 2 designs at once. Dark mode is where it shows first. A variable carries both values; a palette class carries one, so someone adds dark:bg-gray-900 next to it, and now there are 2 places to keep in step. An AI agent reading that file copies the pair.',
+    'The ideal is 25 per 100 files. That is where the tidiest third of 15 shadcn repos in the benchmark sit; the median is 62. shadcn\'s own rules for agents say it in 1 line: use semantic colours, never bg-blue-500.',
+  ],
+  doorOverrides: [
+    'A shadcn component ships with variants: outline, ghost, destructive, small, large. The variant decides how the component looks, once, in the component file you own. Passing bg-blue-100 or font-bold through className decides it again at the call site, where the component can not see it.',
+    'Do this in 5 places and 1 button has 6 designs. The next person can not tell which one is intended, and an agent copies whichever it finds first. shadcn\'s guidance gives 3 routes: use a variant that exists, add a variant to the component, or add a variable to the theme file. className is for layout: width, margin, position.',
+    'The ideal is 15 per 100 files, the tidiest third of the same 15 repos; the median is 22.',
+  ],
   colors: [
     'Look, nobody consciously chooses 100 colours to start with. They were added with each change, and the change after that. No one was checking for the drift, because no one expected it. Then come the multipliers: a rebrand, a dark-mode pass, ad hoc files, each one multiplying the values again.',
     'When an agent looks for Brand Blue it scans the repo, sees dozens of blues and logically picks the one used most often, which is how the most-used stray outvotes the actual token.',

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

@@ -27,7 +27,8 @@ import { ruleStaleness } from '../lib/staleness.mjs';
 import { neverImportedComponents } from '../lib/neverimported.mjs';
 import { lastTouchedDates } from '../lib/lasttouched.mjs';
 import { SCHEMA_VERSION } from '../lib/version.mjs';
-import { decideProfile } from '../profiles/index.mjs';
+import { decideProfile, profileOf } from '../profiles/index.mjs';
+import { countPaint } from './paint.mjs';
 
 function arg(name, fallback) {
   const i = process.argv.indexOf(`--${name}`);
@@ -57,6 +58,22 @@ const exclusions = loadExclusions(target, argAll('exclude'));
 const files = walkRepo(target, 14, exclusions);
 const profile = profileRepo(target, files);
 const { components } = harvestComponents(target, files.code);
+// Repo kind and measurability, decided once in profiles/ and read everywhere
+// (a published components package with no app pages is a LIBRARY; a stack the
+// detector cannot read is NOT MEASURED, never scored as zeros). The decision
+// lands on the profile with its evidence, so the JSON says why.
+decideProfile(profile, components, files, target);
+// A shadcn kitchen gets the 2 paint checks from shadcn's own agent rules,
+// counted over own code only (never the kit's doors, never exempt files).
+{
+  const P = profileOf(profile);
+  if (P.isShadcn) {
+    const doorFiles = new Set([...P.uiDirs.flatMap((d) => files.code.filter((f) => f.startsWith(`${d}/`))), ...(profile.shadcn.blockFiles ?? [])]);
+    const kitNames = new Set(components.filter((c) => doorFiles.has(c.file)).map((c) => c.name));
+    profile.shadcn.paint = countPaint(target, files.code, { uiDirs: [...P.uiDirs, ...(profile.shadcn.blockFiles ?? [])], kitNames });
+  }
+}
+
 const tokens = harvestTokens(target, files.styles, files.code);
 const duplicates = findDuplicates(components, profile.uiDir, target);
 const context = harvestContext(target);
@@ -64,12 +81,6 @@ const staleRules = ruleStaleness(target, components,
   new Set(neverImportedComponents(components, profile.uiDir).map((c) => c.name)),
   [...files.code, ...files.styles, ...files.other]);
 
-// Repo kind and measurability, decided once in profiles/ and read everywhere
-// (a published components package with no app pages is a LIBRARY; a stack the
-// detector cannot read is NOT MEASURED, never scored as zeros). The decision
-// lands on the profile with its evidence, so the JSON says why.
-decideProfile(profile, components, files);
-
 // Orphans carry a receipt: the last time git saw anyone touch the file.
 // Only never-imported components are dated (the adoption map's evidence);
 // outside a git repo the field is simply absent and no claim is made.

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

@@ -0,0 +1,105 @@
+/**
+ * Paint — the two shadcn checks shadcn's own agent rules name
+ * (skills/shadcn/rules/styling.md in shadcn-ui/ui):
+ *
+ *  1. Paint from a tin. A Tailwind palette colour in own code where a sheet
+ *     row exists: `text-gray-500`, `bg-blue-100`, `text-emerald-600` on a
+ *     status, and the manual evening override `dark:bg-gray-950`. 1 sin, 3
+ *     spellings. The rule: "Semantic colors. Never `bg-blue-500`."
+ *  2. Doors repainted from outside. A colour or typography class passed into
+ *     a kit component through className (`<Card className="bg-blue-100
+ *     font-bold">`) where a variant or a sheet row was the intended route.
+ *     The rule: "className for layout only."
+ *
+ * Both are counted over OWN code only: never inside the catalogue (those are
+ * kit doors, edited on purpose), never in exempt files (email, print,
+ * artwork, render-to-image), never under demo folders. Counts are reported
+ * per 100 own-code files so a 40-file starter and a 4,000-file product sit on
+ * one ruler. Receipts name the files and the exact classes.
+ */
+import { join } from 'node:path';
+import { readSource } from './walk.mjs';
+import { exemptReason } from '../lib/exempt.mjs';
+import { PALETTE } from '../profiles/shadcn-data.mjs';
+
+// A palette utility with any variant prefix (hover:, md:, dark:, group-hover:)
+// and any opacity suffix. Word boundary at the front stops `text-primary` and
+// `bg-sidebar` from matching; the shade stops `border-gray` (no shade) which
+// is not a Tailwind class.
+const TIN_RE = new RegExp(`(?<![\\w-])(?:[\\w-]+:)*(?:bg|text|border|ring|outline|from|to|via|fill|stroke|divide|decoration|placeholder|caret|accent|shadow)-(?:${PALETTE})-(?:50|[1-9]00|950)(?:/\\d+)?(?![\\w-])`, 'g');
+// Evening overrides painted by hand with white or black (the palette shades
+// are already caught above with their dark: prefix).
+const DARK_WB_RE = /(?<![\w-])dark:(?:bg|text|border)-(?:white|black)(?:\/\d+)?(?![\w-])/g;
+// Colour or typography passed into a kit door: palette colour, black/white,
+// weight, size.
+const DOOR_CLASS_RE = new RegExp(`(?<![\\w-])(?:[\\w-]+:)*(?:(?:bg|text|border)-(?:${PALETTE}|white|black)(?:-(?:50|[1-9]00|950))?(?:/\\d+)?|font-(?:thin|extralight|light|normal|medium|semibold|bold|extrabold|black)|text-(?:xs|sm|base|lg|xl|[2-9]xl))(?![\\w-])`, 'g');
+
+const DEMO_PATH_RE = /(^|\/)(stories|storybook|__stories__|examples?|demos?|templates?|playground|fixtures?|__tests__|__mocks__|e2e|cypress)\//i;
+
+/**
+ * @param root repo root
+ * @param codeFiles relative code paths from the walk
+ * @param opts { uiDirs: string[], kitNames: Set<string> } — catalogue folders
+ *   and the component names defined in them
+ */
+export function countPaint(root, codeFiles, { uiDirs = [], kitNames = new Set() } = {}) {
+  const inCatalogue = (f) => uiDirs.some((d) => f === d || f.startsWith(`${d}/`));
+  const names = [...kitNames].filter((n) => /^[A-Z][A-Za-z0-9]*$/.test(n));
+  const doorOpen = names.length
+    ? new RegExp(`<(${names.join('|')})\\b[^>]*?className=(?:"([^"]*)"|\\{cn\\(\\s*["'\`]([^"'\`]*)["'\`])`, 'g')
+    : null;
+
+  let ownFiles = 0;
+  const tin = { uses: 0, files: 0, top: [], samples: new Map() };
+  const doors = { uses: 0, files: 0, top: [], samples: new Map() };
+  const bump = (bucket, file, count, sample) => {
+    bucket.uses += count; bucket.files += 1;
+    bucket.top.push({ file, count });
+    if (sample) bucket.samples.set(sample, (bucket.samples.get(sample) ?? 0) + count);
+  };
+
+  for (const f of codeFiles) {
+    if (!/\.(tsx|jsx)$/.test(f) || inCatalogue(f) || DEMO_PATH_RE.test(f)) continue;
+    const src = readSource(join(root, f));
+    if (src === null || exemptReason(f, src)) continue;
+    ownFiles += 1;
+
+    const tinHits = [...src.matchAll(TIN_RE)].map((m) => m[0]);
+    const wbHits = [...src.matchAll(DARK_WB_RE)].map((m) => m[0]);
+    const all = [...tinHits, ...wbHits];
+    if (all.length) {
+      const counts = new Map();
+      for (const c of all) counts.set(c, (counts.get(c) ?? 0) + 1);
+      const most = [...counts.entries()].sort((a, b) => b[1] - a[1])[0][0];
+      bump(tin, f, all.length, null);
+      for (const [c, k] of counts) tin.samples.set(c, (tin.samples.get(c) ?? 0) + k);
+      tin.top.at(-1).sample = most;
+    }
+
+    if (doorOpen) {
+      let hits = 0; const seen = new Map();
+      for (const m of src.matchAll(doorOpen)) {
+        const cls = m[2] ?? m[3] ?? '';
+        const bad = cls.match(DOOR_CLASS_RE);
+        if (!bad) continue;
+        hits += 1;
+        const key = `<${m[1]} className="${bad.slice(0, 3).join(' ')}">`;
+        seen.set(key, (seen.get(key) ?? 0) + 1);
+      }
+      if (hits) {
+        bump(doors, f, hits, null);
+        for (const [k, v] of seen) doors.samples.set(k, (doors.samples.get(k) ?? 0) + v);
+        doors.top.at(-1).sample = [...seen.entries()].sort((a, b) => b[1] - a[1])[0][0];
+      }
+    }
+  }
+
+  const finish = (b) => ({
+    uses: b.uses,
+    files: b.files,
+    per100: ownFiles ? Math.round((b.uses / ownFiles) * 100) : 0,
+    top: b.top.sort((a, b2) => b2.count - a.count).slice(0, 8),
+    samples: [...b.samples.entries()].sort((a, b2) => b2[1] - a[1]).slice(0, 10).map(([value, count]) => ({ value, count })),
+  });
+  return { ownFiles, tin: finish(tin), doors: finish(doors) };
+}

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

@@ -3,6 +3,7 @@
  * widened: the harvester must see EVERYTHING that styles the app — code,
  * stylesheets of any flavor, and config — not just a happy-path shadcn layout.
  */
+import { CATALOGUE } from '../profiles/shadcn-data.mjs';
 import { readFileSync, readdirSync, existsSync, statSync } from 'node:fs';
 import { join, relative, extname } from 'node:path';
 
@@ -204,14 +205,7 @@ export function profileRepo(root, files) {
 
   // Catalogue filenames as shipped by `shadcn add --all` (61 as of 2026-09).
   // Only used to recognise a vendored folder, never to judge one.
-  const SHADCN_CATALOGUE = new Set(['accordion', 'alert-dialog', 'alert', 'aspect-ratio', 'attachment',
-    'avatar', 'badge', 'breadcrumb', 'bubble', 'button-group', 'button', 'calendar', 'card', 'carousel',
-    'chart', 'checkbox', 'collapsible', 'combobox', 'command', 'context-menu', 'dialog', 'direction',
-    'drawer', 'dropdown-menu', 'empty', 'field', 'hover-card', 'input-group', 'input-otp', 'input',
-    'item', 'kbd', 'label', 'marker', 'menubar', 'message-scroller', 'message', 'native-select',
-    'navigation-menu', 'pagination', 'popover', 'progress', 'questionnaire', 'radio-group', 'resizable',
-    'scroll-area', 'select', 'separator', 'sheet', 'sidebar', 'skeleton', 'slider', 'spinner', 'switch',
-    'table', 'tabs', 'textarea', 'toast', 'toggle-group', 'toggle', 'tooltip']);
+  const SHADCN_CATALOGUE = CATALOGUE;
   let catalogueNames = 0;
   if (uiDir) {
     try {

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

@@ -1,7 +1,7 @@
 // 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 = '7.1.0';
+export const VERSION = '7.2.0';
 // The shape of harvest.json and summary.json. Bumped only when a field is
 // renamed, removed or changes meaning; a new field is not a new schema. A
 // tool comparing two scans compares like with like by this number, not by VERSION.

package/skills/roast-my-design-system/scripts/mcp/knowledge.mjs

@@ -48,7 +48,7 @@ export function loadKnowledge(root) {
   const { components } = harvestComponents(root, files.code);
   // The MCP path used to skip the kind decision the CLI harvest makes, so a
   // library read as a product here. Same call, same answer, both doors.
-  decideProfile(profile, components, files);
+  decideProfile(profile, components, files, root);
   const tokens = harvestTokens(root, files.styles, files.code);
   const duplicates = findDuplicates(components, profile.uiDir, root);
   const context = harvestContext(root);

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

@@ -19,11 +19,15 @@
  * Step 2 of the shadcn-profile plan: this move changes no number. Every
  * fixture's expected output is byte-identical before and after.
  */
+import shadcn from './shadcn.mjs';
 import library from './library.mjs';
 import product from './product.mjs';
 
-// Specific kinds first; product is the fallback and must stay last.
-export const PROFILES = [library, product];
+// Specific kinds first; product is the fallback and must stay last. The kind
+// says how to READ the repo (shadcn: a kit over a sheet); the role says what
+// usage MEANS (library: composition, not adoption). A shadcn library keeps
+// both: kind shadcn, role library.
+export const PROFILES = [shadcn, library, product];
 
 /**
  * Decide the repo's kind from the profiler's facts and what the scan found.
@@ -32,7 +36,7 @@ export const PROFILES = [library, product];
  * `componentDetection` (measurability), and the new `kind`, `kindConfidence`
  * and `kindEvidence` (the receipt: why this kind, in words).
  */
-export function decideProfile(profile, components, files) {
+export function decideProfile(profile, components, files, root = null) {
   const reusable = components.filter((c) => !c.isPage);
   const counts = {
     reusable: reusable.length,
@@ -43,12 +47,14 @@ export function decideProfile(profile, components, files) {
   // riding in a library's monorepo (Siemens iX ships two), not a product.
   counts.fewPages = counts.pages <= counts.reusable * 0.05;
 
+  const ctx = { root, files };
   let picked = product, decision = null;
   for (const p of PROFILES) {
-    const d = p.recognise(profile, counts);
+    const d = p.recognise(profile, counts, ctx);
     if (d) { picked = p; decision = d; break; }
   }
-  profile.role = picked.kind;
+  // role is usage semantics, decided by the library rule whatever the kind
+  profile.role = library.recognise(profile, counts, ctx) ? 'library' : 'product';
   profile.kind = picked.kind;
   profile.kindConfidence = decision.confidence;
   profile.kindEvidence = decision.evidence;
@@ -75,11 +81,16 @@ export function decideProfile(profile, components, files) {
 export function profileOf(h) {
   const p = h?.profile ?? h ?? {};
   const kind = p.kind ?? p.role ?? 'product';
+  const role = p.role ?? (kind === 'library' ? 'library' : 'product');
   return {
     kind,
+    role,
     confidence: p.kindConfidence ?? null,
     evidence: p.kindEvidence ?? [],
-    isLibrary: kind === 'library',
+    isLibrary: role === 'library',
+    isShadcn: kind === 'shadcn',
+    shadcn: p.shadcn ?? null,
+    uiDirs: p.uiDirs ?? (p.uiDir ? [p.uiDir] : []),
     // A vendored shadcn catalogue is stock on a shelf, not abandonment.
     vendoredUi: p.vendoredUi === true,
     uiDir: p.uiDir ?? null,
@@ -88,3 +99,14 @@ export function profileOf(h) {
     designSystem: p.designSystem ?? { kind: 'none' },
   };
 }
+
+/**
+ * Question 6, per kind: ideals and fleet stats a profile owns until the
+ * benchmark slice for that kind exists. Overlaid on the general benchmark by
+ * score.mjs; the general profile owns none, so nothing moves for it.
+ */
+export function profileYardstick(h) {
+  const kind = profileOf(h).kind;
+  const p = PROFILES.find((x) => x.kind === kind);
+  return { ideals: p?.ideals ?? {}, stats: p?.stats ?? {} };
+}

package/skills/roast-my-design-system/scripts/profiles/shadcn-data.mjs

@@ -0,0 +1,107 @@
+/**
+ * shadcn facts the profile reads against. Everything here was read out of
+ * shadcn/ui's own source and docs (MIT, github.com/shadcn-ui/ui) on
+ * 2026-09-11 and out of tweakcn (Apache 2.0, github.com/jnsahaj/tweakcn).
+ * Data only: names, rows, values. No judgement lives in this file.
+ */
+
+// Catalogue filenames as shipped by `shadcn add --all` (61 as of 2026-09),
+// plus the older names still found in installs from before 2025. Used only to
+// recognise a vendored folder, never to judge one.
+export const CATALOGUE = new Set(['accordion', 'alert-dialog', 'alert', 'aspect-ratio', 'attachment',
+  'avatar', 'badge', 'breadcrumb', 'bubble', 'button-group', 'button', 'calendar', 'card', 'carousel',
+  'chart', 'checkbox', 'collapsible', 'combobox', 'command', 'context-menu', 'dialog', 'direction',
+  'drawer', 'dropdown-menu', 'empty', 'field', 'hover-card', 'input-group', 'input-otp', 'input',
+  'item', 'kbd', 'label', 'marker', 'menubar', 'message-scroller', 'message', 'native-select',
+  'navigation-menu', 'pagination', 'popover', 'progress', 'questionnaire', 'radio-group', 'resizable',
+  'scroll-area', 'select', 'separator', 'sheet', 'sidebar', 'skeleton', 'slider', 'spinner', 'switch',
+  'table', 'tabs', 'textarea', 'toast', 'toggle-group', 'toggle', 'tooltip',
+  // older installs
+  'form', 'sonner', 'toaster', 'use-toast']);
+
+// Component files the block kits install into OWN code (login-form.tsx lands
+// under the components alias, not under ui/). Kit doors stored outside the
+// catalogue, judged like the catalogue.
+export const BLOCK_COMPONENTS = new Set(['app-sidebar', 'calendars', 'chart-area-interactive',
+  'data-table', 'date-picker', 'login-form', 'nav-actions', 'nav-documents', 'nav-favorites',
+  'nav-main', 'nav-projects', 'nav-secondary', 'nav-user', 'nav-workspaces', 'prompt-input',
+  'search-form', 'section-cards', 'settings-dialog', 'sidebar-left', 'sidebar-opt-in-form',
+  'sidebar-right', 'signup-form', 'site-header', 'team-switcher', 'version-switcher']);
+
+// The sheet: rows shadcn's theming docs define, each in :root and .dark.
+export const SHADCN_ROWS = ['background', 'foreground', 'card', 'card-foreground', 'popover',
+  'popover-foreground', 'primary', 'primary-foreground', 'secondary', 'secondary-foreground',
+  'muted', 'muted-foreground', 'accent', 'accent-foreground', 'destructive', 'border', 'input',
+  'ring', 'chart-1', 'chart-2', 'chart-3', 'chart-4', 'chart-5', 'sidebar', 'sidebar-foreground',
+  'sidebar-primary', 'sidebar-primary-foreground', 'sidebar-accent', 'sidebar-accent-foreground',
+  'sidebar-border', 'sidebar-ring', 'radius'];
+
+// Rows tweakcn adds to every theme it exports.
+export const TWEAKCN_ROWS = ['destructive-foreground', 'font-sans', 'font-serif', 'font-mono',
+  'shadow-2xs', 'shadow-xs', 'shadow-sm', 'shadow', 'shadow-md', 'shadow-lg', 'shadow-xl', 'shadow-2xl',
+  'shadow-color', 'shadow-opacity', 'shadow-blur', 'shadow-spread', 'shadow-offset-x', 'shadow-offset-y',
+  'tracking-tighter', 'tracking-tight', 'tracking-normal', 'tracking-wide', 'tracking-wider',
+  'tracking-widest', 'spacing'];
+
+// Rows the docs or the shipped agent skill mention beyond the two lists.
+export const OTHER_KNOWN_ROWS = ['font-heading', 'surface', 'surface-foreground',
+  'sidebar-background', 'destructive-foreground', 'radius-sm', 'radius-md', 'radius-lg',
+  'radius-xl', 'radius-2xl', 'radius-3xl', 'radius-4xl'];
+
+export const KNOWN_ROWS = new Set([...SHADCN_ROWS, ...TWEAKCN_ROWS, ...OTHER_KNOWN_ROWS]);
+
+// Rows that only exist once (no evening value expected).
+export const LIGHT_ONLY_ROWS = new Set(['radius', 'spacing', 'font-sans', 'font-serif', 'font-mono',
+  'font-heading', 'tracking-tighter', 'tracking-tight', 'tracking-normal', 'tracking-wide',
+  'tracking-wider', 'tracking-widest', 'radius-sm', 'radius-md', 'radius-lg', 'radius-xl',
+  'radius-2xl', 'radius-3xl', 'radius-4xl', 'shadow-2xs', 'shadow-xs', 'shadow-sm', 'shadow',
+  'shadow-md', 'shadow-lg', 'shadow-xl', 'shadow-2xl', 'shadow-color', 'shadow-opacity',
+  'shadow-blur', 'shadow-spread', 'shadow-offset-x', 'shadow-offset-y']);
+
+// The factory value of the gap multiplier. Changing it moves every gap in the
+// kitchen at once; shadcn's own changelog (Rhea, 2026-05) says not to.
+export const FACTORY_SPACING = '0.25rem';
+
+// The kit vocabulary, from packages/shadcn/src/preset/preset.ts, with the
+// legacy values still found in the fleet (slate and gray base colours, the
+// pre-create fronts).
+export const FRONTS = ['nova', 'vega', 'maia', 'lyra', 'mira', 'luma', 'sera', 'rhea'];
+export const LEGACY_FRONTS = ['new-york', 'new-york-v4', 'default'];
+export const BASES = ['base', 'radix', 'aria'];
+export const BASE_COLORS = ['neutral', 'stone', 'zinc', 'gray', 'mauve', 'olive', 'mist', 'taupe', 'slate'];
+export const ICON_LIBRARIES = ['lucide', 'hugeicons', 'tabler', 'phosphor', 'remixicon', 'radix'];
+export const RADIUS_MAP = { '0': 'none', '0rem': 'none', '0px': 'none', '0.45rem': 'small',
+  '0.5rem': 'legacy default', '0.625rem': 'default', '0.875rem': 'large' };
+
+// Light-mode `--primary` and `--chart-2` per named theme, from
+// apps/v4/registry/themes.ts. The accent is matched by primary, the chart
+// colour by chart-2. A sheet in the v3 hsl form never matches, by design.
+export const THEMES = [
+  { name: 'neutral', primary: 'oklch(0.205 0 0)', chart2: 'oklch(0.556 0 0)' },
+  { name: 'stone', primary: 'oklch(0.216 0.006 56.043)', chart2: 'oklch(0.553 0.013 58.071)' },
+  { name: 'zinc', primary: 'oklch(0.21 0.006 285.885)', chart2: 'oklch(0.552 0.016 285.938)' },
+  { name: 'mauve', primary: 'oklch(0.212 0.019 322.12)', chart2: 'oklch(0.542 0.034 322.5)' },
+  { name: 'olive', primary: 'oklch(0.228 0.013 107.4)', chart2: 'oklch(0.58 0.031 107.3)' },
+  { name: 'mist', primary: 'oklch(0.218 0.008 223.9)', chart2: 'oklch(0.56 0.021 213.5)' },
+  { name: 'taupe', primary: 'oklch(0.214 0.009 43.1)', chart2: 'oklch(0.547 0.021 43.1)' },
+  { name: 'amber', primary: 'oklch(0.555 0.163 48.998)', chart2: 'oklch(0.769 0.188 70.08)' },
+  { name: 'blue', primary: 'oklch(0.488 0.243 264.376)', chart2: 'oklch(0.623 0.214 259.815)' },
+  { name: 'cyan', primary: 'oklch(0.52 0.105 223.128)', chart2: 'oklch(0.715 0.143 215.221)' },
+  { name: 'emerald', primary: 'oklch(0.508 0.118 165.612)', chart2: 'oklch(0.696 0.17 162.48)' },
+  { name: 'fuchsia', primary: 'oklch(0.518 0.253 323.949)', chart2: 'oklch(0.667 0.295 322.15)' },
+  { name: 'green', primary: 'oklch(0.527 0.154 150.069)', chart2: 'oklch(0.723 0.219 149.579)' },
+  { name: 'indigo', primary: 'oklch(0.457 0.24 277.023)', chart2: 'oklch(0.585 0.233 277.117)' },
+  { name: 'lime', primary: 'oklch(0.841 0.238 128.85)', chart2: 'oklch(0.768 0.233 130.85)' },
+  { name: 'orange', primary: 'oklch(0.553 0.195 38.402)', chart2: 'oklch(0.705 0.213 47.604)' },
+  { name: 'pink', primary: 'oklch(0.525 0.223 3.958)', chart2: 'oklch(0.656 0.241 354.308)' },
+  { name: 'purple', primary: 'oklch(0.496 0.265 301.924)', chart2: 'oklch(0.627 0.265 303.9)' },
+  { name: 'red', primary: 'oklch(0.505 0.213 27.518)', chart2: 'oklch(0.637 0.237 25.331)' },
+  { name: 'rose', primary: 'oklch(0.514 0.222 16.935)', chart2: 'oklch(0.645 0.246 16.439)' },
+  { name: 'sky', primary: 'oklch(0.5 0.134 242.749)', chart2: 'oklch(0.685 0.169 237.323)' },
+  { name: 'teal', primary: 'oklch(0.511 0.096 186.391)', chart2: 'oklch(0.704 0.14 182.503)' },
+  { name: 'violet', primary: 'oklch(0.491 0.27 292.581)', chart2: 'oklch(0.606 0.25 292.717)' },
+  { name: 'yellow', primary: 'oklch(0.681 0.162 75.834)', chart2: 'oklch(0.795 0.184 86.047)' },
+];
+
+// Tailwind's named palette: paint from a tin when it appears in own code.
+export const PALETTE = 'slate|gray|zinc|neutral|stone|red|orange|amber|yellow|lime|green|emerald|teal|cyan|sky|blue|indigo|violet|purple|fuchsia|pink|rose';

package/skills/roast-my-design-system/scripts/profiles/shadcn.mjs

@@ -0,0 +1,267 @@
+/**
+ * shadcn — a kitchen built from the shadcn kit.
+ *
+ * The team ordered a catalogue of components they are meant to edit, over a
+ * sheet of named finishes they are meant to change. The kit is recognised by
+ * its own marker file (components.json, at the root or in any workspace) and
+ * by the catalogue's door names; the kit's flavour is read the way shadcn's
+ * own `preset resolve` reads it; the sheet is read by row name. Everything
+ * this profile decides lands on `profile.shadcn` with its evidence.
+ *
+ * What it changes, and only this:
+ *   - the catalogue folder(s) are found through the config's alias, not only
+ *     at the three usual paths; installed blocks in own code are kit doors too
+ *   - the sheet is the CSS file the config names, never a file chosen by
+ *     counting colour literals
+ *   - 2 checks from shadcn's own agent rules become tiles: paint from a tin,
+ *     and kit doors repainted from outside (harvest/paint.mjs)
+ *   - the sheet check is receipts: rows present for daylight and evening,
+ *     custom rows, the gap multiplier
+ *
+ * What it does NOT do: compare doors to upstream drawings (editing is the
+ * intended use, and the drawings for the create-era fronts are generated,
+ * not stored), or add any "unless shadcn" clause to a counter.
+ *
+ * Kitchen words used in comments: kit, front (style), hinge system (base),
+ * door (component file), sheet (theme CSS), row (one --name: value).
+ */
+import { existsSync, readFileSync } from 'node:fs';
+import { join, dirname, basename } from 'node:path';
+import { CATALOGUE, BLOCK_COMPONENTS, KNOWN_ROWS, TWEAKCN_ROWS, LIGHT_ONLY_ROWS, FACTORY_SPACING,
+  FRONTS, LEGACY_FRONTS, BASES, BASE_COLORS, ICON_LIBRARIES, RADIUS_MAP, THEMES, SHADCN_ROWS } from './shadcn-data.mjs';
+
+const readJSON = (p) => { try { return JSON.parse(readFileSync(p, 'utf8')); } catch { return null; } };
+const read = (p) => { try { return readFileSync(p, 'utf8'); } catch { return ''; } };
+const stripExt = (f) => f.replace(/\.[cm]?[jt]sx?$/, '');
+
+/**
+ * Every folder in the walk holding 8+ catalogue door names AND the marks of
+ * shadcn-generated code in at least 3 of them: a headless base import
+ * (radix-ui, @radix-ui/react-*, @base-ui/react, react-aria-components), a
+ * `data-slot` attribute, or the cva + cn pair. Names alone are not enough:
+ * button, card, input and label are what anyone calls their components, and
+ * a hand-written library must never be handed the shadcn card.
+ */
+const SHADCN_CODE_RE = /from ['"](?:radix-ui|@radix-ui\/react-[\w-]+|@base-ui\/react(?:\/[\w-]+)?|react-aria-components)['"]|data-slot=|cva\(/;
+function catalogueSweep(root, files) {
+  const byDir = new Map();
+  for (const f of files.code) {
+    if (!/\.[jt]sx$/.test(f)) continue;
+    const d = dirname(f);
+    if (!CATALOGUE.has(stripExt(basename(f)))) continue;
+    const e = byDir.get(d) ?? { names: 0, marked: 0 };
+    e.names += 1;
+    if (e.marked < 3 && SHADCN_CODE_RE.test(read(join(root, f)))) e.marked += 1;
+    byDir.set(d, e);
+  }
+  return [...byDir.entries()].filter(([, e]) => e.names >= 8 && e.marked >= 3).map(([dir, e]) => ({ dir, catalogueNames: e.names }));
+}
+
+/** Resolve an alias like "@/components/ui" or "@acme/ui/components" to a folder under wsRoot, if it exists. */
+function resolveAlias(root, wsRoot, alias) {
+  if (!alias) return null;
+  const rel = (p) => p.replace(/\\/g, '/').replace(/^\.\//, '');
+  // tsconfig paths in the workspace, then the root
+  for (const dir of [wsRoot, root]) {
+    const ts = readJSON(join(dir, 'tsconfig.json'));
+    const paths = ts?.compilerOptions?.paths ?? {};
+    const baseUrl = ts?.compilerOptions?.baseUrl ?? '.';
+    for (const [key, targets] of Object.entries(paths)) {
+      const stem = key.replace(/\/?\*$/, '');
+      if (alias !== stem && !alias.startsWith(`${stem}/`)) continue;
+      const tail = alias.slice(stem.length).replace(/^\//, '');
+      for (const t of targets) {
+        const cand = join(dir, baseUrl, rel(t).replace(/\/?\*$/, ''), tail);
+        if (existsSync(cand)) return cand;
+      }
+    }
+  }
+  // package.json#imports ("#components/*": "./src/components/*.tsx")
+  const pkg = readJSON(join(wsRoot, 'package.json'));
+  for (const [key, target] of Object.entries(pkg?.imports ?? {})) {
+    const stem = key.replace(/\/?\*$/, '');
+    if (typeof target !== 'string' || (alias !== stem && !alias.startsWith(`${stem}/`))) continue;
+    const tail = alias.slice(stem.length).replace(/^\//, '');
+    const cand = join(wsRoot, rel(target).replace(/\/?\*(\.\w+)?$/, ''), tail);
+    if (existsSync(cand)) return cand;
+  }
+  // the common shapes when no alias map resolves
+  const tail = alias.replace(/^[@~#]\/?/, '').replace(/^[^/]+\/ui\//, '');
+  for (const cand of [join(wsRoot, 'src', tail), join(wsRoot, tail), join(wsRoot, 'app', tail)]) {
+    if (existsSync(cand)) return cand;
+  }
+  return null;
+}
+
+/** Rows declared under a selector in a CSS text: name -> value. */
+function rowsUnder(css, selector) {
+  const out = new Map();
+  const re = new RegExp(`(^|[\\s,}])${selector}\\s*\\{([^}]*)\\}`, 'g');
+  for (const m of css.matchAll(re)) {
+    for (const d of m[2].matchAll(/--([a-z0-9-]+)\s*:\s*([^;]+);/gi)) out.set(d[1], d[2].trim());
+  }
+  return out;
+}
+
+const norm = (v) => (v ?? '').replace(/\s+/g, ' ').trim();
+
+/** Read the sheet: the CSS file the config names. */
+function readSheet(root, wsRoot, cssPath) {
+  if (!cssPath) return null;
+  const file = [join(wsRoot, cssPath), join(wsRoot, 'src', cssPath)].find((p) => existsSync(p));
+  if (!file) return { file: cssPath, found: false };
+  const css = read(file);
+  const light = rowsUnder(css, ':root'), dark = rowsUnder(css, '\\.dark');
+  const themeInline = rowsUnder(css, '@theme\\s+inline');
+  const rows = [...light.keys()];
+  const known = rows.filter((r) => KNOWN_ROWS.has(r));
+  const custom = rows.filter((r) => !KNOWN_ROWS.has(r));
+  const missingDark = rows.filter((r) => KNOWN_ROWS.has(r) && !LIGHT_ONLY_ROWS.has(r) && !dark.has(r));
+  const customMissingDark = custom.filter((r) => !dark.has(r) && !/^(font|shadow|tracking|spacing|radius|ease|breakpoint)/.test(r));
+  const shadcnPresent = SHADCN_ROWS.filter((r) => light.has(r)).length;
+  const tweakcnPresent = TWEAKCN_ROWS.filter((r) => light.has(r) && !/^font-(sans|mono)$/.test(r)).length;
+  const hslEra = /--background:\s*\d+(\.\d+)?\s+\d+(\.\d+)?%\s+\d+(\.\d+)?%/.test(css);
+  const spacing = light.get('spacing') ?? null;
+  const themeVars = new Set(themeInline.keys());
+  // a custom colour row must be registered for Tailwind 4 to see it
+  const customUnregistered = themeVars.size
+    ? custom.filter((r) => !/^(font|shadow|tracking|spacing|radius|ease|breakpoint|typeset)/.test(r) && !themeVars.has(`color-${r}`))
+    : [];
+  return {
+    file: file.slice(root.length + 1), found: true, hslEra,
+    lightRows: rows.length, darkRows: dark.size, shadcnPresent, shadcnMissing: SHADCN_ROWS.filter((r) => !light.has(r)),
+    known: known.length, custom, missingDark, customMissingDark, customUnregistered, tweakcnPresent,
+    spacing, spacingChanged: spacing !== null && norm(spacing) !== FACTORY_SPACING,
+    radius: light.get('radius') ?? null,
+    primary: norm(light.get('primary')), chart2: norm(light.get('chart-2')), fontSans: light.get('font-sans') ?? null,
+  };
+}
+
+/** The kit's flavour, the way `shadcn preset resolve` reads it. */
+function readKit(cfg, sheet, deps) {
+  const style = cfg.style ?? null;
+  let base = null, front = null;
+  if (style) {
+    const m = style.match(/^(base|radix|aria)-(.+)$/);
+    if (m) { base = m[1]; front = m[2]; }
+    else { front = style; base = LEGACY_FRONTS.includes(style) ? 'radix' : null; }
+  }
+  const fellBack = [];
+  const theme = THEMES.find((t) => t.primary === sheet?.primary)?.name ?? null;
+  const chartColor = THEMES.find((t) => t.chart2 === sheet?.chart2)?.name ?? null;
+  if (!theme) fellBack.push('theme');
+  if (!chartColor) fellBack.push('chartColor');
+  const radius = sheet?.radius ? (RADIUS_MAP[norm(sheet.radius)] ?? `custom (${norm(sheet.radius)})`) : null;
+  if (!radius) fellBack.push('radius');
+  const twRaw = deps?.tailwindcss ?? null;
+  const twClean = twRaw ? String(twRaw).replace(/^[\^~>=<\s]*/, '') : '';
+  const tailwind = /^\d/.test(twClean) ? twClean.split('.')[0] : null;
+  return {
+    style, front: front && (FRONTS.includes(front) || LEGACY_FRONTS.includes(front)) ? front : front,
+    frontKnown: front ? FRONTS.includes(front) || LEGACY_FRONTS.includes(front) : false,
+    base, baseKnown: base ? BASES.includes(base) : false,
+    baseColor: cfg.tailwind?.baseColor ?? null, baseColorKnown: BASE_COLORS.includes(cfg.tailwind?.baseColor),
+    cssVariables: cfg.tailwind?.cssVariables !== false,
+    iconLibrary: cfg.iconLibrary ?? null, iconLibraryKnown: ICON_LIBRARIES.includes(cfg.iconLibrary),
+    rtl: cfg.rtl === true, menuAccent: cfg.menuAccent ?? null, menuColor: cfg.menuColor ?? null,
+    theme, chartColor, radius, tailwind, fellBack,
+  };
+}
+
+export default {
+  kind: 'shadcn',
+
+  /**
+   * @param profile the profiler's facts (mutated: uiDir, uiDirs, vendoredUi, designSystem, shadcn)
+   * @param counts reusable/pages/codeFiles
+   * @param ctx { root, files } the walk
+   */
+  recognise(profile, counts, ctx) {
+    if (!ctx?.root || !ctx?.files) return null;
+    const { root, files } = ctx;
+    const evidence = [];
+
+    // 1. marker files, root or any workspace
+    const configs = (files.other ?? []).filter((f) => basename(f) === 'components.json')
+      .map((f) => ({ file: f, cfg: readJSON(join(root, f)) }))
+      .filter((c) => c.cfg && (c.cfg.aliases || c.cfg.style !== undefined || c.cfg.tailwind));
+
+    // 2. catalogues: through each config's alias, then a sweep for door names
+    const installs = [];
+    for (const { file, cfg } of configs) {
+      const wsRoot = join(root, dirname(file));
+      const uiAlias = cfg.aliases?.ui ?? (cfg.aliases?.components ? `${cfg.aliases.components}/ui` : '@/components/ui');
+      let uiAbs = resolveAlias(root, wsRoot, uiAlias);
+      let catalogueNames = 0;
+      if (uiAbs) {
+        const rel = uiAbs.slice(root.length + 1).replace(/\\/g, '/');
+        catalogueNames = files.code.filter((f) => f.startsWith(`${rel}/`) && CATALOGUE.has(stripExt(basename(f)))).length;
+        uiAbs = rel;
+      }
+      const pkg = readJSON(join(wsRoot, 'package.json')) ?? readJSON(join(root, 'package.json')) ?? {};
+      const deps = { ...(pkg.dependencies ?? {}), ...(pkg.devDependencies ?? {}) };
+      const sheet = readSheet(root, wsRoot, cfg.tailwind?.css);
+      installs.push({ config: file, uiDir: uiAbs, catalogueNames, kit: readKit(cfg, sheet, deps), sheet });
+    }
+    const swept = catalogueSweep(root, files);
+    for (const s of swept) {
+      if (!installs.some((i) => i.uiDir === s.dir)) installs.push({ config: null, uiDir: s.dir, catalogueNames: s.catalogueNames, kit: null, sheet: null });
+      else { const i = installs.find((x) => x.uiDir === s.dir); i.catalogueNames = Math.max(i.catalogueNames, s.catalogueNames); }
+    }
+
+    // 3. the decision: a marker plus a folder, or a folder that is plainly the
+    // catalogue by name (8+). Same bar the profiler used for vendoredUi.
+    const withConfigAndDir = installs.filter((i) => i.config && i.uiDir);
+    const byName = installs.filter((i) => i.catalogueNames >= 8);
+    if (!withConfigAndDir.length && !byName.length) return null;
+
+    installs.sort((a, b) => b.catalogueNames - a.catalogueNames || (b.config ? 1 : 0) - (a.config ? 1 : 0));
+    const primary = installs[0];
+    // the kit is read from a config; when the biggest catalogue has none
+    // (a shared package found by sweep), the nearest configured install speaks
+    const configured = installs.find((i) => i.kit) ?? null;
+    const kit = primary.kit ?? configured?.kit ?? null;
+    const sheet = primary.sheet ?? configured?.sheet ?? null;
+    const confidence = primary.config && primary.catalogueNames >= 8 ? 'high'
+      : primary.catalogueNames >= 8 || (primary.config && primary.uiDir) ? 'medium' : 'low';
+
+    for (const i of installs) {
+      if (i.config) evidence.push(`components.json in ${dirname(i.config) === '.' ? 'the root' : dirname(i.config)}${i.uiDir ? `, ${i.catalogueNames} catalogue component${i.catalogueNames === 1 ? '' : 's'} in ${i.uiDir}` : ', no catalogue folder found through its alias'}`);
+      else evidence.push(`${i.catalogueNames} catalogue components in ${i.uiDir}, no components.json`);
+    }
+    if (kit?.style) evidence.push(`style ${kit.style}${kit.baseColor ? `, base colour ${kit.baseColor}` : ''}${kit.tailwind ? `, Tailwind ${kit.tailwind}` : ''}`);
+
+    // 4. write the facts every consumer reads
+    profile.uiDirs = installs.map((i) => i.uiDir).filter(Boolean);
+    profile.uiDir = primary.uiDir ?? profile.uiDir ?? null;
+    profile.vendoredUi = true;
+    const cssVars = kit ? kit.cssVariables : true;
+    profile.designSystem = { kind: 'shadcn', name: 'shadcn/ui', confidence: 'high', cssVariables: cssVars };
+    // kit doors installed as blocks into own code
+    const blockFiles = files.code.filter((f) => /\.[jt]sx$/.test(f) && !profile.uiDirs.some((d) => f.startsWith(`${d}/`)) && BLOCK_COMPONENTS.has(stripExt(basename(f))));
+    profile.shadcn = {
+      installs: installs.map((i) => ({ config: i.config, uiDir: i.uiDir, catalogueNames: i.catalogueNames })),
+      kit,
+      sheet,
+      blockFiles,
+    };
+    return { confidence, evidence };
+  },
+
+  /**
+   * Question 6: the ideal for this kind. Per 100 own-code files. The numbers
+   * come from the 2026-09-11 fleet probe over the 15 shadcn kitchens in the
+   * benchmark (docs: shadcn-step1-fleet-probe): the ideal is the tidiest third,
+   * the stats are the whole set so the "cleaner than" line is honest. The
+   * step 4 slice replaces these with a rebuilt ruler.
+   */
+  ideals: {
+    paintTin: { value: 25, note: 'palette colours per 100 own-code files; the tidiest third of 15 shadcn repos sit under this' },
+    doorOverrides: { value: 15, note: 'kit components repainted through className, per 100 own-code files; the tidiest third sit under this' },
+  },
+  // the 15 shadcn kitchens in the benchmark fleet, 2026-09-11, per 100 own-code files
+  stats: {
+    paintTin: { values: [5, 8, 20, 21, 22, 32, 33, 58, 62, 66, 100, 179, 296, 488, 658], median: 62 },
+    doorOverrides: { values: [0, 3, 5, 14, 16, 16, 16, 17, 22, 26, 28, 31, 40, 44, 55], median: 22 },
+  },
+};

package/skills/roast-my-design-system/scripts/rules/build.mjs

@@ -163,6 +163,28 @@ if (neverImported.length >= 3) {
     rule(`The repo uses ${faces.length} typeface${faces.length === 1 ? '' : 's'}: ${faces.slice(0, 4).join(', ')}. Do not introduce another, and do not re-declare font stacks by hand; use the existing setup.`);
   }
   
+  // ---------- shadcn: the kit's own rules ----------
+  // Written from shadcn's docs and the agent skill it ships (skills/shadcn in
+  // shadcn-ui/ui), with this repo's receipts where the scan has them.
+  const P = profileOf(h);
+  if (P.isShadcn) {
+    const sc = P.shadcn ?? {};
+    const sheetFile = sc.sheet?.found ? sc.sheet.file : null;
+    const paint = sc.paint ?? null;
+    section('shadcn: the components and the theme');
+    rule(`This is a shadcn install${sc.kit?.style ? ` (style \`${sc.kit.style}\`${sc.kit.baseColor ? `, base colour ${sc.kit.baseColor}` : ''})` : ''}. The theme is a set of CSS variables${sheetFile ? ` in \`${sheetFile}\`` : ''}: background, foreground, primary, muted, border and the rest, each with a light and a dark value. Change a colour there, never in a component.`);
+    rule('Use the semantic classes the theme gives you (`bg-background`, `text-muted-foreground`, `border-border`), never a palette colour like `bg-blue-500` or `text-gray-600`, and never a hand-written `dark:` colour. The variables already carry both modes.');
+    if (paint?.tin?.uses) lines.push(`  (${paint.tin.uses} palette colour${paint.tin.uses === 1 ? '' : 's'} already sit in own code, ${paint.tin.samples.slice(0, 3).map((s) => `\`${s.value}\` ×${s.count}`).join(', ')}; do not add to them.)`);
+    rule('Before adding classes to a shadcn component, use one of its variants (`variant="outline"`, `size="sm"`). `className` on a shadcn component is for layout only: width, margin, position. Never colour, never typography.');
+    if (paint?.doors?.uses) lines.push(`  (${paint.doors.uses} shadcn component${paint.doors.uses === 1 ? '' : 's'} already restyled through className, like ${paint.doors.samples.slice(0, 2).map((s) => `\`${s.value}\``).join(' and ')}; do not add to them.)`);
+    rule(`Edit the component you own in \`${P.uiDir ?? 'components/ui'}\`. Never build a second one beside it under another name. A wrapper that composes shadcn components is fine; a second implementation is not.`);
+    rule('Merge classes with `cn()`. Never concatenate strings and never write a ternary inside a className string.');
+    rule('Add a component with `npx shadcn@latest add <name>`, then edit it. To see what changed upstream, run `npx shadcn@latest add <name> --diff`.');
+    rule('Radius comes from the `--radius` variable and the scale derived from it; never a bracket value. Never change `--spacing`: it resizes every gap in the app.');
+    if (sc.sheet?.customUnregistered?.length) rule(`Custom variables must be registered where Tailwind can see them: ${sc.sheet.customUnregistered.slice(0, 3).map((r) => `\`--${r}\``).join(', ')} ${sc.sheet.customUnregistered.length === 1 ? 'is' : 'are'} defined but never mapped in \`@theme inline\`, so \`bg-${sc.sheet.customUnregistered[0]}\` does nothing.`);
+    rule('Small habits shadcn expects: `gap-*` not `space-x/y-*`; `size-4` not `w-4 h-4`; `truncate` for one-line clipping; no `z-index` on Dialog, Sheet, Popover or Tooltip, they stack themselves; icons from the project\'s own icon library only.');
+  }
+
   // ---------- styling discipline ----------
   section('Styling discipline');
   rule(`Never write ${reusable.length ? '`style={{ ... }}`' : 'inline `style="..."` attributes'} for static values; styling belongs to classes and tokens where the system can see it.`);