@vint.tri/report_gen_mcp 1.7.0 → 1.7.2

package/FIXED_NEURAL_NETWORK_INSTRUCTIONS_v1.6.5.md

@@ -498,6 +498,50 @@
 }
 ```
 
+### Изображения, сгенерированные с помощью generate-image (Chutes AI)
+
+При использовании инструмента `generate-image` для генерации изображений с помощью Chutes AI, важно правильно форматировать параметры. Все числовые параметры должны быть представлены как числа, а не как строки.
+
+✅ ПРАВИЛЬНО (числовые параметры как числа):
+```json
+{
+  "method": "tools/call",
+  "params": {
+    "name": "generate-image",
+    "arguments": {
+      "prompt": "a dog made of US dollar bills",
+      "width": 512,
+      "height": 512,
+      "guidance_scale": 7.5,
+      "negative_prompt": "blurry, low quality",
+      "num_inference_steps": 50,
+      "seed": 0
+    }
+  }
+}
+```
+
+❌ НЕПРАВИЛЬНО (числовые параметры как строки):
+```json
+{
+  "method": "tools/call",
+  "params": {
+    "name": "generate-image",
+    "arguments": {
+      "prompt": "a dog made of US dollar bills",
+      "width": "512",
+      "height": "512",
+      "guidance_scale": "7.5",
+      "negative_prompt": "blurry, low quality",
+      "num_inference_steps": "50",
+      "seed": "0"
+    }
+  }
+}
+```
+
+**ВАЖНО:** Все числовые параметры (`width`, `height`, `guidance_scale`, `num_inference_steps`, `seed`) должны быть представлены как числа, а не как строки в кавычках. Это критически важно для корректной работы инструмента.
+
 ### Изображения по URL
 
 ```json

package/IMAGE_ELEMENT_FIX_SUMMARY.md

@@ -0,0 +1,66 @@
+# Image Element Fix Summary
+
+## Problem Description
+The MCP server was failing to generate reports when image elements with type "image" were provided. The error occurred because:
+
+1. The `generate-report` tool schema in `src/index.ts` only accepted "url" as a valid image type, but not "image"
+2. The report generator in `src/utils/reportGenerator.ts` had the same limitation
+
+## Solution Implemented
+
+### 1. Updated Schema Validation (src/index.ts)
+Added support for "image" type elements in the Zod schema validation:
+
+```javascript
+z.object({
+  type: z.literal("image"),
+  config: z.object({
+    url: z.string().url().describe("File URL of the image"),
+    alt: z.string().optional().describe("Alternative text for the image"),
+    width: z.number().optional().describe("Width of the image in pixels"),
+    height: z.number().optional().describe("Height of the image in pixels"),
+  }),
+}).describe("Local image file configuration"),
+```
+
+### 2. Enhanced Report Generator (src/utils/reportGenerator.ts)
+Updated the report generator to handle "image" type elements by:
+
+1. Extending the `ImageType` type definition:
+   ```typescript
+   type ImageType = 'url' | 'image';
+   ```
+
+2. Updating the interface definition:
+   ```typescript
+   interface UrlImageElement {
+     type: 'url' | 'image';
+     config: z.infer<typeof urlImageSchema>;
+   }
+   ```
+
+3. Adding "image" to the image renderers mapping:
+   ```typescript
+   const imageRenderers: Record<ImageType, { schema: z.ZodObject<any>; renderer: (config: any) => Promise<string> }> = {
+     url: { schema: urlImageSchema, renderer: renderUrlImage },
+     image: { schema: urlImageSchema, renderer: renderUrlImage },
+   };
+   ```
+
+## Testing
+Created comprehensive tests to verify that the fix works correctly:
+
+1. **File URL Test**: Verified that local file URLs work correctly with the "image" type
+2. **Web URL Test**: Verified that web URLs also work correctly with the "image" type
+
+Both tests passed successfully, confirming that the fix resolves the original issue.
+
+## Benefits
+1. **Backward Compatibility**: Existing code using "url" type continues to work
+2. **Consistency**: Both "url" and "image" types now behave identically
+3. **Neural Network Integration**: The neural network can now successfully generate reports with image elements
+4. **Flexibility**: Supports both local file URLs and web URLs with the same interface
+
+## Files Modified
+1. `src/index.ts` - Updated schema validation for the generate-report tool
+2. `src/utils/reportGenerator.ts` - Enhanced report generator to handle "image" type elements

package/IMAGE_GENERATION_STRING_PARAMS_FIX.md

