Skip to content

Generalize CodexAgentBackend → LLMAgentBackend (#234)#362

Closed
larstalian wants to merge 1 commit into
mainfrom
generalize-codex-agent-backend-234
Closed

Generalize CodexAgentBackend → LLMAgentBackend (#234)#362
larstalian wants to merge 1 commit into
mainfrom
generalize-codex-agent-backend-234

Conversation

@larstalian

Copy link
Copy Markdown
Collaborator

Summary

CodexAgentBackend (src/openrange/agent_backend.py) had no codex-specific code — it just wraps an LLMBackend into a single-shot, tool-less AgentBackend, defaulting to CodexBackend when no backend is passed. The "Codex" in the name was misleading, since the only codex-specific code lives in CodexBackend (src/openrange/llm.py).

This renames the class to LLMAgentBackend so "Codex" appears in exactly one place.

  • Clean rename, no backward-compat alias (repo is pre-1.0; an alias would defeat the point of Generalize CodexAgentBackend — it's not actually codex-specific #234).
  • The CodexBackend default-fallback behavior is kept intact — the constructor still defaults to CodexBackend(model=model) when no backend is given. Only the error strings and docstrings were updated to say LLMAgentBackend.
  • Updated every reference: agent_backend.py, __init__.py (import + __all__), tests/test_agent_backend.py, tests/test_office_persona.py (including the tool-injection error-message assertions), examples/codex_eval.py, packages/openrange-pack-sdk/_protocols.py, and packages/openrange-pack-sdk/README.md.
  • Re-grepped CodexAgentBackend across the repo: zero hits remaining.

Closes #234

Test plan

  • ruff check on impacted files — clean
  • mypy on src/openrange/agent_backend.py + src/openrange/__init__.py — clean
  • pytest tests/test_agent_backend.py tests/test_office_persona.py — 29 passed, 2 skipped (strands optional dep)

🤖 Generated with Claude Code

CodexAgentBackend had no codex-specific code; it just wraps an LLMBackend
into a single-shot, tool-less AgentBackend, defaulting to CodexBackend
when none is passed. Rename it to LLMAgentBackend so "Codex" appears in
exactly one place (CodexBackend). The CodexBackend default fallback is
kept intact. No backward-compat alias (pre-1.0).

Closes #234

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@larstalian larstalian closed this Jul 6, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generalize CodexAgentBackend — it's not actually codex-specific

1 participant