q792602257andClaude Sonnet 5
61ff3b425d
Replaces the stub Planner's fixed describe_screen/[] behavior with a real decision-maker: AIPlanner uses native tool/function calling (Anthropic or OpenAI, pluggable via AI_PLANNER_PROVIDER) to select exactly one grounded action per turn, with an explicit finish_task(success, reason) tool for completion/failure instead of an ambiguous "no tool call" signal. Default disabled (AI_PLANNER_ENABLED=false) and additive; TaskRunner falls back to the existing stub Planner unchanged when disabled. Amends CONSTITUTION.md's Perception Boundary with one narrow exception: only the AI Planner may receive the current step's raw screenshot bytes alongside Scene, for vision-grounded coordinate grounding. Also fixes a latent gap in TaskRunner.run(): observe/plan exceptions are now caught per iteration and turned into a failed task with a failure_reason, instead of propagating uncaught. openspec change: ai-planner-runtime. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
64 lines
1.8 KiB
Python
64 lines
1.8 KiB
Python
from __future__ import annotations
|
|
|
|
import os
|
|
from collections.abc import Mapping
|
|
from dataclasses import dataclass
|
|
|
|
DEFAULT_PROVIDER = "anthropic"
|
|
DEFAULT_MODEL_BY_PROVIDER = {
|
|
"anthropic": "claude-sonnet-5",
|
|
"openai": "gpt-5.6",
|
|
}
|
|
DEFAULT_TIMEOUT_SECONDS = 30.0
|
|
|
|
ENABLED_ENV = "AI_PLANNER_ENABLED"
|
|
PROVIDER_ENV = "AI_PLANNER_PROVIDER"
|
|
MODEL_ENV = "AI_PLANNER_MODEL"
|
|
TIMEOUT_ENV = "AI_PLANNER_TIMEOUT_SECONDS"
|
|
|
|
SUPPORTED_PROVIDERS = frozenset(DEFAULT_MODEL_BY_PROVIDER)
|
|
|
|
|
|
@dataclass(frozen=True)
|
|
class PlannerConfig:
|
|
enabled: bool = False
|
|
provider: str = DEFAULT_PROVIDER
|
|
model: str = ""
|
|
timeout: float = DEFAULT_TIMEOUT_SECONDS
|
|
|
|
def resolved_model(self) -> str:
|
|
return self.model or DEFAULT_MODEL_BY_PROVIDER[self.provider]
|
|
|
|
|
|
def load_config(env: Mapping[str, str] | None = None) -> PlannerConfig:
|
|
values = env or os.environ
|
|
return PlannerConfig(
|
|
enabled=_parse_bool(values.get(ENABLED_ENV), default=False),
|
|
provider=_parse_provider(values.get(PROVIDER_ENV)),
|
|
model=values.get(MODEL_ENV) or "",
|
|
timeout=_parse_timeout(values.get(TIMEOUT_ENV)),
|
|
)
|
|
|
|
|
|
def _parse_bool(value: str | None, *, default: bool) -> bool:
|
|
if value is None:
|
|
return default
|
|
return value.strip().lower() in {"1", "true", "yes", "on", "enabled"}
|
|
|
|
|
|
def _parse_provider(value: str | None) -> str:
|
|
if value is None:
|
|
return DEFAULT_PROVIDER
|
|
provider = value.strip().lower()
|
|
return provider if provider in SUPPORTED_PROVIDERS else DEFAULT_PROVIDER
|
|
|
|
|
|
def _parse_timeout(value: str | None) -> float:
|
|
if value is None:
|
|
return DEFAULT_TIMEOUT_SECONDS
|
|
try:
|
|
timeout = float(value)
|
|
except ValueError:
|
|
return DEFAULT_TIMEOUT_SECONDS
|
|
return timeout if timeout > 0 else DEFAULT_TIMEOUT_SECONDS
|