-
Notifications
You must be signed in to change notification settings - Fork 3
Add Dependency Review and Submission #21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 109 commits
Commits
Show all changes
123 commits
Select commit
Hold shift + click to select a range
c7899b5
Added using Dependency Review on branches
aegilops d2a6b9f
Added Dependency Submission with Component Detection
aegilops ec32784
Added language choice to submission
aegilops 5043ccb
Debug of submission, partial progress, but not working yet
aegilops a729fb3
Single repo scanning added. More scaffolding added to enable Dep Revi…
aegilops 506b9a2
Runs component-detection now, but is killed on MacOS
aegilops 68f5d22
Allow using component-detection at a provided path
aegilops 51a111e
Refactor, make branch submission independent of freshly fetching SBOM
aegilops 6f43ed0
Removed unused function
aegilops c002406
Works for branch changes
aegilops 4787c2e
Formatting
aegilops 290c070
Updated malware matching to do diffs
aegilops e55933e
Updated README
aegilops 3e94895
Removed default branch limit
aegilops 5364da0
Update to better handling rate limiting when querying malware database
aegilops 00116c5
Allows just malware sync/caching
aegilops 5c92626
Fix single repo behaviour on first pass
aegilops 9f284c2
Fixed missing diff from skipped branches
aegilops f6d37c2
Allow forcing submission with Component Detection
aegilops 33415a9
Fixed not waiting for process
aegilops 3e65ebc
Now runs Component Detection correctly, on correct path
aegilops 6314b14
Fixed mistake reading wrong path in Component Detection
aegilops 99b20d9
Fixed missing graphs by normalising appropriately
aegilops 3a89bfb
CHANGELOG
aegilops 1887425
Fixed submission by fixing calling of git to capture STDOUT properly
aegilops 9ea2fa0
Update docs
aegilops 7f13c04
Update GHES URL for listing orgs
aegilops dab9c61
CHANGELOG
aegilops 42bc4ac
Merge branch 'main' into dependency-review
aegilops 12b5390
Potential fix for pull request finding 'Unused variable, import, func…
aegilops 3b35d1c
Potential fix for pull request finding 'Semicolon insertion'
aegilops f3135f5
Improved property assignment from untrusted value
aegilops 5cea67b
Update src/componentDetection.ts
aegilops 9364ebc
Update src/cli.ts
aegilops afca11d
Update src/test-branch-search.ts
aegilops 5d18ee5
Update src/componentDetection.ts
aegilops b4fa046
Update src/types.ts
aegilops fc20a4c
Update src/componentSubmission.ts
aegilops 50cc8fb
Initial plan
Copilot 8554a0a
Fix some issues
aegilops 9c54209
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops 83fb834
Update src/componentDetection.ts
aegilops ac8ea00
Fix submitSnapshot to return boolean indicating success/failure
Copilot 15bd0e9
Initial plan
Copilot 988b8ef
Add missing JSDoc parameter documentation for octokit
Copilot 1d0d38c
Fix numbering in MD
aegilops c16aa51
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops e262cf5
Update src/cli.ts
aegilops ebf5b45
Fixed const/let
aegilops 410f95e
Fix latestCommitDate to use actual commit date from API
Copilot a7a725b
Removed newVersion
aegilops 5544df5
Update src/sbomCollector.ts
aegilops 6c6c8b9
Update src/componentDetection.ts
aegilops a001a19
Update src/componentSubmission.ts
aegilops aafcd0f
Update package.json
aegilops 3c40bc2
Allow undefined commit date
aegilops 09cafe8
Fixed up Component Detection/Submission
aegilops da53c0b
Fixed missing await
aegilops a67db56
Updated packages
aegilops 98dee3d
Fixed test
aegilops bc125d7
Updated test running
aegilops 8b51d9a
Remove test file and put in ignore file
aegilops 95896ab
Update src/componentSubmission.ts
aegilops 95a7fd0
Update src/componentSubmission.ts
aegilops 93d441c
Update src/componentDetection.ts
aegilops bd656c6
Update README.md
aegilops 51bde41
Update src/sbomCollector.ts
aegilops f909bdc
Update src/componentSubmission.ts
aegilops f3673af
Update src/componentDetection.ts
aegilops 938d965
Fix repo URL
aegilops 8e6e341
Update src/componentSubmission.ts
aegilops 580711e
Made job id nearly impossible to clash
aegilops 5c8606b
Removed pointless cast to any
aegilops 8e33d30
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops bdfe2f6
Move error count outside scope
aegilops ca7d8b6
Limit retries of diff
aegilops f2296bd
Initial plan
Copilot 22fde03
Convert async forEach loops to for...of loops to fix race conditions
Copilot f119054
Add proper type definitions for ComponentDetectionPackage constructor
Copilot 72477cf
Merge branch 'dependency-review' into copilot/sub-pr-21-another-one
aegilops dccef0c
Merge pull request #24 from advanced-security/copilot/sub-pr-21-anoth…
aegilops 3991ca0
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops ee77207
Merge branch 'dependency-review' into copilot/sub-pr-21-one-more-time
aegilops 6babca9
Merge pull request #26 from advanced-security/copilot/sub-pr-21-one-m…
aegilops 8d93e53
Better repo handling for --repo
aegilops d6cad1f
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops e103f8a
Merge pull request #27 from advanced-security/copilot/sub-pr-21-pleas…
aegilops 5b00aaf
Sort out retries parameter and handling skipping submission
aegilops 546e33d
Fixed missing backticks
aegilops 82b58da
Added checking retries and made it pre-decrement
aegilops e6b11d9
Update src/componentDetection.ts
aegilops f963d36
Use exact comparison
aegilops 1d04ee8
Update src/sbomCollector.ts
aegilops 46f8f8c
Clarified comment
aegilops fb52fb9
Removed type ignore
aegilops 4997078
Update src/componentDetection.ts
aegilops e3aaf6f
Formatted
aegilops 4dcf2b3
Update src/sbomCollector.ts
aegilops 5ac59f7
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops ba8ff05
Update src/componentDetection.ts
aegilops 5431556
Update src/sbomCollector.ts
aegilops 4ad422a
Initial plan
Copilot a1e931f
Merge branch 'main' into dependency-review
aegilops 395c925
Make snapshot ingestion delays configurable
Copilot 4b2e75c
Update src/malwareMatcher.ts
aegilops 3d5f9ad
Remove redundant 'ms' from log message
Copilot e7f34a6
Add CLI arguments for snapshot ingestion delays
Copilot de9be57
Merge pull request #28 from advanced-security/copilot/sub-pr-21-a1e28…
aegilops bd2e92e
Update src/componentDetection.ts
aegilops a798460
Fix mismerged code
aegilops bfa1e6a
Remove redundant assignment
aegilops 5607399
Fix SBOM counts and caching
aegilops 9a78c60
Update src/malwareMatcher.ts
aegilops 6be40f1
Remove default for malware-cache
aegilops d5bad5a
Added namespace to type
aegilops 3bd2d15
Merge branch 'dependency-review' of https://github.com/advanced-secur…
aegilops b2d1d03
Minor changes to logging
aegilops a502377
Remove unused type
aegilops 4efe35e
Add more delays to processing
aegilops abe838a
Update CHANGELOG
aegilops 2339190
Minor README updates
aegilops 810fda3
Encoding URI parts in PURL, removed unneeded code
aegilops 2d3fe41
Update src/cli.ts
aegilops File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,4 +3,6 @@ dist/ | |
| .env | ||
| data/ | ||
| .vscode/ | ||
| .DS_Store | ||
| .DS_Store | ||
| component-detection | ||
| tmp-branch-search-cache/ | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| [submodule "component-detection-dependency-submission-action"] | ||
| path = component-detection-dependency-submission-action | ||
| url = https://github.com/advanced-security/component-detection-dependency-submission-action.git |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| # Changelog | ||
|
|
||
| ## [2025-12-04] – 0.2.0 - Branch scanning and dependency submission | ||
|
|
||
| Added: | ||
|
|
||
| - Branch scanning: | ||
| - Fetch SBOM diffs for non‑default branches via Dependency Review API. | ||
| - Added `--branch-scan`, `--branch-limit`, and `--diff-base` CLI flags. | ||
| - Dependency Submission integration: | ||
| - Automatically submits dependency snapshots for branches being scanned, if not already present, using Component Detection. | ||
| - Language-aware sparse checkout. | ||
| - Use a pre-downloaded binary (`--component-detection-bin`) or an auto-downloaded release. | ||
| - Search and matching: | ||
| - Refactored search to de-duplicate logic and include branch diffs (added/updated packages only). | ||
aegilops marked this conversation as resolved.
Show resolved
Hide resolved
aegilops marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| - Malware matching enhanced to enumerate packages from diffs; matches annotated with branch. | ||
| - CLI and CSV outputs include branch context; CSV adds a `branch` column. | ||
| - CLI and UX improvements: | ||
| - Argument validation updated: `--sync-sboms` requires `--sbom-cache`. | ||
| - Malware-only mode: allow `--sync-malware` without `--sbom-cache` (requires `--malware-cache`). | ||
| - JSON/CLI/CSV interaction clarified and documented. | ||
| - Added examples for malware-only sync and branch scanning. | ||
| - Advisory sync robustness: | ||
| - GraphQL advisory sync now implements adaptive retries with exponential backoff and `Retry-After` support; respects `--quiet`. | ||
|
|
||
| Fixed: | ||
|
|
||
| - Added `--ghes` flag to ensure proper API URL construction for GitHub Enterprise Server instances. | ||
|
|
||
| ## [2025-10-06] - 0.1.0 - Initial public release | ||
|
|
||
| - Initial release, with: SBOM sync; malware sync; malware matching; CLI, file based and interactive PURL searching. SARIF, CSV and JSON outputs supported. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.