Skip to content

fix(javascript): patch SDK tooling vulnerabilities - #5335

Merged
turip merged 1 commit into
mainfrom
feat/security-javascript-dependencies
Oct 22, 2026
Merged

turip merged 1 commit into
mainfrom
feat/security-javascript-dependencies

Conversation

@turip

@turip turip commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Patch the JavaScript SDK build, generation, and test dependency graph to address its 13 high and 7 medium Dependabot records. Select Vite 7.3.7 explicitly, upgrade Vitest to 4.1.12 for follow-up mocker fixes, and enforce patched transitive versions of brace-expansion, fast-uri, form-data, js-yaml, and markdown-it.

Validation:

  • SDK generation and dual ESM/CommonJS build; regeneration produces no tracked output changes.
  • Vitest tests and TypeScript/Biome checks.
  • Live dependency audit has no moderate, high, or critical findings. One existing low-severity esbuild advisory remains in development tooling.

The published SDK runtime dependencies are unchanged.

RetriggerView in GreptileConfidence Score: 5/5

This PR appears safe to merge; no actionable issues were found.

Summary

Updates the JavaScript SDK’s development tools to address dependency advisories.

  • SDK build and test tools use newer Vite and Vitest releases.
  • SDK installs select patched versions of five transitive packages.

No actionable issues found. Build, generation, tests, and dependency audit were not run during this review.

Reviews (1) · Last reviewed commit: "fix(javascript): patch SDK tooling vulne..." · Reviewed by Greptile

@turip
turip requested a review from a team as a code owner October 22, 2026 08:04
@coderabbitai

coderabbitai Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters: 3
  • api/client/javascript/package.json is excluded by !api/client/**
  • api/client/javascript/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml, !api/client/**
  • api/client/javascript/pnpm-workspace.yaml is excluded by !api/client/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@turip turip added area/dependencies Pull requests that update a dependency file dependency/javascript Pull requests that update Javascript code release-note/dependency-update Release note: Dependency Updates labels Oct 22, 2026
@turip
turip merged commit 2dc4a35 into main Oct 22, 2026
39 of 44 checks passed
@turip
turip deleted the feat/security-javascript-dependencies branch October 22, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dependencies Pull requests that update a dependency file dependency/javascript Pull requests that update Javascript code release-note/dependency-update Release note: Dependency Updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants