Skip to content

fix: map uv to correct semver definition#6719

Merged
thomasschafer merged 2 commits intomainfrom
fix/map-uv-semver
Apr 13, 2026
Merged

fix: map uv to correct semver definition#6719
thomasschafer merged 2 commits intomainfrom
fix/map-uv-semver

Conversation

@thomasschafer
Copy link
Copy Markdown
Contributor

@thomasschafer thomasschafer commented Apr 10, 2026

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages
    are release-note ready, emphasizing
    what was changed, not how.
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High) - Low
  • Highlights breaking API changes (if applicable) - n/a
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___) - n/a
  • Includes product update to be announced in the next stable release notes - n/a

What does this PR do?

This PR makes use of the fix here to map to the correct semver definition for uv. This resolves an issue with testing SBOMs generated directly by uv (it does not affect SBOMs of uv projects generated by snyk sbom).

I also had to update fake-server.ts. Extensions using a localhost API URL (e.g. os-flows/sbom) have it canonicalised by go-application-framework, stripping the /api prefix, so they call /hidden/orgs/.... Extensions using a non-localhost IP (e.g. aibom) skip canonicalisation and call /api/hidden/orgs/.... The fix registers the upload_revisions handlers for both prefixes, and updates response bodies to use dynamic path params rather than hardcoded UUIDs.

Finally, I also added an override for axios to address https://security.snyk.io/vuln/SNYK-JS-AXIOS-15965856.

How should this be manually tested?

Generate an SBOM for a uv project using

uv export --format=cyclonedx1.5 --preview --frozen --no-dev > sbom.json

then test it using

snyk sbom test --file=sbom.json

On main you should see an error like the following:

 ERROR   Unspecified Error (SNYK-CLI-0000)
         The encountered error only provides basic information, please take a look at
         the given details. If they do not help to resolve the issue, consider
         debugging or consulting support.

           failed to compute remediation summary: failed to resolve semver library: no
           semver library defined for ecosystem: uv

but on this branch you should see a successful run.

What's the product update that needs to be communicated to CLI users?

None

@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Apr 10, 2026

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.

@thomasschafer thomasschafer force-pushed the fix/map-uv-semver branch 5 times, most recently from d270b12 to cbf5cac Compare April 10, 2026 13:46
@thomasschafer thomasschafer marked this pull request as ready for review April 10, 2026 13:47
@thomasschafer thomasschafer requested review from a team as code owners April 10, 2026 13:47
@snyk-pr-review-bot

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

Warnings
⚠️ There are multiple commits on your branch, please squash them locally before merging!

Generated by 🚫 dangerJS against 0c4bdcc

@snyk-pr-review-bot

This comment has been minimized.

@snyk-pr-review-bot

This comment has been minimized.

@snyk-pr-review-bot
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 PR contains tests
🔒 No security concerns identified
⚡ No major issues detected
📚 Repository Context Analyzed

This review considered 13 relevant code sections from 3 files (average relevance: 0.78)

@thomasschafer thomasschafer enabled auto-merge April 13, 2026 11:15
@thomasschafer thomasschafer merged commit 1d055f4 into main Apr 13, 2026
9 checks passed
@thomasschafer thomasschafer deleted the fix/map-uv-semver branch April 13, 2026 12:06
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.

2 participants