victor-coding 0.5.7__tar.gz → 0.7.1__tar.gz

{victor_coding-0.5.7 → victor_coding-0.7.1}/PKG-INFO

@@ -1,10 +1,10 @@
 Metadata-Version: 2.4
 Name: victor-coding
-Version: 0.5.7
+Version: 0.7.1
 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>
-License-Expression: Apache-2.0
+License: Apache-2.0
 Project-URL: Homepage, https://github.com/vjsingh1984/victor-coding
 Project-URL: Repository, https://github.com/vjsingh1984/victor-coding
 Project-URL: Documentation, https://docs.victor.dev/verticals/coding
@@ -13,21 +13,34 @@ 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.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: Software Development
-Requires-Python: >=3.10
+Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: victor-ai>=0.5.6
+Requires-Dist: victor-contracts[yaml]<1.0,>=0.6.0
+Requires-Dist: numpy<2.3,>=1.24
+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.6.0; extra == "runtime"
+Provides-Extra: native
+Provides-Extra: proximadb
+Requires-Dist: proximadb<0.3,>=0.2; 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: 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: ruff>=0.5; extra == "dev"
 Requires-Dist: mypy>=1.10; extra == "dev"
+Provides-Extra: native-dev
+Requires-Dist: maturin<2.0,>=1.0; extra == "native-dev"
 Dynamic: license-file
 
 # victor-coding

victor_coding-0.7.1/pyproject.toml

@@ -0,0 +1,200 @@
+[build-system]
+requires = [
+    "setuptools>=61.0",
+    "wheel",
+]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "victor-coding"
+version = "0.7.1"
+description = "Coding vertical for Victor AI - Software development, debugging, and code generation"
+readme = "README.md"
+requires-python = ">=3.11"
+license = {text = "Apache-2.0"}
+authors = [
+    { name = "Vijaykumar Singh", email = "singhvjd@gmail.com" },
+]
+maintainers = [
+    { name = "Vijaykumar Singh", email = "singhvjd@gmail.com" },
+]
+keywords = [
+    "victor",
+    "vertical",
+    "coding",
+    "development",
+    "debugging",
+    "refactoring",
+]
+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-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
+]
+
+[project.optional-dependencies]
+runtime = [
+    "victor-ai>=0.6.0,<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
+]
+# 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 = [
+    "proximadb>=0.2,<0.3",
+]
+# Shared code->CPG chunker (ADR-014). Opt-in: when installed AND
+# VICTOR_CODING_USE_CODEGRAPH=1, CodeChunker delegates to it. Published on PyPI
+# (`pip install victor-codegraph`); for local source work: pip install -e ../../victor-codegraph
+codegraph = [
+    "victor-codegraph>=0.1.0,<1.0",
+]
+dev = [
+    "pytest>=8.0",
+    "pytest-asyncio>=0.23",
+    "pytest-cov>=4.1",
+    "pytest-xdist>=3.0",
+    "pytest-benchmark>=5.0",
+    "black==26.1.0",
+    "ruff>=0.5",
+    "mypy>=1.10",
+]
+native-dev = [
+    "maturin>=1.0,<2.0",  # Build Rust extensions
+]
+
+[project.urls]
+Homepage = "https://github.com/vjsingh1984/victor-coding"
+Repository = "https://github.com/vjsingh1984/victor-coding"
+Documentation = "https://docs.victor.dev/verticals/coding"
+Issues = "https://github.com/vjsingh1984/victor-coding/issues"
+
+[project.entry-points."victor.plugins"]
+coding = "victor_coding.plugin:plugin"
+
+[project.entry-points."victor.tool_dependencies"]
+coding = "victor_coding.tool_dependencies:get_provider"
+
+[project.entry-points."victor.safety_rules"]
+coding = "victor_coding.safety:create_all_coding_safety_rules"
+
+[project.entry-points."victor.api_routers"]
+coding = "victor_coding.api.router_provider:get_fastapi_router_provider"
+
+[project.entry-points."victor.prompt_contributors"]
+coding = "victor_coding.prompts:CodingPromptContributor"
+
+[project.entry-points."victor.mode_configs"]
+coding = "victor_coding.mode_config:CodingModeConfigProvider"
+
+[project.entry-points."victor.workflow_providers"]
+coding = "victor_coding.workflows.provider:CodingWorkflowProvider"
+
+[project.entry-points."victor.capability_providers"]
+coding = "victor_coding.capabilities:CodingCapabilityProvider"
+
+[project.entry-points."victor.service_providers"]
+coding = "victor_coding.service_provider:CodingServiceProvider"
+coding-performance = "victor_coding.performance.service_provider:PerformanceServiceProvider"
+
+[project.entry-points."victor.team_spec_providers"]
+coding = "victor_coding.teams:CodingTeamSpecProvider"
+
+# Victor extension protocol implementations
+[project.entry-points."victor.extension.protocols"]
+coding-tools = "victor_coding.protocols:CodingToolProvider"
+coding-safety = "victor_coding.protocols:CodingSafetyProvider"
+coding-prompts = "victor_coding.protocols:CodingPromptProvider"
+coding-workflows = "victor_coding.protocols:CodingWorkflowProvider"
+
+# Victor extension capability providers
+[project.entry-points."victor.extension.capabilities"]
+coding-lsp = "victor_coding.protocols:LSPCapabilityProvider"
+coding-git = "victor_coding.protocols:GitCapabilityProvider"
+coding-style = "victor_coding.protocols:CodeStyleCapabilityProvider"
+coding-test = "victor_coding.protocols:TestingCapabilityProvider"
+coding-refactor = "victor_coding.protocols:RefactoringCapabilityProvider"
+
+# Extended providers: sandbox isolation, permission hierarchy, hooks, compaction
+[project.entry-points."victor.sandbox.providers"]
+coding-sandbox = "victor_coding.protocols:CodingSandboxProvider"
+
+[project.entry-points."victor.permission.providers"]
+coding-permissions = "victor_coding.protocols:CodingPermissionProvider"
+
+[project.entry-points."victor.hook.providers"]
+coding-hooks = "victor_coding.protocols:CodingHookProvider"
+
+[project.entry-points."victor.compaction.providers"]
+coding-compaction = "victor_coding.protocols:CodingCompactionProvider"
+
+[tool.setuptools]
+packages = [
+    "victor_coding",
+    "victor_coding.api",
+    "victor_coding.codebase",
+    "victor_coding.completion",
+    "victor_coding.coverage",
+    "victor_coding.docgen",
+    "victor_coding.editing",
+    "victor_coding.languages",
+    "victor_coding.lsp",
+    "victor_coding.refactor",
+    "victor_coding.review",
+    "victor_coding.rl",
+    "victor_coding.teams",
+    "victor_coding.testgen",
+    "victor_coding.tools",
+    "victor_coding.workflows",
+]
+
+[tool.black]
+line-length = 100
+target-version = [
+    "py311",
+    "py312",
+]
+
+[tool.ruff]
+line-length = 100
+target-version = "py311"
+
+[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
+    "B019",  # lru_cache on methods (valid pattern for class methods)
+    "B023",  # function uses loop variable (common pattern)
+    "B024",  # abstract class with no abstract methods (valid for protocols)
+    "B025",  # try-except with empty except clause (handled elsewhere)
+    "B904",  # allow broad raising for now
+    "B905",  # zip() without strict= parameter (not critical)
+    "C401",  # unnecessary generator (minor optimization)
+    "C414",  # unnecessary list/set call (minor optimization)
+]
+
+[tool.ruff.lint.per-file-ignores]
+"__init__.py" = ["F401"]
+"tests/**/*.py" = ["B017", "B018", "E741", "F841"]
+[tool.mypy]
+python_version = "3.11"
+warn_return_any = true

