Skip to content

feat: add HeroDevs attribution to SBOM generation#289

Merged
rlmestre merged 4 commits into
mainfrom
feat/add-sbom-attribution
Jul 29, 2025
Merged

feat: add HeroDevs attribution to SBOM generation#289
rlmestre merged 4 commits into
mainfrom
feat/add-sbom-attribution

Conversation

@rlmestre
Copy link
Copy Markdown
Contributor

Adds HeroDevs author and tools to generated SBOM.

Closes neverendingsupport/data-and-integrations/issues/208

@rlmestre rlmestre requested a review from a team as a code owner July 22, 2025 16:42
@rlmestre rlmestre requested a review from Copilot July 22, 2025 16:45
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds HeroDevs attribution to generated SBOM files by including HeroDevs as the author and adding the HeroDevs CLI as a tool in the metadata. The PR also improves logging consistency by only showing the "SBOM saved" message when files are actually saved to disk.

  • Updates SBOM configuration to include HeroDevs attribution in author and tools metadata
  • Refactors logging to show save confirmation message in the correct flow
  • Adds comprehensive e2e tests to verify SBOM generation and attribution

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/service/eol/cdx.svc.ts Adds HeroDevs author and CLI tool attribution to SBOM default options
src/commands/scan/sbom.ts Refactors logging to show save message only when files are saved
e2e/scan/sbom.test.ts Adds comprehensive e2e tests for SBOM generation and attribution verification
e2e/fixtures/npm/simple/sbom.json Updates test fixture with new HeroDevs attribution and additional dependencies

Comment thread src/service/eol/cdx.svc.ts
Comment thread src/service/eol/cdx.svc.ts Outdated
Comment thread src/service/eol/cdx.svc.ts
Comment thread e2e/scan/sbom.test.ts Outdated
@rlmestre rlmestre merged commit 790da15 into main Jul 29, 2025
23 of 24 checks passed
@rlmestre rlmestre deleted the feat/add-sbom-attribution branch July 29, 2025 14:36
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.

5 participants