Skip to content

Fix CI: install basedpyright language server#22

Merged
observerw merged 2 commits intomainfrom
copilot/fix-ci-failure-issue
Jan 16, 2026
Merged

Fix CI: install basedpyright language server#22
observerw merged 2 commits intomainfrom
copilot/fix-ci-failure-issue

Conversation

Copy link
Contributor

Copilot AI commented Jan 16, 2026

CI tests were failing because the basedpyright language server executable was not available in the test environment. The LSP client attempted to spawn basedpyright processes, resulting in ConnectionResetError and BrokenResourceError exceptions that surfaced as cryptic "unhandled errors in a TaskGroup" failures.

Changes

  • Add npm installation step for basedpyright in CI workflow before test execution

The basedpyright package is required at runtime by the LSP client but was not explicitly installed in the CI environment, unlike development environments where language servers are typically pre-installed.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)
  • dl.deno.land
    • Triggering command: /bin/deno deno lsp (dns block)
  • esm.sh
    • Triggering command: /bin/deno deno lsp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

排查并解决ci失败问题


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: observerw <20661574+observerw@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI failure issues Fix CI: install basedpyright language server Jan 16, 2026
Copilot AI requested a review from observerw January 16, 2026 09:46
@observerw observerw marked this pull request as ready for review January 16, 2026 12:03
@observerw observerw merged commit 1ab5089 into main Jan 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants