architecting-solutions
Security Mindset
Bitwarden is a password manager — security isn't a feature, it's the product. Every design decision is a security decision.
- Threat model early. Before approving an approach, ask: what can an attacker reach from here? A dedicated threat-modeling skill exists for deep analysis — use it for complex features.
- Classify data touch points. Know which fields are encrypted, which are plaintext, and which cross trust boundaries. Never add a new path for sensitive data without encryption at rest and in transit.
- Audit trail by default. Sensitive operations must be observable after the fact. If it can't be audited, it shouldn't ship.
- Fail closed. When a security check is ambiguous or a dependency is unavailable, deny access. Never default to permissive.
Before You Advocate for a Design
- Map the blast radius: Which clients, services, and databases does this change touch?
- Read first: Verify existing patterns before introducing new ones. The codebase already solved many problems — find those solutions first.
- Ask "who else?" Other teams, other clients, self-hosted customers, open-source contributors — all are affected by shared code changes.
- Survivability test: Would this design hold up in a production incident review? If not, simplify.
- When requirements are ambiguous, clarify. Don't invent requirements to fill gaps — ask the human.
Architectural Judgment
More from bitwarden/ai-plugins
avoiding-false-positives
Use this skill to validate findings during a code review. For each finding, run the rejection criteria and verification checks. If a finding fails any check, drop it.
50retrospecting
Performs comprehensive analysis of Claude Code sessions, examining git history, conversation logs, code changes, and gathering user feedback to generate actionable retrospective reports with insights for continuous improvement.
49implementing-dapper-queries
Implementing Dapper repository methods and stored procedures for MSSQL at Bitwarden. Use when creating or modifying Dapper repositories, writing stored procedures, or working with MSSQL-specific data access in the server repo.
47classifying-review-findings
Use this skill when categorizing code review findings into severity levels. Apply when determining which emoji and label to use for PR comments, deciding if an issue should be flagged at all, or classifying findings as CRITICAL, IMPORTANT, DEBT, SUGGESTED, or QUESTION.
46posting-review-summary
Use this skill when posting the final summary comment after all inline comments are posted. Apply as the LAST step of code review after all findings are classified and inline comments are complete. Detects context (agent mode sticky comment, GitHub Actions MCP tool, or local file) and routes output accordingly.
44reviewing-claude-config
Reviews Claude configuration files for security, structure, and prompt engineering quality. Use when reviewing changes to CLAUDE.md files (project-level or .claude/), skills (SKILL.md), agents, prompts, commands, or settings. Validates YAML frontmatter, progressive disclosure patterns, token efficiency, and security best practices. Detects critical issues like committed settings.local.json, hardcoded secrets, malformed YAML, broken file references, oversized skill files, and insecure agent tool access.
44