Skip to content

chore: remove the ai-config-policies group from the author verification contract - #3753

Merged
jiparis merged 1 commit into
chainloop-dev:mainfrom
jiparis:jiparis/pfm-8051-author-verification-contract
Oct 29, 2026
Merged

jiparis merged 1 commit into
chainloop-dev:mainfrom
jiparis:jiparis/pfm-8051-author-verification-contract

Conversation

@jiparis

@jiparis jiparis commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

The author-verification workflow fails the ai-instructions-size-bounds policy, because CLAUDE.md has more than 200 lines. The policy comes from the ai-config-policies group in the contract of this workflow.

This change removes the ai-config-policies group from the contract. The source-commit policy and the slsa-checks group stay.

Refs PFM-8051.

AI disclosure: Claude Code helped to write this change.

View guided diff

…on contract

Refs: PFM-8051

Assisted-by: Claude Code
Signed-off-by: Jose I. Paris <jiparis@chainloop.dev>
@chainloop-platform

chainloop-platform Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

PR validation — ✅ 3 passing

Status Policy Material Messages
✅ Passed pr-min-approvals pr-info -
✅ Passed pr-description-required pr-info -
✅ Passed pr-user-story-linked pr-info -

View attestation ↗

AI Session Checks — ⚠️ no AI session found

Missing AI Coding Sessions

This organization requires every PR to be backed by a Chainloop Trace AI coding session, and none was found for this one.

Please make sure the AI coding session evidence has been sent by the Chainloop CLI, or add the skip-ai-session label to this PR to bypass this check.

Learn more about Chainloop Trace.

Security Checks — ✅ 7 passing

✅ secret-scan

Status Policy Messages
✅ Passed secrets-detection -

✅ github-actions-scan

Status Policy Messages
✅ Passed ci-pipeline-security -

✅ sast-scan

Status Policy Messages
✅ Passed owasp-top10-2025 -
✅ Passed sast -
✅ Passed cwe-top25 -
✅ Passed cwe-top26-40-cusp -

✅ iac-scan

Status Policy Messages
✅ Passed iac-misconfiguration -
Scan not applied (1)
Scan Reason
vulnerability-scan no manifest/lockfile changed

View attestation ↗

Security context

✅ Nothing this change touches has a recorded security-fix history.

View in Chainloop ↗ · How this works ↗


Powered by Chainloop and Chainloop Trace

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 1 file

View guided diff | Re-trigger cubic

@jiparis
jiparis requested a review from a team October 29, 2026 17:37
@jiparis
jiparis merged commit 0e0dfe1 into chainloop-dev:main Oct 29, 2026
17 of 18 checks passed
@jiparis
jiparis deleted the jiparis/pfm-8051-author-verification-contract branch October 29, 2026 17:56
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