Skip to content

feat: import chisel in scalibrplugin#2772

Open
zhijie-yang wants to merge 4 commits intogoogle:mainfrom
zhijie-yang:ROCKS-2010/add-chisel-scalibr-plugin
Open

feat: import chisel in scalibrplugin#2772
zhijie-yang wants to merge 4 commits intogoogle:mainfrom
zhijie-yang:ROCKS-2010/add-chisel-scalibr-plugin

Conversation

@zhijie-yang
Copy link
Copy Markdown

Description

This PR imports the os/chisel extractor from the osv-scalibr in the scalibrplugin/presets.go to enable the scanning of container images built with Chisel.

The description of Ubuntu chiseled packages is added to docs/supported_languages_and_lockfiles.md, which corresponds to the changes of this PR.

This PR expects no breaking changes nor regressive UX to be introduced to the OSV-Scanner.

Related pull requests

google/osv-scalibr#764
google/osv-scalibr#2018

P.S. I've run make refresh-all REBUILD_IMAGES=true to update the snapshots.

FYI: @cjdcordeiro

@another-rex another-rex force-pushed the ROCKS-2010/add-chisel-scalibr-plugin branch from 64e6842 to 8b9aa1d Compare May 7, 2026 03:58
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.09%. Comparing base (22fbc22) to head (9168fb5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2772      +/-   ##
==========================================
+ Coverage   79.06%   79.09%   +0.02%     
==========================================
  Files         120      120              
  Lines        8154     8154              
==========================================
+ Hits         6447     6449       +2     
+ Misses       1324     1323       -1     
+ Partials      383      382       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Collaborator

@another-rex another-rex left a comment

Choose a reason for hiding this comment

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

Can you added a e2e test by added a dockerfile that can build a chisel image, which can be scanned by scanner?

@zhijie-yang
Copy link
Copy Markdown
Author

Added as cmd/osv-scanner/scan/image/testdata/test-chisel.Dockerfile.

@zhijie-yang zhijie-yang requested a review from another-rex May 7, 2026 19:27
@another-rex another-rex force-pushed the ROCKS-2010/add-chisel-scalibr-plugin branch from 8a3f52e to 9168fb5 Compare May 8, 2026 00:29
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