taste-lint 0.0.6 → 0.0.8

package/README.md

@@ -19,20 +19,20 @@ Scan your project with [Jev by TypeSafe AI](https://docs.typesafe.ai/introductio
 npx taste-lint@latest init
 ```
 
-Requires Node 24.11 or later. Run from your project directory to install locally and add a scan script.
+Requires Node 24.11 or later. Run it from your project directory. Init installs locally and adds a scan script.
 
 ## Quickstart
 
-Create a [Vercel AI Gateway key](https://vercel.com/docs/ai-gateway/authentication-and-byok/api-keys). From your project directory, set your key and run:
+Create a [Vercel AI Gateway key](https://vercel.com/docs/ai-gateway/authentication-and-byok/api-keys). Then:
 
 ```bash
 export AI_GATEWAY_API_KEY="your-vercel-ai-gateway-key"
 npm run taste
 ```
 
-No account or config for taste-lint. AI checks send selected text and rule context to Vercel AI Gateway, billed to your account. Answers are cached for repeat runs.
+No taste-lint account or config. AI checks send selected text and rule context to Vercel AI Gateway, billed to your account. Repeat runs reuse cached answers.
 
-Use your package manager in place of npm. Add `--agent` to init for agent instructions, or `--dry-run` to preview setup.
+Use your package manager in place of npm. Pass `--agent` to init for agent instructions, or `--dry-run` to preview setup.
 
 ## What it checks
 
@@ -40,17 +40,13 @@ Use your package manager in place of npm. Add `--agent` to init for agent instru
 - **Writing:** Markdown, MDX, and READMEs with `--profile writing`.
 - **Agent instructions:** AGENTS.md and skills with `--profile instructions`.
 
-Rules draw on [Agent Skills](https://github.com/mblode/agent-skills) and [Taste Training](https://blode.co/taste-training). Local checks handle measurable rules. Jev judges meaning and returns probabilities. Uncalibrated AI rules stay advisory; active findings can fail a run.
+Rules come from [Agent Skills](https://github.com/mblode/agent-skills) and [Taste Training](https://blode.co/taste-training). Local checks handle measurable rules. Jev judges meaning and returns probabilities. Uncalibrated AI rules stay advisory. Active findings can fail a run.
 
-## Useful options
+## Docs
 
-| Option           | What it does                                         |
-| ---------------- | ---------------------------------------------------- |
-| `--dry-run`      | Preview scope and estimated cost without model calls |
-| `--output json`  | Save findings for scripts and agents                 |
-| `--output sarif` | Export findings for code review tools                |
+[taste-lint.blode.md](https://taste-lint.blode.md)
 
-Run `taste-lint scan --help` for all options. See the [scan guide](https://github.com/mblode/taste-lint/blob/main/docs/SCANS.md), [usage reference](https://github.com/mblode/taste-lint/blob/main/docs/USAGE.md), and [changelog](https://github.com/mblode/taste-lint/blob/main/CHANGELOG.md) for more.
+`taste-lint scan --help` lists every option. `--dry-run` previews scope and cost. `--output json` and `--output sarif` are for scripts and code review.
 
 ## License
 

package/dist/cli.js

@@ -16,7 +16,7 @@ import { mdxjs } from "micromark-extension-mdxjs";
 import { styleText } from "node:util";
 import os from "node:os";
 //#region package.json
-var version = "0.0.6";
+var version = "0.0.8";
 //#endregion
 //#region src/lib/stats.ts
 const makePRNG = (seed) => {
@@ -444,6 +444,7 @@ const loadCorpus = (corpusDir, rules, options = {}) => {
 			if (r.context !== void 0) {
 				if (!isRecord$1(r.context)) throw new Error(`Invalid corpus line ${where}: context must be an object`);
 				const { docType, role } = r.context;
+				if (r.context.fontScale !== void 0 && (!isRecord$1(r.context.fontScale) || Object.values(r.context.fontScale).some((value) => typeof value !== "string"))) throw new Error(`Invalid corpus line ${where}: fontScale must map tokens to strings`);
 				if (docType !== void 0 && !DOC_TYPES.includes(docType) || role !== void 0 && !ROLES.includes(role)) throw new Error(`Invalid corpus line ${where}: unknown context docType or role`);
 			}
 			if (r.neighbours !== void 0) {
@@ -661,7 +662,7 @@ const stripVariants = (cls) => {
 	};
 };
 const isColour = (token) => {
-	if (token.startsWith("[#") || token.startsWith("[rgb") || token.startsWith("[hsl") || token.startsWith("[oklch") || token.startsWith("[var(")) return true;
+	if (token.startsWith("[#") || token.startsWith("[rgb") || token.startsWith("[hsl") || token.startsWith("[oklch") || token.startsWith("[oklab") || token.startsWith("[color:") || token.startsWith("(color:")) return true;
 	return COLOUR_NAMES.has(token.replace(/-\d{2,3}$/, ""));
 };
 const resolveTypography = (classes, config) => {
@@ -680,7 +681,7 @@ const resolveTypography = (classes, config) => {
 		if (variants.some((v) => !["dark"].includes(v))) continue;
 		if (base.startsWith("text-")) {
 			const [sizeToken, leadingToken] = base.slice(5).split("/");
-			if (isColour(sizeToken)) {
+			if (!Object.hasOwn(theme, sizeToken) && isColour(sizeToken)) {
 				out.colourClasses.push(cls);
 				continue;
 			}
@@ -699,21 +700,25 @@ const resolveTypography = (classes, config) => {
 				"clip"
 			].includes(sizeToken)) continue;
 			const known = FONT_SIZES[sizeToken];
-			if (known) {
+			const declared = Object.hasOwn(theme, sizeToken);
+			const arbitrary = sizeToken.startsWith("[") && sizeToken.endsWith("]");
+			if (declared || arbitrary) {
+				const raw = declared ? theme[sizeToken] : sizeToken.slice(1, -1).replace(/^length:/u, "");
+				const px = parseLength(raw);
+				out.fontSizePx = px ?? void 0;
+				if (lineHeightFromSize) {
+					out.lineHeightPx = void 0;
+					out.lineHeight = void 0;
+					lineHeightFromSize = false;
+				}
+				if (px === null) out.unresolved.push(cls);
+			} else if (known) {
 				out.fontSizePx = known[0];
 				if (!leadingToken && (lineHeightFromSize || out.lineHeightPx === void 0 && out.lineHeight === void 0)) {
 					out.lineHeightPx = known[1];
 					out.lineHeight = void 0;
 					lineHeightFromSize = true;
 				}
-			} else if (sizeToken.startsWith("[") && sizeToken.endsWith("]")) {
-				const px = parseLength(sizeToken.slice(1, -1));
-				if (px === null) out.unresolved.push(cls);
-				else out.fontSizePx = px;
-			} else if (theme[sizeToken]) {
-				const px = parseLength(theme[sizeToken]);
-				if (px === null) out.unresolved.push(cls);
-				else out.fontSizePx = px;
 			} else out.unresolved.push(cls);
 			if (leadingToken) applyLeading(leadingToken, cls);
 			continue;
@@ -784,6 +789,51 @@ const resolveLeading = (out, token, cls) => {
 	out.unresolved.push(cls);
 };
 const splitClasses = (value) => value.split(/\s+/u).filter(Boolean);
+/** Only explicit pixel font tokens establish a scale for this comparison.
+* Relative lengths need a rendered root size; other families need their own
+* theme evidence. Neither is inferred from Tailwind's default typography.
+*/
+const compareScale = (classes, theme = {}) => {
+	const matches = [];
+	const unresolved = [];
+	for (const className of classes) {
+		const { base, variants } = stripVariants(className);
+		const candidate = base.match(/^(text|p[xytrblse]?|m[xytrblse]?|gap(?:-[xy])?|space-[xy]|rounded(?:-[a-z]+)?|leading|w|h|size|inset|top|bottom|left|right)-\[(-?(?:\d+(?:\.\d+)?|\.\d+))px\]$/u);
+		if (!candidate || Math.abs(Number(candidate[2])) <= 2) continue;
+		if (candidate[1] !== "text") {
+			unresolved.push(`${className}: ${candidate[1]} scale family is unsupported`);
+			continue;
+		}
+		const entries = Object.entries(theme);
+		if (!entries.length) {
+			unresolved.push(`${className}: no explicitly declared font scale`);
+			continue;
+		}
+		const value = Number(candidate[2]);
+		const resolved = entries.flatMap(([name, raw]) => {
+			if (!/^[\w-]+$/u.test(name) || !/^(?:\d+(?:\.\d+)?|\.\d+)px$/u.test(raw)) return [];
+			const stepPx = Number(raw.slice(0, -2));
+			return Number.isFinite(stepPx) && stepPx > 0 ? [{
+				difference: Math.abs(value - stepPx),
+				name,
+				stepPx
+			}] : [];
+		});
+		const nearest = resolved.toSorted((a, b) => a.difference - b.difference || a.name.localeCompare(b.name))[0];
+		const tolerance = Number.EPSILON * Math.max(1, Math.abs(value), nearest?.stepPx ?? 0);
+		if (nearest && nearest.difference <= 1 + tolerance) matches.push({
+			className,
+			difference: Math.min(nearest.difference, 1),
+			stepPx: nearest.stepPx,
+			token: [...variants, `text-${nearest.name}`].join(":")
+		});
+		else if (resolved.length < entries.length) unresolved.push(`${className}: declared font scale contains unresolved values`);
+	}
+	return {
+		matches,
+		unresolved
+	};
+};
 //#endregion
 //#region src/lib/errors.ts
 var InputError = class extends Error {
@@ -2543,7 +2593,7 @@ const ARCHITECTURE_RULES = [
 const MODULE$1 = "src/rules/code/classes.ts";
 const ANIMATION = "skills/ui-animation/references";
 const bases = (unit) => (unit.classes ?? []).map(baseClass);
-const animates = (list) => list.some((c) => c.startsWith("transition") || c.startsWith("duration-") || c.startsWith("animate-") && c !== "animate-none");
+const animates = (list) => list.some((c) => c.startsWith("transition") && c !== "transition-none" || c.startsWith("duration-") || c.startsWith("animate-") && c !== "animate-none");
 const durationMs = (cls) => {
 	const named = cls.match(/^duration-(\d+)$/u);
 	if (named) return Number(named[1]);
@@ -2573,6 +2623,29 @@ const shadcn = (spec) => rule$1({
 	unit: ["class-list"]
 });
 const CLASS_RULES = [
+	rule$1({
+		categoryId: "look-constraints",
+		check: (unit) => {
+			const compared = compareScale(unit.classes ?? [], unit.context.fontScale);
+			if (!compared.matches.length) {
+				if (compared.unresolved.length) throw new UnresolvedError(compared.unresolved.join("; "));
+				return none;
+			}
+			if (unit.context.dynamic) throw new UnresolvedError("Dynamic classes may override the near-scale candidate");
+			return hit(compared.matches.map(({ className, token, stepPx, difference }) => `${className} is ${difference}px from ${token} (${stepPx}px)`).join("; "));
+		},
+		hint: "Review the matching font token before changing this value. If the difference is intentional or recurring, name the existing size in the theme instead of changing its appearance. This comparison does not establish design intent.",
+		id: "craft-near-duplicate-scale",
+		source: {
+			line: 10,
+			path: "skills/ui-design/rules/slop-near-duplicate-scale.md",
+			repo: "mblode/agent-skills",
+			ruleId: "slop-near-duplicate-scale"
+		},
+		status: "review-only",
+		title: "Arbitrary font size within 1px of a declared theme step",
+		unit: ["class-list"]
+	}),
 	motion({
 		categoryId: "easing-and-duration",
 		check: (unit) => {
@@ -2606,7 +2679,7 @@ const CLASS_RULES = [
 		categoryId: "easing-and-duration",
 		check: (unit) => {
 			const list = bases(unit);
-			if (!list.some((c) => c.startsWith("transition"))) return none;
+			if (!list.some((c) => c.startsWith("transition") && c !== "transition-none")) return none;
 			const slow = list.filter((c) => (durationMs(c) ?? 0) > 300);
 			return slow.length > 0 ? hit(slow.join(" ")) : none;
 		},
@@ -2675,13 +2748,18 @@ const CLASS_RULES = [
 	shadcn({
 		categoryId: "look-constraints",
 		check: (unit) => {
-			const found = bases(unit).filter((c) => /-\[[^\]]+\]/u.test(c));
+			const found = bases(unit).filter((c) => {
+				const value = c.match(/^-?[\w-]+-\[(.+?)\](?:\/[^\s]+)?!?$/u)?.[1]?.replace(/^(?:length|color):/u, "");
+				if (!value) return false;
+				if (/var\(|env\(|theme\(|^--|^(?:calc|min|max|clamp|minmax|repeat|url)\(/u.test(value)) return false;
+				return /^-?(?:\d+(?:\.\d+)?|\.\d+)(?:px|rem|em|ms|s|deg)?$/u.test(value) || /^#[\da-f]{3,8}$/iu.test(value) || /^(?:rgb|rgba|hsl|hsla|oklab|oklch|color)\(/u.test(value);
+			});
 			return found.length > 0 ? hit(found.join(" ")) : none;
 		},
-		hint: "Step the scale (p-3, text-sm, rounded-md) or register the value in the theme so the next component can reach it by name. p-[13px] is a nudge, not a decision.",
+		hint: "Review whether an existing token serves this literal value. Keep deliberate layout, asset and optical constraints; register a recurring design value if useful. This syntax check does not establish a defect or a safe replacement.",
 		id: "craft-arbitrary-value-class",
 		ruleId: "shadcn/no-arbitrary-values",
-		title: "Arbitrary value in a utility class"
+		title: "Literal arbitrary utility value to review"
 	}),
 	shadcn({
 		categoryId: "look-constraints",
@@ -4475,6 +4553,7 @@ const extractTsx = (file, source, options) => {
 				docType,
 				dynamic: classInfo.dynamic || void 0,
 				element: name,
+				fontScale: config?.tailwind.theme,
 				interpolated: classInfo.interpolated || void 0,
 				role
 			},
@@ -4673,6 +4752,7 @@ const agentText = `${marker}
 Taste Lint uses Jev to judge copy and UI. Preview the taste script with --dry-run, then run it with a user-supplied AI_GATEWAY_API_KEY.
 Fix act findings, review advisory findings in context, and recheck the edited files.
 Never invent a key or treat unknown checks as passes.
+Docs: https://taste-lint.blode.md
 <!-- /taste-lint -->
 `;
 function object$1(value) {
@@ -5486,7 +5566,7 @@ const renderSarif = (result, rules, version) => {
 				ruleIndex: index.get(f.ruleId)
 			})),
 			tool: { driver: {
-				informationUri: "https://github.com/mblode/taste-lint",
+				informationUri: "https://taste-lint.blode.md",
 				name: "taste-lint",
 				rules: ruleList.map((r) => ({
 					fullDescription: { text: r.question?.instructions ?? r.fix.hint },
@@ -5760,6 +5840,14 @@ const artifacts = [
 	"**/docs/archive/**",
 	"**/.captain/**"
 ];
+const stylePreferences = [
+	"copywriting-long-sentence",
+	"typography-straight-quotes",
+	"typography-ellipsis",
+	"typography-dashes",
+	"typography-midpoint-separators",
+	"typography-line-height-out-of-band"
+];
 const internal = [
 	"**/.claude/**",
 	"**/.agents/**",
@@ -5804,6 +5892,7 @@ const profileFor = (name) => {
 		};
 		case "product": return {
 			...common,
+			advisory: stylePreferences,
 			exclude: [
 				...artifacts,
 				...internal,
@@ -5814,7 +5903,7 @@ const profileFor = (name) => {
 		};
 		case "writing": return {
 			...common,
-			advisory: ["copywriting-long-sentence", "typography-straight-quotes"],
+			advisory: stylePreferences,
 			docTypes: [{
 				glob: "**/content/writing/**/*.{md,mdx}",
 				type: "personal"
@@ -5834,7 +5923,7 @@ const profileFor = (name) => {
 		};
 		case "instructions": return {
 			...common,
-			advisory: ["copywriting-long-sentence", "typography-straight-quotes"],
+			advisory: stylePreferences,
 			include: [...internal, "**/docs/plans/**/*.md"],
 			objective: "Review repository instructions, skill authoring, and implementation plans"
 		};
@@ -6231,6 +6320,7 @@ const runScan = async (options, ctx = {}) => {
 			...config,
 			root: void 0
 		},
+		engineVersion: version,
 		exclude,
 		graph: graph?.digest,
 		model: options.model ?? "jev-latest",
@@ -6652,6 +6742,7 @@ Quickstart:
 
 Preview: taste-lint scan . --dry-run
 Get a key: https://vercel.com/docs/ai-gateway/authentication-and-byok/api-keys
+Docs: https://taste-lint.blode.md
 `);
 registerLintCommand(program);
 registerInitCommand(program);