@zhuan-ai/zhuanspec 2.9.5 → 2.11.2

package/dist/cli/hooks.js

@@ -240,7 +240,7 @@ program
             codeReview: codeReviewResult,
             unitTest: unitTestResult,
             specConsistency: specConsistencyResult,
-            reportPath: `zhuanspec/changes/${changeId}/review-report.md`,
+            reportPath: `zhuanspec/changes/${changeId}/review/review-report.md`,
         }));
     }
     else {
@@ -355,10 +355,10 @@ async function runPhaseTransitionCheck(options) {
             };
         case 'archive':
             // Review → Archive: Check for review completion
-            const metricsDir = path.join(changeDir, 'metrics');
-            const reviewPath = path.join(metricsDir, 'review.json');
-            const testsPath = path.join(metricsDir, 'tests.json');
-            const reportPath = path.join(changeDir, 'review-report.md');
+            const reviewDir = path.join(changeDir, 'review');
+            const reviewPath = path.join(reviewDir, 'code-review-result.json');
+            const testsPath = path.join(reviewDir, 'unit-test-result.json');
+            const reportPath = path.join(reviewDir, 'review-report.md');
             const checks = {
                 reviewCompleted: false,
                 testsCompleted: false,
@@ -370,11 +370,11 @@ async function runPhaseTransitionCheck(options) {
             if (!checks.reviewCompleted || !checks.testsCompleted || !checks.reportExists) {
                 const missing = [];
                 if (!checks.reviewCompleted)
-                    missing.push('review.json');
+                    missing.push('review/code-review-result.json');
                 if (!checks.testsCompleted)
-                    missing.push('tests.json');
+                    missing.push('review/unit-test-result.json');
                 if (!checks.reportExists)
-                    missing.push('review-report.md');
+                    missing.push('review/review-report.md');
                 return {
                     continue: false,
                     systemMessage: `✗ Phase transition blocked: ${fromPhase} → ${targetPhase}\n\nMissing review artifacts:\n${missing.map(m => `- ${m}`).join('\n')}\n\nPlease complete Review phase before archiving.`,

package/dist/cli/index.js

@@ -220,6 +220,7 @@ program
     .command('review [change-name]')
     .description('Run ZhuanSpec review and unit test verification')
     .option('--json', 'Output review report as JSON')
+    .option('--force', 'Skip task completion check and force review')
     .option('--test-command <command>', 'Test command to execute (default: npm test)')
     .option('--coverage-threshold <n>', 'Coverage threshold percentage (default: 80)')
     .action(async (changeName, options) => {

package/dist/commands/design.d.ts

@@ -18,13 +18,22 @@ export declare class DesignCommand {
     private bindDesignSourceToChange;
     private selectChangeInteractively;
     /**
-     * Validate design.md for required sections and optionally auto-fix
+     * Validate design.md for required sections and optionally auto-fix.
+     * Uses dynamic section derivation: always-required sections are checked for all proposals,
+     * while conditional sections are only required when techDesign/tech-spec.md content matches.
      */
     private validateDesign;
     /**
      * Auto-fix design content by adding missing section skeletons
      */
     private autoFixDesign;
+    /**
+     * Derive required design.md sections dynamically.
+     * Always-required: 背景/Background, 决策/Decision, 风险/Risks.
+     * Conditional sections are added based on techDesign/tech-spec.md content.
+     * When no techDesign directory exists, only always-required sections are enforced (backward compat).
+     */
+    private getRequiredSections;
     /**
      * Generate a kebab-case change name from design context
      */
@@ -32,7 +41,8 @@ export declare class DesignCommand {
     /**
      * Create a new change directory for techDesign phase
      * - Creates change directory using createChange()
-     * - Creates .tech-design marker file
+     * - Creates .tech-design marker file (backward compat)
+     * - Creates techDesign/ directory with tech-spec.md placeholder
      * - Initializes progress.json with phase=techDesign
      * - Creates design.md skeleton
      * - Creates doc/ request file

package/dist/commands/design.js

@@ -14,6 +14,25 @@ const DEFAULT_DESIGN_TEMPLATE = `## 背景
 - Decision: [内容和原因]
 - Alternatives considered: [选项 + 理由]
 
+## 接口设计
+<!-- 接口签名、入参出参字段类型、枚举值、SCF 方法名 -->
+
+## 数据模型
+<!-- DB schema 变更、实体类字段、索引设计 -->
+
+## 业务流程
+<!-- Mermaid 流程图或时序图 -->
+\`\`\`mermaid
+flowchart TD
+  A[开始] --> B[...]
+\`\`\`
+
+## 实现细节
+<!-- 分层实现说明（Component→Application→Domain→Assemble→DAO）、关键逻辑、复用点 -->
+
+## 边界条件 / 异常处理
+<!-- 参数校验规则、错误码、降级策略 -->
+
 ## 风险 / 权衡
 - [风险] -> 缓解措施
 
@@ -190,7 +209,9 @@ export class DesignCommand {
         }
     }
     /**
-     * Validate design.md for required sections and optionally auto-fix
+     * Validate design.md for required sections and optionally auto-fix.
+     * Uses dynamic section derivation: always-required sections are checked for all proposals,
+     * while conditional sections are only required when techDesign/tech-spec.md content matches.
      */
     async validateDesign(changeName, autoFix) {
         const changeDir = path.join(process.cwd(), 'zhuanspec', 'changes', changeName);
@@ -200,13 +221,8 @@ export class DesignCommand {
         }
         const content = await FileSystemUtils.readFile(designPath);
         const issues = [];
-        // Check required sections: 背景/目标/决策/风险
-        const requiredSections = [
-            { pattern: /^##\s+(背景|Background)/im, name: '背景/Background', skeleton: '## 背景\n\n[请补充背景、约束、利益相关者]\n' },
-            { pattern: /^##\s+(目标|Goals|非目标|Non-Goals)/im, name: '目标/Goals', skeleton: '## 目标 / 非目标\n\n- Goals: [...]\n- Non-Goals: [...]\n' },
-            { pattern: /^##\s+(决策|Decision)/im, name: '决策/Decision', skeleton: '## 决策\n\n- Decision: [内容和原因]\n- Alternatives considered: [选项 + 理由]\n' },
-            { pattern: /^##\s+(风险|权衡|Risks)/im, name: '风险/Risks', skeleton: '## 风险 / 权衡\n\n- [风险] -> 缓解措施\n' },
-        ];
+        // Dynamically derive required sections based on techDesign content
+        const requiredSections = await this.getRequiredSections(changeDir);
         for (const section of requiredSections) {
             if (!section.pattern.test(content)) {
                 issues.push({
@@ -270,6 +286,58 @@ export class DesignCommand {
         }
         return result;
     }
+    /**
+     * Derive required design.md sections dynamically.
+     * Always-required: 背景/Background, 决策/Decision, 风险/Risks.
+     * Conditional sections are added based on techDesign/tech-spec.md content.
+     * When no techDesign directory exists, only always-required sections are enforced (backward compat).
+     */
+    async getRequiredSections(changePath) {
+        // Always-required sections for every proposal
+        const sections = [
+            { pattern: /^##\s+(背景|Background)/im, name: '背景/Background', skeleton: '## 背景\n\n[请补充背景、约束、利益相关者]\n' },
+            { pattern: /^##\s+(决策|Decision)/im, name: '决策/Decision', skeleton: '## 决策\n\n- Decision: [内容和原因]\n- Alternatives considered: [选项 + 理由]\n' },
+            { pattern: /^##\s+(风险|权衡|Risks)/im, name: '风险/Risks', skeleton: '## 风险 / 权衡\n\n- [风险] -> 缓解措施\n' },
+        ];
+        // Conditionally add sections based on tech-spec.md content
+        const techSpecPath = path.join(changePath, 'techDesign', 'tech-spec.md');
+        if (await FileSystemUtils.fileExists(techSpecPath)) {
+            const techContent = await fs.readFile(techSpecPath, 'utf-8');
+            // Tech spec has DDL / table creation → require Database Design
+            if (/CREATE\s+TABLE|DDL|建表|ALTER\s+TABLE/i.test(techContent)) {
+                sections.push({
+                    pattern: /^##\s+(数据模型|数据库设计|Database\s*Design)/im,
+                    name: '数据模型/Database Design',
+                    skeleton: '## 数据模型\n\n<!-- DB schema 变更、实体类字段、索引设计 -->\n',
+                });
+            }
+            // Tech spec has API / interface definitions → require API Contract
+            if (/接口|API|SCF|RPC|Controller|@RequestMapping/i.test(techContent)) {
+                sections.push({
+                    pattern: /^##\s+(接口设计|接口契约|API\s*Contract|Interface)/im,
+                    name: '接口设计/API Contract',
+                    skeleton: '## 接口设计\n\n<!-- 接口签名、入参出参字段类型、枚举值、SCF 方法名 -->\n',
+                });
+            }
+            // Tech spec has ES / MQ / Cache → require Infrastructure
+            if (/ElasticSearch|ES.*索引|mapping|MQ|RocketMQ|Consumer|Redis|缓存|Cache/i.test(techContent)) {
+                sections.push({
+                    pattern: /^##\s+(基础设施|Infrastructure)/im,
+                    name: '基础设施/Infrastructure',
+                    skeleton: '## 基础设施\n\n<!-- ES / MQ / Cache 等基础设施设计 -->\n',
+                });
+            }
+            // Tech spec has frontend design → require Frontend Design
+            if (/前端|页面|组件|路由|Frontend|Component|React|Vue/i.test(techContent)) {
+                sections.push({
+                    pattern: /^##\s+(前端设计|Frontend\s*Design)/im,
+                    name: '前端设计/Frontend Design',
+                    skeleton: '## 前端设计\n\n<!-- 页面布局、组件结构、路由设计 -->\n',
+                });
+            }
+        }
+        return sections;
+    }
     /**
      * Generate a kebab-case change name from design context
      */
@@ -307,7 +375,8 @@ export class DesignCommand {
     /**
      * Create a new change directory for techDesign phase
      * - Creates change directory using createChange()
-     * - Creates .tech-design marker file
+     * - Creates .tech-design marker file (backward compat)
+     * - Creates techDesign/ directory with tech-spec.md placeholder
      * - Initializes progress.json with phase=techDesign
      * - Creates design.md skeleton
      * - Creates doc/ request file
@@ -319,7 +388,11 @@ export class DesignCommand {
         const changeDir = path.join(projectRoot, 'zhuanspec', 'changes', changeName);
         // 2. Initialize progress.json with phase=techDesign
         await initializeProgress(changeName, 'techDesign');
-        // 3. Create .tech-design marker file
+        // 3. Create techDesign/ directory and tech-spec.md placeholder
+        const techDesignDir = path.join(changeDir, 'techDesign');
+        await FileSystemUtils.createDirectory(techDesignDir);
+        await FileSystemUtils.writeFile(path.join(techDesignDir, 'tech-spec.md'), `# Tech Spec\n\ncreated: ${new Date().toISOString()}\nsource: ${context.desc || context.dashenPageId || 'manual'}\n\n> This file will be populated by the techDesign skill.\n`);
+        // 3b. Also create .tech-design marker for backward compatibility
         await FileSystemUtils.writeFile(path.join(changeDir, '.tech-design'), `created: ${new Date().toISOString()}\nsource: ${context.desc || context.dashenPageId || 'manual'}\n`);
         // 4. Create design.md skeleton
         await FileSystemUtils.writeFile(path.join(changeDir, 'design.md'), DEFAULT_DESIGN_TEMPLATE);

package/dist/commands/progress.js

@@ -312,6 +312,17 @@ export class ProgressCommand {
         else if (completedCount === totalTasks && totalTasks > 0) {
             console.log('✅ 全部任务完成，准备进入 Review 阶段');
         }
+        // Proposal changes history
+        if (progress?.proposalChanges && progress.proposalChanges.length > 0) {
+            console.log('');
+            console.log('📝 提案修改历史:');
+            for (const change of progress.proposalChanges) {
+                console.log(`  [${change.timestamp}] ${change.triggeredBy} (${change.phase})${change.relatedDeviationId ? ` ← ${change.relatedDeviationId}` : ''}`);
+                for (const file of change.modifiedFiles) {
+                    console.log(`    ${file.changeType}: ${file.filePath} - ${file.summary}`);
+                }
+            }
+        }
     }
     generateProgressBar(percentage, blocks) {
         const filled = Math.round((percentage / 100) * blocks);

package/dist/commands/review.d.ts

@@ -1,32 +1,8 @@
 export declare class ReviewCommand {
     execute(changeName?: string, options?: {
         json?: boolean;
-        testCommand?: string;
-        coverageThreshold?: string;
+        force?: boolean;
     }): Promise<void>;
     private selectChangeInteractively;
-    private collectRequirementNames;
-    private collectTouchedFileContent;
-    private runTests;
-    private buildSummary;
-    private buildSpecConsistency;
-    private printSummary;
-    /**
-     * Fetch Sonar issues using MCP
-     * Uses the first detected repository's branch for SonarQube queries
-     */
-    private fetchSonarIssues;
-    /**
-     * Collect all Scenarios from delta specs
-     */
-    private collectScenarios;
-    /**
-     * Find test files in the project
-     */
-    private findTestFiles;
-    /**
-     * Convert Scenario name to possible test method names
-     */
-    private scenarioToTestMethodNames;
 }
 //# sourceMappingURL=review.d.ts.map