Skip to content

[PM-35116] llm: Remove local agents and delivery skills, migrate to marketplace plugins#2547

Merged
SaintPatrck merged 3 commits into
mainfrom
llm/generic-architect-updates
May 7, 2026
Merged

[PM-35116] llm: Remove local agents and delivery skills, migrate to marketplace plugins#2547
SaintPatrck merged 3 commits into
mainfrom
llm/generic-architect-updates

Conversation

@SaintPatrck

@SaintPatrck SaintPatrck commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

PM-35116

📔 Objective

WIP — Active development. Publishing early to share direction and
prevent duplicate work across the team.

Depends on bitwarden/ai-plugins#85 — do not merge until the marketplace plugins land.

Removes local agent and delivery skill definitions that are now provided by generic marketplace plugins.

Removed

  • ios-architect agent — replaced by bitwarden-architect:architect
  • ios-implementer agent — replaced by bitwarden-software-engineer:bitwarden-software-engineer
  • committing-ios-changes skill — replaced by bitwarden-delivery-tools:committing-changes
  • creating-ios-pull-request skill — replaced by bitwarden-delivery-tools:creating-pull-request
  • labeling-ios-changes skill — replaced by bitwarden-delivery-tools:labeling-changes
  • perform-ios-preflight-checklist skill — replaced by bitwarden-delivery-tools:perform-preflight

Updated

  • .claude/CLAUDE.md — delivery skill references updated to marketplace plugin names in Skills & Commands table and Delivery Workflow section

@github-actions github-actions Bot added the t:llm Change Type - LLM related change (e.g. CLAUDE.md files) label Apr 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Logo
Checkmarx One – Scan Summary & Detailse8880f9f-66cf-4adb-af59-473e602f8be4

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

@codecov

codecov Bot commented Apr 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.12%. Comparing base (3ecf4ce) to head (f7e7710).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2547      +/-   ##
==========================================
- Coverage   87.20%   86.12%   -1.09%     
==========================================
  Files        1894     2120     +226     
  Lines      167519   182319   +14800     
==========================================
+ Hits       146087   157020   +10933     
- Misses      21432    25299    +3867     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SaintPatrck SaintPatrck marked this pull request as ready for review April 30, 2026 17:02
@SaintPatrck SaintPatrck requested review from a team as code owners April 30, 2026 17:02
Replaced by bitwarden-architect:architect marketplace plugin that
discovers platform context dynamically via CLAUDE.md and repo-local
planning skills.
…ugin

Removes committing-ios-changes, creating-ios-pull-request,
labeling-ios-changes, and perform-ios-preflight-checklist.
Updates CLAUDE.md to reference bitwarden-delivery-tools:* plugin skills.
@SaintPatrck SaintPatrck force-pushed the llm/generic-architect-updates branch from 2bcd549 to f7e7710 Compare April 30, 2026 17:03
@SaintPatrck SaintPatrck merged commit e3aed3b into main May 7, 2026
21 checks passed
@SaintPatrck SaintPatrck deleted the llm/generic-architect-updates branch May 7, 2026 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:llm Change Type - LLM related change (e.g. CLAUDE.md files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants