@timekast/factory 2.6.0 → 2.7.0

package/dist/commands/doctor.js

@@ -261,6 +261,7 @@ export function computeDoctor(deps = {}) {
     if (!hasLockfile(rootDir)) {
         return {
             orphans: [],
+            externalSkills: [],
             conflicts: [],
             formatterDrift: [],
             repoOwned: [],
@@ -285,7 +286,7 @@ export function computeDoctor(deps = {}) {
     const manifestPaths = new Set(lock.files.map((f) => f.path));
     // Orphans: files under `.claude/` on disk that the manifest does not track.
     const claudePaths = collectClaudePaths(rootDir);
-    const orphans = claudePaths.filter((p) => !manifestPaths.has(p)).sort();
+    const { orphans, externalSkills } = splitExternalSkills(claudePaths.filter((p) => !manifestPaths.has(p)).sort(), manifestPaths);
     // Pending conflicts: tracked file whose normalized on-disk hash drifted from
     // the recorded hash (a local modification not yet resolved by `update`).
     const conflicts = [];
@@ -335,6 +336,7 @@ export function computeDoctor(deps = {}) {
     }
     return {
         orphans,
+        externalSkills,
         conflicts,
         formatterDrift,
         repoOwned,
@@ -350,6 +352,37 @@ export function computeDoctor(deps = {}) {
         neonRevokePending,
     };
 }
+const SKILLS_DIR_PREFIX = '.claude/skills/';
+/**
+ * Move out of `orphans` every file that lives in a skill directory the manifest knows nothing
+ * about, and name those directories. A loose file inside a KIT skill directory stays an orphan:
+ * that one is a retired or stray kit file, not someone else's skill.
+ */
+export function splitExternalSkills(allOrphans, manifestPaths) {
+    const skillDirOf = (p) => {
+        if (!p.startsWith(SKILLS_DIR_PREFIX))
+            return null;
+        const rest = p.slice(SKILLS_DIR_PREFIX.length);
+        const slash = rest.indexOf('/');
+        return slash > 0 ? rest.slice(0, slash) : null;
+    };
+    const kitSkillDirs = new Set();
+    for (const p of manifestPaths) {
+        const dir = skillDirOf(p);
+        if (dir)
+            kitSkillDirs.add(dir);
+    }
+    const orphans = [];
+    const external = new Set();
+    for (const p of allOrphans) {
+        const dir = skillDirOf(p);
+        if (dir && !kitSkillDirs.has(dir))
+            external.add(dir);
+        else
+            orphans.push(p);
+    }
+    return { orphans, externalSkills: [...external].sort() };
+}
 /** Render the doctor result to the console. The A1 notice is ALWAYS last. */
 function renderDoctor(result, repoVault) {
     const vaultLines = renderRepoVault(repoVault, { suggestAdopt: result.suggestAdopt });
@@ -410,6 +443,13 @@ function renderDoctor(result, repoVault) {
             }
         }
     }
+    // Skills the kit does not manage (#110): informative, never a problem. Shown whatever the rest says.
+    if (result.externalSkills.length > 0) {
+        console.log('\nSkills que no gestiona el kit (propias del proyecto o add-ons con su instalador):');
+        for (const name of result.externalSkills) {
+            console.log(`  • .claude/skills/${name}/ — no es huérfana: no la borres; \`factory update\` no la toca`);
+        }
+    }
     // Repo-owned kit files (#76): informative, never a conflict. Shown whatever the rest says.
     if (result.repoOwned.length > 0) {
         console.log('\nArchivos propios del repo (el kit dejó su versión al lado, en `.kit-example`):');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timekast/factory",
-  "version": "2.6.0",
+  "version": "2.7.0",
   "description": "Public, thin CLI to bootstrap and maintain TimeKast Factory derived projects.",
   "type": "module",
   "license": "UNLICENSED",