steerdev 1.1.44__tar.gz → 1.1.45__tar.gz

{steerdev-1.1.44 → steerdev-1.1.45}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: steerdev
-Version: 1.1.44
+Version: 1.1.45
 Summary: Backend task runner for steerdev.com - orchestrates CLI coding agents with activity reporting
 Project-URL: Homepage, https://github.com/pentoai/steerdev-agent
 Project-URL: Repository, https://github.com/pentoai/steerdev-agent

{steerdev-1.1.44 → steerdev-1.1.45}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "steerdev"
-version = "1.1.44"
+version = "1.1.45"
 description = "Backend task runner for steerdev.com - orchestrates CLI coding agents with activity reporting"
 readme = "README.md"
 authors = [

{steerdev-1.1.44 → steerdev-1.1.45}/src/steerdev_agent/cli.py

@@ -1654,6 +1654,18 @@ def run(
             envvar="STEERDEV_AGENT_ID",
         ),
     ] = None,
+    provider: Annotated[
+        str | None,
+        typer.Option(
+            "--provider",
+            help=(
+                "AI provider / executor type to run tasks with (e.g. claude)."
+                " Overrides steerdev.yaml executor.type."
+                " Reads from STEERDEV_PROVIDER env var."
+            ),
+            envvar="STEERDEV_PROVIDER",
+        ),
+    ] = None,
     model: Annotated[
         str | None,
         typer.Option(
@@ -1815,7 +1827,11 @@ def run(
         steerdev run --project-id abc123 --workflow-id wf-abc123
         steerdev run --config custom.yaml --project-id abc123
     """
-    from steerdev_agent.runner import resolve_max_tasks, run_agent
+    from steerdev_agent.runner import (
+        resolve_executor_config,
+        resolve_max_tasks,
+        run_agent,
+    )
 
     # Get config from context (loaded in app callback)
     config: SteerDevConfig = ctx.obj or SteerDevConfig()
@@ -1844,6 +1860,12 @@ def run(
     resolved_evidence_config = config.evidence.model_copy()
     if evidence is not None:
         resolved_evidence_config.enabled = evidence
+    # Resolve executor config: --provider/STEERDEV_PROVIDER overrides executor.type
+    try:
+        resolved_executor_config = resolve_executor_config(config.executor, provider)
+    except ValueError as e:
+        console.print(f"[red]Error: {e}[/red]")
+        raise typer.Exit(1) from e
 
     # API key: CLI > env (via envvar) > config env var
     resolved_api_key = api_key
@@ -1875,6 +1897,7 @@ def run(
             f"Project ID: {resolved_project_id}\n"
             f"Agent Name: {agent_name_display}\n"
             f"Working Directory: {working_dir or 'current'}\n"
+            f"Provider: {resolved_executor_config.type}\n"
             f"Model: {resolved_model or 'default'}\n"
             f"Max Tasks: {max_tasks_display}\n"
             f"Timeout: {resolved_timeout}s\n"
@@ -1921,6 +1944,7 @@ def run(
                 enable_canals=canals,
                 worktree_config=resolved_worktree_config,
                 evidence_config=resolved_evidence_config,
+                executor_config=resolved_executor_config,
                 force_workflow_id=resolved_workflow_id,
                 dry_run=dry_run,
                 retry_config=retry_config,

{steerdev-1.1.44 → steerdev-1.1.45}/src/steerdev_agent/executor/__init__.py

@@ -12,6 +12,7 @@ if TYPE_CHECKING:
     from steerdev_agent.config.models import ExecutorConfig
 
 __all__ = [
+    "SUPPORTED_EXECUTOR_TYPES",
     "AgentExecutor",
     "ClaudeExecutor",
     "ExecutorFactory",
@@ -19,6 +20,10 @@ __all__ = [
     "StreamParser",
 ]
 
+# Executor types ExecutorFactory can build. `steerdev run --provider` and
+# steerdev.yaml's executor.type are validated against this.
+SUPPORTED_EXECUTOR_TYPES: tuple[str, ...] = ("claude",)
+
 
 class ExecutorFactory:
     """Factory for creating agent executors based on configuration."""
@@ -62,4 +67,7 @@ class ExecutorFactory:
                 extra_args=config.args or None,
             )
 
-        raise ValueError(f"Unsupported executor type: {config.type}")
+        raise ValueError(
+            f"Unsupported executor type: {config.type} "
+            f"(supported: {', '.join(SUPPORTED_EXECUTOR_TYPES)})"
+        )

{steerdev-1.1.44 → steerdev-1.1.45}/src/steerdev_agent/runner.py

@@ -26,7 +26,7 @@ from steerdev_agent.config.models import (
     RetryConfig,
     WorktreeConfig,
 )
-from steerdev_agent.executor import ExecutorFactory
+from steerdev_agent.executor import SUPPORTED_EXECUTOR_TYPES, ExecutorFactory
 from steerdev_agent.executor.base import AgentExecutor, EventType, StreamEvent
 from steerdev_agent.executor.claude import ClaudeExecutorError
 from steerdev_agent.git.commit import commit_task_boundary
@@ -165,6 +165,29 @@ def resolve_max_tasks(max_tasks: int | None, wave_id: str | None) -> int:
     return 0 if wave_id else 1
 
 
+def resolve_executor_config(
+    base: ExecutorConfig,
+    provider: str | None,
+) -> ExecutorConfig:
+    """Resolve the effective executor config for a run.
+
+    - `provider` (from --provider / STEERDEV_PROVIDER) overrides the config
+      file's executor.type; None keeps it.
+    - The resolved type is validated against SUPPORTED_EXECUTOR_TYPES so an
+      unsupported provider fails fast (before task claiming), not deep inside
+      ExecutorFactory mid-run.
+    """
+    resolved = base.model_copy()
+    if provider is not None:
+        resolved.type = provider
+    if resolved.type not in SUPPORTED_EXECUTOR_TYPES:
+        raise ValueError(
+            f"Unsupported provider: {resolved.type} "
+            f"(supported: {', '.join(SUPPORTED_EXECUTOR_TYPES)})"
+        )
+    return resolved
+
+
 def resolve_merge_flow(
     waves_enabled: bool,
     canals_enabled: bool,

steerdev-1.1.45/tests/test_resolve_executor_config.py

@@ -0,0 +1,85 @@
+"""Tests for resolve_executor_config: the `steerdev run --provider` override.
+
+The sandbox bootstrap passes --provider "$STEERDEV_PROVIDER" so the web app
+can choose the executor per dispatch. Resolution: CLI/env > config file >
+default ("claude"). Unsupported providers must fail fast with a clear error,
+before any network or agent work starts.
+"""
+
+import pytest
+from typer.testing import CliRunner
+
+from steerdev_agent.cli import app
+from steerdev_agent.config.models import ExecutorConfig
+from steerdev_agent.executor import SUPPORTED_EXECUTOR_TYPES
+from steerdev_agent.runner import resolve_executor_config
+
+runner = CliRunner()
+
+
+def test_no_provider_keeps_config_type():
+    # Without --provider, the config file's executor.type is preserved.
+    base = ExecutorConfig(type="claude", permission_mode="plan")
+    resolved = resolve_executor_config(base, None)
+    assert resolved.type == "claude"
+    assert resolved.permission_mode == "plan"
+
+
+def test_provider_overrides_config_type():
+    resolved = resolve_executor_config(ExecutorConfig(), "claude")
+    assert resolved.type == "claude"
+
+
+def test_does_not_mutate_base_config():
+    base = ExecutorConfig()
+    resolve_executor_config(base, "claude")
+    assert base.type == "claude"
+    resolved = resolve_executor_config(base, "claude")
+    assert resolved is not base
+
+
+def test_unsupported_provider_raises_with_supported_list():
+    with pytest.raises(ValueError, match="codex"):
+        resolve_executor_config(ExecutorConfig(), "codex")
+    with pytest.raises(ValueError, match="claude"):
+        # The error must name the supported types so the sandbox log is actionable.
+        resolve_executor_config(ExecutorConfig(), "nonsense")
+
+
+def test_unsupported_config_type_raises_even_without_provider():
+    # A bad executor.type in steerdev.yaml should fail fast too, not deep in
+    # ExecutorFactory after task claiming.
+    with pytest.raises(ValueError, match="nonsense"):
+        resolve_executor_config(ExecutorConfig(type="nonsense"), None)
+
+
+def test_supported_types_include_claude():
+    assert "claude" in SUPPORTED_EXECUTOR_TYPES
+
+
+def test_cli_run_help_lists_provider():
+    result = runner.invoke(app, ["run", "--help"])
+    assert result.exit_code == 0
+    assert "--provider" in result.output
+
+
+def test_cli_run_rejects_unsupported_provider_before_running():
+    # Must exit non-zero with an actionable message, without contacting the API.
+    result = runner.invoke(
+        app,
+        ["run", "--project-id", "proj-x", "--provider", "nonsense"],
+    )
+    assert result.exit_code == 1
+    assert "nonsense" in result.output
+    assert "claude" in result.output
+
+
+def test_cli_run_reads_provider_from_env():
+    # The sandbox exports STEERDEV_PROVIDER; the option must honor it.
+    result = runner.invoke(
+        app,
+        ["run", "--project-id", "proj-x"],
+        env={"STEERDEV_PROVIDER": "nonsense"},
+    )
+    assert result.exit_code == 1
+    assert "nonsense" in result.output