musubi-sdd 5.1.0 → 5.6.1

package/src/agents/browser/action-executor.js

@@ -76,7 +76,7 @@ class ActionExecutor {
    */
   async executeNavigate(action, page, timeout) {
     await page.goto(action.url, { timeout, waitUntil: 'domcontentloaded' });
-    
+
     return {
       success: true,
       type: 'navigate',
@@ -93,7 +93,7 @@ class ActionExecutor {
    */
   async executeClick(action, page, timeout) {
     const selectors = action.selector.split(',').map(s => s.trim());
-    
+
     // Try each selector until one works
     for (const selector of selectors) {
       try {
@@ -110,7 +110,7 @@ class ActionExecutor {
 
     // If none worked, try with the original selector and let it fail
     await page.click(action.selector, { timeout });
-    
+
     return {
       success: true,
       type: 'click',
@@ -127,7 +127,7 @@ class ActionExecutor {
    */
   async executeFill(action, page, timeout) {
     const selectors = action.selector.split(',').map(s => s.trim());
-    
+
     // Try each selector until one works
     for (const selector of selectors) {
       try {
@@ -144,7 +144,7 @@ class ActionExecutor {
 
     // If none worked, try with the original selector
     await page.fill(action.selector, action.value, { timeout });
-    
+
     return {
       success: true,
       type: 'fill',
@@ -161,7 +161,7 @@ class ActionExecutor {
    */
   async executeSelect(action, page, timeout) {
     const selectors = action.selector.split(',').map(s => s.trim());
-    
+
     for (const selector of selectors) {
       try {
         await page.selectOption(selector, action.value, { timeout });
@@ -176,7 +176,7 @@ class ActionExecutor {
     }
 
     await page.selectOption(action.selector, action.value, { timeout });
-    
+
     return {
       success: true,
       type: 'select',
@@ -191,7 +191,7 @@ class ActionExecutor {
    */
   async executeWait(action) {
     await new Promise(resolve => setTimeout(resolve, action.delay));
-    
+
     return {
       success: true,
       type: 'wait',
@@ -207,12 +207,12 @@ class ActionExecutor {
    */
   async executeScreenshot(action, context) {
     const { page, screenshot } = context;
-    
+
     const path = await screenshot.capture(page, {
       name: action.name,
       fullPage: action.fullPage,
     });
-    
+
     return {
       success: true,
       type: 'screenshot',
@@ -229,9 +229,9 @@ class ActionExecutor {
    */
   async executeAssert(action, page, timeout) {
     const locator = page.locator(action.selector);
-    
+
     await locator.waitFor({ state: 'visible', timeout });
-    
+
     let text = null;
     if (action.expectedText) {
       text = await locator.textContent();
@@ -243,7 +243,7 @@ class ActionExecutor {
         };
       }
     }
-    
+
     return {
       success: true,
       type: 'assert',

package/src/agents/browser/ai-comparator.js

@@ -4,7 +4,7 @@
  */
 
 const fs = require('fs-extra');
-const path = require('path');
+const _path = require('path');
 
 /**
  * @typedef {Object} ComparisonResult
@@ -45,10 +45,10 @@ class AIComparator {
     const description = options.description || 'Compare visual appearance';
 
     // Check if files exist
-    if (!await fs.pathExists(expectedPath)) {
+    if (!(await fs.pathExists(expectedPath))) {
       throw new Error(`Expected screenshot not found: ${expectedPath}`);
     }
-    if (!await fs.pathExists(actualPath)) {
+    if (!(await fs.pathExists(actualPath))) {
       throw new Error(`Actual screenshot not found: ${actualPath}`);
     }
 
@@ -64,7 +64,7 @@ class AIComparator {
 
       // Call Vision API
       const result = await this.callVisionAPI(expectedBase64, actualBase64, description);
-      
+
       return {
         passed: result.similarity >= threshold * 100,
         similarity: result.similarity,
@@ -101,7 +101,7 @@ class AIComparator {
       method: 'POST',
       headers: {
         'Content-Type': 'application/json',
-        'Authorization': `Bearer ${this.apiKey}`,
+        Authorization: `Bearer ${this.apiKey}`,
       },
       body: JSON.stringify({
         model: this.model,
@@ -193,7 +193,7 @@ class AIComparator {
     // Calculate size difference
     const sizeDiff = Math.abs(expectedStat.size - actualStat.size);
     const maxSize = Math.max(expectedStat.size, actualStat.size);
-    const sizeRatio = 1 - (sizeDiff / maxSize);
+    const sizeRatio = 1 - sizeDiff / maxSize;
 
     // Simple heuristic: if sizes are similar, likely similar images
     // This is a rough approximation and should be replaced with actual image comparison
@@ -202,9 +202,10 @@ class AIComparator {
     return {
       passed: similarity >= threshold * 100,
       similarity,
-      differences: similarity < threshold * 100 
-        ? [`File size differs by ${sizeDiff} bytes (${((1 - sizeRatio) * 100).toFixed(1)}%)`]
-        : [],
+      differences:
+        similarity < threshold * 100
+          ? [`File size differs by ${sizeDiff} bytes (${((1 - sizeRatio) * 100).toFixed(1)}%)`]
+          : [],
       details: {
         method: 'fallback-size-comparison',
         expectedSize: expectedStat.size,
@@ -221,7 +222,7 @@ class AIComparator {
    * @param {Object} options
    * @returns {string}
    */
-  generateReport(result, options = {}) {
+  generateReport(result, _options = {}) {
     const lines = [
       '# Screenshot Comparison Report',
       '',

package/src/agents/browser/context-manager.js

@@ -66,14 +66,14 @@ class ContextManager {
    */
   async getOrCreatePage(contextName = 'default') {
     const pageKey = `${contextName}:main`;
-    
+
     if (this.pages.has(pageKey)) {
       return this.pages.get(pageKey);
     }
 
     const context = await this.getOrCreateContext(contextName);
     const page = await context.newPage();
-    
+
     this.pages.set(pageKey, page);
     return page;
   }
@@ -87,10 +87,10 @@ class ContextManager {
   async createPage(contextName = 'default', pageName) {
     const context = await this.getOrCreateContext(contextName);
     const page = await context.newPage();
-    
+
     const pageKey = `${contextName}:${pageName}`;
     this.pages.set(pageKey, page);
-    
+
     return page;
   }
 
@@ -162,9 +162,9 @@ class ContextManager {
     if (context) {
       await context.close();
       this.contexts.delete(name);
-      
+
       // Remove associated pages
-      for (const [key, page] of this.pages.entries()) {
+      for (const [key, _page] of this.pages.entries()) {
         if (key.startsWith(`${name}:`)) {
           this.pages.delete(key);
         }

package/src/agents/browser/index.js

@@ -102,7 +102,7 @@ class BrowserAgent {
 
     // Parse natural language to actions
     const parseResult = this.parser.parse(command);
-    
+
     if (!parseResult.success) {
       return {
         success: false,
@@ -122,9 +122,9 @@ class BrowserAgent {
         screenshot: this.screenshot,
         timeout: this.options.timeout,
       });
-      
+
       results.push(result);
-      
+
       if (!result.success) {
         return {
           success: false,
@@ -150,11 +150,11 @@ class BrowserAgent {
    */
   async executeSequence(commands) {
     const results = [];
-    
+
     for (const command of commands) {
       const result = await this.execute(command);
       results.push(result);
-      
+
       if (!result.success) {
         return {
           success: false,

package/src/agents/browser/nl-parser.js

@@ -26,36 +26,21 @@
  */
 const ACTION_PATTERNS = {
   navigate: {
-    patterns: [
-      /(?:に)?移動|開く|アクセス/,
-      /(?:go to|navigate|open|visit)/i,
-    ],
+    patterns: [/(?:に)?移動|開く|アクセス/, /(?:go to|navigate|open|visit)/i],
     urlPattern: /(https?:\/\/[^\s]+)/,
   },
   click: {
-    patterns: [
-      /クリック|押す|タップ|選択/,
-      /click|press|tap|select/i,
-    ],
+    patterns: [/クリック|押す|タップ|選択/, /click|press|tap|select/i],
   },
   fill: {
-    patterns: [
-      /(?:に|を)?入力|記入|タイプ/,
-      /fill|type|enter|input/i,
-    ],
+    patterns: [/(?:に|を)?入力|記入|タイプ/, /fill|type|enter|input/i],
     valuePattern: /[「「]([^」」]+)[」」]|"([^"]+)"|'([^']+)'/,
   },
   select: {
-    patterns: [
-      /ドロップダウン.*選択|選択.*オプション/,
-      /select.*dropdown|choose.*option/i,
-    ],
+    patterns: [/ドロップダウン.*選択|選択.*オプション/, /select.*dropdown|choose.*option/i],
   },
   wait: {
-    patterns: [
-      /秒?待つ|待機/,
-      /wait|pause|delay/i,
-    ],
+    patterns: [/秒?待つ|待機/, /wait|pause|delay/i],
     durationPattern: /(\d+)\s*秒|(\d+)\s*(?:seconds?|ms|milliseconds?)/i,
   },
   screenshot: {
@@ -66,10 +51,7 @@ const ACTION_PATTERNS = {
     namePattern: /[「「]([^」」]+)[」」]|"([^"]+)"|として\s*(\S+)/,
   },
   assert: {
-    patterns: [
-      /(?:が)?表示|確認|検証|存在/,
-      /(?:is )?visible|assert|verify|check|exists?/i,
-    ],
+    patterns: [/(?:が)?表示|確認|検証|存在/, /(?:is )?visible|assert|verify|check|exists?/i],
     textPattern: /[「「]([^」」]+)[」」]|"([^"]+)"/,
   },
 };
@@ -79,17 +61,19 @@ const ACTION_PATTERNS = {
  */
 const ELEMENT_PATTERNS = {
   // Japanese element names
-  'ログインボタン': 'button:has-text("ログイン"), [data-testid="login-button"], button[type="submit"]',
-  '送信ボタン': 'button:has-text("送信"), [data-testid="submit-button"], button[type="submit"]',
-  'メール': 'input[type="email"], input[name="email"], [data-testid="email-input"]',
-  'パスワード': 'input[type="password"], [data-testid="password-input"]',
-  '検索': 'input[type="search"], [data-testid="search-input"], input[name="q"]',
+  ログインボタン:
+    'button:has-text("ログイン"), [data-testid="login-button"], button[type="submit"]',
+  送信ボタン: 'button:has-text("送信"), [data-testid="submit-button"], button[type="submit"]',
+  メール: 'input[type="email"], input[name="email"], [data-testid="email-input"]',
+  パスワード: 'input[type="password"], [data-testid="password-input"]',
+  検索: 'input[type="search"], [data-testid="search-input"], input[name="q"]',
   // English element names
   'login button': 'button:has-text("Login"), [data-testid="login-button"], button[type="submit"]',
-  'submit button': 'button:has-text("Submit"), [data-testid="submit-button"], button[type="submit"]',
-  'email': 'input[type="email"], input[name="email"], [data-testid="email-input"]',
-  'password': 'input[type="password"], [data-testid="password-input"]',
-  'search': 'input[type="search"], [data-testid="search-input"], input[name="q"]',
+  'submit button':
+    'button:has-text("Submit"), [data-testid="submit-button"], button[type="submit"]',
+  email: 'input[type="email"], input[name="email"], [data-testid="email-input"]',
+  password: 'input[type="password"], [data-testid="password-input"]',
+  search: 'input[type="search"], [data-testid="search-input"], input[name="q"]',
 };
 
 /**
@@ -138,11 +122,7 @@ class NLParser {
    * @returns {string}
    */
   normalizeCommand(command) {
-    return command
-      .trim()
-      .replace(/\s+/g, ' ')
-      .replace(/、/g, ',')
-      .replace(/。/g, '.');
+    return command.trim().replace(/\s+/g, ' ').replace(/、/g, ',').replace(/。/g, '.');
   }
 
   /**
@@ -172,7 +152,10 @@ class NLParser {
   splitCommand(command) {
     // Split by conjunctions and separators
     const separators = /[,、]|\s+(?:そして|して|and|then)\s+/i;
-    return command.split(separators).map(s => s.trim()).filter(Boolean);
+    return command
+      .split(separators)
+      .map(s => s.trim())
+      .filter(Boolean);
   }
 
   /**
@@ -262,7 +245,7 @@ class NLParser {
   parseFill(text) {
     const selector = this.extractSelector(text);
     const valueMatch = text.match(this.actionPatterns.fill.valuePattern);
-    const value = valueMatch ? (valueMatch[1] || valueMatch[2] || valueMatch[3]) : '';
+    const value = valueMatch ? valueMatch[1] || valueMatch[2] || valueMatch[3] : '';
 
     return {
       type: 'fill',
@@ -280,7 +263,7 @@ class NLParser {
   parseSelect(text) {
     const selector = this.extractSelector(text);
     const valueMatch = text.match(this.actionPatterns.fill.valuePattern);
-    const value = valueMatch ? (valueMatch[1] || valueMatch[2] || valueMatch[3]) : '';
+    const value = valueMatch ? valueMatch[1] || valueMatch[2] || valueMatch[3] : '';
 
     return {
       type: 'select',
@@ -302,7 +285,7 @@ class NLParser {
     if (durationMatch) {
       const seconds = durationMatch[1] || durationMatch[2];
       delay = parseInt(seconds, 10) * 1000;
-      
+
       // Check if it's milliseconds
       if (/ms|milliseconds?/i.test(text)) {
         delay = parseInt(seconds, 10);
@@ -323,7 +306,7 @@ class NLParser {
    */
   parseScreenshot(text) {
     const nameMatch = text.match(this.actionPatterns.screenshot.namePattern);
-    const name = nameMatch ? (nameMatch[1] || nameMatch[2] || nameMatch[3]) : undefined;
+    const name = nameMatch ? nameMatch[1] || nameMatch[2] || nameMatch[3] : undefined;
     const fullPage = /全体|full\s*page/i.test(text);
 
     return {
@@ -341,7 +324,7 @@ class NLParser {
    */
   parseAssert(text) {
     const textMatch = text.match(this.actionPatterns.assert.textPattern);
-    const expectedText = textMatch ? (textMatch[1] || textMatch[2]) : null;
+    const expectedText = textMatch ? textMatch[1] || textMatch[2] : null;
     const selector = expectedText ? `text="${expectedText}"` : this.extractSelector(text);
 
     return {
@@ -366,7 +349,7 @@ class NLParser {
     }
 
     // Check for CSS selector in the text (match only the selector part)
-    const selectorMatch = text.match(/([#\.][a-zA-Z][\w\-]*|\[[^\]]+\])/);
+    const selectorMatch = text.match(/([#.][a-zA-Z][\w-]*|\[[^\]]+\])/);
     if (selectorMatch) {
       return selectorMatch[0];
     }
@@ -378,7 +361,9 @@ class NLParser {
     }
 
     // Try to extract element description
-    const elementMatch = text.match(/(?:の)?(?:ボタン|リンク|入力欄?|フィールド|テキスト|button|link|input|field|text)\s*[「「]?([^」」\s]*)[」」]?/i);
+    const elementMatch = text.match(
+      /(?:の)?(?:ボタン|リンク|入力欄?|フィールド|テキスト|button|link|input|field|text)\s*[「「]?([^」」\s]*)[」」]?/i
+    );
     if (elementMatch && elementMatch[1]) {
       return `text="${elementMatch[1]}"`;
     }

package/src/agents/browser/screenshot.js

@@ -43,7 +43,7 @@ class ScreenshotCapture {
 
     const timestamp = Date.now();
     this.counter++;
-    
+
     const name = options.name || `screenshot-${this.counter}`;
     const extension = options.type || 'png';
     const filename = `${timestamp}-${name}.${extension}`;
@@ -88,7 +88,7 @@ class ScreenshotCapture {
 
     const timestamp = Date.now();
     this.counter++;
-    
+
     const name = options.name || `element-${this.counter}`;
     const extension = options.type || 'png';
     const filename = `${timestamp}-${name}.${extension}`;

package/src/agents/browser/test-generator.js

@@ -94,9 +94,10 @@ class TestGenerator {
       case 'wait':
         return `await page.waitForTimeout(${action.delay});`;
 
-      case 'screenshot':
+      case 'screenshot': {
         const name = action.name || 'screenshot';
         return `await page.screenshot({ path: 'screenshots/${name}.png'${action.fullPage ? ', fullPage: true' : ''} });`;
+      }
 
       case 'assert':
         if (action.expectedText) {
@@ -173,9 +174,10 @@ class TestGenerator {
       case 'wait':
         return `await new Promise(r => setTimeout(r, ${action.delay}));`;
 
-      case 'screenshot':
+      case 'screenshot': {
         const name = action.name || 'screenshot';
         return `await page.screenshot({ path: 'screenshots/${name}.png'${action.fullPage ? ', fullPage: true' : ''} });`;
+      }
 
       case 'assert':
         return `await page.waitForSelector('${this.escapeSelector(action.selector)}');`;
@@ -191,7 +193,7 @@ class TestGenerator {
    * @param {Object} options
    * @returns {string}
    */
-  generateFromSpec(specification, options = {}) {
+  generateFromSpec(specification, _options = {}) {
     const lines = [
       `import { test, expect } from '@playwright/test';`,
       ``,
@@ -217,7 +219,7 @@ class TestGenerator {
    */
   requirementToTest(requirement) {
     const testName = `${requirement.id}: ${requirement.title || requirement.action || 'Requirement'}`;
-    
+
     const lines = [
       `test('${this.escapeString(testName)}', async ({ page }) => {`,
       `  // Pattern: ${requirement.pattern}`,