@pixel-normal-edit/mcp 2.1.0 → 2.2.0

package/domains/art-tree/{material_tools.js → tools/materials.js}

@@ -1,6 +1,6 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const materialLessons = require('./material_lessons');
+const { registerTool, sendCommand } = require('../../../core/server');
+const materialLessons = require('../lessons/material');
 
 /**
  * Register all material & surface rendering tools on the MCP server.

package/domains/art-tree/{perspective_tools.js → tools/perspective.js}

@@ -1,6 +1,6 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const perspLessons = require('./perspective_lessons');
+const { registerTool, sendCommand } = require('../../../core/server');
+const perspLessons = require('../lessons/perspective');
 
 const hexColor = z.string().regex(/^#[0-9a-fA-F]{6}$/).describe('Hex color e.g. #ff0000');
 

package/domains/art-tree/{sky_tools.js → tools/sky.js}

@@ -1,6 +1,6 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const skyLessons1 = require('./sky_lessons_1');
+const { registerTool, sendCommand } = require('../../../core/server');
+const skyLessons1 = require('../lessons/sky_1');
 
 /**
  * Register all sky and sun tools on the MCP server.

package/domains/art-tree/{structure_tools.js → tools/structure.js}

@@ -1,6 +1,6 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const structureLessons = require('./structure_lessons');
+const { registerTool, sendCommand } = require('../../../core/server');
+const structureLessons = require('../lessons/structure');
 
 const hexColor = z.string().regex(/^#[0-9a-fA-F]{6}$/).describe('Hex color e.g. #ff0000');
 

package/domains/art-tree/{surface_tools.js → tools/surfaces.js}

@@ -1,6 +1,6 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const surfaceLessons = require('./surface_lessons');
+const { registerTool, sendCommand } = require('../../../core/server');
+const surfaceLessons = require('../lessons/surface');
 
 const hexColor = z.string().regex(/^#[0-9a-fA-F]{6}$/).describe('Hex color e.g. #ff0000');
 

package/domains/art-tree/{transform_tools.js → tools/transforms.js}

@@ -1,7 +1,7 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const lessons1 = require('./transform_lessons_1');
-const lessons2 = require('./transform_lessons_2');
+const { registerTool, sendCommand } = require('../../../core/server');
+const lessons1 = require('../lessons/transform_1');
+const lessons2 = require('../lessons/transform_2');
 
 const hexColor = z.string().regex(/^#[0-9a-fA-F]{6}$/).describe('Hex color e.g. #ff0000');
 

package/domains/art-tree/{vocab_tools.js → tools/vocab.js}

@@ -1,6 +1,6 @@
 const { z } = require('zod');
-const { registerTool, sendCommand } = require('../../core/server');
-const vocabLessons = require('./vocab_lessons');
+const { registerTool, sendCommand } = require('../../../core/server');
+const vocabLessons = require('../lessons/vocab');
 
 /**
  * Register all visual vocabulary tools on the MCP server.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pixel-normal-edit/mcp",
-  "version": "2.1.0",
+  "version": "2.2.0",
   "description": "MCP server for Pixel Normal Edit canvas - enables AI agents to draw pixel art",
   "main": "index.js",
   "bin": {

package/rules/prerequisites.js

@@ -20,7 +20,7 @@ const { registerTool } = require('../core/server');
  */
 function register(server) {
   registerTool(server, 'workflow_start',
-    `🚀 REQUIRED: Call this tool FIRST before drawing anything.
+    `REQUIRED: Call this tool FIRST before drawing anything.
 This tool initializes a drawing workflow with sequential steps for a specific mode.
 Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
     { mode: z.enum([MODES.CREATE_IMAGE, MODES.EDIT_IMAGE, MODES.CREATE_ANIMATION]).describe('Workflow mode to start') },
@@ -28,7 +28,7 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const session = require('../core/command-bus').SESSION;
       const state = workflow.start(session, p.mode);
       return {
-        content: [{ type: 'text', text: `🚀 **Workflow started in mode: ${p.mode}!**\n\n${workflow.getProgressReport(session)}\n\n---\nStart with step: **${state.steps[0].label}** → ${state.steps[0].description}` }]
+        content: [{ type: 'text', text: `**Workflow started in mode: ${p.mode}!**\n\n${workflow.getProgressReport(session)}\n\n---\nStart with step: **${state.steps[0].label}** → ${state.steps[0].description}` }]
       };
     });
 
