Skip to content

chore(deps-dev): bump @types/vscode from 1.99.1 to 1.120.0#65

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/types/vscode-1.120.0
Open

chore(deps-dev): bump @types/vscode from 1.99.1 to 1.120.0#65
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/types/vscode-1.120.0

Conversation

@dependabot
Copy link
Copy Markdown
Contributor

@dependabot dependabot Bot commented on behalf of github May 14, 2026

Bumps @types/vscode from 1.99.1 to 1.120.0.

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [@types/vscode](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/vscode) from 1.99.1 to 1.120.0.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/vscode)

---
updated-dependencies:
- dependency-name: "@types/vscode"
  dependency-version: 1.120.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot @github
Copy link
Copy Markdown
Contributor Author

dependabot Bot commented on behalf of github May 14, 2026

Labels

The following labels could not be found: npm. Please create it before Dependabot can add it to a pull request.

Please fix the above issues or remove invalid values from dependabot.yml.

@dependabot dependabot Bot requested a review from oaslananka as a code owner May 14, 2026 00:28
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 14, 2026

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated development dependencies to improve development environment compatibility.

Walkthrough

The @types/vscode devDependency is bumped from version 1.99.1 to 1.120.0 in package.json, bringing the TypeScript type definitions for the VS Code API up to a newer release.

Changes

TypeScript typings update

Layer / File(s) Summary
@types/vscode version update
package.json
@types/vscode bumped from 1.99.1 to 1.120.0 in devDependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • oaslananka
  • oaslananka-repo-ops

Poem

🐰 One dependency hops along,
From ninety-nine to one-twenty strong,
Type definitions fresh and bright,
VS Code's API typed just right! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The PR title accurately describes the main change: a dependency version bump of @types/vscode in devDependencies, using conventional commit format.
Description check ✅ Passed The description is a standard Dependabot-generated message directly related to the dependency update, including version change details and bot commands.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dependabot/npm_and_yarn/types/vscode-1.120.0
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch dependabot/npm_and_yarn/types/vscode-1.120.0

Comment @coderabbitai help to get the list of available commands and usage tips.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​types/​vscode@​1.120.01001008092100

View full report

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@package.json`:
- Line 1857: The package.json currently pins "@types/vscode" to 1.120.0 while
engines.vscode is "^1.99.0", causing a type/runtime mismatch; pick one fix:
either downgrade the dependency to "@types/vscode" matching the declared minimum
(e.g., set "@types/vscode" to 1.99.x) so typechecking only uses APIs available
in VS Code ^1.99.0, or raise the declared minimum engine by updating
engines.vscode to match the types package (e.g., change engines.vscode to
^1.120.0) so the runtime guarantee aligns with the typings; update package.json
accordingly and run a fresh install and a typecheck to verify no remaining API
mismatches.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f75aa534-77d8-416e-b563-a8fb98b64270

📥 Commits

Reviewing files that changed from the base of the PR and between bc2ade7 and 5b81fe9.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

Comment thread package.json
"@types/mocha": "10.0.10",
"@types/node": "24.12.3",
"@types/vscode": "1.99.1",
"@types/vscode": "1.120.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

python - <<'PY'
import json, re, sys

with open("package.json", "r", encoding="utf-8") as f:
    pkg = json.load(f)

engine = pkg.get("engines", {}).get("vscode", "")
types = pkg.get("devDependencies", {}).get("@types/vscode", "")

def parse(v: str):
    m = re.search(r'(\d+)\.(\d+)\.(\d+)', v or "")
    return tuple(map(int, m.groups())) if m else None

e = parse(engine)
t = parse(types)

print(f"engines.vscode: {engine}")
print(f"@types/vscode:  {types}")

if not e or not t:
    print("ERROR: Could not parse one or both semver values.")
    sys.exit(2)

if (t[0], t[1]) > (e[0], e[1]):
    print("FAIL: typings target a newer VS Code minor than the declared engine floor.")
    sys.exit(1)

print("PASS: no typings/engine floor mismatch detected.")
PY

Repository: oaslananka/kicad-studio

Length of output: 190


Align VS Code typings with the declared minimum engine version.

@types/vscode is pinned to 1.120.0 (line 1857), while the extension declares support from ^1.99.0 (line 8). This creates a runtime compatibility risk: code will typecheck against APIs added in VS Code 1.100–1.120 but fail for users on the supported VS Code 1.99–1.119 range.

Either downgrade the type definitions to match the declared minimum or raise the engine requirement. Choose one:

Option 1: Keep types at minimum version
--- a/package.json
+++ b/package.json
@@ -1855,7 +1855,7 @@
     "@types/node": "^20.0.0",
-    "@types/vscode": "1.120.0",
+    "@types/vscode": "1.99.1",
     "eslint": "^8.45.0",
Option 2: Raise minimum supported version
--- a/package.json
+++ b/package.json
@@ -6,7 +6,7 @@
     "vscode": "^1.120.0",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"@types/vscode": "1.120.0",
"@types/vscode": "1.99.1",
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@package.json` at line 1857, The package.json currently pins "@types/vscode"
to 1.120.0 while engines.vscode is "^1.99.0", causing a type/runtime mismatch;
pick one fix: either downgrade the dependency to "@types/vscode" matching the
declared minimum (e.g., set "@types/vscode" to 1.99.x) so typechecking only uses
APIs available in VS Code ^1.99.0, or raise the declared minimum engine by
updating engines.vscode to match the types package (e.g., change engines.vscode
to ^1.120.0) so the runtime guarantee aligns with the typings; update
package.json accordingly and run a fresh install and a typecheck to verify no
remaining API mismatches.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Copy link
Copy Markdown
Contributor

@oaslananka-repo-ops oaslananka-repo-ops Bot left a comment

Choose a reason for hiding this comment

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

Approved for policy-controlled Dependabot auto-merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants