invar-tools 1.15.2__tar.gz → 1.15.4__tar.gz

{invar_tools-1.15.2 → invar_tools-1.15.4}/CHANGELOG.md

@@ -9,6 +9,32 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+## [1.15.4] - 2026-01-03
+
+### Fixed
+- **Pi Custom Tools: Error Message Capture** - Improved error diagnostics
+  - Enhanced error message capture to include both stderr and stdout
+  - Prevents empty error messages like "Failed to extract TOC: "
+  - Fallback to "Unknown error" if both streams are empty
+  - **Solves diagnostic issue** where error details were missing:
+    - Root cause: Some errors may output to stdout instead of stderr
+    - Pi's exec() stderr might be empty even when command fails
+  - All 9 tools now provide clear error messages for debugging
+  - Fixed in: invar_sig, invar_map, invar_doc_toc, invar_doc_read, invar_doc_find, invar_doc_replace, invar_doc_insert, invar_doc_delete
+
+## [1.15.3] - 2026-01-03
+
+### Fixed
+- **Pi Custom Tools: Parameter Handling** - Fixed parameter validation and default values
+  - Added validation for all required parameters (file, target, section, etc.)
+  - Added explicit default value handling using `??` operator for optional parameters
+  - Prevents tools from failing when Pi doesn't pass parameter values
+  - **Solves critical issue** where tools failed with missing parameters:
+    - Root cause: TypeBox default values not automatically populated in params object
+    - Pi may call tools without passing values, even for required parameters
+  - All 9 tools now validate inputs and handle defaults correctly
+  - Clear error messages guide LLM to provide required parameters
+
 
 ## [1.15.2] - 2026-01-03
 
@@ -365,7 +391,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
-[Unreleased]: https://github.com/yourusername/invar/compare/v1.15.2...HEAD
+[Unreleased]: https://github.com/yourusername/invar/compare/v1.15.4...HEAD
+[1.15.4]: https://github.com/yourusername/invar/compare/v1.15.3...v1.15.4
+[1.15.3]: https://github.com/yourusername/invar/compare/v1.15.2...v1.15.3
 [1.15.2]: https://github.com/yourusername/invar/compare/v1.15.1...v1.15.2
 [1.15.1]: https://github.com/yourusername/invar/compare/v1.15.0...v1.15.1
 [1.15.0]: https://github.com/yourusername/invar/compare/v1.14.0...v1.15.0