@@ -40,24 +40,24 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const report = workflow.getProgressReport(session);
       const current = workflow.getCurrentStep(session);
       if (!current) {
-        return { content: [{ type: 'text', text: '⚠️ Workflow has not started. Call **workflow_start** first.' }] };
+        return { content: [{ type: 'text', text: '  Workflow has not started. Call **workflow_start** first.' }] };
       }
       return {
         content: [{
           type: 'text',
-          text: `${report}\n\n👉 **Pending:** ${current.label} — ${current.description}`
+          text: `${report}\n\n  **Pending:** ${current.label} — ${current.description}`
         }]
       };
     });
 
   registerTool(server, 'workflow_advance',
-    '✅ Mark current step as complete and move to the next step. Only call AFTER successfully finishing the current step.',
+    '  Mark current step as complete and move to the next step. Only call AFTER successfully finishing the current step.',
     {},
     () => {
       const session = require('../core/command-bus').SESSION;
       const result = workflow.advance(session);
       if (!result.success) {
-        return { isError: true, content: [{ type: 'text', text: `❌ ${result.message}` }] };
+        return { isError: true, content: [{ type: 'text', text: `  ${result.message}` }] };
       }
       if (!result.nextStep) {
         return { content: [{ type: 'text', text: `🎉 ${result.message}` }] };
@@ -66,7 +66,7 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
     });
 
   registerTool(server, 'workflow_validate',
-    `🔍 Check if the agent is allowed to perform a certain action.
+    `  Check if the agent is allowed to perform a certain action.
   Call this tool BEFORE performing operations to ensure correct workflow order.`,
     { stepId: z.string().describe('ID of the step to validate (e.g., "INITIALIZE_CANVAS")') },
     (p) => {
@@ -75,7 +75,7 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       if (!result.allowed) {
         return { isError: true, content: [{ type: 'text', text: result.message }] };
       }
-      return { content: [{ type: 'text', text: result.message || `✅ You can perform "${p.stepId}". Please proceed!` }] };
+      return { content: [{ type: 'text', text: result.message || `  You can perform "${p.stepId}". Please proceed!` }] };
     });
 
   registerTool(server, 'workflow_set_layer',
@@ -85,9 +85,9 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const session = require('../core/command-bus').SESSION;
       const success = workflow.setLayerAndFrame(session, p.layer, undefined);
       if (!success) {
-        return { isError: true, content: [{ type: 'text', text: '❌ Workflow not started' }] };
+        return { isError: true, content: [{ type: 'text', text: '  Workflow not started' }] };
       }
-      return { content: [{ type: 'text', text: `✅ Active layer set to ${p.layer}` }] };
+      return { content: [{ type: 'text', text: `  Active layer set to ${p.layer}` }] };
     });
 
   registerTool(server, 'workflow_set_frame',
@@ -97,9 +97,9 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const session = require('../core/command-bus').SESSION;
       const success = workflow.setLayerAndFrame(session, undefined, p.frame);
       if (!success) {
-        return { isError: true, content: [{ type: 'text', text: '❌ Workflow not started' }] };
+        return { isError: true, content: [{ type: 'text', text: '  Workflow not started' }] };
       }
-      return { content: [{ type: 'text', text: `✅ Active frame set to ${p.frame}` }] };
+      return { content: [{ type: 'text', text: `  Active frame set to ${p.frame}` }] };
     });
 
   registerTool(server, 'workflow_set_phase',
@@ -109,14 +109,14 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const session = require('../core/command-bus').SESSION;
       const success = workflow.setPhase(session, p.phase);
       if (!success) {
-        return { isError: true, content: [{ type: 'text', text: '❌ Workflow not started' }] };
+        return { isError: true, content: [{ type: 'text', text: '  Workflow not started' }] };
       }
-      return { content: [{ type: 'text', text: `✅ Active phase set to ${p.phase}` }] };
+      return { content: [{ type: 'text', text: `  Active phase set to ${p.phase}` }] };
     });
 
   registerTool(server, 'workflow_setup_animation',
     'Declare animation metadata for tracking (fps, total_frames, loop).',
-    { 
+    {
       fps: z.number().int().optional().describe('Frames per second'),
       total_frames: z.number().int().optional().describe('Total frames in the animation'),
       loop: z.boolean().optional().describe('Whether the animation loops')
@@ -125,14 +125,14 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const session = require('../core/command-bus').SESSION;
       const success = workflow.setAnimationMeta(session, p.fps, p.total_frames, p.loop);
       if (!success) {
-        return { isError: true, content: [{ type: 'text', text: '❌ Workflow not started' }] };
+        return { isError: true, content: [{ type: 'text', text: '  Workflow not started' }] };
       }
-      return { content: [{ type: 'text', text: `✅ Animation meta updated: FPS=${p.fps}, Frames=${p.total_frames}, Loop=${p.loop}` }] };
+      return { content: [{ type: 'text', text: `  Animation meta updated: FPS=${p.fps}, Frames=${p.total_frames}, Loop=${p.loop}` }] };
     });
 
   registerTool(server, 'workflow_create_input_request',
     'Pause the workflow and ask the user for specific input via the UI (e.g. CANVAS_SIZE, APPROVE_PLAN, SELECT_EDIT_REGION).',
-    { 
+    {
       type: z.enum(['CANVAS_SIZE', 'APPROVE_PLAN', 'REVIEW_RESULT', 'SELECT_EDIT_REGION', 'ANIMATION_SETUP']).describe('The type of request'),
       fields: z.record(z.any()).optional().describe('Context or fields for the request')
     },
@@ -140,15 +140,15 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const { sendCommand, SESSION } = require('../core/command-bus');
       const reqId = workflow.createInputRequest(SESSION, p.type, p.fields || {});
       if (!reqId) {
-        return { isError: true, content: [{ type: 'text', text: '❌ Workflow not started' }] };
+        return { isError: true, content: [{ type: 'text', text: '  Workflow not started' }] };
       }
-      
+
       const res = await sendCommand({ action: 'showUserInputRequest', type: p.type, fields: p.fields || {}, reqId }, 35000);
-      
+
       if (!res.isError) {
         workflow.completeInputRequest(SESSION, reqId, res.content[0].text);
       }
-      
+
       return res;
     });
 
@@ -159,18 +159,18 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
       const session = require('../core/command-bus').SESSION;
       const success = workflow.setApprovalRequired(session, p.required);
       if (!success) {
-        return { isError: true, content: [{ type: 'text', text: '❌ Workflow not started' }] };
+        return { isError: true, content: [{ type: 'text', text: '  Workflow not started' }] };
       }
-      return { content: [{ type: 'text', text: p.required ? '✅ Workflow paused, waiting for user approval.' : '✅ Workflow resumed.' }] };
+      return { content: [{ type: 'text', text: p.required ? '  Workflow paused, waiting for user approval.' : '  Workflow resumed.' }] };
     });
 
   registerTool(server, 'workflow_reset',
-    '🔄 Reset workflow to initial state. Clears all progress.',
+    '  Reset workflow to initial state. Clears all progress.',
     {},
     () => {
       const session = require('../core/command-bus').SESSION;
       workflow.reset(session);
-      return { content: [{ type: 'text', text: '🔄 Workflow reset. Please start from the beginning.' }] };
+      return { content: [{ type: 'text', text: '  Workflow reset. Please start from the beginning.' }] };
     });
 
   registerTool(server, 'validate_canvas_name',
@@ -181,9 +181,9 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
     (p) => {
       const result = validateCanvasName(p.name);
       if (!result.valid) {
-        return { isError: true, content: [{ type: 'text', text: `❌ ${result.message}` }] };
+        return { isError: true, content: [{ type: 'text', text: `  ${result.message}` }] };
       }
-      return { content: [{ type: 'text', text: `✅ Name "${p.name}" is valid!` }] };
+      return { content: [{ type: 'text', text: `  Name "${p.name}" is valid!` }] };
     });
 
   registerTool(server, 'validate_canvas_size',
@@ -193,9 +193,9 @@ Modes: CREATE_IMAGE, EDIT_IMAGE, CREATE_ANIMATION`,
     (p) => {
       const result = validateCanvasSize(p.width, p.height);
       if (!result.valid) {
-        return { isError: true, content: [{ type: 'text', text: `❌ ${result.message}` }] };
+        return { isError: true, content: [{ type: 'text', text: `  ${result.message}` }] };
       }
-      return { content: [{ type: 'text', text: `✅ Size ${p.width}x${p.height} is valid!` }] };
+      return { content: [{ type: 'text', text: `  Size ${p.width}x${p.height} is valid!` }] };
     });
 }
 

