signalwire-sdk 3.0.2.dev62__tar.gz → 3.0.2.dev64__tar.gz

{signalwire_sdk-3.0.2.dev62/signalwire/signalwire_sdk.egg-info → signalwire_sdk-3.0.2.dev64}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: signalwire-sdk
-Version: 3.0.2.dev62
+Version: 3.0.2.dev64
 Summary: SignalWire SDK
 Author-email: SignalWire Team <info@signalwire.com>
 License: MIT

{signalwire_sdk-3.0.2.dev62 → signalwire_sdk-3.0.2.dev64}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "signalwire-sdk"
-version = "3.0.2.dev62"
+version = "3.0.2.dev64"
 description = "SignalWire SDK"
 authors = [
     {name = "SignalWire Team", email = "info@signalwire.com"}
@@ -165,3 +165,51 @@ signalwire = [
 [tool.ruff]
 target-version = "py310"
 line-length = 88
+
+[tool.ruff.lint]
+# E4/E7/E9 + F are the ruff defaults (E501 line-length intentionally excluded —
+# formatting is owned by `ruff format`, not the linter). B = flake8-bugbear
+# (likely-bug patterns), S = flake8-bandit (security smells).
+select = ["E4", "E7", "E9", "F", "B", "S", "C4", "PERF", "SIM", "PTH", "RET", "RUF"]
+
+[tool.ruff.lint.per-file-ignores]
+# Tests are built on assert; bandit's S101 is noise there.
+"tests/**" = ["S101"]
+# sw-agent-dokku is a developer-run deploy CLI that shells out to git/ssh with
+# fixed list-form arg vectors (shell=False, so no shell interpolation). The
+# dynamic parts (app name, dokku host, config vars) are the operator's own CLI
+# args for their own deployment — not untrusted server input. S603 (subprocess
+# call) and S607 (partial executable path: git/ssh resolved via PATH, the normal
+# idiom) are expected and safe across this file.
+"signalwire/signalwire/cli/dokku.py" = ["S603", "S607"]
+
+[tool.ruff.lint.flake8-bugbear]
+# FastAPI's dependency-injection idiom puts Depends()/Query()/etc. in argument
+# defaults on purpose; B008 (function-call-in-default) is a false positive for it.
+# Both the bare (`from fastapi import Depends`) and qualified forms are listed
+# because the code imports these names directly.
+extend-immutable-calls = [
+    "Depends", "fastapi.Depends",
+    "Query", "fastapi.Query",
+    "Body", "fastapi.Body",
+    "Header", "fastapi.Header",
+    "Cookie", "fastapi.Cookie",
+    "Path", "fastapi.Path",
+    "Form", "fastapi.Form",
+    "File", "fastapi.File",
+    "Security", "fastapi.Security",
+]
+
+[tool.mypy]
+python_version = "3.10"
+files = ["signalwire/signalwire"]
+# Third-party packages without type stubs (nltk, sentence_transformers, the
+# search/relay stacks) are out of scope — we type our own code, not theirs.
+ignore_missing_imports = true
+# The bar: our own code is clean. Bodies of untyped defs are checked so real
+# type errors inside them surface (this is what found the mixin cluster).
+check_untyped_defs = true
+warn_unused_ignores = true
+# `# type: ignore` is reserved for genuinely-unfixable third-party-stub gaps and
+# must carry an error code + reason. warn_unused_ignores keeps them honest.
+show_error_codes = true

{signalwire_sdk-3.0.2.dev62 → signalwire_sdk-3.0.2.dev64}/signalwire/signalwire/__init__.py

@@ -17,7 +17,7 @@ from .core.logging_config import configure_logging
 
 configure_logging()
 
-__version__ = "3.0.2.dev62"
+__version__ = "3.0.2.dev64"
 
 # Import core classes for easier access.
 # These imports are intentionally placed after configure_logging() so the SDK's
@@ -148,26 +148,26 @@ def RestClient(*args, **kwargs):
 __all__ = [
     "AgentBase",
     "AgentServer",
-    "SWMLService",
-    "SWMLBuilder",
-    "FunctionResult",
-    "SWAIGFunction",
-    "DataMap",
-    "create_simple_api_tool",
-    "create_expression_tool",
-    "ContextBuilder",
+    "BedrockAgent",
     "Context",
-    "Step",
+    "ContextBuilder",
+    "DataMap",
+    "FunctionResult",
     "GatherInfo",
     "GatherQuestion",
-    "create_simple_context",
-    "SwaigFunctionResult",
+    "RestClient",
+    "SWAIGFunction",
+    "SWMLBuilder",
+    "SWMLService",
     "SchemaValidationError",
+    "Step",
+    "SwaigFunctionResult",
     "WebService",
+    "add_skill_directory",
+    "create_expression_tool",
+    "create_simple_api_tool",
+    "create_simple_context",
     "list_skills",
     "list_skills_with_params",
     "register_skill",
-    "add_skill_directory",
-    "BedrockAgent",
-    "RestClient",
 ]

{signalwire_sdk-3.0.2.dev62 → signalwire_sdk-3.0.2.dev64}/signalwire/signalwire/agent_server.py

@@ -11,6 +11,7 @@ AgentServer - Class for hosting multiple SignalWire AI Agents in a single server
 
 import os
 import re
+from pathlib import Path
 from typing import Dict, Any, Optional, List, Tuple, Callable
 
 try:
@@ -19,11 +20,12 @@ try:
 except ImportError:
     raise ImportError(
         "fastapi and uvicorn are required. Install them with: pip install fastapi uvicorn"
-    )
+    ) from None
 
 from signalwire.core.agent_base import AgentBase
 from signalwire.core.swml_service import SWMLService
 from signalwire.core.logging_config import get_logger
+import contextlib
 
 
 class AgentServer:
@@ -41,7 +43,10 @@ class AgentServer:
     """
 
     def __init__(
-        self, host: str = "0.0.0.0", port: int = 3000, log_level: str = "info"
+        self,
+        host: str = "0.0.0.0",  # noqa: S104  # intended server default: listen on all interfaces (overridable)
+        port: int = 3000,
+        log_level: str = "info",
     ):
         """
         Initialize a new agent server
@@ -61,7 +66,7 @@ class AgentServer:
         self.app = FastAPI(
             title="SignalWire AI Agents",
             description="Hosted SignalWire AI Agents",
-            version="3.0.2.dev62",
+            version="3.0.2.dev64",
             redirect_slashes=False,
             docs_url=None,
             redoc_url=None,
@@ -87,7 +92,7 @@ class AgentServer:
 
         # Keep track of SIP routing configuration
         self._sip_routing_enabled = False
-        self._sip_route = None
+        self._sip_route: Optional[str] = None
         self._sip_username_mapping: Dict[str, str] = {}  # Maps SIP usernames to routes
 
         # Register health endpoints immediately so they're available
@@ -145,10 +150,9 @@ class AgentServer:
                 self._auto_map_agent_sip_usernames(agent, route)
 
             # Register the SIP routing callback with this agent if we have one
-            if hasattr(self, "_sip_routing_callback") and self._sip_routing_callback:
-                agent.register_routing_callback(
-                    self._sip_routing_callback, path=self._sip_route
-                )
+            sip_cb = getattr(self, "_sip_routing_callback", None)
+            if sip_cb is not None and self._sip_route is not None:
+                agent.register_routing_callback(sip_cb, path=self._sip_route)
 
     def setup_sip_routing(self, route: str = "/sip", auto_map: bool = True) -> None:
         """
@@ -199,10 +203,7 @@ class AgentServer:
                 if target_route:
                     self.logger.info(f"Routing SIP request to {target_route}")
                     return target_route
-                else:
-                    self.logger.warning(
-                        f"No route found for SIP username: {sip_username}"
-                    )
+                self.logger.warning(f"No route found for SIP username: {sip_username}")
 
             # No routing needed (will be handled by the current agent)
             return None
@@ -373,11 +374,10 @@ class AgentServer:
 
         if mode == "cgi":
             return self._handle_cgi_request()
-        elif mode == "lambda":
+        if mode == "lambda":
             return self._handle_lambda_request(event, context)
-        else:
-            # Server mode - use existing logic
-            return self._run_server(host, port)
+        # Server mode - use existing logic
+        return self._run_server(host, port)
 
     def _handle_cgi_request(self) -> str:
         """Handle CGI request using same routing logic as server"""
@@ -406,7 +406,7 @@ class AgentServer:
                         swml, content_type="application/json"
                     )
                 except Exception as e:
-                    self.logger.error(f"Failed to generate SWML: {str(e)}")
+                    self.logger.error(f"Failed to generate SWML: {e!s}")
                     error_response = {"error": "Failed to generate SWML"}
                     return self._format_cgi_response(
                         error_response, status="500 Internal Server Error"
@@ -450,7 +450,7 @@ class AgentServer:
                         )
 
                     except Exception as e:
-                        self.logger.error(f"SWAIG function failed: {str(e)}")
+                        self.logger.error(f"SWAIG function failed: {e!s}")
                         error_response = {"error": "SWAIG function failed"}
                         return self._format_cgi_response(
                             error_response, status="500 Internal Server Error"
@@ -500,7 +500,7 @@ class AgentServer:
                         )
 
                     except Exception as e:
-                        self.logger.error(f"Function call failed: {str(e)}")
+                        self.logger.error(f"Function call failed: {e!s}")
                         error_response = {"error": "Function call failed"}
                         return self._format_cgi_response(
                             error_response, status="500 Internal Server Error"
@@ -545,7 +545,7 @@ class AgentServer:
                         "body": json.dumps(swml) if isinstance(swml, dict) else swml,
                     }
                 except Exception as e:
-                    self.logger.error(f"Failed to generate SWML: {str(e)}")
+                    self.logger.error(f"Failed to generate SWML: {e!s}")
                     return {
                         "statusCode": 500,
                         "headers": {"Content-Type": "application/json"},
@@ -569,10 +569,8 @@ class AgentServer:
                         # Get POST data from Lambda event body
                         post_data = {}
                         if event and "body" in event and event["body"]:
-                            try:
+                            with contextlib.suppress(json.JSONDecodeError, ValueError):
                                 post_data = json.loads(event["body"])
-                            except (json.JSONDecodeError, ValueError):
-                                pass
 
                         result = agent._execute_swaig_function(
                             function_name, post_data, None, None
@@ -586,7 +584,7 @@ class AgentServer:
                         }
 
                     except Exception as e:
-                        self.logger.error(f"Function call failed: {str(e)}")
+                        self.logger.error(f"Function call failed: {e!s}")
                         return {
                             "statusCode": 500,
                             "headers": {"Content-Type": "application/json"},
@@ -608,10 +606,7 @@ class AgentServer:
         import sys
 
         # Format the body
-        if isinstance(data, dict):
-            body = json.dumps(data)
-        else:
-            body = str(data)
+        body = json.dumps(data) if isinstance(data, dict) else str(data)
 
         # Build CGI response with headers
         response_lines = [
@@ -669,10 +664,10 @@ class AgentServer:
 
         # Validate SSL configuration if enabled
         if ssl_enabled:
-            if not ssl_cert_path or not os.path.exists(ssl_cert_path):
+            if not ssl_cert_path or not Path(ssl_cert_path).exists():
                 self.logger.warning(f"SSL cert not found: {ssl_cert_path}")
                 ssl_enabled = False
-            elif not ssl_key_path or not os.path.exists(ssl_key_path):
+            elif not ssl_key_path or not Path(ssl_key_path).exists():
                 self.logger.warning(f"SSL key not found: {ssl_key_path}")
                 ssl_enabled = False
 
@@ -688,8 +683,11 @@ class AgentServer:
             display_host = f"{host}:{port}"
 
         self.logger.info(f"Starting server on {protocol}://{display_host}")
-        for route, agent in self.agents.items():
-            username, password = agent.get_basic_auth_credentials()
+        for agent in self.agents.values():
+            # include_source defaults False -> 2-tuple at runtime; index to
+            # avoid the union-tuple unpack ambiguity mypy sees.
+            creds = agent.get_basic_auth_credentials()
+            username = creds[0]
             agent_url = agent.get_full_url(include_auth=False)
             self.logger.info(f"Agent '{agent.get_name()}' available at:")
             self.logger.info(f"URL: {agent_url}")
@@ -858,7 +856,7 @@ class AgentServer:
                 if full_path == route.lstrip("/"):
                     # This is a request to an agent's root without trailing slash
                     return await agent._handle_root_request(request)
-                elif full_path.startswith(route.lstrip("/") + "/"):
+                if full_path.startswith(route.lstrip("/") + "/"):
                     # This is a request to an agent's sub-path
                     relative_path = full_path[len(route.lstrip("/")) :]
                     relative_path = relative_path.lstrip("/")
@@ -870,21 +868,18 @@ class AgentServer:
                     clean_path = relative_path.rstrip("/")
                     if clean_path == "debug":
                         return await agent._handle_debug_request(request)
-                    elif clean_path == "swaig":
+                    if clean_path == "swaig":
                         from fastapi import Response
 
                         return await agent._handle_swaig_request(request, Response())
-                    elif clean_path == "post_prompt":
+                    if clean_path == "post_prompt":
                         return await agent._handle_post_prompt_request(request)
-                    elif clean_path == "check_for_input":
+                    if clean_path == "check_for_input":
                         return await agent._handle_check_for_input_request(request)
 
                     # Check for custom routing callbacks
                     if hasattr(agent, "_routing_callbacks"):
-                        for (
-                            callback_path,
-                            callback_fn,
-                        ) in agent._routing_callbacks.items():
+                        for callback_path in agent._routing_callbacks:
                             cb_path_clean = callback_path.strip("/")
                             if clean_path == cb_path_clean:
                                 request.state.callback_path = callback_path
@@ -893,7 +888,7 @@ class AgentServer:
             # No matching agent - check for static files
             if hasattr(self, "_static_directories"):
                 # Check each static directory route
-                for static_route, static_dir in self._static_directories.items():
+                for static_route in self._static_directories:
                     # For root static route, serve any unmatched path
                     if static_route == "" or static_route == "/":
                         response = self._serve_static_file(full_path, "")

{signalwire_sdk-3.0.2.dev62 → signalwire_sdk-3.0.2.dev64}/signalwire/signalwire/agents/bedrock.py

@@ -77,7 +77,7 @@ class BedrockAgent(AgentBase):
         logger.info(f"BedrockAgent initialized: {name} on route {route}")
 
     def _render_swml(
-        self, call_id: str = None, modifications: Optional[dict] = None
+        self, call_id: Optional[str] = None, modifications: Optional[dict] = None
     ) -> str:
         """
         Render SWML document with amazon_bedrock verb

{signalwire_sdk-3.0.2.dev62 → signalwire_sdk-3.0.2.dev64}/signalwire/signalwire/cli/build_search.py

@@ -11,6 +11,7 @@ import argparse
 import sys
 from pathlib import Path
 from datetime import datetime
+from typing import List, cast
 from urllib.parse import urlparse, urlunparse
 
 from signalwire.search.models import MODEL_ALIASES, DEFAULT_MODEL, resolve_model_alias
@@ -26,7 +27,7 @@ def _mask_connection_string(conn_str):
                 + (f":{parsed.port}" if parsed.port else "")
             )
             return urlunparse(masked)
-    except Exception:
+    except Exception:  # noqa: S110  # best-effort password masking for display; falls through to "****" on any parse failure
         pass
     return "****"
 
@@ -518,7 +519,7 @@ Examples:
                         relative_path = (
                             file_path.relative_to(base_dir)
                             if base_dir
-                            else file_path.name
+                            else Path(file_path.name)
                         )
                         json_filename = relative_path.with_suffix(".json")
                         json_path = Path(args.output_dir) / json_filename
@@ -537,7 +538,7 @@ Examples:
                             },
                         }
 
