sumulige-claude 1.0.9 → 1.0.11

package/lib/commands.js

@@ -9,6 +9,7 @@ const path = require('path');
 const { execSync } = require('child_process');
 const { loadConfig, CONFIG_DIR, CONFIG_FILE, SKILLS_DIR, ensureDir, saveConfig } = require('./config');
 const { copyRecursive, toTitleCase } = require('./utils');
+const { runMigrations, TEMPLATE_VERSION } = require('./migrations');
 
 const TEMPLATE_DIR = path.join(__dirname, '../template');
 
@@ -62,6 +63,7 @@ const commands = {
   // -------------------------------------------------------------------------
   sync: () => {
     console.log('🔄 Syncing Sumulige Claude to current project...');
+    console.log('');
 
     const projectDir = process.cwd();
     const projectConfigDir = path.join(projectDir, '.claude');
@@ -71,6 +73,17 @@ const commands = {
 
     // Create .claude directory
     ensureDir(projectConfigDir);
+
+    // Run migrations (version-aware)
+    const result = runMigrations(projectDir);
+    if (result.migrations.length > 0) {
+      console.log(`📦 Migrating project template → v${TEMPLATE_VERSION}`);
+      result.migrations.forEach(m => {
+        console.log(`   ✅ ${m.description}`);
+      });
+      console.log('');
+    }
+
     console.log('✅ Created .claude directory');
 
     // Sync config
@@ -118,6 +131,55 @@ const commands = {
     console.log('✅ Sync complete!');
   },
 
+  // -------------------------------------------------------------------------
+  migrate: () => {
+    console.log('🔧 Migrating project to latest format...');
+    console.log('');
+
+    const projectDir = process.cwd();
+    const settingsFile = path.join(projectDir, '.claude', 'settings.json');
+
+    if (!fs.existsSync(settingsFile)) {
+      console.log('⚠️  No settings.json found in .claude/');
+      console.log('   Run: smc template');
+      return;
+    }
+
+    let settings;
+    try {
+      settings = JSON.parse(fs.readFileSync(settingsFile, 'utf-8'));
+    } catch (e) {
+      console.log('❌ Invalid JSON in settings.json');
+      return;
+    }
+
+    // 检测旧格式
+    const isOldFormat = settings.matcher || (settings.hooks && typeof settings.hooks === 'object');
+
+    if (!isOldFormat) {
+      console.log('✅ Already using latest format');
+      return;
+    }
+
+    // 读取新模板
+    const templateSettings = path.join(TEMPLATE_DIR, '.claude', 'settings.json');
+    if (!fs.existsSync(templateSettings)) {
+      console.log('❌ Template settings.json not found');
+      return;
+    }
+
+    const newSettings = fs.readFileSync(templateSettings, 'utf-8');
+    fs.writeFileSync(settingsFile, newSettings);
+
+    console.log('✅ Migrated settings.json to latest format');
+    console.log('');
+    console.log('Changes:');
+    console.log('  - Old format: {"matcher": "...", "hooks": [...]}');
+    console.log('  - New format: {"UserPromptSubmit": [...], "PostToolUse": [...]}');
+    console.log('');
+    console.log('ℹ️  Hooks will now work correctly!');
+  },
+
   // -------------------------------------------------------------------------
   agent: (task) => {
     if (!task) {
@@ -565,6 +627,11 @@ const commands = {
     fs.writeFileSync(path.join(targetClaudeDir, 'ANCHORS.md'), anchorsContent);
     console.log('   ✅ .claude/ANCHORS.md');
 
+    // Write version file
+    const { setProjectVersion } = require('./migrations');
+    setProjectVersion(targetDir, TEMPLATE_VERSION);
+    console.log(`   ✅ .claude/.version (v${TEMPLATE_VERSION})`);
+
     // Initialize Sumulige Claude if installed
     console.log('');
     console.log('🤖 Initializing Sumulige Claude...');

package/lib/migrations.js

@@ -0,0 +1,154 @@
+/**
+ * Migrations - Version control for project templates
+ *
+ * 每个迁移函数：
+ * - from: 起始版本
+ * - to: 目标版本
+ * - migrate: 实际迁移逻辑
+ *
+ * 迁移链：v1.0.0 → v1.0.2 → v1.0.10 → ...
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+// 当前模板版本
+const TEMPLATE_VERSION = '1.0.10';
+
+// 版本文件路径
+const getVersionFile = (projectDir) => path.join(projectDir, '.claude', '.version');
+
+// 读取项目版本
+function getProjectVersion(projectDir) {
+  const versionFile = getVersionFile(projectDir);
+  if (!fs.existsSync(versionFile)) {
+    // 没有 version 文件说明是旧版本项目
+    return '1.0.0';
+  }
+  try {
+    return fs.readFileSync(versionFile, 'utf-8').trim();
+  } catch {
+    return '1.0.0';
+  }
+}
+
+// 写入项目版本
+function setProjectVersion(projectDir, version) {
+  const versionFile = getVersionFile(projectDir);
+  fs.writeFileSync(versionFile, version + '\n');
+}
+
+// 版本比较
+function compareVersions(v1, v2) {
+  const parts1 = v1.split('.').map(Number);
+  const parts2 = v2.split('.').map(Number);
+
+  for (let i = 0; i < 3; i++) {
+    if (parts1[i] < parts2[i]) return -1;
+    if (parts1[i] > parts2[i]) return 1;
+  }
+  return 0;
+}
+
+// 迁移列表
+const migrations = [
+  {
+    from: '1.0.0',
+    to: '1.0.10',
+    description: 'Migrate hooks format from old to new',
+    migrate: (projectDir) => {
+      const settingsFile = path.join(projectDir, '.claude', 'settings.json');
+      if (!fs.existsSync(settingsFile)) return;
+
+      let settings;
+      try {
+        settings = JSON.parse(fs.readFileSync(settingsFile, 'utf-8'));
+      } catch {
+        return; // 无效 JSON，跳过
+      }
+
+      // 检测旧格式
+      const isOldFormat = settings.matcher || (settings.hooks && typeof settings.hooks === 'object');
+      if (!isOldFormat) return;
+
+      // 读取新模板
+      const templateDir = path.join(__dirname, '../template');
+      const templateSettings = path.join(templateDir, '.claude', 'settings.json');
+
+      if (fs.existsSync(templateSettings)) {
+        fs.writeFileSync(settingsFile, fs.readFileSync(templateSettings, 'utf-8'));
+      }
+    }
+  }
+];
+
+/**
+ * 执行迁移
+ * @param {string} projectDir - 项目目录
+ * @param {boolean} silent - 是否静默模式
+ * @returns {object} { success, migrations, currentVersion }
+ */
+function runMigrations(projectDir, silent = false) {
+  const currentVersion = getProjectVersion(projectDir);
+
+  // 已经是最新版本
+  if (compareVersions(currentVersion, TEMPLATE_VERSION) >= 0) {
+    return {
+      success: true,
+      migrations: [],
+      currentVersion,
+      templateVersion: TEMPLATE_VERSION
+    };
+  }
+
+  // 找到需要执行的迁移
+  const pendingMigrations = migrations.filter(m => {
+    const cmp = compareVersions(currentVersion, m.to);
+    return cmp < 0; // 只执行版本高于当前版本的迁移
+  });
+
+  // 按版本顺序执行
+  pendingMigrations.sort((a, b) => compareVersions(a.to, b.to));
+
+  const executed = [];
+
+  for (const migration of pendingMigrations) {
+    try {
+      if (!silent) {
+        console.log(`⬆️  Migrating: ${currentVersion} → ${migration.to}`);
+        console.log(`   ${migration.description}`);
+      }
+      migration.migrate(projectDir);
+      executed.push(migration);
+      setProjectVersion(projectDir, migration.to);
+      if (!silent) {
+        console.log(`   ✅ Done`);
+      }
+    } catch (error) {
+      if (!silent) {
+        console.log(`   ❌ Failed: ${error.message}`);
+      }
+      return {
+        success: false,
+        error: error.message,
+        currentVersion,
+        executed
+      };
+    }
+  }
+
+  return {
+    success: true,
+    migrations: executed,
+    currentVersion: TEMPLATE_VERSION,
+    templateVersion: TEMPLATE_VERSION
+  };
+}
+
+module.exports = {
+  TEMPLATE_VERSION,
+  getProjectVersion,
+  setProjectVersion,
+  compareVersions,
+  runMigrations
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sumulige-claude",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "The Best Agent Harness for Claude Code",
   "main": "cli.js",
   "bin": {
@@ -8,7 +8,9 @@
     "sumulige-claude": "cli.js"
   },
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1",
+    "test": "jest",
+    "test:coverage": "jest --coverage",
+    "test:watch": "jest --watch",
     "postinstall": "node cli.js init",
     "sync": "node scripts/sync-external.mjs",
     "update-registry": "node scripts/update-registry.mjs",
@@ -35,7 +37,10 @@
     "node": ">=16.0.0"
   },
   "devDependencies": {
+    "jest": "^30.2.0",
+    "mock-fs": "^5.5.0",
     "prettier": "^3.7.4",
+    "sinon": "^21.0.1",
     "yaml": "^2.8.2"
   }
 }

package/scripts/fix-hooks.mjs

@@ -0,0 +1,97 @@
+#!/usr/bin/env node
+/**
+ * Fix hooks format from old to new
+ * 旧格式：{"matcher": "...", "hooks": [...]}
+ * 新格式：{"UserPromptSubmit": [{"matcher": {}, "hooks": [...]}], ...}
+ */
+
+import fs from 'fs';
+import path from 'path';
+
+const NEW_FORMAT = {
+  "UserPromptSubmit": [
+    {
+      "matcher": {},
+      "hooks": [
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/project-kickoff.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/rag-skill-loader.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs", "timeout": 1000 }
+      ]
+    }
+  ],
+  "PreToolUse": [
+    {
+      "matcher": {},
+      "hooks": [
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs", "timeout": 1000 }
+      ]
+    }
+  ],
+  "PostToolUse": [
+    {
+      "matcher": {},
+      "hooks": [
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/code-formatter.cjs", "timeout": 5000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/verify-work.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs", "timeout": 1000 }
+      ]
+    }
+  ],
+  "AgentStop": [
+    {
+      "matcher": {},
+      "hooks": [
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/verify-work.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs", "timeout": 1000 },
+        { "type": "command", "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs", "timeout": 1000 }
+      ]
+    }
+  ]
+};
+
+function fixProject(projectPath) {
+  const settingsFile = path.join(projectPath, '.claude', 'settings.json');
+
+  if (!fs.existsSync(settingsFile)) {
+    console.log(`⚠️  ${projectPath} - no settings.json`);
+    return false;
+  }
+
+  let settings;
+  try {
+    settings = JSON.parse(fs.readFileSync(settingsFile, 'utf-8'));
+  } catch (e) {
+    console.log(`❌ ${projectPath} - invalid JSON`);
+    return false;
+  }
+
+  // 检测旧格式
+  const isOldFormat = settings.matcher || (settings.hooks && typeof settings.hooks === 'object');
+
+  if (!isOldFormat) {
+    console.log(`✅ ${projectPath} - already new format`);
+    return false;
+  }
+
+  // 写入新格式
+  fs.writeFileSync(settingsFile, JSON.stringify(NEW_FORMAT, null, 2) + '\n');
+  console.log(`🔧 ${projectPath} - fixed!`);
+  return true;
+}
+
+// CLI
+const args = process.argv.slice(2);
+
+if (args.length === 0) {
+  // 当前目录
+  fixProject(process.cwd());
+} else {
+  args.forEach(fixProject);
+}

package/template/.claude/commands/commit-push-pr.md

@@ -22,7 +22,27 @@ Analyze the changes and:
 2. Identify the main purpose of the changes
 3. Check if there are any sensitive files that shouldn't be committed
 
-## Step 3: Stage and Commit
+## Step 3: Code Simplification (Before Commit)
+
+**IMPORTANT**: Before creating the commit, simplify and refine the code changes:
+
+1. Review the recently modified code sections
+2. Apply code simplification focusing on:
+   - Code clarity and consistency
+   - Reducing unnecessary complexity and nesting
+   - Eliminating redundant code and abstractions
+   - Following project standards (if CLAUDE.md exists)
+   - Preserving exact functionality
+3. Use explicit, readable code over overly compact solutions
+
+**Trigger phrases to apply simplification**:
+- "简化此代码" (Simplify this code)
+- "使这更清晰" (Make this clearer)
+- "优化此实现" (Optimize this implementation)
+
+Ask the user: "是否需要简化代码？(Need code simplification?)" - if yes, apply code-simplifier agent before proceeding.
+
+## Step 4: Stage and Commit
 
 If there are unstaged changes that should be included, ask the user first.
 
@@ -36,7 +56,7 @@ Then create a commit with a clear message following this format:
 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
 ```
 
-## Step 4: Push and Create PR
+## Step 5: Push and Create PR
 
 1. Push the changes to remote
 2. Check if there's an open PR for this branch using:
@@ -51,7 +71,7 @@ Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
      - Test plan (checklist)
      - Link to any related issues
 
-## Step 5: Verify
+## Step 6: Verify
 
 After creating the PR, show the PR URL and check CI status using:
 ```bash

package/template/.claude/settings.json

@@ -1,51 +1,115 @@
 {
-  "hooks": {
-    "PostToolUse": [
-      {
-        "matcher": "*",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/project-kickoff.cjs",
-            "timeout": 1000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs",
-            "timeout": 1000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/rag-skill-loader.cjs",
-            "timeout": 1000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/code-formatter.cjs",
-            "timeout": 5000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/verify-work.cjs",
-            "timeout": 1000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs",
-            "timeout": 1000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs",
-            "timeout": 1000
-          },
-          {
-            "type": "command",
-            "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/conversation-logger.cjs",
-            "timeout": 1000
-          }
-        ]
-      }
-    ]
-  }
-}
+  "UserPromptSubmit": [
+    {
+      "matcher": {},
+      "hooks": [
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/project-kickoff.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/rag-skill-loader.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs",
+          "timeout": 1000
+        }
+      ]
+    }
+  ],
+  "PreToolUse": [
+    {
+      "matcher": {},
+      "hooks": [
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs",
+          "timeout": 1000
+        }
+      ]
+    }
+  ],
+  "PostToolUse": [
+    {
+      "matcher": {},
+      "hooks": [
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/code-formatter.cjs",
+          "timeout": 5000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/verify-work.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs",
+          "timeout": 1000
+        }
+      ]
+    }
+  ],
+  "AgentStop": [
+    {
+      "matcher": {},
+      "hooks": [
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/verify-work.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/thinking-silent.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/multi-session.cjs",
+          "timeout": 1000
+        },
+        {
+          "type": "command",
+          "command": "node \"$CLAUDE_PROJECT_DIR\"/.claude/hooks/todo-manager.cjs",
+          "timeout": 1000
+        }
+      ]
+    }
+  ]
+}