Skip to content

Re-enable local Claude Code orchestration#11571

Open
kjankov wants to merge 1 commit into
masterfrom
katarina/reenable-claude-code-local
Open

Re-enable local Claude Code orchestration#11571
kjankov wants to merge 1 commit into
masterfrom
katarina/reenable-claude-code-local

Conversation

@kjankov
Copy link
Copy Markdown
Contributor

@kjankov kjankov commented May 22, 2026

Description

Re-enables the local Claude Code child harness for orchestration while keeping local Codex disabled.

This updates the local harness selector so Claude Code remains visible but disabled when the claude CLI is missing, with the tooltip Harness installation required. Learn more and a docs link. When Claude Code is installed, the selector allows local Claude Code and silently reconciles missing/outdated Warp Claude plugins during launch.

Linked Issue

N/A

  • The linked issue is labeled ready-to-spec or ready-to-implement.
  • Where appropriate, screenshots or a short video of the implementation are included below (especially for user-visible or UI changes).

Testing

  • cargo fmt

  • cargo clippy --workspace --all-targets --all-features --tests -- -D warnings

  • cargo test -p warp --lib local_child_harnesses

  • Targeted module-path tests for orchestration_controls::tests, run_agents_card_view::tests, local_harness_launch::tests, plugin_manager::claude::tests, block::tests, and start_agent::tests

  • I have manually tested my changes locally with ./script/run

Screenshots / Videos

N/A

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

Agent Artifacts

CHANGELOG-OZ: Re-enabled local Claude Code child agents for orchestration when Claude Code is installed.

Co-Authored-By: Oz oz-agent@warp.dev

Co-Authored-By: Oz <oz-agent@warp.dev>
@cla-bot cla-bot Bot added the cla-signed label May 22, 2026
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 22, 2026

@kjankov

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR re-enables local Claude Code orchestration, keeps local Codex disabled, adds install-state handling for missing Claude Code, and reconciles Claude plugins during local child launch.

Concerns

  • ⚠️ [IMPORTANT] This is a user-facing change to the orchestration controls and launch behavior, but the PR description marks screenshots/videos as N/A. For this user-facing change, please include screenshots or a screen recording demonstrating the local harness picker, the missing-Claude disabled state/tooltip, and the Learn more path working end to end.

Verdict

Found: 0 critical, 1 important, 0 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant