roast-my-design-system 6.0.0 → 6.0.1

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=6.0.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=6.0.1" width="72" alt="roast-my-design-system">
 
 # roast-my-design-system
 
@@ -71,11 +71,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=6.0.0)
+![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=6.0.1)
 
 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=6.0.0)
+![The diagnosis report in light mode](https://raw.githubusercontent.com/gregkozakiewicz/roast-my-design-system/main/assets/report-light-hero.png?v=6.0.1)
 
 ## What makes the numbers trustworthy
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roast-my-design-system",
-  "version": "6.0.0",
+  "version": "6.0.1",
   "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/harvest/tokens.mjs

@@ -12,6 +12,7 @@ import {
 } from '../lib/exempt.mjs';
 import { join } from 'node:path';
 import { canonical } from '../lib/color.mjs';
+import { TOKEN_REF_SRC } from '../lib/declarations.mjs';
 import { tokenCollisions, workspaceMatcher } from './collisions.mjs';
 
 // ---------- counters ----------
@@ -102,8 +103,9 @@ const FONTSIZE_PROPS = /(?:^|[^-\w])font-size\s*:\s*([^;{}]+)[;}]/g;
  * sizes of which 48 were var(). Same fault that once made one Roboto look like
  * 59 typefaces (found 2026-09-11 while checking the density axis).
  */
+const TOKEN_REF_RE = new RegExp(`^(${TOKEN_REF_SRC})$`, 'i');
 const isTokenRef = (v) =>
-  /^(var\(\s*--[\w-]+[^)]*\)|inherit|initial|unset|revert)$/i.test(v)
+  TOKEN_REF_RE.test(v)
   || v.startsWith('$') || /\bmap[.-]get\b/.test(v);
 const FONTFAMILY_PROPS = /font-family\s*:\s*([^;{}]+)[;}]/g;
 const SHADOW_PROPS = /box-shadow\s*:\s*([^;{}]+)[;}]/g;

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

@@ -20,9 +20,19 @@ export const EXTRA_KINDS = [
 
 export const FONT_LINE_RE = /font-family\s*:\s*([^;{}]+)/i;
 
+// One definition of "this value is a reference into the system", shared with
+// the harvest's isTokenRef so the counter and the checker can never disagree
+// about what a token reference looks like again (the 5.11 contradiction, and
+// docs/variant-counting-findings.md §10). Fallbacks are allowed: var(--x, 4px)
+// is still the system deciding.
+export const TOKEN_REF_SRC = String.raw`var\(\s*--[\w-]+[^)]*\)|inherit|initial|unset|revert`;
+
 // Disciplined values that are never sins: token use, resets, inheritance.
-export const BENIGN_VALUE_RE = /^(var\(--[\w-]+\)|inherit|initial|unset|none|normal|0)$/i;
-const FONT_BENIGN_RE = /^(var\(--[\w-]+\)|inherit)$/i;
+// Deliberately wider than a token reference: none, normal and 0 are resets a
+// checker must not flag, but they reference nothing, so the counter ignores
+// them. The difference is design, not drift.
+export const BENIGN_VALUE_RE = new RegExp(`^(${TOKEN_REF_SRC}|none|normal|0)$`, 'i');
+const FONT_BENIGN_RE = new RegExp(String.raw`^(var\(\s*--[\w-]+[^)]*\)|inherit)$`, 'i');
 
 /** The first declaration of one kind on one line, or null. The guard's door. */
 export function extraValue(re, text) {

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

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