Skip to content

docs(security): add SECURITY.md pointer — closes 2 alerts + un-reds check-critical - #641

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/security-md-pointer
Aug 25, 2026
Merged

docs(security): add SECURITY.md pointer — closes 2 alerts + un-reds check-critical#641
hyperpolymath merged 1 commit into
mainfrom
fix/security-md-pointer

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Three things are currently broken by the absence of a literal SECURITY.md, and one pointer file fixes all of them:

code-scanning alert #705 hypatia/scorecard/SecurityPolicy
code-scanning alert #704 hypatia/cicd_rules/missing_requirement
scorecard-enforcer.ymlcheck-critical runs if [ ! -f "SECURITY.md" ] and fails on every push — confirmed on the 2026-08-24 and 2026-08-25 runs

The policy is not missing. This repo has a full 12KB SECURITY.adoc plus SECURITY-ADVISORIES.adoc. Only the filename those checks look for is absent, because every community-health document in this estate is .adoc by deliberate convention.

A pointer, not a copy — so there stays exactly one source of truth and the policy can't drift between two files.

Why these two will actually close

Both are Hypatia-sourced, and Hypatia's SARIF pipeline is live (its alerts were updated 2026-08-24). They'll clear on the next scan.

⚠️ Worth stating because it is not intuitive: this is not true of the 12 Scorecard-sourced alerts in this repo. Those are frozen at 2026-06-03 and cannot auto-close — scorecard-reusable.yml emits results_format: json with no upload-sarif step, so nothing has fed the code-scanning tab for nearly three months. That is a separate finding with its own remediation.

The alternative I did not take

Teaching the Hypatia rule and the enforcer script to accept SECURITY.adoc is the more consistent long-term answer and is worth doing. It changes two pieces of shared infrastructure rather than adding one file, so it belongs in its own change rather than bundled into a red-CI fix.

🤖 Generated with Claude Code

Three things are currently broken by the absence of a literal
SECURITY.md, and one file fixes all of them:

  * code-scanning alert #705 hypatia/scorecard/SecurityPolicy
  * code-scanning alert #704 hypatia/cicd_rules/missing_requirement
  * scorecard-enforcer.yml's check-critical job, which runs
    `if [ ! -f "SECURITY.md" ]` and has been FAILING ON EVERY PUSH
    (confirmed on the 2026-08-24 and 2026-08-25 runs)

The repo has a full 12KB SECURITY.adoc plus SECURITY-ADVISORIES.adoc; the
policy is not missing, only the filename these checks look for. Every
community-health doc in this estate is .adoc by deliberate convention.

A pointer rather than a copy, so there stays exactly one source of truth.

Both alerts come from Hypatia, whose SARIF pipeline is LIVE (alerts
updated 2026-08-24), so they will close on the next scan. Note this is
NOT true of the 12 Scorecard-sourced alerts in this repo, which are
frozen at 2026-06-03 — see the separate finding on that.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@hyperpolymath
hyperpolymath merged commit 4ac10c8 into main Aug 25, 2026
11 of 13 checks passed
@hyperpolymath
hyperpolymath deleted the fix/security-md-pointer branch August 25, 2026 22:02
@sonarqubecloud

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

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.

1 participant