Skip to content

Fix Copilot Chat debug experience#308264

Open
lramos15 wants to merge 5 commits intomainfrom
lramos15/distinctive-aardwolf
Open

Fix Copilot Chat debug experience#308264
lramos15 wants to merge 5 commits intomainfrom
lramos15/distinctive-aardwolf

Conversation

@lramos15
Copy link
Copy Markdown
Member

@lramos15 lramos15 commented Apr 7, 2026

  • Launch config wasn't matching chat built to /dist
  • Npm install was producing a diff

Copilot AI review requested due to automatic review settings April 7, 2026 17:00
@lramos15 lramos15 enabled auto-merge (squash) April 7, 2026 17:00
@lramos15 lramos15 self-assigned this Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Screenshot Changes

Base: 55a24659 Current: cd9d5710

Changed (2)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Dark
Before After
before after
chat/aiCustomizations/aiCustomizationManagementEditor/PluginBrowseMode/Dark
Before After
before after

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates VS Code repo debugging and dependency lock state to improve the Copilot Chat debug workflow and keep npm install deterministic for the built-in Copilot extension.

Changes:

  • Extend the repo’s “Attach to Extension Host” launch config to include Copilot’s compiled /dist output in outFiles for correct source mapping while debugging.
  • Normalize extensions/copilot/package-lock.json metadata (marking certain packages as dev) to prevent npm install from producing an unexpected diff.
Show a summary per file
File Description
extensions/copilot/package-lock.json Adjusts lockfile metadata (dev: true) to stabilize install output/diffs.
.vscode/launch.json Adds Copilot’s dist/**/*.js to outFiles for extension-host debugging/source maps.

Copilot's findings

Files not reviewed (1)
  • extensions/copilot/package-lock.json: Language not supported
  • Files reviewed: 1/2 changed files
  • Comments generated: 0

@joaomoreno joaomoreno closed this Apr 7, 2026
auto-merge was automatically disabled April 7, 2026 18:40

Pull request was closed

@joaomoreno joaomoreno reopened this Apr 7, 2026
@joaomoreno joaomoreno enabled auto-merge (squash) April 7, 2026 18:40
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.

4 participants