@vint.tri/report_gen_mcp 1.0.25 → 1.0.26

package/dist/index.js

@@ -15,36 +15,16 @@ const zod_1 = require("zod");
 const app = (0, express_1.default)();
 const port = 3000;
 app.use(express_1.default.json());
-// Parse command line arguments to get the reports directory
-commander_1.program
-    .option('-d, --directory <path>', 'Directory for generated reports (required)')
-    .parse(process.argv);
-const options = commander_1.program.opts();
-// Check if directory parameter is provided
-if (!options.directory) {
-    console.error('Error: The --directory (-d) parameter is required to specify the folder for generated reports.');
-    console.error('Example usage:');
-    console.error('  report_gen_mcp --directory /path/to/reports');
-    console.error('  report_gen_mcp --directory ./reports start-server');
-    console.error('  report_gen_mcp --directory ./reports generate --document doc.md --charts charts.json');
-    process.exit(1);
-}
-// Validate that the directory exists or can be created
-let reportsDir;
-try {
-    reportsDir = path_1.default.resolve(options.directory);
-    fs_extra_1.default.ensureDirSync(reportsDir);
-}
-catch (error) {
-    console.error(`Error: Cannot access or create directory "${options.directory}"`);
-    process.exit(1);
-}
-// Use the reports directory as the base directory for all operations
-let homeDir = reportsDir;
 app.post('/generate-report', async (req, res) => {
+    // Check if tempDirectory parameter is provided
     const { document, charts, outputFile = 'report.html', tempDirectory } = req.body;
-    const effectiveTempDir = tempDirectory || homeDir;
-    const outputPath = path_1.default.resolve(effectiveTempDir, outputFile);
+    if (!tempDirectory) {
+        return res.status(400).json({
+            error: 'tempDirectory parameter is required. Please provide the directory where reports should be generated.',
+            example: 'curl -X POST http://localhost:3000/generate-report -H "Content-Type: application/json" -d \'{"document": "# Report", "charts": {}, "tempDirectory": "/path/to/reports"}\''
+        });
+    }
+    const outputPath = path_1.default.resolve(tempDirectory, outputFile);
     try {
         const result = await (0, reportGenerator_1.generateReport)(document, charts, outputPath);
         // Send the file content back to the client
@@ -67,13 +47,20 @@ commander_1.program
     });
 });
 commander_1.program
-    .command('generate')
+    .command('generate <directory>')
     .option('--document <md>', 'Markdown document with placeholders [[chart:id]]')
     .option('--charts <json>', 'JSON string of charts {id: {type: "bar", config: {...}}}')
     .option('--output <file>', 'Output HTML file', 'report.html')
-    .action(async (opts) => {
+    .description('Generate a report in the specified directory')
+    .action(async (directory, opts) => {
+    // Check if directory parameter is provided
+    if (!directory) {
+        console.error('Error: Directory parameter is required. Please specify the directory where reports should be generated.');
+        console.error('Example: npx @vint.tri/report_gen_mcp@latest generate /path/to/reports --document "# Report" --charts \'{}\'');
+        process.exit(1);
+    }
     const charts = JSON.parse(opts.charts);
-    const result = await (0, reportGenerator_1.generateReport)(opts.document, charts, opts.output);
+    const result = await (0, reportGenerator_1.generateReport)(opts.document, charts, path_1.default.resolve(directory, opts.output));
     console.log(result);
 });
 // Handle stdio mode for Claude Desktop integration
@@ -126,7 +113,11 @@ if (process.argv.length === 2) {
         else if (params.arguments && typeof params.arguments === 'object') {
             processedParams = params.arguments;
         }
-        const { document, charts, outputFile = 'report.html', tempDirectory = process.cwd() } = processedParams;
+        // Check if tempDirectory parameter is provided
+        if (!processedParams.tempDirectory) {
+            throw new Error('tempDirectory parameter is required. Please provide the directory where reports should be generated.');
+        }
+        const { document, charts, outputFile = 'report.html', tempDirectory } = processedParams;
         const outputPath = path_1.default.resolve(tempDirectory, outputFile);
         try {
             const result = await (0, reportGenerator_1.generateReport)(document, charts, outputPath);

package/package.json

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

package/src/index.ts

@@ -15,40 +15,18 @@ const port = 3000;
 
 app.use(express.json());
 
-// Parse command line arguments to get the reports directory
-program
-  .option('-d, --directory <path>', 'Directory for generated reports (required)')
-  .parse(process.argv);
-
-const options = program.opts();
-
-// Check if directory parameter is provided
-if (!options.directory) {
-  console.error('Error: The --directory (-d) parameter is required to specify the folder for generated reports.');
-  console.error('Example usage:');
-  console.error('  report_gen_mcp --directory /path/to/reports');
-  console.error('  report_gen_mcp --directory ./reports start-server');
-  console.error('  report_gen_mcp --directory ./reports generate --document doc.md --charts charts.json');
-  process.exit(1);
-}
-
-// Validate that the directory exists or can be created
-let reportsDir: string;
-try {
-  reportsDir = path.resolve(options.directory);
-  fs.ensureDirSync(reportsDir);
-} catch (error) {
-  console.error(`Error: Cannot access or create directory "${options.directory}"`);
-  process.exit(1);
-}
-
-// Use the reports directory as the base directory for all operations
-let homeDir = reportsDir;
-
 app.post('/generate-report', async (req, res) => {
+  // Check if tempDirectory parameter is provided
   const { document, charts, outputFile = 'report.html', tempDirectory } = req.body;
-  const effectiveTempDir = tempDirectory || homeDir;
-  const outputPath = path.resolve(effectiveTempDir, outputFile);
+
+  if (!tempDirectory) {
+    return res.status(400).json({
+      error: 'tempDirectory parameter is required. Please provide the directory where reports should be generated.',
+      example: 'curl -X POST http://localhost:3000/generate-report -H "Content-Type: application/json" -d \'{"document": "# Report", "charts": {}, "tempDirectory": "/path/to/reports"}\''
+    });
+  }
+
+  const outputPath = path.resolve(tempDirectory, outputFile);
 
   try {
     const result = await generateReport(document, charts, outputPath);
@@ -74,13 +52,21 @@ program
   });
 
 program
-  .command('generate')
+  .command('generate <directory>')
   .option('--document <md>', 'Markdown document with placeholders [[chart:id]]')
   .option('--charts <json>', 'JSON string of charts {id: {type: "bar", config: {...}}}')
   .option('--output <file>', 'Output HTML file', 'report.html')
-  .action(async (opts) => {
+  .description('Generate a report in the specified directory')
+  .action(async (directory, opts) => {
+    // Check if directory parameter is provided
+    if (!directory) {
+      console.error('Error: Directory parameter is required. Please specify the directory where reports should be generated.');
+      console.error('Example: npx @vint.tri/report_gen_mcp@latest generate /path/to/reports --document "# Report" --charts \'{}\'');
+      process.exit(1);
+    }
+
     const charts = JSON.parse(opts.charts);
-    const result = await generateReport(opts.document, charts, opts.output);
+    const result = await generateReport(opts.document, charts, path.resolve(directory, opts.output));
     console.log(result);
   });
 
@@ -134,7 +120,12 @@ if (process.argv.length === 2) {
       processedParams = params.arguments;
     }
 
-    const { document, charts, outputFile = 'report.html', tempDirectory = process.cwd() } = processedParams;
+    // Check if tempDirectory parameter is provided
+    if (!processedParams.tempDirectory) {
+      throw new Error('tempDirectory parameter is required. Please provide the directory where reports should be generated.');
+    }
+
+    const { document, charts, outputFile = 'report.html', tempDirectory } = processedParams;
     const outputPath = path.resolve(tempDirectory, outputFile);
 
     try {