Skip to content

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

Merged
rootulp merged 1 commit intomainfrom
rootulp/add-claude-md
Apr 10, 2026
Merged

chore: add CLAUDE.md with Hacken bug bounty PR guidelines#315
rootulp merged 1 commit intomainfrom
rootulp/add-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 instructing Claude to not include bug details in PR descriptions for Hacken bug bounty fixes

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
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 CLAUDE.md file to provide guidance for AI tools regarding security PRs. The review suggests expanding this file to include standard build, test, and lint commands and updating the formatting of the security guidelines to adhere to the project's 80-character line length limit (MD013).

@rootulp rootulp marked this pull request as ready for review April 7, 2026 19:06
@rootulp rootulp requested a review from a team as a code owner April 7, 2026 19:06
@rootulp rootulp requested review from walldiss and removed request for a team April 7, 2026 19:06
@rootulp rootulp enabled auto-merge (squash) April 7, 2026 19:06
@rootulp rootulp requested review from mcrakhman and removed request for walldiss April 9, 2026 20:29
@rootulp rootulp merged commit d0a6f62 into main Apr 10, 2026
9 checks passed
@rootulp rootulp deleted the rootulp/add-claude-md branch April 10, 2026 08:15
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