@skillsmith/cli 0.4.3 → 0.5.1

package/dist/src/utils/skill-name.js

@@ -0,0 +1,26 @@
+/**
+ * Shared skill name validation utilities.
+ *
+ * Used by both `skillsmith author init` and `skillsmith create` to enforce
+ * a consistent, registry-safe naming convention across all scaffolding paths.
+ */
+/**
+ * Valid skill names: lowercase letters, digits, and hyphens only.
+ * Must start with a lowercase letter.
+ * Matches the Skillsmith registry slug format.
+ */
+export const VALID_SKILL_NAME_RE = /^[a-z][a-z0-9-]*$/;
+/**
+ * Validate a skill name against the canonical Skillsmith naming convention.
+ *
+ * @returns `true` if valid, or a string error message if invalid.
+ */
+export function validateSkillName(name) {
+    if (!name.trim())
+        return 'Skill name is required';
+    if (!VALID_SKILL_NAME_RE.test(name)) {
+        return 'Skill name must start with a lowercase letter and contain only lowercase letters, digits, and hyphens (e.g. my-skill)';
+    }
+    return true;
+}
+//# sourceMappingURL=skill-name.js.map

package/dist/src/utils/skill-name.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"skill-name.js","sourceRoot":"","sources":["../../../src/utils/skill-name.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;GAIG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,mBAAmB,CAAA;AAEtD;;;;GAIG;AACH,MAAM,UAAU,iBAAiB,CAAC,IAAY;IAC5C,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE;QAAE,OAAO,wBAAwB,CAAA;IACjD,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACpC,OAAO,uHAAuH,CAAA;IAChI,CAAC;IACD,OAAO,IAAI,CAAA;AACb,CAAC"}

package/dist/src/utils/skills-directory.d.ts

@@ -12,14 +12,23 @@ export interface InstalledSkill {
     hasUpdates: boolean;
 }
 /**
- * Get skills from a specific directory
+ * Get skills from a specific directory.
+ *
+ * When dbPath is provided, opens the skill_versions table to determine
+ * whether a newer content hash has been recorded since the skill was installed.
+ * Falls back to hasUpdates: false when the database is unavailable.
+ *
+ * @param skillsDir Directory to scan for installed skills
+ * @param dbPath    Optional path to the Skillsmith SQLite database
  */
-export declare function getSkillsFromDirectory(skillsDir: string): Promise<InstalledSkill[]>;
+export declare function getSkillsFromDirectory(skillsDir: string, dbPath?: string): Promise<InstalledSkill[]>;
 /**
  * Get list of installed skills from both global (~/.claude/skills) and
  * local (.claude/skills) directories.
  *
  * SMI-1630: Local skills take precedence over global skills with the same name.
+ *
+ * @param dbPath Optional path to the Skillsmith SQLite database for update detection
  */
-export declare function getInstalledSkills(): Promise<InstalledSkill[]>;
+export declare function getInstalledSkills(dbPath?: string): Promise<InstalledSkill[]>;
 //# sourceMappingURL=skills-directory.d.ts.map

