Skip to content

Fix black formatter activation and bundled startup regressions#745

Draft
bschnurr wants to merge 2 commits into
microsoft:mainfrom
bschnurr:digest-20260515-1428
Draft

Fix black formatter activation and bundled startup regressions#745
bschnurr wants to merge 2 commits into
microsoft:mainfrom
bschnurr:digest-20260515-1428

Conversation

@bschnurr

Copy link
Copy Markdown
Member

Summary

  • hide the restart command until the extension has activated
  • keep bundled runtime dependencies on the bundled sys.path during server and runner startup
  • fall back safely when Windows mapped-drive and UNC paths make relpath() fail
  • add regression tests and the requested reviewCritic agent file

Validation

  • npm run compile
  • npm run lint
  • python -m pytest src/test/python_tests/test_get_cwd.py src/test/python_tests/test_sys_path_bootstrap.py

Fixes #514
Fixes #729
Fixes #731

Hide inactive commands until activation, keep bundled runtime dependencies on the bundled path during startup, add Windows cross-mount cwd fallback coverage, and add reviewCritic guidance.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bschnurr bschnurr added the bug Issue identified by VS Code Team member as probable bug label May 16, 2026
Restore LF endings for the touched text files so the existing PR shows the intended minimal diff.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment thread bundled/tool/lsp_runner.py
Comment thread bundled/tool/lsp_server.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue identified by VS Code Team member as probable bug

Projects

None yet

2 participants