Skip to content

docs: describe signed sbom bundles in the sbom attestation type - #538

Merged
mbevc1 merged 2 commits into
mainfrom
autopilot/9045152c
Nov 13, 2026
Merged

mbevc1 merged 2 commits into
mainfrom
autopilot/9045152c

Conversation

@mintlify

@mintlify mintlify Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

The SBOM attestation type now explains that kosli attest sbom accepts signed SBOMs in Sigstore bundles or DSSE envelopes, checks their subject and can verify the signer. It also points to the beta kosli attest in-toto. GA for attest sbom bundles, beta for attest in-toto; both released in CLI v2.56.0.

Based on feat: add kosli attest in-toto, recording attestations from bundle files.

@mintlify
mintlify Bot requested a review from a team as a code owner November 13, 2026 07:09
@mintlify
mintlify Bot requested a review from dangrondahl November 13, 2026 07:09
@mintlify

mintlify Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor Author

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
kosli 🟢 Ready View Preview Nov 13, 2026, 7:15 AM

💡 Tip: Enable Automations to automatically generate PRs for you.

@claude

claude Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @mintlify[bot]'s task in 1m 3s —— View job


Follow-up review of latest commits

Previous finding — resolved. The earlier comment flagged that the sentence on fingerprint-based subject checking didn't cover the case where a bundled SBOM is bound by template name and commit, or to the trail alone (no fingerprint, so the subject goes unchecked) — which could let a reader mistake an unchecked subject for verified evidence.

Commit 6885611 ("note when a bundled sbom subject is not checked") applies essentially the suggested fix verbatim in getting_started/attestations.md:310-316:

"When the artifact's fingerprint is known, calculated or given with --fingerprint, the attestation must list that artifact as its subject. Attested to a trail alone, or bound by template name and commit, it has no fingerprint, so the subject is not checked."

I cross-checked this against the generated reference (client_reference/kosli_attest_sbom.md:28-32) and the wording is consistent and accurate.

New issues in the changed lines: none. The addition is a self-contained clarification; no bugs or regressions introduced.

Note: I wasn't able to query the GitHub review-comments API in this session (sandboxed, requires approval I can't grant), so I couldn't programmatically resolve the original inline thread — flagging this status here instead. The underlying docs change itself is sound.
· branch autopilot/9045152c

Comment thread getting_started/attestations.md Outdated
Comment on lines +313 to +314
CycloneDX document from inside it. When the artifact's fingerprint is known, the attestation
must list that artifact as its subject. To verify who signed it, pass `--certificate-identity`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Improvement: this leaves out the case where the subject is not checked. The generated reference (client_reference/kosli_attest_sbom.md:28-32) says that when the attestation is bound by template name and commit, or to the trail alone, there is no fingerprint, so the subject is not checked. Line 336 now says only a plain SBOM's subject goes unchecked. Read together, the two lines suggest a bundled SBOM's subject is always checked, and a reader attesting to a trail could treat it as verified evidence when it isn't.

Suggested change
CycloneDX document from inside it. When the artifact's fingerprint is known, the attestation
must list that artifact as its subject. To verify who signed it, pass `--certificate-identity`
CycloneDX document from inside it. When the artifact's fingerprint is known, calculated or
given with `--fingerprint`, the attestation must list that artifact as its subject. Attested
to a trail alone, or bound by template name and commit, it has no fingerprint, so the subject
is not checked. To verify who signed it, pass `--certificate-identity`

@mbevc1
mbevc1 merged commit b72a143 into main Nov 13, 2026
6 checks passed
@mbevc1
mbevc1 deleted the autopilot/9045152c branch November 13, 2026 09:24

This branch was successfully deployed

1 active deployment
staging — 3930db1b Deployed Nov 13, 2026 by mintlify[bot]
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.

2 participants