sdd-mcp-server 2.2.1 → 3.0.1

package/dist/utils/sddPaths.js

@@ -0,0 +1,138 @@
+import * as fs from 'fs';
+import * as path from 'path';
+/**
+ * SDD directory names
+ * .spec is the new standard (v2.1.0+)
+ * .kiro is legacy (for backwards compatibility)
+ */
+export const SDD_DIR = '.spec';
+export const SDD_DIR_LEGACY = '.kiro';
+/**
+ * Check if a directory exists
+ * @param dirPath - Directory path to check
+ * @returns true if directory exists
+ */
+function dirExists(dirPath) {
+    try {
+        return fs.statSync(dirPath).isDirectory();
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Get the SDD root directory for a project
+ * Prefers .spec, falls back to .kiro for legacy projects
+ *
+ * @param projectPath - Project root path
+ * @returns SDD root directory path (.spec or .kiro)
+ */
+export function getSddRoot(projectPath) {
+    const specPath = path.join(projectPath, SDD_DIR);
+    const kiroPath = path.join(projectPath, SDD_DIR_LEGACY);
+    // Prefer .spec if it exists
+    if (dirExists(specPath)) {
+        return specPath;
+    }
+    // Fall back to .kiro for legacy projects
+    if (dirExists(kiroPath)) {
+        return kiroPath;
+    }
+    // Default to .spec for new projects
+    return specPath;
+}
+/**
+ * Get the SDD directory name for a project
+ * Returns the actual directory name used (.spec or .kiro)
+ *
+ * @param projectPath - Project root path
+ * @returns Directory name (.spec or .kiro)
+ */
+export function getSddDirName(projectPath) {
+    const specPath = path.join(projectPath, SDD_DIR);
+    const kiroPath = path.join(projectPath, SDD_DIR_LEGACY);
+    if (dirExists(specPath)) {
+        return SDD_DIR;
+    }
+    if (dirExists(kiroPath)) {
+        return SDD_DIR_LEGACY;
+    }
+    return SDD_DIR;
+}
+/**
+ * Get the specs directory path
+ * @param projectPath - Project root path
+ * @returns Specs directory path
+ */
+export function getSpecsPath(projectPath) {
+    return path.join(getSddRoot(projectPath), 'specs');
+}
+/**
+ * Get the steering directory path
+ * @param projectPath - Project root path
+ * @returns Steering directory path
+ */
+export function getSteeringPath(projectPath) {
+    return path.join(getSddRoot(projectPath), 'steering');
+}
+/**
+ * Get a feature's spec directory path
+ * @param projectPath - Project root path
+ * @param featureName - Feature name
+ * @returns Feature spec directory path
+ */
+export function getFeatureSpecPath(projectPath, featureName) {
+    return path.join(getSpecsPath(projectPath), featureName);
+}
+/**
+ * Get the spec.json file path for a feature
+ * @param projectPath - Project root path
+ * @param featureName - Feature name
+ * @returns spec.json file path
+ */
+export function getSpecJsonPath(projectPath, featureName) {
+    return path.join(getFeatureSpecPath(projectPath, featureName), 'spec.json');
+}
+/**
+ * Check if this is a legacy project (using .kiro instead of .spec)
+ * @param projectPath - Project root path
+ * @returns true if project uses .kiro
+ */
+export function isLegacyProject(projectPath) {
+    const specPath = path.join(projectPath, SDD_DIR);
+    const kiroPath = path.join(projectPath, SDD_DIR_LEGACY);
+    // If .spec exists, not legacy
+    if (dirExists(specPath)) {
+        return false;
+    }
+    // If only .kiro exists, it's legacy
+    return dirExists(kiroPath);
+}
+/**
+ * Get the appropriate SDD directory for new projects
+ * Always returns .spec for new projects
+ *
+ * @param projectPath - Project root path
+ * @returns Path to create SDD directory (.spec)
+ */
+export function getNewSddRoot(projectPath) {
+    return path.join(projectPath, SDD_DIR);
+}
+/**
+ * Create all required SDD directories for a new project
+ * @param projectPath - Project root path
+ * @returns Created paths
+ */
+export function createSddDirectories(projectPath) {
+    const sddRoot = getNewSddRoot(projectPath);
+    const paths = [
+        sddRoot,
+        path.join(sddRoot, 'specs'),
+        path.join(sddRoot, 'steering'),
+    ];
+    for (const p of paths) {
+        fs.mkdirSync(p, { recursive: true });
+    }
+    return paths;
+}
+//# sourceMappingURL=sddPaths.js.map

package/dist/utils/sddPaths.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"sddPaths.js","sourceRoot":"","sources":["../../src/utils/sddPaths.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B;;;;GAIG;AACH,MAAM,CAAC,MAAM,OAAO,GAAG,OAAO,CAAC;AAC/B,MAAM,CAAC,MAAM,cAAc,GAAG,OAAO,CAAC;AAEtC;;;;GAIG;AACH,SAAS,SAAS,CAAC,OAAe;IAChC,IAAI,CAAC;QACH,OAAO,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,WAAW,EAAE,CAAC;IAC5C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,UAAU,CAAC,WAAmB;IAC5C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IACjD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,cAAc,CAAC,CAAC;IAExD,4BAA4B;IAC5B,IAAI,SAAS,CAAC,QAAQ,CAAC,EAAE,CAAC;QACxB,OAAO,QAAQ,CAAC;IAClB,CAAC;IAED,yCAAyC;IACzC,IAAI,SAAS,CAAC,QAAQ,CAAC,EAAE,CAAC;QACxB,OAAO,QAAQ,CAAC;IAClB,CAAC;IAED,oCAAoC;IACpC,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,aAAa,CAAC,WAAmB;IAC/C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IACjD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,cAAc,CAAC,CAAC;IAExD,IAAI,SAAS,CAAC,QAAQ,CAAC,EAAE,CAAC;QACxB,OAAO,OAAO,CAAC;IACjB,CAAC;IAED,IAAI,SAAS,CAAC,QAAQ,CAAC,EAAE,CAAC;QACxB,OAAO,cAAc,CAAC;IACxB,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,YAAY,CAAC,WAAmB;IAC9C,OAAO,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,WAAW,CAAC,EAAE,OAAO,CAAC,CAAC;AACrD,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,eAAe,CAAC,WAAmB;IACjD,OAAO,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,WAAW,CAAC,EAAE,UAAU,CAAC,CAAC;AACxD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,kBAAkB,CAAC,WAAmB,EAAE,WAAmB;IACzE,OAAO,IAAI,CAAC,IAAI,CAAC,YAAY,CAAC,WAAW,CAAC,EAAE,WAAW,CAAC,CAAC;AAC3D,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,eAAe,CAAC,WAAmB,EAAE,WAAmB;IACtE,OAAO,IAAI,CAAC,IAAI,CAAC,kBAAkB,CAAC,WAAW,EAAE,WAAW,CAAC,EAAE,WAAW,CAAC,CAAC;AAC9E,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,eAAe,CAAC,WAAmB;IACjD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IACjD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,cAAc,CAAC,CAAC;IAExD,8BAA8B;IAC9B,IAAI,SAAS,CAAC,QAAQ,CAAC,EAAE,CAAC;QACxB,OAAO,KAAK,CAAC;IACf,CAAC;IAED,oCAAoC;IACpC,OAAO,SAAS,CAAC,QAAQ,CAAC,CAAC;AAC7B,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,aAAa,CAAC,WAAmB;IAC/C,OAAO,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;AACzC,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,oBAAoB,CAAC,WAAmB;IACtD,MAAM,OAAO,GAAG,aAAa,CAAC,WAAW,CAAC,CAAC;IAC3C,MAAM,KAAK,GAAG;QACZ,OAAO;QACP,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,OAAO,CAAC;QAC3B,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC;KAC/B,CAAC;IAEF,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QACtB,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACvC,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC"}

package/documentGenerator.js

@@ -977,7 +977,7 @@ function generateDeploymentConfig(analysis) {
 }
 
 function buildDirectoryTree(analysis) {
-  let tree = `├── .kiro/                    # SDD workflow files
+  let tree = `├── .spec/                    # SDD workflow files
 │   ├── steering/            # Project steering documents
 │   └── specs/              # Feature specifications\n`;
   

package/mcp-server.js

@@ -58,7 +58,7 @@ const server = new McpServer(
 
 // Helper functions for file operations
 async function createKiroDirectory(projectPath) {
-  const kiroPath = path.join(projectPath, ".kiro");
+  const kiroPath = path.join(projectPath, ".spec");
   const specsPath = path.join(kiroPath, "specs");
   const steeringPath = path.join(kiroPath, "steering");
 
@@ -393,12 +393,12 @@ server.registerTool(
       }
 
       // Proceed with spec creation (either quality is good or answers were provided)
-      // Create .kiro directory structure in current directory (not in a subdirectory)
+      // Create .spec directory structure in current directory (not in a subdirectory)
       const { specsPath, steeringPath } =
         await createKiroDirectory(currentPath);
 
       // Generate feature name from description
-      const featureName = await generateFeatureName(description || projectName);
+      const featureName = projectName || await generateFeatureName(description);
       const featurePath = path.join(specsPath, featureName);
       await fs.mkdir(featurePath, { recursive: true });
 
@@ -477,7 +477,7 @@ server.registerTool(
         content: [
           {
             type: "text",
-            text: `## Spec Initialization Complete\n\n**Generated Feature Name**: ${featureName}\n**Project Name**: ${projectName}\n**Project Description**: ${description}${clarificationNote}\n\n**Created Files**:\n- \`.kiro/specs/${featureName}/spec.json\` - Metadata and approval tracking\n- \`.kiro/specs/${featureName}/requirements.md\` - Requirements template with enriched description\n\n**Next Step**: Use \`sdd-requirements ${featureName}\` to generate comprehensive requirements\n\nThe spec has been initialized following stage-by-stage development principles.`,
+            text: `## Spec Initialization Complete\n\n**Generated Feature Name**: ${featureName}\n**Project Name**: ${projectName}\n**Project Description**: ${description}${clarificationNote}\n\n**Created Files**:\n- \`.spec/specs/${featureName}/spec.json\` - Metadata and approval tracking\n- \`.spec/specs/${featureName}/requirements.md\` - Requirements template with enriched description\n\n**Next Step**: Use \`sdd-requirements ${featureName}\` to generate comprehensive requirements\n\nThe spec has been initialized following stage-by-stage development principles.`,
           },
         ],
       };
@@ -507,7 +507,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
 
       // Check if spec exists
       const specPath = path.join(featurePath, "spec.json");
@@ -558,7 +558,7 @@ server.registerTool(
         content: [
           {
             type: "text",
-            text: `## Requirements Generated\n\nRequirements document generated for feature: **${featureName}**\n\n**Generated**: .kiro/specs/${featureName}/requirements.md\n**Status**: Requirements phase completed\n\n**Next Step**: Review the requirements, then use \`sdd-design\` to proceed to design phase`,
+            text: `## Requirements Generated\n\nRequirements document generated for feature: **${featureName}**\n\n**Generated**: .spec/specs/${featureName}/requirements.md\n**Status**: Requirements phase completed\n\n**Next Step**: Review the requirements, then use \`sdd-design\` to proceed to design phase`,
           },
         ],
       };
@@ -588,7 +588,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const specPath = path.join(featurePath, "spec.json");
 
       // Read and validate spec
@@ -640,7 +640,7 @@ server.registerTool(
         content: [
           {
             type: "text",
-            text: `## Design Generated\n\nTechnical design document generated for feature: **${featureName}**\n\n**Generated**: .kiro/specs/${featureName}/design.md\n**Status**: Design phase completed\n\n**Next Step**: Review the design document, then use \`sdd-tasks\` to proceed to task planning phase`,
+            text: `## Design Generated\n\nTechnical design document generated for feature: **${featureName}**\n\n**Generated**: .spec/specs/${featureName}/design.md\n**Status**: Design phase completed\n\n**Next Step**: Review the design document, then use \`sdd-tasks\` to proceed to task planning phase`,
           },
         ],
       };
@@ -670,7 +670,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const specPath = path.join(featurePath, "spec.json");
 
       // Read and validate spec
@@ -778,7 +778,7 @@ ${designContext.substring(0, 1000)}${designContext.length > 1000 ? "...\n[Design
         content: [
           {
             type: "text",
-            text: `## Implementation Tasks Generated\n\nImplementation tasks document generated for feature: **${featureName}**\n\n**Generated**: .kiro/specs/${featureName}/tasks.md\n**Status**: Tasks phase completed\n**Ready for Implementation**: Yes\n\n**Next Step**: Review tasks, then use \`sdd-implement\` to begin implementation or \`sdd-status\` to check progress`,
+            text: `## Implementation Tasks Generated\n\nImplementation tasks document generated for feature: **${featureName}**\n\n**Generated**: .spec/specs/${featureName}/tasks.md\n**Status**: Tasks phase completed\n**Ready for Implementation**: Yes\n\n**Next Step**: Review tasks, then use \`sdd-implement\` to begin implementation or \`sdd-status\` to check progress`,
           },
         ],
       };
@@ -808,7 +808,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const specPath = path.join(featurePath, "spec.json");
 
       // Read spec
@@ -863,9 +863,9 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const kiroPath = path.join(currentPath, ".kiro");
+      const kiroPath = path.join(currentPath, ".spec");
 
-      // Check if .kiro directory exists
+      // Check if .spec directory exists
       const kiroExists = await fs
         .access(kiroPath)
         .then(() => true)
@@ -1013,7 +1013,7 @@ server.registerTool(
   async ({ featureName, phase }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const specPath = path.join(featurePath, "spec.json");
 
       // Read spec
@@ -1165,7 +1165,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
 
       // Load all context files
       const files = ["spec.json", "requirements.md", "design.md", "tasks.md"];
@@ -1252,7 +1252,7 @@ server.registerTool(
   async ({ templateType, featureName, customTemplate }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const specPath = path.join(featurePath, "spec.json");
 
       // Load spec for context
@@ -1335,7 +1335,7 @@ server.registerTool(
   async ({ updateMode }) => {
     try {
       const currentPath = process.cwd();
-      const steeringPath = path.join(currentPath, ".kiro", "steering");
+      const steeringPath = path.join(currentPath, ".spec", "steering");
 
       // Create steering directory if it doesn't exist
       await fs.mkdir(steeringPath, { recursive: true });
@@ -1636,17 +1636,17 @@ Kiro-style Spec Driven Development implementation using ai agent slash commands,
 ## Project Context
 
 ### Paths
-- Steering: \`.kiro/steering/\`
-- Specs: \`.kiro/specs/\`
+- Steering: \`.spec/steering/\`
+- Specs: \`.spec/specs/\`
 - Commands: \`.ai agent/commands/\`
 
 ### Steering vs Specification
 
-**Steering** (\`.kiro/steering/\`) - Guide AI with project-wide rules and context
-**Specs** (\`.kiro/specs/\`) - Formalize development process for individual features
+**Steering** (\`.spec/steering/\`) - Guide AI with project-wide rules and context
+**Specs** (\`.spec/specs/\`) - Formalize development process for individual features
 
 ### Active Specifications
-- Check \`.kiro/specs/\` for active specifications
+- Check \`.spec/specs/\` for active specifications
 - Use \`/kiro:spec-status [feature-name]\` to check progress
 
 **Current Specifications:**
@@ -2082,14 +2082,14 @@ Refer to full principles.md for detailed examples and language-specific guidance
 **Generated**: ${new Date().toISOString()}
 
 **${mode} Files**:
-- \`.kiro/steering/product.md\` - Product overview and business context (AI analysis template)
-- \`.kiro/steering/tech.md\` - Technology stack and development environment (AI analysis template)
-- \`.kiro/steering/structure.md\` - Project organization and architectural decisions (AI analysis template)
-- \`.kiro/steering/linus-review.md\` - Code review guidelines (full content)
-- \`.kiro/steering/commit.md\` - Commit message standards (full content)
-- \`.kiro/steering/tdd-guideline.md\` - Test-Driven Development practices and workflow (full content)
-- \`.kiro/steering/security-check.md\` - Security checklist aligned to OWASP Top 10 (full content)
-- \`.kiro/steering/AGENTS.md\` - Universal AI agent workflow guidance
+- \`.spec/steering/product.md\` - Product overview and business context (AI analysis template)
+- \`.spec/steering/tech.md\` - Technology stack and development environment (AI analysis template)
+- \`.spec/steering/structure.md\` - Project organization and architectural decisions (AI analysis template)
+- \`.spec/steering/linus-review.md\` - Code review guidelines (full content)
+- \`.spec/steering/commit.md\` - Commit message standards (full content)
+- \`.spec/steering/tdd-guideline.md\` - Test-Driven Development practices and workflow (full content)
+- \`.spec/steering/security-check.md\` - Security checklist aligned to OWASP Top 10 (full content)
+- \`.spec/steering/AGENTS.md\` - Universal AI agent workflow guidance
 
 **AI-Driven Approach**:
 The steering documents now contain analysis instructions for AI agents rather than hardcoded templates. This ensures:
@@ -2150,7 +2150,7 @@ server.registerTool(
   async ({ fileName, topic, inclusionMode, filePattern }) => {
     try {
       const currentPath = process.cwd();
-      const steeringPath = path.join(currentPath, ".kiro", "steering");
+      const steeringPath = path.join(currentPath, ".spec", "steering");
 
       // Create steering directory if it doesn't exist
       await fs.mkdir(steeringPath, { recursive: true });
@@ -2230,7 +2230,7 @@ ${
             type: "text",
             text: `## Custom Steering Document Created
 
-**File**: .kiro/steering/${fileName}
+**File**: .spec/steering/${fileName}
 **Topic**: ${topic}
 **Inclusion Mode**: ${inclusionMode}${inclusionMode === "conditional" ? ` (Pattern: "${filePattern}")` : ""}
 
@@ -2273,7 +2273,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const designPath = path.join(featurePath, "design.md");
       const specPath = path.join(featurePath, "spec.json");
 
@@ -2444,7 +2444,7 @@ server.registerTool(
   async ({ featureName }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const requirementsPath = path.join(featurePath, "requirements.md");
       const specPath = path.join(featurePath, "spec.json");
 
@@ -2664,7 +2664,7 @@ server.registerTool(
   async ({ featureName, taskNumbers }) => {
     try {
       const currentPath = process.cwd();
-      const featurePath = path.join(currentPath, ".kiro", "specs", featureName);
+      const featurePath = path.join(currentPath, ".spec", "specs", featureName);
       const tasksPath = path.join(featurePath, "tasks.md");
       const requirementsPath = path.join(featurePath, "requirements.md");
       const designPath = path.join(featurePath, "design.md");
@@ -2788,10 +2788,10 @@ server.registerTool(
       report += `**TDD Methodology**: Kent Beck's Red → Green → Refactor cycle\n\n`;
 
       report += `### Context Loaded\n`;
-      report += `- ✅ Requirements: .kiro/specs/${featureName}/requirements.md\n`;
-      report += `- ✅ Design: .kiro/specs/${featureName}/design.md\n`;
-      report += `- ✅ Tasks: .kiro/specs/${featureName}/tasks.md\n`;
-      report += `- ✅ Metadata: .kiro/specs/${featureName}/spec.json\n\n`;
+      report += `- ✅ Requirements: .spec/specs/${featureName}/requirements.md\n`;
+      report += `- ✅ Design: .spec/specs/${featureName}/design.md\n`;
+      report += `- ✅ Tasks: .spec/specs/${featureName}/tasks.md\n`;
+      report += `- ✅ Metadata: .spec/specs/${featureName}/spec.json\n\n`;
 
       report += `### TDD Implementation Plan\n\n`;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sdd-mcp-server",
-  "version": "2.2.1",
+  "version": "3.0.1",
   "description": "MCP server for spec-driven development workflows across AI-agent CLIs and IDEs",
   "main": "dist/index.js",
   "bin": {

package/skills/sdd-review/SKILL.md

@@ -0,0 +1,191 @@
+---
+name: sdd-review
+description: Perform thorough Linus-style code review focusing on correctness, maintainability, and adherence to project conventions. Use after completing implementation to ensure code quality. Invoked via /sdd-review [file-path or PR-number].
+---
+
+# SDD Code Review
+
+Perform comprehensive code reviews in the style of Linus Torvalds - direct, thorough, and focused on what matters: correctness, simplicity, and long-term maintainability.
+
+## Review Philosophy
+
+> "Talk is cheap. Show me the code."
+> — Linus Torvalds
+
+This review focuses on:
+1. **Correctness** - Does it actually work? Does it handle edge cases?
+2. **Simplicity** - Is it more complex than necessary?
+3. **Maintainability** - Will future developers understand this?
+4. **Convention Adherence** - Does it follow project patterns?
+
+## Workflow
+
+### Step 1: Identify Review Scope
+
+Determine what to review:
+- **Single file**: `/sdd-review src/services/UserService.ts`
+- **Directory**: `/sdd-review src/services/`
+- **Git diff**: `/sdd-review HEAD~3..HEAD`
+- **PR/MR**: `/sdd-review PR-123` or `/sdd-review MR-45`
+
+### Step 2: Load Project Context
+
+Before reviewing:
+1. Read project steering documents from `.spec/steering/`
+2. Understand existing patterns in the codebase
+3. Check if there's a related spec in `.spec/specs/`
+
+### Step 3: Perform Review
+
+#### Code Correctness Checks
+
+```markdown
+## Correctness Issues
+
+### Critical
+- [ ] Logic errors that will cause bugs
+- [ ] Race conditions or threading issues
+- [ ] Resource leaks (files, connections, memory)
+- [ ] Unhandled error conditions
+
+### Important
+- [ ] Edge cases not handled
+- [ ] Assumptions that may not hold
+- [ ] Off-by-one errors
+- [ ] Type mismatches or unsafe casts
+```
+
+#### Simplicity Assessment
+
+Ask these questions:
+- Could this be done with less code?
+- Is there a standard library function for this?
+- Is this abstraction earning its keep?
+- Would a junior developer understand this in 6 months?
+
+#### Pattern Violations
+
+Check against project conventions:
+```markdown
+## Pattern Violations
+
+### Naming
+- [ ] Variables don't follow naming convention
+- [ ] Functions named for implementation, not purpose
+
+### Structure
+- [ ] Logic in wrong layer (controller doing business logic)
+- [ ] Missing separation of concerns
+- [ ] Circular dependencies introduced
+
+### Error Handling
+- [ ] Swallowed exceptions
+- [ ] Generic error messages
+- [ ] Missing error propagation
+```
+
+### Step 4: Provide Feedback
+
+Structure feedback with clear categories:
+
+```markdown
+# Code Review: {file/PR description}
+
+## Summary
+Brief overall assessment (1-2 sentences)
+
+## 🚨 Must Fix (Blocking)
+Issues that must be resolved before merge:
+1. **Line 42**: Memory leak - connection never closed
+   ```diff
+   - const conn = await getConnection();
+   + const conn = await getConnection();
+   + try { ... } finally { conn.close(); }
+   ```
+
+## ⚠️ Should Fix (Non-blocking)
+Issues that should be addressed but won't block:
+1. **Line 78**: Magic number should be a named constant
+2. **Line 103**: Consider extracting this to a helper function
+
+## 💡 Suggestions (Optional)
+Improvements that would be nice but are truly optional:
+1. **Line 156**: This could be simplified with `Array.flatMap()`
+
+## ✅ What's Good
+Acknowledge good patterns to reinforce them:
+1. Good use of dependency injection
+2. Clear separation of concerns
+3. Comprehensive error handling in auth module
+```
+
+### Step 5: Verify Tests
+
+For any code changes:
+1. Check if tests exist for modified code
+2. Verify edge cases are tested
+3. Run existing tests to ensure no regressions
+
+```bash
+# Run tests for affected files
+npm test -- --findRelatedTests {changed-files}
+```
+
+## Review Severity Levels
+
+| Level | Meaning | Action Required |
+|-------|---------|-----------------|
+| 🚨 **Critical** | Bug, security issue, data loss risk | Must fix before merge |
+| ⚠️ **Warning** | Code smell, potential issue | Should fix, discuss if disagree |
+| 💡 **Info** | Suggestion, style preference | Optional, author's choice |
+
+## Common Issues to Watch For
+
+### TypeScript/JavaScript Specific
+- `any` type usage without justification
+- Missing null/undefined checks
+- Promises not awaited
+- Event listener memory leaks
+- Mutable shared state
+
+### General
+- Functions doing too much
+- Deep nesting (> 3 levels)
+- Boolean parameters (use options object)
+- Comments explaining *what* instead of *why*
+- Dead code or unused imports
+
+## Integration with SDD Workflow
+
+When reviewing implementation:
+1. Compare against requirements in `.spec/specs/{feature}/requirements.md`
+2. Verify design patterns from `.spec/specs/{feature}/design.md`
+3. Check task completion against `.spec/specs/{feature}/tasks.md`
+
+## Example Review
+
+```markdown
+# Code Review: UserAuthService.ts
+
+## Summary
+Good overall structure but has a critical security issue and some error handling gaps.
+
+## 🚨 Must Fix
+1. **Line 67**: Password stored in plain text in error log
+   ```typescript
+   // BAD: Leaks credentials
+   logger.error(`Login failed for ${email} with password ${password}`);
+
+   // GOOD: Never log credentials
+   logger.error(`Login failed for ${email}`);
+   ```
+
+## ⚠️ Should Fix
+1. **Line 89**: Catch block swallows all errors
+2. **Line 112-130**: This block should be extracted to a private method
+
+## ✅ What's Good
+- Clean separation between auth logic and data access
+- Good use of TypeScript discriminated unions for auth result
+- Comprehensive input validation
+```