Skip to content

chore: remove unused uuid type dependency - #1679

Merged
Changyong Gong (chagong) merged 1 commit into
mainfrom
chore/remove-unused-uuid-types
Aug 31, 2026
Merged

chore: remove unused uuid type dependency#1679
Changyong Gong (chagong) merged 1 commit into
mainfrom
chore/remove-unused-uuid-types

Conversation

@chagong

Copy link
Copy Markdown
Contributor

Removes the unused @types/uuid development dependency. Its latest version is a deprecated stub that installs uuid transitively, so this supersedes #1670.

Validation:

  • npm run tslint
  • npm test (29 passing)
  • VSIX package and isolated installation
  • npm audit --omit=dev --audit-level=high

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

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

Removes the unused @types/uuid dev dependency to avoid pulling in a deprecated stub package that can install uuid transitively, keeping the dependency graph cleaner for the VS Code Java Debug extension.

Changes:

  • Removed @types/uuid from devDependencies in package.json.
  • Removed the corresponding @types/uuid entries from package-lock.json.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Drops the unused @types/uuid dev dependency.
package-lock.json Removes the lockfile entries for @types/uuid to match package.json.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@chagong
Changyong Gong (chagong) merged commit 6855444 into main Aug 31, 2026
5 checks passed
@chagong
Changyong Gong (chagong) deleted the chore/remove-unused-uuid-types branch August 31, 2026 05:59
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.

3 participants