janito 2.17.0__tar.gz → 2.17.1__tar.gz

{janito-2.17.0 → janito-2.17.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: janito
-Version: 2.17.0
+Version: 2.17.1
 Summary: A new Python package called janito.
 Author-email: João Pinto <janito@ikignosis.org>
 Project-URL: Homepage, https://github.com/ikignosis/janito

{janito-2.17.0 → janito-2.17.1}/janito/tools/adapters/local/read_files.py

@@ -52,4 +52,4 @@ class ReadFilesTool(ToolBase):
                 results.append(
                     f"--- File: {disp_path} (error) ---\nError reading file: {e}\n"
                 )
-        return "\n".join(results)
+        return "\n".join(results)

{janito-2.17.0 → janito-2.17.1}/janito/tools/path_security.py

@@ -135,6 +135,7 @@ def _extract_path_keys_from_schema(schema: Mapping[str, Any]) -> set[str]:
     path_keys: set[str] = set()
     if schema is not None:
         for k, v in schema.get("properties", {}).items():
+            # Handle direct path strings
             if v.get("format") == "path" or (
                 v.get("type") == "string"
                 and (
@@ -144,6 +145,19 @@ def _extract_path_keys_from_schema(schema: Mapping[str, Any]) -> set[str]:
                 )
             ):
                 path_keys.add(k)
+            # Handle arrays of path strings
+            elif v.get("type") == "array" and "items" in v:
+                items = v["items"]
+                if items.get("format") == "path" or (
+                    items.get("type") == "string"
+                    and (
+                        "path" in items.get("description", "").lower()
+                        or "path" in v.get("description", "").lower()
+                        or k.endswith("paths")
+                        or k == "paths"
+                    )
+                ):
+                    path_keys.add(k)
     return path_keys
 
 

{janito-2.17.0 → janito-2.17.1}/janito/tools/tools_adapter.py

@@ -131,9 +131,15 @@ class ToolsAdapterBase:
 
         if arguments is None:
             arguments = {}
-        # Ensure the input is a dict to avoid breaking the inspect-based logic
+        # If arguments are provided as a non-dict (e.g. a list or a scalar)
+        # we skip signature *keyword* validation completely and defer the
+        # decision to Python's own call mechanics when the function is
+        # eventually invoked.  This allows positional / variadic arguments to
+        # be supplied by callers that intentionally bypass the structured
+        # JSON-schema style interface.
         if not isinstance(arguments, dict):
-            return "Tool arguments should be provided as an object / mapping"
+            # Nothing to validate at this stage – treat as OK.
+            return None
 
         sig = inspect.signature(func)
         params = sig.parameters
@@ -193,30 +199,32 @@ class ToolsAdapterBase:
             )
 
             schema = getattr(tool, "schema", None)
