Skip to content

chore: add CLAUDE.md with Hacken bug bounty PR guidelines#316

Closed
rootulp wants to merge 1 commit intomainfrom
rootulp/update-claude-md
Closed

chore: add CLAUDE.md with Hacken bug bounty PR guidelines#316
rootulp wants to merge 1 commit intomainfrom
rootulp/update-claude-md

Conversation

@rootulp
Copy link
Copy Markdown
Collaborator

@rootulp rootulp commented Apr 7, 2026

Summary

  • Add a CLAUDE.md file with a "Security PRs" section
  • Instructs Claude to not include bug details in PR descriptions for Hacken bug bounty fixes; instead link to a Linear issue

Closes https://linear.app/celestia/issue/PROTOCO-1445/update-claudemd

Test plan

  • Verify CLAUDE.md is present at the repo root with the "Security PRs" section

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rootulp rootulp self-assigned this Apr 7, 2026
@rootulp rootulp closed this Apr 7, 2026
@rootulp rootulp deleted the rootulp/update-claude-md branch April 7, 2026 19:07
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new CLAUDE.md file which establishes guidelines for security-related pull requests, specifically instructing contributors to link to Linear issues for Hacken bug bounty reports rather than including sensitive details in the PR description. A review comment suggests wrapping a long line in the new file to comply with markdown formatting standards (MD013) and provides a code suggestion to fix it.


## Security PRs

- For PRs that resolve Hacken bug bounty reports, do not include details about the bug in the PR description. Instead, link to a Linear issue that contains more details on the bug and the link to the Hacken bug bounty report.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The line length exceeds 200 characters. The repository's README.md (lines 113-115) specifies that markdown files must be formatted with markdownlint, which by default enforces a line length limit (MD013). Wrapping the text improves readability and ensures consistency with the project's formatting standards.

Suggested change
- For PRs that resolve Hacken bug bounty reports, do not include details about the bug in the PR description. Instead, link to a Linear issue that contains more details on the bug and the link to the Hacken bug bounty report.
- For PRs that resolve Hacken bug bounty reports, do not include details about the bug in the PR description.
Instead, link to a Linear issue that contains more details on the bug and the link to the Hacken bug bounty report.
References
  1. Markdown files must be formatted with markdownlint (MD013 for line length). (link)

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