@@ -0,0 +1,108 @@
+# Fix for Image Generation String Parameters Issue
+
+## Problem Description
+
+The neural network was sending numeric parameters as strings to the `generate-image` tool, causing validation errors:
+
+```
+Error calling tool generate-image: Error: Error invoking remote method 'mcp:call-tool': McpError: MCP error -32602: Invalid arguments for tool generate-image: [
+  {
+    "code": "invalid_type",
+    "expected": "number",
+    "received": "string",
+    "path": ["width"],
+    "message": "Expected number, received string"
+  },
+  // Similar errors for height, guidance_scale, num_inference_steps, seed
+]
+```
+
+## Root Cause
+
+The Zod schema validation in the MCP server expected actual numbers for numeric parameters, but the neural network was sending them as JSON strings.
+
+## Solution Implemented
+
+### 1. Updated Image Generation Server
+
+Modified `src/mcp/imageGenerationServer.ts` to automatically convert string parameters to numbers:
+
+```typescript
+// Преобразуем строковые значения в числа, если необходимо
+const generationParams: Partial<ImageGenerationSettings> = {};
+if (args.width !== undefined) {
+  generationParams.width = typeof args.width === 'string' ? parseFloat(args.width) : args.width;
+}
+if (args.height !== undefined) {
+  generationParams.height = typeof args.height === 'string' ? parseFloat(args.height) : args.height;
+}
+if (args.guidance_scale !== undefined) {
+  generationParams.guidance_scale = typeof args.guidance_scale === 'string' ? parseFloat(args.guidance_scale) : args.guidance_scale;
+}
+if (args.negative_prompt !== undefined) generationParams.negative_prompt = args.negative_prompt;
+if (args.num_inference_steps !== undefined) {
+  generationParams.num_inference_steps = typeof args.num_inference_steps === 'string' ? parseInt(args.num_inference_steps) : args.num_inference_steps;
+}
+if (args.seed !== undefined) {
+  generationParams.seed = typeof args.seed === 'string' ? parseInt(args.seed) : args.seed;
+}
+```
+
+This change was applied to both `generate_image` and `generate_image_to_file` tools in the server.
+
+### 2. Updated Neural Network Instructions
+
+Added clear guidelines in `FIXED_NEURAL_NETWORK_INSTRUCTIONS_v1.6.5.md` about proper parameter formatting:
+
+#### Correct Format (Numbers as Numbers):
+```json
+{
+  "method": "tools/call",
+  "params": {
+    "name": "generate-image",
+    "arguments": {
+      "prompt": "a dog made of US dollar bills",
+      "width": 512,
+      "height": 512,
+      "guidance_scale": 7.5,
+      "negative_prompt": "blurry, low quality",
+      "num_inference_steps": 50,
+      "seed": 0
+    }
+  }
+}
+```
+
+#### Incorrect Format (Numbers as Strings):
+```json
+{
+  "method": "tools/call",
+  "params": {
+    "name": "generate-image",
+    "arguments": {
+      "prompt": "a dog made of US dollar bills",
+      "width": "512",
+      "height": "512",
+      "guidance_scale": "7.5",
+      "negative_prompt": "blurry, low quality",
+      "num_inference_steps": "50",
+      "seed": "0"
+    }
+  }
+}
+```
+
+## Testing
+
+Created `test-image-generation-string-params.js` to verify the fix works correctly with both string and mixed parameter types.
+
+## Backward Compatibility
+
+The fix maintains backward compatibility by:
+1. Still accepting numeric parameters as numbers
+2. Converting string parameters to numbers when needed
+3. Not changing the external API interface
+
+## Impact
+
+This fix resolves the issue where neural networks sending string-formatted numeric parameters would receive validation errors, allowing seamless image generation with the Chutes AI service.

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AAieA;;;;;GAKG;AACH,wBAAsB,aAAa,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;IAC5D,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,IAAI,CAAC,EAAE,MAAM,CAAC;CACf,GAAG,OAAO,CAAC;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,CAgCjD"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AA0eA;;;;;GAKG;AACH,wBAAsB,aAAa,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;IAC5D,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,IAAI,CAAC,EAAE,MAAM,CAAC;CACf,GAAG,OAAO,CAAC;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,CAgCjD"}

package/dist/index.js

