@atomixstudio/mcp 1.0.22 → 1.0.24

package/dist/index.js

@@ -1480,15 +1480,6 @@ function figmaColorNameWithGroup(key) {
   const groupDisplay = group.charAt(0).toUpperCase() + group.slice(1).toLowerCase();
   return `${groupDisplay} / ${name}`;
 }
-var FIGMA_COLOR_GROUP_ORDER = {
-  background: 0,
-  text: 1,
-  border: 2,
-  icon: 3,
-  brand: 4,
-  action: 5,
-  feedback: 6
-};
 var FIGMA_SHADOW_ORDER = {
   none: 0,
   xs: 1,
@@ -1582,8 +1573,11 @@ function buildFigmaPayloadsFromDS(data) {
     if (Object.keys(light).length > 0) modes.push("Light");
     if (Object.keys(dark).length > 0 && !modes.includes("Dark")) modes.push("Dark");
     if (modes.length === 0) modes.push("Light");
-    const allKeys = /* @__PURE__ */ new Set([...Object.keys(light), ...Object.keys(dark)]);
-    for (const key of allKeys) {
+    const orderedKeys = [...Object.keys(light)];
+    for (const k of Object.keys(dark)) {
+      if (!orderedKeys.includes(k)) orderedKeys.push(k);
+    }
+    for (const key of orderedKeys) {
       const lightHex = light[key];
       const darkHex = dark[key];
       if (typeof lightHex === "string" && hexRe.test(lightHex)) {
@@ -1612,26 +1606,8 @@ function buildFigmaPayloadsFromDS(data) {
     }
   }
   if (variables.length === 0 && modes.length === 0) modes.push("Light");
-  const colorSortKey = (figmaName) => {
-    const slash = figmaName.indexOf(" / ");
-    const group = slash >= 0 ? figmaName.slice(0, slash).toLowerCase() : "";
-    const name = slash >= 0 ? figmaName.slice(slash + 3) : figmaName;
-    const order = FIGMA_COLOR_GROUP_ORDER[group] ?? 999;
-    return [order, name];
-  };
-  variables.sort((a, b) => {
-    const [oA, nA] = colorSortKey(a.name);
-    const [oB, nB] = colorSortKey(b.name);
-    if (oA !== oB) return oA - oB;
-    return nA.localeCompare(nB);
-  });
-  paintStyles.sort((a, b) => {
-    const [oA, nA] = colorSortKey(a.name);
-    const [oB, nB] = colorSortKey(b.name);
-    if (oA !== oB) return oA - oB;
-    return nA.localeCompare(nB);
-  });
-  const collectionName = data.meta?.name ? `${data.meta.name} Colors` : "Atomix Colors";
+  const dsName = data.meta?.name;
+  const collectionName = dsName ? `${dsName} Foundations` : "Foundations";
   const textStyles = [];
   const sizeToPx = (val, basePx = 16) => {
     if (typeof val === "number") return Math.round(val);
@@ -1735,7 +1711,6 @@ function buildFigmaPayloadsFromDS(data) {
     if (a.fontSize !== b.fontSize) return a.fontSize - b.fontSize;
     return (a.name || "").localeCompare(b.name || "");
   });
-  const dsName = data.meta?.name ?? "Atomix";
   const numberVariableCollections = [];
   const spacing = tokens?.spacing;
   if (spacing?.scale && typeof spacing.scale === "object") {
@@ -1745,7 +1720,7 @@ function buildFigmaPayloadsFromDS(data) {
       if (n >= 0) variables2.push({ name: key, value: n });
     }
     variables2.sort((a, b) => a.value - b.value);
-    if (variables2.length > 0) numberVariableCollections.push({ collectionName: `${dsName} Spacing`, categoryKey: "Spacing", variables: variables2, scopes: ["GAP"] });
+    if (variables2.length > 0) numberVariableCollections.push({ collectionName, categoryKey: "Spacing", variables: variables2, scopes: ["GAP"] });
   }
   const radius = tokens?.radius;
   if (radius?.scale && typeof radius.scale === "object") {
@@ -1755,7 +1730,7 @@ function buildFigmaPayloadsFromDS(data) {
       if (n >= 0) variables2.push({ name: key, value: n });
     }
     variables2.sort((a, b) => a.value - b.value);
-    if (variables2.length > 0) numberVariableCollections.push({ collectionName: `${dsName} Radius`, categoryKey: "Radius", variables: variables2, scopes: ["CORNER_RADIUS"] });
+    if (variables2.length > 0) numberVariableCollections.push({ collectionName, categoryKey: "Radius", variables: variables2, scopes: ["CORNER_RADIUS"] });
   }
   const borders = tokens?.borders;
   if (borders?.width && typeof borders.width === "object") {
@@ -1765,7 +1740,7 @@ function buildFigmaPayloadsFromDS(data) {
       if (n >= 0) variables2.push({ name: key, value: n });
     }
     variables2.sort((a, b) => a.value - b.value);
-    if (variables2.length > 0) numberVariableCollections.push({ collectionName: `${dsName} Borders`, categoryKey: "Borders", variables: variables2, scopes: ["STROKE_FLOAT"] });
+    if (variables2.length > 0) numberVariableCollections.push({ collectionName, categoryKey: "Borders", variables: variables2, scopes: ["STROKE_FLOAT"] });
   }
   const sizing = tokens?.sizing;
   if (sizing?.height && typeof sizing.height === "object") {
@@ -1775,7 +1750,7 @@ function buildFigmaPayloadsFromDS(data) {
       if (n >= 0) variables2.push({ name: key, value: n });
     }
     variables2.sort((a, b) => a.value - b.value);
-    if (variables2.length > 0) numberVariableCollections.push({ collectionName: `${dsName} Height`, categoryKey: "Height", variables: variables2, scopes: ["WIDTH_HEIGHT"] });
+    if (variables2.length > 0) numberVariableCollections.push({ collectionName, categoryKey: "Height", variables: variables2, scopes: ["WIDTH_HEIGHT"] });
   }
   if (sizing?.icon && typeof sizing.icon === "object") {
     const variables2 = [];
@@ -1784,7 +1759,7 @@ function buildFigmaPayloadsFromDS(data) {
       if (n >= 0) variables2.push({ name: key, value: n });
     }
     variables2.sort((a, b) => a.value - b.value);
-    if (variables2.length > 0) numberVariableCollections.push({ collectionName: `${dsName} Icon`, categoryKey: "Icon", variables: variables2, scopes: ["WIDTH_HEIGHT"] });
+    if (variables2.length > 0) numberVariableCollections.push({ collectionName, categoryKey: "Icon", variables: variables2, scopes: ["WIDTH_HEIGHT"] });
   }
   const layout = tokens?.layout;
   if (layout?.breakpoint && typeof layout.breakpoint === "object") {
@@ -1794,7 +1769,7 @@ function buildFigmaPayloadsFromDS(data) {
       if (n >= 0) variables2.push({ name: key, value: n });
     }
     variables2.sort((a, b) => a.value - b.value);
-    if (variables2.length > 0) numberVariableCollections.push({ collectionName: `${dsName} Breakpoint`, categoryKey: "Breakpoint", variables: variables2, scopes: ["WIDTH_HEIGHT"] });
+    if (variables2.length > 0) numberVariableCollections.push({ collectionName, categoryKey: "Breakpoint", variables: variables2, scopes: ["WIDTH_HEIGHT"] });
   }
   const effectStyles = [];
   const shadows = tokens?.shadows;
@@ -1909,6 +1884,43 @@ ${changes.summary}`);
     }
   }
 }
+function validateTokenFileAfterWrite(outputPath, format, expectedMinVariables) {
+  if (!fs2.existsSync(outputPath)) {
+    return { path: outputPath, status: "FAIL", detail: "File not found after write." };
+  }
+  const content = fs2.readFileSync(outputPath, "utf-8");
+  if (!content || content.trim().length === 0) {
+    return { path: outputPath, status: "FAIL", detail: "File is empty after write." };
+  }
+  if (["css", "scss", "less"].includes(format)) {
+    const varPattern = /(--[a-zA-Z0-9-]+):\s*[^;]+;/g;
+    let count = 0;
+    let m;
+    while ((m = varPattern.exec(content)) !== null) count++;
+    if (count < expectedMinVariables) {
+      return {
+        path: outputPath,
+        status: "FAIL",
+        detail: `Expected at least ${expectedMinVariables} variables; found ${count}.`
+      };
+    }
+    return { path: outputPath, status: "OK", detail: `${count} variables written.` };
+  }
+  return { path: outputPath, status: "OK", detail: "File written (non-CSS format)." };
+}
+function formatValidationBlock(entries) {
+  if (entries.length === 0) return "";
+  const displayPath = (p) => p.startsWith("(") ? p : path2.relative(process.cwd(), p);
+  const lines = [
+    "",
+    "\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501",
+    "VALIDATION (confirm sync succeeded)",
+    "\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501\u2501",
+    ...entries.map((e) => `  ${displayPath(e.path)}: ${e.status} \u2014 ${e.detail}`),
+    ""
+  ];
+  return lines.join("\n");
+}
 async function fetchDesignSystemForMCP(forceRefresh = false) {
   if (!dsId) throw new Error("Missing --ds-id. Usage: npx @atomixstudio/mcp --ds-id <id> --atomix-token <token>");
   if (!accessToken) throw new Error("Missing --atomix-token. Get your token from the Export modal or Settings.");
@@ -1931,7 +1943,7 @@ var TOKEN_CATEGORIES = ["colors", "typography", "spacing", "sizing", "shadows",
 var server = new Server(
   {
     name: "atomix-mcp-user",
-    version: "1.0.22"
+    version: "1.0.24"
   },
   {
     capabilities: {
@@ -2083,7 +2095,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
     },
     {
       name: "syncAll",
-      description: "Sync tokens, AI rules, skills files (SKILL.md, design-in-figma.md), and atomix-dependencies.json. One tool for full project sync. Use /--sync prompt or call when the user wants to sync. Optional: output (default ./tokens.css), format (default css), skipTokens (if true, only writes skills and manifest).",
+      description: "Sync tokens, AI rules, skills (SKILL.md, design-in-figma.md), and atomix-dependencies.json. Use dryRun: true first to report what would change without writing; then dryRun: false to apply. Response includes a VALIDATION section\u2014agent must check it to confirm success. Optional: output (default ./tokens.css), format (default css), skipTokens (if true, only skills and manifest), dryRun (if true, report only; no files written).",
       inputSchema: {
         type: "object",
         properties: {
@@ -2099,6 +2111,10 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
           skipTokens: {
             type: "boolean",
             description: "If true, skip token file and rules sync; only write skills and dependencies manifest. Default: false."
+          },
+          dryRun: {
+            type: "boolean",
+            description: "If true, compute and return what would be written (paths, token counts, diff summary) but do NOT write any files. Call with dryRun: false to apply. Default: false."
           }
         },
         required: []
@@ -2311,7 +2327,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         isError: true
       };
     }
-    async function performTokenSyncAndRules(designSystemData, tokenOutput, tokenFormat) {
+    async function performTokenSyncAndRules(designSystemData, tokenOutput, tokenFormat, dryRun) {
       const output = tokenOutput;
       const format = tokenFormat;
       const outputPath = path2.resolve(process.cwd(), output);
@@ -2374,7 +2390,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         const lightChanges = diff.added.length + diff.modified.length;
         const darkChanges = diff.addedDark.length + diff.modifiedDark.length;
         const totalChanges = lightChanges + darkChanges + deprecatedCount;
-        if (totalChanges === 0) {
+        if (totalChanges === 0 && !dryRun) {
           const lastUpdated = designSystemData.meta.exportedAt ? new Date(designSystemData.meta.exportedAt).toLocaleString() : "N/A";
           return {
             responseText: `\u2713 Already up to date!
@@ -2383,7 +2399,19 @@ File: ${output}
 Tokens: ${tokenCount}
 Version: ${designSystemData.meta.version}
 Last updated: ${lastUpdated}`,
-            rulesResults: []
+            rulesResults: [],
+            validation: [{ path: outputPath, status: "OK", detail: "No changes needed; file already up to date." }]
+          };
+        }
+        if (totalChanges === 0 && dryRun) {
+          return {
+            responseText: `[DRY RUN] Already up to date. No changes would be written.
+
+File: ${output}
+Tokens: ${tokenCount}
+Version: ${designSystemData.meta.version}`,
+            rulesResults: [],
+            validation: [{ path: "(dry run)", status: "OK", detail: "No files written." }]
           };
         }
         changes = [...diff.modified, ...diff.modifiedDark];
@@ -2397,9 +2425,34 @@ Last updated: ${lastUpdated}`,
           timestamp: (/* @__PURE__ */ new Date()).toISOString()
         };
       }
+      if (dryRun) {
+        const added = diff ? diff.added.length + diff.addedDark.length : 0;
+        const modified = diff ? diff.modified.length + diff.modifiedDark.length : 0;
+        const changeLine = fileExists && diff ? `  Changes: ${added} added, ${modified} modified` : !fileExists ? `  New file would be created with ${tokenCount} tokens.` : "";
+        const report = [
+          "[DRY RUN] No files were written. The following would be written if you run syncAll with dryRun: false.",
+          "",
+          "Token file:",
+          `  Path: ${output}`,
+          `  Format: ${format}`,
+          `  Tokens: ${tokenCount} (${deprecatedCount} deprecated preserved)`,
+          changeLine,
+          "",
+          "Rules: .cursorrules (or existing rules files in project)",
+          "",
+          "Call syncAll again with dryRun: false to apply."
+        ].filter(Boolean).join("\n");
+        return {
+          responseText: report,
+          rulesResults: [],
+          validation: [{ path: "(dry run)", status: "OK", detail: "No files written." }]
+        };
+      }
       const outputDir = path2.dirname(outputPath);
       if (!fs2.existsSync(outputDir)) fs2.mkdirSync(outputDir, { recursive: true });
       fs2.writeFileSync(outputPath, newContent);
+      const validation = [];
+      validation.push(validateTokenFileAfterWrite(outputPath, format, tokenCount));
       let rulesResults = [];
       try {
         rulesResults = await syncRulesFiles({
@@ -2408,6 +2461,14 @@ Last updated: ${lastUpdated}`,
           apiBase: apiBase ?? void 0,
           rulesDir: process.cwd()
         });
+        for (const r of rulesResults) {
+          const fullPath = path2.resolve(process.cwd(), r.path);
+          validation.push({
+            path: fullPath,
+            status: r.success && fs2.existsSync(fullPath) ? "OK" : "FAIL",
+            detail: r.success ? "Written." : r.error || "Write failed."
+          });
+        }
       } catch (error) {
         console.error(`[syncAll] Failed to sync rules: ${error}`);
       }
@@ -2428,7 +2489,7 @@ Last updated: ${lastUpdated}`,
         hasRefactorRecommendation: !!lastSyncAffectedTokens?.removed.length,
         deprecatedTokenCount: lastSyncAffectedTokens?.removed.length || 0
       });
-      return { responseText: response, rulesResults };
+      return { responseText: response, rulesResults, validation };
     }
     switch (name) {
       case "getToken": {
@@ -2796,26 +2857,50 @@ Last updated: ${lastUpdated}`,
       }
       case "syncAll": {
         const skipTokens = args?.skipTokens === true;
+        const dryRun = args?.dryRun === true;
         const output = args?.output || "./tokens.css";
         const format = args?.format || "css";
-        const parts = ["\u2713 syncAll complete."];
+        const parts = [dryRun ? "[DRY RUN] syncAll report (no files written)." : "\u2713 syncAll complete."];
         let tokenResponseText = "";
+        const allValidation = [];
         if (!skipTokens) {
-          const { responseText, rulesResults } = await performTokenSyncAndRules(data, output, format);
-          tokenResponseText = responseText;
-          parts.push(`Tokens: ${output} (${format})`);
-          if (rulesResults.length > 0) {
-            parts.push(`Rules: ${rulesResults.map((r) => r.path).join(", ")}`);
+          const result = await performTokenSyncAndRules(data, output, format, dryRun);
+          tokenResponseText = result.responseText;
+          allValidation.push(...result.validation);
+          if (!dryRun && result.rulesResults.length > 0) {
+            parts.push(`Rules: ${result.rulesResults.map((r) => r.path).join(", ")}`);
+          }
+          if (dryRun) {
+            parts.push(`Would write tokens: ${output} (${format})`);
+          } else {
+            parts.push(`Tokens: ${output} (${format})`);
           }
         }
-        const skillsDir = path2.resolve(process.cwd(), ".cursor/skills/atomix-ds");
-        if (!fs2.existsSync(skillsDir)) fs2.mkdirSync(skillsDir, { recursive: true });
         const dsVersion = String(data.meta.version ?? "1.0.0");
         const dsExportedAt = data.meta.exportedAt;
+        const skillsDir = path2.resolve(process.cwd(), ".cursor/skills/atomix-ds");
+        const skillPath1 = path2.join(skillsDir, "SKILL.md");
+        const skillPath2 = path2.join(skillsDir, "design-in-figma.md");
+        const manifestPath = path2.resolve(process.cwd(), "atomix-dependencies.json");
+        if (dryRun) {
+          parts.push("Would write skills: .cursor/skills/atomix-ds/SKILL.md, .cursor/skills/atomix-ds/design-in-figma.md");
+          parts.push("Would write manifest: atomix-dependencies.json");
+          const reportText = [parts.join("\n"), tokenResponseText].filter(Boolean).join("\n\n---\n\n");
+          return {
+            content: [{ type: "text", text: `syncAllResult: DRY_RUN (no files written)
+
+${reportText}` }]
+          };
+        }
+        if (!fs2.existsSync(skillsDir)) fs2.mkdirSync(skillsDir, { recursive: true });
         const genericWithVersion = injectSkillVersion(GENERIC_SKILL_MD, dsVersion, dsExportedAt);
         const figmaWithVersion = injectSkillVersion(FIGMA_DESIGN_SKILL_MD, dsVersion, dsExportedAt);
-        fs2.writeFileSync(path2.join(skillsDir, "SKILL.md"), genericWithVersion);
-        fs2.writeFileSync(path2.join(skillsDir, "design-in-figma.md"), figmaWithVersion);
+        fs2.writeFileSync(skillPath1, genericWithVersion);
+        fs2.writeFileSync(skillPath2, figmaWithVersion);
+        allValidation.push(
+          { path: skillPath1, status: fs2.existsSync(skillPath1) ? "OK" : "FAIL", detail: "Written." },
+          { path: skillPath2, status: fs2.existsSync(skillPath2) ? "OK" : "FAIL", detail: "Written." }
+        );
         parts.push("Skills: .cursor/skills/atomix-ds/SKILL.md, .cursor/skills/atomix-ds/design-in-figma.md (synced at DS v" + dsVersion + ")");
         const tokens = data.tokens;
         const typography = tokens?.typography;
@@ -2851,15 +2936,18 @@ Last updated: ${lastUpdated}`,
             syncedAtVersion: data.meta.version ?? "1.0.0"
           }
         };
-        const manifestPath = path2.resolve(process.cwd(), "atomix-dependencies.json");
         fs2.writeFileSync(manifestPath, JSON.stringify(manifest, null, 2));
+        allValidation.push({ path: manifestPath, status: fs2.existsSync(manifestPath) ? "OK" : "FAIL", detail: "Written." });
         parts.push("Manifest: atomix-dependencies.json (icons, fonts, skill paths)");
         const summary = parts.join("\n");
-        const fullText = tokenResponseText ? `${summary}
+        const validationBlock = formatValidationBlock(allValidation);
+        const hasFailure = allValidation.some((e) => e.status === "FAIL");
+        const resultLine = hasFailure ? "syncAllResult: FAIL \u2014 Check VALIDATION section below. Do not report success to the user.\n\n" : "syncAllResult: OK\n\n";
+        const fullText = resultLine + (tokenResponseText ? `${summary}
 
 ---
 
-${tokenResponseText}` : summary;
+${tokenResponseText}${validationBlock}` : `${summary}${validationBlock}`);
         return {
           content: [{ type: "text", text: fullText }]
         };
@@ -2971,13 +3059,14 @@ ${tokenResponseText}` : summary;
           if (payloads.numberVariableCollections.length > 0) {
             const numberResults = [];
             try {
+              const foundationsName = payloads.colorVariables.collectionName;
               for (const coll of payloads.numberVariableCollections) {
                 const result = await sendBridgeRequest("create_number_variables", {
-                  collectionName: coll.collectionName,
-                  variables: coll.variables,
+                  collectionName: foundationsName,
+                  variables: coll.variables.map((v) => ({ name: `${coll.categoryKey} / ${v.name}`, value: v.value })),
                   scopes: coll.scopes
                 });
-                numberResults.push({ collectionName: coll.collectionName, result });
+                numberResults.push({ categoryKey: coll.categoryKey, result });
               }
               out.numberVariables = numberResults;
             } catch (e) {