package/rules/rules.js

@@ -84,7 +84,7 @@ const DRAWING_STEPS = [
 ];
 
 const DRAWING_CONSTRAINTS = {
-  maxCanvasSize: { width: 1024, height: 1024 },
+  maxCanvasSize: { width: 256, height: 256 },
   minCanvasSize: { width: 8, height: 8 },
   allowedColors: null,
   maxStampNameLength: 32,

package/rules/workflow.js

@@ -66,7 +66,7 @@ function start(sessionId, mode = MODES.CREATE_IMAGE) {
     history: [],
     startedAt: Date.now(),
   };
-  
+
   sessions.set(sessionId, state);
   saveState();
   return getState(sessionId);
@@ -185,7 +185,7 @@ function createInputRequest(sessionId, type, fields = {}) {
   if (!state) return false;
 
   state.user_input_request = {
-    id: `req_${Date.now()}_${Math.floor(Math.random()*1000)}`,
+    id: `req_${Date.now()}_${Math.floor(Math.random() * 1000)}`,
     type,
     fields,
     status: 'WAITING_FOR_USER',
@@ -210,7 +210,15 @@ function completeInputRequest(sessionId, reqId, response) {
   state.user_input_request.status = 'COMPLETED';
   state.user_input_request.response = response;
   state.step_status = STATUSES.IN_PROGRESS; // Unpause workflow
-  
+
+  // Auto-set flags based on completed request type
+  if (state.user_input_request.type === 'CANVAS_SIZE') {
+    state.step_completion_flags.canvas_initialized = true;
+  }
+  if (state.user_input_request.type === 'APPROVE_PLAN') {
+    state.step_completion_flags.plan_created = true;
+  }
+
   state.history.push({ event: 'input_request_completed', reqId, time: Date.now() });
   saveState();
   return true;
@@ -222,18 +230,18 @@ function completeInputRequest(sessionId, reqId, response) {
 function setLayerAndFrame(sessionId, layer, frame = null) {
   const state = sessions.get(sessionId);
   if (!state) return false;
-  
+
   if (layer !== undefined && layer !== null) {
     state.current_layer = layer;
   }
   if (frame !== undefined && frame !== null) {
     state.current_frame = frame;
   }
-  state.history.push({ 
-    event: 'layer_frame_change', 
-    layer: state.current_layer, 
-    frame: state.current_frame, 
-    time: Date.now() 
+  state.history.push({
+    event: 'layer_frame_change',
+    layer: state.current_layer,
+    frame: state.current_frame,
+    time: Date.now()
   });
   saveState();
   return true;
@@ -245,12 +253,12 @@ function setLayerAndFrame(sessionId, layer, frame = null) {
 function setPhase(sessionId, phase) {
   const state = sessions.get(sessionId);
   if (!state) return false;
-  
+
   state.current_phase = phase;
-  state.history.push({ 
-    event: 'phase_change', 
-    phase: phase, 
-    time: Date.now() 
+  state.history.push({
+    event: 'phase_change',
+    phase: phase,
+    time: Date.now()
   });
   saveState();
   return true;
@@ -262,11 +270,11 @@ function setPhase(sessionId, phase) {
 function setAnimationMeta(sessionId, fps, total_frames, loop) {
   const state = sessions.get(sessionId);
   if (!state) return false;
-  
+
   if (fps !== undefined) state.animation_meta.fps = fps;
   if (total_frames !== undefined) state.animation_meta.total_frames = total_frames;
   if (loop !== undefined) state.animation_meta.loop = loop;
-  
+
   saveState();
   return true;
 }
@@ -292,7 +300,7 @@ function advance(sessionId) {
   if (!state) {
     return { success: false, message: 'Workflow has not started' };
   }
-  
+
   if (state.user_approval_required) {
     return { success: false, message: 'Cannot advance: Waiting for user approval.' };
   }
@@ -308,7 +316,7 @@ function advance(sessionId) {
 
   // Pre-condition validation block
   if (current.id === 'INITIALIZE_CANVAS' && !state.step_completion_flags.canvas_initialized) {
-    return { success: false, message: 'Cannot advance: Canvas has not been initialized. Create a canvas first.' };
+    return { success: false, message: 'Cannot advance: Canvas has not been initialized. You MUST use workflow_create_input_request with type CANVAS_SIZE first.' };
   }
   if (current.id === 'PLAN_DRAWING' && !state.step_completion_flags.plan_created) {
     return { success: false, message: 'Cannot advance: Drawing plan has not been finalized.' };
@@ -374,10 +382,10 @@ function getProgressReport(sessionId) {
     state.user_input_request && state.user_input_request.status === 'WAITING_FOR_USER' ? `⚠️ PENDING REQUEST: ${state.user_input_request.type}` : '',
     `\n**Steps:**`
   ];
-  
+
   state.steps.forEach((step, i) => {
     const icon = step.status === STATUSES.COMPLETED ? '✅' :
-                 step.status === STATUSES.IN_PROGRESS ? '🔄' : '⬜';
+      step.status === STATUSES.IN_PROGRESS ? '🔄' : '⬜';
     const marker = i === state.currentStepIndex ? ' ← **IN PROGRESS**' : '';
     lines.push(`${icon} **${step.label}**${marker}`);
     if (step.status === STATUSES.IN_PROGRESS || i === state.currentStepIndex) {

package/tools/canvas-tools.js

@@ -16,8 +16,8 @@ function register(server) {
   registerTool(server, 'canvas_resize',
     'Resize canvas to new dimensions. mode=clear resets pixels, extend keeps them.',
     {
-      width: z.number().int().min(1).max(1024).describe('New width in pixels'),
-      height: z.number().int().min(1).max(1024).describe('New height in pixels'),
+      width: z.number().int().min(1).max(256).describe('New width in pixels (max 256)'),
+      height: z.number().int().min(1).max(256).describe('New height in pixels (max 256)'),
       mode: z.enum(['clear', 'extend', 'fit']).default('clear').describe('How to handle existing content'),
       dx: z.number().int().default(0).describe('X offset for content when extending'),
       dy: z.number().int().default(0).describe('Y offset for content when extending')

package/tools/drawing-tools.js

@@ -14,54 +14,65 @@ const hexColor = z.string().regex(/^#[0-9a-fA-F]{6}$/).describe('Hex color e.g.
 function register(server) {
   registerTool(server, 'draw_pixel',
     'Set a single pixel to a color',
-    { x: z.number().int().describe('X coordinate'), y: z.number().int().describe('Y coordinate'), color: hexColor },
+    { x: z.number().int().max(1024).describe('X coordinate'), y: z.number().int().max(1024).describe('Y coordinate'), color: hexColor },
     (p) => ({ action: 'drawPixel', ...p }));
 
   registerTool(server, 'draw_erase',
     'Erase a single pixel (make transparent)',
-    { x: z.number().int(), y: z.number().int() },
+    { x: z.number().int().max(1024), y: z.number().int().max(1024) },
     (p) => ({ action: 'erasePixel', ...p }));
 
+  registerTool(server, 'draw_pixels_bulk',
+    'Set multiple pixels at once (much faster than calling draw_pixel repeatedly)',
+    { 
+      pixels: z.array(z.object({ 
+        x: z.number().int().max(1024), 
+        y: z.number().int().max(1024), 
+        color: hexColor.optional() 
+      })).max(5000).describe('Array of pixels {x, y, color}. Omit color to erase.')
+    },
+    (p) => ({ action: 'drawPixelsBulk', pixels: p.pixels }));
+
   registerTool(server, 'draw_line',
     'Draw a straight line using Bresenham algorithm',
-    { x0: z.number().int(), y0: z.number().int(), x1: z.number().int(), y1: z.number().int(), color: hexColor },
+    { x0: z.number().int().max(1024), y0: z.number().int().max(1024), x1: z.number().int().max(1024), y1: z.number().int().max(1024), color: hexColor },
     (p) => ({ action: 'drawLine', ...p }));
 
   registerTool(server, 'draw_rect',
     'Draw a rectangle (outline or filled)',
     {
-      x: z.number().int().describe('Top-left X'), y: z.number().int().describe('Top-left Y'),
-      w: z.number().int().min(1).describe('Width'), h: z.number().int().min(1).describe('Height'),
+      x: z.number().int().max(1024).describe('Top-left X'), y: z.number().int().max(1024).describe('Top-left Y'),
+      w: z.number().int().min(1).max(256).describe('Width (max 256)'), h: z.number().int().min(1).max(256).describe('Height (max 256)'),
       color: hexColor, filled: z.boolean().default(false).describe('Fill interior?')
     },
     (p) => ({ action: 'drawRect', ...p }));
 
   registerTool(server, 'draw_circle',
     'Draw a circle (outline or filled)',
-    { cx: z.number().int().describe('Center X'), cy: z.number().int().describe('Center Y'),
-      r: z.number().int().min(1).describe('Radius'), color: hexColor, filled: z.boolean().default(false) },
+    { cx: z.number().int().max(1024).describe('Center X'), cy: z.number().int().max(1024).describe('Center Y'),
+      r: z.number().int().min(1).max(128).describe('Radius (max 128)'), color: hexColor, filled: z.boolean().default(false) },
     (p) => ({ action: 'drawCircle', ...p }));
 
   registerTool(server, 'draw_ellipse',
     'Draw an ellipse (outline or filled)',
-    { cx: z.number().int(), cy: z.number().int(), rx: z.number().int().min(1).describe('Horizontal radius'),
-      ry: z.number().int().min(1).describe('Vertical radius'), color: hexColor, filled: z.boolean().default(false) },
+    { cx: z.number().int().max(1024), cy: z.number().int().max(1024), rx: z.number().int().min(1).max(128).describe('Horizontal radius'),
+      ry: z.number().int().min(1).max(128).describe('Vertical radius'), color: hexColor, filled: z.boolean().default(false) },
     (p) => ({ action: 'drawEllipse', ...p }));
 
   registerTool(server, 'draw_polygon',
     'Draw a polygon from a list of points (outline or filled)',
-    { points: z.array(z.object({ x: z.number().int(), y: z.number().int() })).min(3).describe('Array of {x,y} vertices'),
+    { points: z.array(z.object({ x: z.number().int().max(1024), y: z.number().int().max(1024) })).min(3).describe('Array of {x,y} vertices'),
       color: hexColor, filled: z.boolean().default(false) },
     (p) => ({ action: 'drawPolygon', ...p }));
 
   registerTool(server, 'draw_fill',
     'Flood-fill starting from (x,y) with a color',
-    { x: z.number().int(), y: z.number().int(), color: hexColor },
+    { x: z.number().int().max(1024), y: z.number().int().max(1024), color: hexColor },
     (p) => ({ action: 'fill', ...p }));
 
   registerTool(server, 'draw_gradient_rect',
     'Draw a rectangle filled with a smooth gradient',
-    { x: z.number().int(), y: z.number().int(), w: z.number().int().min(1), h: z.number().int().min(1),
+    { x: z.number().int().max(1024), y: z.number().int().max(1024), w: z.number().int().min(1).max(256).describe('Width (max 256)'), h: z.number().int().min(1).max(256).describe('Height (max 256)'),
       colorFrom: hexColor.describe('Start color'), colorTo: hexColor.describe('End color'),
       direction: z.enum(['h', 'v']).default('h').describe('h=horizontal, v=vertical') },
     (p) => ({ action: 'drawGradientRect', ...p }));

package/tools/workspace-tools.js

@@ -19,21 +19,22 @@ function register(server) {
     {},
     () => ({ action: 'getActiveTabId' }));
 
-  server.tool('workspace_create_tab',
+  registerTool(server, 'workspace_create_tab',
     'Create a new blank canvas tab',
     {
       name: z.string().optional().describe('Tab name'),
-      width: z.number().int().optional().describe('Canvas width'),
-      height: z.number().int().optional().describe('Canvas height')
+      width: z.number().int().min(1).max(256).optional().describe('Canvas width (max 256)'),
+      height: z.number().int().min(1).max(256).optional().describe('Canvas height (max 256)')
     },
     async (p) => {
       if (p.width === undefined || p.height === undefined) {
         return {
+          action: 'noop',
           isError: true,
           content: [{ type: 'text', text: 'Please provide the width and height dimensions (resize) when creating a new canvas.' }]
         };
       }
-      return sendCommand({ action: 'createTab', ...p });
+      return { action: 'createTab', ...p };
     }
   );
 

package/domains/art-tree/3d_lessons.js

@@ -1,114 +0,0 @@
-const shapes3d = require('./3d_shapes');
-const advShapes = require('./advanced_shapes');
-
-/**
- * Lesson: Box Properties (Width, Height, Depth, Planes, Edges)
- */
-function lessonBoxProperties(canvasSize = 32, color = '#2196f3') {
-  const commands = [];
-  const cx = Math.floor(canvasSize / 2);
-  const cy = Math.floor(canvasSize / 2);
-  const w = Math.floor(canvasSize * 0.4);
-  const h = Math.floor(canvasSize * 0.4);
-  const d = Math.floor(canvasSize * 0.3);
-
-  // Draw wireframe box
-  commands.push(...shapes3d.drawWireframeBox(cx, cy, w, h, d, color));
-  
-  // Highlight dimensions
-  const fX = cx - Math.floor(w/2);
-  const fY = cy - Math.floor(h/2);
-  
-  // Width
-  commands.push(...advShapes.drawDistance(fX, fY + h + 2, fX + w, fY + h + 2, '#4caf50'));
-  // Height
-  commands.push(...advShapes.drawDistance(fX - 2, fY, fX - 2, fY + h, '#ff9800'));
-  
-  // Depth (approximate along the oblique edge)
-  const dx = Math.floor(d * 0.7);
-  const dy = Math.floor(d * 0.5);
-  commands.push(...advShapes.drawDistance(fX + w + 2, fY + h, fX + w + 2 + dx, fY + h - dy, '#e91e63'));
-
-  return commands;
-}
-
-/**
- * Lesson: Sphere Properties (Center, Radius, Cross contours, Axis)
- */
-function lessonSphereProperties(canvasSize = 32, color = '#9c27b0') {
-  const commands = [];
-  const cx = Math.floor(canvasSize / 2);
-  const cy = Math.floor(canvasSize / 2);
-  const r = Math.floor(canvasSize * 0.35);
-
-  // Draw sphere with volume lines
-  commands.push(...shapes3d.drawSphereVolume(cx, cy, r, color));
-  
-  // Highlight radius
-  commands.push(...advShapes.drawDistance(cx, cy, cx + r, cy, '#ff9800'));
-
-  return commands;
-}
-
-/**
- * Lesson: Cylinder Properties (Central axis, Bases, Perspective change)
- */
-function lessonCylinderProperties(canvasSize = 32, color = '#00bcd4') {
-  const commands = [];
-  const cx = Math.floor(canvasSize / 2);
-  const cy = Math.floor(canvasSize / 2);
-  const r = Math.floor(canvasSize * 0.3);
-  const h = Math.floor(canvasSize * 0.6);
-
-  // Draw cylinder volume
-  commands.push(...shapes3d.drawCylinderVolume(cx, cy, r, h, color));
-  
-  // Highlight Height
-  commands.push(...advShapes.drawDistance(cx + r + 4, cy - Math.floor(h/2), cx + r + 4, cy + Math.floor(h/2), '#4caf50'));
-
-  // Highlight Radius (top base)
-  commands.push(...advShapes.drawDistance(cx, cy - Math.floor(h/2), cx + r, cy - Math.floor(h/2), '#ff9800'));
-
-  return commands;
-}
-
-/**
- * Lesson: Cone Properties (Apex, Base, Axis)
- */
-function lessonConeProperties(canvasSize = 32, color = '#ff5722') {
-  const commands = [];
-  const cx = Math.floor(canvasSize / 2);
-  const cy = Math.floor(canvasSize / 2);
-  const r = Math.floor(canvasSize * 0.35);
-  const h = Math.floor(canvasSize * 0.6);
-
-  // Draw cone volume
-  commands.push(...shapes3d.drawConeVolume(cx, cy, r, h, color));
-  
-  // Highlight Height
-  const apexY = cy - Math.floor(h/2);
-  const baseY = cy + Math.floor(h/2);
-  commands.push(...advShapes.drawDistance(cx + r + 4, apexY, cx + r + 4, baseY, '#4caf50'));
-  
-  // Highlight Radius (base)
-  commands.push(...advShapes.drawDistance(cx, baseY, cx + r, baseY, '#ff9800'));
-
-  return commands;
-}
-
-function get3DLessonCatalog() {
-  return [
-    { id: '3d_box', name: '3D Box Properties', description: 'Width, height, depth, planes, inner structure', fn: lessonBoxProperties },
-    { id: '3d_sphere', name: '3D Sphere Properties', description: 'Center, radius, axis, cross contours', fn: lessonSphereProperties },
-    { id: '3d_cylinder', name: '3D Cylinder Properties', description: 'Axis, bases, height, perspective changes', fn: lessonCylinderProperties },
-    { id: '3d_cone', name: '3D Cone Properties', description: 'Apex, base, axis, radius', fn: lessonConeProperties },
-  ];
-}
-
-module.exports = {
-  lessonBoxProperties,
-  lessonSphereProperties,
-  lessonCylinderProperties,
-  lessonConeProperties,
-  get3DLessonCatalog
-};