onsystem 0.7.0 → 0.8.0

package/dist/eslint-worker.d.ts

@@ -1,4 +1,4 @@
-import { h as DesignSystemModel, i as ResolvedConfig } from "./config-1XSTYI44.js";
+import { h as DesignSystemModel, i as ResolvedConfig } from "./config-kYGiD5oW.js";
 import { MessagePort } from "node:worker_threads";
 //#region src/eslint/worker.d.ts
 export interface LoadRequest {

package/dist/eslint-worker.js

@@ -1,5 +1,5 @@
-import { Ct as relativePath, K as DEFAULT_TEST_EXCLUDE, St as matchesGlob, i as componentFiles, q as DEFAULT_TEST_FOLDERS, r as buildModel } from "./design-system-C4bhqLLw.js";
-import { r as loadTarget } from "./workspace-Z0tZf_Dv.js";
+import { Q as excludedBy, Tt as relativePath, i as componentFiles, r as buildModel } from "./design-system-Dmcv7eHR.js";
+import { r as loadTarget } from "./workspace-CM1RyqZA.js";
 import fs from "node:fs";
 import path from "node:path";
 import { parentPort } from "node:worker_threads";
@@ -14,8 +14,6 @@ const warned = /* @__PURE__ */ new Set();
 /** How long a loaded design system is trusted before its files are looked at again. */
 const FRESH_MS = 2e3;
 const targets = /* @__PURE__ */ new Map();
-/** The default test and story patterns: the recommended config ignores those files itself. */
-const TEST_PATTERNS = /* @__PURE__ */ new Set([DEFAULT_TEST_EXCLUDE, DEFAULT_TEST_FOLDERS]);
 const projects = /* @__PURE__ */ new Map();
 function realPath(file) {
 	try {
@@ -65,7 +63,7 @@ async function load(request) {
 		projects.set(config.root, project);
 	}
 	const relative = relativePath(config.root, request.file);
-	const excluded = config.exclude.find((pattern) => !TEST_PATTERNS.has(pattern) && matchesGlob(relative, pattern));
+	const excluded = excludedBy(config, relative, { includeTests: true });
 	if (excluded) return { skip: `excluded by "${excluded}"` };
 	if (!config.includeDesignSystem && project.own.has(realPath(request.file))) return { skip: "one of the design system’s own component files" };
 	const { version } = project;

package/dist/eslint.js

@@ -1,4 +1,4 @@
-import { Ct as relativePath, G as DEFAULT_SEVERITY, K as DEFAULT_TEST_EXCLUDE, _ as NAME, k as RULES, q as DEFAULT_TEST_FOLDERS, t as DesignSystem, v as VERSION } from "./design-system-C4bhqLLw.js";
+import { G as DEFAULT_SEVERITY, K as DEFAULT_TEST_EXCLUDE, Tt as relativePath, _ as NAME, k as RULES, q as DEFAULT_TEST_FOLDERS, t as DesignSystem, v as VERSION } from "./design-system-Dmcv7eHR.js";
 import path from "node:path";
 import { MessageChannel, Worker, receiveMessageOnPort } from "node:worker_threads";
 //#region src/eslint/bridge.ts

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { C as TOKEN_CATEGORIES, D as VariantInfo, E as TokenCategory, O as ImportMapping, S as SourceLocation, T as Token, _ as DocSection, a as ResolvedRule, b as PropInfo, c as configJsonSchema, d as CheckResult, f as ComponentDocs, g as Diagnostic, h as DesignSystemModel, i as ResolvedConfig, l as defineConfig, m as CompoundVariantInfo, n as LoadConfigOptions, o as RuleId, p as ComponentInfo, r as RULE_IDS, s as RuleOptions, t as Config, u as loadConfig, v as ExampleInfo, w as TextEdit, x as Severity, y as InheritedProps } from "./config-1XSTYI44.js";
+import { C as TOKEN_CATEGORIES, D as VariantInfo, E as TokenCategory, O as ImportMapping, S as SourceLocation, T as Token, _ as DocSection, a as ResolvedRule, b as PropInfo, c as configJsonSchema, d as CheckResult, f as ComponentDocs, g as Diagnostic, h as DesignSystemModel, i as ResolvedConfig, l as defineConfig, m as CompoundVariantInfo, n as LoadConfigOptions, o as RuleId, p as ComponentInfo, r as RULE_IDS, s as RuleOptions, t as Config, u as loadConfig, v as ExampleInfo, w as TextEdit, x as Severity, y as InheritedProps } from "./config-kYGiD5oW.js";
 import ts from "typescript";
 import { Oklch } from "culori";
 import { McpServer, Transport } from "@modelcontextprotocol/server";
@@ -541,11 +541,24 @@ export declare class DesignSystem {
   private readonly searchIndex;
   /**
    * The components the tools offer: all of them, except a library's (MUI's) where the project
-   * has its own of that name, a wrapper the team means to be used. The rules still check code
-   * that imports the library's directly against the library's.
+   * has its own of that name, a wrapper the team means to be used, and the helpers in a
+   * registry folder's `lib/` (signoff-ui's `Img` and icons). The rules still check code that
+   * imports either directly against them.
    */
   private readonly offered;
+  /** The design systems the rules rank, with the registry folders a config lists. */
+  private readonly sources;
   constructor(model: DesignSystemModel, config: ResolvedConfig);
+  /**
+   * The shadcn registry a component was copied from (`@signoff-ui`), when its folder is one
+   * components.json names: a read-only design system, whose code is the registry's.
+   */
+  registryOf(component: ComponentInfo): {
+    registry: string;
+    folder: string;
+  } | undefined;
+  /** A component in a registry folder's `lib/`: an internal of its items, not one of them. */
+  private registryHelper;
   get root(): string;
   get components(): ComponentInfo[];
   get tokens(): Token[];

package/dist/index.js

@@ -1,3 +1,3 @@
-import { $ as loadConfig, A as TokenIndex, B as parseDoc, I as parseDtcgTokens, L as parseDesignMd, M as mergeTokens, N as toDesignMd, P as toDtcg, R as parseCssTokens, V as extractComponents, X as configJsonSchema, Y as RULE_IDS, Z as defineConfig, a as fingerprint, d as searchComponents, f as applyFixes, g as formatDiagnostics, j as loadTokens, k as RULES, n as DesignSystemHost, o as loadDesignSystem, p as checkSource, r as buildModel, rt as findVariantDefinitions, t as DesignSystem, v as VERSION, z as attachDocs } from "./design-system-C4bhqLLw.js";
-import { n as INSTRUCTIONS, r as createServer, t as serveStdio } from "./stdio-BojAQ6JP.js";
+import { A as TokenIndex, B as parseDoc, I as parseDtcgTokens, L as parseDesignMd, M as mergeTokens, N as toDesignMd, P as toDtcg, R as parseCssTokens, V as extractComponents, X as configJsonSchema, Y as RULE_IDS, Z as defineConfig, a as fingerprint, d as searchComponents, et as loadConfig, f as applyFixes, g as formatDiagnostics, it as findVariantDefinitions, j as loadTokens, k as RULES, n as DesignSystemHost, o as loadDesignSystem, p as checkSource, r as buildModel, t as DesignSystem, v as VERSION, z as attachDocs } from "./design-system-Dmcv7eHR.js";
+import { n as INSTRUCTIONS, r as createServer, t as serveStdio } from "./stdio-MTS6QAgo.js";
 export { DesignSystem, DesignSystemHost, INSTRUCTIONS, RULES, RULE_IDS, TokenIndex, VERSION, applyFixes, attachDocs, buildModel, checkSource, configJsonSchema, createServer, defineConfig, extractComponents, findVariantDefinitions, fingerprint, formatDiagnostics, loadConfig, loadDesignSystem, loadTokens, mergeTokens, parseCssTokens, parseDesignMd, parseDoc, parseDtcgTokens, searchComponents, serveStdio, toDesignMd, toDtcg };

package/dist/{stdio-BojAQ6JP.js → stdio-MTS6QAgo.js}

@@ -1,5 +1,5 @@
-import { $ as loadConfig, Ct as relativePath, F as TOKEN_CATEGORIES, G as DEFAULT_SEVERITY, H as CONFIG_FILES, J as LEGACY_CONFIG_FILES, Tt as defaultProjectConfig, Y as RULE_IDS, _ as NAME, _t as closest, at as LIBRARY_IDS, bt as unique, d as searchComponents, dt as detectProject, ft as findDesignSystemPackage, it as LIBRARIES, m as uncheckedNotice, n as DesignSystemHost, nt as formatPx, ot as libraryTitle, st as notReadYet, u as buildSearchIndex, v as VERSION, vt as plural, xt as isInside, yt as truncate } from "./design-system-C4bhqLLw.js";
-import { n as hasProjectMarker, r as loadTarget, t as findProjectRoot } from "./workspace-Z0tZf_Dv.js";
+import { Ct as isInside, Dt as defaultProjectConfig, F as TOKEN_CATEGORIES, G as DEFAULT_SEVERITY, H as CONFIG_FILES, J as LEGACY_CONFIG_FILES, Q as excludedBy, St as SCRIPT_FILE, Tt as relativePath, Y as RULE_IDS, _ as NAME, at as LIBRARIES, bt as truncate, ct as notReadYet, d as searchComponents, et as loadConfig, ft as detectProject, m as uncheckedNotice, n as DesignSystemHost, ot as LIBRARY_IDS, pt as findDesignSystemPackage, rt as formatPx, st as libraryTitle, u as buildSearchIndex, v as VERSION, vt as closest, xt as unique, yt as plural } from "./design-system-Dmcv7eHR.js";
+import { n as hasProjectMarker, r as loadTarget, t as findProjectRoot } from "./workspace-CM1RyqZA.js";
 import fs from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
@@ -47,7 +47,10 @@ function explainProject(ds, cwd, indent = "") {
 	field("importPath", importPaths(ds));
 	field("libraries", config.libraryIds === void 0 ? `every library onsystem reads (${LIBRARY_IDS.join(", ")})` : config.libraryIds !== false && config.libraryIds.length ? config.libraryIds.join(", ") : `none ("libraries": ${JSON.stringify(config.libraryIds)})`);
 	if (config.tailwindConfig) field("tailwind", display(config.root, config.tailwindConfig));
-	if (config.designSystems) field("primary", config.designSystems.map((d, i) => `${i ? "then " : ""}${d.source} (${d.imports} imports)`).join("; "));
+	if (config.designSystems) {
+		const ranked = config.designSystems.filter((d) => !d.registry).length > 1;
+		field("primary", config.designSystems.map((d, i) => `${i ? "then " : ""}${d.source}${d.registry ? ", read-only" : ranked ? ` (${d.imports} imports)` : ""}`).join("; "));
+	}
 	const changed = RULE_IDS.filter((id) => config.rules[id].severity !== DEFAULT_SEVERITY[id]);
 	field("rules", `${RULE_IDS.map((id) => `${id} ${config.rules[id].severity}`).join(", ")}${changed.length ? ` (changed: ${changed.join(", ")})` : ""}`);
 	if (config.overrides?.length) for (const override of config.overrides) field("override", `${override.files.join(", ")}: ${Object.entries(override.rules).map(([id, rule]) => `${id} ${rule.severity}`).join(", ")}`);
@@ -236,7 +239,7 @@ function renderComponentList(ds, project) {
 	const parts = ds.components.length - roots.length;
 	const lines = [`${plural(roots.length, "component")}${parts ? ` (+${plural(parts, "part")})` : ""}${project ? ` in ${project}` : ""}. Call get_component before using one.`, ""];
 	const own = roots.filter((c) => !c.library);
-	for (const component of own) lines.push(...listItem(component));
+	for (const component of own) lines.push(...listItem(component, registryNote(ds, component)));
 	for (const library of new Set(roots.flatMap((c) => c.library ? [c.library] : []))) {
 		if (own.length) lines.push("");
 		lines.push(`From ${library} (import { Name } from "${library}"):`, "");
@@ -244,6 +247,11 @@ function renderComponentList(ds, project) {
 	}
 	return lines.join("\n");
 }
+/** ` (from @signoff-ui, read-only)` for a component copied from a shadcn registry. */
+function registryNote(ds, component) {
+	const from = ds.registryOf(component);
+	return from ? ` (from ${from.registry}, read-only)` : "";
+}
 function listItem(component, note = "", withImport = true) {
 	const element = component.element ? ` <${component.element}>` : "";
 	const lines = [`${component.name}${element}${note} — ${firstSentence(component.description) || "No description."}`];
@@ -376,6 +384,8 @@ function renderComponent(ds, component, origin) {
 	].filter(Boolean);
 	lines.push(facts.join(" · "));
 	if (origin) lines.push(`From ${origin.package}, used by ${listOf$1(origin.projects)}.`);
+	const registry = ds.registryOf(component);
+	if (registry) lines.push(`Installed from the shadcn registry ${registry.registry} into ${registry.folder}: use it as it is. Its code is the registry's, so change it upstream, not here; \`npx shadcn add --overwrite\` replaces the file.`);
 	lines.push("", "## Props");
 	if (!component.props.length && !component.inherits.length) lines.push("No props.");
 	for (const prop of component.props) lines.push(renderProp(prop));
@@ -565,6 +575,10 @@ const MAX_CODE = 1e6;
 /** Diagnostics check_ui returns by default: a 3,000-line file can have thousands, more than a context holds. */
 const DEFAULT_LIMIT = 50;
 const MAX_LIMIT = 1e3;
+/** check_ui's answer for a path or filename that is not JS or TS. */
+function notScript(name) {
+	return errorResult(`check_ui checks .tsx, .jsx, .ts and .js files; ${name} is not one, so it was not checked.`);
+}
 /**
 * An integer, without the ±9007199254740991 bounds zod adds to the JSON Schema,
 * which every client would otherwise read in each tool definition.
@@ -999,7 +1013,7 @@ function createServer({ getDesignSystem, getProjects }) {
 		description: "Lint TSX/JSX against the design system. Reports hardcoded colors, spacing and radius (with the nearest token), native elements that have a design-system component, unknown components, props and variant values, and icon-only buttons without an accessible name. Each diagnostic has line/column, a rule id, a message and a suggested fix. Run it on every snippet or file you write and fix all errors. Pass `code` for unsaved code, or `path` for a file in the project.",
 		inputSchema: z.object({
 			code: z.string().max(MAX_CODE).optional().describe(`TSX/JSX source to check (up to ${MAX_CODE.toLocaleString("en")} characters).`),
-			path: z.string().max(MAX_PATH).optional().describe("Path of a file to check: relative to the project root (the monorepo root, when served from there), or absolute."),
+			path: z.string().max(MAX_PATH).optional().describe("Path of a .tsx, .jsx, .ts or .js file to check: relative to the project root (the monorepo root, when served from there), or absolute."),
 			filename: z.string().max(MAX_PATH).optional().describe("Name to report `code` under; a .jsx extension parses it as JSX. At a monorepo root, the path the code will be saved under, which picks the project whose design system checks it. When the server was started outside your project, pass it as an absolute path: the project is found from it."),
 			limit: z.number().int().min(1).max(MAX_LIMIT).default(DEFAULT_LIMIT).describe(`Most diagnostics to return, errors first (default ${DEFAULT_LIMIT}). The counts and byRule cover all of them.`)
 		}),
@@ -1014,7 +1028,7 @@ function createServer({ getDesignSystem, getProjects }) {
 				errors: int(),
 				warnings: int()
 			})).describe("Every finding per rule, shown or not."),
-			skipped: z.string().optional().describe("Why the file was not checked: it renders an image (next/og) or an email."),
+			skipped: z.string().optional().describe("Why the file was not checked: it renders an image (next/og) or an email, or the config's `exclude` leaves it out."),
 			unchecked: z.object({
 				names: z.array(z.string()),
 				modules: z.array(z.string())
@@ -1024,6 +1038,7 @@ function createServer({ getDesignSystem, getProjects }) {
 		annotations: READ_ONLY
 	}, async ({ code, path: filePath, filename, limit }) => {
 		if (code === void 0 && !filePath) return errorResult("Pass either `code` or `path`.");
+		if (code !== void 0 && filename !== void 0 && !SCRIPT_FILE.test(filename)) return notScript(filename);
 		const routeBy = code === void 0 ? filePath : filename ?? void 0;
 		const set = await getProjects?.(routeBy);
 		let ds;
@@ -1048,6 +1063,7 @@ function createServer({ getDesignSystem, getProjects }) {
 			const absolute = path.resolve(base, filePath);
 			const outside = errorResult(`${filePath} is outside the project root (${base}).`);
 			if (!isInside(base, absolute)) return outside;
+			if (!SCRIPT_FILE.test(absolute)) return notScript(filePath);
 			try {
 				const [root, real] = await Promise.all([fsp.realpath(base), fsp.realpath(absolute)]);
 				if (!isInside(root, real)) return outside;
@@ -1071,7 +1087,14 @@ function createServer({ getDesignSystem, getProjects }) {
 			file = relativePath(ds.root, absolute);
 			shown = relativePath(base, absolute);
 		}
-		const checked = ds.check(source, file);
+		const excluded = (filename !== void 0 || code === void 0) && !path.isAbsolute(file) ? excludedBy(ds.config, file) : void 0;
+		const checked = excluded ? {
+			file,
+			diagnostics: [],
+			errorCount: 0,
+			warningCount: 0,
+			skipped: `It is left out by "${excluded}" in the config's exclude, as in check, the ESLint plugin and the Claude Code hook.`
+		} : ds.check(source, file);
 		const result = file === shown ? checked : {
 			...checked,
 			file: shown

package/dist/{workspace-Z0tZf_Dv.js → workspace-CM1RyqZA.js}

@@ -1,4 +1,4 @@
-import { $ as loadConfig, Ct as relativePath, H as CONFIG_FILES, J as LEGACY_CONFIG_FILES, Q as hasOwnConfig, et as loadConfigWith, ft as findDesignSystemPackage, gt as workspacePatterns, ht as workspacePackageDirs, i as componentFiles, lt as PackageFinder, mt as isWorkspaceRoot, xt as isInside } from "./design-system-C4bhqLLw.js";
+import { $ as hasOwnConfig, Ct as isInside, H as CONFIG_FILES, J as LEGACY_CONFIG_FILES, Tt as relativePath, _t as workspacePatterns, et as loadConfig, gt as workspacePackageDirs, ht as isWorkspaceRoot, i as componentFiles, pt as findDesignSystemPackage, tt as loadConfigWith, ut as PackageFinder } from "./design-system-Dmcv7eHR.js";
 import fs from "node:fs";
 import path from "node:path";
 import os from "node:os";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "onsystem",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "mcpName": "io.github.dgesteves/onsystem",
   "description": "Keeps coding agents on your design system: catches palette colors, native elements and APIs your system lacks, at every edit and pull request, with the fix. Set it up once in your design-system package; every app in your monorepo gets it.",
   "keywords": [

package/schema.json

@@ -22,7 +22,7 @@
       ]
     },
     "exclude": {
-      "description": "Globs to ignore when finding components and when `check` expands folders and globs (stories, tests). Replaces the defaults; node_modules is skipped either way unless a pattern names it.",
+      "description": "Globs for files to leave out, relative to the root: `check` in folders and globs, the ESLint plugin, the Claude Code hook and check_ui leave them out, and finding components skips them. A folder (`app/legacy`) covers everything in it. Adds to the defaults (node_modules, .d.ts files, tests and stories); a pattern that starts with `!` takes out the default it names, such as `!**/*.d.ts`.",
       "type": "array",
       "items": {
         "type": "string"