victor-coding 0.7.2__tar.gz → 0.14.0__tar.gz

{victor_coding-0.7.2 → victor_coding-0.14.0}/LICENSE

@@ -175,7 +175,7 @@
 
    END OF TERMS AND CONDITIONS
 
-   Copyright 2025 Vijaykumar Singh <singhvjd@gmail.com>
+   Copyright 2025 Vijaykumar Singh <vijay@anvaiops.com>
 
    Licensed under the Apache License, Version 2.0 (the "License");
    you may not use this file except in compliance with the License.

{victor_coding-0.7.2 → victor_coding-0.14.0}/PKG-INFO

@@ -1,9 +1,9 @@
 Metadata-Version: 2.4
 Name: victor-coding
-Version: 0.7.2
+Version: 0.14.0
 Summary: Coding vertical for Victor AI - Software development, debugging, and code generation
-Author-email: Vijaykumar Singh <singhvjd@gmail.com>
-Maintainer-email: Vijaykumar Singh <singhvjd@gmail.com>
+Author-email: Vijaykumar Singh <vijay@anvaiops.com>
+Maintainer-email: Vijaykumar Singh <vijay@anvaiops.com>
 License: Apache-2.0
 Project-URL: Homepage, https://github.com/vjsingh1984/victor-coding
 Project-URL: Repository, https://github.com/vjsingh1984/victor-coding
@@ -13,34 +13,33 @@ Keywords: victor,vertical,coding,development,debugging,refactoring
 Classifier: Development Status :: 4 - Beta
 Classifier: Intended Audience :: Developers
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: Software Development
-Requires-Python: >=3.11
+Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: victor-contracts[yaml]<1.0,>=0.6.0
-Requires-Dist: numpy<2.3,>=1.24
+Requires-Dist: victor-contracts[yaml]<1.0,>=0.11.0
 Requires-Dist: victor-codegraph[treesitter]>=0.1.0
 Requires-Dist: defusedxml<1.0,>=0.7
 Provides-Extra: runtime
 Requires-Dist: victor-ai<1.0,>=0.7.6; extra == "runtime"
 Provides-Extra: native
 Provides-Extra: proximadb
-Requires-Dist: proximadb<0.3,>=0.2; extra == "proximadb"
+Requires-Dist: proximadb<0.5,>=0.4.1; extra == "proximadb"
 Provides-Extra: codegraph
 Requires-Dist: victor-codegraph<1.0,>=0.1.0; extra == "codegraph"
 Provides-Extra: dev
-Requires-Dist: pytest>=8.0; extra == "dev"
+Requires-Dist: numpy<2.6,>=1.24; extra == "dev"
+Requires-Dist: pytest<9.2,>=8.0; extra == "dev"
 Requires-Dist: pytest-asyncio>=0.23; extra == "dev"
 Requires-Dist: pytest-cov>=4.1; extra == "dev"
 Requires-Dist: pytest-xdist>=3.0; extra == "dev"
 Requires-Dist: pytest-benchmark>=5.0; extra == "dev"
-Requires-Dist: black==26.1.0; extra == "dev"
+Requires-Dist: black==26.10.1; extra == "dev"
 Requires-Dist: ruff>=0.5; extra == "dev"
-Requires-Dist: mypy>=1.10; extra == "dev"
+Requires-Dist: mypy==2.4.0; extra == "dev"
 Provides-Extra: native-dev
-Requires-Dist: maturin<2.0,>=1.0; extra == "native-dev"
+Requires-Dist: maturin<2.0,>=1.10; extra == "native-dev"
 Dynamic: license-file
 
 # victor-coding

{victor_coding-0.7.2 → victor_coding-0.14.0}/pyproject.toml

@@ -1,22 +1,22 @@
 [build-system]
 requires = [
-    "setuptools>=61.0",
+    "setuptools>=83.0.0",
     "wheel",
 ]
 build-backend = "setuptools.build_meta"
 
 [project]
 name = "victor-coding"
-version = "0.7.2"
+version = "0.14.0"
 description = "Coding vertical for Victor AI - Software development, debugging, and code generation"
 readme = "README.md"
