[PR #6575] [MERGED] [PM-21659] llm: Add workflow skills and finalize CLAUDE.md restructuring #43779

Closed
opened 2026-04-23 22:26:57 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/bitwarden/android/pull/6575
Author: @SaintPatrck
Created: 2/24/2026
Status: Merged
Merged: 3/3/2026
Merged by: @theMickster

Base: mainHead: claude/add-workflow-skills


📝 Commits (6)

  • 733151e Add workflow skills and finalize CLAUDE.md restructuring
  • dde42e6 Rename skills for consistent naming convention and add followup commit guidance
  • 4a9c75b Address code review feedback
  • f4912d8 Add preflight skill reference and multi-line git add format
  • 0293e51 Address second round of review feedback
  • a02c41d Remove meta files and tweak workflow phase descriptions

📊 Changes

15 files changed (+306 additions, -382 deletions)

View changed files

📝 .claude/CLAUDE.md (+16 -38)
.claude/commands/work-on-android.md (+66 -0)
.claude/skills/build-test-verify/CHANGELOG.md (+0 -19)
.claude/skills/build-test-verify/CONTRIBUTING.md (+0 -43)
.claude/skills/build-test-verify/README.md (+0 -75)
📝 .claude/skills/build-test-verify/SKILL.md (+1 -1)
.claude/skills/committing-android-changes/SKILL.md (+81 -0)
.claude/skills/creating-android-pull-request/SKILL.md (+64 -0)
.claude/skills/implementing-android-code/CHANGELOG.md (+0 -40)
.claude/skills/implementing-android-code/CONTRIBUTING.md (+0 -44)
.claude/skills/implementing-android-code/README.md (+0 -77)
.claude/skills/labeling-android-changes/SKILL.md (+40 -0)
.claude/skills/perform-android-preflight-checklist/SKILL.md (+37 -0)
.claude/skills/testing-android-code/README.md (+0 -44)
📝 docs/TROUBLESHOOTING.md (+1 -1)

📄 Description

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-21659

📔 Objective

Add workflow skills and finalize CLAUDE.md restructuring.

Skills added

  • committing-android-changes — Git commit conventions, message format, followup commit guidance, pre-commit checklist, staging best practices (multi-line git add format)
  • creating-android-pull-request — PR title format, body template guidance, pre-PR checklist, branch management
  • perform-android-preflight-checklist — Quality gate checklist covering tests, code quality, security, Bitwarden patterns, and file hygiene
  • labeling-android-changes — Conventional commit type keywords for PR titles and commit messages (synced with .github/label-pr.json)
  • work-on-android — Slash command guiding the full development lifecycle through all phases

CLAUDE.md changes

  • Replace verbose inline "Adding New Feature Screen" and "Other Skills" sections with a numbered workflow skill list
  • Add Quick Reference entries for committing, code review, and PR creation
  • Flatten Security & Configuration into Security Rules
  • Remove redundant Data Models and Security Components sections

Naming convention

All skills now follow a consistent verb-noun pattern: implementing-android-code, testing-android-code, committing-android-changes, creating-android-pull-request, reviewing-changes, labeling-android-changes, perform-android-preflight-checklist.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/bitwarden/android/pull/6575 **Author:** [@SaintPatrck](https://github.com/SaintPatrck) **Created:** 2/24/2026 **Status:** ✅ Merged **Merged:** 3/3/2026 **Merged by:** [@theMickster](https://github.com/theMickster) **Base:** `main` ← **Head:** `claude/add-workflow-skills` --- ### 📝 Commits (6) - [`733151e`](https://github.com/bitwarden/android/commit/733151eb6bd126f19b26d6e4a654db9d4aef7cb4) Add workflow skills and finalize CLAUDE.md restructuring - [`dde42e6`](https://github.com/bitwarden/android/commit/dde42e6a859619d9f00655893379e4ea84118766) Rename skills for consistent naming convention and add followup commit guidance - [`4a9c75b`](https://github.com/bitwarden/android/commit/4a9c75be8fcadae8e6733692c23fc37aaf9a26b7) Address code review feedback - [`f4912d8`](https://github.com/bitwarden/android/commit/f4912d8153e029d11bf7f5ae65fef2dd06d1b842) Add preflight skill reference and multi-line git add format - [`0293e51`](https://github.com/bitwarden/android/commit/0293e5102149e3bcb955ad50cb908f97e4cc9746) Address second round of review feedback - [`a02c41d`](https://github.com/bitwarden/android/commit/a02c41d81fb249d5269d627eef222d4e86006457) Remove meta files and tweak workflow phase descriptions ### 📊 Changes **15 files changed** (+306 additions, -382 deletions) <details> <summary>View changed files</summary> 📝 `.claude/CLAUDE.md` (+16 -38) ➕ `.claude/commands/work-on-android.md` (+66 -0) ➖ `.claude/skills/build-test-verify/CHANGELOG.md` (+0 -19) ➖ `.claude/skills/build-test-verify/CONTRIBUTING.md` (+0 -43) ➖ `.claude/skills/build-test-verify/README.md` (+0 -75) 📝 `.claude/skills/build-test-verify/SKILL.md` (+1 -1) ➕ `.claude/skills/committing-android-changes/SKILL.md` (+81 -0) ➕ `.claude/skills/creating-android-pull-request/SKILL.md` (+64 -0) ➖ `.claude/skills/implementing-android-code/CHANGELOG.md` (+0 -40) ➖ `.claude/skills/implementing-android-code/CONTRIBUTING.md` (+0 -44) ➖ `.claude/skills/implementing-android-code/README.md` (+0 -77) ➕ `.claude/skills/labeling-android-changes/SKILL.md` (+40 -0) ➕ `.claude/skills/perform-android-preflight-checklist/SKILL.md` (+37 -0) ➖ `.claude/skills/testing-android-code/README.md` (+0 -44) 📝 `docs/TROUBLESHOOTING.md` (+1 -1) </details> ### 📄 Description ## 🎟️ Tracking https://bitwarden.atlassian.net/browse/PM-21659 ## 📔 Objective Add workflow skills and finalize CLAUDE.md restructuring. ### Skills added - `committing-android-changes` — Git commit conventions, message format, followup commit guidance, pre-commit checklist, staging best practices (multi-line `git add` format) - `creating-android-pull-request` — PR title format, body template guidance, pre-PR checklist, branch management - `perform-android-preflight-checklist` — Quality gate checklist covering tests, code quality, security, Bitwarden patterns, and file hygiene - `labeling-android-changes` — Conventional commit type keywords for PR titles and commit messages (synced with `.github/label-pr.json`) - `work-on-android` — Slash command guiding the full development lifecycle through all phases ### CLAUDE.md changes - Replace verbose inline "Adding New Feature Screen" and "Other Skills" sections with a numbered workflow skill list - Add Quick Reference entries for committing, code review, and PR creation - Flatten Security & Configuration into Security Rules - Remove redundant Data Models and Security Components sections ### Naming convention All skills now follow a consistent verb-noun pattern: `implementing-android-code`, `testing-android-code`, `committing-android-changes`, `creating-android-pull-request`, `reviewing-changes`, `labeling-android-changes`, `perform-android-preflight-checklist`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-23 22:26:57 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#43779