@sangheepark/figma-ds-mcp 0.2.5 → 0.2.7

package/dist/tools/pipeline-tools.js

@@ -234,9 +234,14 @@ function enrichSpec(traversal, mapping) {
                 node.style['line-height'] = `${Math.round(lh * 100)}%`;
             }
         }
+        // _ds_type 제거 (downstream 미소비 — build 전에 정리)
+        if ('_ds_type' in node) {
+            delete node['_ds_type'];
+        }
         // Step 6: CSS → Figma 정규화
-        // 6-pre: overflow:hidden → layout.clip-content 변환 (6-A 필터링 전에 처리)
-        if (node.style && node.style['overflow'] === 'hidden') {
+        // 6-pre: overflow → layout.clip-content 변환 (6-A 필터링 전에 처리)
+        const overflow = node.style && node.style['overflow'];
+        if (overflow === 'hidden' || overflow === 'scroll' || overflow === 'auto') {
             node.layout = node.layout || {};
             node.layout['clip-content'] = true;
             delete node.style['overflow'];
@@ -277,13 +282,13 @@ function enrichSpec(traversal, mapping) {
         }
     }
     walk(spec, 'root');
-    // Step 6-C: root frame sizing (fill → 1440px width, hug height)
+    // Step 6-C: root frame sizing (viewport defaults: 1440×900 PC)
     if (spec.layout) {
         if (spec.layout.width === 'fill') {
             spec.layout.width = '1440';
         }
         if (spec.layout.height === 'fill') {
-            spec.layout.height = 'hug';
+            spec.layout.height = '900';
         }
     }
     return { spec, errors };
@@ -467,6 +472,127 @@ function gateCheck(spec) {
     const pass = results.every(r => r.pass);
     return { pass, results };
 }
+// --- _repeat: Component Auto-Extraction ---
+function extractRepeatComponents(spec, defaultCount = 3) {
+    const extracted = new Map(); // name → component spec
+    function deepClone(obj) {
+        return JSON.parse(JSON.stringify(obj));
+    }
+    function getRepeatCount(node) {
+        if (typeof node._repeat === 'number' && node._repeat > 0)
+            return node._repeat;
+        return defaultCount;
+    }
+    // Depth-first walk: process children first so nested _repeat resolves before parent
+    function walk(node) {
+        // Process children first (depth-first)
+        if (node.children) {
+            for (const child of node.children) {
+                walk(child);
+            }
+        }
+        // Now process this node's children for _repeat
+        if (!node.children)
+            return;
+        const newChildren = [];
+        for (const child of node.children) {
+            if (!child._repeat) {
+                newChildren.push(child);
+                continue;
+            }
+            const count = getRepeatCount(child);
+            const nodeAny = child;
+            const hasLibraryKey = !!nodeAny['library-key'];
+            const hasDs = !!child._ds;
+            // Library instance: just duplicate N times, no component extraction
+            if (hasLibraryKey || hasDs) {
+                const cleanChild = deepClone(child);
+                delete cleanChild._repeat;
+                for (let i = 0; i < count; i++) {
+                    const instance = deepClone(cleanChild);
+                    if (instance.name)
+                        instance.name = `${instance.name}_${i + 1}`;
+                    newChildren.push(instance);
+                }
+                continue;
+            }
+            // Custom frame: extract as component + insert instances
+            const compName = child.name || `RepeatComponent_${extracted.size + 1}`;
+            if (!extracted.has(compName)) {
+                // Extract as component
+                const compSpec = deepClone(child);
+                compSpec.type = 'component';
+                compSpec.name = compName;
+                delete compSpec._repeat;
+                extracted.set(compName, compSpec);
+            }
+            // Insert N instances in place
+            for (let i = 0; i < count; i++) {
+                newChildren.push({
+                    type: 'instance',
+                    component: compName,
+                    name: `${compName}_${i + 1}`,
+                });
+            }
+        }
+        node.children = newChildren;
+    }
+    const page = deepClone(spec);
+    // Skip if root itself has _repeat (standalone component, no extraction needed)
+    if (page._repeat) {
+        delete page._repeat;
+        return { page, components: [] };
+    }
+    walk(page);
+    return { page, components: Array.from(extracted.values()) };
+}
+function countNodes(node) {
+    let count = 1;
+    if (node.children) {
+        for (const child of node.children)
+            count += countNodes(child);
+    }
+    return count;
+}
+function splitSections(pageSpec, components, outputDir) {
+    mkdirSync(join(outputDir, 'components'), { recursive: true });
+    mkdirSync(join(outputDir, 'sections'), { recursive: true });
+    // Save components
+    const compEntries = [];
+    for (const comp of components) {
+        const name = comp.name || `Component_${compEntries.length}`;
+        const relPath = `components/${name}.json`;
+        writeFileSync(join(outputDir, relPath), JSON.stringify(comp, null, 2));
+        compEntries.push({ name, specPath: relPath, nodeCount: countNodes(comp) });
+    }
+    // Split page into sections: use _section markers or top-level children
+    const sectionEntries = [];
+    const rootLayout = { ...(pageSpec.layout || {}) };
+    const rootStyle = pageSpec.style ? { ...pageSpec.style } : undefined;
+    const pageName = pageSpec.name || 'Page';
+    if (pageSpec.children && pageSpec.children.length > 0) {
+        for (let i = 0; i < pageSpec.children.length; i++) {
+            const child = pageSpec.children[i];
+            const name = child.name || `Section_${i}`;
+            const relPath = `sections/${name}.json`;
+            writeFileSync(join(outputDir, relPath), JSON.stringify(child, null, 2));
+            sectionEntries.push({ name, specPath: relPath, nodeCount: countNodes(child), order: i });
+        }
+    }
+    let totalNodes = 1; // root
+    for (const c of compEntries)
+        totalNodes += c.nodeCount;
+    for (const s of sectionEntries)
+        totalNodes += s.nodeCount;
+    const manifest = {
+        page: { name: pageName, rootLayout, rootStyle },
+        components: compEntries,
+        sections: sectionEntries,
+        stats: { totalNodes, components: compEntries.length, sections: sectionEntries.length },
+    };
+    writeFileSync(join(outputDir, 'manifest.json'), JSON.stringify(manifest, null, 2));
+    return manifest;
+}
 // fontWeight name → CSS number (reverse of display names in data files)
 const FONT_WEIGHT_NAME_TO_NUM = {
     thin: '100', hairline: '100',
@@ -838,16 +964,28 @@ Run after validate_traversal. If outputPath is provided, spec is saved to file a
                 }],
         };
     });
+    // Helper: resolve traversals from either JSON param or file paths
+    function resolveTraversals(traversals, traversalPaths) {
+        if (traversalPaths && traversalPaths.length > 0) {
+            return traversalPaths.map(p => JSON.parse(readFileSync(p, 'utf-8')));
+        }
+        if (traversals && traversals.length > 0) {
+            return traversals;
+        }
+        throw new Error('Either traversals or traversalPaths must be provided');
+    }
     // generate_mapping — D5
     server.tool('generate_mapping', `Generate mapping.json from traversal JSON + blueprint data files. Zero LLM involvement.
 Reads *-component-sets.json and *-style.json from dataDir. Matches components (3-stage: exact → case-insensitive → word-split), tokens (HEX → variable), text styles (fontSize/fontWeight → style name).
 Saves result to outputPath. Replaces ds-data-lookup sub-agent.`, {
-        traversals: z.array(z.record(z.unknown())).describe('Array of traversal JSON root nodes'),
+        traversals: z.array(z.record(z.unknown())).optional().describe('Array of traversal JSON root nodes'),
+        traversalPaths: z.array(z.string()).optional().describe('File paths to traversal JSON files (alternative to traversals — avoids large param)'),
         dataDir: z.string().describe('Path to blueprint data directory (e.g., .claude/blueprint/data/)'),
         outputPath: z.string().describe('File path to save mapping.json'),
-    }, async ({ traversals, dataDir, outputPath }) => {
+    }, async ({ traversals, traversalPaths, dataDir, outputPath }) => {
         try {
-            const result = generateMapping(traversals, dataDir);
+            const tNodes = resolveTraversals(traversals, traversalPaths);
+            const result = generateMapping(tNodes, dataDir);
             mkdirSync(dirname(outputPath), { recursive: true });
             writeFileSync(outputPath, JSON.stringify(result, null, 2));
             // Summary for context (not the full mapping)
@@ -859,7 +997,7 @@ Saves result to outputPath. Replaces ds-data-lookup sub-agent.`, {
                     allDs.add(node._ds);
                 node.children?.forEach(collectDs);
             }
-            traversals.forEach(collectDs);
+            tNodes.forEach(collectDs);
             const unmatchedDs = [];
             for (const ds of allDs) {
                 if (!matchedDs.has(ds) && !reviewDs.has(ds))
@@ -900,13 +1038,14 @@ Saves result to outputPath. Replaces ds-data-lookup sub-agent.`, {
     server.tool('run_pipeline', `Run the full spec pipeline: generate_mapping → validate → enrich → gate → tree in one call.
 Supports 2-pass pattern: Pass 1 returns reviewNeeded (ambiguous matches). Agent decides, then Pass 2 with resolutions auto-applies decisions.
 If no reviewNeeded, completes in 1 pass.`, {
-        traversals: z.array(z.record(z.unknown())).describe('Array of traversal JSON root nodes'),
+        traversals: z.array(z.record(z.unknown())).optional().describe('Array of traversal JSON root nodes'),
+        traversalPaths: z.array(z.string()).optional().describe('File paths to traversal JSON files (alternative to traversals — avoids large param)'),
         dataDir: z.string().describe('Path to blueprint data directory (e.g., .claude/blueprint/data/)'),
         outputPath: z.string().describe('File path to save spec.json (mapping.json saved alongside)'),
         resolutions: z.record(z.string()).optional().describe('Agent-decided mappings: { dsName: resolvedName }. Pass 2 only.'),
-    }, async ({ traversals, dataDir, outputPath, resolutions }) => {
+    }, async ({ traversals, traversalPaths, dataDir, outputPath, resolutions }) => {
         try {
-            let tNodes = traversals;
+            let tNodes = resolveTraversals(traversals, traversalPaths);
             // Step 0: Apply resolutions — replace _ds in traversal nodes
             if (resolutions && Object.keys(resolutions).length > 0) {
                 // Deep clone to avoid mutating input
@@ -993,9 +1132,20 @@ If no reviewNeeded, completes in 1 pass.`, {
                         }],
                 };
             }
+            // Step 3.5: Extract _repeat components
+            // Components are placed before pages so builder creates them first
+            const finalSpecs = [];
+            const allComponents = [];
+            const allPages = [];
+            for (const spec of specs) {
+                const { page, components } = extractRepeatComponents(spec);
+                allComponents.push(...components);
+                allPages.push(page);
+                finalSpecs.push(...components, page);
+            }
             // Step 4: Gate check each spec
             const gateResults = [];
-            for (const spec of specs) {
+            for (const spec of finalSpecs) {
                 gateResults.push(gateCheck(spec));
             }
             const allGatesPass = gateResults.every(g => g.pass);
@@ -1003,18 +1153,25 @@ If no reviewNeeded, completes in 1 pass.`, {
             mkdirSync(dirname(outputPath), { recursive: true });
             const trees = [];
             const allStats = [];
-            if (specs.length === 1) {
-                writeFileSync(outputPath, JSON.stringify(specs[0], null, 2));
-                trees.push(specToTree(specs[0]));
-                allStats.push(countSpecNodes(specs[0]));
+            // Also produce manifest + section files for incremental build
+            const outputDir = dirname(outputPath);
+            const manifests = [];
+            for (const page of allPages) {
+                const manifest = splitSections(page, allComponents, outputDir);
+                manifests.push(manifest);
+            }
+            // Save full specs (legacy format) alongside manifest
+            if (finalSpecs.length === 1) {
+                writeFileSync(outputPath, JSON.stringify(finalSpecs[0], null, 2));
+                trees.push(specToTree(finalSpecs[0]));
+                allStats.push(countSpecNodes(finalSpecs[0]));
             }
             else {
-                // Multiple specs → save each with index
-                for (let i = 0; i < specs.length; i++) {
+                for (let i = 0; i < finalSpecs.length; i++) {
                     const specPath = outputPath.replace(/\.json$/, `_${i}.json`);
-                    writeFileSync(specPath, JSON.stringify(specs[i], null, 2));
-                    trees.push(specToTree(specs[i]));
-                    allStats.push(countSpecNodes(specs[i]));
+                    writeFileSync(specPath, JSON.stringify(finalSpecs[i], null, 2));
+                    trees.push(specToTree(finalSpecs[i]));
+                    allStats.push(countSpecNodes(finalSpecs[i]));
                 }
             }
             // Collect unresolved
@@ -1037,7 +1194,9 @@ If no reviewNeeded, completes in 1 pass.`, {
                         text: JSON.stringify({
                             success: allGatesPass,
                             savedTo: outputPath,
+                            manifestSavedTo: join(outputDir, 'manifest.json'),
                             mappingSavedTo: mappingPath,
+                            manifest: manifests.length === 1 ? manifests[0] : manifests,
                             tree: trees.length === 1 ? trees[0] : trees,
                             stats: allStats.length === 1 ? allStats[0] : allStats,
                             gateResult: gateResults.length === 1 ? gateResults[0] : gateResults,

package/dist/tools/utility-tools.js

@@ -57,20 +57,47 @@ function analyzeLayoutWarnings(spec) {
     return warnings;
 }
 export function registerUtilityTools(server, bridge) {
-    // get_status — Check connection status
+    // get_status — Check connection status + bridge version/capabilities
     server.tool('get_status', 'Check if the Figma Bridge plugin is connected and ready', {}, async () => {
         const connected = bridge.isConnected();
-        return {
-            content: [{
-                    type: 'text',
-                    text: JSON.stringify({
-                        connected,
-                        message: connected
-                            ? 'Figma Bridge plugin is connected and ready.'
-                            : 'Figma Bridge plugin is NOT connected. Please open the "Code to Figma Bridge" plugin in Figma.',
-                    }, null, 2),
-                }],
-        };
+        if (!connected) {
+            return {
+                content: [{
+                        type: 'text',
+                        text: JSON.stringify({
+                            connected: false,
+                            message: 'Figma Bridge plugin is NOT connected. Please open the "Code to Figma Bridge" plugin in Figma.',
+                        }, null, 2),
+                    }],
+            };
+        }
+        try {
+            const bridgeStatus = await bridge.send('bridge-get-status');
+            return {
+                content: [{
+                        type: 'text',
+                        text: JSON.stringify({
+                            connected: true,
+                            message: 'Figma Bridge plugin is connected and ready.',
+                            bridge: {
+                                version: bridgeStatus.version || 'unknown',
+                                capabilities: bridgeStatus.capabilities || [],
+                            },
+                        }, null, 2),
+                    }],
+            };
+        }
+        catch {
+            return {
+                content: [{
+                        type: 'text',
+                        text: JSON.stringify({
+                            connected: true,
+                            message: 'Figma Bridge plugin is connected but bridge-get-status failed. Bridge plugin dist may need rebuild.',
+                        }, null, 2),
+                    }],
+            };
+        }
     });
     // validate_spec — Validate a JSON spec without creating anything
     server.tool('validate_spec', 'Validate a component/variable/style spec (JSON) without creating anything in Figma. Returns validation errors if any.', {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sangheepark/figma-ds-mcp",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "MCP server for Code to Figma Bridge — bridges Claude Code to Figma plugin via WebSocket",
   "type": "module",
   "main": "dist/index.js",