docs(phase-2b): Phase 2B Skills Consolidation Planning & Architecture by ashleyshaw · Pull Request #1375 · lightspeedwp/.github

ashleyshaw · 2026-07-24T18:38:06Z

Summary

Establish Phase 2B planning structure with 4 comprehensive documents for skills consolidation:

README.md — Project overview, timeline (W3-W4), success criteria, 3 key architectural decisions
SKILL-DEPENDENCY-MAP.md — Tier classification framework, agent-skill matrix, conflict analysis
ARCHITECTURE-PLAN.md — Decisions on Tier 0 scope (65 utilities), override system design, conflict resolution
IMPLEMENTATION-ROADMAP.md — Phase C execution plan (40–60 hours, W5–W12) with per-agent tasks & risk mitigation

Provides actionable roadmap for Phase C: extract utilities to Tier 0, implement override system, migrate skills, consolidate tiers, release v2.0.0-rc1.

Linked Issues

Closes#1316 (Phase 2B planning)
Related: Epic #1079 (agent-standardisation)
Builds on: Phase A (merged PR #1283, Issue #1305)

Test plan

All 4 documents present in .github/projects/active/phase-2b-skills-audit/
Verify README.md links to other documents work
Confirm architecture decisions address 3 key questions (Tier 0 scope, override system, conflict resolution)
Check implementation roadmap Phase C tasks are sequenced correctly (W5–W12)
Validate frontmatter & link formats per standards

Changelog

Added:

Phase 2B Skills Consolidation Planning & Architecture documents
- SKILL-DEPENDENCY-MAP.md (Tier classification, agent relationships)
- ARCHITECTURE-PLAN.md (Consolidation scope, override system, conflict resolution)
- IMPLEMENTATION-ROADMAP.md (Phase C execution plan, 40–60 hours, W5–W12)
- README.md (Project overview, timeline, success criteria)

Related: Issue #1316, Epic #1079

Checklist (Global DoD / PR)

All AC met and demonstrated
Tests added/updated (unit/E2E as appropriate)
Accessibility checklist completed (where relevant)
Docs/readme/changelog updated (if user-facing)
Security checklist completed (where relevant)
Code/design reviews approved
CI green; linked issues closed; release notes prepared (if shipping)

🤖 Generated with Claude Code

coderabbitai · 2026-07-24T18:38:17Z

Warning

Review limit reached

@ashleyshaw, you've reached your PR review limit, so we couldn't start this review.

Next review available in:59 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 1465f955-3446-4d27-836e-337ae4f9b8e4

📥 Commits

Reviewing files that changed from the base of the PR and between cc3fc00 and 2bf9d2c.

⛔ Files ignored due to path filters (1)

.github/projects/active/phase-2b-skills-audit/SKILL-DEPENDENCY-MATRIX.csv is excluded by !**/*.csv

📒 Files selected for processing (20)

.github/projects/active/phase-2b-skills-audit/ARCHITECTURE-PLAN.md
.github/projects/active/phase-2b-skills-audit/IMPLEMENTATION-ROADMAP.md
.github/projects/active/phase-2b-skills-audit/IMPLEMENTATION_NOTES.md
.github/projects/active/phase-2b-skills-audit/INDEX.md
.github/projects/active/phase-2b-skills-audit/PHASE-2B-SKILLS-AUDIT.md
.github/projects/active/phase-2b-skills-audit/PHASE-2B-SKILLS-PLANNING.md
.github/projects/active/phase-2b-skills-audit/PHASE-C-TASK-BREAKDOWN.md
.github/projects/active/phase-2b-skills-audit/README.md
.github/projects/active/phase-2b-skills-audit/SKILL-DEPENDENCY-MAP.md
projects/active/milestone-planning-v1/ROADMAP.md
projects/active/milestone-planning-v1/ROADMAP_VISUAL.md
projects/active/prd-combined-agent/COMPLETION_SUMMARY.md
projects/active/prd-combined-agent/IMPLEMENTATION_NOTES.md
projects/active/prd-combined-agent/README.md
projects/active/repository-maintenance-infrastructure/COMPLETION_SUMMARY.md
projects/active/repository-maintenance-infrastructure/IMPLEMENTATION_NOTES.md
projects/active/repository-maintenance-infrastructure/README.md
projects/active/repository-maintenance-infrastructure/SCHEMA-CONSOLIDATION-MIGRATION-PLAN.md
projects/active/workflows-consolidation-2026-q3/IMPLEMENTATION_NOTES.md
projects/active/workflows-consolidation-2026-q3/README.md

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch docs/phase-2b-skills-consolidation-planning

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

github-actions · 2026-07-24T18:39:55Z

✅ Template check passed after update. Thanks for fixing the PR description.

github-actions · 2026-07-24T19:23:44Z

📄 README Validation

❌ One or more README checks failed.

Check	Result
❌ Frontmatter	Failed
✅ Structure	Passed

github-actions · 2026-07-24T19:24:05Z

🔍 Reviewer Summary for PR #1375

CI Status: ✅ success
Files changed: 4
Risk Distribution: 0 critical, 4 high, 0 medium, 0 low

Recommendations

Ready to proceed pending human review

github-actions · 2026-07-24T19:30:23Z

Metadata governance

Item: docs(phase-2b): Phase 2B Skills Consolidation Planning & Architecture #1375
Assignee: ashleyshaw
Milestone: v1.0
Linked issues/PRs: Phase 2B: Phase B — Skills Consolidation Planning & Architecture #1316

github-actions · 2026-07-24T19:30:27Z

⏱️ Aging and SLA annotation

Age: 5 day(s)
SLA state: Within SLA
Thresholds: warn at 7 days, breach at 14 days
Last updated: 2026-07-30T06:14:35.003Z

Maintained by project-meta-sync workflow.

mergify · 2026-07-31T10:38:08Z

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

Queue this pull request

Copilot

Pull request overview

Adds a Phase 2B planning package for skills consolidation, capturing the dependency mapping, architectural decisions (tiers + override design), and a Phase C implementation roadmap so Phase C can start with agreed structure and sequencing.

Changes:

Introduces a Skill Dependency Map with tier framework, reuse ranking, conflict analysis, and override patterns.
Defines an Architecture Plan covering Tier 0 scope, Tier 1 override system approach, and conflict resolution decisions.
Provides a Phase C Implementation Roadmap with milestones, per-agent tasks, estimates, and risk mitigation.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File	Description
.github/projects/active/phase-2b-skills-audit/README.md	Project overview for Phase 2B deliverables, timeline, decision questions, and references.
.github/projects/active/phase-2b-skills-audit/SKILL-DEPENDENCY-MAP.md	Tier classification + dependency/reuse mapping structure and conflict/override pattern analysis.
.github/projects/active/phase-2b-skills-audit/ARCHITECTURE-PLAN.md	Consolidation architecture and governance decisions for tiers and overrides, including conflict resolution strategy.
.github/projects/active/phase-2b-skills-audit/IMPLEMENTATION-ROADMAP.md	Phase C execution plan with milestones, per-agent migration tasks, estimates, and risk mitigation.

+---
+name: phase-2b-skills-audit
+title: Phase 2B Skills Audit — Planning & Architecture
+status: in-progress
+phase: 2B
+created: 2026-07-24
+updated: 2026-07-24
+effort_hours: "10–12"
+schedule: "W3–W4 (July 28 – Aug 8)"
+owner: Ash Shaw
+related_issues:
+ - "#1316" # Phase 2B planning issue
+ - "#1305" # Phase A completion
+ - "#1225" # Parent initiative
+ - "#1079" # Epic: agent-standardisation
+---


+---
+file_type: documentation
+title: Skill Dependency Map — Phase 2B
+description: Agent-skill relationship matrix with reuse frequency ranking, tier classification (Tier 0/1/2/3), and override pattern analysis
+status: in-progress
+created: 2026-07-24
+updated: 2026-07-24
+owner: Ash Shaw
+owners:
+ - Ash Shaw
+tags:
+ - phase-2b
+ - skills-audit
+ - dependency-mapping
+ - architecture
+format: Markdown + CSV/JSON
+---


+---
+file_type: documentation
+title: Implementation Roadmap — Phase C Skills Consolidation
+description: Phase C execution plan (40–60 hours, W5–W12) with per-agent tasks, tier rollout sequence, effort estimates, and risk mitigation
+status: in-progress
+created: 2026-07-24
+updated: 2026-07-24
+owner: Ash Shaw
+owners:
+ - Ash Shaw
+tags:
+ - phase-2b
+ - phase-c
+ - skills-consolidation
+ - roadmap
+ - implementation
+---


+---
+file_type: documentation
+title: Architecture Plan — Phase 2B Skills Consolidation
+description: Technical architecture for 4-tier skills consolidation with Tier 0 scope, override system design, and conflict resolution strategy
+status: in-progress
+created: 2026-07-24
+updated: 2026-07-24
+owner: Ash Shaw
+owners:
+ - Ash Shaw
+tags:
+ - phase-2b
+ - skills-audit
+ - architecture
+ - planning
+---


+```
+Phase C Schedule: Aug 7 – Oct 3, 2026
+├── Week 1–2 (Aug 7–20): Tier 0 Core Utilities [20h]
+├── Week 3–4 (Aug 21–Sep 3): Tier 1 Override System & Conflict Fixes [30h]
+├── Week 5–8 (Sep 4–Oct 1): Tier 1 Agent Migration [20h]
+├── Week 9–12 (Oct 2–3): Tier 2/3 Cleanup & Validation [10h]
+└── Total: 40–60 hours
+```


+| design-partner-agent | 12 | 2h | W5 | Parameters, templates, hooks | Figma conflict resolved; owns figma-code-connect (Tier 2) |
+| prd-agent | 8 | 1.5h | W5 | Parameters (output format, validation) | 117 shared → PRD-shared; 86 local |
+| prd-factory-planner-agent | 8 | 1.5h | W5 | Parameters (output format, planning rules) | 117 shared → PRD-shared; 63 local |
+| tour-operator-config-agent | 6 | 1h | W6 | Parameters (config schema, validation) | Multi-provider standardization |


+Phase C Schedule: Aug 7 – Oct 3, 2026
+├── Week 1–2 (Aug 7–20): Tier 0 Core Utilities [20h]
+├── Week 3–4 (Aug 21–Sep 3): Tier 1 Override System & Conflict Fixes [30h]
+├── Week 5–8 (Sep 4–Oct 1): Tier 1 Agent Migration [20h]
+├── Week 9–12 (Oct 2–3): Tier 2/3 Cleanup & Validation [10h]


Corrected frontmatter in .github/projects/active/phase-2b-skills-audit/README.md: - Changed `status: in-progress` to `status: active` (must be in allowed enum) - Renamed `created` to `created_date` (correct schema field name) - Renamed `updated` to `last_updated` (correct schema field name) - Added `version: 1.0.0` (recommended field for readme type) - Removed non-schema fields: phase, effort_hours, schedule, owner, related_issues - Standardized frontmatter to match LightSpeed readme schema Also added `file_type: readme` to projects/active/phase-2b-skills-audit/README.md Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Corrected frontmatter in Phase 2B skills audit README files: - Added `file_type: readme` (required schema field) - Changed `created` to `created_date` (correct schema field name) - Removed non-schema fields: name, related_issues, related_branches - Standardised frontmatter to match LightSpeed readme schema Applies fix across both locations: - .github/projects/active/phase-2b-skills-audit/README.md - projects/active/phase-2b-skills-audit/README.md Resolves issue #1375 with proper frontmatter validation. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Added two changelog entries under Fixed section: - README frontmatter schema compliance fix for Phase 2B Skills Audit - Labeling agent module import error fix References issue #1375 (README frontmatter) and documents the labeling agent bug fix that was blocking PR validation workflows.

Corrected frontmatter in Phase 2B skills audit README files: - Added `file_type: readme` (required schema field) - Changed `created` to `created_date` (correct schema field name) - Removed non-schema fields: name, related_issues, related_branches - Standardised frontmatter to match LightSpeed readme schema Applies fix across both locations: - .github/projects/active/phase-2b-skills-audit/README.md - projects/active/phase-2b-skills-audit/README.md Resolves issue #1375 with proper frontmatter validation. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Added two changelog entries under Fixed section: - README frontmatter schema compliance fix for Phase 2B Skills Audit - Labeling agent module import error fix References issue #1375 (README frontmatter) and documents the labeling agent bug fix that was blocking PR validation workflows.

* fix(#1375): Resolve README frontmatter validation issue Corrected frontmatter in Phase 2B skills audit README files: - Added `file_type: readme` (required schema field) - Changed `created` to `created_date` (correct schema field name) - Removed non-schema fields: name, related_issues, related_branches - Standardised frontmatter to match LightSpeed readme schema Applies fix across both locations: - .github/projects/active/phase-2b-skills-audit/README.md - projects/active/phase-2b-skills-audit/README.md Resolves issue #1375 with proper frontmatter validation. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix: Correct labeling agent import - use fetchCanonicalLabels (without underscore) The labeling agent was trying to import _fetchCanonicalLabels but the actual function in label-lookup.js is named fetchCanonicalLabels. Fixed import name to match the actual export. * docs: Add changelog entries for PR #1540 fixes Added two changelog entries under Fixed section: - README frontmatter schema compliance fix for Phase 2B Skills Audit - Labeling agent module import error fix References issue #1375 (README frontmatter) and documents the labeling agent bug fix that was blocking PR validation workflows. * fix: Correct label-reporting import - use formatErrors (without underscore) * fix: Address Copilot review feedback - Remove unused fetchCanonicalLabels import from labeling.agent.js - Align README version field (1.0.1 → 1.1.0) with Quick Facts table - Clarify CHANGELOG entry to specify exact file changed (.github/projects/active/phase-2b-skills-audit/README.md) --------- Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>

github-actionsBot assigned ashleyshaw Jul 24, 2026

github-actionsBot added this to the v1.0 milestone Jul 24, 2026

ashleyshaw force-pushed the docs/phase-2b-skills-consolidation-planning branch from f9b083e to 744095fCompare July 24, 2026 19:23

github-actionsBot added status:needs-review Awaiting code review type:documentation Documentation priority:normal Default priority type:chore Chore / small hygiene change meta:needs-changelog Requires a changelog entry before merge labels Jul 24, 2026

ashleyshaw enabled auto-merge (squash) July 24, 2026 19:28

github-actionsBot removed the type:chore Chore / small hygiene change label Jul 24, 2026

ashleyshaw mentioned this pull request Jul 30, 2026
Phase 2B: Phase B — Skills Consolidation Planning & Architecture #1316
Closed

ashleyshaw requested a lite review from Copilot August 4, 2026 15:18

Copilotstarted reviewing on behalf of ashleyshaw August 4, 2026 15:19 View session

CopilotAI reviewed Aug 4, 2026
View reviewed changes

ashleyshaw closed this Aug 5, 2026

auto-merge was automatically disabled August 5, 2026 06:15
Pull request was closed

ashleyshaw force-pushed the docs/phase-2b-skills-consolidation-planning branch from 2bf9d2c to c970ed5Compare August 5, 2026 06:15

ashleyshaw mentioned this pull request Aug 5, 2026
fix(#1375): Resolve README frontmatter validation issue #1540
Merged
20 tasks

Navigation Menu

Uh oh!

docs(phase-2b): Phase 2B Skills Consolidation Planning & Architecture - #1375

docs(phase-2b): Phase 2B Skills Consolidation Planning & Architecture#1375
ashleyshaw wants to merge 0 commit into
developfrom
docs/phase-2b-skills-consolidation-planning

ashleyshaw commented Jul 24, 2026•
edited
Loading

Uh oh!

coderabbitaiBot commented Jul 24, 2026•
edited
Loading

Review limit reached

Uh oh!

github-actionsBot commented Jul 24, 2026•
edited
Loading

Uh oh!

github-actionsBot commented Jul 24, 2026

Uh oh!

github-actionsBot commented Jul 24, 2026

Uh oh!

github-actionsBot commented Jul 24, 2026

Uh oh!

github-actionsBot commented Jul 24, 2026•
edited
Loading

Uh oh!

mergifyBot commented Jul 31, 2026

Uh oh!

CopilotAI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

ashleyshaw commented Jul 24, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Linked Issues

Test plan

Changelog

Checklist (Global DoD / PR)

Uh oh!

coderabbitaiBot commented Jul 24, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Uh oh!

github-actionsBot commented Jul 24, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Jul 24, 2026

📄 README Validation

Uh oh!

github-actionsBot commented Jul 24, 2026

🔍 Reviewer Summary for PR #1375

Recommendations

Uh oh!

github-actionsBot commented Jul 24, 2026

Metadata governance

Uh oh!

github-actionsBot commented Jul 24, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⏱️ Aging and SLA annotation

Uh oh!

mergifyBot commented Jul 31, 2026

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ashleyshaw commented Jul 24, 2026•
edited
Loading

coderabbitaiBot commented Jul 24, 2026•
edited
Loading

github-actionsBot commented Jul 24, 2026•
edited
Loading

github-actionsBot commented Jul 24, 2026•
edited
Loading