Skip to content

feat!: codex-only removal (drop Claude/Anthropic paths)#10

Merged
eve-senara merged 2 commits intomasterfrom
feat/codex-only-remove-claude
Feb 23, 2026
Merged

feat!: codex-only removal (drop Claude/Anthropic paths)#10
eve-senara merged 2 commits intomasterfrom
feat/codex-only-remove-claude

Conversation

@eve-senara
Copy link
Collaborator

@eve-senara eve-senara commented Feb 23, 2026

Codex-only removal plan

This PR completes the codex-only migration for Orca by removing Claude/Anthropic runtime and setup paths, updating config/types/flags, and syncing tests/docs.

Scope

  • Runtime executor paths
  • Setup/auth detection and config writing
  • CLI flags/help surface
  • Types/config schema
  • Tests and integration setup script
  • README and SKILL docs

What changed

Runtime

  • Removed Claude executor implementation and tests:
    • src/agents/claude/session.ts (deleted)
    • src/agents/claude/session.test.ts (deleted)
  • Run/resume now execute via Codex only.
  • Removed nested-Claude guard logic (no longer relevant once Claude runtime is removed).

Setup / config

  • orca setup is codex-only:
    • No Anthropic key prompts/detection
    • No Claude availability checks
  • Codex auth detection supports both ~/.codex/auth.json shapes:
    • tokens.access_token (chatgpt/oauth mode)
    • OPENAI_API_KEY (api_key mode)
  • Added/kept TS config output support (--ts):
    • Global: ~/.orca/config.ts
    • Project: ./orca.config.ts
  • Config resolution includes global TS + JS precedence.

CLI / flags

  • Removed Claude flags/options:
    • --claude-only
    • --claude-effort
  • Kept codex equivalents:
    • --codex-only
    • --codex-effort

Types / schema

  • Removed Claude/Anthropic config surface from active config types/docs.
  • Executor surface is codex-only.

Docs

  • Updated in-repo docs (README/SKILL) for codex-only behavior.
  • Removed stale Claude/Anthropic setup guidance.

Breaking changes

  • Orca no longer supports Claude execution.
  • Claude-related flags/config are removed.
  • Anthropic setup/config paths are removed.

Migration

  1. Remove Claude fields from config (claude.*, Anthropic key fields).
  2. Use codex-only config:
    • executor: "codex" (or omit; codex is default/only)
  3. Ensure Codex auth via one of:
    • OPENAI_API_KEY
    • ORCA_OPENAI_API_KEY
    • ~/.codex/auth.json
  4. Replace old Claude flags with codex equivalents.

Verification

  • npm run lint
  • bun test src
  • bun run build
  • bun run test:setup

Residual risk

  • Low. Main risk is user configs/scripts still referencing removed Claude flags/fields.

@github-actions
Copy link

PR Review: codex-only removal (drop Claude/Anthropic paths)

The PR cleanly removes all Claude/Anthropic executor paths, making Orca a codex-only tool. Most changes are consistent. Found two issues:

BUG: --project-config-template flag is silently broken

src/cli/commands/setup.ts:212

The --project-config-template and --skip-project-config flags are still registered in commander:

.option("--project-config-template", "Write a typed project hook template to ./orca.config.ts")
.option("--skip-project-config", "Do not prompt to generate a project config template")

However, the maybeWriteProjectTemplate function and buildProjectConfigTemplate were both deleted, and the call to maybeWriteProjectTemplate was removed from setupCommandHandler. The validation for conflicting flags (--project-config-template + --skip-project-config) was also removed.

Failure scenario: A user runs orca setup --project-config-template. The flag is accepted without error, but no template file is written. The user believes the operation succeeded.

Fix: Either remove the two orphaned .option(...) lines from registerSetupCommand, or restore the template-writing logic.


BUG: orca help no longer lists PR commands

src/cli/commands/help.ts:40-42

The entire PULL REQUESTS help section was removed from printStyledHelpPage:

// Removed:
printSection("PULL REQUESTS", [
  { command: "orca pr", description: "interactive — pick run + action" },
  { command: "orca pr draft --run <id>", description: "create draft PR" },
  { command: "orca pr create --run <id>", description: "create ready-for-review PR" },
  { command: "orca pr publish --run <id>", description: "publish draft → ready for review" },
  { command: "orca pr status --run <id>", description: "check PR state and CI" }
]);

The PR commands themselves are still registered (src/cli/commands/pr/index.ts is unchanged, registerPrCommand is still called from src/cli/index.ts:34). This removal appears unrelated to the Claude removal and looks accidental.

Failure scenario: Users running orca help no longer see PR commands, making them undiscoverable. The commands still work if invoked directly.

Fix: Re-add the PULL REQUESTS section to printStyledHelpPage.

@github-actions
Copy link

No issues found. This PR cleanly removes all Claude/Anthropic runtime, setup, config, CLI flags, types, and tests — making Orca a codex-only tool — with no dangling references, type inconsistencies, or broken code paths.

@eve-senara eve-senara merged commit d44fbd0 into master Feb 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant