github-starred/android
2
0
Fork 0
mirror of https://github.com/bitwarden/android.git synced 2026-05-07 00:09:31 -05:00
Code Issues 1.1k Packages Projects Releases 122 Wiki Activity

[PR #6650] llm: Add resolving-sdk-updates skill #26130

New Issue
Open
opened 2026-04-17 01:39:54 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-04-17 01:39:54 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/bitwarden/android/pull/6650
Author: @SaintPatrck
Created: 3/12/2026
Status: 🔄 Open

Base: main ← Head: llm/add-resolving-sdk-updates-skill

📝 Commits (4)

  • 97bc0d2 llm: Add resolving-sdk-updates skill
  • 90b8806 Add preprocessed context injection to resolving-sdk-updates skill
  • b31c448 Fix skill preprocessing: remove $() substitution and add allowed-tools
  • 8630094 Apply suggestions from code review

📊 Changes

2 files changed (+407 additions, -0 deletions)

View changed files

➕ .claude/skills/resolving-sdk-updates/SKILL.md (+212 -0)
➕ .claude/skills/resolving-sdk-updates/references/common-fix-patterns.md (+195 -0)

📄 Description

🎟️ Tracking

Internal tooling — no Jira ticket.
Used to resolve breaking changes in #6615

📔 Objective

Adds a new Claude Code skill (resolving-sdk-updates) that teaches Claude to diagnose and resolve build failures from Bitwarden SDK (com.bitwarden:sdk-android) version updates.

The skill provides a five-phase workflow:

  1. Identify — extract SDK version diff and parse PR body for sdk-internal PR references and Jira tickets
  2. Diagnose — categorize compiler errors (exhaustive when, removed API, renamed type, new required parameter, new error type)
  3. Investigate — read sdk-internal PR diffs to understand the author's intent behind breaking changes
  4. Fix — apply appropriate patterns from the bundled reference guide, then grep for non-compiler-caught usages
  5. Assess — classify changes as compile-only vs feature-requiring and trace Jira tickets for follow-up work

Preprocessed context injection:

  • Skill automatically runs git branch, git diff, and gh run list on load to detect current branch, SDK version delta, and CI failures
  • Proactive behavior skips to Phase 2 when CI failures are detected

Allowed-tools (least-privilege):

  • Read-only git: git branch --show-current, git diff *
  • Read-only GitHub CLI: gh run list *, gh run view *, gh pr view *, gh pr diff *
  • No write/destructive operations permitted

Files added:

  • .claude/skills/resolving-sdk-updates/SKILL.md — main skill instructions with preprocessed context and allowed-tools frontmatter
  • .claude/skills/resolving-sdk-updates/references/common-fix-patterns.md — error category to fix strategy quick-reference

🔄 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/6650 **Author:** [@SaintPatrck](https://github.com/SaintPatrck) **Created:** 3/12/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `llm/add-resolving-sdk-updates-skill` --- ### 📝 Commits (4) - [`97bc0d2`](https://github.com/bitwarden/android/commit/97bc0d29e90cf8142702204020d0239956e1f989) llm: Add resolving-sdk-updates skill - [`90b8806`](https://github.com/bitwarden/android/commit/90b88067ba8b58c78b21d2a1f6b0f33ab6d116fc) Add preprocessed context injection to resolving-sdk-updates skill - [`b31c448`](https://github.com/bitwarden/android/commit/b31c448d25f23a47ebc438ea59b004fd4821d60c) Fix skill preprocessing: remove $() substitution and add allowed-tools - [`8630094`](https://github.com/bitwarden/android/commit/86300942def4fa62855cb92167a5a8317186529f) Apply suggestions from code review ### 📊 Changes **2 files changed** (+407 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `.claude/skills/resolving-sdk-updates/SKILL.md` (+212 -0) ➕ `.claude/skills/resolving-sdk-updates/references/common-fix-patterns.md` (+195 -0) </details> ### 📄 Description ## 🎟️ Tracking Internal tooling — no Jira ticket. Used to resolve breaking changes in #6615 ## 📔 Objective Adds a new Claude Code skill (`resolving-sdk-updates`) that teaches Claude to diagnose and resolve build failures from Bitwarden SDK (`com.bitwarden:sdk-android`) version updates. **The skill provides a five-phase workflow:** 1. **Identify** — extract SDK version diff and parse PR body for `sdk-internal` PR references and Jira tickets 2. **Diagnose** — categorize compiler errors (exhaustive `when`, removed API, renamed type, new required parameter, new error type) 3. **Investigate** — read `sdk-internal` PR diffs to understand the author's intent behind breaking changes 4. **Fix** — apply appropriate patterns from the bundled reference guide, then grep for non-compiler-caught usages 5. **Assess** — classify changes as compile-only vs feature-requiring and trace Jira tickets for follow-up work **Preprocessed context injection:** - Skill automatically runs `git branch`, `git diff`, and `gh run list` on load to detect current branch, SDK version delta, and CI failures - Proactive behavior skips to Phase 2 when CI failures are detected **Allowed-tools (least-privilege):** - Read-only git: `git branch --show-current`, `git diff *` - Read-only GitHub CLI: `gh run list *`, `gh run view *`, `gh pr view *`, `gh pr diff *` - No write/destructive operations permitted **Files added:** - `.claude/skills/resolving-sdk-updates/SKILL.md` — main skill instructions with preprocessed context and allowed-tools frontmatter - `.claude/skills/resolving-sdk-updates/references/common-fix-patterns.md` — error category to fix strategy quick-reference --- <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-17 01:39:54 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
app:authenticator
app:password-manager
bug
bug-passkey
customer-support
duplicate
enhancement
good first issue
help wanted
needs-reply
needs-response
pull-request
Mirrored from GitHub Pull Request
 question
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#26130
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?