Skip to content

docs: Add SECURITY.md - #7322

Merged
avallete merged 4 commits into
developfrom
add_security_md
Oct 26, 2026
Merged

avallete merged 4 commits into
developfrom
add_security_md

Conversation

@idawson-sb

Copy link
Copy Markdown
Contributor

Summary

Adds a SECURITY.md file so we can get better signal on our security reports.

Linked issue

Linear Issue

  • The linked issue is open and carries the open-for-contribution label (or I'm a Supabase maintainer).

Checklist

  • The PR title follows Conventional Commits (e.g. fix(cli): …).
  • Tests added or updated for the change.
  • From the repository root, pnpm check:all passes; relevant package tests pass for every touched workspace, and pnpm types:check passes for each touched TypeScript workspace (or workspace declaring it).

@idawson-sb
idawson-sb requested a review from a team as a code owner October 26, 2026 01:24

@github-actions github-actions 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.

🤖 AI Review

Verified all five input findings against SECURITY.md. After merging overlapping formatting findings, three confirmed nits remain: two inaccurate section references and one formatting issue. No substantive policy defect was identified.

Findings

Severity Location Category Sources Claim
⚪ NIT SECURITY.md:12 documentation claude The introduction incorrectly says all program-wide rules are repeated at the end of the file.
⚪ NIT SECURITY.md:127 documentation claude The sentence describing the preceding sections as repository-specific sits immediately after a program-wide section.
⚪ NIT SECURITY.md:31 formatting claude+codex Lines 31 and 145 contain unnecessary trailing spaces, and line 145 extends beyond the surrounding prose wrapping.

Stats

Claude findings: 4 · Codex findings: 1 · Confirmed: 3 · Refuted: 0 · Uncertain: 0


Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: auto · Workflow run

This review runs once per PR. A maintainer can request another with a /ai-review comment.

Comment thread SECURITY.md Outdated
Comment thread SECURITY.md Outdated
Comment thread SECURITY.md Outdated
idawson-sb and others added 3 commits October 26, 2026 10:31
Drop the intro paragraph, the repeated Reporting block and the label sentence
in the hosted vs. self-hosted section, which restate other sections. Keep the
blank line after the target label list that the formatter requires.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@avallete
avallete added this pull request to the merge queue Oct 26, 2026
Merged via the queue into develop with commit e3d4723 Oct 26, 2026
43 checks passed
@avallete
avallete deleted the add_security_md branch October 26, 2026 08:25
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