ouroboros-ai 0.27.3.dev1__tar.gz → 0.27.3.dev2__tar.gz

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/PKG-INFO

@@ -1,32 +1,40 @@
 Metadata-Version: 2.4
 Name: ouroboros-ai
-Version: 0.27.3.dev1
+Version: 0.27.3.dev2
 Summary: Specification-first workflow engine for AI coding agents. Works with Claude Code and Codex CLI.
 Author-email: Q00 <jqyu.lee@gmail.com>
 License-File: LICENSE
 Requires-Python: >=3.12
-Requires-Dist: aiosqlite>=0.20.0
-Requires-Dist: pydantic>=2.0.0
-Requires-Dist: pyyaml>=6.0.0
-Requires-Dist: rich>=13.0.0
-Requires-Dist: sqlalchemy[asyncio]>=2.0.0
-Requires-Dist: structlog>=24.0.0
-Requires-Dist: typer>=0.12.0
+Requires-Dist: aiosqlite<1.0.0,>=0.20.0
+Requires-Dist: prompt-toolkit<4.0.0,>=3.0.0
+Requires-Dist: pydantic<3.0.0,>=2.0.0
+Requires-Dist: pyyaml<7.0.0,>=6.0.0
+Requires-Dist: rich<15.0.0,>=13.0.0
+Requires-Dist: sqlalchemy[asyncio]<3.0.0,>=2.0.0
+Requires-Dist: structlog<26.0.0,>=24.0.0
+Requires-Dist: typer<1.0.0,>=0.12.0
 Provides-Extra: all
-Requires-Dist: anthropic>=0.52.0; extra == 'all'
-Requires-Dist: claude-agent-sdk>=0.1.0; extra == 'all'
+Requires-Dist: anthropic<1.0.0,>=0.52.0; extra == 'all'
+Requires-Dist: claude-agent-sdk<1.0.0,>=0.1.0; extra == 'all'
 Requires-Dist: litellm<=1.82.6,>=1.80.0; extra == 'all'
-Requires-Dist: mcp>=1.26.0; extra == 'all'
-Requires-Dist: textual>=1.0.0; extra == 'all'
+Requires-Dist: mcp<2.0.0,>=1.26.0; extra == 'all'
+Requires-Dist: pandas<3.0.0,>=2.2.0; extra == 'all'
+Requires-Dist: plotly<7.0.0,>=5.24.0; extra == 'all'
+Requires-Dist: streamlit<2.0.0,>=1.40.0; extra == 'all'
+Requires-Dist: textual<9.0.0,>=1.0.0; extra == 'all'
 Provides-Extra: claude
-Requires-Dist: anthropic>=0.52.0; extra == 'claude'
-Requires-Dist: claude-agent-sdk>=0.1.0; extra == 'claude'
+Requires-Dist: anthropic<1.0.0,>=0.52.0; extra == 'claude'
+Requires-Dist: claude-agent-sdk<1.0.0,>=0.1.0; extra == 'claude'
+Provides-Extra: dashboard
+Requires-Dist: pandas<3.0.0,>=2.2.0; extra == 'dashboard'
+Requires-Dist: plotly<7.0.0,>=5.24.0; extra == 'dashboard'
+Requires-Dist: streamlit<2.0.0,>=1.40.0; extra == 'dashboard'
 Provides-Extra: litellm
 Requires-Dist: litellm<=1.82.6,>=1.80.0; extra == 'litellm'
 Provides-Extra: mcp
-Requires-Dist: mcp>=1.26.0; extra == 'mcp'
+Requires-Dist: mcp<2.0.0,>=1.26.0; extra == 'mcp'
 Provides-Extra: tui
-Requires-Dist: textual>=1.0.0; extra == 'tui'
+Requires-Dist: textual<9.0.0,>=1.0.0; extra == 'tui'
 Description-Content-Type: text/markdown
 
 <p align="right">
@@ -128,10 +136,12 @@ pip install ouroboros-ai[claude]        # + Claude Code deps
 pip install ouroboros-ai[litellm]       # + LiteLLM multi-provider
 pip install ouroboros-ai[mcp]           # + MCP server/client support
 pip install ouroboros-ai[tui]           # + Textual terminal UI
