Skip to content

Compliance: claude-missing-check-run-trigger #64

Closed

Description

@don-petry

Compliance Finding

Category: ci-workflows
Severity: warning
Check: claude-missing-check-run-trigger

Detail

The claude.yml workflow is missing the check_run trigger. Without it the claude-ci-fix job cannot respond to CI failures on PRs automatically. Add check_run: types: [completed] to the on: block.

Standard Reference

standards/ci-standards.md#4-claude-code-claudeyml

Remediation

Please review the linked standard and bring this repository into compliance.

See the full standards documentation for implementation guidance.


This issue was automatically created by the weekly compliance audit.

Activity

  1. don-petry commented on May 8, 2026

    @don-petry
    CollaboratorAuthor

    Claude finished @don-petry's task in 2m 43s —— View job • claude/issue-64-20260508-1419 • [Create PR ➔](https://github.com/petry-projects/.github-private/compare/main...claude/issue-64-20260508-1419?quick_pull=1&title=Issue%20%2364%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%2364%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)


    Tasks

    • Gather context
    • Check for existing open PRs (none found)
    • Read CLAUDE.md and standard template
    • Read current claude.yml
    • Apply fix — replaced inlined workflow with thin-caller stub
    • Commit and push
    • Self-review PR (no issues found)
    • Check CODEOWNERS (no owners file for this path)

    Summary

    Replaced the repo-local inlined claude.yml with the org-standard thin-caller stub per petry-projects/.github/standards/workflows/claude.yml. Key changes:

    • Added check_run: types: [completed] trigger (fixes the compliance finding)
    • Added paths-ignore OIDC guard on pull_request trigger (prevents 401 errors on future PRs that only change this file)
    • Delegated all logic (prompt, allowedTools, claude-ci-fix job, claude-issue job) to the org reusable workflow claude-code-reusable.yml@v1
    • Reduced the file from 181 lines to 65 lines

    ⚠️ The Claude Code CI check on this PR will likely show a 401 OIDC failure — this is expected behavior documented in the file header. The paths-ignore guard prevents this on future PRs once this lands on main.

    PR #84 |

  2. don-petry commented on May 11, 2026

    @don-petry
    CollaboratorAuthor

    Resolved! This check is now passing as of 2026-05-11. Closing automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    compliance-auditAutomated compliance audit finding

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions