dodraw-mcp-server 0.1.6 → 0.1.7

package/dist/src/tools/diagramTools.js

@@ -211,8 +211,6 @@ exports.toolDefinitions = [
                         required: ["name", "type"]
                     }
                 },
-                x: { type: "number" },
-                y: { type: "number" },
                 id: { type: "string", description: "Optional explicit ID" },
                 referenceNodeId: { type: "string", description: "Optional. ID of an existing node to place this new table relative to." },
                 direction: { type: "string", enum: ["UP", "DOWN", "LEFT", "RIGHT"], description: "Optional. Direction to place relative to referenceNodeId." },
@@ -598,9 +596,9 @@ async function handleToolCall(name, args) {
             const estimatedHeight = 0.8 + cols.length * 0.4;
             const height = Math.max(2, estimatedHeight);
             const width = 3;
-            // Auto-placement logic if X/Y not provided
-            let x = args.x !== undefined ? Number(args.x) : 0;
-            let z = args.y !== undefined ? Number(args.y) : 0;
+            // Auto-placement logic (defaults to 0,0 for first node)
+            let x = 0;
+            let z = 0;
             // Relative Placement Logic
             if (args.referenceNodeId && args.direction) {
                 const sourceNode = state.nodes.find(n => n.id === args.referenceNodeId);
@@ -646,16 +644,8 @@ async function handleToolCall(name, args) {
                         return dx < combinedHalfWidth && dz < combinedHalfHeight;
                     });
                 };
-                let iterations = 0;
-                while (checkCollision(x, z) && iterations < 50) {
-                    iterations++;
-                    if (dirX !== 0)
-                        z += shiftStepZ;
-                    else
-                        x += shiftStepX;
-                }
             }
-            else if (args.x === undefined && args.y === undefined && state.nodes.length > 0) {
+            else if (state.nodes.length > 0) {
                 // Fallback to "next to last" if no explicit relative info
                 const lastNode = state.nodes[state.nodes.length - 1];
                 x = lastNode.x + 5;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dodraw-mcp-server",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "MCP server for DoDraw",
   "main": "dist/src/index.js",
   "bin": {

package/src/tools/diagramTools.ts

@@ -210,10 +210,9 @@ export const toolDefinitions: Tool[] = [
                   isFk: { type: "boolean" }
                },
                required: ["name", "type"]
-            } 
+            }
         },
-        x: { type: "number" },
-        y: { type: "number" },
+
         id: { type: "string", description: "Optional explicit ID" },
         referenceNodeId: { type: "string", description: "Optional. ID of an existing node to place this new table relative to." },
         direction: { type: "string", enum: ["UP", "DOWN", "LEFT", "RIGHT"], description: "Optional. Direction to place relative to referenceNodeId." },
@@ -621,9 +620,9 @@ export async function handleToolCall(name: string, args: any): Promise<any> {
 
             const width = 3;
 
-            // Auto-placement logic if X/Y not provided
-            let x = args.x !== undefined ? Number(args.x) : 0;
-            let z = args.y !== undefined ? Number(args.y) : 0;
+            // Auto-placement logic (defaults to 0,0 for first node)
+            let x = 0;
+            let z = 0;
             
             // Relative Placement Logic
             if (args.referenceNodeId && args.direction) {
@@ -666,13 +665,7 @@ export async function handleToolCall(name: string, args: any): Promise<any> {
                     });
                 };
 
-                let iterations = 0;
-                 while (checkCollision(x, z) && iterations < 50) {
-                    iterations++;
-                    if (dirX !== 0) z += shiftStepZ;
-                    else x += shiftStepX;
-                }
-            } else if (args.x === undefined && args.y === undefined && state.nodes.length > 0) {
+            } else if (state.nodes.length > 0) {
                  // Fallback to "next to last" if no explicit relative info
                  const lastNode = state.nodes[state.nodes.length - 1];
                  x = lastNode.x + 5;