-                        with open(json_path, "w", encoding="utf-8") as f:
+                        with json_path.open("w", encoding="utf-8") as f:
                             json.dump(chunk_data, f, indent=2, ensure_ascii=False)
 
                         chunk_files_created.append(json_path)
@@ -547,7 +548,7 @@ Examples:
                         # Accumulate all chunks for single file output
                         all_chunks.extend(chunks)
 
-                except Exception as e:
+                except Exception as e:  # noqa: PERF203  # per-iteration error isolation: one failed file must not abort the batch
                     print(f"Error processing {file_path}: {e}")
                     if args.verbose:
                         import traceback
@@ -566,7 +567,7 @@ Examples:
                     },
                 }
 
-                with open(args.output, "w", encoding="utf-8") as f:
+                with Path(args.output).open("w", encoding="utf-8") as f:
                     json.dump(output_data, f, indent=2, ensure_ascii=False)
 
                 print(f"✓ Exported {len(all_chunks)} chunks to {args.output}")
@@ -575,8 +576,8 @@ Examples:
                     f"✓ Created {len(chunk_files_created)} JSON files in {args.output_dir}"
                 )
                 total_chunks = 0
-                for f in chunk_files_created:
-                    with open(f) as fh:
+                for chunk_file in chunk_files_created:
+                    with chunk_file.open() as fh:
                         total_chunks += len(json.load(fh)["chunks"])
                 print(f"  Total chunks: {total_chunks}")
 