{invar_tools-1.15.2 → invar_tools-1.15.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: invar-tools
-Version: 1.15.2
+Version: 1.15.4
 Summary: AI-native software engineering tools with design-by-contract verification
 Project-URL: Homepage, https://github.com/tefx/invar
 Project-URL: Documentation, https://github.com/tefx/invar#readme

{invar_tools-1.15.2 → invar_tools-1.15.4}/pyproject.toml

@@ -14,7 +14,7 @@ artifacts = ["src/invar/node_tools/**/*.js", "src/invar/node_tools/**/MANIFEST"]
 
 [project]
 name = "invar-tools"
-version = "1.15.2"
+version = "1.15.4"
 description = "AI-native software engineering tools with design-by-contract verification"
 readme = "README.md"
 license = "GPL-3.0-or-later"

{invar_tools-1.15.2 → invar_tools-1.15.4}/src/invar/templates/pi-tools/invar/index.ts

@@ -69,18 +69,24 @@ const factory: CustomToolFactory = (pi) => {
         const cmd = await resolveInvarCommand();
         const args = [...cmd.args, "guard"];
 
+        // Handle optional parameters with defaults
+        const changed = params.changed ?? true;
+        const contractsOnly = params.contracts_only ?? false;
+        const coverage = params.coverage ?? false;
+        const strict = params.strict ?? false;
+
         // Default is --changed (check modified files)
-        if (params.changed === false) {
+        if (changed === false) {
           args.push("--all");
         }
 
-        if (params.contracts_only) {
+        if (contractsOnly) {
           args.push("-c");
         }
-        if (params.coverage) {
+        if (coverage) {
           args.push("--coverage");
         }
-        if (params.strict) {
+        if (strict) {
           args.push("--strict");
         }
 
@@ -117,6 +123,11 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameter
+        if (!params.target) {
+          throw new Error("Missing required parameter: target (file path or file::symbol)");
+        }
+
         if (!isValidPath(params.target)) {
           throw new Error("Invalid target path: contains unsafe characters or path traversal");
         }
@@ -131,7 +142,8 @@ const factory: CustomToolFactory = (pi) => {
         }
 
         if (result.exitCode !== 0) {
-          throw new Error(`Failed to get signatures: ${result.stderr}`);
+          const errorMsg = result.stderr || result.stdout || "Unknown error";
+          throw new Error(`Failed to get signatures: ${errorMsg}`);
         }
 
         return {
@@ -163,19 +175,21 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
-        if (params.path && params.path !== "." && !isValidPath(params.path)) {
+        // Handle optional parameters with defaults
+        const targetPath = params.path ?? ".";
+        const topN = params.top ?? 10;
+
+        if (targetPath !== "." && !isValidPath(targetPath)) {
           throw new Error("Invalid path: contains unsafe characters or path traversal");
         }
 
         const args = [...cmd.args, "map"];
 
-        if (params.path && params.path !== ".") {
-          args.push(params.path);
+        if (targetPath !== ".") {
+          args.push(targetPath);
         }
 
-        if (params.top) {
-          args.push("--top", params.top.toString());
-        }
+        args.push("--top", topN.toString());
 
         const result = await pi.exec(cmd.command, args, {
           cwd: pi.cwd,
@@ -187,14 +201,15 @@ const factory: CustomToolFactory = (pi) => {
         }
 
         if (result.exitCode !== 0) {
-          throw new Error(`Failed to generate map: ${result.stderr}`);
+          const errorMsg = result.stderr || result.stdout || "Unknown error";
+          throw new Error(`Failed to generate map: ${errorMsg}`);
         }
 
         return {
           content: [{ type: "text", text: result.stdout }],
           details: {
-            path: params.path || ".",
-            top: params.top || 10,
+            path: targetPath,
+            top: topN,
           },
         };
       },
@@ -221,14 +236,22 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameter
+        if (!params.file) {
+          throw new Error("Missing required parameter: file (markdown file path)");
+        }
+
         if (!isValidPath(params.file)) {
           throw new Error("Invalid file path: contains unsafe characters or path traversal");
         }
 
+        // Handle optional parameter with default
+        const maxDepth = params.depth ?? 6;
+
         const args = [...cmd.args, "doc", "toc", params.file];
 
-        if (params.depth && params.depth !== 6) {
-          args.push("--depth", params.depth.toString());
+        if (maxDepth !== 6) {
+          args.push("--depth", maxDepth.toString());
         }
 
         const result = await pi.exec(cmd.command, args, {
@@ -241,14 +264,15 @@ const factory: CustomToolFactory = (pi) => {
         }
 
         if (result.exitCode !== 0) {
-          throw new Error(`Failed to extract TOC: ${result.stderr}`);
+          const errorMsg = result.stderr || result.stdout || "Unknown error";
+          throw new Error(`Failed to extract TOC: ${errorMsg}`);
         }
 
         return {
           content: [{ type: "text", text: result.stdout }],
           details: {
             file: params.file,
-            depth: params.depth || 6,
+            depth: maxDepth,
           },
         };
       },
@@ -276,13 +300,24 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameters
+        if (!params.file) {
+          throw new Error("Missing required parameter: file (markdown file path)");
+        }
+        if (!params.section) {
+          throw new Error("Missing required parameter: section (section path or identifier)");
+        }
+
         if (!isValidPath(params.file) || !isValidPath(params.section)) {
           throw new Error("Invalid file or section path: contains unsafe characters or path traversal");
         }
 
+        // Handle optional parameter with default
+        const includeChildren = params.children ?? true;
+
         const args = [...cmd.args, "doc", "read", params.file, params.section, "--json"];
 
-        if (params.children === false) {
+        if (includeChildren === false) {
           args.push("--no-children");
         }
 
@@ -296,7 +331,8 @@ const factory: CustomToolFactory = (pi) => {
         }
 
         if (result.exitCode !== 0) {
-          throw new Error(`Failed to read section: ${result.stderr}`);
+          const errorMsg = result.stderr || result.stdout || "Unknown error";
+          throw new Error(`Failed to read section: ${errorMsg}`);
         }
 
         return {
@@ -335,12 +371,21 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameters
+        if (!params.file) {
+          throw new Error("Missing required parameter: file (markdown file path)");
+        }
+        if (!params.pattern) {
+          throw new Error("Missing required parameter: pattern (glob pattern for section titles)");
+        }
+
         if (!isValidPath(params.file)) {
           throw new Error("Invalid file path: contains unsafe characters or path traversal");
         }
 
         const args = [...cmd.args, "doc", "find", params.pattern, params.file, "--json"];
 
+        // Handle optional parameters
         if (params.content) {
           args.push("--content", params.content);
         }
@@ -359,7 +404,8 @@ const factory: CustomToolFactory = (pi) => {
         }
 
         if (result.exitCode !== 0) {
-          throw new Error(`Failed to find sections: ${result.stderr}`);
+          const errorMsg = result.stderr || result.stdout || "Unknown error";
+          throw new Error(`Failed to find sections: ${errorMsg}`);
         }
 
         return {
@@ -397,10 +443,24 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameters
+        if (!params.file) {
+          throw new Error("Missing required parameter: file (markdown file path)");
+        }
+        if (!params.section) {
+          throw new Error("Missing required parameter: section (section path or identifier)");
+        }
+        if (!params.content) {
+          throw new Error("Missing required parameter: content (new content for section)");
+        }
+
         if (!isValidPath(params.file) || !isValidPath(params.section)) {
           throw new Error("Invalid file or section path: contains unsafe characters or path traversal");
         }
 
+        // Handle optional parameter with default
+        const keepHeading = params.keep_heading ?? true;
+
         // Write content to temporary file to avoid shell injection
         const tmpFile = path.join(pi.cwd, `.invar-tmp-${Date.now()}.txt`);
 
@@ -417,7 +477,7 @@ const factory: CustomToolFactory = (pi) => {
             tmpFile,
           ];
 
-          if (params.keep_heading === false) {
+          if (keepHeading === false) {
             args.push("--no-keep-heading");
           }
 
@@ -431,7 +491,8 @@ const factory: CustomToolFactory = (pi) => {
           }
 
           if (result.exitCode !== 0) {
-            throw new Error(`Failed to replace section: ${result.stderr}`);
+            const errorMsg = result.stderr || result.stdout || "Unknown error";
+            throw new Error(`Failed to replace section: ${errorMsg}`);
           }
 
           return {
@@ -478,10 +539,24 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameters
+        if (!params.file) {
+          throw new Error("Missing required parameter: file (markdown file path)");
+        }
+        if (!params.anchor) {
+          throw new Error("Missing required parameter: anchor (section path for insertion point)");
+        }
+        if (!params.content) {
+          throw new Error("Missing required parameter: content (content to insert)");
+        }
+
         if (!isValidPath(params.file) || !isValidPath(params.anchor)) {
           throw new Error("Invalid file or anchor path: contains unsafe characters or path traversal");
         }
 
+        // Handle optional parameter with default
+        const position = params.position ?? "after";
+
         // Write content to temporary file
         const tmpFile = path.join(pi.cwd, `.invar-tmp-${Date.now()}.txt`);
 
@@ -498,8 +573,8 @@ const factory: CustomToolFactory = (pi) => {
             tmpFile,
           ];
 
-          if (params.position && params.position !== "after") {
-            args.push("--position", params.position);
+          if (position !== "after") {
+            args.push("--position", position);
           }
 
           const result = await pi.exec(cmd.command, args, {
@@ -512,7 +587,8 @@ const factory: CustomToolFactory = (pi) => {
           }
 
           if (result.exitCode !== 0) {
-            throw new Error(`Failed to insert content: ${result.stderr}`);
+            const errorMsg = result.stderr || result.stdout || "Unknown error";
+            throw new Error(`Failed to insert content: ${errorMsg}`);
           }
 
           return {
@@ -556,13 +632,24 @@ const factory: CustomToolFactory = (pi) => {
       async execute(toolCallId, params, onUpdate, ctx, signal) {
         const cmd = await resolveInvarCommand();
 
+        // Validate required parameters
+        if (!params.file) {
+          throw new Error("Missing required parameter: file (markdown file path)");
+        }
+        if (!params.section) {
+          throw new Error("Missing required parameter: section (section path or identifier)");
+        }
+
         if (!isValidPath(params.file) || !isValidPath(params.section)) {
           throw new Error("Invalid file or section path: contains unsafe characters or path traversal");
         }
 
+        // Handle optional parameter with default
+        const includeChildren = params.children ?? true;
+
         const args = [...cmd.args, "doc", "delete", params.file, params.section];
 
-        if (params.children === false) {
+        if (includeChildren === false) {
           args.push("--no-children");
         }
 
@@ -576,7 +663,8 @@ const factory: CustomToolFactory = (pi) => {
         }
 
         if (result.exitCode !== 0) {
-          throw new Error(`Failed to delete section: ${result.stderr}`);
+          const errorMsg = result.stderr || result.stdout || "Unknown error";
+          throw new Error(`Failed to delete section: ${errorMsg}`);
         }
 
         return {