Skip to content

agent_ui: Show global skill paths in edits summary#57767

Merged
MartinYe1234 merged 2 commits into
mainfrom
file-creation-ui-no-file-name-for-global-skills
May 27, 2026
Merged

agent_ui: Show global skill paths in edits summary#57767
MartinYe1234 merged 2 commits into
mainfrom
file-creation-ui-no-file-name-for-global-skills

Conversation

@MartinYe1234

@MartinYe1234 MartinYe1234 commented May 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Display the full path for global skill files in the agent edits summary when they do not have a project-relative file name.

Closes AI-320

Validation

  • cargo check -p agent_ui --lib

Release Notes:

  • Fixed global skill edits appearing without a file path in the agent changes summary.

@MartinYe1234 MartinYe1234 self-assigned this May 27, 2026
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 27, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label May 27, 2026
@MartinYe1234 MartinYe1234 marked this pull request as ready for review May 27, 2026 01:56
Comment on lines +2748 to +2753
global_skill_full_path.as_ref().map(|path| {
Label::new(path.clone())
.size(LabelSize::XSmall)
.buffer_font(cx)
.ml_1()
})

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we just fallback to the full path no matter what location path is in? If an agent edits a file I would expect to see the full path instead of an empty icon for basically any case

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah that makes sense to me

@Anthony-Eid Anthony-Eid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

great work!

@MartinYe1234 MartinYe1234 added this pull request to the merge queue May 27, 2026
Merged via the queue into main with commit 867e511 May 27, 2026
32 checks passed
@MartinYe1234 MartinYe1234 deleted the file-creation-ui-no-file-name-for-global-skills branch May 27, 2026 02:27
@danilo-leal

Copy link
Copy Markdown
Member

/cherry-pick preview

@zed-zippy

zed-zippy Bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

🍒💥 Cherry-pick did not succeed
https://github.com/zed-industries/zed/actions/runs/26512124785

MartinYe1234 added a commit that referenced this pull request May 27, 2026
…ick to preview) (#57838)

Cherry-pick of #57767 to preview

----
Summary

- Display the full path for global skill files in the agent edits
summary when they do not have a project-relative file name.

Closes AI-320

Validation

- cargo check -p agent_ui --lib

Release Notes:

- Fixed global skill edits appearing without a file path in the agent
changes summary.

Co-authored-by: MartinYe1234 <52641447+MartinYe1234@users.noreply.github.com>
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 2, 2026
)

Summary

- Display the full path for global skill files in the agent edits
summary when they do not have a project-relative file name.

Closes AI-320

Validation

- cargo check -p agent_ui --lib

Release Notes:

- Fixed global skill edits appearing without a file path in the agent
changes summary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants