Skip to content

PM-35200 - Create contributing guide for Claude tooling#482

Merged
theMickster merged 2 commits into
mainfrom
adding-claude-contribution-guide
Apr 29, 2026
Merged

PM-35200 - Create contributing guide for Claude tooling#482
theMickster merged 2 commits into
mainfrom
adding-claude-contribution-guide

Conversation

@theMickster
Copy link
Copy Markdown
Contributor

@theMickster theMickster commented Apr 29, 2026 •
edited by atlassian Bot
Loading

🎟️ Tracking

PM-35200

📔 Objective

Craft guidance for engineers to understand contributing to Claude tooling.
The guidance in the other-side-of-the-coin for ai-plugins/pull/88.

@theMickster theMickster requested a review from a team as a code owner April 29, 2026 08:30
@theMickster theMickster added the ai-review Request a Claude code review label Apr 29, 2026
@theMickster theMickster requested a review from audreyality April 29, 2026 08:30
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026 •
edited
Loading

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR adds a single documentation file, .claude/CONTRIBUTING.md, that guides engineers on contributing Claude tooling (rules, skills, agents, commands, hooks) to this repo and on the boundary between this repo's .claude/ and the shared bitwarden/ai-plugins repo. The content is grounded in real Claude Code primitives, links to canonical Anthropic docs, and is consistent in voice with the existing .claude/CLAUDE.md. There are no executable changes, no dependency updates, and no security-sensitive content.

Code Review Details

No findings. The guide is self-consistent, references the correct primitives (CLAUDE.md, .claude/rules/, .claude/skills/, .claude/agents/, .claude/commands/), correctly notes that subagents run in a fresh context and need security definitions inlined, and correctly points to /bitwarden-code-review:code-review-local for pre-push validation.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – b9e21f42-8e78-439b-a344-9bdc47ad3485

Great job! No new security vulnerabilities introduced in this pull request

audreyality
audreyality approved these changes Apr 29, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@audreyality audreyality left a comment •
edited
Loading

Choose a reason for hiding this comment

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

⛏️ Consider adding this to the template repository so that it's clear that this is a common document.

Addressed by bitwarden/template#195

@theMickster
Copy link
Copy Markdown
Contributor Author

theMickster commented Apr 29, 2026

⛏️ Consider adding this to the template repository so that it's clear that this is a common document.

Addressed by bitwarden/template#195

Great minds think alike! 😁

@theMickster theMickster merged commit 3c35e18 into main Apr 29, 2026
13 checks passed
@theMickster theMickster deleted the adding-claude-contribution-guide branch April 29, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@audreyality audreyality audreyality approved these changes

Assignees

No one assigned

Labels

ai-review Request a Claude code review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@theMickster @audreyality