-
Notifications
You must be signed in to change notification settings - Fork 36k
Display page titles in chat reference labels instead of URLs #275958
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: dmitrivMS <[email protected]>
Co-authored-by: dmitrivMS <[email protected]>
|
🤖 Copilot has tried out this change! Members of the team can view the recording here or download the Playwright trace here (run) |
|
@copilot In chat UI we have an item with label "Fetched ...." which, when expanded, contains the link with "globe" icon. That link now has the title. Update the code so that the parent link has the title and the inner link has the full URL instead |
Co-authored-by: dmitrivMS <[email protected]>
Updated in ac0668a. Now for single web page fetches:
For multiple resources, the parent label shows the count as before ("Fetched 2 resources"), and the inner links show the URLs. |
Fixes rendering of fetched web page references in chat to show page titles in the collapsible header label instead of raw URLs.
Before:
Fetched https://docs.expo.dev/versions/latest/sdk/imagepicker/After:
Fetched ImagePicker - Expo DocumentationWhen the collapsible section is expanded, the inner link displays the full URL with a globe icon for reference.
Changes
Web content extraction - Extended
WebContentExtractResultto include optionaltitlefield;NativeWebContentExtractorServicenow extracts titles viawebContents.getTitle()Tool result pipeline - Added
IToolResultReferenceinterface to carry URI + title metadata throughtoolResultDetailsFetch tool -
FetchWebPageToolcollects titles from extraction results and setstoolResultMessagewith the title for single web page fetchesUI rendering - For single web resource fetches, the collapsible header label shows the page title via
toolResultMessage. The inner reference link shows the full URL. For multiple resources, the header shows the resource count as before.Serialization - Updated
IChatToolInvocationSerialized.resultDetailstype to supportIToolResultReferenceURL fallback preserved for cases without titles or non-web resources.
Fixes #247800
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.