Skip to content

chore: bump extensions for SBOM per-component results - #7149

Open
MoisBrigitte wants to merge 1 commit into
mainfrom
chore/bump-gaf-asset-link
Open

chore: bump extensions for SBOM per-component results#7149
MoisBrigitte wants to merge 1 commit into
mainfrom
chore/bump-gaf-asset-link

Conversation

@MoisBrigitte

@MoisBrigitte MoisBrigitte commented Aug 18, 2026

Copy link
Copy Markdown

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)
  • Highlights breaking API changes (if applicable) — none
  • 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

What does this PR do?

Dependency bumps only, no source changes. Both cliv2 and cliv2-private:

Module From (current main) To Upstream PR
go-application-framework v0.16.0 v0.16.1-0.20260819102712-53729422c5cd (5372942) snyk/go-application-framework#707
cli-extension-os-flows v0.0.0-20260818092349-91c745a9ee7a v0.0.0-20260819103218-611f8d16dc54 (611f8d1) snyk/cli-extension-os-flows#270

Together they change how SBOM test results are presented:

Draft: both pins are branch commits on unmerged PRs. Needs re-pointing at main pseudo-versions once they land, then it can come out of draft.

Where should the reviewer start?

cliv2/go.mod and cliv2-private/go.mod, plus the corresponding go.sum entries. No source changes.

How should this be manually tested?

Requires an org with SBOM testing and asset inventory enabled:

snyk sbom test --experimental --file=<multi-component-sbom>.json --report
  • One result section per component, each with its own header and test summary, then an overall test summary — rather than one combined result.
  • View your asset(s) at: … appears exactly once, immediately below the Overall Test Summary box. Before, it appeared once per component, mid-output.

Worth smoke testing that plain snyk test and snyk test --all-projects are unchanged — neither reports an asset, so neither should print that line at all.

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

snyk sbom test now reports one result per component in the SBOM, rather than a single combined result. With --report, the asset inventory link is shown once at the end of the results instead of under every component.

Risk assessment (Low | Medium | High)?

Medium. Two things carry the risk, neither of them in this repo:

  • The os-flows half changes the shape of snyk sbom test output — one result per component, and one entry per component in the legacy JSON — so anything parsing that output sees a different structure.
  • The GAF half is presentation-only: one template block moves, and a helper reads the link from whichever result carries it. No payload or API change.

Any background context you want to provide?

A modelling gap sits underneath the asset link fix: asset is a property of the test, but the UFM payload is a bare array of results with nowhere to put test-level facts, so it is copied onto every result and then stripped from all but one by the producer. Closing that gap is prepared separately, stacked on these branches — it also needs a change to cli-extension-axi, which parses the payload itself rather than using the framework's decoder. Not part of this PR.

Links to automated tests

Both upstream PRs carry their own coverage:

  • GAF: multi_project_with_asset in Test_UfmPresenter_HumanReadable — there was no test for the asset link at all before.
  • os-flows: Test_RunSbomFlow_ReportsTestWideLinksOnce and the TestUnifiedFindingPresenter_AssetLink subtests.

Both modules build clean here (go build ./... in cliv2 and cliv2-private).

🤖 Generated with Claude Code

@snyk-io

snyk-io Bot commented Aug 18, 2026

Copy link
Copy Markdown

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.

@MoisBrigitte MoisBrigitte changed the title chore: bump go-application-framework for the asset link fix chore: bump go-application-framework and cli-extension-os-flows for SBOM per-component results Aug 18, 2026
@MoisBrigitte
MoisBrigitte force-pushed the chore/bump-gaf-asset-link branch 8 times, most recently from 72629a4 to eba5b1f Compare August 19, 2026 10:33
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor
Warnings
⚠️

"chore: bump go-application-framework and cli-extension-os-flows for SBOM per-component results" is too long. Keep the first line of your commit message under 72 characters.

Generated by 🚫 dangerJS against ae7be42

@MoisBrigitte
MoisBrigitte marked this pull request as ready for review August 19, 2026 13:55
@MoisBrigitte
MoisBrigitte requested a review from a team as a code owner August 19, 2026 13:55
@MoisBrigitte
MoisBrigitte force-pushed the chore/bump-gaf-asset-link branch from 73cae97 to b79c7ae Compare August 19, 2026 13:55
@MoisBrigitte MoisBrigitte changed the title chore: bump go-application-framework and cli-extension-os-flows for SBOM per-component results chore: bump extensions for SBOM per-component results Aug 19, 2026
@snyk-pr-review-bot

This comment has been minimized.

@snyk-pr-review-bot

This comment has been minimized.

…BOM per-component results

cli-extension-os-flows now reports one result per component for SBOM
tests, so `snyk sbom test` renders a section per component instead of a
single section covering the whole document.

go-application-framework v0.16.1 carries the matching change to the UFM
human readable template, which renders the asset inventory link once
below the overall test summary rather than under each result.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@MoisBrigitte
MoisBrigitte force-pushed the chore/bump-gaf-asset-link branch from b79c7ae to ae7be42 Compare August 19, 2026 13:58
@snyk-pr-review-bot

Copy link
Copy Markdown

PR Reviewer Guide 🔍

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

This review considered 7 relevant code sections from 4 files (average relevance: 1.00)

🤖 Repository instructions applied (from AGENTS.md)

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.

1 participant