Skip to content

agent_ui: Fix send failure for pasted file#line mention links#49786

Merged
benbrandt merged 1 commit intozed-industries:mainfrom
cppcoffee:agent_ui_mention
Mar 17, 2026
Merged

agent_ui: Fix send failure for pasted file#line mention links#49786
benbrandt merged 1 commit intozed-industries:mainfrom
cppcoffee:agent_ui_mention

Conversation

@cppcoffee
Copy link
Copy Markdown
Contributor

Release Notes:

  • Fixed send failure for pasted file#line mention links

Handle MentionUri::Selection with a file path in confirm_mention_for_uri
by resolving it to text content via existing line-range loading logic, instead
of returning "Unsupported mention URI type for paste".

Keep untitled-buffer selections (abs_path: None) as an explicit unsupported
error for now.

Add a regression test for pasting selection mentions to ensure they resolve to
Mention::Text with the expected line range.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 21, 2026
@SomeoneToIgnore SomeoneToIgnore added the area:ai Related to Agent Panel, Edit Prediction, Copilot, or other AI features label Feb 21, 2026
@cppcoffee
Copy link
Copy Markdown
Contributor Author

cppcoffee commented Feb 28, 2026

before:

2026-02-28.11.30.44.mov

after:

2026-02-28.11.41.06.mov

Signed-off-by: Xiaobo Liu <cppcoffee@gmail.com>
Copy link
Copy Markdown
Member

@benbrandt benbrandt left a comment

Choose a reason for hiding this comment

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

thanks!

@benbrandt benbrandt enabled auto-merge (squash) March 17, 2026 10:50
@benbrandt benbrandt merged commit a3a28ea into zed-industries:main Mar 17, 2026
29 checks passed
@cppcoffee cppcoffee deleted the agent_ui_mention branch March 17, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ai Related to Agent Panel, Edit Prediction, Copilot, or other AI features cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants