Skip to content

fix(security): include pinned versions in OSV vulnerability queries - #79

Merged
knight22-21 merged 1 commit into
knight22-21:mainfrom
Subodh-17:feat/version-aware-cve
Nov 5, 2026
Merged

knight22-21 merged 1 commit into
knight22-21:mainfrom
Subodh-17:feat/version-aware-cve

Conversation

@Subodh-17

Copy link
Copy Markdown
Contributor

Summary

Fixes #57.

  • Include exact dependency versions in OSV vulnerability query payloads when available.
  • Parse exact == and === pins from requirements files.
  • Handle package extras, inline comments and environment markers.
  • Keep wildcard, ranged and unpinned dependencies on the existing unversioned query path.
  • Preserve existing network-failure fallback behavior.

Tests

  • CVE tests: 13 passed.
  • Security subsystem tests: 81 passed.
  • Ruff lint: passed.
  • Ruff formatting: passed.
  • Diff whitespace check: passed.

The full repository suite has known baseline environment failures involving optional tree-sitter grammars and Windows CLI behavior; these are documented separately and are not claimed as passing.

@knight22-21 knight22-21 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

All CI checks pass. Changes look solid — pinned version extraction is clean, the fallback path for unversioned deps is preserved, and the OSV payload construction handles extras, inline comments, and environment markers correctly. Good work.

@knight22-21
knight22-21 merged commit 27f9f4a into knight22-21:main Nov 5, 2026
4 checks passed
@knight22-21

Copy link
Copy Markdown
Owner

Thanks @Subodh-17! This is a meaningful improvement to the security subsystem — OSV queries now carry exact pinned versions when available, making vulnerability lookups significantly more precise. The extras/comment/marker parsing is a nice touch too. Appreciate the thorough PR description and clean commit.

@knight22-21

Copy link
Copy Markdown
Owner

Hey @Subodh-17 — if you find CodePrism useful, consider giving the repo a ⭐ at https://github.com/knight22-21/CodePrism. It really helps with visibility!

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.

check_package ignores the version: patched pins are reported with every historical CVE

2 participants