Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions .claude/skills/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ Specialized knowledge for AI agents working on Video.js 10.
| Reviewing accessibility | `aria` → `review/workflow.md` |
| Accessibility audit | `aria` |
| Committing / creating PRs | `git` or `/commit-pr` |
| Reviewer-oriented PR descriptions (SPF) | `/spf-describe-pr` |
| CSS → Tailwind migration | `css-to-tailwind` |
| Reviewing Tailwind migration | `css-to-tailwind` → `review/workflow.md` |
| Reviewing branch changes | `/review-branch` |
Expand All @@ -43,17 +44,18 @@ Specialized knowledge for AI agents working on Video.js 10.
| [docs](docs/SKILL.md) | Write Video.js 10 documentation (concepts, how-to, READMEs) | Yes |
| [gh-issue](gh-issue/SKILL.md) | Analyze GitHub issues and create implementation plans | No |
| [git](git/SKILL.md) | Git workflow — commit messages, PRs, branch naming, scope inference | No |
| [merge-behaviors](merge-behaviors/SKILL.md) | Merge two SPF behaviors with cleaned-shape-first discipline | No |
| [refactor-behavior](refactor-behavior/SKILL.md) | Refactor an SPF behavior using purpose-first discipline | No |
| [review-branch](review-branch/SKILL.md) | Review branch changes and suggest improvements | No |
| [rfc](rfc/SKILL.md) | Write RFCs — proposals needing buy-in (public API, product, DX) | No |
| [spf-create-behavior](spf-create-behavior/SKILL.md) | Create a new SPF behavior with conventions-aligned shape (stub; grows with use) | No |
| [spf-describe-pr](spf-describe-pr/SKILL.md) | Draft/revise reviewer-oriented PR descriptions for SPF-area PRs (bucket-calibrated reading order, five-label set, smoke-test discipline) | No |
| [spf-document-feature](spf-document-feature/SKILL.md) | Produce/update SPF feature registry docs (triangulation, cluster heuristics, cross-cutting checks, cascade) | No |
| [spf-document-use-case](spf-document-use-case/SKILL.md) | Produce/update SPF use-case-composition registry docs (four-mechanism taxonomy, constituent-feature cascade) | No |
| [spf-implement-feature](spf-implement-feature/SKILL.md) | Implement an SPF feature from its registry doc (disambiguation, phase scoping, chunk decomposition, downstream-skill routing, doc-as-starting-point) | No |
| [spf-implement-use-case](spf-implement-use-case/SKILL.md) | Implement an SPF use-case composition from its registry doc (disambiguation/routing, constituent-feature readiness, variant assembly, doc-as-starting-point) | No |
| [spf-update-behavior](spf-update-behavior/SKILL.md) | Update an existing SPF behavior whose purpose is changing (stub; distinct from /refactor-behavior which preserves purpose) | No |
| [split-behavior](split-behavior/SKILL.md) | Split one SPF behavior into N with axis-declared, constraints-audited discipline | No |
| [spf-merge-behaviors](spf-merge-behaviors/SKILL.md) | Merge two SPF behaviors with cleaned-shape-first discipline | No |
| [spf-refactor-behavior](spf-refactor-behavior/SKILL.md) | Refactor an SPF behavior using purpose-first discipline | No |
| [spf-split-behavior](spf-split-behavior/SKILL.md) | Split one SPF behavior into N with axis-declared, constraints-audited discipline | No |
| [spf-update-behavior](spf-update-behavior/SKILL.md) | Update an existing SPF behavior whose purpose is changing (stub; distinct from /spf-refactor-behavior which preserves purpose) | No |

## Review Workflows

Expand Down
14 changes: 7 additions & 7 deletions .claude/skills/spf-create-behavior/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
name: spf-create-behavior
description: >-
Create a new SPF behavior with conventions-aligned shape. Walks through
purpose articulation (carries /refactor-behavior's purpose-first discipline),
purpose articulation (carries /spf-refactor-behavior's purpose-first discipline),
signal type choice, slot map design, composition placement, cleanup pattern
selection, test placement, and engine wiring. Distinct from
/refactor-behavior (which modifies an existing behavior preserving its
/spf-refactor-behavior (which modifies an existing behavior preserving its
purpose) and /spf-update-behavior (which modifies an existing behavior whose
purpose is changing). Triggers: "create behavior", "new behavior", "create
SPF behavior", "add behavior", "scaffold behavior", "new SPF behavior".
Expand Down Expand Up @@ -58,7 +58,7 @@ Required reading before drafting:

## Failure-mode catalog (seeded; grows with use)

- **Purpose-articulation skipped.** Same failure mode as `/refactor-behavior`:
- **Purpose-articulation skipped.** Same failure mode as `/spf-refactor-behavior`:
jumping from "we need a behavior" to `defineBehavior` without naming what
the behavior *does* in business terms. Carries the purpose-first
discipline. The articulation should answer: what business rule does this
Expand Down Expand Up @@ -94,7 +94,7 @@ Required reading before drafting:

### Step 1 — Articulate purpose

Carry `/refactor-behavior`'s purpose-first discipline forward to new
Carry `/spf-refactor-behavior`'s purpose-first discipline forward to new
behaviors. Before any code:

- **What business rule does this behavior implement?** Name it in plain
Expand Down Expand Up @@ -170,10 +170,10 @@ Audit checklist:

## When this is the wrong skill

- **Refactoring an existing behavior, purpose preserved** → `/refactor-behavior`
- **Refactoring an existing behavior, purpose preserved** → `/spf-refactor-behavior`
- **Updating an existing behavior, purpose changing** → `/spf-update-behavior`
- **Splitting / merging existing behaviors** → `/split-behavior` /
`/merge-behaviors` (often routed through `/refactor-behavior`)
- **Splitting / merging existing behaviors** → `/spf-split-behavior` /
`/spf-merge-behaviors` (often routed through `/spf-refactor-behavior`)
- **Creating a media-layer or network-layer helper (not a behavior)** →
manual for now; future media-layer / network-layer skills will own this

Expand Down
Loading
Loading