Skip to content

chore(deps): update definitelytyped#80

Closed
renovate[bot] wants to merge 1 commit intomainfrom
renovate/definitelytyped
Closed

chore(deps): update definitelytyped#80
renovate[bot] wants to merge 1 commit intomainfrom
renovate/definitelytyped

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Oct 18, 2025

This PR contains the following updates:

Package Change Age Confidence
@types/chai (source) 4.3.64.3.20 age confidence
@types/chai-arrays (source) 2.0.12.0.3 age confidence
@types/chai-as-promised (source) 7.1.67.1.8 age confidence
@types/cors (source) 2.8.122.8.19 age confidence
@types/debug (source) 4.1.74.1.12 age confidence
@types/dedent (source) 0.7.00.7.2 age confidence
@​types/del 4.0.04.0.3 age confidence
@types/format-util (source) 1.0.21.0.4 age confidence
@​types/glob 5.0.375.0.38 age confidence
@types/json2csv (source) 5.0.35.0.7 age confidence
@types/loadable__component (source) 5.13.45.13.10 age confidence
@types/lodash (source) 4.14.1814.17.21 age confidence
@types/node (source) 18.19.3118.19.130 age confidence
@types/node (source) 22.15.3122.19.3 age confidence
@types/node-fetch (source) 2.6.122.6.13 age confidence
@types/pdfkit (source) ^0.11.0^0.17.0 age confidence
@types/promisify-node (source) 0.4.00.4.3 age confidence
@types/react-dom (source) 16.9.1416.9.25 age confidence
@​types/react-json-tree ^0.6.8^0.13.0 age confidence
@types/react-redux (source) 7.1.237.1.34 age confidence
@types/react-virtualized (source) 9.21.209.22.3 age confidence
@types/redux-logger (source) 3.0.93.0.13 age confidence
@types/sinon (source) 10.0.1510.0.20 age confidence
@types/stack-trace (source) 0.0.290.0.33 age confidence
@types/strip-comments (source) 2.0.12.0.4 age confidence
@types/svg-to-pdfkit (source) 0.1.00.1.3 age confidence
@types/tcp-port-used (source) 1.0.11.0.4 age confidence
@types/temp (source) ^0.8.32^0.9.0 age confidence
@types/tmp (source) 0.2.30.2.6 age confidence
@types/url-parse (source) 1.4.81.4.11 age confidence
@types/vscode (source) 1.88.01.107.0 age confidence
@types/vscode-notebook-renderer (source) 1.60.01.72.4 age confidence

Configuration

📅 Schedule: Branch creation - "every weekend" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

Summary by CodeRabbit

  • Chores
    • Updated several development type-definition packages to newer versions for improved type coverage and compatibility during development and testing.
    • No changes to public/exported API signatures; runtime behavior and public interfaces remain unchanged.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 18, 2025

📝 Walkthrough

Walkthrough

Bumps dev-only TypeScript declaration versions in package.json: @types/pdfkit ^0.11.0 → ^0.17.0, @types/react-json-tree ^0.6.8 → ^0.13.0, @types/stack-trace 0.0.29 → 0.0.33, and @types/temp ^0.8.32 → ^0.9.0. Changes are limited to devDependencies; no source code, runtime behavior, or exported/public API signatures were modified.

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating DefinitelyTyped packages in devDependencies.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 9811516 and 13cad35.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: copilot-setup-steps

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

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 18, 2025

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

coderabbitai[bot]
coderabbitai Bot previously approved these changes Oct 18, 2025
coderabbitai[bot]
coderabbitai Bot previously approved these changes Oct 18, 2025
@renovate renovate Bot force-pushed the renovate/definitelytyped branch from 136521d to c44d7c3 Compare October 18, 2025 07:19
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 136521d and c44d7c3.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

@renovate renovate Bot force-pushed the renovate/definitelytyped branch 2 times, most recently from 324357d to b43aa9e Compare October 20, 2025 16:14
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 324357d and b43aa9e.

⛔ Files ignored due to path filters (1)
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)
🧰 Additional context used
🪛 GitHub Actions: CI
package.json

[error] 1-1: Command failed with exit code 1: npm ci --prefer-offline --no-audit

@renovate renovate Bot force-pushed the renovate/definitelytyped branch 3 times, most recently from b71a11c to 2b76688 Compare October 20, 2025 17:25
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 658b854 and 2b76688.

⛔ Files ignored due to path filters (1)
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)
🔇 Additional comments (1)
package.json (1)

2276-2276: Clarify @types/temp vs runtime tmp package.

Line 2276 references @types/temp@^0.9.0, but the runtime dependency is tmp@^0.2.4 (line 2218). Verify these refer to the same package or clarify the intent.

@renovate renovate Bot force-pushed the renovate/definitelytyped branch 6 times, most recently from 3ba1ac3 to 68e1475 Compare October 21, 2025 13:28
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between f86d239 and ce9bd10.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)
🔇 Additional comments (2)
package.json (2)

2590-2590: No active codebase usage detected for react-json-tree

Searches found no imports or component usage in the codebase. If this package is only a transitive dependency or not actively used, the version bump poses no compatibility risk. Verify manually that this dependency serves a purpose and check the DefinitelyTyped changelog if direct usage exists.


2586-2586: @types/pdfkit mismatch: types v0.17.0 but pdfkit runtime is v0.13.0

pdfkit is used in src/webviews/extension-side/plotting/plotViewer.node.ts (lines 71–81). The types version (0.17.0) is ahead of the runtime (0.13.0)—types should be backward compatible, but verify the types work correctly with the runtime version and that the basic pdfkit constructor and SVGtoPDF integration have no issues.

Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 6e2b641 and 81e1715.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

coderabbitai[bot]
coderabbitai Bot previously approved these changes Nov 5, 2025
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 9e35723 and b3d3f13.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between b3d3f13 and 36dc8cd.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

coderabbitai[bot]
coderabbitai Bot previously approved these changes Nov 12, 2025
coderabbitai[bot]
coderabbitai Bot previously approved these changes Nov 15, 2025
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 739f08b and d886ba0.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between d886ba0 and d4a1aad.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between d4a1aad and c5e8e5e.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 641fd54 and e504c33.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

coderabbitai[bot]
coderabbitai Bot previously approved these changes Nov 25, 2025
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between da125aa and d4833ff.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

coderabbitai[bot]
coderabbitai Bot previously approved these changes Dec 3, 2025
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1bc22db and 95d3541.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 0f18ad5 and 6baaf92.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)

coderabbitai[bot]
coderabbitai Bot previously approved these changes Dec 12, 2025
coderabbitai[bot]
coderabbitai Bot previously approved these changes Dec 13, 2025
Copy link
Copy Markdown
Contributor

@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

📜 Review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1903285 and 9b16e5b.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json (1 hunks)
🔇 Additional comments (3)
package.json (3)

2625-2625: Remove orphaned @types/temp: runtime temp package not present.

The runtime temp package is not listed in dependencies. Having @types/temp ^0.9.0 without the runtime package suggests unused type definitions—either remove the type definition or confirm the package is a transitive dependency.

Likely an incorrect or invalid review comment.


2614-2614: Remove orphaned @types/react-json-tree dependency.

The base package react-json-tree is not installed, and @types/react-json-tree is unused throughout the codebase. Delete this dead dependency from package.json.


2610-2610: No action needed. pdfkit is not used in the codebase—the version bump for @types/pdfkit has no impact.

Copy link
Copy Markdown
Contributor

@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: 2

♻️ Duplicate comments (1)
package.json (1)

2621-2621: Version mismatch already flagged.

Types at 0.0.33 vs runtime stack-trace at 0.0.10 (line 2560) already noted in past reviews.

📜 Review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 9b16e5b and cb5168f.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json
🔇 Additional comments (1)
package.json (1)

2610-2610: No pdfkit usage detected in the codebase, making the type/runtime version mismatch a non-issue. If pdfkit is unused, consider removing the dependency and its types rather than fixing version alignment.

Likely an incorrect or invalid review comment.

Copy link
Copy Markdown
Contributor

@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

♻️ Duplicate comments (3)
package.json (3)

2614-2614: Remove deprecated @types/react-json-tree.

This package remains deprecated. The upstream library provides its own types, and there's no react-json-tree runtime dependency.


2621-2621: Version mismatch persists.

@types/stack-trace at 0.0.33 but runtime stack-trace (line 2560) remains at 0.0.10.


2625-2625: Wrong types package.

@types/temp is for the "temp" package, but this codebase uses "tmp" (line 2565). Use only @types/tmp (line 2626).

📜 Review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between cb5168f and 3525cae.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

Copy link
Copy Markdown
Contributor

@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: 4

