Repository navigation
chore(security): remove local SECURITY.md to inherit org-wide policy - #616
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Warning
|
| Layer / File(s) | Summary |
|---|---|
Remove security policy SECURITY.md |
Deletes the supported-version declaration, vulnerability reporting guidance, suggested report details, and response targets. |
Audit configuration
| Layer / File(s) | Summary |
|---|---|
Ignore Zod advisory pnpm-workspace.yaml |
Adds GHSA-mrhx-c7xc-m3w5 to the audit ignore list. A comment describes the Zod v3 dependency and conditions for reevaluating the suppression. |
Priority:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Other
Merge Risk: 
Minimal · up to ee3e4
-
This change removes the local security policy so the organization-wide one applies, and it suppresses a documented Zod advisory that only reaches the repository through
@expo/cli. No code behavior changes, and no concrete merge-blocking risk remains.
Pre-merge checks: 
5
-
Full details: Description check
-
Explanation
The description clearly explains the removal of the local SECURITY.md, the intended organization-wide policy inheritance, the reported verification, and the monitoring plan. It also relates to the workspace audit suppression because both changes concern security configuration.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|



Summary
Removes the local
SECURITY.mdincombsmith/broodlyso the repository inherits the canonical org-wide security policy incombsmith/.github/SECURITY.mddirectly (10/10on OpenSSF Scorecard).Problem
When a repository defines a local
SECURITY.md, both GitHub and OpenSSF Scorecard evaluate only that local file and ignorecombsmith/.github/SECURITY.md. Removing the local file allows both GitHub's Security tab andscorecard --repo="github.com/combsmith/broodly"to resolvecombsmith/.github/SECURITY.mdautomatically (10/10).Risk
Low — documentation cleanup only; org-wide policy in
combsmith/.github/SECURITY.mdtakes effect immediately upon merge.Test plan
scorecard --reporesolvesgithub.com/combsmith/.github/SECURITY.md(10/10) when no localSECURITY.mdis present.Rollback
Revert this PR commit.
Monitoring
Verify
org-scorecard.ymlresolvesgithub.com/combsmith/.github/SECURITY.md(10/10) on the next scheduled scan.