-pip install ouroboros-ai[all]           # everything
+pip install ouroboros-ai[all]           # everything (claude + litellm + mcp + tui + dashboard)
 ouroboros setup                         # configure runtime
 ```
 
+Legacy compatibility: `ouroboros-ai[dashboard]` is still accepted as a compatibility alias while extras migrate.
+
 See runtime guides: [Claude Code](./docs/runtime-guides/claude-code.md) · [Codex CLI](./docs/runtime-guides/codex.md)
 
 </details>

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/README.ko.md

@@ -125,6 +125,8 @@ pip install ouroboros-ai[all]           # 전부
 ouroboros setup                         # 런타임 설정
 ```
 
+호환성 참고: extras 전환 기간 동안 `ouroboros-ai[dashboard]`도 no-op alias로 계속 허용됩니다.
+
 런타임별 가이드: [Claude Code](./docs/runtime-guides/claude-code.md) · [Codex CLI](./docs/runtime-guides/codex.md)
 
 </details>

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/README.md

@@ -97,10 +97,12 @@ pip install ouroboros-ai[claude]        # + Claude Code deps
 pip install ouroboros-ai[litellm]       # + LiteLLM multi-provider
 pip install ouroboros-ai[mcp]           # + MCP server/client support
 pip install ouroboros-ai[tui]           # + Textual terminal UI
-pip install ouroboros-ai[all]           # everything
+pip install ouroboros-ai[all]           # everything (claude + litellm + mcp + tui + dashboard)
 ouroboros setup                         # configure runtime
 ```
 
+Legacy compatibility: `ouroboros-ai[dashboard]` is still accepted as a compatibility alias while extras migrate.
+
 See runtime guides: [Claude Code](./docs/runtime-guides/claude-code.md) · [Codex CLI](./docs/runtime-guides/codex.md)
 
 </details>

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/docs/cli-reference.md

@@ -160,10 +160,6 @@ ouroboros init start --resume interview_20260116_120000
 ouroboros init
 ```
 
-> **Interactive multiline input:** when `init` or other interview-style commands prompt for a
-> longer response, press `Ctrl+D` (macOS/Linux) or `Ctrl+Z` then Enter (Windows) to submit the
-> full response. Plain Enter inserts a newline.
-
 ### `init list`
 
 List all interview sessions.

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/docs/getting-started.md

@@ -82,13 +82,14 @@ pip install ouroboros-ai[claude]      # + Claude Code runtime deps (anthropic, c
 pip install ouroboros-ai[litellm]     # + LiteLLM multi-provider support (100+ models)
 pip install ouroboros-ai[mcp]         # + MCP server/client runtime support
 pip install ouroboros-ai[tui]         # + Textual terminal UI
-pip install ouroboros-ai[all]         # Everything (claude + litellm + mcp + tui)
+pip install ouroboros-ai[all]         # Everything (claude + litellm + mcp + tui + dashboard)
 
 ouroboros --version                   # verify CLI
 ```
 
 > **Which extra do I need?** If you only use Claude Code as your runtime, `ouroboros-ai[claude]` is sufficient.
 > For multi-model support via LiteLLM, use `ouroboros-ai[litellm]` or just grab everything with `ouroboros-ai[all]`.
+> Legacy note: `ouroboros-ai[dashboard]` is still accepted as a compatibility extra during the extras transition.
 
 **One-liner alternative** (auto-detects your runtime and installs matching extras):
 ```bash

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/pyproject.toml

@@ -8,21 +8,27 @@ authors = [
 ]
 requires-python = ">=3.12"
 dependencies = [
-    "aiosqlite>=0.20.0",
-    "pydantic>=2.0.0",
-    "pyyaml>=6.0.0",
-    "rich>=13.0.0",
-    "sqlalchemy[asyncio]>=2.0.0",
-    "structlog>=24.0.0",
-    "typer>=0.12.0",
+    "aiosqlite>=0.20.0,<1.0.0",
+    "pydantic>=2.0.0,<3.0.0",
+    "prompt-toolkit>=3.0.0,<4.0.0",
+    "pyyaml>=6.0.0,<7.0.0",
+    "rich>=13.0.0,<15.0.0",
+    "sqlalchemy[asyncio]>=2.0.0,<3.0.0",
+    "structlog>=24.0.0,<26.0.0",
+    "typer>=0.12.0,<1.0.0",
 ]
 
 [project.optional-dependencies]
-claude = ["claude-agent-sdk>=0.1.0", "anthropic>=0.52.0"]
+claude = ["claude-agent-sdk>=0.1.0,<1.0.0", "anthropic>=0.52.0,<1.0.0"]
 litellm = ["litellm>=1.80.0,<=1.82.6"]
-mcp = ["mcp>=1.26.0"]
-tui = ["textual>=1.0.0"]
-all = ["ouroboros-ai[claude,litellm,mcp,tui]"]
+dashboard = [
+    "streamlit>=1.40.0,<2.0.0",
+    "plotly>=5.24.0,<7.0.0",
+    "pandas>=2.2.0,<3.0.0",
+]
+mcp = ["mcp>=1.26.0,<2.0.0"]
+tui = ["textual>=1.0.0,<9.0.0"]
+all = ["ouroboros-ai[claude,litellm,mcp,tui,dashboard]"]
 
 [project.scripts]
 ouroboros = "ouroboros.cli.main:app"

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/src/ouroboros/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.27.3.dev1'
-__version_tuple__ = version_tuple = (0, 27, 3, 'dev1')
+__version__ = version = '0.27.3.dev2'
+__version_tuple__ = version_tuple = (0, 27, 3, 'dev2')
 
 __commit_id__ = commit_id = None

ouroboros_ai-0.27.3.dev2/src/ouroboros/cli/formatters/prompting.py

@@ -0,0 +1,38 @@
+"""Prompt helpers for interactive CLI input."""
+
+from __future__ import annotations
+
+from prompt_toolkit import PromptSession
+from prompt_toolkit.key_binding import KeyBindings, KeyPressEvent
+from prompt_toolkit.patch_stdout import patch_stdout
+
+from ouroboros.cli.formatters import console
+
+
+async def multiline_prompt_async(prompt_text: str) -> str:
+    """Get multiline-safe input while allowing logs above the active prompt."""
+    bindings = KeyBindings()
+
+    @bindings.add("c-j")
+    def insert_newline(event: KeyPressEvent) -> None:
+        event.current_buffer.insert_text("\n")
+
+    @bindings.add("c-m")
+    def submit(event: KeyPressEvent) -> None:
+        event.current_buffer.validate_and_handle()
+
+    console.print(f"[bold green]{prompt_text}[/] [dim](Enter: submit, Ctrl+J: newline)[/]")
+
+    session: PromptSession[str] = PromptSession(
+        message="> ",
+        multiline=True,
+        prompt_continuation="  ",
+        key_bindings=bindings,
+    )
+
+    # prompt_toolkit proxies both stdout and stderr above the prompt.
+    with patch_stdout(raw=True):
+        return await session.prompt_async()
+
+
+__all__ = ["multiline_prompt_async"]

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/src/ouroboros/config/loader.py

@@ -54,6 +54,8 @@ from ouroboros.core.errors import ConfigError  # noqa: E402
 _CODEX_LLM_BACKENDS = frozenset({"codex", "codex_cli", "opencode", "opencode_cli"})
 _OPENCODE_BACKENDS = frozenset({"opencode", "opencode_cli"})
 _CODEX_DEFAULT_MODEL = "default"
