diffing 0.18.1 → 0.18.3

package/dist/{mcp-DoNBucix.mjs → mcp-45VPdWIr.mjs}

@@ -1,9 +1,9 @@
-import { i as buildGitDiffArgs, s as parseDiffOptions, t as startServer } from "./server-BRQdb64n.mjs";
-import { r as formatPlanReview, t as formatMockupReview } from "./mockup-format-D4Ln0WxJ.mjs";
-import { t as formatComments } from "./comment-format-D4mCqr7T.mjs";
-import { _ as generateSessionToken, c as removeServerLockIfOwned, i as isLockAlive, p as writeServerLock, r as diffScopeKey, s as removeServerLock, t as acquireServerStartupLease, u as resolveActiveServerLock, x as SESSION_TOKEN_HEADER, y as isLoopbackHost } from "./server-lock-CdaUNZyX.mjs";
-import { i as reviewSessionUrl, n as joinSessionApiUrl, r as reviewSessionBaseUrl, t as appendSessionToken } from "./session-url-B9FOnF3F.mjs";
-import { n as AWAIT_REVIEW_TIMEOUT_NEXT_ACTION, r as AWAIT_TOOL_DESCRIPTION_SUFFIX, s as PLAN_SUBMIT_NEXT_ACTION, t as AWAIT_PLAN_TIMEOUT_NEXT_ACTION } from "./handoff-DcShCxWJ.mjs";
+import { i as buildGitDiffArgs, s as parseDiffOptions, t as startServer } from "./server-CH-UPGFR.mjs";
+import { a as formatPlanReview, r as formatMockupReview, t as formatSubmitHints } from "./mockup-lint-BRjhxXSw.mjs";
+import { t as formatComments } from "./comment-format-DS1wOWj_.mjs";
+import { _ as generateSessionToken, c as removeServerLockIfOwned, i as isLockAlive, p as writeServerLock, r as diffScopeKey, s as removeServerLock, t as acquireServerStartupLease, u as resolveActiveServerLock, x as SESSION_TOKEN_HEADER, y as isLoopbackHost } from "./server-lock-ACw2cKDI.mjs";
+import { i as reviewSessionUrl, n as joinSessionApiUrl, r as reviewSessionBaseUrl, t as appendSessionToken } from "./session-url-CyBpcLAH.mjs";
+import { n as AWAIT_REVIEW_TIMEOUT_NEXT_ACTION, r as AWAIT_TOOL_DESCRIPTION_SUFFIX, s as PLAN_SUBMIT_NEXT_ACTION, t as AWAIT_PLAN_TIMEOUT_NEXT_ACTION } from "./handoff-B85WLpcN.mjs";
 import { execFileSync } from "node:child_process";
 import { randomUUID } from "node:crypto";
 import { fileURLToPath } from "node:url";
@@ -1517,11 +1517,18 @@ function createMcpServer(options) {
 			screens: z.array(z.object({
 				id: z.string().optional(),
 				label: z.string().optional(),
-				html: z.string()
+				html: z.string(),
+				stateOf: z.string().optional(),
+				flow: z.string().optional()
 			})).optional(),
 			mockupId: z.string().optional(),
 			source: z.string().optional(),
-			model: z.string().optional()
+			model: z.string().optional(),
+			mode: z.enum(["fragment", "document"]).optional(),
+			designSystem: z.string().optional(),
+			planId: z.string().optional(),
+			fromMockupId: z.string().optional(),
+			blank: z.boolean().optional()
 		},
 		outputSchema: {
 			mockupId: z.string(),
@@ -1533,7 +1540,7 @@ function createMcpServer(options) {
 			readOnlyHint: false,
 			idempotentHint: false
 		}
-	}, async ({ title, html, screens, mockupId, source, model }) => {
+	}, async ({ title, html, screens, mockupId, source, model, mode, designSystem, planId, fromMockupId, blank }) => {
 		const session = requireWebSession();
 		await seedMockupCursor(session, void 0, true);
 		const mockup = await requestSessionJson(session, "/api/mockups", {
@@ -1545,12 +1552,17 @@ function createMcpServer(options) {
 				html,
 				screens,
 				source,
-				model
+				model,
+				mode,
+				designSystemId: designSystem,
+				planId,
+				fromMockupId,
+				blank
 			})
 		});
 		const url = appendSessionToken(`${session.apiOrigin}/mockup/${mockup.id}`, session.lock.authToken);
 		const hints = mockup.hints ?? [];
-		const hintText = hints.length > 0 ? `\n⚠️ In-page state UI detected — split each state into its own screen:\n${hints.map((h) => `  • ${h.screenId}: ${h.patterns.join(", ")}`).join("\n")}` : "";
+		const hintText = formatSubmitHints(hints);
 		return textResult(`Submitted mockup ${mockup.id} (v${mockup.version}) at ${url}.${hintText} ${MOCKUP_SUBMIT_NEXT_ACTION}`, {
 			mockupId: mockup.id,
 			version: mockup.version,
@@ -1724,7 +1736,8 @@ function createMcpServer(options) {
 				"summary",
 				"comments",
 				"comment",
-				"screen"
+				"screen",
+				"preview"
 			]).default("summary"),
 			status: z.enum(["open", "resolved"]).optional(),
 			screenId: z.string().optional(),
@@ -1761,18 +1774,20 @@ function createMcpServer(options) {
 	});
 	server.registerTool("revise_mockup", {
 		title: "Revise a mockup screen",
-		description: "One-screen revision of an HTML mockup. op=upsert adds/replaces a screen (html inline, never files on disk), op=remove deletes a screen, op=patch replaces the first exact occurrence of expectedText with replacement. Every success bumps the mockup version; pass expectedVersion to guard against racing edits (409 version-mismatch on conflict, nothing applied). For revisions touching many screens, resubmit via submit_mockup with the same mockupId instead.",
+		description: "One-screen revision of an HTML mockup. op=upsert adds/replaces a screen (html inline, never files on disk), op=remove deletes a screen, op=patch replaces the first exact occurrence of expectedText with replacement, op=replace-region replaces the inner HTML of the first [data-diffing=region] element. Every success bumps the mockup version; pass expectedVersion to guard against racing edits (409 version-mismatch on conflict, nothing applied). For revisions touching many screens, resubmit via submit_mockup with the same mockupId instead.",
 		inputSchema: {
 			mockupId: z.string().min(1),
 			op: z.enum([
 				"upsert",
 				"remove",
-				"patch"
+				"patch",
+				"replace-region"
 			]),
 			screenId: z.string().min(1),
 			html: z.string().optional(),
 			label: z.string().optional(),
 			expectedText: z.string().optional(),
+			region: z.string().optional(),
 			replacement: z.string().optional(),
 			expectedVersion: z.number().int().positive().optional()
 		},
@@ -1783,7 +1798,7 @@ function createMcpServer(options) {
 			occurrences: z.number().optional()
 		},
 		annotations: MUTATING
-	}, async ({ mockupId, op, screenId, html, label, expectedText, replacement, expectedVersion }) => {
+	}, async ({ mockupId, op, screenId, html, label, expectedText, region, replacement, expectedVersion }) => {
 		const headers = { "Content-Type": "application/json" };
 		const base = `/api/mockups/${encodeURIComponent(mockupId)}/screens/${encodeURIComponent(screenId)}`;
 		if (op === "upsert") {
@@ -1798,7 +1813,7 @@ function createMcpServer(options) {
 				})
 			});
 			const hints = mockup.hints ?? [];
-			const hintText = hints.length > 0 ? `\n⚠️ In-page state UI detected — split each state into its own screen: ${hints.map((h) => h.patterns.join(", ")).join("; ")}` : "";
+			const hintText = formatSubmitHints(hints);
 			return textResult(`Upserted screen ${screenId} on ${mockupId} → v${mockup.version}.${hintText}`, {
 				mockupId,
 				version: mockup.version,
@@ -1814,6 +1829,24 @@ function createMcpServer(options) {
 				screenIds: mockup.screens.map((s) => s.id)
 			});
 		}
+		if (op === "replace-region") {
+			if (!region || replacement === void 0) throw new Error("revise_mockup op=replace-region requires region and replacement");
+			const data = await requestBaseJson(base, {
+				method: "PATCH",
+				headers,
+				body: JSON.stringify({
+					region,
+					replacement,
+					expectedVersion
+				})
+			});
+			return textResult(`Replaced region "${region}" on ${screenId} (${data.occurrences} match(es)) → v${data.mockup.version}`, {
+				mockupId,
+				version: data.mockup.version,
+				screenIds: data.mockup.screens.map((s) => s.id),
+				occurrences: data.occurrences
+			});
+		}
 		if (expectedText === void 0 || replacement === void 0) throw new Error("revise_mockup op=patch requires expectedText and replacement");
 		const data = await requestBaseJson(base, {
 			method: "PATCH",
@@ -1882,6 +1915,92 @@ function createMcpServer(options) {
 			results: data.results
 		});
 	});
