@shepai/cli 1.136.0-pr438.9e52ee5 → 1.136.0-pr439.2a7af41

package/dist/packages/core/src/infrastructure/services/spec/spec-initializer.service.js

@@ -7,6 +7,8 @@
  */
 import { mkdir, readdir, writeFile } from 'node:fs/promises';
 import { join } from 'node:path';
+import { computeRepoHash } from '../../services/filesystem/repo-hash.js';
+import { getShepHomeDir } from '../../services/filesystem/shep-directory.service.js';
 /**
  * Pad a number to 3 digits with leading zeros.
  */
@@ -240,13 +242,14 @@ const TEMPLATES = [
     { filename: 'feature.yaml', content: FEATURE_YAML },
 ];
 export class SpecInitializerService {
-    async initialize(basePath, slug, featureNumber, description, mode) {
-        // Scan existing specs/ directory for highest NNN prefix to avoid collisions
+    async initialize(basePath, slug, featureNumber, description, mode, options) {
+        const storageMode = options?.storageMode ?? 'in-repo';
+        // Scan existing spec directories for highest NNN prefix to avoid collisions
         // (specs may have been created outside the DB, e.g., via /shep-kit:new-feature)
-        const resolvedNumber = await this.resolveNextNumber(basePath, featureNumber);
+        const resolvedNumber = await this.resolveNextNumber(basePath, featureNumber, options?.repositoryPath);
         const nnn = padNumber(resolvedNumber);
         const specDirName = `${nnn}-${slug}`;
-        const specDir = join(basePath, 'specs', specDirName);
+        const specDir = this.computeSpecDir(basePath, specDirName, storageMode, options?.repositoryPath);
         // Create the spec directory
         await mkdir(specDir, { recursive: true });
         const now = new Date();
@@ -263,26 +266,59 @@ export class SpecInitializerService {
         return { specDir, featureNumber: nnn };
     }
     /**
-     * Scan specs/ for existing NNN-* directories and return the next available number.
+     * Compute the spec directory path based on storage mode.
+     */
+    computeSpecDir(basePath, specDirName, storageMode, repositoryPath) {
+        if (storageMode === 'shep-managed') {
+            if (!repositoryPath) {
+                throw new Error('repositoryPath is required when storageMode is shep-managed');
+            }
+            const repoHash = computeRepoHash(repositoryPath);
+            return join(getShepHomeDir(), 'repos', repoHash, 'specs', specDirName);
+        }
+        return join(basePath, '.shep', 'specs', specDirName);
+    }
+    /**
+     * Scan spec directories for existing NNN-* entries and return the next available number.
+     * Scans up to 3 locations to prevent collisions during the transition period:
+     * 1. specs/ (legacy location)
+     * 2. .shep/specs/ (new in-repo location)
+     * 3. ~/.shep/repos/<hash>/specs/ (shep-managed location, when repositoryPath provided)
+     *
      * Uses the DB-derived hint as a minimum, but always respects filesystem state.
      */
-    async resolveNextNumber(basePath, hint) {
-        const specsDir = join(basePath, 'specs');
+    async resolveNextNumber(basePath, hint, repositoryPath) {
+        const dirsToScan = [join(basePath, 'specs'), join(basePath, '.shep', 'specs')];
+        if (repositoryPath) {
+            const repoHash = computeRepoHash(repositoryPath);
+            dirsToScan.push(join(getShepHomeDir(), 'repos', repoHash, 'specs'));
+        }
+        let maxExisting = 0;
+        for (const dir of dirsToScan) {
+            const max = await this.scanDirForMaxNumber(dir);
+            maxExisting = Math.max(maxExisting, max);
+        }
+        return Math.max(maxExisting + 1, hint);
+    }
+    /**
+     * Scan a single directory for NNN-* prefixed entries and return the max number found.
+     * Returns 0 if the directory does not exist or is empty.
+     */
+    async scanDirForMaxNumber(dirPath) {
         let entries;
         try {
-            entries = await readdir(specsDir);
+            entries = await readdir(dirPath);
         }
         catch {
-            // specs/ doesn't exist yet — use hint
-            return hint;
+            return 0;
         }
-        let maxExisting = 0;
+        let max = 0;
         for (const entry of entries) {
             const match = entry.match(/^(\d{3})-/);
             if (match) {
-                maxExisting = Math.max(maxExisting, parseInt(match[1], 10));
+                max = Math.max(max, parseInt(match[1], 10));
             }
         }
-        return Math.max(maxExisting + 1, hint);
+        return max;
     }
 }

package/dist/src/presentation/cli/index.js

@@ -42,7 +42,6 @@ import { createIdeOpenCommand } from './commands/ide-open.command.js';
 import { createInstallCommand } from './commands/install.command.js';
 import { createUpgradeCommand } from './commands/upgrade.command.js';
 import { createToolsCommand } from './commands/tools.command.js';
-import { createDoctorCommand } from './commands/doctor.command.js';
 import { messages } from './ui/index.js';
 // Daemon lifecycle commands
 import { createStartCommand } from './commands/start.command.js';
@@ -111,7 +110,6 @@ async function bootstrap() {
         program.addCommand(createInstallCommand());
         program.addCommand(createToolsCommand());
         program.addCommand(createUpgradeCommand());
-        program.addCommand(createDoctorCommand());
         // Daemon lifecycle commands (task-9)
         program.addCommand(createStartCommand());
         program.addCommand(createStopCommand());