Skip to content

docs(security): add SECURITY.md - #179

Open
jjramirezn wants to merge 1 commit into
mainfrom
docs/security-md
Open

docs(security): add SECURITY.md#179
jjramirezn wants to merge 1 commit into
mainfrom
docs/security-md

Conversation

@jjramirezn

Copy link
Copy Markdown
Contributor

GitHub surfaces a Report a vulnerability link only when it finds SECURITY.md. Researchers browsing this repo saw no intake path — even though the disclosure page and /.well-known/security.txt already exist and both return 200.

That gap is concrete: of the three researchers who reported in July 2026, one mailed an unmonitored address twice, one landed in the customer support chat behind KYC tickets, and one went through a founder's Telegram. Three front doors, none of them this one.

This file points at the existing policy rather than restating it, so mono/product/security.md stays the single source of truth. Scope and reward wording are copied from it verbatim:

  • discretionary rewards, no fixed schedule, no severity tiers
  • the retired tiered programme explicitly named as retired
  • no service promises — the only commitment is "we read every report"

GitHub shows a 'Report a vulnerability' link only when it finds this file. Researchers browsing the repo saw no intake path, even though the disclosure page and /.well-known/security.txt already exist.

This points at them rather than restating the policy, so product/security.md stays the single source of truth.
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

Added SECURITY.md with vulnerability reporting channels, product scope, out-of-scope issues, and discretionary reward terms.

Changes

Security Policy

Layer / File(s) Summary
Security policy content
SECURITY.md
Documents the reporting URL and security.txt contact, covered Peanut systems, excluded issue categories, and discretionary rewards without fixed tiers or payout amounts.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes adding SECURITY.md, matching the pull request's primary change.
Description check ✅ Passed The description directly explains the purpose of SECURITY.md and its connection to the existing security disclosure policy.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@SECURITY.md`:
- Around line 9-10: Update SECURITY.md to explicitly reference the repository’s
canonical policy source at mono/product/security.md, while retaining the
existing link to the public security.txt disclosure page.
- Around line 26-28: Update the out-of-scope list in SECURITY.md to preserve the
complete policy wording by adding exclusions for spam and issues requiring
another person’s fingerprint or face, while keeping the existing exclusions
unchanged and aligned with the linked disclosure policy.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 942c7f9f-735c-4b58-8609-9645d9d59725

📥 Commits

Reviewing files that changed from the base of the PR and between c44e56c and 1f2ee37.

📒 Files selected for processing (1)
  • SECURITY.md

Comment thread SECURITY.md
Comment on lines +9 to +10
That page is the current policy. Machine-readable contact details are at
[`/.well-known/security.txt`](https://peanut.me/.well-known/security.txt).

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Reference the repository’s canonical policy source.

The PR objective identifies mono/product/security.md as the single source of truth, but this file only points to the public disclosure page. Add an explicit reference to the canonical file to reduce policy drift.

Proposed fix
-That page is the current policy. Machine-readable contact details are at
+That page is the current policy. The canonical source is
+[`mono/product/security.md`](mono/product/security.md). Machine-readable contact details are at
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
That page is the current policy. Machine-readable contact details are at
[`/.well-known/security.txt`](https://peanut.me/.well-known/security.txt).
That page is the current policy. The canonical source is
[`mono/product/security.md`](mono/product/security.md). Machine-readable contact details are at
[`/.well-known/security.txt`](https://peanut.me/.well-known/security.txt).
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@SECURITY.md` around lines 9 - 10, Update SECURITY.md to explicitly reference
the repository’s canonical policy source at mono/product/security.md, while
retaining the existing link to the public security.txt disclosure page.

Comment thread SECURITY.md
Comment on lines +26 to +28
- Denial of service and volumetric testing
- Social engineering
- Anything that needs physical access to another person's unlocked device

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔒 Security & Privacy | 🟡 Minor | ⚡ Quick win

Preserve the complete out-of-scope wording.

The current disclosure page also excludes spam and issues requiring another person’s fingerprint or face; both exclusions are missing here. Keep this copy aligned with the linked policy. (peanut.me)

Proposed fix
-- Denial of service and volumetric testing
+- Denial of service, spam, and volumetric testing
...
-- Anything that needs physical access to another person's unlocked device
+- Anything requiring physical access to another person's unlocked device or their fingerprint or face
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- Denial of service and volumetric testing
- Social engineering
- Anything that needs physical access to another person's unlocked device
- Denial of service, spam, and volumetric testing
- Social engineering
- Anything requiring physical access to another person's unlocked device or their fingerprint or face
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@SECURITY.md` around lines 26 - 28, Update the out-of-scope list in
SECURITY.md to preserve the complete policy wording by adding exclusions for
spam and issues requiring another person’s fingerprint or face, while keeping
the existing exclusions unchanged and aligned with the linked disclosure policy.

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