Route bundled skills through active Agent Mode host

[moirahuang]

Description

Routes Agent Mode bundled-skill listing, read_skill resolution, and slash-command skill invocation through the active session's bundled-skill catalog. Local sessions continue using the local catalog, while connected remote sessions use the ready catalog for their exact HostId.

The Agent input context passes the active session's SkillPathOrigin into skill listing, and ReadSkillExecutor receives ActiveSession so bundled IDs resolve from the same selected host catalog. Slash-command invocations (/skill-name) resolve through a new BlocklistAIController::skill_path_origin accessor backed by the same active session, so remote sessions embed host-rendered content instead of client-local paths. Remote sessions whose host identity/catalog is unavailable return no bundled skills and do not fall back to client-local bundled content — both read_skill and the slash-command toast surface "Bundled skills are not available on this remote session". Existing path-based skill behavior remains unchanged.

This is the third PR in the stack:

• Introduce single-host BundledSkill abstraction #12377 introduces the single-host BundledSkill abstraction.
• Install and manage bundled skills across remote host lifecycles #12378 installs complete remote bundles and manages per-host rendered catalogs.
• This PR connects those catalogs to Agent Mode listing, read_skill, and slash-command invocation.

Linked Issue

• 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

• Added coverage for host-specific bundled-skill listing, same-ID host isolation, unavailable remote-session no-fallback behavior, remote bundled-skill reads, and preservation of local reads.
• cargo check -p warp --all-targets passes cleanly (including removal of the now-unused BundledSkills::local accessor); tests, formatters, and Clippy were not run per request.
• I have manually tested my changes locally with ./script/run

Agent Mode

• Warp Agent Mode - This PR was created via Warp's AI Agent Mode
• Agent conversation: https://staging.warp.dev/conversation/5f4415d3-76ab-4edf-aa62-ada66bac74f4
• Plan: https://staging.warp.dev/drive/notebook/F5HotaELiARZxrQCvUmKqD

CHANGELOG-NONE

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

[oz-for-oss]

@moirahuang

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

[oz-for-oss]

Overview

This PR routes Agent Mode bundled-skill listing and read_skill resolution through the active session's skill origin so local sessions use local bundles, connected remote sessions use their host-specific remote catalog, and unavailable remote sessions do not fall back to client-local bundled content. I did not find correctness or security issues in the changed diff.

Concerns

• This is a user-facing Agent Mode behavior change, but the PR description does not include screenshots or a screen recording demonstrating the remote bundled-skill listing/read_skill behavior working end to end. For this user-facing change, please include screenshots or a short recording; if a local desktop is not available, capture it from a coding agent that supports computer use and attach it to the PR.

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

[moirahuang]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Route bundled skills through active Agent Mode host #12418 👈 (View in Graphite)
• Install and manage bundled skills across remote host lifecycles #12378
• Introduce single-host BundledSkill abstraction #12377
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.