Skip to content

Plan: Repo-wide Skill Review, Simplification, and Docs Alignment #222

@graysurf

Description

@graysurf

Plan: Repo-wide Skill Review, Simplification, and Docs Alignment

Overview

This plan reviews and corrects all repo skills using create-skill governance rules, then simplifies skill entrypoints and scripts with the same method used in PR #221 (release-workflow single-entrypoint refactor).
Work is sequenced as sprint gates (no cross-sprint execution parallelism), with parallelism only inside each sprint.
The implementation target is a final, production-ready skill/doc set: no stale scripts, no outdated guidance, and final-state deliverables that do not include progress-tracking language.

Scope

  • In scope:
    • All tracked skills under skills/**/SKILL.md (currently 44 tracked files across automation, tools, workflows).
    • Skill contracts, entrypoints, tests, script specs, fixtures, and linked docs.
    • Repo docs alignment for skill behavior and validation workflow.
  • Out of scope:
    • Ignored local skills under skills/.system/ for committed repo deliverables unless explicitly promoted into tracked files.
    • New feature workflows unrelated to skill governance/simplification.
    • Non-skill production code outside docs/scripts/tests impacted by skill entrypoint or docs parity.

Assumptions

  1. The canonical skill governance checks remain:
    • skills/tools/skill-management/skill-governance/scripts/validate_skill_contracts.sh
    • skills/tools/skill-management/skill-governance/scripts/audit-skill-layout.sh
  2. plan-tooling, bash, git, and python3 stay available locally.
  3. PR Simplify release-workflow to a single publish entrypoint #221 patterns are treated as the reference simplification approach for entrypoint consolidation and script retirement.
  4. Final output wording requirement means user-facing deliverables must describe confirmed final state, not intermediate progress states.

Review + Modification Method (derived from PR #221)

  1. Per skill, identify the smallest stable public command surface and keep one primary entrypoint whenever possible.
  2. Merge or inline fragmented legacy wrappers into the primary entrypoint flow.
  3. Remove obsolete scripts only together with their dependent smoke specs/fixtures/tests.
  4. Update SKILL.md contract + usage text + referenced docs in the same change.
  5. Update contract tests, entrypoint ownership checks, and smoke tests to match the simplified interface.
  6. If script removal is breaking for callers, add explicit migration notes (replacement command, old->new mapping, impact).

Required per-skill decision record:

Field Requirement
old entrypoint Current command/script path before remediation
decision keep | merge | remove
new entrypoint Final supported command/script path
test/spec updates Exact tests + script specs changed/removed/added
migration note Caller-impact statement and old->new mapping

Task Decomposition

Task Summary Owner Branch Worktree Execution Mode PR Status Notes
S1T1 Build Complete Skill Inventory subagent-s1-t1 issue/s1-t1-build-complete-skill-inventory issue-s1-t1 pr-shared #223 done sprint=S1; plan-task:Task 1.1; validate=git ls-files 'skills/**/SKILL.md' / sort; pr-grouping=group; pr-group=s1-auto-g1; shared-pr-anchor=S1T1
S1T2 Freeze create-skill Compliance Checklist subagent-s1-t1 issue/s1-t1-build-complete-skill-inventory issue-s1-t1 pr-shared #223 done sprint=S1; plan-task:Task 1.2; deps=Task 1.1; validate=skills/tools/skill-management/skill-governance/scripts/validate_skill_contracts.sh; pr-grouping=group; pr-group=s1-auto-g1; shared-pr-anchor=S1T1
S1T3 Freeze PR #221 Simplification Playbook subagent-s1-t1 issue/s1-t1-build-complete-skill-inventory issue-s1-t1 pr-shared #223 done sprint=S1; plan-task:Task 1.3; deps=Task 1.1; validate=rg -n "release-publish-from-changelog/removed legacy/contract/entrypoint tests/migration" docs/plans/pr-221-reference-notes.md; pr-grouping=group; pr-group=s1-auto-g1; shared-pr-anchor=S1T1
S1T4 Generate Script Remediation Map subagent-s1-t4 issue/s1-t4-generate-script-remediation-map issue-s1-t4 pr-isolated #224 done sprint=S1; plan-task:Task 1.4; deps=Task 1.2,Task 1.3; validate=bash scripts/ci/stale-skill-scripts-audit.sh --check; pr-grouping=group; pr-group=s1-auto-g2
S2T1 Normalize Workflow Skill Contracts and References subagent-s2-t1 issue/s2-t1-normalize-workflow-skill-contracts-and-reference issue-s2-t1 pr-shared #225 done sprint=S2; plan-task:Task 2.1; validate=skills/tools/skill-management/skill-governance/scripts/validate_skill_contracts.sh; pr-grouping=group; pr-group=s2-auto-g1; shared-pr-anchor=S2T1
S2T2 Simplify Workflow Scripts Using PR #221 Method subagent-s2-t1 issue/s2-t1-normalize-workflow-skill-contracts-and-reference issue-s2-t1 pr-shared #225 done sprint=S2; plan-task:Task 2.2; deps=Task 2.1; validate=bash scripts/ci/stale-skill-scripts-audit.sh --check; pr-grouping=group; pr-group=s2-auto-g1; shared-pr-anchor=S2T1
S2T3 Normalize Tools Skill Contracts and References subagent-s2-t3 issue/s2-t3-normalize-tools-skill-contracts-and-references issue-s2-t3 pr-shared #226 done sprint=S2; plan-task:Task 2.3; validate=skills/tools/skill-management/skill-governance/scripts/validate_skill_contracts.sh; pr-grouping=group; pr-group=s2-auto-g2; shared-pr-anchor=S2T3
S2T4 Simplify Tools Scripts and Clean Companion Specs/Fixtures subagent-s2-t3 issue/s2-t3-normalize-tools-skill-contracts-and-references issue-s2-t3 pr-shared #226 done sprint=S2; plan-task:Task 2.4; deps=Task 2.3; validate=bash scripts/ci/stale-skill-scripts-audit.sh --check; pr-grouping=group; pr-group=s2-auto-g2; shared-pr-anchor=S2T3
S2T5 Cross-Domain Integration Pass for Workflows + Tools subagent-s2-t1 issue/s2-t1-normalize-workflow-skill-contracts-and-reference issue-s2-t1 pr-shared #225 done sprint=S2; plan-task:Task 2.5; deps=Task 2.2,Task 2.4; validate=scripts/check.sh --docs; pr-grouping=group; pr-group=s2-auto-g1; shared-pr-anchor=S2T1
S3T1 Normalize Automation Skill Contracts and Coverage subagent-s3-t1 issue/s3-t1-normalize-automation-skill-contracts-and-coverag issue-s3-t1 pr-shared #228 done sprint=S3; plan-task:Task 3.1; validate=skills/tools/skill-management/skill-governance/scripts/validate_skill_contracts.sh; pr-grouping=group; pr-group=s3-auto-g1; shared-pr-anchor=S3T1
S3T2 Simplify Automation Scripts with Single-Entrypoint Bias subagent-s3-t1 issue/s3-t1-normalize-automation-skill-contracts-and-coverag issue-s3-t1 pr-shared #228 done sprint=S3; plan-task:Task 3.2; deps=Task 3.1; validate=bash scripts/ci/stale-skill-scripts-audit.sh --check; pr-grouping=group; pr-group=s3-auto-g1; shared-pr-anchor=S3T1
S3T3 Normalize Automation Shared References and Library Boundaries subagent-s3-t3 issue/s3-t3-normalize-automation-shared-references-and-libra issue-s3-t3 pr-shared #227 done sprint=S3; plan-task:Task 3.3; validate=rg -n "shared primitive/entrypoint/wrapper" skills/automation/README.md skills/automation/_libs/README.md; pr-grouping=group; pr-group=s3-auto-g2; shared-pr-anchor=S3T3
S3T4 Migration Notes + Deprecation Cleanup for Removed Entrypoints subagent-s3-t3 issue/s3-t3-normalize-automation-shared-references-and-libra issue-s3-t3 pr-shared #227 done sprint=S3; plan-task:Task 3.4; deps=Task 3.2,Task 3.3; validate=scripts/check.sh --docs; pr-grouping=group; pr-group=s3-auto-g2; shared-pr-anchor=S3T3
S4T1 Align Root + Runbook Docs to Final Skill Surface subagent-s4-t1 issue/s4-t1-align-root-runbook-docs-to-final-skill-surface issue-s4-t1 per-sprint #229 done sprint=S4; plan-task:Task 4.1; validate=scripts/check.sh --docs; pr-grouping=group; pr-group=s4-auto-g1; shared-pr-anchor=S4T1
S4T2 Remove Outdated/Inappropriate Content and Progress-Style Wording subagent-s4-t1 issue/s4-t1-align-root-runbook-docs-to-final-skill-surface issue-s4-t1 per-sprint #229 done sprint=S4; plan-task:Task 4.2; deps=Task 4.1; validate=scripts/check.sh --docs; pr-grouping=group; pr-group=s4-auto-g1; shared-pr-anchor=S4T1
S4T3 Full Quality Gate + Regression Verification subagent-s4-t1 issue/s4-t1-align-root-runbook-docs-to-final-skill-surface issue-s4-t1 per-sprint #229 done sprint=S4; plan-task:Task 4.3; deps=Task 4.2; validate=scripts/check.sh --all; pr-grouping=group; pr-group=s4-auto-g1; shared-pr-anchor=S4T1
S4T4 Publish Final Outcome Artifacts (No Progress Ledger) subagent-s4-t1 issue/s4-t1-align-root-runbook-docs-to-final-skill-surface issue-s4-t1 per-sprint #229 done sprint=S4; plan-task:Task 4.4; deps=Task 4.3; validate=scripts/check.sh --docs; pr-grouping=group; pr-group=s4-auto-g1; shared-pr-anchor=S4T1

Consistency Rules

  • Status must be one of: planned, in-progress, blocked, done.
  • Status = in-progress or done requires non-TBD execution metadata (Owner, Branch, Worktree, Execution Mode, PR).
  • Owner must be a subagent identifier (contains subagent) once the task is assigned; main-agent ownership is invalid for implementation tasks.
  • Execution Mode should be one of: per-sprint, pr-isolated, pr-shared (or TBD before assignment).
  • Branch and Worktree uniqueness is enforced only for rows using Execution Mode = pr-isolated.

Risks / Uncertainties

  • Sprint approvals may be recorded before final close; issue stays open until final plan acceptance.
  • Close gate fails if task statuses or PR merge states in the issue body are incomplete.

Evidence

  • Plan source: docs/plans/skills-repo-review-simplification-plan.md
  • Sprint approvals: issue comments (one comment per accepted sprint)
  • Final approval: issue/pull comment URL passed to close-plan

Metadata

Metadata

Assignees

No one assigned

    Labels

    issueIssue workflow trackingplanPlan execution tracking

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions