-
Notifications
You must be signed in to change notification settings - Fork 4
Add spec-kit integration and rewrite spec-kit-auto skill #30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| { | ||
| "name": "reference", | ||
| "version": "1.0.0", | ||
| "description": "AI-assisted development patterns and skills for the Ambient Code reference repository.", | ||
| "skills": "auto" | ||
| } | ||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,184 @@ | ||||||
| --- | ||||||
| description: Perform a non-destructive cross-artifact consistency and quality analysis across spec.md, plan.md, and tasks.md after task generation. | ||||||
| --- | ||||||
|
|
||||||
| ## User Input | ||||||
|
|
||||||
| ```text | ||||||
| $ARGUMENTS | ||||||
| ``` | ||||||
|
|
||||||
| You **MUST** consider the user input before proceeding (if not empty). | ||||||
|
|
||||||
| ## Goal | ||||||
|
|
||||||
| Identify inconsistencies, duplications, ambiguities, and underspecified items across the three core artifacts (`spec.md`, `plan.md`, `tasks.md`) before implementation. This command MUST run only after `/speckit.tasks` has successfully produced a complete `tasks.md`. | ||||||
|
|
||||||
| ## Operating Constraints | ||||||
|
|
||||||
| **STRICTLY READ-ONLY**: Do **not** modify any files. Output a structured analysis report. Offer an optional remediation plan (user must explicitly approve before any follow-up editing commands would be invoked manually). | ||||||
|
|
||||||
| **Constitution Authority**: The project constitution (`.specify/memory/constitution.md`) is **non-negotiable** within this analysis scope. Constitution conflicts are automatically CRITICAL and require adjustment of the spec, plan, or tasks—not dilution, reinterpretation, or silent ignoring of the principle. If a principle itself needs to change, that must occur in a separate, explicit constitution update outside `/speckit.analyze`. | ||||||
|
|
||||||
| ## Execution Steps | ||||||
|
|
||||||
| ### 1. Initialize Analysis Context | ||||||
|
|
||||||
| Run `.specify/scripts/bash/check-prerequisites.sh --json --require-tasks --include-tasks` once from repo root and parse JSON for FEATURE_DIR and AVAILABLE_DOCS. Derive absolute paths: | ||||||
|
|
||||||
| - SPEC = FEATURE_DIR/spec.md | ||||||
| - PLAN = FEATURE_DIR/plan.md | ||||||
| - TASKS = FEATURE_DIR/tasks.md | ||||||
|
|
||||||
| Abort with an error message if any required file is missing (instruct the user to run missing prerequisite command). | ||||||
| For single quotes in args like "I'm Groot", use escape syntax: e.g 'I'\''m Groot' (or double-quote if possible: "I'm Groot"). | ||||||
|
|
||||||
| ### 2. Load Artifacts (Progressive Disclosure) | ||||||
|
|
||||||
| Load only the minimal necessary context from each artifact: | ||||||
|
|
||||||
| **From spec.md:** | ||||||
|
|
||||||
| - Overview/Context | ||||||
| - Functional Requirements | ||||||
| - Success Criteria (measurable outcomes — e.g., performance, security, availability, user success, business impact) | ||||||
| - User Stories | ||||||
| - Edge Cases (if present) | ||||||
|
|
||||||
| **From plan.md:** | ||||||
|
|
||||||
| - Architecture/stack choices | ||||||
| - Data Model references | ||||||
| - Phases | ||||||
| - Technical constraints | ||||||
|
|
||||||
| **From tasks.md:** | ||||||
|
|
||||||
| - Task IDs | ||||||
| - Descriptions | ||||||
| - Phase grouping | ||||||
| - Parallel markers [P] | ||||||
| - Referenced file paths | ||||||
|
|
||||||
| **From constitution:** | ||||||
|
|
||||||
| - Load `.specify/memory/constitution.md` for principle validation | ||||||
|
|
||||||
| ### 3. Build Semantic Models | ||||||
|
|
||||||
| Create internal representations (do not include raw artifacts in output): | ||||||
|
|
||||||
| - **Requirements inventory**: For each Functional Requirement (FR-###) and Success Criterion (SC-###), record a stable key. Use the explicit FR-/SC- identifier as the primary key when present, and optionally also derive an imperative-phrase slug for readability (e.g., "User can upload file" → `user-can-upload-file`). Include only Success Criteria items that require buildable work (e.g., load-testing infrastructure, security audit tooling), and exclude post-launch outcome metrics and business KPIs (e.g., "Reduce support tickets by 50%"). | ||||||
| - **User story/action inventory**: Discrete user actions with acceptance criteria | ||||||
| - **Task coverage mapping**: Map each task to one or more requirements or stories (inference by keyword / explicit reference patterns like IDs or key phrases) | ||||||
| - **Constitution rule set**: Extract principle names and MUST/SHOULD normative statements | ||||||
|
|
||||||
| ### 4. Detection Passes (Token-Efficient Analysis) | ||||||
|
|
||||||
| Focus on high-signal findings. Limit to 50 findings total; aggregate remainder in overflow summary. | ||||||
|
|
||||||
| #### A. Duplication Detection | ||||||
|
|
||||||
| - Identify near-duplicate requirements | ||||||
| - Mark lower-quality phrasing for consolidation | ||||||
|
|
||||||
| #### B. Ambiguity Detection | ||||||
|
|
||||||
| - Flag vague adjectives (fast, scalable, secure, intuitive, robust) lacking measurable criteria | ||||||
| - Flag unresolved placeholders (TODO, TKTK, ???, `<placeholder>`, etc.) | ||||||
|
|
||||||
| #### C. Underspecification | ||||||
|
|
||||||
| - Requirements with verbs but missing object or measurable outcome | ||||||
| - User stories missing acceptance criteria alignment | ||||||
| - Tasks referencing files or components not defined in spec/plan | ||||||
|
|
||||||
| #### D. Constitution Alignment | ||||||
|
|
||||||
| - Any requirement or plan element conflicting with a MUST principle | ||||||
| - Missing mandated sections or quality gates from constitution | ||||||
|
|
||||||
| #### E. Coverage Gaps | ||||||
|
|
||||||
| - Requirements with zero associated tasks | ||||||
| - Tasks with no mapped requirement/story | ||||||
| - Success Criteria requiring buildable work (performance, security, availability) not reflected in tasks | ||||||
|
|
||||||
| #### F. Inconsistency | ||||||
|
|
||||||
| - Terminology drift (same concept named differently across files) | ||||||
| - Data entities referenced in plan but absent in spec (or vice versa) | ||||||
| - Task ordering contradictions (e.g., integration tasks before foundational setup tasks without dependency note) | ||||||
| - Conflicting requirements (e.g., one requires Next.js while other specifies Vue) | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick | 🔵 Trivial Minor grammar improvement. The phrase "one requires Next.js while other specifies Vue" is missing an article. Consider: "one requires Next.js while the other specifies Vue" or "one requires Next.js while another specifies Vue". 📝 Suggested fix-- Conflicting requirements (e.g., one requires Next.js while other specifies Vue)
+- Conflicting requirements (e.g., one requires Next.js while the other specifies Vue)📝 Committable suggestion
Suggested change
🧰 Tools🪛 LanguageTool[grammar] ~112-~112: Ensure spelling is correct (QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1) 🤖 Prompt for AI Agents |
||||||
|
|
||||||
| ### 5. Severity Assignment | ||||||
|
|
||||||
| Use this heuristic to prioritize findings: | ||||||
|
|
||||||
| - **CRITICAL**: Violates constitution MUST, missing core spec artifact, or requirement with zero coverage that blocks baseline functionality | ||||||
| - **HIGH**: Duplicate or conflicting requirement, ambiguous security/performance attribute, untestable acceptance criterion | ||||||
| - **MEDIUM**: Terminology drift, missing non-functional task coverage, underspecified edge case | ||||||
| - **LOW**: Style/wording improvements, minor redundancy not affecting execution order | ||||||
|
|
||||||
| ### 6. Produce Compact Analysis Report | ||||||
|
|
||||||
| Output a Markdown report (no file writes) with the following structure: | ||||||
|
|
||||||
| ## Specification Analysis Report | ||||||
|
|
||||||
| | ID | Category | Severity | Location(s) | Summary | Recommendation | | ||||||
| |----|----------|----------|-------------|---------|----------------| | ||||||
| | A1 | Duplication | HIGH | spec.md:L120-134 | Two similar requirements ... | Merge phrasing; keep clearer version | | ||||||
|
|
||||||
| (Add one row per finding; generate stable IDs prefixed by category initial.) | ||||||
|
|
||||||
| **Coverage Summary Table:** | ||||||
|
|
||||||
| | Requirement Key | Has Task? | Task IDs | Notes | | ||||||
| |-----------------|-----------|----------|-------| | ||||||
|
|
||||||
| **Constitution Alignment Issues:** (if any) | ||||||
|
|
||||||
| **Unmapped Tasks:** (if any) | ||||||
|
|
||||||
| **Metrics:** | ||||||
|
|
||||||
| - Total Requirements | ||||||
| - Total Tasks | ||||||
| - Coverage % (requirements with >=1 task) | ||||||
| - Ambiguity Count | ||||||
| - Duplication Count | ||||||
| - Critical Issues Count | ||||||
|
|
||||||
| ### 7. Provide Next Actions | ||||||
|
|
||||||
| At end of report, output a concise Next Actions block: | ||||||
|
|
||||||
| - If CRITICAL issues exist: Recommend resolving before `/speckit.implement` | ||||||
| - If only LOW/MEDIUM: User may proceed, but provide improvement suggestions | ||||||
| - Provide explicit command suggestions: e.g., "Run /speckit.specify with refinement", "Run /speckit.plan to adjust architecture", "Manually edit tasks.md to add coverage for 'performance-metrics'" | ||||||
|
|
||||||
| ### 8. Offer Remediation | ||||||
|
|
||||||
| Ask the user: "Would you like me to suggest concrete remediation edits for the top N issues?" (Do NOT apply them automatically.) | ||||||
|
|
||||||
| ## Operating Principles | ||||||
|
|
||||||
| ### Context Efficiency | ||||||
|
|
||||||
| - **Minimal high-signal tokens**: Focus on actionable findings, not exhaustive documentation | ||||||
| - **Progressive disclosure**: Load artifacts incrementally; don't dump all content into analysis | ||||||
| - **Token-efficient output**: Limit findings table to 50 rows; summarize overflow | ||||||
| - **Deterministic results**: Rerunning without changes should produce consistent IDs and counts | ||||||
|
|
||||||
| ### Analysis Guidelines | ||||||
|
|
||||||
| - **NEVER modify files** (this is read-only analysis) | ||||||
| - **NEVER hallucinate missing sections** (if absent, report them accurately) | ||||||
| - **Prioritize constitution violations** (these are always CRITICAL) | ||||||
| - **Use examples over exhaustive rules** (cite specific instances, not generic patterns) | ||||||
| - **Report zero issues gracefully** (emit success report with coverage statistics) | ||||||
|
|
||||||
| ## Context | ||||||
|
|
||||||
| $ARGUMENTS | ||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🌐 Web query:
Claude plugin.json schema skills auto configuration💡 Result:
The plugin.json schema for Claude Code (Anthropic's terminal-based AI coding tool) defines the manifest for plugins that extend functionality with skills, agents, hooks, MCP servers, etc. It is located at .claude-plugin/plugin.json in the plugin root directory. The manifest is optional; without it, Claude Code auto-discovers components in default locations (e.g., skills/ directory) and uses the directory name as the plugin name. Complete schema from official Claude Code documentation: { "name": "plugin-name", "version": "1.2.0", "description": "Brief plugin description", "author": { "name": "Author Name", "email": "author@example.com", "url": "https://github.com/author" }, "homepage": "https://docs.example.com/plugin", "repository": "https://github.com/author/plugin", "license": "MIT", "keywords": ["keyword1", "keyword2"], "commands": ["./custom/commands/special.md"], "agents": "./custom/agents/", "skills": "./custom/skills/", "hooks": "./config/hooks.json", "mcpServers": "./mcp-config.json", "outputStyles": "./styles/", "lspServers": "./.lsp.json" } Required field: name (kebab-case string, unique identifier and namespace prefix for skills like /plugin-name:skill). Skills auto-configuration: Skills (in skills/ directories) are automatically discovered and loaded by Claude Code from multiple scopes: project (.claude/skills/), user (~/.claude/skills/), plugins, etc. Claude invokes them model-autonomously based on descriptions/context, or via slash commands (/skill-name). No manual configuration needed beyond placing SKILL.md files with YAML frontmatter (e.g., description, disable-model-invocation: true). Higher-priority scopes override lower ones; plugin skills are namespaced to avoid conflicts. For IDE validation/autocomplete, community tools like claudelint provide plugin-manifest.schema.json; unofficial schemas exist on GitHub (e.g., hesreallyhim/claude-code-json-schema). settings.json uses official SchemaStore schema: "$schema": "https://json.schemastore.org/claude-code-settings.json". Official sources: Claude Code Docs (plugins reference, create plugins, skills, settings). No direct JSON Schema URI published by Anthropic for plugin.json.
Citations:
Remove the invalid
"skills"configuration.The value
"skills": "auto"is not valid per the Claude Code plugin schema. According to the official documentation, the"skills"field should either be omitted (for automatic discovery in default locations likeskills/) or set to a path string (e.g.,"./custom/skills/"). Remove or replace this field with a valid path.🤖 Prompt for AI Agents