Breakdown initialization and spec definition skills#141
Conversation
Plugin Validation Report —
|
| Check | Result |
|---|---|
plugin.json schema, semver, required fields |
PASS |
marketplace.json ↔ plugin.json version + description consistency (both at 1.4.0) |
PASS |
CHANGELOG.md Keep-a-Changelog format and matching [1.4.0] - 2026-06-09 entry |
PASS |
README.md surfaces both new skills with triggers and purposes |
PASS |
New starting-breakdown skill frontmatter and content |
PASS |
New developing-breakdown-spec skill frontmatter and content |
PASS |
developing-breakdown-spec/references/process-flow.dot exists and matches the documented flow |
PASS |
writing-tech-breakdowns deprecation banner well-signaled in description and body |
PASS |
Cross-skill Skill(...) references resolve to existing siblings |
PASS |
| No secrets, no overly broad permissions, no dangerous auto-approvals | PASS |
Recommendation: ready to merge. The two optional follow-ups (examples/ in developing-breakdown-spec, tightening allowed-tools on the deprecated writing-tech-breakdowns) can be deferred to a later PR.
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE This PR introduces the first two skills ( Code Review DetailsNo new findings. All previously raised issues across the 11 review threads have either been resolved in commits on this branch or were dismissed with justification (the |
theMickster
left a comment
There was a problem hiding this comment.
I support these changes and have added only a couple idea comments as feedback opportunities to ensure Claude does exactly as instructed.
Co-authored-by: Mick Letofsky <mick.tosk@gmail.com>
|
@theMickster for your review, I did add in a nudge at the end of the skills saying "offer to continue inline by invoking |
|
@bitwarden/tech-leads I've added you as optional reviewers on this, as the intent is for these skills and subsequent ones in the follow-up PRs to leverage the new template you're reviewing in https://github.com/bitwarden/tech-breakdowns/pull/10. I've spent a lot of time iterating on these skills (way more than I expected when starting out), trying to hone them down to what the highest-leverage work for Claude to assist you or other engineers doing breakdowns; where Claude can help, and how to make sure the skills are clear and known to provide that help consistently. I've taken as input:
I'd love any input on these and also on the (still Draft) #143 and #144, which build on top of this. These PRs are in Draft because I'll need to rebase on If preferred, we can merge these and you can provide feedback on skill effectiveness after the fact, but I want to make sure that these are actually helpful for you and your teams doing breakdowns. |
|
Approving on behalf of the |
|
I'm not really sure a tech lead is gonna review this independently as it's entirely about AI skills. Regarding the "deprecated" skills though, is that necessary? Just get rid of the ones you're no longer supporting. This is entirely contained here and we're not building anything repo-specific using them so you can just delete the skills, no? |
@withinfocus my fear was that we'd be left with no full solution as this is broken up into several PRs; in the interim we'd have no full breakdown solution in the plugins. Is that OK? Would you be OK with me removing them as part of the |
|
I don't personally think it's some critical gap and as long as you get them purged once your wave of work completes, that's fine. I just don't want it seen as more traditional deprecation with some long tail. |
|
@bitwarden/tech-leads I merged this PR, as Matt pointed out, your input will likely be more valuable as users of the skill versus evaluators of the skill as written. I'm preferencing merging them so that they can be used and iterated upon in action. Please don't take this as ignoring feedback. |
…#141) * Initial skils for new breakdown. * PR feedback. * Clarifications on gate. * PR feedback. * Allowing MCP for querying Jira. * PR feedback. * More explicitly handle reading content. * Cleaned up process flow. * Remove spec-kit reference Co-authored-by: Mick Letofsky <mick.tosk@gmail.com> * Removed "the" from skill names. * Removed duplicate folder. * Removing Spec-Kit, again. * Updated to use folder structure to support separate files for tasks. * Prompt for breakdown location. * Added required tools. * PR feedback. * PR feedback. * Updated template reference. --------- Co-authored-by: Mick Letofsky <mick.tosk@gmail.com>
…#141) * Initial skils for new breakdown. * PR feedback. * Clarifications on gate. * PR feedback. * Allowing MCP for querying Jira. * PR feedback. * More explicitly handle reading content. * Cleaned up process flow. * Remove spec-kit reference Co-authored-by: Mick Letofsky <mick.tosk@gmail.com> * Removed "the" from skill names. * Removed duplicate folder. * Removing Spec-Kit, again. * Updated to use folder structure to support separate files for tasks. * Prompt for breakdown location. * Added required tools. * PR feedback. * PR feedback. * Updated template reference. --------- Co-authored-by: Mick Letofsky <mick.tosk@gmail.com>
📔 Objective
The first in a series of changes to decompose and enhance the existing
writing-tech-breakdownsskill, for use with our newtech-breakdownsrepo.The plan for skill decomposition is roughly defined below, and these are the first 2 skills in that plan:
Depends upon https://github.com/bitwarden/tech-breakdowns/pull/10 for naming of
breakdown.mdtemplate.For current ideation on all the remaining skills, you can reference #134, which will not be merged in its current state