@@ -633,7 +634,11 @@ Examples:
             # Check if the index was actually created
             import os
 
-            if os.path.exists(args.output):
+            # Rationale for PTH110 suppression below: tests patch global
+            # os.path.exists separately from pathlib.Path.exists to drive this
+            # post-build check; Path.exists() would pick up the unrelated Path
+            # mock and break the test seam.
+            if os.path.exists(args.output):  # noqa: PTH110
                 print(f"\n✓ Search index created successfully: {args.output}")
             else:
                 print("\n✗ Search index creation failed - no files were processed")
@@ -770,10 +775,11 @@ def search_command():
         args.model = MODEL_ALIASES[args.model]
 
     # Validate keyword weight if provided
-    if args.keyword_weight is not None:
-        if args.keyword_weight < 0.0 or args.keyword_weight > 1.0:
-            print("Error: --keyword-weight must be between 0.0 and 1.0")
-            sys.exit(1)
+    if args.keyword_weight is not None and (
+        args.keyword_weight < 0.0 or args.keyword_weight > 1.0
+    ):
+        print("Error: --keyword-weight must be between 0.0 and 1.0")
+        sys.exit(1)
 
     # Validate backend configuration
     if args.backend == "pgvector" and not args.connection_string:
@@ -920,7 +926,9 @@ def search_command():
 
                     # Perform search
                     results = engine.search(
-                        query_vector=enhanced.get("vector"),
+                        query_vector=cast(
+                            List[float], enhanced.get("vector")
+                        ),  # vector is None for keyword-only search; search() tolerates it at runtime
                         enhanced_text=enhanced.get("enhanced_text", query),
                         count=args.count,
                         similarity_threshold=args.distance_threshold,
@@ -1003,7 +1011,9 @@ def search_command():
 
         # Perform search
         results = engine.search(
-            query_vector=enhanced.get("vector"),
+            query_vector=cast(
+                List[float], enhanced.get("vector")
+            ),  # vector is None for keyword-only search; search() tolerates it at runtime
             enhanced_text=enhanced.get("enhanced_text", args.query),
             count=args.count,
             similarity_threshold=args.distance_threshold,
@@ -1535,17 +1545,17 @@ Examples:
             sys.argv.pop(1)
             validate_command()
             return
-        elif sys.argv[1] == "search":
+        if sys.argv[1] == "search":
             # Remove 'search' from argv and call search_command
             sys.argv.pop(1)
             search_command()
             return
-        elif sys.argv[1] == "remote":
+        if sys.argv[1] == "remote":
             # Remove 'remote' from argv and call remote_command
             sys.argv.pop(1)
             remote_command()
             return
-        elif sys.argv[1] == "migrate":
+        if sys.argv[1] == "migrate":
             # Remove 'migrate' from argv and call migrate_command
             sys.argv.pop(1)
             migrate_command()