Skip to content

Engineering - add GitHub action stub #252544

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

Closed
wants to merge 85 commits into from
Closed

Engineering - add GitHub action stub #252544

wants to merge 85 commits into from

Conversation

lszomoru
Copy link
Member

No description provided.

@lszomoru lszomoru self-assigned this Jun 26, 2025
@lszomoru lszomoru added this to the July 2025 milestone Jun 26, 2025
@lszomoru lszomoru added the engineering VS Code - Build / issue tracking / etc. label Jun 26, 2025
connor4312 and others added 29 commits June 30, 2025 09:28
* trust markdown

* remove unneeded trust options
* shellEnv: disable powershell discovery on Windows

Closes #251957
Closes #251901
Closes #251446

* comments
* remove css for coding agent prompt file

* but localized
chat - prepare dialog experiments with alternate provider
* Fix incomplete markdown rendering issue with trailing spaces

* Fix issue with trailing dash
* Initial refactoring

* More observables

* Fix define-class-fields-check
)

* Initial plan

* Fix fetch tool to only handle HTTP/HTTPS schemes

This prevents MCP resource URIs (like test://static/resource/50) from being
incorrectly processed by the fetch tool. MCP resources should be handled
by VS Code's filesystem providers instead.

- Modified FetchWebPageTool._parseUris() to filter out non-HTTP(S) schemes
- Added comprehensive tests for the new scheme filtering behavior
- Non-HTTP(S) URIs will now be treated as invalid and skipped

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Refactor tests to use TestWebContentExtractorService instead of testing private methods

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Enhance fetch tool to serve file content via IFileService for non-HTTP schemes

Co-authored-by: connor4312 <2230985+connor4312@users.noreply.github.com>

* Revert tool description to original text as requested

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Implement FileService-based WebContentExtractorService for web platform

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Handle binary data

* style

* don't adopt this yet

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>
Co-authored-by: connor4312 <2230985+connor4312@users.noreply.github.com>
Co-authored-by: Tyler Leonhardt <tyleonha@microsoft.com>
* Proper observables in service

* Persist mode data

* Mode observables for rendering

* Handle observable more consistently

* Rename to ChatModeKind

* More "kind" renames

* Add unit test for the service

* Fire event less

* Fix throwing from deserialize method

* Revert this change

* Cleanup '2' names
…252891)

Update chat widget welcome view with suggested prompts for startup experiments
* add telemetry and switch setting to exp

* fix names

* change to onExp

* clean up telemetry call

* change telemetry name again
@lszomoru lszomoru closed this Jun 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering VS Code - Build / issue tracking / etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.