roast-my-design-system 7.6.0 → 7.7.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.6.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.7.0" width="72" alt="roast-my-design-system">
 
 # roast-my-design-system
 
@@ -10,15 +10,7 @@
 
 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.6: a fourth repo profile, `registry`.** A repo that publishes a shadcn registry is read as a registry, not as an app that installed shadcn. Recognition: a `registry.json` with typed items (`registry:ui`, `registry:style`, ...) or a route that builds one from a packages folder. Covers shadcn's own source, magicui, kibo-ui and tweakcn. The header states what the repo publishes (components, blocks, styles, demos) and how many variants of the same components it keeps. Scoring is unchanged in this release: a registry is measured with the shadcn tiles against the shadcn slice, and the registry counting rules follow in the next one. `summary.json` gains `kind: "registry"` and a `registry` block. Also in 7.6: the theme file is checked for the variables tweakcn adds to every export, shown as a `tweakcn theme` pill on a match; typefaces declared in a single file are reported as a picker rather than as N typefaces.
-
-> **New in 7.5: the kit is read as it is built, and every file left out of a count is named.** A fresh `npx shadcn create` install is told apart from a repo built on one: "The score is the kit's, not yours." A rules file a framework wrote for itself is named as such, and so is the shadcn skill when it is installed. The theme file is found where it is, even when `components.json` points at a file that moved; `button/index.tsx` is a Button. The repaint advice checks whether the theme file holds the variables before telling you to swap to them. And the side panel now lists everything a check leaves out, with the reason and one honest line: your agent reads it anyway. New on that list: the Next.js crash page, an embedded widget's `!important`, and `!important` aimed at a library's own class names.
-
-> **New in 7.4: the score says what it measures, and the report has a new shape.** Under the number: "How safely an AI agent can build on this repo without going off-system." A fixed side panel holds the score, the stack, how the repo was read, an index of every section, and what is not yours and not counted. Installed code splits by one test, does it teach an agent a wrong lesson: shadcn's own bracket values are out of the count and named; palette colours inside an installed registry stay in the score and are never prompted; catalogue stock is not a trap. A factory install scores 100. Leftover theme variables become a fix.
-
-> **New in 7.3: shadcn repos are compared with shadcn repos.** The benchmark carries a shadcn slice, 17 of the 34 fleet repos, and on a shadcn repo every fleet line reads against it: "Avg shadcn repo", "cleaner than 60% of shadcn repos". Ideals and the reputable-systems line stay the same for everyone.
-
-> **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.7: four repo profiles, and the fourth is `registry`.** The scanner decides once what kind of repo it is reading, `product`, `library`, `shadcn` or `registry`, and every count, ideal and benchmark line reads that decision. `registry` is new: a repo that publishes a shadcn registry (a `registry.json` with typed items, or a route that builds one from a packages folder) is counted on what it publishes and nothing else. The docs site, demos and examples are kept out and named in the header with file counts; the same components kept in several variants count once; a name repeated only inside published blocks is the range, not a duplicate; published components are the project's own work, with no installed-code exemption. Published themes get their own tile: every shadcn colour variable present for light and for dark, ideal 0. For a registry that publishes themes and no code, that check is the score. Covers shadcn's own source (84), magicui (69), kibo-ui (78) and tweakcn (100, 36 themes, all complete). No other kind of repo moves. `summary.json`: `kind: "registry"` with a `registry` block (`publishes`, `counted`, `showcase`, `variantsDropped`, `themes`); tiles gain `themesIncomplete` on registries.
 
 > **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`.
 
@@ -83,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: a fixed side panel with the health score and what it measures, the stack, how the repo was read as a shadcn install, an index of every section and what is not the team's and not counted; then the summary, 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.6.0)
+![The full diagnosis report for vercel/ai-chatbot in dark mode: a fixed side panel with the health score and what it measures, the stack, how the repo was read as a shadcn install, an index of every section and what is not the team's and not counted; then the summary, 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.7.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.6.0)
+![The diagnosis report in light mode](https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/report-light-hero.png?v=7.7.0)
 
 ## What makes the numbers trustworthy
 
@@ -243,7 +235,7 @@ After the roast, the skill also offers to write `design-system-rules.md` to disk
 - **[adobe/spectrum-web-components](https://gregkozakiewicz.github.io/roast-my-design-system/examples/adobe-spectrum.html)**: Lit, the `--spectrum-*` namespace named in the header
 - **[npx shadcn create, fresh](https://gregkozakiewicz.github.io/roast-my-design-system/examples/shadcn-create-fresh.html)**: a factory install with all 61 components, scanned as is and read as fresh
 - **[vercel/ai-chatbot](https://gregkozakiewicz.github.io/roast-my-design-system/examples/vercel-ai-chatbot.html)**: React on shadcn, with Claude's notes embedded
-- **[magicuidesign/magicui](https://gregkozakiewicz.github.io/roast-my-design-system/examples/magicui.html)**: read as a shadcn registry, what it publishes named under the score
+- **[magicuidesign/magicui](https://gregkozakiewicz.github.io/roast-my-design-system/examples/magicui.html)**: read as a shadcn registry, counted on the 78 components it publishes
 - **[excalidraw/excalidraw](https://gregkozakiewicz.github.io/roast-my-design-system/examples/excalidraw-excalidraw.html)**
 - **[dubinc/dub](https://gregkozakiewicz.github.io/roast-my-design-system/examples/dubinc-dub.html)**
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roast-my-design-system",
-  "version": "7.6.0",
+  "version": "7.7.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/benchmark/benchmark.json

@@ -93,6 +93,10 @@
     "doorOverrides": {
       "value": 2,
       "note": "shadcn components given a colour through className, per 100 files; the tidiest half of 16 shadcn repos sit under this (2026-09-13 audit)"
+    },
+    "themesIncomplete": {
+      "value": 0,
+      "note": "published themes missing a shadcn colour variable in light or dark; a gap here ships into every repo that installs the theme"
     }
   },
   "stats": {
@@ -2677,4 +2681,4 @@
       ]
     }
   }
-}
+}

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

@@ -368,7 +368,7 @@ function tile(t, pLabel, pMetric, pFallback) {
   const { metric, label, value, healthValue, health } = t;
   if (health === 'na') {
     return { num: '—', label, health, metric, healthValue: null,
-      rows: [{ label: `not measured: ${notMeasuredReason}`, val: '', dir: '' }] };
+      rows: [{ label: t.naReason ? t.naReason : `not measured: ${notMeasuredReason}`, val: '', dir: '' }] };
   }
   const iv = ideal(metric), rm = refMedian(metric);
   const pct = percentile(metric, value);
@@ -441,7 +441,7 @@ function projectedScore(applied) {
 // A utility-class shadcn repo with no hardcoded colours has zero literal
 // colours and a real system: the palette is bg-zinc-900 and friends.
 const utilityPalette = ds.kind === 'shadcn' && ds.cssVariables === false && twColorUtils >= 5;
-const noSystemLikely = !utilityPalette && (colors.length === 0 || (colors.length < 3 && spacingTotal === 0));
+const noSystemLikely = !utilityPalette && !(P.isRegistry && P.registry?.themes?.total) && (colors.length === 0 || (colors.length < 3 && spacingTotal === 0));
 
 // ---------- section renderers ----------
 const DIR = {
@@ -619,6 +619,11 @@ function sheetSection() {
   } 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 (P.isRegistry && P.registry?.themes?.total) {
+    const t = P.registry.themes;
+    const rows = t.worst.map((w) => `<div class="ledger-row"><span class="mono strong">${esc(w.name)}</span><span class="dim">${w.missingLight.length ? `light missing ${w.missingLight.slice(0, 4).map((r) => `--${esc(r)}`).join(', ')}${w.missingLight.length > 4 ? ` and ${w.missingLight.length - 4} more` : ''}` : ''}${w.missingLight.length && w.missingDark.length ? ' · ' : ''}${w.missingDark.length ? `dark missing ${w.missingDark.slice(0, 4).map((r) => `--${esc(r)}`).join(', ')}${w.missingDark.length > 4 ? ` and ${w.missingDark.length - 4} more` : ''}` : ''}</span></div>`).join('');
+    parts.push(`<div class="receipts">${eyebrow(`${n(t.total)} published theme${t.total === 1 ? '' : 's'} · ${n(t.incomplete)} incomplete`)}<p class="sub">${t.incomplete ? `A theme missing a variable ships that gap into every repo that installs it. ${t.incomplete === 1 ? 'The one' : `The ${t.incomplete}`} below ${t.incomplete === 1 ? 'is' : 'are'} short of the full set in at least one mode.` : 'Every published theme carries every shadcn colour variable, for light and for dark.'}</p>${rows ? `<div class="ledger">${rows}</div>` : ''}${whyToggle('themesIncomplete')}</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('');
@@ -648,7 +653,7 @@ function duplicatesSection() {
     </div>` : '';
   const dupeCards = exactDupes.slice(0, 8).map((d) => `
     <div class="fam">
-      ${eyebrow(`&lt;${esc(d.name)}&gt; · ${d.upstream ? 'two shadcn components define it: upstream\'s overlap, not counted' : d.wrapped ? 'defined twice, one wraps the other' : `${d.files.length} implementations`}`)}
+      ${eyebrow(`&lt;${esc(d.name)}&gt; · ${d.blocks ? `defined in ${d.files.length} blocks that each install alone: the range, not counted` : d.upstream ? 'two shadcn components define it: upstream\'s overlap, not counted' : d.wrapped ? 'defined twice, one wraps the other' : `${d.files.length} implementations`}`)}
       <div class="fam-rows">
       ${d.files.slice(0, 6).map((f) => `<div class="mini-card">${fileLink(f)}</div>`).join('')}
       ${d.files.length > 6 ? `<div class="mini-card dim">…and ${d.files.length - 6} more</div>` : ''}
@@ -1326,8 +1331,12 @@ function shadcnReceipt() {
   if (P.isRegistry && P.registry) {
     const r = P.registry;
     const what = r.builtFrom ? `publishes ${r.items} component${r.items === 1 ? '' : 's'}, built from its packages folder` : `publishes ${esc(publishesLine(r))}`;
-    const pages = comps.filter((c) => c.isPage).length;
-    return `<div class="excl fresh">Read as a shadcn registry: ${what}${r.variants.length ? `, the same components kept in ${r.variants.length} variants` : ''}.${pages ? ` Its own site: ${n(pages)} page${pages === 1 ? '' : 's'}.` : ''} Scored as a shadcn install for now; the registry rules come next.</div><div class="excl">Evidence: ${receipt}</div>`;
+    const kept = (r.showcase ?? []).reduce((a, e) => a + e.files, 0);
+    const th = r.themes?.total ? ` ${n(r.themes.total)} published theme${r.themes.total === 1 ? '' : 's'} checked, ${n(r.themes.incomplete)} incomplete.` : '';
+    const counted = (r.counted?.code ?? 0) === 0 && r.themes?.total
+      ? `<b>It publishes no code, so the themes check is the score</b>: the ${n(kept)} files of the app that makes them are not counted.`
+      : `<b>Only what it publishes is counted</b>: ${n(r.counted?.code ?? 0)} code file${(r.counted?.code ?? 0) === 1 ? '' : 's'}${kept ? `, with ${n(kept)} more kept out as its site, demos and examples` : ''}.`;
+    return `<div class="excl fresh">Read as a shadcn registry: ${what}${r.variants.length ? `, the same components kept in ${r.variants.length} variants` : ''}. ${counted}${th}</div><div class="excl">Evidence: ${receipt}</div>`;
   }
   return `<div class="excl">Read as a shadcn install (${esc(P.confidence ?? 'medium')} confidence): ${receipt}</div>`;
 }
@@ -1392,6 +1401,13 @@ function exceptionsBlock() {
     const si = spacingOwn.installed;
     const sv = si.values.length;
     if (sv) lines.push(`${n(sv)} off-scale spacing value${sv === 1 ? '' : 's'} inside ${esc(P.uiDirs.map((d) => basename(d)).join(', '))} ${sv === 1 ? 'is' : 'are'} shadcn's own (${si.values.slice(0, 3).map((v) => esc(v.value)).join(', ')}) and ${sv === 1 ? 'is' : 'are'} not counted, for the same reason as the bracket values.`);
+    if (P.isRegistry && P.registry) {
+      const r = P.registry;
+      const sc = r.showcase ?? [];
+      if (sc.length) lines.push(`${n(sc.reduce((a, e) => a + e.files, 0))} files outside what the registry publishes (${sc.slice(0, 4).map((e) => `${esc(e.dir)} ${n(e.files)}`).join(', ')}${sc.length > 4 ? ` and ${sc.length - 4} more folders` : ''}) are not counted: the docs site, demos and examples nobody installs. Your agent reads them like everything else.`);
+      const vd = r.variantsDropped ?? [];
+      if (vd.length) lines.push(`${vd.length} variant${vd.length === 1 ? '' : 's'} of the same components (${vd.map((e) => esc(basename(dirname(e.dir)) === 'bases' ? basename(dirname(e.dir)) + '/' + basename(e.dir) : e.dir)).join(', ')}, ${n(vd.reduce((a, e) => a + e.files, 0))} files) counted once, through the variant the registry file names.`);
+    }
     if (vendoredUi && neverImported.length) lines.push(`${n(neverImported.length)} catalogue component${neverImported.length === 1 ? '' : 's'} not used yet: stock on the shelf, not scored.`);
     const rp = P.shadcn?.registryPaint;
     if (rp) lines.push(`Installed registr${rp.dirs.length === 1 ? 'y' : 'ies'} ${esc(rp.dirs.map((d) => basename(d)).join(', '))}: ${n(rp.files)} file${rp.files === 1 ? '' : 's'}${rp.tinUses ? `, ${n(rp.tinUses)} palette colour${rp.tinUses === 1 ? '' : 's'}` : ''}. Kept in the score, left out of the fixes. Your agent reads them like everything else.`);
@@ -2039,7 +2055,7 @@ if (summaryPath) {
     verdict,
     role: P.role,
     kind: P.kind,
-    ...(P.isRegistry && P.registry ? { registry: { source: P.registry.source, builtFrom: P.registry.builtFrom, items: P.registry.items, publishes: P.registry.publishes, variants: P.registry.variants } } : {}),
+    ...(P.isRegistry && P.registry ? { registry: { source: P.registry.source, builtFrom: P.registry.builtFrom, items: P.registry.items, publishes: P.registry.publishes, variants: P.registry.variants, counted: P.registry.counted ?? null, showcase: P.registry.showcase ?? [], variantsDropped: P.registry.variantsDropped ?? [], themes: P.registry.themes ? { total: P.registry.themes.total, incomplete: P.registry.themes.incomplete } : null } } : {}),
     ...(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, registries: P.shadcn.registryDirs ?? [], ...(P.shadcn.registryPaint ? { registryFiles: P.shadcn.registryPaint.files, registryPaletteColours: P.shadcn.registryPaint.tinUses } : {}), ownFiles: P.shadcn.paint?.ownFiles ?? null, fresh: P.shadcn.fresh?.fresh === true } } : {}),
     componentsMeasured,
     metrics: (({ colors, colorTokens, colorStrays, greys, greyStrays, spacing, exactDuplicates, inlineStyles, nearPairs, important, neverImported, arbitrary, tokenLed }) =>

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

@@ -78,8 +78,8 @@ export function benchHelpers(bench, kind = 'product') {
   return { percentile, cleanerPct, ideal, median, displayAvg, refMedian, sliceInfo };
 }
 
-export const ZERO_IDEAL = new Set(['exactDuplicates', 'inlineStyles', 'nearPairs', 'important', 'neverImported']);
-export const WARN_TOLERANCE = { exactDuplicates: 2, inlineStyles: 10, nearPairs: 2, important: 5, neverImported: 2 };
+export const ZERO_IDEAL = new Set(['exactDuplicates', 'inlineStyles', 'nearPairs', 'important', 'neverImported', 'themesIncomplete']);
+export const WARN_TOLERANCE = { exactDuplicates: 2, inlineStyles: 10, nearPairs: 2, important: 5, neverImported: 2, themesIncomplete: 1 };
 export const SCORE_OF = { good: 100, warn: 55, bad: 10 };
 
 /** healthOf(metric, value) for one benchmark: 'good' | 'warn' | 'bad' | 'info'. */
@@ -107,7 +107,7 @@ export const PROFILE_TILES = {
   ],
 };
 // a registry measures what a shadcn repo measures (release (a), 2026-09-13)
-PROFILE_TILES.registry = PROFILE_TILES.shadcn;
+PROFILE_TILES.registry = [...PROFILE_TILES.shadcn, ['themesIncomplete', 'published themes incomplete']];
 export const tilesFor = (kind) => [...TILES, ...(PROFILE_TILES[kind] ?? [])];
 
 /** The tiles, in report order: metric key, the label the report prints. */
@@ -176,6 +176,11 @@ export function coreMetrics(h, opts = {}) {
     utilityPalette: profileOf(h).designSystem?.cssVariables === false,
     paintTin: paint?.tin?.per100 ?? 0,
     doorOverrides: paint?.doors?.per100 ?? 0,
+    // a registry's published themes, checked for every variable in both modes
+    themesIncomplete: P.registry?.themes?.incomplete ?? 0,
+    themesPublished: P.registry?.themes?.total ?? 0,
+    // a registry that publishes themes and no code: the themes check is the score
+    stylesOnly: !!(P.isRegistry && P.registry?.themes?.total && (P.registry?.counted?.code ?? 0) === 0),
   };
 }
 
@@ -203,12 +208,14 @@ 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,
+    paintTin: m.paintTin ?? 0, doorOverrides: m.doorOverrides ?? 0, themesIncomplete: m.themesIncomplete ?? 0,
   };
   const judged = { ...shown, colors: m.tokenLed ? m.colorStrays : m.colors, greys: m.tokenLed ? m.greyStrays : m.greys };
   return tilesFor(m.kind ?? 'product').map(([metric, label]) => {
     let health = healthOf(metric, judged[metric]);
     let value = shown[metric], healthValue = judged[metric];
+    // a registry that publishes no themes has nothing to check here
+    if (metric === 'themesIncomplete' && !m.themesPublished) { health = 'na'; value = null; healthValue = null; }
     if (!m.componentsMeasured && (metric === 'exactDuplicates' || metric === 'neverImported')) {
       health = 'na'; value = null; healthValue = null;
     } else if (metric === 'neverImported' && (m.isLibrary || (m.vendoredUi && m.neverImported > 0))) {
@@ -216,7 +223,10 @@ export function tileHealths(m, healthOf) {
     } else if (metric === 'paintTin' && m.utilityPalette) {
       health = 'info';
     }
-    return { metric, label, value, healthValue, health };
+    // a registry that publishes themes and no code: nothing else applies
+    let naReason = null;
+    if (m.stylesOnly && metric !== 'themesIncomplete') { health = 'na'; value = null; healthValue = null; naReason = 'this registry publishes themes and no code, so there is nothing here to measure'; }
+    return { metric, label, value, healthValue, health, ...(naReason ? { naReason } : {}) };
   });
 }
 

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

