xibecode 0.9.1 → 0.9.3

package/dist/core/tools.js

@@ -9,11 +9,9 @@ import { GitUtils } from '../utils/git.js';
 import { TestRunnerDetector } from '../utils/testRunner.js';
 import { SafetyChecker, sanitizePath, sanitizeUrl } from '../utils/safety.js';
 import { PluginManager } from './plugins.js';
-import { BrowserManager } from '../tools/browser.js';
 import * as os from 'os';
 import { SkillManager } from './skills.js';
 import { TestGenerator, writeTestFile } from '../tools/test-generator.js';
-import { VisualFeedbackProvider } from './visual-feedback.js';
 import { PatternMiner } from './pattern-miner.js';
 import { BackgroundAgentManager } from './background-agent.js';
 import { CodeGraph } from './code-graph.js';
@@ -21,6 +19,8 @@ import { ConflictSolver } from './conflict-solver.js';
 import { SwarmOrchestrator } from './swarm.js';
 import { PermissionManager } from './permissions.js';
 const execAsync = promisify(exec);
+/** Returned by former Playwright-backed browser tools; XibeCode does not bundle browsers. */
+export const NO_EMBEDDED_BROWSER_MESSAGE = 'XibeCode does not bundle Playwright or download Chromium. Use run_command with agent-browser (e.g. agent-browser open <url>, agent-browser screenshot out.png) or your environment browser MCP. For Playwright E2E in a repo, add @playwright/test there and run it via run_command.';
 /**
  * Main tool executor for XibeCode agent
  *
@@ -32,7 +32,7 @@ const execAsync = promisify(exec);
  * - Context Operations: code search, file finding, context discovery
  * - Test Operations: run tests, get results
  * - Memory Operations: update neural memory
- * - Browser Operations: web automation with Playwright
+ * - Browser guidance: use run_command + agent-browser (no bundled browser)
  *
  * Features:
  * - Mode-based tool permissions
@@ -76,8 +76,6 @@ export class CodingToolExecutor {
     pluginManager;
     mcpClientManager;
     memory;
-    browserManager;
-    visualFeedback;
     skillManager;
     patternMiner;
     backgroundAgent;
@@ -131,8 +129,6 @@ export class CodingToolExecutor {
         this.pluginManager = options?.pluginManager || new PluginManager();
         this.mcpClientManager = options?.mcpClientManager;
         this.memory = options?.memory;
-        this.browserManager = new BrowserManager();
-        this.visualFeedback = new VisualFeedbackProvider(workingDir);
         this.patternMiner = new PatternMiner(workingDir);
         this.backgroundAgent = new BackgroundAgentManager(workingDir);
         this.codeGraph = new CodeGraph(workingDir);
@@ -536,56 +532,15 @@ export class CodingToolExecutor {
                     await this.memory.addMemory(p.trigger, p.action, p.outcome, p.tags || []);
                     return { success: true, message: 'Lesson learned and saved to neural memory.' };
                 }
-                case 'take_screenshot': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    if (!p.path || typeof p.path !== 'string')
-                        return { error: true, success: false, message: 'Missing path' };
-                    return this.browserManager.takeScreenshot(p.url, p.path, p.fullPage !== false);
-                }
-                case 'get_console_logs': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    return this.browserManager.getConsoleLogs(p.url);
-                }
-                case 'run_visual_test': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    if (!p.baseline_path || typeof p.baseline_path !== 'string')
-                        return { error: true, success: false, message: 'Missing baseline_path' };
-                    const outputDir = p.output_dir || '.playwright-baselines';
-                    return this.browserManager.runVisualTest(p.url, this.resolvePath(p.baseline_path), this.resolvePath(outputDir));
-                }
-                case 'check_accessibility': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    return this.browserManager.checkAccessibility(p.url);
-                }
-                case 'measure_performance': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    return this.browserManager.measurePerformance(p.url);
-                }
-                case 'test_responsive': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    const outputDir = p.output_dir || '.responsive-screenshots';
-                    return this.browserManager.testResponsive(p.url, this.resolvePath(outputDir), p.viewports);
-                }
-                case 'capture_network': {
-                    if (!p.url || typeof p.url !== 'string')
-                        return { error: true, success: false, message: 'Missing url' };
-                    return this.browserManager.captureNetworkRequests(p.url);
-                }
-                case 'run_playwright_test': {
-                    if (!p.test_path || typeof p.test_path !== 'string')
-                        return { error: true, success: false, message: 'Missing test_path' };
-                    return this.browserManager.runPlaywrightTest(this.resolvePath(p.test_path), {
-                        headed: p.headed,
-                        browser: p.browser,
-                        timeout: p.timeout,
-                    });
-                }
+                case 'take_screenshot':
+                case 'get_console_logs':
+                case 'run_visual_test':
+                case 'check_accessibility':
+                case 'measure_performance':
+                case 'test_responsive':
+                case 'capture_network':
+                case 'preview_app':
+                    return { error: true, success: false, message: NO_EMBEDDED_BROWSER_MESSAGE };
                 case 'search_skills_sh': {
                     if (!p.query || typeof p.query !== 'string') {
                         return { error: true, success: false, message: 'Missing required parameter: query (string)' };
@@ -611,12 +566,6 @@ export class CodingToolExecutor {
                         maxTestsPerFunction: p.max_tests_per_function,
                     }, p.write_file);
                 }
-                case 'preview_app': {
-                    if (!p.url || typeof p.url !== 'string') {
-                        return { error: true, success: false, message: 'Missing required parameter: url (string)' };
-                    }
-                    return this.visualFeedback.capture(p.url, { fullPage: p.full_page });
-                }
                 case 'mine_project_patterns': {
                     const patterns = await this.patternMiner.mine();
                     if (patterns.length === 0) {
@@ -725,6 +674,51 @@ export class CodingToolExecutor {
                         worker: p.worker_type
                     };
                 }
+                case 'run_swarm': {
+                    const subtasks = p.subtasks;
+                    if (!Array.isArray(subtasks) || subtasks.length === 0) {
+                        return { error: true, success: false, message: 'run_swarm requires a non-empty subtasks array' };
+                    }
+                    const normalized = [];
+                    for (let i = 0; i < subtasks.length; i++) {
+                        const st = subtasks[i];
+                        if (!st || typeof st !== 'object') {
+                            return { error: true, success: false, message: `Invalid subtasks[${i}]: expected object` };
+                        }
+                        const task = st.task;
+                        const worker_type = st.worker_type;
+                        if (typeof task !== 'string' || !task.trim()) {
+                            return { error: true, success: false, message: `subtasks[${i}].task must be a non-empty string` };
+                        }
+                        if (typeof worker_type !== 'string' || !isValidMode(worker_type)) {
+                            return {
+                                error: true,
+                                success: false,
+                                message: `subtasks[${i}].worker_type must be a valid AgentMode (got ${String(worker_type)})`,
+                            };
+                        }
+                        normalized.push({ mode: worker_type, task: task.trim() });
+                    }
+                    const timeoutMs = typeof p.timeout_ms === 'number' && Number.isFinite(p.timeout_ms) && p.timeout_ms > 0
+                        ? Math.floor(p.timeout_ms)
+                        : undefined;
+                    const maxConcurrent = typeof p.max_parallel === 'number' && Number.isFinite(p.max_parallel) && p.max_parallel > 0
+                        ? Math.floor(p.max_parallel)
+                        : undefined;
+                    const results = await this.swarmOrchestrator.delegateSubtasksParallel(normalized, {
+                        timeoutMs,
+                        maxConcurrent,
+                    });
+                    const success = results.every((r) => r.success);
+                    return {
+                        success,
+                        parallel: true,
+                        results,
+                        message: success
+                            ? 'All swarm subtasks finished successfully.'
+                            : 'One or more swarm subtasks failed, timed out, or were killed.',
+                    };
+                }
                 case 'synthesize_tool': {
                     const name = typeof p.name === 'string' ? p.name.trim() : '';
                     const description = typeof p.description === 'string' ? p.description.trim() : '';
@@ -743,7 +737,7 @@ export class CodingToolExecutor {
                     return { success: true, message: `Tool "${name}" registered. You can call it with the same name. Execution is sandboxed.` };
                 }
                 default:
-                    return { error: true, success: false, message: `Unknown tool: ${toolName}. Available tools: read_file, read_multiple_files, write_file, edit_file, edit_lines, insert_at_line, verified_edit, list_directory, search_files, run_command, create_directory, delete_file, move_file, get_context, revert_file, run_tests, get_test_status, get_git_status, get_git_diff_summary, get_git_changed_files, create_git_checkpoint, revert_to_git_checkpoint, git_show_diff, get_mcp_status, grep_code, web_search, fetch_url, remember_lesson, synthesize_tool, take_screenshot, get_console_logs, run_visual_test, check_accessibility, measure_performance, test_responsive, capture_network, run_playwright_test, search_skills_sh, install_skill_from_skills_sh, preview_app, delegate_subtask` };
+                    return { error: true, success: false, message: `Unknown tool: ${toolName}. Available tools: read_file, read_multiple_files, write_file, edit_file, edit_lines, insert_at_line, verified_edit, list_directory, search_files, run_command, create_directory, delete_file, move_file, get_context, revert_file, run_tests, get_test_status, get_git_status, get_git_diff_summary, get_git_changed_files, create_git_checkpoint, revert_to_git_checkpoint, git_show_diff, get_mcp_status, grep_code, web_search, fetch_url, remember_lesson, synthesize_tool, take_screenshot, get_console_logs, run_visual_test, check_accessibility, measure_performance, test_responsive, capture_network, search_skills_sh, install_skill_from_skills_sh, preview_app, delegate_subtask, run_swarm` };
             }
         }
         catch (err) {
@@ -955,7 +949,7 @@ export class CodingToolExecutor {
             },
             {
                 name: 'preview_app',
-                description: 'Capture a visual preview of a web application. Returns a screenshot path and a simplified semantic representation of the DOM. Useful for validating UI changes and layouts.',
+                description: 'Disabled: no bundled browser. Returns guidance to use run_command with agent-browser or a browser MCP. Previously captured a screenshot and simplified DOM summary.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1329,6 +1323,41 @@ export class CodingToolExecutor {
                     required: ['task', 'worker_type']
                 }
             },
+            {
+                name: 'run_swarm',
+                description: 'Run multiple specialized sub-agents in parallel (separate background processes) to save wall-clock time. Each entry has worker_type + task. Cap concurrent workers with max_parallel (default 6). Risk: workers editing the same files can conflict—split work by disjoint paths or use delegate_subtask serially when unsure.',
+                input_schema: {
+                    type: 'object',
+                    properties: {
+                        subtasks: {
+                            type: 'array',
+                            description: 'One object per worker; each runs as an isolated sub-agent.',
+                            items: {
+                                type: 'object',
+                                properties: {
+                                    task: { type: 'string', description: 'Task for this worker only.' },
+                                    worker_type: {
+                                        type: 'string',
+                                        description: 'Agent mode for this worker',
+                                        enum: ['plan', 'agent', 'tester', 'debugger', 'security', 'review', 'team_leader', 'seo', 'product', 'architect', 'engineer', 'data', 'researcher']
+                                    }
+                                },
+                                required: ['task', 'worker_type']
+                            },
+                            minItems: 1
+                        },
+                        timeout_ms: {
+                            type: 'number',
+                            description: 'Optional per-subtask timeout in ms (same as delegate_subtask; default five minutes).'
+                        },
+                        max_parallel: {
+                            type: 'number',
+                            description: 'Max concurrent background agents (default 6). Lower on small machines; raise only if subtasks are independent.'
+                        }
+                    },
+                    required: ['subtasks']
+                }
+            },
             {
                 name: 'grep_code',
                 description: 'Search for a text pattern across your codebase using ripgrep (or grep fallback). Returns matching file paths, line numbers, and line content. Use this to find function usages, variable references, imports, error messages, etc. Much faster than reading files one by one.',
@@ -1406,7 +1435,7 @@ export class CodingToolExecutor {
             },
             {
                 name: 'take_screenshot',
-                description: 'Take a screenshot of a web page. Useful for verifying UI appearance or capturing the state of a web app.',
+                description: 'Disabled: no bundled browser. Returns guidance; use run_command with agent-browser screenshot or a browser MCP for captures.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1419,7 +1448,7 @@ export class CodingToolExecutor {
             },
             {
                 name: 'get_console_logs',
-                description: 'Get browser console logs (and errors) from a URL. Useful for debugging frontend issues.',
+                description: 'Disabled: no bundled browser. Returns guidance; use agent-browser or host browser tooling for console capture.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1430,20 +1459,20 @@ export class CodingToolExecutor {
             },
             {
                 name: 'run_visual_test',
-                description: 'Run visual regression testing by comparing screenshots. Creates baseline on first run, then compares subsequent screenshots against it. Perfect for catching unintended UI changes.',
+                description: 'Disabled: no bundled browser. Returns guidance; use external visual regression or agent-browser workflows in the target project.',
                 input_schema: {
                     type: 'object',
                     properties: {
                         url: { type: 'string', description: 'URL to test (e.g., http://localhost:3000)' },
                         baseline_path: { type: 'string', description: 'Path to baseline screenshot file (e.g., baselines/homepage.png)' },
-                        output_dir: { type: 'string', description: 'Directory for test output (default: .playwright-baselines)' }
+                        output_dir: { type: 'string', description: 'Directory for test output (unused; tool disabled)' }
                     },
                     required: ['url', 'baseline_path']
                 }
             },
             {
                 name: 'check_accessibility',
-                description: 'Run accessibility audit on a webpage. Checks for missing alt text, form labels, heading hierarchy, color contrast, and other WCAG issues. Returns errors, warnings, and notices.',
+                description: 'Disabled: no bundled browser. Returns guidance; use Lighthouse, axe, or agent-browser in the environment.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1454,7 +1483,7 @@ export class CodingToolExecutor {
             },
             {
                 name: 'measure_performance',
-                description: 'Measure Core Web Vitals and performance metrics: First Contentful Paint (FCP), Largest Contentful Paint (LCP), Cumulative Layout Shift (CLS), Time to Interactive (TTI), and DOM Content Loaded. Essential for UX and SEO.',
+                description: 'Disabled: no bundled browser. Returns guidance; use Lighthouse or browser DevTools via your workflow.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1465,7 +1494,7 @@ export class CodingToolExecutor {
             },
             {
                 name: 'test_responsive',
-                description: 'Test a page across multiple viewport sizes (mobile, tablet, desktop). Takes screenshots at each breakpoint and reports any JavaScript errors. Perfect for responsive design testing.',
+                description: 'Disabled: no bundled browser. Returns guidance; use agent-browser or project E2E tooling for viewport checks.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1489,7 +1518,7 @@ export class CodingToolExecutor {
             },
             {
                 name: 'capture_network',
-                description: 'Capture all network requests made during page load. Shows URLs, methods, status codes, resource types, and timing. Useful for debugging API calls, detecting failed requests, and performance analysis.',
+                description: 'Disabled: no bundled browser. Returns guidance; use browser DevTools HAR, agent-browser, or MCP browser network logs.',
                 input_schema: {
                     type: 'object',
                     properties: {
@@ -1498,20 +1527,6 @@ export class CodingToolExecutor {
                     required: ['url']
                 }
             },
-            {
-                name: 'run_playwright_test',
-                description: 'Execute a Playwright test file. Runs the test and returns pass/fail results with output. Great for running E2E tests, integration tests, and component tests.',
-                input_schema: {
-                    type: 'object',
-                    properties: {
-                        test_path: { type: 'string', description: 'Path to the Playwright test file (e.g., tests/homepage.spec.ts)' },
-                        headed: { type: 'boolean', description: 'Run with visible browser window (default: false)' },
-                        browser: { type: 'string', enum: ['chromium', 'firefox', 'webkit'], description: 'Browser to use (default: chromium)' },
-                        timeout: { type: 'number', description: 'Test timeout in milliseconds (default: 120000)' }
-                    },
-                    required: ['test_path']
-                }
-            },
             {
                 name: 'fetch_url',
                 description: 'Fetch and read content from any URL. HTML is automatically stripped to plain text. Use this to read documentation pages, API references, blog posts, or any web content. Supports HTML, JSON, and plain text.',