+	server.registerTool("get_design_system", {
+		title: "Get the repo design system",
+		description: "Read the published (or draft) per-repo design system: tokens, guidelines, components. Call this before authoring mockup HTML. Omit id for the default system.",
+		inputSchema: { id: z.string().optional() },
+		outputSchema: { system: z.unknown().nullable() },
+		annotations: READ_ONLY
+	}, async ({ id }) => {
+		const path = id ? `/api/design-systems/${encodeURIComponent(id)}` : "/api/design-systems/default";
+		try {
+			return textResult("design system", { system: await requestBaseJson(path) });
+		} catch {
+			return textResult("No design system published yet.", { system: null });
+		}
+	});
+	server.registerTool("extract_design_system", {
+		title: "Extract a draft design system from the repo",
+		description: "Scan the consumer repo for CSS custom properties / token JSON and write a draft design system. Does not publish. Human must publish before fragment mockups inherit it.",
+		inputSchema: {
+			id: z.string().optional(),
+			title: z.string().optional()
+		},
+		outputSchema: {
+			system: z.unknown(),
+			extract: z.unknown()
+		},
+		annotations: MUTATING
+	}, async ({ id, title }) => {
+		const systemId = id || "default";
+		const data = await requestBaseJson(`/api/design-systems/${encodeURIComponent(systemId)}/extract`, {
+			method: "POST",
+			headers: { "Content-Type": "application/json" },
+			body: JSON.stringify({
+				from: "css",
+				title
+			})
+		});
+		return textResult(`Draft design system ${systemId}. Human must publish before it wraps mockups.`, data);
+	});
+	server.registerTool("propose_design_system", {
+		title: "Propose a design-system draft",
+		description: "Update the draft design system (tokens, guidelines, components). Does not publish. Agents may propose; only the human publishes.",
+		inputSchema: {
+			id: z.string().optional(),
+			title: z.string().optional(),
+			guidelines: z.string().optional(),
+			tokens: z.unknown().optional()
+		},
+		outputSchema: { system: z.unknown() },
+		annotations: MUTATING
+	}, async ({ id, title, guidelines, tokens }) => {
+		const systemId = id || "default";
+		const system = await requestBaseJson(`/api/design-systems/${encodeURIComponent(systemId)}`, {
+			method: "PUT",
+			headers: { "Content-Type": "application/json" },
+			body: JSON.stringify({
+				title,
+				guidelines,
+				tokens
+			})
+		});
+		return textResult(`Proposed draft on ${systemId}`, { system });
+	});
+	server.registerTool("publish_design_system", {
+		title: "Publish the design system",
+		description: "Publish the current draft as a new revision. Prefer letting the human do this in the UI. Use only when the user explicitly asked to publish.",
+		inputSchema: { id: z.string().optional() },
+		outputSchema: { system: z.unknown() },
+		annotations: MUTATING
+	}, async ({ id }) => {
+		const systemId = id || "default";
+		const system = await requestBaseJson(`/api/design-systems/${encodeURIComponent(systemId)}/publish`, { method: "POST" });
+		return textResult(`Published ${systemId}`, { system });
+	});
+	server.registerTool("get_mockup_handoff", {
+		title: "Get mockup implementation handoff",
+		description: "Compact handoff after a mockup is approved: tokens, screens with intent, components used, leftover nits. Prefer this over dumping every screen's HTML.",
+		inputSchema: { mockupId: z.string().min(1) },
+		outputSchema: {
+			xml: z.string().optional(),
+			mockupId: z.string().optional()
+		},
+		annotations: READ_ONLY
+	}, async ({ mockupId }) => {
+		const data = await requestBaseJson(`/api/mockups/${encodeURIComponent(mockupId)}/handoff`);
+		return textResult(typeof data.xml === "string" ? data.xml : "handoff", data);
+	});
 	server.registerResource("agent-guide", "diffing://agent-guide", {
 		title: "diffing agent guide",
 		description: "Portable quick reference for the native code-review and plan-review workflows.",
@@ -1916,7 +2035,7 @@ This MCP connection is immutably bound to \`${repoRoot}\`.
 1. Call \`submit_mockup\`, share the URL, and park unless asked to wait; resubmit with the same \`mockupId\` to bump the version.
 2. On resume or sync wait, call \`await_mockup_review\` (or \`list_mockups\` / \`get_mockup\`).
 3. For bounded reads prefer \`inspect_mockup\` (view=summary/comments/comment/screen, filters by status/screenId/viewport/version).
-4. To revise one screen use \`revise_mockup\` (upsert/remove/patch with \`expectedVersion\` guard); thread ops go through atomic \`update_mockup_threads\`.
+4. To revise one screen use \`revise_mockup\` (upsert/remove/patch/replace-region with \`expectedVersion\` guard); thread ops go through atomic \`update_mockup_threads\`.
 5. On \`approved\`, implement; on \`changes-requested\`, revise and resubmit; on \`rejected\`, stop.
 
 ## GitHub PR

package/dist/{mockup-format-D4Ln0WxJ.mjs → mockup-lint-BRjhxXSw.mjs}

@@ -35,9 +35,9 @@ function normalizeSubmitScreens(input) {
 		ok: false,
 		error: "A mockup html body or screens[] is required"
 	};
-	if (raw.length > 12) return {
+	if (raw.length > 24) return {
 		ok: false,
-		error: `At most 12 screens are allowed`
+		error: `At most 24 screens are allowed`
 	};
 	const seen = /* @__PURE__ */ new Set();
 	const screens = [];
@@ -60,12 +60,14 @@ function normalizeSubmitScreens(input) {
 			error: `Duplicate screen id "${id}"`
 		};
 		seen.add(id);
-		const label = typeof item.label === "string" && item.label.trim() ? item.label.trim() : defaultScreenLabel(id);
-		screens.push({
+		const screen = {
 			id,
-			label,
+			label: typeof item.label === "string" && item.label.trim() ? item.label.trim() : defaultScreenLabel(id),
 			html: item.html.replace(/\r\n/g, "\n")
-		});
+		};
+		if (item.stateOf) screen.stateOf = item.stateOf;
+		if (item.flow) screen.flow = item.flow;
+		screens.push(screen);
 	}
 	return {
 		ok: true,
@@ -324,4 +326,155 @@ function formatComment(comment, fallbackVersion, includeContext) {
 	return parts.join("\n");
 }
 //#endregion
-export { MOCKUP_MAX_SCREEN_BYTES as a, defaultScreenLabel as c, normalizeSubmitScreens as d, slugifyScreenId as f, sectionTitleForLine as i, isMockupViewport as l, extractPlanLines as n, MOCKUP_SCREEN_ID_RE as o, formatPlanReview as r, commentViewport as s, formatMockupReview as t, normalizeMockupViewport as u };
+//#region src/lib/mockup-lint.ts
+const RULES = [
+	{
+		label: "tabs",
+		re: /role\s*=\s*["']tab(list|panel)?["']/i
+	},
+	{
+		label: "tabs",
+		re: /aria-selected\s*=/i
+	},
+	{
+		label: "tabs",
+		re: /data-(?:bs-)?toggle\s*=\s*["'](?:tab|pill)["']/i
+	},
+	{
+		label: "tabs",
+		re: /\bdata-tabs?\s*=/i
+	},
+	{
+		label: "accordion",
+		re: /<details[\s>]/i
+	},
+	{
+		label: "accordion",
+		re: /data-(?:bs-)?toggle\s*=\s*["']collapse["']/i
+	},
+	{
+		label: "accordion",
+		re: /class\s*=\s*["'][^"']*\baccordion\b/i
+	},
+	{
+		label: "modal",
+		re: /role\s*=\s*["'](?:alert)?dialog["']/i
+	},
+	{
+		label: "modal",
+		re: /data-(?:bs-)?toggle\s*=\s*["']modal["']/i
+	},
+	{
+		label: "modal",
+		re: /class\s*=\s*["'][^"']*\bmodal\b/i
+	},
+	{
+		label: "dropdown",
+		re: /role\s*=\s*["']menu(?:bar)?["']/i
+	},
+	{
+		label: "dropdown",
+		re: /data-(?:bs-)?toggle\s*=\s*["']dropdown["']/i
+	},
+	{
+		label: "dropdown",
+		re: /class\s*=\s*["'][^"']*\bdropdown\b/i
+	},
+	{
+		label: "toggle",
+		re: /role\s*=\s*["']switch["']/i
+	},
+	{
+		label: "toggle",
+		re: /class\s*=\s*["'][^"']*\b(?:toggle|switch)\b/i
+	}
+];
+const ORDERED_LABELS = [
+	"tabs",
+	"accordion",
+	"modal",
+	"dropdown",
+	"toggle"
+];
+/** Find the in-page state UI patterns present in one screen's HTML. */
+function detectInPageState(html) {
+	const found = /* @__PURE__ */ new Set();
+	for (const rule of RULES) {
+		if (found.has(rule.label)) continue;
+		if (rule.re.test(html)) found.add(rule.label);
+	}
+	return ORDERED_LABELS.filter((label) => found.has(label));
+}
+const STYLE_RULES = [
+	{
+		label: "tailwind-cdn",
+		re: /cdn\.tailwindcss\.com/i
+	},
+	{
+		label: "tailwind-cdn",
+		re: /(?:unpkg\.com|cdn\.jsdelivr\.net|esm\.sh)\/(?:npm\/)?tailwindcss/i
+	},
+	{
+		label: "google-fonts",
+		re: /fonts\.googleapis\.com/i
+	},
+	{
+		label: "google-fonts",
+		re: /fonts\.gstatic\.com/i
+	}
+];
+const GENERIC_FONT_RE = /font-family\s*:\s*[^;{]*\bInter\b/i;
+const GENERIC_INDIGO_RE = /#(?:4f46e5|4338ca|6366f1|818cf8|7c3aed|8b5cf6|a78bfa)\b|\b(?:indigo|violet)-(?:[4-9]00|950)\b/i;
+/** Off-brand / generic-AI styling that should not ship as the product look. */
+function detectGenericStyle(html) {
+	const found = /* @__PURE__ */ new Set();
+	for (const rule of STYLE_RULES) {
+		if (found.has(rule.label)) continue;
+		if (rule.re.test(html)) found.add(rule.label);
+	}
+	if (GENERIC_FONT_RE.test(html) && GENERIC_INDIGO_RE.test(html)) found.add("generic-palette");
+	return [
+		"tailwind-cdn",
+		"google-fonts",
+		"generic-palette"
+	].filter((label) => found.has(label));
+}
+/** Build submit-time hints for a set of screens (empty when all clean). */
+function lintMockupScreens(screens) {
+	const hints = [];
+	for (const screen of screens) {
+		const state = detectInPageState(screen.html);
+		if (state.length > 0) hints.push({
+			screenId: screen.id,
+			kind: "state",
+			patterns: state,
+			message: `Screen "${screen.id}" contains in-page state UI (${state.join(", ")}). Split each state into its own screen instead.`
+		});
+		const style = detectGenericStyle(screen.html);
+		if (style.length > 0) hints.push({
+			screenId: screen.id,
+			kind: "style",
+			patterns: style,
+			message: `Screen "${screen.id}" looks generic (${style.join(", ")}). Drop CDN Tailwind / Google Fonts and do not invent Inter + indigo — match the product.`
+		});
+	}
+	return hints;
+}
+/** Compact submit/revise warning for CLI and MCP. */
+function formatSubmitHints(hints) {
+	if (hints.length === 0) return "";
+	const state = hints.filter((h) => h.kind !== "style");
+	const style = hints.filter((h) => h.kind === "style");
+	const lines = [];
+	if (state.length > 0) {
+		lines.push("⚠️ In-page state UI detected — split each state into its own screen:");
+		for (const hint of state) lines.push(`  • ${hint.screenId}: ${hint.patterns.join(", ")}`);
+	}
+	if (style.length > 0) {
+		lines.push("⚠️ Generic styling — match the product (no CDN Tailwind, Google Fonts, or Inter+indigo):");
+		for (const hint of style) lines.push(`  • ${hint.screenId}: ${hint.patterns.join(", ")}`);
+	}
+	return `\n${lines.join("\n")}`;
+}
+//#endregion
+export { formatPlanReview as a, MOCKUP_SCREEN_ID_RE as c, isMockupViewport as d, normalizeMockupViewport as f, extractPlanLines as i, commentViewport as l, slugifyScreenId as m, lintMockupScreens as n, sectionTitleForLine as o, normalizeSubmitScreens as p, formatMockupReview as r, MOCKUP_MAX_SCREEN_BYTES as s, formatSubmitHints as t, defaultScreenLabel as u };

package/dist/{review-export-DoST2oPt.mjs → review-export-BXCeRNjk.mjs}

@@ -1,4 +1,4 @@
-import { n as reviewDecisionSummary } from "./comment-format-D4mCqr7T.mjs";
+import { n as reviewDecisionSummary } from "./comment-format-DS1wOWj_.mjs";
 //#region src/lib/review-export.ts
 /**
 * Export a review as human-readable Markdown (clipboard / download / CLI).