Skip to content

docs: update material types list in README - #3377

Merged
migmartri merged 1 commit into
chainloop-dev:mainfrom
migmartri:dw/23053-1787780149
Aug 28, 2026
Merged

docs: update material types list in README#3377
migmartri merged 1 commit into
chainloop-dev:mainfrom
migmartri:dw/23053-1787780149

Conversation

@migmartri

@migmartrimigmartri commented Aug 26, 2026

Copy link
Copy Markdown
Member

Update the "Supported Pieces of Evidence / Materials" section in the README to reflect the full set of material types defined in crafting_schema.proto, including the recently added types (GHAS scans, Gitleaks, OpenAPI/AsyncAPI/GraphQL specs, TruffleHog, Cobertura, Checkmarx, Oversecured, PIT, radamsa, Sysinternals, and more).

🤖 Posted by Maximus bot (Claude Code) on behalf of @migmartri

Review in cubic

Assisted-by: OpenCode
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Chainloop-Trace-Sessions: ses_fbfff3addffe5TVsaanKY6jGSB
@chainloop-platform

chainloop-platformBot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

AI Session Checks — 🟢 84% · ✅ 0 failing

Avg scoreSessionsFailing policiesAttributionFilesLinesTotal Duration
🟢 84%1✅ 0100% AI / 0% Human1+34 / -112m31s

🟢 84% — 100% AI — ✅ All policies passing

Aug 26, 2026 21:36 UTC · 2m31s · $0.16 · 101.1k in / 2.6k out · opencode 1.17.20 (deepseek/deepseek-v4-pro)

View session details ↗

Change Summary

Documentation-only change. README's material list was expanded from the linked MaterialType proto enum, and the follow-up work moved toward a README-only commit.

AI Session Overall Score

🟢 84% — Docs update was clean, but the PR-creation follow-up appears incomplete.

AI Session Analysis Breakdown

🟢 96% · scope-discipline

🟢 The visible boundary stayed within README.md only. · High Impact

🟢 92% · context-and-planning

🟢 User supplied the target file and authoritative source up front. · High Impact

🟢 90% · user-trust-signal

No notes.

🟡 72% · alignment

🟢 The update stayed anchored to the linked proto-backed material list. · High Impact

🟠 After the user asked for a PR, the session shows git prep and commit steps but no visible PR creation or blocker. · Medium Severity

💡 When a user asks for a PR, either complete the push/PR flow or explicitly say what blocked you.

abstained · solution-quality

🟡 Solution-quality abstained because the session only edited documentation, not executable code. · Low Severity

abstained · verification

🟡 Verification abstained: this was a docs-only README change with no runtime behavior to validate. · Low Severity

Missing criteria: solution-quality, verification


File Attribution

████████████████████ 100% AI / 0% Human

StatusAttributionFileLines
modifiedaiREADME.md+34 / -11

Policies (4)

StatusPolicyMaterialMessages
✅ Passedai-config-ai-agents-allowedai-coding-session-ses-fb-
✅ Passedai-config-no-dangerous-commandsai-coding-session-ses-fb-
✅ Passedai-config-no-secretsai-coding-session-ses-fb-
✅ Passedai-config-mcp-servers-allowedai-coding-session-ses-fb-

Security Checks — ⚠️ 1 failing

PR info

StatusPolicyMessages
✅ Passedpr-description-required-
⚠️ Failedpr-user-story-linkedPR/MR #3377 does not reference a user story or issue in title, description, or branch 'docs: update material types list in README'. Expected patterns: ["(?i)[A-Z]+-[0-9]+", "#[0-9]+", "(?i)[A-Z]{2", "}-[0-9]+", "(?i)gh-[0-9]+", "(?i)\[[A-Z]+-[0-9]+\]"]

⏭️ 3 scans not applied

ScanReason
vulnerability-scanno manifest/lockfile changed
github-actions-scanno workflow files changed
iac-scanno IaC files changed

View attestation ↗


Powered by Chainloop and Chainloop Trace

@migmartri
migmartri requested a review from a teamAugust 26, 2026 21:40

@cubic-dev-aicubic-dev-aiBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

@migmartri
migmartri merged commit 294cad6 into chainloop-dev:mainAug 28, 2026
16 of 17 checks passed
Sign up for freeto 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

@migmartri@jiparis