pretext-pdf-mcp 1.0.7 → 1.0.8

package/CHANGELOG.md

@@ -1,10 +1,26 @@
 # Changelog
 
+<!-- markdownlint-disable MD024 -->
+
 All notable changes to pretext-pdf-mcp are documented here.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ---
 
+## [1.0.8] — 2026-04-13
+
+### Fixed
+
+- **Critical: /mcp endpoint crashed on malformed JSON** — Missing try-catch around `JSON.parse()` caused unhandled exceptions. Now returns HTTP 400 with `{error: "Invalid JSON body"}`, consistent with `/api/generate`.
+- **Critical: generate_pdf error response was plain text** — Validation error returned `{type: 'text', text: 'Error: ...'}` instead of JSON, breaking clients that expected `{success, error, message}`. Now consistent with all other tools.
+- **High: isClientError() misclassified internal errors as 400** — Non-PretextPdfError exceptions (TypeError, RangeError, etc.) were mapped to HTTP 400. Now correctly returns 500 for unexpected server errors; 400 only for known client-caused PretextPdfErrors.
+- **High: PORT env var NaN on invalid input** — `parseInt("abc")` returns NaN, causing server to bind to a random port silently. Now exits with a clear error message.
+- **Medium: GST calculation floating-point drift** — Accumulated rounding errors in per-rate GST totals (e.g. `18% of ₹250,000 = ₹45,000.00000000001`). All GST amounts now rounded to 2 decimal places at each step.
+- **Low: Dead `columns` variable in generate-report.ts** — Unused intermediate `columns` array was computed and suppressed with `void columns`. Removed entirely.
+- **Low: smithery.yaml version was stale** — Showed `1.0.1` instead of matching package.json. Now tracks current version.
+
+---
+
 ## [1.0.7] — 2026-04-13
 
 ### Fixed

package/dist/index.js

@@ -26,7 +26,7 @@ function validatePdfDocumentInput(data) {
  */
 function isClientError(err) {
     if (!(err instanceof PretextPdfError))
-        return true; // Unknown errors → client error by default
+        return false; // Unknown errors → server error (500)
     const clientErrors = [
         'VALIDATION_ERROR',
         'IMAGE_LOAD_FAILED',
@@ -41,7 +41,7 @@ function isClientError(err) {
     return clientErrors.includes(err.code);
 }
 function createServer() {
-    const server = new Server({ name: 'pretext-pdf', version: '1.0.7' }, { capabilities: { tools: {} } });
+    const server = new Server({ name: 'pretext-pdf', version: '1.0.8' }, { capabilities: { tools: {} } });
     const tools = [generatePdfTool, generateInvoiceTool, generateReportTool, listElementsTool];
     server.setRequestHandler(ListToolsRequestSchema, async () => ({
         tools: tools.map(t => t.schema),
@@ -63,7 +63,12 @@ function setCorsHeaders(res) {
     res.setHeader('Access-Control-Allow-Methods', 'POST, OPTIONS');
     res.setHeader('Access-Control-Allow-Headers', 'Content-Type');
 }
-const port = process.env.PORT ? parseInt(process.env.PORT, 10) : null;
+const rawPort = process.env.PORT;
+const port = rawPort ? parseInt(rawPort, 10) : null;
+if (port !== null && isNaN(port)) {
+    process.stderr.write(`[pretext-pdf-mcp] Error: PORT="${rawPort}" is not a valid port number\n`);
+    process.exit(1);
+}
 if (port) {
     const { createServer: createHttpServer } = await import('node:http');
     const { StreamableHTTPServerTransport } = await import('@modelcontextprotocol/sdk/server/streamableHttp.js');
@@ -145,7 +150,15 @@ if (port) {
                 }
                 chunks.push(chunk);
             }
-            const body = JSON.parse(Buffer.concat(chunks).toString());
+            let body;
+            try {
+                body = JSON.parse(Buffer.concat(chunks).toString());
+            }
+            catch {
+                res.writeHead(400, { 'Content-Type': 'application/json' });
+                res.end(JSON.stringify({ error: 'Invalid JSON body' }));
+                return;
+            }
             const transport = new StreamableHTTPServerTransport({
                 sessionIdGenerator: undefined,
             });

package/dist/tools/generate-invoice.js

@@ -70,7 +70,7 @@ function buildInvoiceDocument(input) {
             return sum + (amount * rate) / 100;
         }, 0)
         : 0;
-    const grandTotal = subtotal + totalGst;
+    const grandTotal = Math.round((subtotal + totalGst) * 100) / 100;
     // Build totals section
     const totalsContent = [
         { type: 'hr', color: '#dddddd', thickness: 0.5, spaceBelow: 6 },
@@ -88,7 +88,8 @@ function buildInvoiceDocument(input) {
             if (!item.gst_rate)
                 continue;
             const amount = item.quantity * item.rate;
-            rateGroups[item.gst_rate] = (rateGroups[item.gst_rate] ?? 0) + (amount * item.gst_rate) / 100;
+            const gst = Math.round((amount * item.gst_rate) / 100 * 100) / 100;
+            rateGroups[item.gst_rate] = Math.round(((rateGroups[item.gst_rate] ?? 0) + gst) * 100) / 100;
         }
         for (const [rate, gstAmt] of Object.entries(rateGroups)) {
             totalsContent.push({

package/dist/tools/generate-pdf.js

@@ -24,7 +24,7 @@ export const generatePdfTool = {
         try {
             if (!args.document || typeof args.document !== 'object') {
                 return {
-                    content: [{ type: 'text', text: 'Error: document is required and must be an object' }],
+                    content: [{ type: 'text', text: JSON.stringify({ success: false, error: 'VALIDATION_ERROR', message: 'document is required and must be an object' }) }],
                     isError: true,
                 };
             }

package/dist/tools/generate-report.js

@@ -85,8 +85,6 @@ function buildReportDocument(input) {
         }
         if (section.table) {
             const { headers, rows } = section.table;
-            const columns = headers.map(() => ({ width: `${Math.floor(100 / headers.length)}%` }));
-            // Use equal fractional widths
             const fracColumns = headers.map(() => ({ width: '1*', align: 'left' }));
             const headerRow = {
                 isHeader: true,
@@ -106,8 +104,6 @@ function buildReportDocument(input) {
                 cellPaddingV: 6,
                 spaceAfter: 12,
             });
-            // Suppress unused variable warning
-            void columns;
         }
         if (section.callout) {
             const borderColor = CALLOUT_COLORS[section.callout.style] ?? '#888888';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pretext-pdf-mcp",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "description": "MCP server for pretext-pdf — generate professional PDFs from JSON in Claude, Cursor, or any AI agent",
   "keywords": [
     "mcp",

package/smithery.yaml

@@ -1,6 +1,6 @@
 name: pretext-pdf
 description: "Generate professional PDFs from structured JSON. Supports invoices (with GST), reports, tables, forms, encryption, and more. No headless browser — pure Node.js."
-version: "1.0.1"
+version: "1.0.8"
 startCommand:
   type: stdio
   command: npx