package/dist/src/utils/skills-directory.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"skills-directory.d.ts","sourceRoot":"","sources":["../../../src/utils/skills-directory.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAKH,OAAO,EAAe,KAAK,SAAS,EAAE,MAAM,kBAAkB,CAAA;AAE9D,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAA;IACZ,IAAI,EAAE,MAAM,CAAA;IACZ,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IACtB,SAAS,EAAE,SAAS,CAAA;IACpB,WAAW,EAAE,MAAM,CAAA;IACnB,UAAU,EAAE,OAAO,CAAA;CACpB;AAoBD;;GAEG;AACH,wBAAsB,sBAAsB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CAqDzF;AAED;;;;;GAKG;AACH,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,cAAc,EAAE,CAAC,CAqBpE"}
+{"version":3,"file":"skills-directory.d.ts","sourceRoot":"","sources":["../../../src/utils/skills-directory.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAMH,OAAO,EAML,KAAK,SAAS,EACf,MAAM,kBAAkB,CAAA;AAGzB,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAA;IACZ,IAAI,EAAE,MAAM,CAAA;IACZ,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IACtB,SAAS,EAAE,SAAS,CAAA;IACpB,WAAW,EAAE,MAAM,CAAA;IACnB,UAAU,EAAE,OAAO,CAAA;CACpB;AAoBD;;;;;;;;;GASG;AACH,wBAAsB,sBAAsB,CAC1C,SAAS,EAAE,MAAM,EACjB,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,cAAc,EAAE,CAAC,CAiG3B;AAED;;;;;;;GAOG;AACH,wBAAsB,kBAAkB,CAAC,MAAM,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CAsBnF"}

package/dist/src/utils/skills-directory.js

@@ -3,9 +3,11 @@
  * from the global and local ~/.claude/skills directories.
  */
 import { readdir, readFile, stat } from 'fs/promises';
+import { createHash } from 'crypto';
 import { join } from 'path';
 import { homedir } from 'os';
-import { SkillParser } from '@skillsmith/core';
+import { SkillParser, createDatabaseAsync, initializeSchema, SkillVersionRepository, } from '@skillsmith/core';
+import { DEFAULT_DB_PATH } from '../config.js';
 /**
  * SMI-1630: Search both global and local skill directories
  *
@@ -23,10 +25,32 @@ function getLocalSkillsDir() {
     return join(process.cwd(), '.claude', 'skills');
 }
 /**
- * Get skills from a specific directory
+ * Get skills from a specific directory.
+ *
+ * When dbPath is provided, opens the skill_versions table to determine
+ * whether a newer content hash has been recorded since the skill was installed.
+ * Falls back to hasUpdates: false when the database is unavailable.
+ *
+ * @param skillsDir Directory to scan for installed skills
+ * @param dbPath    Optional path to the Skillsmith SQLite database
  */
-export async function getSkillsFromDirectory(skillsDir) {
+export async function getSkillsFromDirectory(skillsDir, dbPath) {
     const skills = [];
+    // Open the version repository if a db path was provided
+    let versionRepo = null;
+    let dbConn = null;
+    if (dbPath) {
+        try {
+            dbConn = await createDatabaseAsync(dbPath);
+            initializeSchema(dbConn);
+            versionRepo = new SkillVersionRepository(dbConn);
+        }
+        catch {
+            // DB not available yet — fall back to hasUpdates: false
+            versionRepo = null;
+            dbConn = null;
+        }
+    }
     try {
         const entries = await readdir(skillsDir, { withFileTypes: true });
         for (const entry of entries) {
@@ -38,13 +62,39 @@ export async function getSkillsFromDirectory(skillsDir) {
                     const content = await readFile(skillMdPath, 'utf-8');
                     const parser = new SkillParser();
                     const parsed = parser.parse(content);
+                    // Determine hasUpdates by comparing the current SKILL.md hash to the
+                    // most-recently recorded hash in skill_versions for this skill id.
+                    let hasUpdates = false;
+                    if (versionRepo && parsed) {
+                        try {
+                            const parsedAny = parsed;
+                            const skillId = parsedAny['id'] ?? entry.name;
+                            const latestVersion = await versionRepo.getLatestVersion(skillId);
+                            if (latestVersion) {
+                                const currentHash = createHash('sha256').update(content, 'utf8').digest('hex');
+                                const storedHash = parsedAny['contentHash'] ??
+                                    parsedAny['originalContentHash'] ??
+                                    '';
+                                // hasUpdates = latest recorded hash differs from what we have locally
+                                hasUpdates = storedHash !== '' && latestVersion.content_hash !== storedHash;
+                                // If we have no stored hash, compare against current content hash
+                                if (!storedHash) {
+                                    hasUpdates = latestVersion.content_hash !== currentHash;
+                                }
+                            }
+                        }
+                        catch {
+                            // Version check failed — safe to ignore, fall back to false
+                            hasUpdates = false;
+                        }
+                    }
                     skills.push({
                         name: parsed?.name || entry.name,
                         path: skillPath,
                         version: parsed?.version || null,
                         trustTier: parsed ? parser.inferTrustTier(parsed) : 'unknown',
                         installDate: skillMdStat.mtime.toISOString().split('T')[0] || 'Unknown',
-                        hasUpdates: false, // Would check remote for updates
+                        hasUpdates,
                     });
                 }
                 catch (error) {
@@ -73,6 +123,9 @@ export async function getSkillsFromDirectory(skillsDir) {
             throw error;
         }
     }
+    finally {
+        dbConn?.close();
+    }
     return skills;
 }
 /**
@@ -80,12 +133,15 @@ export async function getSkillsFromDirectory(skillsDir) {
  * local (.claude/skills) directories.
  *
  * SMI-1630: Local skills take precedence over global skills with the same name.
+ *
+ * @param dbPath Optional path to the Skillsmith SQLite database for update detection
  */
-export async function getInstalledSkills() {
+export async function getInstalledSkills(dbPath) {
+    const resolvedDbPath = dbPath ?? DEFAULT_DB_PATH;
     // Get skills from both directories
     const [globalSkills, localSkills] = await Promise.all([
-        getSkillsFromDirectory(GLOBAL_SKILLS_DIR),
-        getSkillsFromDirectory(getLocalSkillsDir()),
+        getSkillsFromDirectory(GLOBAL_SKILLS_DIR, resolvedDbPath),
+        getSkillsFromDirectory(getLocalSkillsDir(), resolvedDbPath),
     ]);
     // Create a map for deduplication, local skills take precedence
     const skillMap = new Map();

package/dist/src/utils/skills-directory.js.map

@@ -1 +1 @@
-{"version":3,"file":"skills-directory.js","sourceRoot":"","sources":["../../../src/utils/skills-directory.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,aAAa,CAAA;AACrD,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAA;AAC3B,OAAO,EAAE,OAAO,EAAE,MAAM,IAAI,CAAA;AAC5B,OAAO,EAAE,WAAW,EAAkB,MAAM,kBAAkB,CAAA;AAW9D;;;;;;;GAOG;AACH,MAAM,iBAAiB,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAA;AAE9D;;;GAGG;AACH,SAAS,iBAAiB;IACxB,OAAO,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAA;AACjD,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAAC,SAAiB;IAC5D,MAAM,MAAM,GAAqB,EAAE,CAAA;IAEnC,IAAI,CAAC;QACH,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,SAAS,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAA;QAEjE,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;YAC5B,IAAI,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;gBACxB,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,EAAE,KAAK,CAAC,IAAI,CAAC,CAAA;gBAC7C,MAAM,WAAW,GAAG,IAAI,CAAC,SAAS,EAAE,UAAU,CAAC,CAAA;gBAE/C,IAAI,CAAC;oBACH,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,WAAW,CAAC,CAAA;oBAC3C,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,WAAW,EAAE,OAAO,CAAC,CAAA;oBACpD,MAAM,MAAM,GAAG,IAAI,WAAW,EAAE,CAAA;oBAChC,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAA;oBAEpC,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,MAAM,EAAE,IAAI,IAAI,KAAK,CAAC,IAAI;wBAChC,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,MAAM,EAAE,OAAO,IAAI,IAAI;wBAChC,SAAS,EAAE,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,SAAS;wBAC7D,WAAW,EAAE,WAAW,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,SAAS;wBACvE,UAAU,EAAE,KAAK,EAAE,iCAAiC;qBACrD,CAAC,CAAA;gBACJ,CAAC;gBAAC,OAAO,KAAK,EAAE,CAAC;oBACf,sDAAsD;oBACtD,yDAAyD;oBACzD,MAAM,KAAK,GAAI,KAA+B,CAAC,IAAI,CAAA;oBACnD,IAAI,KAAK,KAAK,QAAQ,EAAE,CAAC;wBACvB,MAAM,KAAK,CAAA;oBACb,CAAC;oBAED,sCAAsC;oBACtC,MAAM,OAAO,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,CAAA;oBACrC,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,CAAC,IAAI;wBAChB,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,IAAI;wBACb,SAAS,EAAE,SAAS;wBACpB,WAAW,EAAE,OAAO,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,SAAS;wBACnE,UAAU,EAAE,KAAK;qBAClB,CAAC,CAAA;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,IAAK,KAA+B,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;YACvD,MAAM,KAAK,CAAA;QACb,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB;IACtC,mCAAmC;IACnC,MAAM,CAAC,YAAY,EAAE,WAAW,CAAC,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC;QACpD,sBAAsB,CAAC,iBAAiB,CAAC;QACzC,sBAAsB,CAAC,iBAAiB,EAAE,CAAC;KAC5C,CAAC,CAAA;IAEF,+DAA+D;IAC/D,MAAM,QAAQ,GAAG,IAAI,GAAG,EAA0B,CAAA;IAElD,0BAA0B;IAC1B,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;QACjC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,CAAA;IACjC,CAAC;IAED,6DAA6D;IAC7D,KAAK,MAAM,KAAK,IAAI,WAAW,EAAE,CAAC;QAChC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,CAAA;IACjC,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAA;AACtC,CAAC"}
+{"version":3,"file":"skills-directory.js","sourceRoot":"","sources":["../../../src/utils/skills-directory.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,aAAa,CAAA;AACrD,OAAO,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAA;AACnC,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAA;AAC3B,OAAO,EAAE,OAAO,EAAE,MAAM,IAAI,CAAA;AAC5B,OAAO,EACL,WAAW,EACX,mBAAmB,EACnB,gBAAgB,EAChB,sBAAsB,GAGvB,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAAE,eAAe,EAAE,MAAM,cAAc,CAAA;AAW9C;;;;;;;GAOG;AACH,MAAM,iBAAiB,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAA;AAE9D;;;GAGG;AACH,SAAS,iBAAiB;IACxB,OAAO,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAA;AACjD,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAC1C,SAAiB,EACjB,MAAe;IAEf,MAAM,MAAM,GAAqB,EAAE,CAAA;IAEnC,wDAAwD;IACxD,IAAI,WAAW,GAAkC,IAAI,CAAA;IACrD,IAAI,MAAM,GAAoB,IAAI,CAAA;IAClC,IAAI,MAAM,EAAE,CAAC;QACX,IAAI,CAAC;YACH,MAAM,GAAG,MAAM,mBAAmB,CAAC,MAAM,CAAC,CAAA;YAC1C,gBAAgB,CAAC,MAAM,CAAC,CAAA;YACxB,WAAW,GAAG,IAAI,sBAAsB,CAAC,MAAM,CAAC,CAAA;QAClD,CAAC;QAAC,MAAM,CAAC;YACP,wDAAwD;YACxD,WAAW,GAAG,IAAI,CAAA;YAClB,MAAM,GAAG,IAAI,CAAA;QACf,CAAC;IACH,CAAC;IAED,IAAI,CAAC;QACH,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,SAAS,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAA;QAEjE,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;YAC5B,IAAI,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;gBACxB,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,EAAE,KAAK,CAAC,IAAI,CAAC,CAAA;gBAC7C,MAAM,WAAW,GAAG,IAAI,CAAC,SAAS,EAAE,UAAU,CAAC,CAAA;gBAE/C,IAAI,CAAC;oBACH,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,WAAW,CAAC,CAAA;oBAC3C,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,WAAW,EAAE,OAAO,CAAC,CAAA;oBACpD,MAAM,MAAM,GAAG,IAAI,WAAW,EAAE,CAAA;oBAChC,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAA;oBAEpC,qEAAqE;oBACrE,mEAAmE;oBACnE,IAAI,UAAU,GAAG,KAAK,CAAA;oBACtB,IAAI,WAAW,IAAI,MAAM,EAAE,CAAC;wBAC1B,IAAI,CAAC;4BACH,MAAM,SAAS,GAAG,MAA4C,CAAA;4BAC9D,MAAM,OAAO,GAAI,SAAS,CAAC,IAAI,CAAwB,IAAI,KAAK,CAAC,IAAI,CAAA;4BACrE,MAAM,aAAa,GAAG,MAAM,WAAW,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAA;4BACjE,IAAI,aAAa,EAAE,CAAC;gCAClB,MAAM,WAAW,GAAG,UAAU,CAAC,QAAQ,CAAC,CAAC,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;gCAC9E,MAAM,UAAU,GACb,SAAS,CAAC,aAAa,CAAwB;oCAC/C,SAAS,CAAC,qBAAqB,CAAwB;oCACxD,EAAE,CAAA;gCACJ,sEAAsE;gCACtE,UAAU,GAAG,UAAU,KAAK,EAAE,IAAI,aAAa,CAAC,YAAY,KAAK,UAAU,CAAA;gCAC3E,kEAAkE;gCAClE,IAAI,CAAC,UAAU,EAAE,CAAC;oCAChB,UAAU,GAAG,aAAa,CAAC,YAAY,KAAK,WAAW,CAAA;gCACzD,CAAC;4BACH,CAAC;wBACH,CAAC;wBAAC,MAAM,CAAC;4BACP,4DAA4D;4BAC5D,UAAU,GAAG,KAAK,CAAA;wBACpB,CAAC;oBACH,CAAC;oBAED,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,MAAM,EAAE,IAAI,IAAI,KAAK,CAAC,IAAI;wBAChC,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,MAAM,EAAE,OAAO,IAAI,IAAI;wBAChC,SAAS,EAAE,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,SAAS;wBAC7D,WAAW,EAAE,WAAW,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,SAAS;wBACvE,UAAU;qBACX,CAAC,CAAA;gBACJ,CAAC;gBAAC,OAAO,KAAK,EAAE,CAAC;oBACf,sDAAsD;oBACtD,yDAAyD;oBACzD,MAAM,KAAK,GAAI,KAA+B,CAAC,IAAI,CAAA;oBACnD,IAAI,KAAK,KAAK,QAAQ,EAAE,CAAC;wBACvB,MAAM,KAAK,CAAA;oBACb,CAAC;oBAED,sCAAsC;oBACtC,MAAM,OAAO,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,CAAA;oBACrC,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,CAAC,IAAI;wBAChB,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,IAAI;wBACb,SAAS,EAAE,SAAS;wBACpB,WAAW,EAAE,OAAO,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,SAAS;wBACnE,UAAU,EAAE,KAAK;qBAClB,CAAC,CAAA;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,IAAK,KAA+B,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;YACvD,MAAM,KAAK,CAAA;QACb,CAAC;IACH,CAAC;YAAS,CAAC;QACT,MAAM,EAAE,KAAK,EAAE,CAAA;IACjB,CAAC;IAED,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CAAC,MAAe;IACtD,MAAM,cAAc,GAAG,MAAM,IAAI,eAAe,CAAA;IAChD,mCAAmC;IACnC,MAAM,CAAC,YAAY,EAAE,WAAW,CAAC,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC;QACpD,sBAAsB,CAAC,iBAAiB,EAAE,cAAc,CAAC;QACzD,sBAAsB,CAAC,iBAAiB,EAAE,EAAE,cAAc,CAAC;KAC5D,CAAC,CAAA;IAEF,+DAA+D;IAC/D,MAAM,QAAQ,GAAG,IAAI,GAAG,EAA0B,CAAA;IAElD,0BAA0B;IAC1B,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;QACjC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,CAAA;IACjC,CAAC;IAED,6DAA6D;IAC7D,KAAK,MAAM,KAAK,IAAI,WAAW,EAAE,CAAC;QAChC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,CAAA;IACjC,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAA;AACtC,CAAC"}

package/dist/tests/create.test.d.ts

@@ -0,0 +1,5 @@
+/**
+ * SMI-3083: skillsmith create command tests
+ */
+export {};
+//# sourceMappingURL=create.test.d.ts.map

package/dist/tests/create.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"create.test.d.ts","sourceRoot":"","sources":["../../tests/create.test.ts"],"names":[],"mappings":"AAAA;;GAEG"}

package/dist/tests/create.test.js

@@ -0,0 +1,449 @@
+/**
+ * SMI-3083: skillsmith create command tests
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { Command } from 'commander';
+// Mock file system
+vi.mock('fs/promises', () => ({
+    mkdir: vi.fn(),
+    writeFile: vi.fn(),
+    stat: vi.fn(),
+}));
+// Mock inquirer
+vi.mock('@inquirer/prompts', () => ({
+    input: vi.fn(),
+    confirm: vi.fn(),
+    select: vi.fn(),
+}));
+// Mock ora
+vi.mock('ora', () => ({
+    default: vi.fn(() => ({
+        start: vi.fn().mockReturnThis(),
+        stop: vi.fn().mockReturnThis(),
+        succeed: vi.fn().mockReturnThis(),
+        fail: vi.fn().mockReturnThis(),
+        warn: vi.fn().mockReturnThis(),
+        text: '',
+    })),
+}));
+// ---------------------------------------------------------------------------
+// validateSkillName (re-exported from utils/skill-name via create.ts)
+// ---------------------------------------------------------------------------
+describe('SMI-3083: validateSkillName', () => {
+    it('accepts valid lowercase-hyphen names', async () => {
+        const { validateSkillName } = await import('../src/commands/create.js');
+        expect(validateSkillName('my-skill')).toBe(true);
+        expect(validateSkillName('skill')).toBe(true);
+        expect(validateSkillName('a1b2-c3')).toBe(true);
+    });
+    it('rejects names with uppercase letters', async () => {
+        const { validateSkillName } = await import('../src/commands/create.js');
+        const result = validateSkillName('My-Skill');
+        expect(result).not.toBe(true);
+        expect(typeof result).toBe('string');
+    });
+    it('rejects names with spaces', async () => {
+        const { validateSkillName } = await import('../src/commands/create.js');
+        expect(validateSkillName('my skill')).not.toBe(true);
+    });
+    it('rejects names starting with a digit', async () => {
+        const { validateSkillName } = await import('../src/commands/create.js');
+        expect(validateSkillName('1skill')).not.toBe(true);
+    });
+    it('rejects empty string', async () => {
+        const { validateSkillName } = await import('../src/commands/create.js');
+        expect(validateSkillName('')).not.toBe(true);
+    });
+    it('rejects names with underscores', async () => {
+        const { validateSkillName } = await import('../src/commands/create.js');
+        expect(validateSkillName('my_skill')).not.toBe(true);
+    });
+});
+// ---------------------------------------------------------------------------
+// createCreateCommand structure
+// ---------------------------------------------------------------------------
+describe('SMI-3083: createCreateCommand', () => {
+    it('creates a command with name "create"', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd).toBeInstanceOf(Command);
+        expect(cmd.name()).toBe('create');
+    });
+    it('has --output option with no baked-in default', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        const opt = cmd.options.find((o) => o.long === '--output');
+        expect(opt).toBeDefined();
+        // default must be undefined — resolved lazily inside createSkill()
+        expect(opt?.defaultValue).toBeUndefined();
+    });
+    it('has --type option', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--type')).toBeDefined();
+    });
+    it('has --behavior option', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--behavior')).toBeDefined();
+    });
+    it('has --description option', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--description')).toBeDefined();
+    });
+    it('has --author option', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--author')).toBeDefined();
+    });
+    it('has --category option', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--category')).toBeDefined();
+    });
+    it('has --scripts flag', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--scripts')).toBeDefined();
+    });
+    it('has --yes flag', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--yes')).toBeDefined();
+    });
+    it('has --dry-run flag', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.options.find((o) => o.long === '--dry-run')).toBeDefined();
+    });
+    it('has description mentioning ~/.claude/skills', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.description()).toContain('~/.claude/skills');
+    });
+    it('has description mentioning skillsmith author init', async () => {
+        const { createCreateCommand } = await import('../src/commands/create.js');
+        const cmd = createCreateCommand();
+        expect(cmd.description()).toContain('author init');
+    });
+});
+// ---------------------------------------------------------------------------
+// createSkill scaffold behaviour
+// ---------------------------------------------------------------------------
+describe('SMI-3083: createSkill scaffold', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    afterEach(() => {
+        vi.restoreAllMocks();
+    });
+    /**
+     * Set up prompt mocks for tests that provide type/behavior/scripts via options.
+     * Prompt order (when those options are not provided via flags):
+     *   input:  description, author
+     *   select: category, [type, behavior if not in options]
+     *   confirm: [scripts if not in options], [overwrite if dir exists and !yes]
+     */
+    async function setupMocks(overrides = {}) {
+        const { mkdir, writeFile, stat } = await import('fs/promises');
+        const { input, confirm, select } = await import('@inquirer/prompts');
+        if (overrides.statRejects !== false) {
+            vi.mocked(stat).mockRejectedValue(new Error('ENOENT'));
+        }
+        else {
+            vi.mocked(stat).mockResolvedValue({ isDirectory: () => true });
+        }
+        vi.mocked(mkdir).mockResolvedValue(undefined);
+        vi.mocked(writeFile).mockResolvedValue(undefined);
+        // Defaults assume type/behavior/scripts are passed via options (skips those prompts).
+        // Only description (input), author (input), and category (select) are prompted.
+        const selectResponses = overrides.selectResponses ?? ['development'];
+        const confirmResponses = overrides.confirmResponses ?? [];
+        const inputResponses = overrides.inputResponses ?? ['A test skill', 'testuser'];
+        for (const val of selectResponses) {
+            vi.mocked(select).mockResolvedValueOnce(val);
+        }
+        for (const val of confirmResponses) {
+            vi.mocked(confirm).mockResolvedValueOnce(val);
+        }
+        for (const val of inputResponses) {
+            vi.mocked(input).mockResolvedValueOnce(val);
+        }
+        return { mkdir, writeFile, stat };
+    }
+    it('scaffolds exactly 4 files when scripts=false (SKILL.md, README.md, CHANGELOG.md, .gitignore)', async () => {
+        const { writeFile } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        });
+        expect(vi.mocked(writeFile)).toHaveBeenCalledTimes(4);
+    });
+    it('scaffolds exactly 5 files when scripts=true (+ scripts/example.js)', async () => {
+        const { writeFile } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: true,
+        });
+        expect(vi.mocked(writeFile)).toHaveBeenCalledTimes(5);
+    });
+    it('creates resources/ directory', async () => {
+        const { mkdir } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        });
+        const mkdirCalls = vi.mocked(mkdir).mock.calls.map((c) => c[0]);
+        expect(mkdirCalls.some((p) => p.endsWith('resources'))).toBe(true);
+    });
+    it('CHANGELOG.md content contains [1.0.0]', async () => {
+        const { writeFile } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        });
+        const call = vi
+            .mocked(writeFile)
+            .mock.calls.find((c) => c[0].endsWith('CHANGELOG.md'));
+        expect(call).toBeDefined();
+        expect(call[1]).toContain('[1.0.0]');
+    });
+    it('SKILL.md content contains Behavioral Classification section', async () => {
+        const { writeFile } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'advanced',
+            behavior: 'guided',
+            scripts: false,
+        });
+        const call = vi.mocked(writeFile).mock.calls.find((c) => c[0].endsWith('SKILL.md'));
+        expect(call).toBeDefined();
+        expect(call[1]).toContain('Behavioral Classification');
+    });
+    it('.gitignore is written with node_modules entry', async () => {
+        const { writeFile } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        });
+        const call = vi
+            .mocked(writeFile)
+            .mock.calls.find((c) => c[0].endsWith('.gitignore'));
+        expect(call).toBeDefined();
+        expect(call[1]).toContain('node_modules/');
+    });
+    it('scripts/example.js uses JSON.stringify (safe for skill names with quotes)', async () => {
+        const { writeFile } = await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: true,
+        });
+        const call = vi
+            .mocked(writeFile)
+            .mock.calls.find((c) => c[0].endsWith('example.js'));
+        expect(call).toBeDefined();
+        // JSON.stringify wraps the string in double quotes — no raw single-quote interpolation
+        expect(call[1]).toContain('"test-skill script executed"');
+    });
+    it('does not write files when user declines overwrite', async () => {
+        // stat resolves (dir exists); confirm returns false (decline)
+        const { writeFile, stat } = await setupMocks({
+            statRejects: false,
+            confirmResponses: [false],
+        });
+        vi.mocked(stat).mockResolvedValue({ isDirectory: () => true });
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('existing-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        });
+        expect(vi.mocked(writeFile)).not.toHaveBeenCalled();
+    });
+    it('--yes always overwrites without prompting even if directory exists', async () => {
+        const { writeFile, stat, mkdir } = await setupMocks();
+        vi.mocked(stat).mockResolvedValue({ isDirectory: () => true });
+        vi.mocked(mkdir).mockResolvedValue(undefined);
+        vi.mocked(writeFile).mockResolvedValue(undefined);
+        const { createSkill } = await import('../src/commands/create.js');
+        const { confirm } = await import('@inquirer/prompts');
+        await createSkill('existing-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+            yes: true,
+        });
+        expect(vi.mocked(confirm)).not.toHaveBeenCalled();
+        expect(vi.mocked(writeFile)).toHaveBeenCalledTimes(4);
+    });
+    it('--dry-run prints preview without writing any files', async () => {
+        await setupMocks();
+        const { createSkill } = await import('../src/commands/create.js');
+        const { writeFile, mkdir } = await import('fs/promises');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+            dryRun: true,
+        });
+        expect(vi.mocked(writeFile)).not.toHaveBeenCalled();
+        expect(vi.mocked(mkdir)).not.toHaveBeenCalled();
+    });
+    it('supports fully non-interactive mode via --description, --author, --category, --type, --behavior, --scripts', async () => {
+        const { mkdir, writeFile, stat } = await import('fs/promises');
+        const { input, confirm, select } = await import('@inquirer/prompts');
+        vi.mocked(stat).mockRejectedValue(new Error('ENOENT'));
+        vi.mocked(mkdir).mockResolvedValue(undefined);
+        vi.mocked(writeFile).mockResolvedValue(undefined);
+        const { createSkill } = await import('../src/commands/create.js');
+        await createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            description: 'A test skill',
+            author: 'testuser',
+            category: 'development',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        });
+        // No prompts should have been called
+        expect(vi.mocked(input)).not.toHaveBeenCalled();
+        expect(vi.mocked(select)).not.toHaveBeenCalled();
+        expect(vi.mocked(confirm)).not.toHaveBeenCalled();
+        expect(vi.mocked(writeFile)).toHaveBeenCalledTimes(4);
+    });
+    it('calls spinner.fail and rethrows when mkdir fails', async () => {
+        const { mkdir, writeFile, stat } = await import('fs/promises');
+        const { input, select } = await import('@inquirer/prompts');
+        const ora = await import('ora');
+        const failMock = vi.fn().mockReturnThis();
+        vi.mocked(ora.default).mockReturnValue({
+            start: vi.fn().mockReturnThis(),
+            stop: vi.fn().mockReturnThis(),
+            succeed: vi.fn().mockReturnThis(),
+            fail: failMock,
+            warn: vi.fn().mockReturnThis(),
+            text: '',
+        });
+        vi.mocked(stat).mockRejectedValue(new Error('ENOENT'));
+        vi.mocked(mkdir).mockRejectedValue(new Error('Permission denied'));
+        vi.mocked(writeFile).mockResolvedValue(undefined);
+        // All input prompts return a valid value; all select prompts return a valid category
+        vi.mocked(input).mockResolvedValue('desc');
+        vi.mocked(select).mockResolvedValue('development');
+        const { createSkill } = await import('../src/commands/create.js');
+        await expect(createSkill('fail-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            scripts: false,
+        })).rejects.toThrow();
+        expect(failMock).toHaveBeenCalled();
+    });
+});
+// ---------------------------------------------------------------------------
+// Validation — CLI flag guards
+// ---------------------------------------------------------------------------
+describe('SMI-3083: createSkill validation guards', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    it('rejects invalid --type value and exits', async () => {
+        const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => {
+            throw new Error('process.exit called');
+        });
+        const { createSkill } = await import('../src/commands/create.js');
+        await expect(createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'invalid-type',
+            behavior: 'autonomous',
+            description: 'desc',
+            author: 'testuser',
+            category: 'development',
+            scripts: false,
+        })).rejects.toThrow();
+        expect(exitSpy).toHaveBeenCalledWith(1);
+        exitSpy.mockRestore();
+    });
+    it('rejects invalid --behavior value and exits', async () => {
+        const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => {
+            throw new Error('process.exit called');
+        });
+        const { createSkill } = await import('../src/commands/create.js');
+        await expect(createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'invalid-behavior',
+            description: 'desc',
+            author: 'testuser',
+            category: 'development',
+            scripts: false,
+        })).rejects.toThrow();
+        expect(exitSpy).toHaveBeenCalledWith(1);
+        exitSpy.mockRestore();
+    });
+    it('rejects invalid --category value and exits', async () => {
+        const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => {
+            throw new Error('process.exit called');
+        });
+        const { createSkill } = await import('../src/commands/create.js');
+        await expect(createSkill('test-skill', {
+            output: '/tmp/test-skills',
+            type: 'basic',
+            behavior: 'autonomous',
+            description: 'desc',
+            author: 'testuser',
+            category: 'invalid-category',
+            scripts: false,
+        })).rejects.toThrow();
+        expect(exitSpy).toHaveBeenCalledWith(1);
+        exitSpy.mockRestore();
+    });
+});
+// ---------------------------------------------------------------------------
+// Template exports
+// ---------------------------------------------------------------------------
+describe('SMI-3083: CHANGELOG_MD_TEMPLATE', () => {
+    it('is exported from templates/index', async () => {
+        const { CHANGELOG_MD_TEMPLATE } = await import('../src/templates/index.js');
+        expect(typeof CHANGELOG_MD_TEMPLATE).toBe('string');
+    });
+    it('contains [1.0.0] placeholder pattern', async () => {
+        const { CHANGELOG_MD_TEMPLATE } = await import('../src/templates/index.js');
+        expect(CHANGELOG_MD_TEMPLATE).toContain('[1.0.0]');
+    });
+    it('contains {{name}} and {{date}} placeholders', async () => {
+        const { CHANGELOG_MD_TEMPLATE } = await import('../src/templates/index.js');
+        expect(CHANGELOG_MD_TEMPLATE).toContain('{{name}}');
+        expect(CHANGELOG_MD_TEMPLATE).toContain('{{date}}');
+    });
+});
+describe('SMI-3083: SKILL_MD_TEMPLATE has {{behavioralClassification}} placeholder', () => {
+    it('contains {{behavioralClassification}} for create/init branching', async () => {
+        const { SKILL_MD_TEMPLATE } = await import('../src/templates/index.js');
+        expect(SKILL_MD_TEMPLATE).toContain('{{behavioralClassification}}');
+    });
+});
+//# sourceMappingURL=create.test.js.map