victor_coding-0.7.1/tests/test_api_router_provider.py

@@ -0,0 +1,46 @@
+"""Tests for victor-coding FastAPI router provider."""
+
+import pytest
+
+# Check if FastAPI is available
+try:
+    import fastapi  # noqa: F401
+
+    FASTAPI_AVAILABLE = True
+except ImportError:
+    FASTAPI_AVAILABLE = False
+
+# Only import if FastAPI is available
+if FASTAPI_AVAILABLE:
+    from victor_coding.api.router_provider import (
+        create_coding_lsp_router,
+        get_fastapi_router_provider,
+    )
+else:
+    # Create dummy imports for pytestmark
+    create_coding_lsp_router = None
+    get_fastapi_router_provider = None
+
+pytestmark = pytest.mark.skipif(
+    not FASTAPI_AVAILABLE, reason="FastAPI not installed. Install with: pip install fastapi"
+)
+
+
+def _route_paths(app) -> set[str]:
+    return {getattr(route, "path", "") for route in app.routes}
+
+
+def test_create_coding_lsp_router_registers_expected_routes() -> None:
+    router = create_coding_lsp_router()
+    paths = _route_paths(router)
+
+    assert "/lsp/completions" in paths
+    assert "/lsp/hover" in paths
+    assert "/lsp/definition" in paths
+    assert "/lsp/references" in paths
+    assert "/lsp/diagnostics" in paths
+
+
+def test_entry_point_provider_returns_router() -> None:
+    router = get_fastapi_router_provider(workspace_root="/tmp")
+    assert "/lsp/completions" in _route_paths(router)

{victor_coding-0.5.7 → victor_coding-0.7.1}/tests/test_coding_vertical_rl.py

@@ -47,7 +47,7 @@ class TestCodingRLConfig:
         assert "feature" in config.task_type_mappings
 
         # Check tools are lists
-        for task_type, tools in config.task_type_mappings.items():
+        for _task_type, tools in config.task_type_mappings.items():
             assert isinstance(tools, list)
             assert len(tools) > 0
 
@@ -55,7 +55,7 @@ class TestCodingRLConfig:
         """Test quality thresholds are reasonable."""
         config = CodingRLConfig()
 
-        for task_type, threshold in config.quality_thresholds.items():
+        for _task_type, threshold in config.quality_thresholds.items():
             assert 0.0 <= threshold <= 1.0
 
         # Refactoring should have highest threshold
@@ -69,7 +69,7 @@ class TestCodingRLConfig:
         config = CodingRLConfig()
 
         # All patience values should be positive
-        for provider, patience in config.default_patience.items():
+        for _provider, patience in config.default_patience.items():
             assert patience > 0
 
         # Local models should have more patience

{victor_coding-0.5.7 → victor_coding-0.7.1}/tests/test_coding_vertical_teams.py

@@ -14,10 +14,7 @@
 
 """Tests for coding vertical teams integration."""
 
-import pytest
-
-from victor.framework.teams import TeamFormation
-from victor.framework.team_schema import TeamSpec
+from victor_contracts.team_schema import TeamFormation, TeamSpec
 from victor_coding.teams import (
     CodingRoleConfig,
     CODING_ROLES,
@@ -43,7 +40,7 @@ class TestCodingRoles:
 
     def test_role_config_structure(self):
         """Test role configurations have required fields."""
-        for name, config in CODING_ROLES.items():
+        for _name, config in CODING_ROLES.items():
             assert isinstance(config, CodingRoleConfig)
             assert config.base_role in ["researcher", "planner", "executor", "reviewer"]
             assert isinstance(config.tools, list)
@@ -81,7 +78,7 @@ class TestCodingTeamSpecs:
 
     def test_team_spec_structure(self):
         """Test team specs have required fields."""
-        for name, spec in CODING_TEAM_SPECS.items():
+        for _name, spec in CODING_TEAM_SPECS.items():
             assert isinstance(spec, TeamSpec)
             assert spec.name
             assert spec.description
@@ -194,7 +191,7 @@ class TestTeamMemberSpecs:
 
     def test_member_budgets_reasonable(self):
         """Test member tool budgets are reasonable."""
-        for name, spec in CODING_TEAM_SPECS.items():
+        for _name, spec in CODING_TEAM_SPECS.items():
             total_member_budget = sum(m.tool_budget or 0 for m in spec.members)
             # Individual budgets should sum to around team total
             # (allow some flexibility)

{victor_coding-0.5.7 → victor_coding-0.7.1}/tests/test_coding_vertical_workflows.py

@@ -17,7 +17,7 @@
 import pytest
 
 from victor_coding.workflows import CodingWorkflowProvider
-from victor.workflows.definition import (
+from victor.framework.extensions import (
     AgentNode,
     ConditionNode,
     ParallelNode,
@@ -208,7 +208,7 @@ class TestBugFixWorkflow:
         wf = bug_fix_workflow
 
         # Check for condition node (workflow may have varying structures)
-        has_condition = any(isinstance(node, ConditionNode) for node in wf.nodes.values())
+        any(isinstance(node, ConditionNode) for node in wf.nodes.values())
         # Some bug fix workflows use simple linear flow
         assert len(wf.nodes) >= 3  # At minimum has investigate, fix, verify
 
@@ -228,7 +228,7 @@ class TestCodeReviewWorkflow:
         wf = code_review_workflow
 
         # Check for parallel node (some workflows may use sequential)
-        has_parallel = any(isinstance(node, ParallelNode) for node in wf.nodes.values())
+        any(isinstance(node, ParallelNode) for node in wf.nodes.values())
         # Parallel is optional - workflow may use sequential reviews
         assert len(wf.nodes) >= 2
 
@@ -268,7 +268,7 @@ class TestWorkflowIntegration:
         provider = CodingWorkflowProvider()
         workflows = provider.get_workflows()
 
-        for name, wf in workflows.items():
+        for _name, wf in workflows.items():
             d = wf.to_dict()
             assert d["name"] == wf.name
             assert "nodes" in d
@@ -279,8 +279,8 @@ class TestWorkflowIntegration:
         provider = CodingWorkflowProvider()
         workflows = provider.get_workflows()
 
-        for name, wf in workflows.items():
-            for node_id, node in wf.nodes.items():
+        for _name, wf in workflows.items():
+            for _node_id, node in wf.nodes.items():
                 if isinstance(node, AgentNode):
                     # Most agent nodes should have allowed_tools
                     # (not all require it, but most do)