-requires-python = ">=3.11"
+requires-python = ">=3.12"
 license = {text = "Apache-2.0"}
 authors = [
-    { name = "Vijaykumar Singh", email = "singhvjd@gmail.com" },
+    { name = "Vijaykumar Singh", email = "vijay@anvaiops.com" },
 ]
 maintainers = [
-    { name = "Vijaykumar Singh", email = "singhvjd@gmail.com" },
+    { name = "Vijaykumar Singh", email = "vijay@anvaiops.com" },
 ]
 keywords = [
     "victor",
@@ -30,13 +30,11 @@ classifiers = [
     "Development Status :: 4 - Beta",
     "Intended Audience :: Developers",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Topic :: Software Development",
 ]
 dependencies = [
-    "victor-contracts[yaml]>=0.6.0,<1.0",  # Definition-layer contract for external verticals
-    "numpy>=1.24,<2.3",  # Required by victor.storage.embeddings; numba-compatible
+    "victor-contracts[yaml]>=0.11.0,<1.0",  # Definition-layer contract for external verticals
     "victor-codegraph[treesitter]>=0.1.0",  # Shared code->CPG chunker (ADR-014); [treesitter] supplies the grammars the delegation parses JS/Go/TS/Rust with
     "defusedxml>=0.7,<1.0",  # XXE-hardened XML parsing for coverage reports
 ]
@@ -46,16 +44,15 @@ runtime = [
     "victor-ai>=0.7.6,<1.0",  # Runtime support - optional, contract-first architecture
 ]
 native = [
-    # Native extensions are handled by maturin
-    # Install with: pip install -e .[native]
-    # or build with: maturin develop --release
+    # Compatibility extra. Build the separate distribution from ./native;
+    # see native/README.md. Installing this empty extra does not build Rust.
 ]
-# ProximaDB embedding provider (optional backend).
-# Imported lazily with graceful ImportError fallback in
-# victor_coding/codebase/embeddings/proximadb_provider.py. Pin kept identical
-# to victor-ai's proximadb extra so the backend version stays aligned.
+# ProximaDB embedding backend (optional). The provider lives in victor-ai
+# (victor/storage/vector_stores/proximadb_provider.py) and is resolved through
+# victor_contracts.embeddings_runtime; this pass-through extra keeps the pin
+# identical to victor-ai's proximadb extra.
 proximadb = [
-    "proximadb>=0.2,<0.3",
+    "proximadb>=0.4.1,<0.5",
 ]
 # Shared code->CPG chunker (ADR-014). Opt-in: when installed AND
 # VICTOR_CODING_USE_CODEGRAPH=1, CodeChunker delegates to it. Published on PyPI
@@ -64,17 +61,18 @@ codegraph = [
     "victor-codegraph>=0.1.0,<1.0",
 ]
 dev = [
-    "pytest>=8.0",
+    "numpy>=1.24,<2.6",  # Synthetic embeddings in indexer tests
+    "pytest>=8.0,<9.2",  # pytest 9.2 (2026-10-27) removed Function._nodeid, which pytest-benchmark reads in fixture setup (verticals CI _nodeid AttributeErrors)
     "pytest-asyncio>=0.23",
     "pytest-cov>=4.1",
     "pytest-xdist>=3.0",
     "pytest-benchmark>=5.0",
-    "black==26.1.0",
+    "black==26.10.1",
     "ruff>=0.5",
-    "mypy>=1.10",
+    "mypy==2.4.0",
 ]
 native-dev = [
-    "maturin>=1.0,<2.0",  # Build Rust extensions
+    "maturin>=1.10,<2.0",  # Build Rust extensions
 ]
 
 [project.urls]
@@ -145,7 +143,7 @@ coding-compaction = "victor_coding.protocols:CodingCompactionProvider"
 [tool.setuptools.packages.find]
 # Auto-discover all `victor_coding*` subpackages. An explicit `packages` list
 # here previously omitted nested subpackages (languages.plugins,
-# codebase.embeddings, codebase.graph, compat, completion.providers,
+# codebase.embeddings, codebase.graph, compat,
 # coverage.parsers, native, performance, refactor.transforms), which silently
 # broke the published wheel (ModuleNotFoundError) while editable installs
 # worked — because the whole source tree is on sys.path under `-e`.
@@ -156,20 +154,23 @@ exclude = ["tests*", "docs*", "native*"]
 [tool.black]
 line-length = 100
 target-version = [
-    "py311",
     "py312",
 ]
 
 [tool.ruff]
+extend-exclude = [
+    # Parser input corpus for the golden tests: data, not code. Editing it
+    # shifts the pinned line numbers.
+    "tests/codebase/goldens/samples",
+]
 line-length = 100
-target-version = "py311"
+target-version = "py312"
 
 [tool.ruff.lint]
 select = ["E", "W", "F", "B", "C4"]
 ignore = [
     "E501",  # line too long (handled by formatter)
     "E402",  # module level import not at top of file (sometimes necessary)
-    "F401",  # unused imports (often intentional re-exports)
     "F811",  # redefined while unused (type imports shadowing runtime)
     "B007",  # loop control variable not used within loop body
     "B008",  # do not perform function calls in argument defaults
@@ -187,5 +188,5 @@ ignore = [
 "__init__.py" = ["F401"]
 "tests/**/*.py" = ["B017", "B018", "E741", "F841"]
 [tool.mypy]
-python_version = "3.11"
+python_version = "3.12"
 warn_return_any = true

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_coding_capabilities.py

@@ -1,4 +1,4 @@
-# Copyright 2026 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2026 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

victor_coding-0.14.0/tests/test_coding_handlers.py

@@ -0,0 +1,147 @@
+# Copyright 2026 Vijaykumar Singh <vijay@anvaiops.com>
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+"""Pre-move characterization for victor-coding compute handlers (dedup F5).
+
+Pins the observable contract before the envelope is extracted to the shared
+kernel: command construction, output-key handling, the two handlers' DIFFERENT
+failure policies (code_validation always COMPLETED with all_passed inside the
+output; test_runner FAILED when the suite fails), and registration semantics.
+"""
+
+from typing import Any, Dict
+from unittest.mock import AsyncMock, MagicMock
+
+import pytest
+
+from victor_contracts.testing import MockComputeNode, MockToolResult, MockWorkflowContext
+from victor_coding.handlers import HANDLERS, CodeValidationHandler, TestRunnerHandler
+
+
+def registry(result: MockToolResult) -> MagicMock:
+    r = MagicMock()
+    r.execute = AsyncMock(return_value=result)
+    return r
+
+
+class TestCodeValidationHandler:
+    @pytest.mark.asyncio
+    async def test_lint_check_passes_through_shell_command(self):
+        reg = registry(MockToolResult(success=True, output="clean"))
+        ctx = MockWorkflowContext()
+        node = MockComputeNode(
+            node_id="v1",
+            input_mapping={"files": ["a.py"], "checks": ["lint"]},
+            output_key="validation",
+        )
+        result = await CodeValidationHandler()(node, ctx, reg)
+        cmd = reg.execute.call_args.kwargs["command"]
+        assert cmd == "ruff check a.py"
+        assert result.status.value == "completed"
+        assert ctx.get("validation")["all_passed"] is True
+        assert ctx.get("validation")["checks"]["lint"]["passed"] is True
+
+    @pytest.mark.asyncio
+    async def test_failed_check_still_completes_with_all_passed_false(self):
+        """code_validation's fail policy: COMPLETED status, failure in output."""
+        reg = registry(MockToolResult(success=False, error="E501"))
+        ctx = MockWorkflowContext()
+        node = MockComputeNode(node_id="v2", input_mapping={"files": ["b.py"], "checks": ["lint"]})
+        result = await CodeValidationHandler()(node, ctx, reg)
+        assert result.status.value == "completed"
+        assert ctx.get("v2")["all_passed"] is False
+
+    @pytest.mark.asyncio
+    async def test_unknown_check_passes_with_message(self):
+        reg = registry(MockToolResult(success=True))
+        node = MockComputeNode(
+            node_id="v3", input_mapping={"files": ["x.py"], "checks": ["nonexistent"]}
+        )
+        ctx = MockWorkflowContext()
+        result = await CodeValidationHandler()(node, ctx, reg)
+        assert result.status.value == "completed"
+        assert ctx.get("v3")["checks"]["nonexistent"]["passed"] is True
+
+    @pytest.mark.asyncio
+    async def test_exception_is_contained_per_check(self):
+        """Actual behavior: _run_check catches exceptions and reports them as
+        failed checks — the node still COMPLETES (no outer try/except)."""
+        reg = MagicMock()
+        reg.execute = AsyncMock(side_effect=RuntimeError("boom"))
+        node = MockComputeNode(node_id="v4", input_mapping={"files": ["c.py"]})
+        ctx = MockWorkflowContext()
+        result = await CodeValidationHandler()(node, ctx, reg)
+        assert result.status.value == "completed"
+        assert ctx.get("v4")["all_passed"] is False
+        assert "boom" in ctx.get("v4")["checks"]["lint"]["error"]
+
+
+class TestTestRunnerHandler:
+    @pytest.mark.asyncio
+    async def test_pytest_command_with_coverage(self):
+        reg = registry(MockToolResult(success=True, output="3 passed"))
+        node = MockComputeNode(
+            node_id="t1",
+            input_mapping={"test_path": "tests/", "framework": "pytest", "coverage": True},
+            output_key="test_results",
+        )
+        ctx = MockWorkflowContext()
+        result = await TestRunnerHandler()(node, ctx, reg)
+        cmd = reg.execute.call_args.kwargs["command"]
+        assert cmd == "pytest tests/ -v --cov --cov-report=json"
+        assert result.status.value == "completed"
+        assert ctx.get("test_results")["passed"] is True
+
+    @pytest.mark.asyncio
+    async def test_failed_suite_fails_the_node(self):
+        """test_runner's fail policy: FAILED status on suite failure (unlike
+        code_validation — the kernel must parameterize this)."""
+        reg = registry(MockToolResult(success=False, output="1 failed"))
+        node = MockComputeNode(node_id="t2", input_mapping={"framework": "pytest"})
+        ctx = MockWorkflowContext()
+        result = await TestRunnerHandler()(node, ctx, reg)
+        assert result.status.value == "failed"
+
+    @pytest.mark.asyncio
+    async def test_unittest_framework_command(self):
+        reg = registry(MockToolResult(success=True))
+        node = MockComputeNode(node_id="t3", input_mapping={"framework": "unittest"})
+        ctx = MockWorkflowContext()
+        await TestRunnerHandler()(node, ctx, reg)
+        cmd = reg.execute.call_args.kwargs["command"]
+        assert cmd.startswith("python -m unittest discover")
+
+
+class TestRegistration:
+    def test_handlers_table_and_registrar_signature(self):
+        assert set(HANDLERS) == {"code_validation", "test_runner"}
+        # The UI introspects this signature; the kernel must keep it.
+        import inspect
+
+        from victor_coding.handlers import register_handlers
+
+        params = inspect.signature(register_handlers).parameters
+        assert "registrar" in params
+
+    def test_register_via_explicit_registrar(self):
+        entries: Dict[str, Any] = {}
+
+        class Registrar:
+            def __call__(self, name: str, handler: Any) -> None:
+                entries[name] = handler
+
+        from victor_coding.handlers import register_handlers
+
+        register_handlers(Registrar())
+        assert set(entries) == set(HANDLERS)

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_coding_vertical_rl.py

@@ -1,4 +1,4 @@
-# Copyright 2025 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2025 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_coding_vertical_teams.py

@@ -1,4 +1,4 @@
-# Copyright 2025 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2025 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_coding_vertical_workflows.py

@@ -1,4 +1,4 @@
-# Copyright 2025 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2025 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -285,3 +285,41 @@ class TestWorkflowIntegration:
                     # Most agent nodes should have allowed_tools
                     # (not all require it, but most do)
                     pass  # Just ensure no crashes
+
+
+class TestSharedEscapeHatches:
+    """Generic hatches come from the framework registry, not a local copy."""
+
+    SHARED = {
+        "complexity_check": "condition",
+        "complexity_assessment": "condition",
+        "format_implementation_plan": "transform",
+    }
+
+    def test_provider_resolves_the_shared_hatches(self):
+        config = CodingWorkflowProvider()._get_config()
+        for name, kind in self.SHARED.items():
+            registry = (
+                config.condition_registry if kind == "condition" else config.transform_registry
+            )
+            assert name in registry, f"{name} missing from the coding workflow {kind}s"
+            assert callable(registry[name])
+
+    def test_refactor_workflow_still_routes_on_complexity(self):
+        """refactor.yaml branches on complexity_assessment."""
+        config = CodingWorkflowProvider()._get_config()
+        assess = config.condition_registry["complexity_assessment"]
+        assert assess({"files_affected": list(range(30))}) in {
+            "high",
+            "complex",
+            "medium",
+            "low",
+            "simple",
+        }
+
+    def test_not_redefined_locally(self):
+        import victor_coding.escape_hatches as local
+
+        for name in self.SHARED:
+            assert not hasattr(local, name), f"{name} copied back into victor_coding.escape_hatches"
+        assert not set(self.SHARED) & (set(local.CONDITIONS) | set(local.TRANSFORMS))

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_enhanced_integration.py

@@ -1,4 +1,4 @@
-# Copyright 2025 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2025 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -300,7 +300,7 @@ class TestEnhancedCodingConversationManager:
         manager = EnhancedCodingConversationManager()
 
         assert manager._conversation_coordinator is not None
-        assert isinstance(manager._coding_context, CodingContext)
+        assert isinstance(manager.get_coding_context(), CodingContext)
 
     def test_add_message(self):
         """Test adding a message."""

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_extension_cache.py

@@ -1,4 +1,4 @@
-# Copyright 2025 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2025 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

{victor_coding-0.7.2 → victor_coding-0.14.0}/tests/test_framework_lsp_adapter.py

@@ -1,4 +1,4 @@
-# Copyright 2026 Vijaykumar Singh <singhvjd@gmail.com>
+# Copyright 2026 Vijaykumar Singh <vijay@anvaiops.com>
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

victor_coding-0.14.0/tests/test_graph_backend_marker.py

@@ -0,0 +1,70 @@
+# Copyright 2026 Vijaykumar Singh <vijay@anvaiops.com>
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+"""The graph tool and index shim must honor `.victor/graph_backend`.
+
+Previously: the `graph` tool probed project.db's sqlite tables (dropping the
+tool from the LLM schema on marker'd repos whose graph lives elsewhere) and
+hardcoded SqliteGraphStore; the indexer accepted `graph_store_name` and
+silently dropped it; the shim defaulted to sqlite. A repo flipped to another
+backend got sqlite everywhere.
+"""
+
+from pathlib import Path
+
+from victor_coding.tools.graph_tool import (
+    _project_graph_has_data,
+    _resolved_graph_backend,
+)
+
+
+def _mark(tmp_path: Path, backend: str) -> Path:
+    marker_dir = tmp_path / ".victor"
+    marker_dir.mkdir(exist_ok=True)
+    (marker_dir / "graph_backend").write_text(backend)
+    return tmp_path
+
+
+def test_resolved_backend_reads_marker(tmp_path: Path):
+    assert _resolved_graph_backend(tmp_path) == "sqlite"
+    _mark(tmp_path, "proxima")
+    assert _resolved_graph_backend(tmp_path) == "proxima"
+
+
+def test_has_data_advertises_tool_on_marked_repos(tmp_path: Path):
+    # A proxima-marked repo must not be probed via project.db's sqlite
+    # tables (which are legitimately empty there) — the tool stays
+    # advertised and the load path constructs the real store.
+    _mark(tmp_path, "proxima")
+    assert _project_graph_has_data(tmp_path) is True
+
+
+def test_shim_default_is_auto():
+    import inspect
+
+    from victor_coding.codebase.graph.registry import create_graph_store
+
+    sig = inspect.signature(create_graph_store)
+    assert sig.parameters["name"].default == "auto"
+
+
+def test_indexer_threads_graph_store_name():
+    import inspect
+
+    from victor_coding.codebase.indexer import CodebaseIndex
+
+    src = inspect.getsource(CodebaseIndex.__init__)
+    assert (
+        'name=graph_store_name or "auto"' in src
+    ), "graph_store_name must reach create_graph_store, not be dropped"

victor_coding-0.14.0/tests/test_personas.py

@@ -0,0 +1,143 @@
+# Copyright 2026 Vijaykumar Singh <vijay@anvaiops.com>
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+"""Pre-move characterization for victor-coding personas (dedup F6).
+
+Coding was the only personas vertical with NO test file. These pin the
+observable contract before the kernel extraction: inventory, trait
+normalization, prompt hints, backstory shape, helper semantics, and explicit-provider
+registration (registration tuples included). Import-time registration was
+removed by FEP-0042.
+"""
+
+from victor_contracts.testing.personas import (
+    assert_explicit_persona_registration,
+    assert_import_registers_no_personas,
+)
+from victor_coding.teams.personas import (
+    CODING_PERSONAS,
+    CodingPersonaTraits,
+    CommunicationStyle,
+    DecisionStyle,
+    apply_persona_to_spec,
+    get_persona,
+    get_persona_by_expertise,
+    get_personas_for_role,
+    list_personas,
+)
+
+
+class TestInventory:
+    def test_eight_personas_with_expected_ids(self):
+        assert set(CODING_PERSONAS) == {
+            "code_archaeologist",
+            "security_auditor",
+            "architect",
+            "refactoring_strategist",
+            "craftsman",
+            "debugger",
+            "quality_guardian",
+            "test_specialist",
+        }
+
+    def test_every_persona_complete(self):
+        for pid, persona in CODING_PERSONAS.items():
+            assert persona.name, pid
+            assert persona.role in ("researcher", "planner", "executor", "reviewer"), pid
+            assert persona.expertise, pid
+            assert persona.strengths, pid
+            assert persona.approach, pid
+
+
+class TestTraits:
+    def test_trait_scales_normalized(self):
+        traits = CodingPersonaTraits(attention_to_detail=2.0, risk_tolerance=-1.0, verbosity=0.5)
+        # values are stored as given (no clamping in the dataclass)
+        assert traits.attention_to_detail == 2.0
+        assert traits.risk_tolerance == -1.0
+
+    def test_prompt_hints_composition(self):
+        traits = CodingPersonaTraits(
+            communication_style=CommunicationStyle.CONCISE,
+            decision_style=DecisionStyle.CONSERVATIVE,
+            attention_to_detail=0.9,
+            risk_tolerance=0.1,
+        )
+        hints = traits.to_prompt_hints()
+        assert "brief" in hints.lower()
+        assert "proven" in hints.lower()
+        assert "edge cases" in hints.lower()
+        assert "thorough testing" in hints.lower()
+
+    def test_framework_traits_mapping(self):
+        traits = CodingPersonaTraits(attention_to_detail=0.8, risk_tolerance=0.4, verbosity=0.6)
+        fw = traits.to_framework_traits(name="X", role="executor", description="d", strengths=["s"])
+        assert fw.risk_tolerance == 0.4
+        assert fw.verbosity == 0.6
+        # coding's mapping: creativity = 1 - attention_to_detail
+        assert abs(fw.creativity - 0.2) < 1e-9
+        assert fw.custom_traits["decision_style"]
+        assert fw.custom_traits["attention_to_detail"] == 0.8
+
+
+class TestPersonaSemantics:
+    def test_get_expertise_list_merges_primary_and_secondary(self):
+        p = get_persona("architect")
+        lst = p.get_expertise_list()
+        assert lst
+        assert all(isinstance(e, str) for e in lst)
+
+    def test_generate_backstory_shape(self):
+        story = get_persona("debugger").generate_backstory()
+        assert story.startswith("You are ")
+        assert "Bug Hunter" in story
+        assert story.endswith(".")
+
+    def test_helpers(self):
+        assert set(list_personas()) == set(CODING_PERSONAS)
+        planners = get_personas_for_role("planner")
+        assert planners
+        assert all(p.role == "planner" for p in planners)
+        found = get_persona_by_expertise(CODING_PERSONAS["architect"].expertise[0])
+        assert found
+
+    def test_apply_persona_to_spec_merges(self):
+        from victor_contracts import TeamMemberSpec
+
+        spec = TeamMemberSpec(role="planner", goal="ship", name="a1")
+        out = apply_persona_to_spec(spec, "architect")
+        # documented contract: same object, enhanced in place
+        assert out is spec
+        assert getattr(spec, "expertise", None) or getattr(spec, "backstory", None)
+        # unknown persona: unchanged spec returned
+        spec2 = TeamMemberSpec(role="planner", goal="ship", name="a2")
+        assert apply_persona_to_spec(spec2, "no_such_persona") is spec2
+
+
+class TestPersonaRegistration:
+    """FEP-0042: personas reach the host through ``get_personas()`` on activation;
+    nothing registers at import, and the explicit-provider helper still works."""
+
+    def test_teams_import_registers_nothing_and_emits_no_deprecation(self):
+        assert_import_registers_no_personas("victor_coding.teams", "victor_coding.teams.personas")
+
+    def test_explicit_provider_helper_registers_every_persona_without_warning(self):
+        from victor_coding.teams.personas import _register_personas_with_framework
+
+        calls = assert_explicit_persona_registration(
+            _register_personas_with_framework, CODING_PERSONAS, "coding"
+        )
+        debugger = next(c for c in calls if c["name"] == "debugger")
+        assert debugger["category"] == "execution"
+        assert debugger["tags"] == ["debugging", "troubleshooting", "root-cause"]