+_PLACEHOLDER_API_KEY_PREFIX = "YOUR_"
+_PLACEHOLDER_API_KEY_SUFFIX = "_API_KEY"
 _DEFAULT_CONSENSUS_MODELS = (
     "openrouter/openai/gpt-4o",
     "openrouter/anthropic/claude-opus-4-6",
@@ -82,6 +84,16 @@ def _parse_env_value(raw_value: str) -> str:
     return candidate.rstrip()
 
 
+def _is_placeholder_api_key(value: str) -> bool:
+    """Treat common template placeholders as unset."""
+    candidate = value.strip()
+    return bool(
+        candidate
+        and candidate.startswith(_PLACEHOLDER_API_KEY_PREFIX)
+        and candidate.endswith(_PLACEHOLDER_API_KEY_SUFFIX)
+    )
+
+
 def _load_env_file(path: Path) -> None:
     if not path.exists():
         return
@@ -99,7 +111,14 @@ def _load_env_file(path: Path) -> None:
         key = key.strip()
         if not key or any(ch.isspace() for ch in key):
             continue
-        os.environ.setdefault(key, _parse_env_value(raw_value))
+
+        parsed_value = _parse_env_value(raw_value)
+        if not parsed_value or _is_placeholder_api_key(parsed_value):
+            continue
+
+        current_value = os.environ.get(key)
+        if current_value is None or _is_placeholder_api_key(current_value):
+            os.environ[key] = parsed_value
 
 
 for env_path in (Path(".env"), Path.home() / ".ouroboros" / ".env"):

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/src/ouroboros/core/file_lock.py

@@ -15,14 +15,14 @@ else:  # pragma: no branch
 
 
 @contextmanager
-def file_lock(file_path: Path, exclusive: bool = True) -> Iterator[None]:  # noqa: ARG001
+def file_lock(file_path: Path, exclusive: bool = True) -> Iterator[None]:
     """Context manager for cross-platform file locking."""
     lock_path = file_path.with_suffix(file_path.suffix + ".lock")
     lock_path.parent.mkdir(parents=True, exist_ok=True)
 
     with lock_path.open("a+", encoding="utf-8") as handle:
         _ensure_lockfile_content(handle)
-        _acquire_lock(handle)
+        _acquire_lock(handle, exclusive=exclusive)
         try:
             yield
         finally:
@@ -37,13 +37,13 @@ def _ensure_lockfile_content(handle: TextIO) -> None:
         handle.seek(0)
 
 
-def _acquire_lock(handle: TextIO) -> None:
+def _acquire_lock(handle: TextIO, *, exclusive: bool = True) -> None:
     if os.name == "nt":  # pragma: no cover - exercised on Windows
         handle.seek(0)
-        msvcrt.locking(handle.fileno(), msvcrt.LK_LOCK, 1)
+        msvcrt.locking(handle.fileno(), msvcrt.LK_RLCK if not exclusive else msvcrt.LK_LOCK, 1)
         return
 
-    fcntl.flock(handle.fileno(), fcntl.LOCK_EX)
+    fcntl.flock(handle.fileno(), fcntl.LOCK_EX if exclusive else fcntl.LOCK_SH)
 
 
 def _release_lock(handle: TextIO) -> None:

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/src/ouroboros/mcp/server/adapter.py

@@ -8,8 +8,10 @@ handling, and server lifecycle.
 import asyncio
 from collections.abc import Sequence
 import inspect
+import keyword
 import os
 from pathlib import Path
+import re
 from typing import Any
 
 import structlog
@@ -105,13 +107,50 @@ def _build_tool_signature(parameters: tuple[MCPToolParameter, ...]) -> inspect.S
     By setting __signature__ with explicit parameters, FastMCP generates the
     correct schema and clients can send flat argument dicts.
     """
+    signature, _ = _build_tool_signature_with_aliases(parameters)
+    return signature
+
+
+def _to_safe_signature_name(name: str) -> str:
+    """Return a valid Python identifier for a tool parameter name."""
+    if name.isidentifier() and not keyword.iskeyword(name):
+        return name
+
+    # Replace invalid characters with underscore and avoid starting with a digit.
+    sanitized = re.sub(r"[^0-9a-zA-Z_]", "_", name)
+    sanitized = re.sub(r"_+", "_", sanitized).strip("_")
+    if not sanitized or sanitized[0].isdigit():
+        sanitized = f"_{sanitized or 'param'}"
+
+    if keyword.iskeyword(sanitized):
+        sanitized = f"_{sanitized}"
+
+    return sanitized
+
+
+def _build_tool_signature_with_aliases(
+    parameters: tuple[MCPToolParameter, ...],
+) -> tuple[inspect.Signature, dict[str, str]]:
+    """Build signature plus map from schema args to original MCP parameter names."""
+
     sig_params = []
+    alias_counts: dict[str, int] = {}
+    alias_to_original: dict[str, str] = {}
+
     for p in parameters:
+        parameter_name = _to_safe_signature_name(p.name)
+        alias_count = alias_counts.get(parameter_name, 0) + 1
+        alias_counts[parameter_name] = alias_count
+        if alias_count > 1:
+            parameter_name = f"{parameter_name}_{alias_count}"
+
+        alias_to_original[parameter_name] = p.name
+
         python_type = _TOOL_TYPE_MAP.get(p.type, Any)
         if p.required:
             sig_params.append(
                 inspect.Parameter(
-                    name=p.name,
+                    name=parameter_name,
                     kind=inspect.Parameter.KEYWORD_ONLY,
                     annotation=python_type,
                 )
@@ -120,13 +159,14 @@ def _build_tool_signature(parameters: tuple[MCPToolParameter, ...]) -> inspect.S
             default = p.default if p.default is not None else None
             sig_params.append(
                 inspect.Parameter(
-                    name=p.name,
+                    name=parameter_name,
                     kind=inspect.Parameter.KEYWORD_ONLY,
                     default=default,
                     annotation=python_type | None,
                 )
             )
-    return inspect.Signature(parameters=sig_params)
+
+    return inspect.Signature(parameters=sig_params), alias_to_original
 
 
 _PROJECT_ROOT_MARKERS = (
@@ -200,11 +240,13 @@ def _project_dir_from_artifact(artifact: str) -> str | None:
     from pathlib import Path
     import re
 
-    # Match quoted paths (spaces allowed) or unquoted paths (no spaces).
+    # Match quoted paths (spaces allowed) and unquoted paths.
     # Examples:  Write: /foo/bar.py  |  File: "/path with spaces/bar.py"
     write_matches: list[str] = []
-    for m in re.finditer(r'(?:Write|Edit|File): (?:"([^"]+)"|(/[^\s]+))', artifact):
-        write_matches.append(m.group(1) or m.group(2))
+    for m in re.finditer(r'(?:Write|Edit|File): (?:"([^"]+)"|(.+))', artifact):
+        path_candidate = m.group(1) or m.group(2)
+        if path_candidate:
+            write_matches.append(path_candidate.strip())
     for path_str in write_matches:
         candidate = Path(path_str).parent
         for _ in range(10):
@@ -529,7 +571,18 @@ class MCPServerAdapter:
                         and isinstance(kwargs["kwargs"], dict)
                     ):
                         kwargs = kwargs["kwargs"]
-                    result = await h.handle(kwargs)
+
+                    _, alias_to_original = _build_tool_signature_with_aliases(
+                        h.definition.parameters,
+                    )
+                    normalized_kwargs: dict[str, Any] = {}
+                    for alias_key, original_key in alias_to_original.items():
+                        if alias_key in kwargs:
+                            normalized_kwargs[original_key] = kwargs[alias_key]
+                    for key, value in kwargs.items():
+                        normalized_kwargs.setdefault(key, value)
+
+                    result = await h.handle(normalized_kwargs)
                     if result.is_ok:
                         # Convert MCPToolResult to FastMCP format
                         tool_result = result.value

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/src/ouroboros/orchestrator/mcp_tools.py

@@ -2059,6 +2059,7 @@ class MCPToolProvider:
 
         try:
             # Retry transient failures with bounded backoff
+            # Use retry_async for retries on transient failures
             result = await self._call_with_retry(
                 tool_info=tool_info,
                 arguments=arguments or {},
@@ -2090,7 +2091,7 @@ class MCPToolProvider:
     ) -> Result[MCPToolResult, MCPToolError]:
         """Call tool with retry logic for transient failures.
 
-        Uses internal exponential backoff retries on:
+        Uses retry_async for exponential backoff retries on:
         - Connection errors
         - Timeout errors (if marked retriable)
         - Other transient MCPClientErrors

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/src/ouroboros/providers/litellm_adapter.py

@@ -160,6 +160,18 @@ class LiteLLMAdapter:
             if configured_api_key:
                 return configured_api_key
 
+        # Unknown/custom models may still be routed through OpenRouter via credentials.
+        provider_name = self._extract_provider(model)
+        if provider_name not in {"openrouter", "openai", "anthropic", "google"}:
+            credentials = self._load_credentials_config()
+            configured = (
+                credentials.providers.get("openrouter") if credentials is not None else None
+            )
+            if configured is not None:
+                configured_api_key = self._normalize_api_key(configured.api_key)
+                if configured_api_key:
+                    return configured_api_key
+
         # Default to OpenRouter for unknown models
         return self._normalize_api_key(os.environ.get("OPENROUTER_API_KEY"))
 

ouroboros_ai-0.27.3.dev2/tests/unit/cli/formatters/test_prompting.py

@@ -0,0 +1,45 @@
+"""Tests for interactive prompt helpers."""
+
+from __future__ import annotations
+
+import sys
+from unittest.mock import AsyncMock, MagicMock, patch
+
+import pytest
+
+from ouroboros.cli.formatters.prompting import multiline_prompt_async
+
+
+@pytest.mark.asyncio
+async def test_multiline_prompt_async_patches_stdout_and_stderr() -> None:
+    """The shared prompt helper should proxy both stdout and stderr during input."""
+    session = MagicMock()
+    session.prompt_async = AsyncMock()
+
+    async def fake_prompt_async() -> str:
+        assert sys.stdout is not sys.__stdout__
+        assert sys.stderr is not sys.__stderr__
+        return "line 1\nline 2"
+
+    session.prompt_async.side_effect = fake_prompt_async
+
+    with (
+        patch(
+            "ouroboros.cli.formatters.prompting.PromptSession", return_value=session
+        ) as mock_session,
+        patch("ouroboros.cli.formatters.prompting.console.print"),
+    ):
+        result = await multiline_prompt_async("Prompt here")
+
+    assert result == "line 1\nline 2"
+    session.prompt_async.assert_awaited_once()
+
+    kwargs = mock_session.call_args.kwargs
+    assert kwargs["message"] == "> "
+    assert kwargs["multiline"] is True
+    assert kwargs["prompt_continuation"] == "  "
+
+    key_bindings = kwargs["key_bindings"]
+    bound_keys = {tuple(binding.keys) for binding in key_bindings.bindings}
+    assert ("c-j",) in bound_keys
+    assert ("c-m",) in bound_keys

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/tests/unit/config/test_loader_env.py

@@ -34,3 +34,19 @@ def test_load_env_file_does_not_override_existing_values(
     _load_env_file(env_file)
 
     assert os.environ["FIRST"] == "existing"
+
+
+def test_load_env_file_skips_template_placeholders(tmp_path: Path, monkeypatch) -> None:
+    """Template placeholders should not block later env values from loading."""
+    repo_env = tmp_path / "repo.env"
+    home_env = tmp_path / "home.env"
+
+    repo_env.write_text("OPENROUTER_API_KEY=YOUR_OPENROUTER_API_KEY")
+    home_env.write_text("OPENROUTER_API_KEY=real-key")
+
+    monkeypatch.delenv("OPENROUTER_API_KEY", raising=False)
+
+    _load_env_file(repo_env)
+    _load_env_file(home_env)
+
+    assert os.environ["OPENROUTER_API_KEY"] == "real-key"

ouroboros_ai-0.27.3.dev2/tests/unit/core/test_file_lock.py

@@ -0,0 +1,52 @@
+"""Tests for stdlib-backed file locking."""
+
+from __future__ import annotations
+
+from pathlib import Path
+from unittest.mock import MagicMock
+
+import ouroboros.core.file_lock as file_lock_module
+from ouroboros.core.file_lock import _acquire_lock, _release_lock, file_lock
+
+
+def test_file_lock_creates_lockfile(tmp_path: Path) -> None:
+    target = tmp_path / "state.json"
+    target.write_text("{}")
+
+    with file_lock(target):
+        lock_path = target.with_suffix(".json.lock")
+        assert lock_path.exists()
+        assert lock_path.read_text() == "0"
+
+
+def test_file_lock_exclusive_false_acquires_shared_lock(tmp_path: Path) -> None:
+    """Shared (non-exclusive) locks should allow concurrent readers."""
+    target = tmp_path / "data.json"
+    target.write_text("{}")
+
+    with file_lock(target, exclusive=False):
+        lock_path = target.with_suffix(".json.lock")
+        assert lock_path.exists()
+        # A second shared lock on the same file should not block
+        with file_lock(target, exclusive=False):
+            assert lock_path.exists()
+
+
+def test_file_lock_windows_shared_uses_read_lock_mode(monkeypatch, tmp_path: Path) -> None:
+    """On Windows, non-exclusive lock requests should use a shared/read mode."""
+    target = tmp_path / "shared.json"
+    target.write_text("{}")
+    with target.open("a+", encoding="utf-8") as handle:
+        fd = handle.fileno()
+        mock_msvcrt = MagicMock()
+        mock_msvcrt.LK_LOCK = 1
+        mock_msvcrt.LK_RLCK = 2
+        mock_msvcrt.LK_UNLCK = 3
+        monkeypatch.setattr(file_lock_module, "msvcrt", mock_msvcrt, raising=False)
+        monkeypatch.setattr(file_lock_module.os, "name", "nt")
+
+        _acquire_lock(handle, exclusive=False)
+        _release_lock(handle)
+
+    mock_msvcrt.locking.assert_any_call(fd, mock_msvcrt.LK_RLCK, 1)
+    mock_msvcrt.locking.assert_any_call(fd, mock_msvcrt.LK_UNLCK, 1)

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/tests/unit/mcp/server/test_adapter.py

@@ -11,6 +11,7 @@ from ouroboros.mcp.errors import MCPResourceNotFoundError, MCPServerError
 from ouroboros.mcp.server.adapter import (
     VALID_TRANSPORTS,
     MCPServerAdapter,
+    _build_tool_signature_with_aliases,
     _extract_feedback_metadata_from_artifact,
     _project_dir_from_artifact,
     _project_dir_from_seed,
@@ -119,6 +120,34 @@ class TestMCPServerAdapter:
 
         assert _project_dir_from_artifact(artifact) == str(project_dir)
 
+    def test_project_dir_from_artifact_handles_spaces_in_paths(self, tmp_path) -> None:
+        """Artifact extraction should detect spaced file paths."""
+        project_dir = tmp_path / "my project"
+        nested_dir = project_dir / "src" / "components"
+        nested_dir.mkdir(parents=True)
+        (project_dir / "pyproject.toml").write_text("[build-system]")
+
+        artifact = f"Edit: {nested_dir / 'app.tsx'}"
+
+        assert _project_dir_from_artifact(artifact) == str(project_dir)
+
+    def test_build_tool_signature_sanitizes_non_identifier_parameter_names(self) -> None:
+        """Invalid MCP parameter names are sanitized to valid Python signatures."""
+        parameters = (
+            MCPToolParameter(name="file-path", type=ToolInputType.STRING),
+            MCPToolParameter(name="max.tokens", type=ToolInputType.INTEGER),
+            MCPToolParameter(name="class", type=ToolInputType.BOOLEAN),
+        )
+
+        signature, aliases = _build_tool_signature_with_aliases(parameters)
+        names = tuple(param.name for param in signature.parameters.values())
+        assert names == ("file_path", "max_tokens", "_class")
+        assert aliases == {
+            "file_path": "file-path",
+            "max_tokens": "max.tokens",
+            "_class": "class",
+        }
+
     def test_extract_feedback_metadata_from_artifact_parses_structured_warning(self) -> None:
         """Execution artifacts should expose structured evaluation feedback metadata."""
         artifact = """

{ouroboros_ai-0.27.3.dev1 → ouroboros_ai-0.27.3.dev2}/tests/unit/providers/test_litellm_adapter.py

@@ -128,6 +128,26 @@ class TestLiteLLMAdapterGetApiKey:
 
         assert result == "or-key"
 
+    def test_unknown_model_uses_openrouter_credentials_fallback(self) -> None:
+        """Unknown models can resolve credentials from the openrouter provider."""
+        adapter = LiteLLMAdapter()
+        credentials = CredentialsConfig(
+            providers={
+                "openrouter": ProviderCredentials(
+                    api_key="cred-openrouter-key",
+                    base_url="https://openrouter.example/v1",
+                ),
+            }
+        )
+
+        with (
+            patch.dict("os.environ", {}, clear=True),
+            patch.object(adapter, "_load_credentials_config", return_value=credentials),
+        ):
+            result = adapter._get_api_key("acme/gpt-4-custom")
+
+        assert result == "cred-openrouter-key"
+
     def test_missing_env_var_returns_none(self) -> None:
         """Returns None if no environment variable is set."""
         adapter = LiteLLMAdapter()