@@ -103,7 +103,7 @@ if (process.argv.length === 2) {
     // No command specified, run in stdio mode using MCP SDK
     const mcpServer = new McpServer({
         name: "report_gen_mcp",
-        version: "1.7.0"
+        version: "1.7.2"
     }, {
         // Disable health check to prevent automatic calls
         capabilities: {
@@ -153,6 +153,15 @@ if (process.argv.length === 2) {
                         height: z.number().optional().describe("Height of the image in pixels"),
                     }),
                 }).describe("Image from URL configuration"),
+                z.object({
+                    type: z.literal("image"),
+                    config: z.object({
+                        url: z.string().url().describe("File URL of the image"),
+                        alt: z.string().optional().describe("Alternative text for the image"),
+                        width: z.number().optional().describe("Width of the image in pixels"),
+                        height: z.number().optional().describe("Height of the image in pixels"),
+                    }),
+                }).describe("Local image file configuration"),
             ])).describe("Report elements (charts and images) mapped by ID"),
             outputFile: z.string().optional().describe("Output HTML file path"),
             tempDirectory: z.string().optional().describe("Temporary directory for file storage (optional, will use REPORTS_DIR environment variable if set)"),

package/dist/mcp/imageGenerationServer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"imageGenerationServer.d.ts","sourceRoot":"","sources":["../../src/mcp/imageGenerationServer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAeH,UAAU,uBAAuB;IAC/B,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,MAAM,CAAC;IACf,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,EAAE,MAAM,CAAC;IACxB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;CACrB;AAED,cAAM,cAAc;IAClB,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,eAAe,CAA0B;;IAiBjD,OAAO,CAAC,aAAa;IAIf,aAAa,CAAC,MAAM,EAAE,MAAM,EAAE,gBAAgB,GAAE,OAAO,CAAC,uBAAuB,CAAM,GAAG,OAAO,CAAC,MAAM,CAAC;CAuE9G;AAyMD,iBAAe,IAAI,kBASlB;AAOD,OAAO,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC"}
+{"version":3,"file":"imageGenerationServer.d.ts","sourceRoot":"","sources":["../../src/mcp/imageGenerationServer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAeH,UAAU,uBAAuB;IAC/B,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,MAAM,CAAC;IACf,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,EAAE,MAAM,CAAC;IACxB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;CACrB;AAED,cAAM,cAAc;IAClB,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,eAAe,CAA0B;;IAiBjD,OAAO,CAAC,aAAa;IAIf,aAAa,CAAC,MAAM,EAAE,MAAM,EAAE,gBAAgB,GAAE,OAAO,CAAC,uBAAuB,CAAM,GAAG,OAAO,CAAC,MAAM,CAAC;CAuE9G;AA+ND,iBAAe,IAAI,kBASlB;AAOD,OAAO,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC"}

package/dist/mcp/imageGenerationServer.js

@@ -121,19 +121,25 @@ app.registerTool("generate_image", {
             throw new Error("Параметр 'prompt' обязателен");
         }
         // Дополнительные параметры (необязательные)
+        // Преобразуем строковые значения в числа, если необходимо
         const generationParams = {};
-        if (args.width !== undefined)
-            generationParams.width = args.width;
-        if (args.height !== undefined)
-            generationParams.height = args.height;
-        if (args.guidance_scale !== undefined)
-            generationParams.guidance_scale = args.guidance_scale;
+        if (args.width !== undefined) {
+            generationParams.width = typeof args.width === 'string' ? parseFloat(args.width) : args.width;
+        }
+        if (args.height !== undefined) {
+            generationParams.height = typeof args.height === 'string' ? parseFloat(args.height) : args.height;
+        }
+        if (args.guidance_scale !== undefined) {
+            generationParams.guidance_scale = typeof args.guidance_scale === 'string' ? parseFloat(args.guidance_scale) : args.guidance_scale;
+        }
         if (args.negative_prompt !== undefined)
             generationParams.negative_prompt = args.negative_prompt;
-        if (args.num_inference_steps !== undefined)
-            generationParams.num_inference_steps = args.num_inference_steps;
-        if (args.seed !== undefined)
-            generationParams.seed = args.seed;
+        if (args.num_inference_steps !== undefined) {
+            generationParams.num_inference_steps = typeof args.num_inference_steps === 'string' ? parseInt(args.num_inference_steps) : args.num_inference_steps;
+        }
+        if (args.seed !== undefined) {
+            generationParams.seed = typeof args.seed === 'string' ? parseInt(args.seed) : args.seed;
+        }
         // Генерируем изображение
         const imageDataUri = await imageGen.generateImage(prompt, generationParams);
         // Извлекаем только base64 данные без префикса
@@ -216,19 +222,25 @@ app.registerTool("generate_image_to_file", {
             throw new Error("Параметр 'filename' обязателен");
         }
         // Дополнительные параметры (необязательные)
+        // Преобразуем строковые значения в числа, если необходимо
         const generationParams = {};
-        if (args.width !== undefined)
-            generationParams.width = args.width;
-        if (args.height !== undefined)
-            generationParams.height = args.height;
-        if (args.guidance_scale !== undefined)
-            generationParams.guidance_scale = args.guidance_scale;
+        if (args.width !== undefined) {
+            generationParams.width = typeof args.width === 'string' ? parseFloat(args.width) : args.width;
+        }
+        if (args.height !== undefined) {
+            generationParams.height = typeof args.height === 'string' ? parseFloat(args.height) : args.height;
+        }
+        if (args.guidance_scale !== undefined) {
+            generationParams.guidance_scale = typeof args.guidance_scale === 'string' ? parseFloat(args.guidance_scale) : args.guidance_scale;
+        }
         if (args.negative_prompt !== undefined)
             generationParams.negative_prompt = args.negative_prompt;
-        if (args.num_inference_steps !== undefined)
-            generationParams.num_inference_steps = args.num_inference_steps;
-        if (args.seed !== undefined)
-            generationParams.seed = args.seed;
+        if (args.num_inference_steps !== undefined) {
+            generationParams.num_inference_steps = typeof args.num_inference_steps === 'string' ? parseInt(args.num_inference_steps) : args.num_inference_steps;
+        }
+        if (args.seed !== undefined) {
+            generationParams.seed = typeof args.seed === 'string' ? parseInt(args.seed) : args.seed;
+        }
         // Генерируем изображение
         const imageDataUri = await imageGen.generateImage(prompt, generationParams);
         // Извлекаем только base64 данные без префикса

package/dist/utils/reportGenerator.d.ts

@@ -6,7 +6,7 @@ interface ChartElement {
     config: any;
 }
 interface UrlImageElement {
-    type: 'url';
+    type: 'url' | 'image';
     config: z.infer<typeof urlImageSchema>;
 }
 type ImageElement = UrlImageElement;

package/dist/utils/reportGenerator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reportGenerator.d.ts","sourceRoot":"","sources":["../../src/utils/reportGenerator.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAOxB,OAAO,EAAE,cAAc,EAAkB,MAAM,wBAAwB,CAAC;AAGxE,KAAK,SAAS,GAAG,KAAK,GAAG,MAAM,GAAG,KAAK,GAAG,UAAU,GAAG,OAAO,GAAG,WAAW,CAAC;AAI7E,UAAU,YAAY;IACpB,IAAI,EAAE,SAAS,CAAC;IAChB,MAAM,EAAE,GAAG,CAAC;CACb;AAED,UAAU,eAAe;IACvB,IAAI,EAAE,KAAK,CAAC;IACZ,MAAM,EAAE,CAAC,CAAC,KAAK,CAAC,OAAO,cAAc,CAAC,CAAC;CACxC;AAED,KAAK,YAAY,GAAG,eAAe,CAAC;AACpC,KAAK,aAAa,GAAG,YAAY,GAAG,YAAY,CAAC;AAgDjD,wBAAsB,cAAc,CAClC,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,aAAa,CAAC,EACvC,UAAU,EAAE,MAAM;;;GA+HnB"}
+{"version":3,"file":"reportGenerator.d.ts","sourceRoot":"","sources":["../../src/utils/reportGenerator.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAOxB,OAAO,EAAE,cAAc,EAAkB,MAAM,wBAAwB,CAAC;AAGxE,KAAK,SAAS,GAAG,KAAK,GAAG,MAAM,GAAG,KAAK,GAAG,UAAU,GAAG,OAAO,GAAG,WAAW,CAAC;AAI7E,UAAU,YAAY;IACpB,IAAI,EAAE,SAAS,CAAC;IAChB,MAAM,EAAE,GAAG,CAAC;CACb;AAED,UAAU,eAAe;IACvB,IAAI,EAAE,KAAK,GAAG,OAAO,CAAC;IACtB,MAAM,EAAE,CAAC,CAAC,KAAK,CAAC,OAAO,cAAc,CAAC,CAAC;CACxC;AAED,KAAK,YAAY,GAAG,eAAe,CAAC;AACpC,KAAK,aAAa,GAAG,YAAY,GAAG,YAAY,CAAC;AAiDjD,wBAAsB,cAAc,CAClC,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,aAAa,CAAC,EACvC,UAAU,EAAE,MAAM;;;GA+HnB"}

package/dist/utils/reportGenerator.js

@@ -18,6 +18,7 @@ const chartRenderers = {
 };
 const imageRenderers = {
     url: { schema: urlImageSchema, renderer: renderUrlImage },
+    image: { schema: urlImageSchema, renderer: renderUrlImage },
 };
 // Utility function to normalize chart configuration data
 // Converts string values to arrays where the schema expects arrays

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vint.tri/report_gen_mcp",
-  "version": "1.7.0",
+  "version": "1.7.2",
   "description": "CLI tool for generating HTML reports with embedded charts and images",
   "main": "dist/index.js",
   "bin": {