@boyingliu01/xp-gate 0.6.1 → 0.6.2

package/lib/__tests__/detect-deps.test.js

@@ -1,6 +1,8 @@
 /**
  * @test detect-deps
- * @intent Verify checkDeps() correctly detects missing/present/outdated dependencies
+ * @intent Verify checkDeps() correctly detects missing/present/outdated dependencies across platforms,
+ *         detectPlatform() identifies the correct AI agent platform,
+ *         and autoInstallDeps() handles install scenarios
  */
 const fs = require('fs');
 const path = require('path');
@@ -16,6 +18,7 @@ describe('detect-deps', () => {
     process.env.HOME = tmpHome;
     vi.resetModules();
     delete require.cache[require.resolve('../detect-deps')];
+    delete require.cache[require.resolve('../shared-paths')];
   });
 
   afterEach(() => {
@@ -24,8 +27,10 @@ describe('detect-deps', () => {
     vi.restoreAllMocks();
   });
 
-  function makeSkillDir(skillName, contents = {}) {
-    const dir = path.join(tmpHome, '.config', 'opencode', 'skills', skillName);
+  function makeSkillDir(skillName, contents = {}, baseDir) {
+    const dir = baseDir
+      ? path.join(baseDir, skillName)
+      : path.join(tmpHome, '.config', 'opencode', 'skills', skillName);
     fs.mkdirSync(dir, { recursive: true });
     if (contents.packageJson) {
       fs.writeFileSync(path.join(dir, 'package.json'), JSON.stringify(contents.packageJson));
@@ -48,6 +53,8 @@ describe('detect-deps', () => {
     return dir;
   }
 
+  // ── checkDeps: backward-compatible tests (default platform = opencode) ──
+
   it('returns ok:false missing:superpowers when no deps exist', async () => {
     const { checkDeps } = require('../detect-deps');
     const result = await checkDeps();
@@ -127,7 +134,6 @@ describe('detect-deps', () => {
   });
 
   it('returns ok:true (no version check) when package.json has no version and no SKILL.md', async () => {
-    // getSkillVersion returns null → skips version check → ok
     makeSkillDir('superpowers', { packageJson: {} });
     makeSkillDir('gstack', { packageJson: {} });
     const { checkDeps } = require('../detect-deps');
@@ -147,13 +153,8 @@ describe('detect-deps', () => {
   });
 
   it('returns null version when neither package.json nor SKILL.md exist (skips version check)', async () => {
-    // Create empty dirs (no metadata files)
-    fs.mkdirSync(path.join(tmpHome, '.config', 'opencode', 'skills', 'superpowers'), {
-      recursive: true,
-    });
-    fs.mkdirSync(path.join(tmpHome, '.config', 'opencode', 'skills', 'gstack'), {
-      recursive: true,
-    });
+    fs.mkdirSync(path.join(tmpHome, '.config', 'opencode', 'skills', 'superpowers'), { recursive: true });
+    fs.mkdirSync(path.join(tmpHome, '.config', 'opencode', 'skills', 'gstack'), { recursive: true });
     const { checkDeps } = require('../detect-deps');
     const result = await checkDeps();
     expect(result.ok).toBe(true);
@@ -164,17 +165,14 @@ describe('detect-deps', () => {
     makeSkillDir('gstack', { packageJson: { version: '1.0.0' } });
     const { checkDeps } = require('../detect-deps');
     const result = await checkDeps();
-    // superpowers version=null → skips version check → ok
     expect(result.ok).toBe(true);
   });
 
-  it('compareVersions handles partial versions (e.g. 1.0 treated as 1.0.0)', async () => {
-    // version "1.0" in SKILL.md regex requires X.Y.Z so won't match; use package.json
+  it('compareVersions handles partial versions (e.g. 1 treated as 1.0.0)', async () => {
     makeSkillDir('superpowers', { packageJson: { version: '1' } });
     makeSkillDir('gstack', { packageJson: { version: '1.0.0' } });
     const { checkDeps } = require('../detect-deps');
     const result = await checkDeps();
-    // '1' parsed as [1] vs [1,0,0] → equal at index 0; index 1: 0 vs 0; index 2: 0 vs 0 → equal → passes
     expect(result.ok).toBe(true);
   });
 
@@ -196,14 +194,187 @@ describe('detect-deps', () => {
   });
 
   it('prefers SKILLS_DIR over OPENCODE_DIR when both exist', async () => {
-    // Put low version in SKILLS_DIR, high in OPENCODE_DIR
     makeSkillDir('superpowers', { packageJson: { version: '0.0.1' } });
     makeOpencodeDir('superpowers', { packageJson: { version: '2.0.0' } });
     makeSkillDir('gstack', { packageJson: { version: '1.0.0' } });
     const { checkDeps } = require('../detect-deps');
     const result = await checkDeps();
-    // Should use SKILLS_DIR (first in possiblePaths) → 0.0.1 fails
     expect(result.ok).toBe(false);
     expect(result.versionMismatch.found).toBe('0.0.1');
   });
+
+  // ── Platform-specific tests (Issue #128) ──
+
+  describe('checkDeps with platform parameter', () => {
+    it('qoder: checks ~/.qoder/skills/ for dependencies', async () => {
+      const qoderSkills = path.join(tmpHome, '.qoder', 'skills');
+      makeSkillDir('superpowers', { packageJson: { version: '2.0.0' } }, path.join(qoderSkills, 'superpowers'));
+      makeSkillDir('gstack', { packageJson: { version: '1.0.0' } }, path.join(qoderSkills, 'gstack'));
+      const { checkDeps } = require('../detect-deps');
+      const result = await checkDeps('qoder');
+      expect(result.ok).toBe(true);
+    });
+
+    it('qoder: returns missing when qoder skills dir is empty', async () => {
+      fs.mkdirSync(path.join(tmpHome, '.qoder', 'skills'), { recursive: true });
+      const { checkDeps } = require('../detect-deps');
+      const result = await checkDeps('qoder');
+      expect(result.ok).toBe(false);
+      expect(result.missing).toBe('superpowers');
+    });
+
+    it('claude-code: checks ~/.claude/skills/ for dependencies', async () => {
+      const claudeSkills = path.join(tmpHome, '.claude', 'skills');
+      makeSkillDir('superpowers', { packageJson: { version: '2.0.0' } }, path.join(claudeSkills, 'superpowers'));
+      makeSkillDir('gstack', { packageJson: { version: '1.0.0' } }, path.join(claudeSkills, 'gstack'));
+      const { checkDeps } = require('../detect-deps');
+      const result = await checkDeps('claude-code');
+      expect(result.ok).toBe(true);
+    });
+
+    it('claude-code: returns missing when claude skills dir is empty', async () => {
+      fs.mkdirSync(path.join(tmpHome, '.claude', 'skills'), { recursive: true });
+      const { checkDeps } = require('../detect-deps');
+      const result = await checkDeps('claude-code');
+      expect(result.ok).toBe(false);
+      expect(result.missing).toBe('superpowers');
+    });
+
+    it('unknown platform falls back to opencode profile', async () => {
+      makeSkillDir('superpowers', { packageJson: { version: '2.0.0' } });
+      makeSkillDir('gstack', { packageJson: { version: '1.0.0' } });
+      const { checkDeps } = require('../detect-deps');
+      const result = await checkDeps('unknown-platform');
+      expect(result.ok).toBe(true);
+    });
+
+    it('qoder platform does NOT find deps in opencode dir', async () => {
+      // Put deps in opencode dir but NOT in qoder dir
+      makeSkillDir('superpowers', { packageJson: { version: '2.0.0' } });
+      makeSkillDir('gstack', { packageJson: { version: '1.0.0' } });
+      const { checkDeps } = require('../detect-deps');
+      const result = await checkDeps('qoder');
+      expect(result.ok).toBe(false);
+      expect(result.missing).toBe('superpowers');
+    });
+  });
+
+  // ── detectPlatform tests ──
+
+  describe('detectPlatform', () => {
+    it('returns "qoder" when ~/.qoder/skills/ exists', () => {
+      fs.mkdirSync(path.join(tmpHome, '.qoder', 'skills'), { recursive: true });
+      const { detectPlatform } = require('../detect-deps');
+      expect(detectPlatform()).toBe('qoder');
+    });
+
+    it('returns "claude-code" when ~/.claude/skills/ exists', () => {
+      fs.mkdirSync(path.join(tmpHome, '.claude', 'skills'), { recursive: true });
+      const { detectPlatform } = require('../detect-deps');
+      expect(detectPlatform()).toBe('claude-code');
+    });
+
+    it('returns "opencode" as default when no platform dirs exist', () => {
+      const { detectPlatform } = require('../detect-deps');
+      expect(detectPlatform()).toBe('opencode');
+    });
+
+    it('prefers qoder over claude-code when both exist', () => {
+      fs.mkdirSync(path.join(tmpHome, '.qoder', 'skills'), { recursive: true });
+      fs.mkdirSync(path.join(tmpHome, '.claude', 'skills'), { recursive: true });
+      const { detectPlatform } = require('../detect-deps');
+      expect(detectPlatform()).toBe('qoder');
+    });
+  });
+
+  // ── getSkillsDirs tests ──
+
+  describe('getSkillsDirs', () => {
+    it('returns opencode paths for opencode platform', () => {
+      const { getSkillsDirs } = require('../detect-deps');
+      const dirs = getSkillsDirs('opencode');
+      expect(dirs[0]).toContain('.config');
+      expect(dirs[0]).toContain('opencode');
+    });
+
+    it('returns qoder paths for qoder platform', () => {
+      const { getSkillsDirs } = require('../detect-deps');
+      const dirs = getSkillsDirs('qoder');
+      expect(dirs[0]).toContain('.qoder');
+    });
+
+    it('returns claude-code paths for claude-code platform', () => {
+      const { getSkillsDirs } = require('../detect-deps');
+      const dirs = getSkillsDirs('claude-code');
+      expect(dirs[0]).toContain('.claude');
+    });
+  });
+
+  // ── PLATFORM_PROFILES tests ──
+
+  describe('PLATFORM_PROFILES', () => {
+    it('all platforms have requiredDeps defined', () => {
+      const { PLATFORM_PROFILES } = require('../detect-deps');
+      expect(PLATFORM_PROFILES.opencode.requiredDeps.length).toBe(2);
+      expect(PLATFORM_PROFILES.qoder.requiredDeps.length).toBe(2);
+      expect(PLATFORM_PROFILES['claude-code'].requiredDeps.length).toBe(2);
+    });
+
+    it('all platforms have skillsDirs defined', () => {
+      const { PLATFORM_PROFILES } = require('../detect-deps');
+      expect(PLATFORM_PROFILES.opencode.skillsDirs.length).toBeGreaterThan(0);
+      expect(PLATFORM_PROFILES.qoder.skillsDirs.length).toBeGreaterThan(0);
+      expect(PLATFORM_PROFILES['claude-code'].skillsDirs.length).toBeGreaterThan(0);
+    });
+  });
+
+  // ── autoInstallDeps tests ──
+
+  describe('autoInstallDeps', () => {
+    it('returns ok:true with empty installed when deps already exist', async () => {
+      makeSkillDir('superpowers', { packageJson: { version: '2.0.0' } });
+      makeSkillDir('gstack', { packageJson: { version: '1.0.0' } });
+      const { autoInstallDeps } = require('../detect-deps');
+      const result = await autoInstallDeps();
+      expect(result.ok).toBe(true);
+      expect(result.installed).toEqual([]);
+    });
+
+    it('creates skills directory if it does not exist', async () => {
+      const { autoInstallDeps } = require('../detect-deps');
+      // Mock execSync to avoid actual git clone
+      const { execSync } = require('child_process');
+      vi.spyOn(require('child_process'), 'execSync').mockImplementation((cmd) => {
+        // Simulate successful clone by creating the target dir
+        const match = cmd.match(/"([^"]+)"$/);
+        if (match) {
+          fs.mkdirSync(match[1], { recursive: true });
+          fs.writeFileSync(path.join(match[1], 'package.json'), JSON.stringify({ version: '2.0.0' }));
+        }
+        return '';
+      });
+
+      const result = await autoInstallDeps();
+      // Should have attempted to install
+      expect(result.ok).toBe(true);
+      expect(result.installed).toContain('superpowers');
+      expect(result.installed).toContain('gstack');
+
+      vi.restoreAllMocks();
+    });
+
+    it('returns errors when git clone fails', async () => {
+      const { autoInstallDeps } = require('../detect-deps');
+      vi.spyOn(require('child_process'), 'execSync').mockImplementation(() => {
+        throw new Error('git not found');
+      });
+
+      const result = await autoInstallDeps();
+      expect(result.ok).toBe(false);
+      expect(result.errors.length).toBeGreaterThan(0);
+      expect(result.errors[0].message).toContain('git not found');
+
+      vi.restoreAllMocks();
+    });
+  });
 });

package/lib/__tests__/doctor.test.js

@@ -27,6 +27,7 @@ describe('doctor', () => {
     delete require.cache[require.resolve('../uninstall')];
     delete require.cache[require.resolve('../init')];
     delete require.cache[require.resolve('../detect-deps.js')];
+    delete require.cache[require.resolve('../shared-paths')];
     logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});

package/lib/__tests__/init.test.js

@@ -24,6 +24,7 @@ describe('init', () => {
     vi.resetModules();
     delete require.cache[require.resolve('../init')];
     delete require.cache[require.resolve('../detect-deps.js')];
+    delete require.cache[require.resolve('../shared-paths')];
     logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
@@ -102,6 +103,13 @@ describe('init', () => {
   });
 
   it('init([]) with missing superpowers warns Missing dependencies', async () => {
+    // Mock execSync to make auto-install fail immediately
+    vi.spyOn(childProcess, 'execSync').mockImplementation((cmd) => {
+      if (typeof cmd === 'string' && cmd.includes('git clone')) {
+        throw new Error('git clone not available in test');
+      }
+      return '';
+    });
     const { init } = require('../init');
     const result = await init([]);
     expect(result).toBe(0);
@@ -110,6 +118,13 @@ describe('init', () => {
   });
 
   it('init([]) with versionMismatch warns version detail', async () => {
+    // Mock execSync to make auto-install fail immediately
+    vi.spyOn(childProcess, 'execSync').mockImplementation((cmd) => {
+      if (typeof cmd === 'string' && cmd.includes('git clone')) {
+        throw new Error('git clone not available in test');
+      }
+      return '';
+    });
     // superpowers too old, gstack good
     const sp = path.join(skillsDir(), 'superpowers');
     fs.mkdirSync(sp, { recursive: true });

package/lib/__tests__/install-skill.test.js

@@ -69,6 +69,7 @@ describe('install-skill', () => {
     vi.resetModules();
     delete require.cache[require.resolve('../install-skill')];
     delete require.cache[require.resolve('../detect-deps')];
+    delete require.cache[require.resolve('../shared-paths')];
     delete require.cache[require.resolve('../download-skill')];
     delete require.cache[require.resolve('../rollback')];
     vi.spyOn(console, 'log').mockImplementation(() => {});

package/lib/__tests__/uninstall.test.js

@@ -27,6 +27,7 @@ describe('uninstall', () => {
     delete require.cache[require.resolve('../uninstall')];
     delete require.cache[require.resolve('../init')];
     delete require.cache[require.resolve('../detect-deps.js')];
+    delete require.cache[require.resolve('../shared-paths')];
     logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});

package/lib/detect-deps.js

@@ -3,20 +3,72 @@ const path = require('path');
 const os = require('os');
 const { execSync } = require('child_process');
 
+const { HOME_DIR } = require('./shared-paths.js');
+
+const REQUIRED_DEPS = [
+  { name: 'superpowers', repo: 'obra/superpowers', minVersion: '1.0.0' },
+  { name: 'gstack', repo: 'garrytan/gstack', minVersion: '1.0.0' }
+];
+
 /**
- * Resolve the user's home directory cross-platform.
- * Fallback chain: HOME → USERPROFILE → os.homedir()
- * This covers all known Windows/Linux/macOS/edge cases.
+ * Platform profiles — each AI agent platform has its own skills directory.
+ * All platforms require the same dependencies (superpowers, gstack).
+ *
+ * @covers Issue #128 Bug 1 — Qoder platform was excluded from dependency checks
  */
-const HOME = process.env.HOME || process.env.USERPROFILE || os.homedir();
+const PLATFORM_PROFILES = {
+  opencode: {
+    skillsDirs: [
+      path.join(HOME_DIR, '.config', 'opencode', 'skills'),
+      path.join(HOME_DIR, '.config', 'opencode'),
+    ],
+    requiredDeps: REQUIRED_DEPS,
+  },
+  'claude-code': {
+    skillsDirs: [
+      path.join(HOME_DIR, '.claude', 'skills'),
+      path.join(HOME_DIR, '.claude'),
+    ],
+    requiredDeps: REQUIRED_DEPS,
+  },
+  qoder: {
+    skillsDirs: [
+      path.join(HOME_DIR, '.qoder', 'skills'),
+      path.join(HOME_DIR, '.qoder'),
+    ],
+    requiredDeps: REQUIRED_DEPS,
+  },
+};
 
-const SKILLS_DIR = path.join(HOME, '.config', 'opencode', 'skills');
-const OPENCODE_DIR = path.join(HOME, '.config', 'opencode');
+/**
+ * Detect which AI agent platform is currently in use.
+ * Checks for platform-specific directories in the user's home.
+ * Falls back to 'opencode' if no platform is detected.
+ *
+ * @returns {'opencode' | 'claude-code' | 'qoder'}
+ */
+function detectPlatform() {
+  // Check for Qoder-specific marker
+  if (fs.existsSync(path.join(HOME_DIR, '.qoder', 'skills'))) {
+    return 'qoder';
+  }
+  // Check for Claude Code-specific marker
+  if (fs.existsSync(path.join(HOME_DIR, '.claude', 'skills'))) {
+    return 'claude-code';
+  }
+  // Default to opencode (most common, backward compatible)
+  return 'opencode';
+}
 
-const REQUIRED_DEPS = [
-  { name: 'superpowers', minVersion: '1.0.0' },
-  { name: 'gstack', minVersion: '1.0.0' }
-];
+/**
+ * Get the skills directories for a given platform.
+ * @param {string} platform - 'opencode' | 'claude-code' | 'qoder'
+ * @returns {string[]}
+ */
+function getSkillsDirs(platform) {
+  const profile = PLATFORM_PROFILES[platform] || PLATFORM_PROFILES.opencode;
+  return profile.skillsDirs;
+}
 
 /**
  * Check if bash is available on the system.
@@ -84,25 +136,28 @@ function checkBash() {
   }
 }
 
-async function checkDeps() {
+/**
+ * Check if required dependencies are installed for the given platform.
+ * @param {string} [platform='opencode'] - AI agent platform
+ * @returns {Promise<{ok: boolean, missing?: string, versionMismatch?: object}>}
+ */
+async function checkDeps(platform = 'opencode') {
+  const skillsDirs = getSkillsDirs(platform);
+
   for (const dep of REQUIRED_DEPS) {
-    const possiblePaths = [
-      path.join(SKILLS_DIR, dep.name),
-      path.join(OPENCODE_DIR, dep.name)
-    ];
-    
     let depDir = null;
-    for (const p of possiblePaths) {
-      if (fs.existsSync(p)) {
-        depDir = p;
+    for (const baseDir of skillsDirs) {
+      const candidate = path.join(baseDir, dep.name);
+      if (fs.existsSync(candidate)) {
+        depDir = candidate;
         break;
       }
     }
-    
+
     if (!depDir) {
       return { ok: false, missing: dep.name };
     }
-    
+
     const version = await getSkillVersion(depDir);
     if (version && compareVersions(version, dep.minVersion) < 0) {
       return {
@@ -115,10 +170,101 @@ async function checkDeps() {
       };
     }
   }
-  
+
   return { ok: true };
 }
 
+/**
+ * Auto-install missing dependencies by cloning from GitHub.
+ * @param {string} [platform='opencode'] - AI agent platform
+ * @returns {Promise<{ok: boolean, installed?: string[], errors?: Array<{name: string, message: string}>}>}
+ */
+async function autoInstallDeps(platform = 'opencode') {
+  const skillsDirs = getSkillsDirs(platform);
+  const targetDir = skillsDirs[0]; // Install to the primary skills directory
+
+  if (!fs.existsSync(targetDir)) {
+    try {
+      fs.mkdirSync(targetDir, { recursive: true });
+    } catch (e) {
+      return {
+        ok: false,
+        errors: [{ name: 'mkdir', message: `Cannot create skills directory: ${e.message}` }]
+      };
+    }
+  }
+
+  const check = await checkDeps(platform);
+  if (check.ok) {
+    return { ok: true, installed: [] };
+  }
+
+  const errors = [];
+  const installed = [];
+
+  // Check each dep: missing → install, version mismatch → can't auto-fix
+  const depCheck = await checkDeps(platform);
+  if (depCheck.ok) {
+    return { ok: true, installed: [] };
+  }
+
+  // If the issue is version mismatch, auto-install can't help
+  if (depCheck.versionMismatch) {
+    return {
+      ok: false,
+      installed: [],
+      errors: [{
+        name: depCheck.versionMismatch.name,
+        message: `version mismatch: need ${depCheck.versionMismatch.required}, found ${depCheck.versionMismatch.found} (auto-install cannot upgrade)`
+      }]
+    };
+  }
+
+  for (const dep of REQUIRED_DEPS) {
+    // Re-check if this specific dep is missing
+    let exists = false;
+    for (const baseDir of skillsDirs) {
+      if (fs.existsSync(path.join(baseDir, dep.name))) {
+        exists = true;
+        break;
+      }
+    }
+    if (exists) continue;
+
+    const destPath = path.join(targetDir, dep.name);
+    const repoUrl = `https://github.com/${dep.repo}.git`;
+
+    try {
+      console.log(`  ${dep.name}: not found → installing from ${repoUrl} ...`);
+      const cp = require('child_process');
+      cp.execSync(`git clone --depth 1 "${repoUrl}" "${destPath}"`, {
+        encoding: 'utf8',
+        stdio: ['ignore', 'pipe', 'pipe'],
+        timeout: 120000, // 2 minute timeout
+      });
+      installed.push(dep.name);
+      console.log(`  ${dep.name}: OK`);
+    } catch (e) {
+      errors.push({ name: dep.name, message: e.message || 'git clone failed' });
+      console.warn(`  ${dep.name}: FAILED (${e.message || 'unknown error'})`);
+      // Clean up partial clone
+      try {
+        if (fs.existsSync(destPath)) {
+          fs.rmSync(destPath, { recursive: true, force: true });
+        }
+      } catch {
+        // Ignore cleanup failures
+      }
+    }
+  }
+
+  if (errors.length > 0) {
+    return { ok: false, installed, errors };
+  }
+
+  return { ok: true, installed };
+}
+
 async function getSkillVersion(skillDir) {
   const pkgFile = path.join(skillDir, 'package.json');
   if (fs.existsSync(pkgFile)) {
@@ -127,7 +273,7 @@ async function getSkillVersion(skillDir) {
       return pkg.version;
     } catch {}
   }
-  
+
   const skillFile = path.join(skillDir, 'SKILL.md');
   if (fs.existsSync(skillFile)) {
     const content = fs.readFileSync(skillFile, 'utf8');
@@ -136,22 +282,30 @@ async function getSkillVersion(skillDir) {
       return versionMatch[1];
     }
   }
-  
+
   return null;
 }
 
 function compareVersions(a, b) {
   const partsA = a.split('.').map(Number);
   const partsB = b.split('.').map(Number);
-  
+
   for (let i = 0; i < 3; i++) {
     const partA = partsA[i] || 0;
     const partB = partsB[i] || 0;
     if (partA > partB) return 1;
     if (partA < partB) return -1;
   }
-  
+
   return 0;
 }
 
-module.exports = { checkDeps, checkBash };
+module.exports = {
+  checkDeps,
+  checkBash,
+  autoInstallDeps,
+  detectPlatform,
+  getSkillsDirs,
+  PLATFORM_PROFILES,
+  REQUIRED_DEPS,
+};

package/lib/doctor.js

@@ -1,15 +1,13 @@
 const fs = require('fs');
 const path = require('path');
-const os = require('os');
 const { execSync } = require('child_process');
-
-// Cross-platform home directory resolution
-const HOME_DIR = process.env.HOME || process.env.USERPROFILE || os.homedir();
-
-const CONFIG_DIR = path.join(HOME_DIR, '.config', 'xp-gate');
-const CONFIG_FILE = path.join(CONFIG_DIR, 'xp-gate.json');
-const GLOBAL_HOOKS_DIR = path.join(CONFIG_DIR, 'hooks');
-const GLOBAL_ADAPTERS_DIR = path.join(CONFIG_DIR, 'adapters');
+const {
+  HOME_DIR,
+  CONFIG_DIR,
+  CONFIG_FILE,
+  GLOBAL_HOOKS_DIR,
+  GLOBAL_ADAPTERS_DIR,
+} = require('./shared-paths.js');
 
 // npm package source dir (template hooks/adapters)
 const PKG_DIR = path.dirname(__dirname);

package/lib/init.js

@@ -1,18 +1,15 @@
 const fs = require('fs');
 const path = require('path');
-const os = require('os');
 const crypto = require('crypto');
-const { checkDeps } = require('./detect-deps.js');
-const { checkBash } = require('./detect-deps.js');
-
-// Cross-platform home directory resolution with os.homedir() fallback
-const HOME_DIR = process.env.HOME || process.env.USERPROFILE || os.homedir();
-
-const CONFIG_DIR = path.join(HOME_DIR, '.config', 'xp-gate');
-const CONFIG_FILE = path.join(CONFIG_DIR, 'xp-gate.json');
-const TEMPLATE_DIR = path.join(HOME_DIR, '.config', 'opencode', 'git-hooks-template');
-const GLOBAL_HOOKS_DIR = path.join(CONFIG_DIR, 'hooks');
-const GLOBAL_ADAPTERS_DIR = path.join(CONFIG_DIR, 'adapters');
+const { checkDeps, checkBash, autoInstallDeps, detectPlatform } = require('./detect-deps.js');
+const {
+  HOME_DIR,
+  CONFIG_DIR,
+  CONFIG_FILE,
+  TEMPLATE_DIR,
+  GLOBAL_HOOKS_DIR,
+  GLOBAL_ADAPTERS_DIR,
+} = require('./shared-paths.js');
 
 function copyHooks(srcDir, destDir) {
   ['pre-commit', 'pre-push'].forEach(hook => {
@@ -54,6 +51,35 @@ function logDeps(depCheck) {
   }
 }
 
+/**
+ * Detect which AI agent platform is in use and check/auto-install deps.
+ * @param {string} platform - 'opencode' | 'claude-code' | 'qoder'
+ */
+async function checkAndInstallDeps(platform) {
+  const depCheck = await checkDeps(platform);
+  if (depCheck.ok) {
+    logDeps(depCheck);
+    return depCheck;
+  }
+
+  // Auto-install missing deps
+  console.log(`Checking dependencies (platform: ${platform})...`);
+  const installResult = await autoInstallDeps(platform);
+  if (installResult.ok) {
+    console.log('Dependencies: OK (auto-installed)\n');
+    return { ok: true };
+  }
+
+  // Auto-install failed — report and continue with warning
+  logDeps(depCheck);
+  if (installResult.errors) {
+    for (const err of installResult.errors) {
+      console.warn(`  Auto-install failed for ${err.name}: ${err.message}`);
+    }
+  }
+  return depCheck;
+}
+
 function printUsage() {
   console.log('Choose installation mode:');
   console.log('  1) Global  — all git projects use the same hooks (recommended)');
@@ -228,7 +254,10 @@ async function init(args) {
     console.warn(`  ${bashCheck.message}\n`);
   }
 
-  logDeps(await checkDeps());
+  // Detect platform and check/auto-install dependencies
+  const platform = detectPlatform();
+  console.log(`Platform: ${platform}\n`);
+  await checkAndInstallDeps(platform);
 
   const installMode = args.includes('--global') ? 'global' :
                       args.includes('--core-only') ? 'local' :

package/lib/install-skill.js

@@ -1,18 +1,14 @@
 const fs = require('fs');
 const path = require('path');
-const os = require('os');
 const https = require('https');
 const http = require('http');
 const { execSync } = require('child_process');
-const { checkDeps } = require('./detect-deps.js');
+const { checkDeps, detectPlatform } = require('./detect-deps.js');
 const { downloadFromGitHub } = require('./download-skill.js');
 const { rollback } = require('./rollback.js');
+const { HOME_DIR, CONFIG_DIR } = require('./shared-paths.js');
 
-// Cross-platform home directory resolution
-const HOME = process.env.HOME || process.env.USERPROFILE || os.homedir();
-
-const CONFIG_DIR = path.join(HOME, '.config', 'xp-gate');
-const SKILLS_DIR = path.join(HOME, '.config', 'opencode', 'skills');
+const SKILLS_DIR = path.join(HOME_DIR, '.config', 'opencode', 'skills');
 
 const SKILLS_REGISTRY = {
   'sprint-flow': { repo: 'boyingliu01/xp-gate', path: 'skills/sprint-flow' },
@@ -24,7 +20,8 @@ const SKILLS_REGISTRY = {
 async function installSkill(name, options = {}) {
   const { offline = false, verbose = false, force = false } = options;
   
-  const depCheck = await checkDeps();
+  const platform = detectPlatform();
+  const depCheck = await checkDeps(platform);
   if (!depCheck.ok) {
     if (depCheck.missing) {
       console.error(`Error: ${depCheck.missing} is required but not installed`);

package/lib/shared-paths.js

@@ -0,0 +1,30 @@
+/**
+ * Shared filesystem path constants used across CLI modules.
+ * Cross-platform home directory resolution with os.homedir() fallback.
+ *
+ * @intent Eliminate duplicate path constants in init.js / uninstall.js / detect-deps.js
+ * @covers Issue #107 — duplicate code between init.js and uninstall.js
+ */
+const path = require('path');
+const os = require('os');
+
+/**
+ * Resolve the user's home directory cross-platform.
+ * Fallback chain: HOME → USERPROFILE → os.homedir()
+ */
+const HOME_DIR = process.env.HOME || process.env.USERPROFILE || os.homedir();
+
+const CONFIG_DIR = path.join(HOME_DIR, '.config', 'xp-gate');
+const CONFIG_FILE = path.join(CONFIG_DIR, 'xp-gate.json');
+const TEMPLATE_DIR = path.join(HOME_DIR, '.config', 'opencode', 'git-hooks-template');
+const GLOBAL_HOOKS_DIR = path.join(CONFIG_DIR, 'hooks');
+const GLOBAL_ADAPTERS_DIR = path.join(CONFIG_DIR, 'adapters');
+
+module.exports = {
+  HOME_DIR,
+  CONFIG_DIR,
+  CONFIG_FILE,
+  TEMPLATE_DIR,
+  GLOBAL_HOOKS_DIR,
+  GLOBAL_ADAPTERS_DIR,
+};

package/lib/uninstall.js

@@ -1,16 +1,15 @@
 const fs = require('fs');
 const path = require('path');
-const os = require('os');
 const crypto = require('crypto');
+const {
+  HOME_DIR,
+  CONFIG_DIR,
+  CONFIG_FILE,
+  TEMPLATE_DIR,
+  GLOBAL_HOOKS_DIR,
+  GLOBAL_ADAPTERS_DIR,
+} = require('./shared-paths.js');
 
-// Cross-platform home directory resolution
-const HOME_DIR = process.env.HOME || process.env.USERPROFILE || os.homedir();
-
-const CONFIG_DIR = path.join(HOME_DIR, '.config', 'xp-gate');
-const CONFIG_FILE = path.join(CONFIG_DIR, 'xp-gate.json');
-const TEMPLATE_DIR = path.join(HOME_DIR, '.config', 'opencode', 'git-hooks-template');
-const GLOBAL_HOOKS_DIR = path.join(CONFIG_DIR, 'hooks');
-const GLOBAL_ADAPTERS_DIR = path.join(CONFIG_DIR, 'adapters');
 const BACKUP_DIR = path.join(CONFIG_DIR, '.uninstall-backup');
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/xp-gate",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "./bin/xp-gate.js"

package/plugins/claude-code/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Claude Code. Includes 6 quality gates, Sprint Flow, and Delphi multi-expert review.",
   "author": {

package/skills/sprint-flow/SKILL.md

@@ -462,9 +462,12 @@ Phase 2 第一步必须执行 DELPHI-GATE 检查。没有 delphi-review APPROVED
 - 输入: phase-4-summary.md（验收结果）+ emergent-issues.md（如有）
 - 输出: `feedback-log.md`
 - **HARD-GATE**: Phase 5 不可跳过。Phase 4 完成后 → 必须进入 Phase 5 → 完成后才能进入 Phase 6。
-- **`learn` (gstack)** — Sprint 级复盘（这是 /learn 在本项目中的主要调用时机）
+- **`learn` (gstack)** — Sprint 级复盘（**Phase 5 必须自动调用，不依赖手动触发**）
+  - **默认提炼模板**（无需用户额外输入）：
+    > 提炼总结并保存可复用的经验教训，把大模型不知道、并且犯错后无法立即发现和纠正的知识保存下来。如果是对其他项目也有价值的，就保存成全局记忆，否则保存为项目记忆。
   - ralph-loop 已在 BUILD Phase 内部实现 per-REQ learn（permanent/contextual 分类）
   - Phase 5 额外进行 Sprint 级复盘，总结全 Phase 经验
+  - **learnings 自动注入**：`/learn export` 时自动 append 到 CLAUDE.md / AGENTS.md 末尾形成 `## Project Learnings` 章节
 - **`retro` (gstack)** — 工程回顾：提交历史、工作模式、代码质量趋势
 - **`systematic-debugging` (superpowers)** — 根因调试
 
@@ -620,10 +623,11 @@ Phase 2 第一步必须执行 DELPHI-GATE 检查。没有 delphi-review APPROVED
 
 3. **等待用户确认 checkpoint**（如适用）
 
-4. **展示进度看板**：读取 `sprint-state.json`，使用 `@templates/sprint-progress-template.md` 渲染进度看板
+4. **展示进度看板**：执行 `node scripts/render-sprint-progress.cjs` 渲染进度看板
+   - 脚本自动读取 `.sprint-state/sprint-state.json` 并输出 ASCII 进度看板
    - 渲染规则：已完成阶段显示 ✅ + 耗时，当前阶段 🔄，待做 ⬜，跳过 ⏭️，失败 ❌
    - 进度条：`[████▓░░░░░░] {pct}%`（已完成数/总阶段数 11）
-   - 下一步行动：根据当前阶段 + 状态，从模板的"下一步行动表"中查找对应提示
+   - 下一步行动：根据当前阶段 + 状态，自动查找对应提示
    - 输出物路径：列出 `outputs` 中已有的文件路径
    - 时机：每个 Phase 完成后的 transition 阶段自动展示，用户无需请求
    - 向后兼容：旧版 `sprint-state.json` 缺少 `phase_history` 时，从 `phase` 字段推断状态
@@ -792,17 +796,16 @@ Sprint state is persisted as JSON in `.sprint-state/sprint-state.json`:
 
 ```bash
 /sprint-flow --status
-# → 读取 .sprint-state/sprint-state.json
-# → 渲染进度看板（使用 @templates/sprint-progress-template.md）
+# → 执行 node scripts/render-sprint-progress.cjs
+# → 读取 .sprint-state/sprint-state.json 并渲染 ASCII 进度看板
 # → 不执行任何 Phase，仅展示当前状态
 # 适用场景：碎片时间恢复时快速查看进度、中断后确认当前阶段和下一步操作
 ```
 
 **行为规则**：
-- 读取 `.sprint-state/sprint-state.json` 获取 `id`, `task_description`, `phase`, `status`, `phase_history`, `outputs`, `isolation`
-- 扫描 `.sprint-state/phase-outputs/phase-*-summary.md` 获取各阶段 `status` 和 `phase_name`
-- 如果 `sprint-state.json` 不存在 → 输出 `[INFO] 未找到活跃的 Sprint。请先运行 /sprint-flow "[需求描述]" 启动新 Sprint。`
-- 如果 `status == "completed"` → 展示完整看板 + `[INFO] Sprint 已完成。` + Sprint Summary 路径
+- 执行 `node scripts/render-sprint-progress.cjs`（脚本自动处理所有渲染逻辑）
+- 如果 `sprint-state.json` 不存在 → 脚本输出 `[INFO] 未找到活跃的 Sprint。请先运行 /sprint-flow "[需求描述]" 启动新 Sprint。`
+- 如果 `status == "completed"` → 脚本输出完整看板 + `[INFO] Sprint 已完成。`
 - `--status` 可与其他参数组合：`--status --resume-from build` → 先展示状态，再提示 "将从 Phase 2 BUILD 继续"
 - 向后兼容：旧版 `sprint-state.json` 缺少 `phase_history`/`task_description` 时，按模板"向后兼容"规则渲染