sdd-skills 1.1.7 → 1.1.9

package/install.js

@@ -72,10 +72,21 @@ async function promptInstallation() {
       // Removed default to force user selection
     },
     {
-      type: 'confirm',
+      type: 'input',
       name: 'configureDingTalk',
-      message: 'Would you like to configure DingTalk notifications?',
-      default: false,
+      message: 'Would you like to configure DingTalk notifications? (y/n)',
+      default: 'n',
+      validate: (input) => {
+        const normalized = input.toLowerCase().trim();
+        if (['y', 'n', 'yes', 'no', ''].includes(normalized)) {
+          return true;
+        }
+        return 'Please enter y or n';
+      },
+      filter: (input) => {
+        const normalized = input.toLowerCase().trim();
+        return normalized === 'y' || normalized === 'yes';
+      },
     },
   ]);
 
@@ -243,10 +254,21 @@ async function install() {
 
     const { confirmInstall } = await inquirer.prompt([
       {
-        type: 'confirm',
+        type: 'input',
         name: 'confirmInstall',
-        message: 'Proceed with installation?',
-        default: true,
+        message: 'Proceed with installation? (y/n)',
+        default: 'y',
+        validate: (input) => {
+          const normalized = input.toLowerCase().trim();
+          if (['y', 'n', 'yes', 'no', ''].includes(normalized)) {
+            return true;
+          }
+          return 'Please enter y or n';
+        },
+        filter: (input) => {
+          const normalized = input.toLowerCase().trim();
+          return normalized === '' || normalized === 'y' || normalized === 'yes';
+        },
       },
     ]);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sdd-skills",
-  "version": "1.1.7",
+  "version": "1.1.9",
   "description": "Spec-Driven Development Skills for Claude Code - SAE/ADE workflow automation",
   "type": "module",
   "main": "install.js",

package/skills/code-reviewer/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: code-reviewer
-description: 代码审查专家，负责代码质量、安全审查和规范检查。当测试通过需要代码审查时激活。检查代码风格、最佳实践、安全漏洞、性能问题，审查不通过时触发钉钉通知。
+description: 代码审查专家，负责代码质量、安全审查和规范检查。当代码实现完成需要代码审查时激活。检查代码风格、最佳实践、安全漏洞、性能问题，审查不通过时触发钉钉通知。
 ---
 
 # Code Reviewer
@@ -211,7 +211,7 @@ const HeavyComponent = defineAsyncComponent(() =>
 - [ ] 避免魔法数字，使用常量
 - [ ] 避免重复代码（DRY 原则）
 - [ ] 复杂逻辑有注释说明
-- [ ] 测试覆盖率达标（后端 >= 90%, 前端 >= 70%）
+- [ ] 测试覆盖率达标（后端 >= 90%, 前端 >= 90%）
 
 ```go
 // ❌ 可维护性问题：魔法数字
@@ -391,7 +391,7 @@ npm audit
 - [ ] Pinia store 设计合理
 - [ ] 类型定义完整
 - [ ] 复杂逻辑有注释
-- [ ] 测试覆盖率 >= 70%
+- [ ] 测试覆盖率 >= 90%
 ```
 
 ## 钉钉通知集成

package/skills/git-engineer/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: git-engineer
-description: Git工程师，负责代码提交、版本管理和MR创建。当代码审查通过需要提交代码时激活。执行预检测（lint/build/test）、规范化提交、创建GitLab MR，失败时触发钉钉通知。
+description: Git工程师，负责代码提交、版本管理和MR创建。当测试通过需要提交代码时激活。执行预检测（lint/build/test）、规范化提交、创建GitLab MR，失败时触发钉钉通知。
 ---
 
 # Git Engineer
@@ -56,7 +56,7 @@ npm run test:unit
 
 # 6. 检查覆盖率
 npm run test:coverage
-# 要求: >= 70%
+# 要求: >= 90%
 ```
 
 ✅ **预检测通过标准**：
@@ -202,7 +202,7 @@ glab mr create \
 
 ## 测试结果
 - 后端覆盖率：92% (>= 90%)
-- 前端覆盖率：75% (>= 70%)
+- 前端覆盖率：92% (>= 90%)
 - E2E 测试：3/3 通过
 
 ## 验收标准
@@ -244,7 +244,7 @@ glab mr create
 
 ## 测试结果
 - 后端覆盖率：XX% (>= 90%)
-- 前端覆盖率：XX% (>= 70%)
+- 前端覆盖率：XX% (>= 90%)
 - E2E 测试：X/X 通过
 
 ## 验收标准
@@ -475,7 +475,7 @@ glab mr status
 ✅ 类型检查：通过
 ✅ 编译检查：通过
 ✅ 单元测试：8/8 通过
-✅ 覆盖率：75% (>= 70%)
+✅ 覆盖率：92% (>= 90%)
 
 ✅ 所有预检测通过！
 

package/skills/sae/SKILL.md

@@ -185,7 +185,7 @@ Response (401):
 ## 决策原则
 
 遵循以下技术决策优先级：
-1. **稳定性 > 性能 > 安全性 > 成本**
+1. **安全性 > 稳定性 > 性能 > 成本**
 2. **技术选型标准**:
    - 成熟度高，社区活跃
    - 团队熟悉程度

package/uninstall.js

@@ -90,26 +90,48 @@ async function promptUninstallation() {
       choices: locationChoices,
     },
     {
-      type: 'confirm',
+      type: 'input',
       name: 'removeConfig',
-      message: 'Also remove DingTalk configuration?',
-      default: true,
+      message: 'Also remove DingTalk configuration? (y/n)',
+      default: 'y',
       when: (answers) => {
         const configPath = answers.location === 'global' ? GLOBAL_CONFIG : LOCAL_CONFIG;
         return existsSync(configPath);
       },
+      validate: (input) => {
+        const normalized = input.toLowerCase().trim();
+        if (['y', 'n', 'yes', 'no', ''].includes(normalized)) {
+          return true;
+        }
+        return 'Please enter y or n';
+      },
+      filter: (input) => {
+        const normalized = input.toLowerCase().trim();
+        return normalized === '' || normalized === 'y' || normalized === 'yes';
+      },
     },
     {
-      type: 'confirm',
+      type: 'input',
       name: 'confirm',
       message: (answers) => {
         const location = answers.location;
         const skillsCount = location === 'global'
           ? globalInstallation.skills.length
           : localInstallation.skills.length;
-        return `${chalk.red('Are you sure you want to remove')} ${chalk.bold.red(skillsCount)} ${chalk.red('Skills?')}`;
+        return `${chalk.red('Are you sure you want to remove')} ${chalk.bold.red(skillsCount)} ${chalk.red('Skills?')} (y/n)`;
+      },
+      default: 'n',
+      validate: (input) => {
+        const normalized = input.toLowerCase().trim();
+        if (['y', 'n', 'yes', 'no', ''].includes(normalized)) {
+          return true;
+        }
+        return 'Please enter y or n';
+      },
+      filter: (input) => {
+        const normalized = input.toLowerCase().trim();
+        return normalized === 'y' || normalized === 'yes';
       },
-      default: false,
     },
   ]);