📜 Review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 21a47d1 and 2e06df3.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • src/test/vscode-notebook-perf/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

Comment thread package.json
"@types/node": "^22.15.1",
"@types/node-fetch": "^2.6.12",
"@types/pdfkit": "^0.11.0",
"@types/pdfkit": "^0.17.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Verify pdfkit runtime version and check if types should match
echo "=== Runtime pdfkit version ==="
rg '"pdfkit"' package.json -A1 -B1

echo ""
echo "=== Type definitions version ==="
rg '"@types/pdfkit"' package.json -A1 -B1

Repository: deepnote/vscode-deepnote

Length of output: 346


Align @types/pdfkit with runtime pdfkit version.

@types/pdfkit (^0.17.0) is ahead of runtime pdfkit (^0.13.0) and may reference unavailable APIs.

🤖 Prompt for AI Agents
In package.json around line 2610, the devDependency "@types/pdfkit": "^0.17.0"
is newer than the runtime "pdfkit" (^0.13.0) and may reference APIs that don't
exist at runtime; change the @types/pdfkit version to match the runtime (pin it
to a 0.13.x range compatible with pdfkit ^0.13.0, e.g. "^0.13.0" or the exact
0.13 version available), then run your package manager to install updated deps
and run the test/build to confirm no type/runtime mismatches.

Comment thread package.json
"@types/react": "^16.4.14",
"@types/react-dom": "^16.0.8",
"@types/react-json-tree": "^0.6.8",
"@types/react-json-tree": "^0.13.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Previously flagged as deprecated—upstream provides own types.

Past review noted package-lock.json deprecation: "react-json-tree provides its own type definitions." Consider removing this devDependency.

🤖 Prompt for AI Agents
In package.json around line 2614, the devDependency "@types/react-json-tree" is
redundant because react-json-tree ships its own types; remove the
"@types/react-json-tree" entry from devDependencies, run npm install (or yarn
install) to update package-lock.json (or yarn.lock), and then run the TypeScript
build/typecheck to ensure no type errors; if any references remain, remove or
update them accordingly.

Comment thread package.json
"@types/sinon": "^10.0.15",
"@types/sinonjs__fake-timers": "^6.0.1",
"@types/stack-trace": "0.0.29",
"@types/stack-trace": "0.0.33",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Verify stack-trace runtime version
echo "=== Runtime stack-trace version ==="
rg '"stack-trace"' package.json -C2

echo ""
echo "=== Type definitions version ==="
rg '"@types/stack-trace"' package.json -C2

Repository: deepnote/vscode-deepnote

Length of output: 511


@types/stack-trace (0.0.33) version ahead of runtime stack-trace (0.0.10).

Type definitions are for a newer version than what's installed at runtime, which may cause type errors if the APIs differ between these versions.

🤖 Prompt for AI Agents
In package.json around line 2621, the @types/stack-trace dependency is at 0.0.33
while the runtime stack-trace package is 0.0.10; update package.json so the
versions match (either set "@types/stack-trace" to "0.0.10" or upgrade the
runtime "stack-trace" to 0.0.33), then run your package manager install to
update lockfiles and re-run TypeScript type-checks (and tests) to ensure no API
mismatches remain.

Comment thread package.json
"@types/svg-to-pdfkit": "^0.1.0",
"@types/tcp-port-used": "^1.0.0",
"@types/temp": "^0.8.32",
"@types/temp": "^0.9.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Search for temp vs tmp usage
echo "=== Searching for 'temp' imports ==="
rg "from ['\"]temp['\"]|require\(['\"]temp['\"]" --type ts --type js -C2

echo ""
echo "=== Searching for 'tmp' imports ==="
rg "from ['\"]tmp['\"]|require\(['\"]tmp['\"]" --type ts --type js -C2

echo ""
echo "=== Package.json dependencies ==="
rg '"temp"|"tmp"' package.json -C1

Repository: deepnote/vscode-deepnote

Length of output: 1837


Remove unused @types/temp dependency.

Codebase imports only tmp package (found in 4 files), never temp. The @types/temp entry at line 2625 is unused; @types/tmp at line 2626 is the correct type definition to keep.

🤖 Prompt for AI Agents
In package.json around line 2625, the dependency entry "@types/temp": "^0.9.0"
is unused and should be removed; keep the correct "@types/tmp" entry. Edit
package.json to delete the "@types/temp" line and run npm/yarn install (or
update lockfile) to ensure the dependency is removed from lock files.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant