Skip to content

refactor: remove legacy Snyk Code client - #7081

Open
fsargent wants to merge 3 commits into
snyk:mainfrom
fsargent:code-client-removal/remove-legacy-typescript-client
Open

refactor: remove legacy Snyk Code client#7081
fsargent wants to merge 3 commits into
snyk:mainfrom
fsargent:code-client-removal/remove-legacy-typescript-client

Conversation

@fsargent

@fsargent fsargent commented Aug 4, 2026

Copy link
Copy Markdown

Summary

  • Remove the unused TypeScript Snyk Code implementation and @snyk/code-client dependency.
  • Keep Snyk Code on the native code-client-go workflow.
  • Reject accidental Snyk Code use through legacy TypeScript entry points.
  • Remove tests for the retired implementation and keep native workflow coverage.

This replaces the stale PR #6944. File Upload API support and user-journey coverage have since merged in PR #7061 and PR #7064.

Test plan

  • npm run lint
  • npx jest --runInBand test/jest/unit/ecosystems.spec.ts test/jest/unit/validate-fix-command-is-supported.spec.ts
  • npm run build:dev
  • Snyk Code scan. No finding affects changed code.

Snyk Code now runs through code-client-go. Remove the unused TypeScript
implementation and guard legacy TypeScript entry points.
@fsargent
fsargent requested a review from a team as a code owner August 4, 2026 12:10
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@snyk-io

snyk-io Bot commented Aug 4, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Comment thread src/cli/commands/test/index.ts
Comment thread src/cli/commands/fix/validate-fix-command-is-supported.ts Outdated
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