-            try:
-                validate_paths_in_arguments(arguments, workdir, schema=schema)
-            except PathSecurityError as sec_err:
-                # Publish both a ToolCallError and a user-facing ReportEvent for path security errors
-                self._publish_tool_call_error(
-                    tool_name, request_id, str(sec_err), arguments
-                )
-                if self._event_bus:
-                    from janito.report_events import (
-                        ReportEvent,
-                        ReportSubtype,
-                        ReportAction,
+            # Only validate paths for dictionary-style arguments
+            if isinstance(arguments, dict):
+                try:
+                    validate_paths_in_arguments(arguments, workdir, schema=schema)
+                except PathSecurityError as sec_err:
+                    # Publish both a ToolCallError and a user-facing ReportEvent for path security errors
+                    self._publish_tool_call_error(
+                        tool_name, request_id, str(sec_err), arguments
                     )
+                    if self._event_bus:
+                        from janito.report_events import (
+                            ReportEvent,
+                            ReportSubtype,
+                            ReportAction,
+                        )
 
-                    self._event_bus.publish(
-                        ReportEvent(
-                            subtype=ReportSubtype.ERROR,
-                            message=f"[SECURITY] Path access denied: {sec_err}",
-                            action=ReportAction.EXECUTE,
-                            tool=tool_name,
-                            context={"arguments": arguments, "request_id": request_id},
+                        self._event_bus.publish(
+                            ReportEvent(
+                                subtype=ReportSubtype.ERROR,
+                                message=f"[SECURITY] Path access denied: {sec_err}",
+                                action=ReportAction.EXECUTE,
+                                tool=tool_name,
+                                context={"arguments": arguments, "request_id": request_id},
+                            )
                         )
-                    )
-                return f"Security error: {sec_err}"
+                    return f"Security error: {sec_err}"
         # --- END SECURITY ---
 
         self._publish_tool_call_started(tool_name, request_id, arguments)
@@ -224,7 +232,18 @@ class ToolsAdapterBase:
             f"[tools-adapter] Executing tool: {tool_name} with arguments: {arguments}"
         )
         try:
-            result = self.execute(tool, **(arguments or {}), **kwargs)
+            # Normalize arguments to ensure proper type handling
+            normalized_args = self._normalize_arguments(arguments, tool, func)
+            
+            if isinstance(normalized_args, (list, tuple)):
+                # Positional arguments supplied as an array → expand as *args
+                result = self.execute(tool, *normalized_args, **kwargs)
+            elif isinstance(normalized_args, dict) or normalized_args is None:
+                # Keyword-style arguments (the default) – pass as **kwargs
+                result = self.execute(tool, **(normalized_args or {}), **kwargs)
+            else:
+                # Single positional argument (scalar/str/int/…)
+                result = self.execute(tool, normalized_args, **kwargs)
         except Exception as e:
             self._handle_execution_error(tool_name, request_id, e, arguments)
         self._print_verbose(
@@ -281,6 +300,65 @@ class ToolsAdapterBase:
         if self.verbose_tools:
             print(message)
 
+    def _normalize_arguments(self, arguments, tool, func):
+        """
+        Normalize arguments to ensure proper type handling at the adapter level.
+        
+        This handles cases where:
+        1. String is passed instead of list for array parameters
+        2. JSON string parsing issues
+        3. Other type mismatches that can be automatically resolved
+        """
+        import inspect
+        import json
+        
+        # If arguments is already a dict or None, return as-is
+        if isinstance(arguments, dict) or arguments is None:
+            return arguments
+            
+        # If arguments is a list/tuple, return as-is (positional args)
+        if isinstance(arguments, (list, tuple)):
+            return arguments
+            
+        # Handle string arguments
+        if isinstance(arguments, str):
+            # Try to parse as JSON if it looks like JSON
+            stripped = arguments.strip()
+            if (stripped.startswith('{') and stripped.endswith('}')) or \
+               (stripped.startswith('[') and stripped.endswith(']')):
+                try:
+                    parsed = json.loads(arguments)
+                    return parsed
+                except json.JSONDecodeError:
+                    # If it looks like JSON but failed, try to handle common issues
+                    pass
+            
+            # Check if the function expects a list parameter
+            try:
+                sig = inspect.signature(func)
+                params = list(sig.parameters.values())
+                
+                # Skip 'self' parameter for methods
+                if len(params) > 0 and params[0].name == 'self':
+                    params = params[1:]
+                
+                # If there's exactly one parameter that expects a list, wrap string in list
+                if len(params) == 1:
+                    param = params[0]
+                    annotation = param.annotation
+                    
+                    # Check if annotation is list[str] or similar
+                    if hasattr(annotation, '__origin__') and annotation.__origin__ is list:
+                        return [arguments]
+                    elif str(annotation).startswith('list[') or str(annotation) == 'list':
+                        return [arguments]
+                        
+            except (ValueError, TypeError):
+                pass
+                
+        # Return original arguments for other cases
+        return arguments
+
     def execute_function_call_message_part(self, function_call_message_part):
         """
         Execute a FunctionCallMessagePart by extracting the tool name and arguments and dispatching to execute_by_name.
@@ -298,9 +376,23 @@ class ToolsAdapterBase:
         # Parse arguments if they are a JSON string
         if isinstance(arguments, str):
             try:
+                # Try to parse as JSON first
                 arguments = json.loads(arguments)
-            except Exception:
-                pass  # Leave as string if not JSON
+            except json.JSONDecodeError:
+                # Handle single quotes in JSON strings
+                try:
+                    # Replace single quotes with double quotes for JSON compatibility
+                    fixed_json = arguments.replace("'", '"')
+                    arguments = json.loads(fixed_json)
+                except (json.JSONDecodeError, ValueError):
+                    # If it's a string that looks like it might be a single path parameter,
+                    # try to handle it gracefully
+                    if arguments.startswith("{") and arguments.endswith("}"):
+                        # Looks like JSON but failed to parse - this is likely an error
+                        pass
+                    else:
+                        # Single string argument - let the normalization handle it
+                        pass
         if self.verbose_tools:
             print(
                 f"[tools-adapter] Executing FunctionCallMessagePart: tool={tool_name}, arguments={arguments}, tool_call_id={tool_call_id}"

{janito-2.17.0 → janito-2.17.1}/janito.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: janito
-Version: 2.17.0
+Version: 2.17.1
 Summary: A new Python package called janito.
 Author-email: João Pinto <janito@ikignosis.org>
 Project-URL: Homepage, https://github.com/ikignosis/janito

{janito-2.17.0 → janito-2.17.1}/janito.egg-info/SOURCES.txt

@@ -30,7 +30,6 @@ docs/driver-flow.md
 docs/driver-request-cancellation.md
 docs/drivers.md
 docs/event-bus.md
-docs/index.md
 docs/llm-drivers-required-config.md
 docs/llm-drivers.md
 docs/moonshotai-setup.md

{janito-2.17.0 → janito-2.17.1}/tests/adapters/local/test_read_files.py

@@ -16,7 +16,7 @@ def test_read_files_tool(tmp_path, read_files_tool):
     file2.write_text("Second file contents.\n")
     missing_file = tmp_path / "missing.txt"
 
-    result = read_files_tool.run([str(file1), str(file2), str(missing_file)])
+    result = read_files_tool.run(paths=[str(file1), str(file2), str(missing_file)])
 
     assert "Hello from file 1!" in result
     assert "Second file contents." in result

janito-2.17.0/docs/index.md

@@ -1,26 +0,0 @@
-<p align="center">
-  <img src="imgs/happy-programmer.svg" alt="Janito Logo" width="180"/>
-</p>
-
-Janito is a lightweight language model client focused on software development activities, enabling automation of programming and project tasks through natural language using LLMs.
-
-## ✨ Key Features
-
--  ⚡ Automate code review, refactoring, and documentation
--  💬 Interactive pair programming chat mode
--  🖥️ Use via command-line with an optional web interface for complex manual edits
--  🔌 Easily extend with custom tools and plugins
--  🛡️ Robust configuration and quality checks
--  👨‍💻 Designed for developers, by developers
-
-
-- - -
-
-## 📚 Learn More
-
-- [What Is a Language Model Client?](concepts/language-model-clients.md) — Thin vs. Thick Clients explained
-- [Configuring Janito for DeepSeek](deepseek-setup.md) — How to set up and use DeepSeek models
-- [Configuring Janito for MoonshotAI](moonshotai-setup.md) — How to set up and use MoonshotAI models
-- [Configuring Janito for Z.ai](z-ai-setup.md) — How to set up and use Z.ai models
-- **Concepts & Terminology:** See the [concepts](concepts/index.md) section for more foundational explanations.
-