musubi-sdd 5.1.0 → 5.6.1

package/src/managers/skill-loader.js

@@ -1,8 +1,8 @@
 /**
  * MUSUBI Skill Loader
- * 
+ *
  * キーワードトリガー型スキルのロードと活性化
- * 
+ *
  * @module src/managers/skill-loader
  * @see REQ-P0-B002
  * @inspired-by OpenHands openhands/microagent/microagent.py
@@ -59,7 +59,7 @@ class Skill {
 
   /**
    * メッセージがこのスキルをトリガーするか判定
-   * @param {string} message 
+   * @param {string} message
    * @returns {boolean}
    */
   matchesTrigger(message) {
@@ -87,7 +87,7 @@ class Skill {
 
   /**
    * エージェントがこのスキルを使用可能か判定
-   * @param {string} agentType 
+   * @param {string} agentType
    * @returns {boolean}
    */
   isAvailableFor(agentType) {
@@ -126,7 +126,7 @@ class SkillLoader {
     this.globalDir = options.globalDir || path.join(__dirname, '../../steering/templates/skills');
     this.userDir = options.userDir || path.join(os.homedir(), '.musubi/skills');
     this.repoDir = options.repoDir || path.join(this.projectRoot, '.musubi/skills');
-    
+
     this.loadedSkills = new Map();
     this.initialized = false;
   }
@@ -153,9 +153,9 @@ class SkillLoader {
 
   /**
    * ディレクトリからスキルをロード
-   * @param {string} dir 
-   * @param {string} defaultType 
-   * @param {number} basePriority 
+   * @param {string} dir
+   * @param {string} defaultType
+   * @param {number} basePriority
    */
   async _loadFromDirectory(dir, defaultType, basePriority) {
     if (!fs.existsSync(dir)) {
@@ -183,9 +183,9 @@ class SkillLoader {
 
   /**
    * スキルファイルをパース
-   * @param {string} filePath 
-   * @param {string} defaultType 
-   * @param {number} basePriority 
+   * @param {string} filePath
+   * @param {string} defaultType
+   * @param {number} basePriority
    * @returns {Skill|null}
    */
   async parseSkill(filePath, defaultType = SkillType.GLOBAL, basePriority = 0) {
@@ -215,7 +215,7 @@ class SkillLoader {
 
   /**
    * Frontmatterをパース
-   * @param {string} content 
+   * @param {string} content
    * @returns {{ frontmatter: Object, body: string }}
    */
   _parseFrontmatter(content) {
@@ -280,7 +280,7 @@ class SkillLoader {
 
   /**
    * 値をパース
-   * @param {string} value 
+   * @param {string} value
    * @returns {any}
    */
   _parseValue(value) {
@@ -296,7 +296,10 @@ class SkillLoader {
     if (value.toLowerCase() === 'false') return false;
     // 配列（インライン）
     if (value.startsWith('[') && value.endsWith(']')) {
-      return value.slice(1, -1).split(',').map(v => v.trim().replace(/^["']|["']$/g, ''));
+      return value
+        .slice(1, -1)
+        .split(',')
+        .map(v => v.trim().replace(/^["']|["']$/g, ''));
     }
     // 文字列
     return value.replace(/^["']|["']$/g, '');
@@ -336,7 +339,7 @@ class SkillLoader {
 
   /**
    * 名前でスキルを取得
-   * @param {string} name 
+   * @param {string} name
    * @returns {Skill|undefined}
    */
   getSkill(name) {
@@ -353,7 +356,7 @@ class SkillLoader {
 
   /**
    * スキルタイプでフィルタリング
-   * @param {string} type 
+   * @param {string} type
    * @returns {Skill[]}
    */
   getSkillsByType(type) {
@@ -365,8 +368,7 @@ class SkillLoader {
    * @returns {boolean}
    */
   hasRepoSkills() {
-    return fs.existsSync(this.repoDir) && 
-           fs.readdirSync(this.repoDir).some(f => f.endsWith('.md'));
+    return fs.existsSync(this.repoDir) && fs.readdirSync(this.repoDir).some(f => f.endsWith('.md'));
   }
 
   /**
@@ -385,9 +387,10 @@ class SkillLoader {
 
     const sortedSkills = this.getSkills().sort((a, b) => b.priority - a.priority);
     for (const skill of sortedSkills) {
-      const triggers = skill.triggers.length > 3 
-        ? skill.triggers.slice(0, 3).join(', ') + '...'
-        : skill.triggers.join(', ');
+      const triggers =
+        skill.triggers.length > 3
+          ? skill.triggers.slice(0, 3).join(', ') + '...'
+          : skill.triggers.join(', ');
       md += `| ${skill.name} | ${skill.type} | ${skill.priority} | ${triggers} | ${skill.agent} |\n`;
     }
 
@@ -396,7 +399,7 @@ class SkillLoader {
 
   /**
    * 活性化されたスキルをプロンプトに変換
-   * @param {Skill[]} skills 
+   * @param {Skill[]} skills
    * @returns {string}
    */
   formatSkillsForPrompt(skills) {

package/src/managers/skill-tools.js

@@ -2,7 +2,7 @@
  * @fileoverview Skill Tools Manager
  * @description Manage allowed-tools configuration for MUSUBI skills
  * @version 3.11.0
- * 
+ *
  * Features:
  * - Load skill tool configurations from YAML
  * - Validate tool availability
@@ -22,19 +22,8 @@ const { EventEmitter } = require('events');
  * Default tool sets for common skill categories
  */
 const DEFAULT_TOOL_SETS = {
-  requirements: [
-    'file_read',
-    'file_write',
-    'search_files',
-    'read_resource'
-  ],
-  design: [
-    'file_read',
-    'file_write',
-    'search_files',
-    'create_directory',
-    'read_resource'
-  ],
+  requirements: ['file_read', 'file_write', 'search_files', 'read_resource'],
+  design: ['file_read', 'file_write', 'search_files', 'create_directory', 'read_resource'],
   implementation: [
     'file_read',
     'file_write',
@@ -42,48 +31,23 @@ const DEFAULT_TOOL_SETS = {
     'create_directory',
     'run_command',
     'read_resource',
-    'code_analysis'
-  ],
-  testing: [
-    'file_read',
-    'file_write',
-    'search_files',
-    'run_command',
-    'test_runner'
-  ],
-  validation: [
-    'file_read',
-    'search_files',
-    'read_resource',
-    'validate'
-  ],
-  documentation: [
-    'file_read',
-    'file_write',
-    'search_files',
-    'read_resource'
-  ],
-  analysis: [
-    'file_read',
-    'search_files',
     'code_analysis',
-    'read_resource'
   ],
-  deployment: [
-    'file_read',
-    'run_command',
-    'deploy'
-  ]
+  testing: ['file_read', 'file_write', 'search_files', 'run_command', 'test_runner'],
+  validation: ['file_read', 'search_files', 'read_resource', 'validate'],
+  documentation: ['file_read', 'file_write', 'search_files', 'read_resource'],
+  analysis: ['file_read', 'search_files', 'code_analysis', 'read_resource'],
+  deployment: ['file_read', 'run_command', 'deploy'],
 };
 
 /**
  * Tool restriction levels
  */
 const RestrictionLevel = {
-  NONE: 'none',           // No restrictions
-  STANDARD: 'standard',   // Default restrictions
-  STRICT: 'strict',       // Minimal tools only
-  CUSTOM: 'custom'        // Custom configuration
+  NONE: 'none', // No restrictions
+  STANDARD: 'standard', // Default restrictions
+  STRICT: 'strict', // Minimal tools only
+  CUSTOM: 'custom', // Custom configuration
 };
 
 /**
@@ -161,7 +125,7 @@ class SkillToolConfig {
       deniedTools: this.deniedTools,
       restrictionLevel: this.restrictionLevel,
       toolOverrides: this.toolOverrides,
-      inheritFrom: this.inheritFrom
+      inheritFrom: this.inheritFrom,
     };
   }
 }
@@ -176,7 +140,7 @@ class SkillToolsManager extends EventEmitter {
       configDir: options.configDir || '.musubi/tools',
       defaultToolSet: options.defaultToolSet || 'standard',
       enableInheritance: options.enableInheritance !== false,
-      ...options
+      ...options,
     };
 
     this.skillConfigs = new Map();
@@ -206,7 +170,6 @@ class SkillToolsManager extends EventEmitter {
       await this._processConfig(config);
       this.emit('configLoaded', configPath);
       return config;
-
     } catch (error) {
       if (error.code === 'ENOENT') {
         return null;
@@ -266,7 +229,7 @@ class SkillToolsManager extends EventEmitter {
    */
   setSkillConfig(skillName, config) {
     const skillConfig = new SkillToolConfig(skillName, config);
-    
+
     // Handle inheritance
     if (this.options.enableInheritance && skillConfig.inheritFrom) {
       const parentConfig = this.skillConfigs.get(skillConfig.inheritFrom);
@@ -302,7 +265,7 @@ class SkillToolsManager extends EventEmitter {
    */
   getAllowedTools(skillName, options = {}) {
     const config = this.skillConfigs.get(skillName);
-    
+
     if (!config) {
       // Return default tools based on skill category
       return this._getDefaultToolsForSkill(skillName);
@@ -324,7 +287,7 @@ class SkillToolsManager extends EventEmitter {
    */
   _getDefaultToolsForSkill(skillName) {
     const lowerName = skillName.toLowerCase();
-    
+
     for (const [category, tools] of Object.entries(DEFAULT_TOOL_SETS)) {
       if (lowerName.includes(category)) {
         return [...tools];
@@ -366,7 +329,7 @@ class SkillToolsManager extends EventEmitter {
       valid: missing.length === 0,
       available,
       missing,
-      coverage: available.length / allowedTools.length
+      coverage: available.length / allowedTools.length,
     };
   }
 
@@ -385,22 +348,22 @@ class SkillToolsManager extends EventEmitter {
 
     if (context.readOnly) {
       // Remove write operations
-      optimizedTools = optimizedTools.filter(t => 
-        !t.includes('write') && !t.includes('create') && !t.includes('delete')
+      optimizedTools = optimizedTools.filter(
+        t => !t.includes('write') && !t.includes('create') && !t.includes('delete')
       );
     }
 
     if (context.noNetwork) {
       // Remove network operations
-      optimizedTools = optimizedTools.filter(t => 
-        !t.includes('http') && !t.includes('api') && !t.includes('fetch')
+      optimizedTools = optimizedTools.filter(
+        t => !t.includes('http') && !t.includes('api') && !t.includes('fetch')
       );
     }
 
     if (context.minimalPermissions) {
       // Keep only essential tools
-      optimizedTools = optimizedTools.filter(t => 
-        t.includes('read') || t.includes('search') || t.includes('list')
+      optimizedTools = optimizedTools.filter(
+        t => t.includes('read') || t.includes('search') || t.includes('list')
       );
     }
 
@@ -408,7 +371,7 @@ class SkillToolsManager extends EventEmitter {
       skillName,
       allowedTools: optimizedTools,
       restrictionLevel: config?.restrictionLevel || RestrictionLevel.STANDARD,
-      context
+      context,
     };
   }
 
@@ -422,19 +385,19 @@ class SkillToolsManager extends EventEmitter {
 
     for (const skill of skills) {
       const skillName = skill.name || skill.id;
-      
+
       // Determine category from skill metadata
       const category = this._detectSkillCategory(skill);
       const defaultTools = DEFAULT_TOOL_SETS[category] || DEFAULT_TOOL_SETS.validation;
 
       // Merge with explicitly defined tools
-      const allowedTools = skill.allowedTools 
+      const allowedTools = skill.allowedTools
         ? [...new Set([...defaultTools, ...skill.allowedTools])]
         : defaultTools;
 
       const config = this.setSkillConfig(skillName, {
         allowedTools,
-        restrictionLevel: skill.restrictionLevel || RestrictionLevel.STANDARD
+        restrictionLevel: skill.restrictionLevel || RestrictionLevel.STANDARD,
       });
 
       configs.set(skillName, config);
@@ -448,7 +411,8 @@ class SkillToolsManager extends EventEmitter {
    * @private
    */
   _detectSkillCategory(skill) {
-    const text = `${skill.name || ''} ${skill.description || ''} ${skill.purpose || ''}`.toLowerCase();
+    const text =
+      `${skill.name || ''} ${skill.description || ''} ${skill.purpose || ''}`.toLowerCase();
 
     const categoryKeywords = {
       requirements: ['requirement', 'ears', 'spec', 'feature'],
@@ -458,7 +422,7 @@ class SkillToolsManager extends EventEmitter {
       validation: ['validate', 'check', 'verify', 'lint'],
       documentation: ['document', 'readme', 'guide', 'doc'],
       analysis: ['analyze', 'audit', 'review', 'inspect'],
-      deployment: ['deploy', 'release', 'publish', 'ci']
+      deployment: ['deploy', 'release', 'publish', 'ci'],
     };
 
     for (const [category, keywords] of Object.entries(categoryKeywords)) {
@@ -491,10 +455,10 @@ class SkillToolsManager extends EventEmitter {
       version: '1.0.0',
       exportedAt: new Date().toISOString(),
       defaults: {
-        toolSet: this.options.defaultToolSet
+        toolSet: this.options.defaultToolSet,
       },
       toolDependencies,
-      skills
+      skills,
     };
   }
 
@@ -535,7 +499,7 @@ class SkillToolsManager extends EventEmitter {
    */
   getStats() {
     const skills = Array.from(this.skillConfigs.values());
-    
+
     const byRestriction = {};
     for (const level of Object.values(RestrictionLevel)) {
       byRestriction[level] = skills.filter(s => s.restrictionLevel === level).length;
@@ -552,7 +516,7 @@ class SkillToolsManager extends EventEmitter {
       totalSkills: this.skillConfigs.size,
       totalUniqueTools: allTools.size,
       availableTools: this.availableTools.size,
-      byRestrictionLevel: byRestriction
+      byRestrictionLevel: byRestriction,
     };
   }
 }
@@ -561,5 +525,5 @@ module.exports = {
   SkillToolsManager,
   SkillToolConfig,
   RestrictionLevel,
-  DEFAULT_TOOL_SETS
+  DEFAULT_TOOL_SETS,
 };

package/src/managers/workflow.js

@@ -1,6 +1,6 @@
 /**
  * Workflow Engine for MUSUBI SDD
- * 
+ *
  * Manages workflow state, stage transitions, and metrics collection.
  */
 
@@ -22,7 +22,7 @@ const WORKFLOW_STAGES = {
   testing: { next: ['deployment', 'implementation', 'requirements'] }, // Feedback loops
   deployment: { next: ['monitoring'] },
   monitoring: { next: ['retrospective'] },
-  retrospective: { next: ['requirements'] } // New iteration
+  retrospective: { next: ['requirements'] }, // New iteration
 };
 
 /**
@@ -51,20 +51,20 @@ class WorkflowEngine {
       currentStage: options.startStage || 'requirements',
       startedAt: new Date().toISOString(),
       stages: {},
-      history: []
+      history: [],
     };
 
     // Record initial stage
     state.stages[state.currentStage] = {
       enteredAt: new Date().toISOString(),
-      status: 'in-progress'
+      status: 'in-progress',
     };
 
     state.history.push({
       timestamp: new Date().toISOString(),
       action: 'workflow-started',
       stage: state.currentStage,
-      feature: featureName
+      feature: featureName,
     });
 
     await this.saveState(state);
@@ -77,7 +77,7 @@ class WorkflowEngine {
    * Get current workflow state
    */
   async getState() {
-    if (!await fs.pathExists(this.stateFile)) {
+    if (!(await fs.pathExists(this.stateFile))) {
       return null;
     }
     const content = await fs.readFile(this.stateFile, 'utf8');
@@ -107,7 +107,7 @@ class WorkflowEngine {
     if (!validTransitions.includes(targetStage)) {
       throw new Error(
         `Invalid transition: ${currentStage} → ${targetStage}. ` +
-        `Valid transitions: ${validTransitions.join(', ')}`
+          `Valid transitions: ${validTransitions.join(', ')}`
       );
     }
 
@@ -135,14 +135,14 @@ class WorkflowEngine {
       action: 'stage-transition',
       from: currentStage,
       to: targetStage,
-      notes
+      notes,
     });
 
     await this.saveState(state);
     await this.recordMetric('stage_transition', {
       from: currentStage,
       to: targetStage,
-      feature: state.feature
+      feature: state.feature,
     });
 
     return state;
@@ -160,7 +160,7 @@ class WorkflowEngine {
       action: 'feedback-loop',
       from: fromStage,
       to: toStage,
-      reason
+      reason,
     });
 
     await this.saveState(state);
@@ -168,7 +168,7 @@ class WorkflowEngine {
       from: fromStage,
       to: toStage,
       reason,
-      feature: state.feature
+      feature: state.feature,
     });
   }
 
@@ -194,14 +194,14 @@ class WorkflowEngine {
     state.history.push({
       timestamp: new Date().toISOString(),
       action: 'workflow-completed',
-      notes
+      notes,
     });
 
     await this.saveState(state);
     await this.recordMetric('workflow_completed', {
       feature: state.feature,
       totalDuration: state.totalDuration,
-      stageCount: Object.keys(state.stages).length
+      stageCount: Object.keys(state.stages).length,
     });
 
     // Generate summary
@@ -221,7 +221,7 @@ class WorkflowEngine {
     metrics.push({
       timestamp: new Date().toISOString(),
       name,
-      data
+      data,
     });
 
     await fs.ensureDir(path.dirname(this.metricsFile));
@@ -232,7 +232,7 @@ class WorkflowEngine {
    * Get workflow metrics summary
    */
   async getMetricsSummary() {
-    if (!await fs.pathExists(this.metricsFile)) {
+    if (!(await fs.pathExists(this.metricsFile))) {
       return { message: 'No metrics recorded yet.' };
     }
 
@@ -245,7 +245,7 @@ class WorkflowEngine {
       feedbackLoops: 0,
       stageTransitions: 0,
       averageDuration: null,
-      stageStats: {}
+      stageStats: {},
     };
 
     const durations = [];
@@ -289,7 +289,7 @@ class WorkflowEngine {
     const stages = Object.entries(state.stages).map(([name, data]) => ({
       name,
       duration: data.duration || 'N/A',
-      attempts: data.attempts || 1
+      attempts: data.attempts || 1,
     }));
 
     const feedbackLoops = state.history.filter(h => h.action === 'feedback-loop');
@@ -299,7 +299,7 @@ class WorkflowEngine {
       totalDuration: state.totalDuration,
       stages,
       feedbackLoops: feedbackLoops.length,
-      feedbackDetails: feedbackLoops
+      feedbackDetails: feedbackLoops,
     };
   }
 
@@ -338,10 +338,18 @@ class WorkflowEngine {
       const value = parseInt(part);
       const unit = part.slice(-1);
       switch (unit) {
-        case 'd': ms += value * 24 * 60 * 60 * 1000; break;
-        case 'h': ms += value * 60 * 60 * 1000; break;
-        case 'm': ms += value * 60 * 1000; break;
-        case 's': ms += value * 1000; break;
+        case 'd':
+          ms += value * 24 * 60 * 60 * 1000;
+          break;
+        case 'h':
+          ms += value * 60 * 60 * 1000;
+          break;
+        case 'm':
+          ms += value * 60 * 1000;
+          break;
+        case 's':
+          ms += value * 1000;
+          break;
       }
     });
     return ms;