@@ -15,6 +15,11 @@ export const WHY = {
     '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.',
   ],
+  themesIncomplete: [
+    'A published theme is a set of CSS variables a repo installs with one command: background, foreground, primary, muted, border and the rest, each with a light value and a dark value. A component in that repo reads the variable; if the theme never set it, the component gets nothing.',
+    'A theme missing --sidebar-border in dark mode looks fine in the editor that made it and breaks in the first repo that uses the sidebar. The gap is invisible until then, and the person who hits it is not the person who can fix it.',
+    'The ideal is 0. Every theme a registry publishes carries every variable, in both modes, or says which component it does not support.',
+  ],
   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 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.',

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

@@ -27,7 +27,7 @@ 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, decideFresh, profileOf, installedDirs, splitArbitrary } from '../profiles/index.mjs';
+import { decideProfile, decideFresh, profileOf, installedDirs, splitArbitrary, scopeFiles } from '../profiles/index.mjs';
 import { countPaint } from './paint.mjs';
 
 function arg(name, fallback) {
@@ -55,15 +55,35 @@ const outPath = resolve(arg('out', 'harvest.json'));
 
 const t0 = Date.now();
 const exclusions = loadExclusions(target, argAll('exclude'));
-const files = walkRepo(target, 14, exclusions);
+let files = walkRepo(target, 14, exclusions);
 const profile = profileRepo(target, files);
-const { components } = harvestComponents(target, files.code);
-const tokens = harvestTokens(target, files.styles, files.code);
+let { components } = harvestComponents(target, files.code);
+let tokens = harvestTokens(target, files.styles, 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 registry is counted on what it publishes, one variant of it. The rest
+// (docs site, demos, an installed catalogue for the site) is kept out and
+// named in the header like any exclusion; secondary variants likewise.
+if (profileOf(profile).isRegistry) {
+  const scope = scopeFiles(profile.registry, files);
+  profile.registry.showcase = scope.showcase;
+  profile.registry.variantsDropped = scope.variantsDropped;
+  for (const e of scope.showcase) exclusions.patterns.push({ pattern: e.dir, source: 'the registry profile (not published)', files: e.files });
+  for (const e of scope.variantsDropped) exclusions.patterns.push({ pattern: e.dir, source: 'the registry profile (a variant counted once)', files: e.files });
+  files = scope.files;
+  // the theme file the published components read stays in scope: it is the
+  // colour system they are judged against, wherever the repo keeps it
+  const sheetFile = profile.shadcn?.sheet?.found ? profile.shadcn.sheet.file : null;
+  if (sheetFile && !files.styles.includes(sheetFile)) files.styles.push(sheetFile);
+  ({ components } = harvestComponents(target, files.code));
+  tokens = harvestTokens(target, files.styles, files.code);
+  // published components are the project's own work, not installed code
+  profile.uiDirs = []; profile.uiDir = null; profile.vendoredUi = false;
+  profile.registry.counted = { code: files.code.length, styles: files.styles.length };
+}
 // 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).
 {
@@ -97,6 +117,17 @@ decideProfile(profile, components, files, target);
 }
 
 const duplicates = findDuplicates(components, profile.uiDir, target, profile.uiDirs ?? null);
+// A registry's blocks are self-contained kits, each installed alone
+// (sidebar-01 to sidebar-16 every one with its own AppSidebar). A name whose
+// every copy sits inside a block is the range, not confusion: listed, never
+// counted, like a wrapped pair.
+{
+  const blockDirs = profile.registry?.blockDirs ?? [];
+  if (blockDirs.length) {
+    const inBlock = (f) => blockDirs.some((d) => f === d || f.startsWith(`${d}/`));
+    for (const d of duplicates.exactDuplicates) if (!d.wrapped && d.files.every(inBlock)) { d.wrapped = true; d.blocks = true; }
+  }
+}
 const context = harvestContext(target);
 const staleRules = ruleStaleness(target, components,
   new Set(neverImportedComponents(components, profile.uiDir).map((c) => c.name)),

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.6.0';
+export const VERSION = '7.7.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/profiles/index.mjs

@@ -22,7 +22,8 @@ import { join, basename } from 'node:path';
  * fixture's expected output is byte-identical before and after.
  */
 import shadcn from './shadcn.mjs';
-import { readRegistry, publishesLine, variantsFromDirs } from './registry.mjs';
+import { readRegistry, publishesLine, variantsFromDirs, scopeFiles } from './registry.mjs';
+export { scopeFiles };
 import library from './library.mjs';
 import product from './product.mjs';
 
@@ -64,16 +65,19 @@ export function decideProfile(profile, components, files, root = null) {
   // A shadcn repo that PUBLISHES a registry is a registry: the fourth kind.
   // Every count still reads the shadcn facts (release (a): zero score change);
   // the kind, the receipt and the header line say what it is.
-  if (picked === shadcn && root) {
+  if (root) {
     const reg = readRegistry(root, files);
     if (reg) {
       if (!reg.variants.length) reg.variants = variantsFromDirs(reg, profile.uiDirs, files);
       delete reg.itemNames;
       profile.kind = 'registry';
+      // consumers of what it publishes live in other repos: library semantics
+      profile.role = 'library';
       profile.registry = reg;
       profile.kindEvidence = [
         reg.builtFrom ? `registry built from ${reg.items} packages by ${reg.source}` : `${reg.source} publishes ${publishesLine(reg)}`,
         ...(reg.variants.length ? [`the same components kept in ${reg.variants.length} variants (${reg.variants.map((d) => basename(d)).join(', ')})`] : []),
+        ...(reg.themes?.total ? [`${reg.themes.total} published theme${reg.themes.total === 1 ? '' : 's'} checked, ${reg.themes.incomplete} incomplete`] : []),
         ...decision.evidence,
       ];
     }

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

@@ -19,7 +19,8 @@
  *      (app/r/registry.json/route.ts, kibo-ui)
  */
 import { readFileSync, readdirSync, existsSync } from 'node:fs';
-import { join, dirname, basename } from 'node:path';
+import { join, dirname, basename, relative } from 'node:path';
+import { SHADCN_ROWS } from './shadcn-data.mjs';
 
 const readJSON = (p) => { try { return JSON.parse(readFileSync(p, 'utf8')); } catch { return null; } };
 
@@ -109,7 +110,18 @@ export function readRegistry(root, files) {
   if (best) {
     const publishes = tally(best.items);
     const itemNames = best.items.filter((it) => /^registry:(ui|component)$/.test(String(it?.type ?? ''))).map((it) => it.name).filter(Boolean);
-    return { source: best.source, builtFrom: null, items: best.items.length, publishes, variants: variantDirs(best.items, root), itemNames };
+    // item file paths are relative to the registry file's folder
+    const base = dirname(best.source);
+    const rel = (p) => (base === '.' ? p : `${base}/${p}`);
+    const dirsOf = (re) => [...new Set(best.items.filter((it) => re.test(String(it?.type ?? '')))
+      .flatMap((it) => (it.files ?? []).map((f) => (typeof f === 'string' ? f : f?.path)).filter(Boolean).map((p) => rel(dirname(p)))))].sort();
+    return {
+      source: best.source, builtFrom: null, items: best.items.length, publishes, variants: variantDirs(best.items, root), itemNames,
+      publishedDirs: dirsOf(/^registry:(ui|component|block|lib|hook|style|theme|file|page|internal)$/),
+      demoDirs: dirsOf(/^registry:example$/),
+      blockDirs: dirsOf(/^registry:block$/),
+      themes: themeCheck(best.items),
+    };
   }
   // 2. a route that builds the registry from packages/* (kibo-ui)
   const route = look.routes[0] ?? null;
@@ -127,7 +139,9 @@ export function readRegistry(root, files) {
       if (!pkgs && existsSync(join(root, 'packages'))) pkgs = join(root, 'packages');
       if (pkgs) {
         const names = readdirSync(pkgs, { withFileTypes: true }).filter((e) => e.isDirectory() && existsSync(join(pkgs, e.name, 'index.tsx'))).map((e) => e.name);
-        return { source: route, builtFrom: 'packages', items: names.length, publishes: { components: names.length, blocks: 0, styles: 0, demos: 0, other: 0 }, variants: [] };
+        const pkgRel = relative(root, pkgs).replace(/\\/g, '/');
+        return { source: route, builtFrom: 'packages', items: names.length, publishes: { components: names.length, blocks: 0, styles: 0, demos: 0, other: 0 }, variants: [], itemNames: names,
+          publishedDirs: names.map((nm) => `${pkgRel}/${nm}`), demoDirs: [], blockDirs: [], themes: { total: 0, incomplete: 0, worst: [] } };
       }
     }
   }
@@ -140,21 +154,77 @@ export function readRegistry(root, files) {
  * registry.json lists one file per component (new-york-v4), while the same
  * components sit again under bases/aria, bases/base and bases/radix.
  */
-export function variantsFromDirs(reg, uiDirs, files) {
+export function variantsFromDirs(reg, _uiDirs, files) {
   const names = new Set();
   // names come from the registry items when it is a file, else from packages
   if (reg.itemNames) for (const nm of reg.itemNames) names.add(nm);
   if (names.size < 5) return [];
   const stem = (f) => { const b = basename(f).replace(/\.[cm]?[jt]sx?$/, ''); return b === 'index' ? basename(dirname(f)) : b; };
+  // every folder holding most of the published names, catalogue or not
+  const byDir = new Map();
+  for (const f of files.code ?? []) {
+    if (!/\.[jt]sx$/.test(f)) continue;
+    const d = dirname(f);
+    if (!byDir.has(d)) byDir.set(d, new Set());
+    byDir.get(d).add(stem(f));
+  }
   const hits = [];
-  for (const d of uiDirs ?? []) {
-    const have = new Set((files.code ?? []).filter((f) => f.startsWith(`${d}/`) && /\.[jt]sx$/.test(f)).map(stem));
+  for (const [d, have] of byDir) {
     const shared = [...names].filter((nm) => have.has(nm)).length;
     if (shared >= Math.max(5, Math.ceil(names.size * 0.5))) hits.push(d);
   }
   return hits.length > 1 ? hits.sort() : [];
 }
 
+/**
+ * Published themes, checked: every shadcn colour variable present for light
+ * and for dark. --radius is not per mode and is left out. A theme with a
+ * missing row ships that gap into every repo that installs it.
+ */
+const THEME_ROWS = SHADCN_ROWS.filter((r) => r !== 'radius');
+function themeCheck(items) {
+  const worst = [];
+  let total = 0, incomplete = 0;
+  for (const it of items) {
+    if (!/^registry:(style|theme)$/.test(String(it?.type ?? ''))) continue;
+    const cv = it.cssVars ?? {};
+    if (!cv.light && !cv.dark) continue;
+    total += 1;
+    const has = (mode) => new Set(Object.keys(cv[mode] ?? {}).map((k) => k.replace(/^--/, '')));
+    const light = has('light'), dark = has('dark');
+    const missingLight = THEME_ROWS.filter((r) => !light.has(r));
+    const missingDark = THEME_ROWS.filter((r) => !dark.has(r));
+    if (missingLight.length || missingDark.length) {
+      incomplete += 1;
+      if (worst.length < 6) worst.push({ name: it.name, missingLight, missingDark });
+    }
+  }
+  return { total, incomplete, worst };
+}
+
+/**
+ * What is counted on a registry: the code it publishes, one variant of it.
+ * Everything else (the docs site, demos, examples, an installed catalogue
+ * for the site) is kept out and named, with file counts per folder.
+ */
+export function scopeFiles(reg, files) {
+  const under = (f, dirs) => dirs.some((d) => f === d || f.startsWith(`${d}/`));
+  const published = reg.publishedDirs ?? [];
+  const secondary = (reg.variants ?? []).filter((d) => !under(d, published) && !published.some((p) => p === d || p.startsWith(`${d}/`)));
+  const top = (f) => { const seg = f.split('/'); return (seg[0] === 'apps' || seg[0] === 'packages') && seg.length > 2 ? seg.slice(0, 2).join('/') : seg.length > 1 ? seg[0] : '(root)'; };
+  const out = { code: [], styles: [], other: files.other ?? [] };
+  const showcase = new Map(), variants = new Map();
+  for (const kind of ['code', 'styles']) {
+    for (const f of files[kind] ?? []) {
+      if (under(f, secondary)) { const d = secondary.find((v) => f.startsWith(`${v}/`)); variants.set(d, (variants.get(d) ?? 0) + 1); continue; }
+      if (under(f, published)) { out[kind].push(f); continue; }
+      showcase.set(top(f), (showcase.get(top(f)) ?? 0) + 1);
+    }
+  }
+  const list = (m) => [...m.entries()].sort((a, b) => b[1] - a[1]).map(([dir, n]) => ({ dir, files: n }));
+  return { files: out, showcase: list(showcase), variantsDropped: list(variants) };
+}
+
 /** The header sentence: what it publishes, in words. */
 export function publishesLine(r) {
   const parts = [];

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

@@ -121,7 +121,9 @@ const repoName = h.profile?.name ?? 'this repo';
       const files = d.files.map((f) => (typeof f === 'string' ? f : f.file));
       const ranked = [...files].sort((a, b) => sharedScore(b) - sharedScore(a));
       const clear = sharedScore(ranked[0]) > sharedScore(ranked[1]);
-      if (d.upstream) {
+      if (d.blocks) {
+        rule(`\`<${d.name}>\` is defined in ${files.length} published blocks, each a self-contained install. Copy the whole block or none of it; never import across blocks.`);
+      } else if (d.upstream) {
         rule(`\`<${d.name}>\` is defined by two shadcn components (${files.map((f) => `\`${f}\``).join(', ')}), upstream's overlap. Import whichever the surrounding code already uses; do not create a third.`);
       } else if (d.wrapped) {
         rule(`\`<${d.name}>\` is defined twice and one wraps the other${clear ? `. Import \`${ranked[0]}\`` : ''}; do not create a third.`);
@@ -179,6 +181,7 @@ if (neverImported.length >= 3) {
     const sheetFile = sc.sheet?.found ? sc.sheet.file : null;
     const paint = sc.paint ?? null;
     section('shadcn: the components and the theme');
+    if (P.isRegistry) rule(`This repo publishes a shadcn registry${P.registry?.publishes ? ` (${[['components', P.registry.publishes.components], ['blocks', P.registry.publishes.blocks], ['styles', P.registry.publishes.styles]].filter(([, v]) => v).map(([k, v]) => `${v} ${k}`).join(', ')})` : ''}. A palette colour, a bracket value or a hand-written dark: colour written here ships into every repo that installs it. Hold published code to the theme variables and the scale harder than app code, and keep demos and examples out of published files.`);
     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.`);
     if ((sc.sheet?.shadcnPresent ?? 0) >= 5 || sc.kit?.cssVariables === false) 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.');
     else rule(`${sheetFile ? `\`${sheetFile}\` defines` : 'The theme file defines'} none of shadcn's colour variables, so \`bg-background\` and \`text-muted-foreground\` have nothing behind them here. Until the theme variables are adopted, stay with the palette classes the surrounding file already uses; do not introduce semantic classes with no variable behind them, and do not add a second palette.`);