Skip to content

feat(vulnera): drop Snyk support (#325)#327

Merged
fraxken merged 2 commits intoNodeSecure:mainfrom
HoyeongJeon:drop-snyk-support
Mar 24, 2026
Merged

feat(vulnera): drop Snyk support (#325)#327
fraxken merged 2 commits intoNodeSecure:mainfrom
HoyeongJeon:drop-snyk-support

Conversation

@HoyeongJeon
Copy link
Contributor

Drops all Snyk-related code as Snyk APIs are locked behind a paywall and maintaining support is no longer viable.

Removed components:

  • Strategy (src/strategies/snyk.ts) — Snyk vulnerability fetching strategy
  • Database (src/database/snyk.ts) — Snyk API client class
  • Format (src/formats/snyk/) — Snyk-specific vulnerability format and mappers
  • VULN_MODE.SNYK constant removed from src/constants.ts
  • Residual mapFromSnyk code removed from standard format mappers
  • All related tests, fixtures, and documentation
  • Updated outdated origin comment in docs/formats/standard.md (Snyk, NodeSWG → Sonatype, GitHub, OSV)

Closes #325

@fraxken
Copy link
Member

fraxken commented Mar 23, 2026

That's not caused by your PR, but can you add a commit to fix lint issues? Thanks

@PierreDemailly
Copy link
Member

@HoyeongJeon tests fails due to last commit

@fraxken
Copy link
Member

fraxken commented Mar 23, 2026

Can you please squash the two last commits together ?

@fraxken fraxken merged commit 28c309c into NodeSecure:main Mar 24, 2026
10 checks passed
@fraxken
Copy link
Member

fraxken commented Mar 24, 2026

@allcontributors please add @HoyeongJeon for code

@allcontributors
Copy link
Contributor

@fraxken

I've put up a pull request to add @HoyeongJeon! 🎉

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.

Drop support of Snyk

3 participants