Skip to content

get OptionsSchema.json back in sync with package.json#14523

Merged
bobbrow merged 4 commits into
mainfrom
bobbrow/optionsSchema
Jun 18, 2026
Merged

get OptionsSchema.json back in sync with package.json#14523
bobbrow merged 4 commits into
mainfrom
bobbrow/optionsSchema

Conversation

@bobbrow

@bobbrow bobbrow commented Jun 17, 2026

Copy link
Copy Markdown
Member

Two changes for the debugger were made directly to package.json without a matching OptionsSchema.json update so I got a surprise when generating it for my last change.

This PR gets it back in sync.

@bobbrow bobbrow requested a review from a team as a code owner June 17, 2026 23:33
@github-project-automation github-project-automation Bot moved this to Pull Request in cpptools Jun 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the generated debugger options schema (Extension/tools/OptionsSchema.json) to match the current debugger configuration schema in Extension/package.json, addressing drift caused by prior direct edits to package.json.

Changes:

  • Removed an extra description field from the SourceFileMapEntry definition to match the inline package.json schema.
  • Updated cppdbg attach program to use markdownDescription (and removed the old default/description form) in line with package.json.
  • Updated cppvsdbg attach to no longer require processId, and added the program property with the same markdownDescription as package.json.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bobbrow bobbrow merged commit 239deaf into main Jun 18, 2026
6 checks passed
@bobbrow bobbrow deleted the bobbrow/optionsSchema branch June 18, 2026 21:05
@github-project-automation github-project-automation Bot moved this from Pull Request to Done in cpptools Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants