Skip to content

Compliance: security_and_analysis_unavailable #111

Closed

Description

@don-petry

Compliance Finding

Category: push-protection
Severity: warning
Check: security_and_analysis_unavailable

Detail

Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

Standard Reference

standards/push-protection.md#required-repo-level-settings

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 Apr 9, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-04-09)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  2. don-petry commented on Apr 9, 2026

    @don-petry
    ContributorAuthor

    Claude encountered an error —— View job


    I'll analyze this and get back to you.

  3. don-petry commented on Apr 10, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-04-10)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  4. don-petry commented on Apr 14, 2026

    @don-petry
    ContributorAuthor

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


    Fixing Compliance Issue #111

    • Gather context: read push-protection standard and existing repo workflows
    • Understand the finding
    • Add secret-scan job to ci.yml
    • Add *.pem to .gitignore
    • Fix: gitleaks CLI binary (no org license fee), SHA-256 pinned to v8.30.1
    • Fix: .gitleaks.toml to suppress 7 false positives in _bmad/ (SHA-256 hashes in manifest CSV + truncated JWT placeholder in test docs)
    • CI passing — all checks green ✓ (SonarCloud Code Analysis failure is pre-existing, also present on merged ci: add agent-shield.yml workflow #97)
    • Tagged @don-petry for review

    PR: petry-projects/broodly#127

    What was changed:

    File Change
    .github/workflows/ci.yml Added secret-scan job — installs gitleaks v8.30.1 (SHA-256 verified), scans full history with --redact --verbose --exit-code 1
    .gitignore Added *.pem (required by gitignore_secrets_block compliance check)
    .gitleaks.toml New file — path allowlist for _bmad/ to suppress false positives

    Required admin action (cannot be done via code):

    The security_and_analysis_unavailable finding (the original warning) requires a repo admin to enable GitHub's native secret scanning and push protection:

    gh api -X PATCH "repos/petry-projects/broodly" \
      -F 'security_and_analysis={"secret_scanning":{"status":"enabled"},"secret_scanning_push_protection":{"status":"enabled"},"secret_scanning_ai_detection":{"status":"enabled"},"secret_scanning_non_provider_patterns":{"status":"enabled"},"dependabot_security_updates":{"status":"enabled"}}'
    

    Or via Settings → Code security in the GitHub UI.

  5. added a commit that references this issue on Apr 14, 2026
    694f3b7
  6. don-petry commented on Apr 17, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-04-17)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  7. don-petry commented on Apr 18, 2026

    @don-petry
    ContributorAuthor

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


    Fixing Compliance Issue #111

    • Gather context: check current branch state, existing PRs, push-protection standard
    • Review what PR fix: add secret-scan CI job and *.pem gitignore entry #127 already implemented (still open, unmerged)
    • Check workflow templates and verify gitleaks-action SHA: ff98106e4c7b2bc287b24eaf42907196329070c7 # v2.3.9 ✓
    • Add secret-scan job to ci.yml (satisfies secret_scan_ci_job_present check)
    • Add *.pem to .gitignore (satisfies gitignore_secrets_block check)
    • Add .gitleaks.toml (suppress _bmad/ false positives)
    • Commit and push
    • Create PR petry-projects/broodly#139
    • Tagged @don-petry for review
    • CI passes (running — checking back shortly)

    What was changed:

    File Change
    .github/workflows/ci.yml Added secret-scan job — gitleaks v2.3.9 (SHA pinned), full-history checkout, --redact output
    .gitignore Added *.pem — satisfies gitignore_secrets_block audit check
    .gitleaks.toml New file — path allowlist for _bmad/ to suppress false positives (SHA-256 hashes in manifests, placeholder tokens in docs)

    Required admin action (cannot be done via code):

    The security_and_analysis_unavailable root finding requires a repo admin to enable GitHub's native secret scanning:

    • GitHub UI: Settings → Code security → enable Secret scanning + Push protection
    • API:
    gh api -X PATCH "repos/petry-projects/broodly" --input - <<'JSON'
    {
      "security_and_analysis": {
        "secret_scanning": {"status": "enabled"},
        "secret_scanning_push_protection": {"status": "enabled"},
        "secret_scanning_ai_detection": {"status": "enabled"},
        "secret_scanning_non_provider_patterns": {"status": "enabled"},
        "dependabot_security_updates": {"status": "enabled"}
      }
    }
    JSON
    

    • Branch

  8. added a commit that references this issue on Apr 18, 2026
    6bba3c6
  9. don-petry commented on Apr 20, 2026

    @don-petry
    ContributorAuthor

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


    Fixing Compliance Issue #111

    • Gather context: check current branch state, existing PRs, push-protection standard, workflow templates
    • Understand what previous PRs (fix: add secret-scan CI job and *.pem gitignore entry #127, fix: add secret-scan CI job and *.pem gitignore entry #139) implemented and current state
    • Check org workflow templates and action SHAs
    • Add secret-scan job to ci.yml — initially tried gitleaks-action but it requires a paid org licence; switched to gitleaks CLI binary v8.30.1 (SHA-256 551f6fc8… verified)
    • Add *.pem, secrets.yaml/yml entries to .gitignore
    • Create .gitleaks.toml (allowlist for _bmad/ and _bmad-output/)
    • Commit and push
    • Create PR petry-projects/broodly#145
    • Self-review: identified and fixed gitleaks-action org licence blocker
    • CI passing — Secret scan ✅ TypeScript ✅ Go ✅ AgentShield ✅ CodeQL ✅ SonarCloud ✅
    • Tagged @don-petry for review

    What was changed:

    File Change
    .github/workflows/ci.yml Added secret-scan job — gitleaks CLI v8.30.1 (SHA-256 pinned), full-history checkout, --redact output
    .gitignore Added *.pem, secrets.yaml, secrets.yml, *.secrets.yaml, *.secrets.yml — satisfies gitignore_secrets_block audit check
    .gitleaks.toml New file — path allowlist for _bmad/ and _bmad-output/ to suppress false positives

    Required admin action (cannot be done via code):

    The security_and_analysis_unavailable root finding requires a repo admin to enable GitHub's native secret scanning:

    gh api -X PATCH "repos/petry-projects/broodly" --input - <<'JSON'
    {
      "security_and_analysis": {
        "secret_scanning": {"status": "enabled"},
        "secret_scanning_push_protection": {"status": "enabled"},
        "secret_scanning_ai_detection": {"status": "enabled"},
        "secret_scanning_non_provider_patterns": {"status": "enabled"},
        "dependabot_security_updates": {"status": "enabled"}
      }
    }
    JSON
    

    Or via Settings → Code security in the GitHub UI.
    • Branch

  10. added a commit that references this issue on Apr 20, 2026
    96b8101
  11. don-petry commented on Apr 21, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-04-21)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  12. don-petry commented on Apr 24, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-04-24)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  13. don-petry commented on May 1, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-05-01)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  14. don-petry commented on May 8, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-05-08)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  15. don-petry commented on May 11, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-05-11)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  16. don-petry commented on May 12, 2026

    @don-petry
    ContributorAuthor

    Weekly Compliance Audit (2026-05-12)

    This finding is still open.

    Detail: Could not fetch security_and_analysis — token may lack admin scope, or the repo's plan does not expose these settings

    Standard: standards/push-protection.md#required-repo-level-settings

  17. don-petry commented on May 12, 2026

    @don-petry
    ContributorAuthor

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

  18. don-petry commented on Jun 5, 2026

    @don-petry
    ContributorAuthor

    Resolved! This check is now passing as of 2026-06-05. 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 findingdev-leadFor dev-lead agent pickup

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions