[5/5] Resolve remote skill locations in tool output#11863
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
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 Powered by Oz |
There was a problem hiding this comment.
Overview
This PR updates AI block output rendering so read-files and search-codebase tool outputs resolve skill buttons from session-aware local or remote locations. I did not find security findings or material spec drift; the attached spec context says no approved repository spec was found.
Concerns
- This is a user-visible behavior change to tool output, but the PR description does not include screenshots or a screen recording. For this user-facing change, please include screenshots or a screen recording demonstrating it 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
278c688 to
5f6fa32
Compare
befa271 to
5e6b798
Compare
8b5d0c1 to
783e4a5
Compare
5e6b798 to
477221e
Compare
783e4a5 to
f7c7e76
Compare
477221e to
63bfd99
Compare
63bfd99 to
556a682
Compare
f7c7e76 to
74d2566
Compare
085de16 to
356c1d5
Compare
Co-Authored-By: Oz <oz-agent@warp.dev>
356c1d5 to
b7ca2b3
Compare
Merge activity
|

Description
PR 5/5 of the remote-aware skills stack.
Resolve read/search result locations through the active session before mapping results back to known skills, preserving remote host identity in rendered output. This layer also removes the superseded local-only output helper and its tests.
Plan: https://staging.warp.dev/drive/notebook/EAngN0Hb9BqY5WiPMTXFV5
Agent run: https://staging.warp.dev/conversation/88702634-8ffe-46a4-b868-1efae92630eb
Linked Issue
ready-to-specorready-to-implement.Testing
cargo fmt --manifest-path Cargo.toml -p warp_util -p ai -p warpcargo test -p warp test_unique_skills --libcargo test -p warp parsed_skill_for_common_locations --libFull workspace clippy was started on the cumulative stack tip but not completed before submission.
I have manually tested my changes locally with
./script/runAgent Mode
Co-Authored-By: Oz oz-agent@warp.dev