Consolidate agents: 48 → 29 (fold tech variants into parent references) - #24
Merged
Conversation
Fold the 18 [Extends X] technology/language variants into their parent role's references/, so a role agent self-routes to the matching stack instead of each being a separate always-in-context agent. Two-tier roster: 15-agent core team + optional specialists; tech stacks live as references. - /fe <- angular, vue, flutter, javafx-desktop (+ stack-selection router) - /be <- kotlin, fastapi, laravel, quarkus, spring-kafka, hmrc-api (+ router) - /rev <- backend/frontend/php review - /e2e <- cucumber-bdd ; /arch <- graphql ; /devops <- terraform ; /ui <- javafx-design - compliance: generic /legal + /fin are the roles; uk-legal/uk-accountant/ uk-self-employment become references/uk, selected by jurisdiction. - Each parent: description enumerates stacks (preserves auto-trigger) + a Stack-selection router + references index. - Rewrote agents.md (core 15 + specialists), simplified disambiguation.md to roles only, updated CLAUDE.md/README counts. - Fixed 2 pre-existing nested/dangling fence bugs (technical-writer, ui-designer). Full-tree fence scan clean; relocations preserve git history.
There was a problem hiding this comment.
Pull request overview
This PR consolidates the agent roster by folding technology/language variants (previous [Extends X] agents) into their parent role agents as on-demand references/, reducing the number of discoverable agents and shifting stack-specific behavior to “stack selection” routing within the parent roles.
Changes:
- Updated core role skills (
/fe,/be,/rev,/e2e,/devops,/legal,/fin) to route to stack/jurisdiction-specific reference docs instead of separate specialist agents. - Relocated/rewrote former specialist agent frontmatter into reference markdown (and added new deep-dive reference content for JavaFX + HMRC + UK legal/tax).
- Updated docs/commands to reflect the new two-tier roster and removed/rewired references to folded agents; fixed a couple of markdown fence issues.
Reviewed changes
Copilot reviewed 43 out of 59 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates quick-start wording for the new roster model. |
| claude/skills/specialized/technical-writer/SKILL.md | Fixes nested/dangling fence issues in templates. |
| claude/skills/quality/testing/frontend-tester/SKILL.md | Repoints review guidance to /rev reference model. |
| claude/skills/quality/testing/e2e-tester/SKILL.md | Reframes Cucumber as a reference; updates references index. |
| claude/skills/quality/testing/e2e-tester/references/cucumber-bdd.md | Converts former Cucumber agent frontmatter into a reference doc header. |
| claude/skills/quality/testing/backend-tester/SKILL.md | Repoints review guidance to /rev reference model. |
| claude/skills/quality/review/reviewer/SKILL.md | Adds language-specific review references under /rev. |
| claude/skills/quality/review/reviewer/references/php-review.md | Converts PHP reviewer into /rev reference content. |
| claude/skills/quality/review/reviewer/references/frontend-review.md | Converts frontend reviewer into /rev reference content. |
| claude/skills/quality/review/reviewer/references/backend-review.md | Converts backend reviewer into /rev reference content. |
| claude/skills/operations/mlops/mlops-engineer/SKILL.md | Repoints FastAPI usage to /be (FastAPI reference). |
| claude/skills/operations/devops/devops-engineer/SKILL.md | Converts Terraform specialist into a devops reference. |
| claude/skills/operations/devops/devops-engineer/references/terraform.md | Converts Terraform specialist frontmatter into a reference doc header. |
| claude/skills/management/product-owner/SKILL.md | Updates finance/legal invocations to consolidated role commands. |
| claude/skills/management/business-analyst/SKILL.md | Updates finance/legal invocations to consolidated role commands. |
| claude/skills/disambiguation.md | Updates routing matrix to roles-only (stacks as references). |
| claude/skills/development/mobile/native/native-mobile-developer/SKILL.md | Updates cross-platform routing to /fe + Flutter reference. |
| claude/skills/development/frontend/react/frontend-developer/SKILL.md | Adds stack selection router and references list (Angular/Vue/Flutter/JavaFX). |
| claude/skills/development/frontend/react/frontend-developer/references/vue.md | Converts Vue specialist frontmatter into a reference doc header. |
| claude/skills/development/frontend/react/frontend-developer/references/javafx-desktop/references/javafx-expertise.md | Adds JavaFX desktop deep-dive expertise content. |
| claude/skills/development/frontend/react/frontend-developer/references/javafx-desktop/overview.md | Adds JavaFX desktop “overview” reference (still skill-like header). |
| claude/skills/development/frontend/react/frontend-developer/references/flutter.md | Converts Flutter specialist frontmatter into a reference doc header. |
| claude/skills/development/frontend/react/frontend-developer/references/angular.md | Converts Angular specialist frontmatter into a reference doc header. |
| claude/skills/development/backend/java/backend-developer/SKILL.md | Adds stack selection router and references list for backend stacks. |
| claude/skills/development/backend/java/backend-developer/references/spring-kafka.md | Converts Kafka specialist frontmatter into a reference doc header. |
| claude/skills/development/backend/java/backend-developer/references/quarkus.md | Converts Quarkus specialist frontmatter into a reference doc header. |
| claude/skills/development/backend/java/backend-developer/references/laravel.md | Converts Laravel specialist frontmatter into a reference doc header. |
| claude/skills/development/backend/java/backend-developer/references/kotlin.md | Converts Kotlin specialist frontmatter into a reference doc header. |
| claude/skills/development/backend/java/backend-developer/references/hmrc-api/references/self-assessment-error-config.md | Adds HMRC Self Assessment integration/error/config reference. |
| claude/skills/development/backend/java/backend-developer/references/hmrc-api/references/api-oauth-fraud.md | Adds HMRC OAuth/fraud-prevention headers reference. |
| claude/skills/development/backend/java/backend-developer/references/hmrc-api/overview.md | Adds HMRC overview reference (still skill-like header + old related skills). |
| claude/skills/development/backend/java/backend-developer/references/fastapi.md | Converts FastAPI specialist frontmatter into a reference doc header. |
| claude/skills/design/ui-designer/SKILL.md | Updates JavaFX design specialization pointers; fixes fence closure. |
| claude/skills/design/ui-designer/references/javafx-design/references/javafx-design-expertise.md | Adds JavaFX design deep-dive expertise content. |
| claude/skills/design/ui-designer/references/javafx-design/overview.md | Adds JavaFX design overview reference (still skill-like header). |
| claude/skills/compliance/legal-counsel/SKILL.md | Updates UK handling to /alex loading references/uk/. |
| claude/skills/compliance/legal-counsel/references/uk/references/templates.md | Adds UK legal templates reference. |
| claude/skills/compliance/legal-counsel/references/uk/references/scenarios.md | Adds UK legal scenario examples reference. |
| claude/skills/compliance/legal-counsel/references/uk/references/penalties-and-compliance.md | Adds UK penalties/compliance calendar reference. |
| claude/skills/compliance/legal-counsel/references/uk/references/legal-domains.md | Adds UK legal domains deep-dive reference. |
| claude/skills/compliance/legal-counsel/references/uk/overview.md | Adds UK legal persona overview (still skill-like header). |
| claude/skills/compliance/accountant/SKILL.md | Updates UK handling to /inga loading references/uk/ + adds UK self-employment ref mention. |
| claude/skills/compliance/accountant/references/uk/references/templates.md | Adds UK accounting templates reference. |
| claude/skills/compliance/accountant/references/uk/references/tax-domains.md | Adds UK tax domains deep-dive reference. |
| claude/skills/compliance/accountant/references/uk/references/scenarios.md | Adds UK accounting scenario examples reference. |
| claude/skills/compliance/accountant/references/uk/overview.md | Adds UK accounting persona overview (still skill-like header). |
| claude/skills/compliance/accountant/references/uk-self-employment.md | Converts self-employment specialist frontmatter into a reference doc header. |
| claude/skills/architecture/solution-architect/SKILL.md | Adds GraphQL reference pointer under /arch. |
| claude/skills/architecture/solution-architect/references/graphql.md | Converts GraphQL specialist frontmatter into a reference doc header. |
| claude/commands/legal.md | Updates legal command to consolidated /legal + UK references. |
| claude/commands/ios.md | Updates Flutter routing to /fe. |
| claude/commands/inga.md | Updates /inga to invoke consolidated accountant skill + UK references. |
| claude/commands/fin.md | Updates /fin to invoke consolidated accountant skill + UK references. |
| claude/commands/android.md | Updates Flutter routing to /fe. |
| claude/commands/alex.md | Updates /alex to invoke consolidated legal skill + UK references. |
| claude/commands/agents.md | Rewrites agent directory for core+specialists + references-as-stacks model. |
| claude/commands/accountant.md | Updates accountant command to consolidated accountant skill + UK references. |
| claude/CLAUDE.md | Updates team note to references-as-stacks model. |
| CLAUDE.md | Updates top-level repo doc comment on skills count/roster. |
Comments suppressed due to low confidence (1)
claude/skills/quality/review/reviewer/references/php-review.md:173
- This reference still points readers to non-existent agents (
laravel-developer,frontend-reviewer) after the consolidation. That will send reviewers to dead commands instead of the new “role + reference” model.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- README: update all stale '40 agents / 40 skill files' mentions to 29 (15-agent core + specialists; stacks as references). - backend-developer: JavaFX desktop is a /fe reference, not /be — remove it from the /be stacks note.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 43 out of 59 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (1)
claude/skills/quality/review/reviewer/references/php-review.md:173
- The related-skills list still references removed agents (
laravel-developer,frontend-reviewer). After consolidation those aren’t invokable skills anymore, so these pointers are broken. Update to current role commands (e.g./befor Laravel patterns and/fefor Blade/Livewire UI concerns).
devops-engineer had no slash command. Add claude/commands/devops.md and declare /devops in the skill (description + header) so the /devops references in agents.md and disambiguation.md resolve.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 44 out of 60 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
claude/skills/quality/review/reviewer/references/php-review.md:173
laravel-developerandfrontend-reviewerare referenced here, but those specialized agents no longer exist after the consolidation (Laravel and frontend review are now handled via/beand/revreferences). This leaves misleading/invalid handoff guidance in the PHP review reference.
…references - /fe: add JavaFX desktop to the description + Stack selection router. - /be: add HMRC MTD to the Stack selection router. Keeps each router consistent with its references index.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 44 out of 60 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
claude/skills/quality/review/reviewer/references/php-review.md:172
- These related-skill pointers reference agents that no longer exist as standalone skills after the consolidation (
laravel-developer,frontend-reviewer). This will misroute readers; point to the consolidated role agents/references instead.
The Cucumber reference index is below the Specializations note, not above.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 44 out of 60 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
claude/skills/quality/review/reviewer/references/php-review.md:173
- This reference still points to removed reviewer/developer agents (e.g.,
laravel-developer,frontend-reviewer). After the consolidation, these should route via the parent role commands/references (e.g.,/befor Laravel and/revfrontend reference), otherwise the guidance will send users to non-existent skills.
olehsvyrydov
added a commit
that referenced
this pull request
Jun 6, 2026
…s) (#24) * Consolidate agents: 48 -> 29 (fold tech variants into parent references) Fold the 18 [Extends X] technology/language variants into their parent role's references/, so a role agent self-routes to the matching stack instead of each being a separate always-in-context agent. Two-tier roster: 15-agent core team + optional specialists; tech stacks live as references. - /fe <- angular, vue, flutter, javafx-desktop (+ stack-selection router) - /be <- kotlin, fastapi, laravel, quarkus, spring-kafka, hmrc-api (+ router) - /rev <- backend/frontend/php review - /e2e <- cucumber-bdd ; /arch <- graphql ; /devops <- terraform ; /ui <- javafx-design - compliance: generic /legal + /fin are the roles; uk-legal/uk-accountant/ uk-self-employment become references/uk, selected by jurisdiction. - Each parent: description enumerates stacks (preserves auto-trigger) + a Stack-selection router + references index. - Rewrote agents.md (core 15 + specialists), simplified disambiguation.md to roles only, updated CLAUDE.md/README counts. - Fixed 2 pre-existing nested/dangling fence bugs (technical-writer, ui-designer). Full-tree fence scan clean; relocations preserve git history. * Address Copilot review on PR #24: fix README counts + /be JavaFX note - README: update all stale '40 agents / 40 skill files' mentions to 29 (15-agent core + specialists; stacks as references). - backend-developer: JavaFX desktop is a /fe reference, not /be — remove it from the /be stacks note. * Address Copilot review on PR #24: add the missing /devops command devops-engineer had no slash command. Add claude/commands/devops.md and declare /devops in the skill (description + header) so the /devops references in agents.md and disambiguation.md resolve. * Address Copilot re-review on PR #24: align stack routers with folded references - /fe: add JavaFX desktop to the description + Stack selection router. - /be: add HMRC MTD to the Stack selection router. Keeps each router consistent with its references index. * Address Copilot re-review on PR #24: fix e2e-tester reference direction The Cucumber reference index is below the Specializations note, not above.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Collapses the roster from 48 → 29 agents by folding the 18
[Extends X]technology/language variants into their parent role'sreferences/. A role agent now self-routes to the matching stack (via a Stack selection router + an enumerated description) instead of every variant being a separate always-in-context agent.Two-tier roster: a 15-agent core team + optional specialists; tech stacks are references, not agents.
Fold map
/fe/be/rev/e2e/arch/devops/ui/legal/finHow it preserves behaviour
descriptionenumerates its stacks (e.g./be: Java/Spring, Kotlin, FastAPI, Laravel, Quarkus, Kafka) so "build a FastAPI endpoint" still auto-triggers/be.## Stack selectionrouter tells the agent to detect the project's stack (build files / request) and load the matching reference./legal+/finare the roles; UK becomesreferences/uk/(personas Alex/Inga) selected by jurisdiction.git mv(history preserved); multi-file variants kept their ownreferences/intact (SKILL.md → overview.md so they're no longer discovered as agents).Docs & cleanup
commands/agents.md(Core 15 + Specialists tiers + stacks-as-references table).skills/disambiguation.mdto roles only.CLAUDE.mdteam note + README counts.Verification
find claude/skills -name SKILL.md | wc -l→ 29 (was 48); no discoverable SKILL.md remains under anyreferences/.Closes the consolidation discussed in the plan; supersedes the standalone graphql refactor (PR #23, closed).