From 51154672486af5203d2bad3273bfae1e7d440392 Mon Sep 17 00:00:00 2001 From: Clay Good Date: Tue, 14 Jul 2026 11:44:59 -0500 Subject: [PATCH 1/4] feat(skills): publish workflow skills to skills.sh Commit the 12 OpenSpec workflow skills as static skills//SKILL.md so `npx skills add Fission-AI/OpenSpec` can install them (skills.sh reads static files from the repo; OpenSpec otherwise only generates skills at init time). Files are generated from the existing templates via `pnpm generate:skills`, not hand-copied, and skillssh-parity.test.ts fails CI if a template changes without regenerating. The volatile generatedBy frontmatter line is stripped so the committed copies stay byte-stable across releases. Closes #1258 Co-Authored-By: Claude Opus 4.8 (1M context) --- package.json | 1 + scripts/generate-skillssh.mjs | 45 ++ scripts/skillssh-shared.mjs | 16 + skills/README.md | 19 + skills/openspec-apply-change/SKILL.md | 159 ++++++ skills/openspec-archive-change/SKILL.md | 117 ++++ skills/openspec-bulk-archive-change/SKILL.md | 248 +++++++++ skills/openspec-continue-change/SKILL.md | 121 ++++ skills/openspec-explore/SKILL.md | 289 ++++++++++ skills/openspec-ff-change/SKILL.md | 104 ++++ skills/openspec-new-change/SKILL.md | 76 +++ skills/openspec-onboard/SKILL.md | 554 +++++++++++++++++++ skills/openspec-propose/SKILL.md | 113 ++++ skills/openspec-sync-specs/SKILL.md | 147 +++++ skills/openspec-update-change/SKILL.md | 85 +++ skills/openspec-verify-change/SKILL.md | 171 ++++++ test/core/templates/skillssh-parity.test.ts | 28 + 17 files changed, 2293 insertions(+) create mode 100644 scripts/generate-skillssh.mjs create mode 100644 scripts/skillssh-shared.mjs create mode 100644 skills/README.md create mode 100644 skills/openspec-apply-change/SKILL.md create mode 100644 skills/openspec-archive-change/SKILL.md create mode 100644 skills/openspec-bulk-archive-change/SKILL.md create mode 100644 skills/openspec-continue-change/SKILL.md create mode 100644 skills/openspec-explore/SKILL.md create mode 100644 skills/openspec-ff-change/SKILL.md create mode 100644 skills/openspec-new-change/SKILL.md create mode 100644 skills/openspec-onboard/SKILL.md create mode 100644 skills/openspec-propose/SKILL.md create mode 100644 skills/openspec-sync-specs/SKILL.md create mode 100644 skills/openspec-update-change/SKILL.md create mode 100644 skills/openspec-verify-change/SKILL.md create mode 100644 test/core/templates/skillssh-parity.test.ts diff --git a/package.json b/package.json index fee580da50..53c3399aae 100644 --- a/package.json +++ b/package.json @@ -42,6 +42,7 @@ "scripts": { "lint": "eslint src/", "build": "node build.js", + "generate:skills": "node scripts/generate-skillssh.mjs", "dev": "tsc --watch", "dev:cli": "pnpm build && node bin/openspec.js", "test": "vitest run", diff --git a/scripts/generate-skillssh.mjs b/scripts/generate-skillssh.mjs new file mode 100644 index 0000000000..2fedb5d49b --- /dev/null +++ b/scripts/generate-skillssh.mjs @@ -0,0 +1,45 @@ +#!/usr/bin/env node + +/** + * Generate the static skills.sh distribution of the OpenSpec workflow skills. + * + * skills.sh installs skills by reading committed `SKILL.md` files straight from + * a GitHub repo (`npx skills add Fission-AI/OpenSpec`). OpenSpec normally + * *generates* these skills into a user's project via `openspec init`, so this + * script mirrors that same output into a committed `skills//SKILL.md` + * tree that skills.sh can discover. + * + * The committed copies are kept honest by `test/core/templates/skillssh-parity.test.ts`, + * which regenerates and diffs against disk. Run this after any skill-template + * change: `pnpm build && pnpm generate:skills`. + */ + +import { mkdirSync, readdirSync, rmSync, writeFileSync } from 'node:fs'; +import { dirname, join } from 'node:path'; +import { fileURLToPath } from 'node:url'; + +import { getSkillTemplates, generateSkillContent } from '../dist/core/shared/skill-generation.js'; +import { stripVolatileFrontmatter, SKILLS_DIR } from './skillssh-shared.mjs'; + +const repoRoot = join(dirname(fileURLToPath(import.meta.url)), '..'); +const outDir = join(repoRoot, SKILLS_DIR); + +// Remove existing skill subdirectories (clears any renamed/removed skills) but +// preserve top-level files like README.md. +mkdirSync(outDir, { recursive: true }); +for (const entry of readdirSync(outDir, { withFileTypes: true })) { + if (entry.isDirectory()) { + rmSync(join(outDir, entry.name), { recursive: true, force: true }); + } +} + +let count = 0; +for (const { template, dirName } of getSkillTemplates()) { + const content = stripVolatileFrontmatter(generateSkillContent(template, 'skills.sh')); + const skillDir = join(outDir, dirName); + mkdirSync(skillDir, { recursive: true }); + writeFileSync(join(skillDir, 'SKILL.md'), content, 'utf8'); + count++; +} + +console.log(`Generated ${count} skills into ${SKILLS_DIR}/`); diff --git a/scripts/skillssh-shared.mjs b/scripts/skillssh-shared.mjs new file mode 100644 index 0000000000..2f4197b709 --- /dev/null +++ b/scripts/skillssh-shared.mjs @@ -0,0 +1,16 @@ +/** + * Shared helpers for the skills.sh distribution generator and its parity test. + */ + +/** Directory (repo-relative) that skills.sh scans for `SKILL.md` files. */ +export const SKILLS_DIR = 'skills'; + +/** + * Drop the per-release `generatedBy` frontmatter line so the committed + * skills.sh copies stay byte-stable across OpenSpec version bumps. The line is + * meaningful only for skills that `openspec init` writes into a project; in the + * standalone distribution it would just churn the files on every release. + */ +export function stripVolatileFrontmatter(content) { + return content.replace(/^ {2}generatedBy: .*\n/m, ''); +} diff --git a/skills/README.md b/skills/README.md new file mode 100644 index 0000000000..44c01c60f1 --- /dev/null +++ b/skills/README.md @@ -0,0 +1,19 @@ +# OpenSpec skills for skills.sh + +Install the OpenSpec workflow skills into any [skills.sh](https://skills.sh)-compatible agent: + +```bash +npx skills add Fission-AI/OpenSpec +``` + +Each `openspec-*/SKILL.md` here is the same skill `openspec init` writes into a +project. The skills drive the `openspec` CLI, so for the full setup (CLI + +`openspec/` project scaffolding + slash commands) run: + +```bash +npx openspec@latest init +``` + +> These files are generated from the skill templates — do not edit by hand. Run +> `pnpm build && pnpm generate:skills` after changing a template; +> `skillssh-parity.test.ts` fails if they drift. diff --git a/skills/openspec-apply-change/SKILL.md b/skills/openspec-apply-change/SKILL.md new file mode 100644 index 0000000000..c62317063e --- /dev/null +++ b/skills/openspec-apply-change/SKILL.md @@ -0,0 +1,159 @@ +--- +name: openspec-apply-change +description: Implement tasks from an OpenSpec change. Use when the user wants to start implementing, continue implementation, or work through tasks. +allowed-tools: Bash(openspec:*) +license: MIT +compatibility: Requires openspec CLI. +metadata: + author: openspec + version: "1.0" +--- + +Implement tasks from an OpenSpec change. + +**Store selection:** If the user names a store (a store is a standalone OpenSpec repo registered on this machine) or the work lives in one, run `openspec store list --json` to discover registered store ids, then pass `--store ` on the commands that read or write specs and changes (`new change`, `status`, `instructions`, `list`, `show`, `validate`, `archive`, `doctor`, `context`). Other commands do not take the flag. Hints printed by commands already carry the flag; keep it on follow-ups. Without a store, commands act on the nearest local `openspec/` root. + +**Input**: Optionally specify a change name. If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes. + +**Steps** + +1. **Select the change** + + If a name is provided, use it. Otherwise: + - Infer from conversation context if the user mentioned a change + - Auto-select if only one active change exists + - If ambiguous, run `openspec list --json` to get available changes and use the **AskUserQuestion tool** to let the user select + + Always announce: "Using change: " and how to override (e.g., `/opsx:apply `). + +2. **Check status to understand the schema** + ```bash + openspec status --change "" --json + ``` + Parse the JSON to understand: + - `schemaName`: The workflow being used (e.g., "spec-driven") + - `planningHome`, `changeRoot`, and `actionContext`: planning scope and edit constraints + - Which artifact contains the tasks (typically "tasks" for spec-driven, check status for others) + +3. **Get apply instructions** + + ```bash + openspec instructions apply --change "" --json + ``` + + This returns: + - `contextFiles`: artifact ID -> array of concrete file paths (varies by schema - could be proposal/specs/design/tasks or spec/tests/implementation/docs) + - Progress (total, complete, remaining) + - Task list with status + - Dynamic instruction based on current state + + **Handle states:** + - If `state: "blocked"` (missing artifacts): show message, suggest using openspec-continue-change + - If `state: "all_done"`: congratulate, suggest archive + - Otherwise: proceed to implementation + +4. **Read context files** + + Read every file path listed under `contextFiles` from the apply instructions output. + The files depend on the schema being used: + - **spec-driven**: proposal, specs, design, tasks + - Other schemas: follow the contextFiles from CLI output + +5. **Show current progress** + + Display: + - Schema being used + - Progress: "N/M tasks complete" + - Remaining tasks overview + - Dynamic instruction from CLI + +6. **Implement tasks (loop until done or blocked)** + + For each pending task: + - Show which task is being worked on + - Make the code changes required + - Keep changes minimal and focused + - Mark task complete in the tasks file: `- [ ]` → `- [x]` + - Continue to next task + + **Pause if:** + - Task is unclear → ask for clarification + - Implementation reveals a design issue → suggest updating artifacts + - Error or blocker encountered → report and wait for guidance + - User interrupts + +7. **On completion or pause, show status** + + Display: + - Tasks completed this session + - Overall progress: "N/M tasks complete" + - If all done: suggest archive + - If paused: explain why and wait for guidance + +**Output During Implementation** + +``` +## Implementing: (schema: ) + +Working on task 3/7: +[...implementation happening...] +✓ Task complete + +Working on task 4/7: +[...implementation happening...] +✓ Task complete +``` + +**Output On Completion** + +``` +## Implementation Complete + +**Change:** +**Schema:** +**Progress:** 7/7 tasks complete ✓ + +### Completed This Session +- [x] Task 1 +- [x] Task 2 +... + +All tasks complete! Ready to archive this change. +``` + +**Output On Pause (Issue Encountered)** + +``` +## Implementation Paused + +**Change:** +**Schema:** +**Progress:** 4/7 tasks complete + +### Issue Encountered + + +**Options:** +1.