Skip to content

Fix CI: patch vscode-jsonrpc exports map to support @github/copilot-sdk 0.1.32#39

Draft
Copilot wants to merge 2 commits intorenovate/github-copilot-sdk-0.xfrom
copilot/sub-pr-38
Draft

Fix CI: patch vscode-jsonrpc exports map to support @github/copilot-sdk 0.1.32#39
Copilot wants to merge 2 commits intorenovate/github-copilot-sdk-0.xfrom
copilot/sub-pr-38

Conversation

Copy link
Contributor

Copilot AI commented Mar 8, 2026

@github/copilot-sdk 0.1.32 imports vscode-jsonrpc/node and vscode-jsonrpc/node.js as ESM subpaths, but vscode-jsonrpc 8.2.1 ships without an exports field — Node's strict ESM resolver can't locate those subpaths, crashing all tests at import time.

Changes

  • scripts/patch-vscode-jsonrpc.cjs — adds the missing exports map to the installed vscode-jsonrpc package, exposing both ./node and ./node.js subpaths that point to the existing node.js file
  • package.json — wires the script as a postinstall hook so it runs automatically after every npm ci / npm install

The patch is idempotent and silently skips if vscode-jsonrpc is absent (e.g. production installs). Any unexpected error (permissions, malformed JSON) is surfaced as a warning rather than swallowed.

// Added to node_modules/vscode-jsonrpc/package.json after install
"exports": {
  ".":          { "default": "./lib/node/main.js", "types": "./lib/common/api.d.ts" },
  "./node":     { "default": "./node.js",           "types": "./node.d.ts" },
  "./node.js":  { "default": "./node.js",           "types": "./node.d.ts" }
}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…ithub/copilot-sdk 0.1.32

Co-authored-by: raphaelpor <8525863+raphaelpor@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 8, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

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:

  • https://api.github.com/copilot_internal/user
    • Triggering command: /opt/hostedtoolcache/node/24.14.0/x64/bin/node /opt/hostedtoolcache/node/24.14.0/x64/bin/node /home/REDACTED/work/katt/katt/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level debug --stdio git ame (http block)
    • Triggering command: /opt/hostedtoolcache/node/24.14.0/x64/bin/node /opt/hostedtoolcache/node/24.14.0/x64/bin/node /home/REDACTED/work/katt/katt/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level debug --stdio it ame (http block)
    • Triggering command: /opt/hostedtoolcache/node/24.14.0/x64/bin/node /opt/hostedtoolcache/node/24.14.0/x64/bin/node /home/REDACTED/work/katt/katt/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level debug --stdio p/bin/bash essage)) (http block)

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

Copilot AI changed the title [WIP] Update dependency @github/copilot-sdk to v0.1.32 Fix CI: patch vscode-jsonrpc exports map to support @github/copilot-sdk 0.1.32 Mar 8, 2026
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