@hybridlabor-api/aos 4.3.0 → 4.4.0

package/mcps/unreal_mcp/tests/unit/tools/inspect_handlers.test.ts

@@ -1,231 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import type { ITools } from '../../../src/types/tools/tool-interfaces.js';
-
-vi.mock('../../../src/tools/handlers/foundation/dispatch/common-handlers.js', () => ({
-  executeAutomationRequest: vi.fn()
-}));
-
-import { handleInspectTools } from '../../../src/tools/handlers/inspect/inspect-handlers.js';
-import { executeAutomationRequest } from '../../../src/tools/handlers/foundation/dispatch/common-handlers.js';
-
-describe('Inspect Handlers', () => {
-  const mockExecuteAutomationRequest = vi.mocked(executeAutomationRequest);
-  let mockTools: ITools;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockTools = {
-      systemTools: {
-        executeConsoleCommand: vi.fn(async () => ({ success: true })),
-        getProjectSettings: vi.fn(async () => ({}))
-      },
-      assetResources: {
-        list: vi.fn(async () => ({}))
-      },
-      automationBridge: {
-        isConnected: vi.fn().mockReturnValue(true),
-        sendAutomationRequest: vi.fn()
-      }
-    };
-  });
-
-  it('routes get_blueprint_details through blueprint_get', async () => {
-    mockExecuteAutomationRequest.mockResolvedValue({ success: true, variables: [{ name: 'Strength_Min' }] });
-
-    const result = await handleInspectTools(
-      'get_blueprint_details',
-      { objectPath: '/Game/Abilities/Shared/AS_CharacterStats' },
-      mockTools
-    );
-
-    expect(mockExecuteAutomationRequest).toHaveBeenCalledWith(
-      mockTools,
-      'blueprint_get',
-      {
-        requestedPath: '/Game/Abilities/Shared/AS_CharacterStats',
-        blueprintCandidates: ['/Game/Abilities/Shared/AS_CharacterStats']
-      },
-      'inspect:get_blueprint_details -> blueprint_get: automation bridge not available'
-    );
-    expect(result).toEqual({ success: true, variables: [{ name: 'Strength_Min' }] });
-  });
-
-  describe('inspect_cdo', () => {
-    it('routes inspect_cdo with blueprintPath to inspect automation', async () => {
-      mockExecuteAutomationRequest.mockResolvedValue({
-        success: true,
-        className: 'BP_Hero_C',
-        components: [{ name: 'CharacterMesh0', class: 'SkeletalMeshComponent' }]
-      });
-
-      const result = await handleInspectTools(
-        'inspect_cdo',
-        { blueprintPath: '/Game/Characters/BP_Hero' },
-        mockTools
-      );
-
-      const payload = mockExecuteAutomationRequest.mock.calls[0][2] as Record<string, unknown>;
-      expect(payload.action).toBe('inspect_cdo');
-      expect(payload.blueprintPath).toBe('/Game/Characters/BP_Hero');
-      expect(result).toEqual({
-        success: true,
-        className: 'BP_Hero_C',
-        components: [{ name: 'CharacterMesh0', class: 'SkeletalMeshComponent' }]
-      });
-    });
-
-    it('delegates to C++ when blueprintPath is missing', async () => {
-      mockExecuteAutomationRequest.mockResolvedValue({
-        success: false,
-        error: 'blueprintPath is required for inspect_cdo'
-      });
-
-      const result = await handleInspectTools(
-        'inspect_cdo',
-        {},
-        mockTools
-      );
-
-      expect(mockExecuteAutomationRequest).toHaveBeenCalled();
-      expect(result).toEqual({
-        success: false,
-        error: 'blueprintPath is required for inspect_cdo'
-      });
-    });
-
-    it('passes optional detailed and componentName params', async () => {
-      mockExecuteAutomationRequest.mockResolvedValue({ success: true });
-
-      await handleInspectTools(
-        'inspect_cdo',
-        { blueprintPath: '/Game/Characters/BP_Hero', detailed: true, componentName: 'CharacterMesh0' },
-        mockTools
-      );
-
-      const payload = mockExecuteAutomationRequest.mock.calls[0][2] as Record<string, unknown>;
-      expect(payload.action).toBe('inspect_cdo');
-      expect(payload.blueprintPath).toBe('/Game/Characters/BP_Hero');
-      expect(payload.detailed).toBe(true);
-      expect(payload.componentName).toBe('CharacterMesh0');
-    });
-
-    it('passes propertyNames array when provided', async () => {
-      mockExecuteAutomationRequest.mockResolvedValue({ success: true });
-
-      await handleInspectTools(
-        'inspect_cdo',
-        { blueprintPath: '/Game/Characters/BP_Hero', propertyNames: ['SkeletalMesh', 'AnimClass'] },
-        mockTools
-      );
-
-      const payload = mockExecuteAutomationRequest.mock.calls[0][2] as Record<string, unknown>;
-      expect(payload.action).toBe('inspect_cdo');
-      expect(payload.propertyNames).toEqual(['SkeletalMesh', 'AnimClass']);
-    });
-
-    it('falls back to objectPath when blueprintPath not provided', async () => {
-      mockExecuteAutomationRequest.mockResolvedValue({ success: true });
-
-      await handleInspectTools(
-        'inspect_cdo',
-        { objectPath: '/Game/Characters/BP_Hero' },
-        mockTools
-      );
-
-      const payload = mockExecuteAutomationRequest.mock.calls[0][2] as Record<string, unknown>;
-      expect(payload.action).toBe('inspect_cdo');
-      expect(payload.blueprintPath).toBe('/Game/Characters/BP_Hero');
-    });
-  });
-
-  it('inspect_cdo is in the tool schema action enum', async () => {
-    const { consolidatedToolDefinitions } = await import('../../../src/tools/catalog/consolidated-tool-definitions.js');
-    const inspectTool = consolidatedToolDefinitions.find((t: { name: string }) => t.name === 'inspect');
-    const actionEnum = (inspectTool?.inputSchema as Record<string, unknown> & {
-      properties: { action: { enum: string[] } }
-    })?.properties?.action?.enum;
-    expect(actionEnum).toContain('inspect_cdo');
-  });
-
-  it('keeps unsupported inspect export file-path params out of the schema', async () => {
-    const { consolidatedToolDefinitions } = await import('../../../src/tools/catalog/consolidated-tool-definitions.js');
-    const { coreToolDefinitions } = await import('../../../src/tools/schemas/core-tools.js');
-    const inspectTool = consolidatedToolDefinitions.find((t: { name: string }) => t.name === 'inspect');
-    const coreInspectTool = coreToolDefinitions.find((t: { name: string }) => t.name === 'inspect');
-    const tools = [inspectTool, coreInspectTool];
-
-    for (const tool of tools) {
-      const properties = (tool?.inputSchema as Record<string, unknown> & {
-        properties: Record<string, unknown>
-      })?.properties;
-      expect(properties).not.toHaveProperty('destinationPath');
-      expect(properties).not.toHaveProperty('outputPath');
-      expect(properties).not.toHaveProperty('format');
-    }
-  });
-
-  it('normalizes name and propertyPath aliases for get_property', async () => {
-    mockExecuteAutomationRequest
-      .mockResolvedValueOnce({ success: true, actors: [{ path: '/Game/Test/ResolvedActor' }] })
-      .mockResolvedValueOnce({ success: true, value: 0 });
-
-    await handleInspectTools(
-      'get_property',
-      { name: 'ReadableActorName', propertyPath: 'InitialLifeSpan' },
-      mockTools
-    );
-
-    expect(mockExecuteAutomationRequest).toHaveBeenNthCalledWith(
-      2,
-      mockTools,
-      'inspect',
-      {
-        name: 'ReadableActorName',
-        propertyPath: 'InitialLifeSpan',
-        action: 'get_property',
-        propertyName: 'InitialLifeSpan',
-        objectPath: '/Game/Test/ResolvedActor'
-      }
-    );
-  });
-
-  it('normalizes classPath alias for inspect_class', async () => {
-    mockExecuteAutomationRequest.mockResolvedValue({ success: true, classPath: '/Script/Engine.StaticMeshActor' });
-
-    await handleInspectTools(
-      'inspect_class',
-      { classPath: '/Script/Engine.StaticMeshActor' },
-      mockTools
-    );
-
-    expect(mockExecuteAutomationRequest).toHaveBeenCalledWith(
-      mockTools,
-      'inspect',
-      {
-        action: 'inspect_class',
-        className: '/Script/Engine.StaticMeshActor'
-      }
-    );
-  });
-
-  it('keeps inspect_object on the inspect automation path', async () => {
-    mockExecuteAutomationRequest.mockResolvedValue({ success: true, objectPath: '/Game/Test/BP_Test' });
-
-    await handleInspectTools(
-      'inspect_object',
-      { objectPath: '/Game/Test/BP_Test' },
-      mockTools
-    );
-
-    expect(mockExecuteAutomationRequest).toHaveBeenCalledWith(
-      mockTools,
-      'inspect',
-      {
-        objectPath: '/Game/Test/BP_Test',
-        action: 'inspect_object',
-        detailed: true
-      },
-      'Automation bridge not available for inspect operations'
-    );
-  });
-});

package/mcps/unreal_mcp/tests/unit/tools/level.test.ts

@@ -1,56 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { LevelTools } from '../../../src/tools/level/level.js';
-import { UnrealBridge } from '../../../src/unreal-bridge.js';
-
-describe('LevelTools Security', () => {
-    let bridge: UnrealBridge;
-    let levelTools: LevelTools;
-
-    beforeEach(() => {
-        bridge = new UnrealBridge();
-        vi.spyOn(bridge, 'executeConsoleCommand').mockImplementation((cmd: string) => {
-            if (cmd.startsWith('GetLevelPath')) {
-                return Promise.resolve({ success: true, message: '/Game/Maps/MyMap' });
-            }
-            return Promise.resolve({ success: true, message: 'OK' });
-        });
-        levelTools = new LevelTools(bridge);
-    });
-
-    it('should use sanitized path in console fallback for loadLevel', async () => {
-        const rawPath = '\\Game\\Maps\\MyMap';
-        const expectedPath = '/Game/Maps/MyMap';
-
-        await levelTools.loadLevel({ levelPath: rawPath });
-
-        const executeCommandMock = vi.mocked(bridge.executeConsoleCommand);
-        expect(executeCommandMock).toHaveBeenCalled();
-
-        const calls = executeCommandMock.mock.calls;
-        const openCall = calls.find((call) => call[0].startsWith('Open'));
-        expect(openCall).toBeDefined();
-        expect(openCall?.[0]).toBe(`Open ${expectedPath}`);
-    });
-
-    it('should validate level path before falling back to console', async () => {
-        const maliciousPath = '../../Secret/Map';
-
-        await expect(levelTools.loadLevel({ levelPath: maliciousPath }))
-            .rejects.toThrow(/Security validation failed/);
-
-        const executeCommandMock = vi.mocked(bridge.executeConsoleCommand);
-        expect(executeCommandMock).not.toHaveBeenCalled();
-    });
-
-    it('should return error when level not found via GetLevelPath', async () => {
-        vi.mocked(bridge.executeConsoleCommand).mockResolvedValueOnce({
-            success: false,
-            message: 'Level not found: /Game/Maps/NonExistent'
-        });
-
-        const result = await levelTools.loadLevel({ levelPath: '/Game/Maps/NonExistent' });
-
-        expect(result.success).toBe(false);
-        expect(result.error).toBe('LEVEL_NOT_FOUND');
-    });
-});

package/mcps/unreal_mcp/tests/unit/tools/level_security.test.ts

@@ -1,45 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { LevelTools } from '../../../src/tools/level/level.js';
-import { UnrealBridge } from '../../../src/unreal-bridge.js';
-
-describe('LevelTools Injection Security', () => {
-    let bridge: UnrealBridge;
-    let levelTools: LevelTools;
-
-    beforeEach(() => {
-        bridge = new UnrealBridge();
-        vi.spyOn(bridge, 'executeConsoleCommand').mockResolvedValue({ success: true, message: 'OK' });
-        vi.spyOn(bridge, 'executeConsoleCommands').mockResolvedValue({ success: true, message: 'OK' });
-        levelTools = new LevelTools(bridge);
-    });
-
-    it('should NOT allow command injection in createSubLevel via name', async () => {
-        const maliciousName = 'MyLevel;Quit';
-        await levelTools.createSubLevel({ name: maliciousName, type: 'Persistent' });
-
-        const executeCommandMock = vi.mocked(bridge.executeConsoleCommand);
-        expect(executeCommandMock).toHaveBeenCalled();
-        const command = executeCommandMock.mock.calls[0][0];
-
-        expect(command).not.toContain(';Quit');
-    });
-
-    it('should NOT allow command injection in setLevelVisibility via levelName', async () => {
-        const maliciousName = 'MyLevel;Quit';
-        await levelTools.setLevelVisibility({ levelName: maliciousName, visible: true });
-
-        const executeCommandMock = vi.mocked(bridge.executeConsoleCommand);
-        expect(executeCommandMock).toHaveBeenCalled();
-        const command = executeCommandMock.mock.calls[0][0];
-
-        expect(command).not.toContain(';Quit');
-    });
-
-    it('should NOT allow command injection in streaming load fallback', async () => {
-        await levelTools.loadLevel({ levelPath: '/Game/Maps/MyLevel;Quit', streaming: true });
-
-        const executeCommandMock = vi.mocked(bridge.executeConsoleCommand);
-        const command = executeCommandMock.mock.calls[0][0];
-        expect(command).toBe('StreamLevel MyLevel_Quit Load Show');
-    });
-});

package/mcps/unreal_mcp/tests/unit/tools/lighting_timeout_propagation.test.ts

@@ -1,123 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { LONG_RUNNING_OP_TIMEOUT_MS } from '../../../src/constants.js';
-import type { ITools } from '../../../src/types/tools/tool-interfaces.js';
-import { handleEnvironmentTools } from '../../../src/tools/handlers/environment/environment-handlers.js';
-import { handleLevelTools } from '../../../src/tools/handlers/level/runtime/level-handlers.js';
-import { handleLightingTools } from '../../../src/tools/handlers/lighting/lighting-handlers.js';
-
-function createTools() {
-  const sendAutomationRequest = vi.fn(async () => ({ success: true }));
-  const tools: ITools = {
-    systemTools: {
-      executeConsoleCommand: vi.fn(async () => ({ success: true })),
-      getProjectSettings: vi.fn(async () => ({}))
-    },
-    assetResources: {
-      list: vi.fn(async () => ({}))
-    },
-    automationBridge: {
-      isConnected: () => true,
-      sendAutomationRequest
-    }
-  };
-
-  return { tools, sendAutomationRequest };
-}
-
-describe('lighting build timeout propagation', () => {
-  it('uses the caller timeout for the lighting tool without sending it in the payload', async () => {
-    const { tools, sendAutomationRequest } = createTools();
-
-    await handleLightingTools('build_lighting', {
-      action: 'build_lighting',
-      quality: 'High',
-      timeoutMs: 240000
-    }, tools);
-
-    expect(sendAutomationRequest).toHaveBeenCalledWith('bake_lightmap', {
-      quality: 'High',
-      buildOnlySelected: false,
-      buildReflectionCaptures: true,
-      levelPath: undefined
-    }, {
-      timeoutMs: 240000
-    });
-  });
-
-  it('uses the caller timeout for the level tool without sending it in the payload', async () => {
-    const { tools, sendAutomationRequest } = createTools();
-
-    await handleLevelTools('build_lighting', {
-      action: 'build_lighting',
-      quality: 'Production',
-      timeoutMs: 300000
-    }, tools);
-
-    expect(sendAutomationRequest).toHaveBeenCalledWith('manage_lighting', {
-      action: 'build_lighting',
-      quality: 'Production',
-      buildOnlySelected: false,
-      buildReflectionCaptures: false
-    }, {
-      timeoutMs: 300000
-    });
-  });
-
-  it('uses the caller timeout for the environment tool without sending it in the payload', async () => {
-    const { tools, sendAutomationRequest } = createTools();
-
-    await handleEnvironmentTools('bake_lightmap', {
-      action: 'bake_lightmap',
-      quality: 'Preview',
-      timeoutMs: 270000
-    }, tools);
-
-    expect(sendAutomationRequest).toHaveBeenCalledWith('bake_lightmap', {
-      quality: 'Preview',
-      buildOnlySelected: false,
-      buildReflectionCaptures: false
-    }, {
-      timeoutMs: 270000
-    });
-  });
-
-  it.each([
-    {
-      name: 'lighting',
-      run: async (tools: ITools) =>
-        handleLightingTools('build_lighting', {
-          action: 'build_lighting',
-          quality: 'High',
-        }, tools),
-      request: 'bake_lightmap',
-    },
-    {
-      name: 'level',
-      run: async (tools: ITools) =>
-        handleLevelTools('build_lighting', {
-          action: 'build_lighting',
-          quality: 'Production',
-        }, tools),
-      request: 'manage_lighting',
-    },
-    {
-      name: 'environment',
-      run: async (tools: ITools) =>
-        handleEnvironmentTools('bake_lightmap', {
-          action: 'bake_lightmap',
-          quality: 'Preview',
-        }, tools),
-      request: 'bake_lightmap',
-    },
-  ])('uses the long-running default timeout for the $name tool', async ({ run, request }) => {
-    const { tools, sendAutomationRequest } = createTools();
-
-    await run(tools);
-
-    expect(sendAutomationRequest).toHaveBeenCalledWith(
-      request,
-      expect.any(Object),
-      { timeoutMs: LONG_RUNNING_OP_TIMEOUT_MS },
-    );
-  });
-});

package/mcps/windows-computer-use-mcp/docs/ARCHITECTURE.md

@@ -1,46 +0,0 @@
-# Architecture
-
-Windows **computer use agent** — MCP host drives portmanteau tools; server actuates the real desktop.
-
-```
-┌─────────────────┐     stdio / HTTP      ┌──────────────────────────┐
-│  MCP host       │ ◄──────────────────► │  windows-computer-use-mcp│
-│  (Cursor,       │      /mcp             │  FastMCP 3.2 + FastAPI    │
-│   Claude, …)    │                       │  port 10789              │
-└─────────────────┘                       └───────────┬──────────────┘
-                                                        │
-                        ┌───────────────────────────────┼───────────────────────────────┐
-                        ▼                               ▼                               ▼
-                 win32_mouse / UIA              pywinauto / PyAutoGUI            OCR / OpenCV
-                        │                               │                               │
-                        └───────────────────────────────┴───────────────────────────────┘
-                                                        │
-                                                        ▼
-                                              Windows desktop session
-```
-
-## Components
-
-| Layer | Path | Role |
-|-------|------|------|
-| MCP surface | `src/windows_computer_use_mcp/app.py`, `tools/` | Portmanteau tool registration |
-| Transport | `src/windows_computer_use_mcp/transport.py`, `server.py` | stdio + ASGI `/mcp` + REST `/api/v1` |
-| Input | `src/windows_computer_use_mcp/win32_mouse.py` | DPI-aware pointer injection |
-| Safety | `src/windows_computer_use_mcp/safety*.py` | HITL, kill switch, rate limits |
-| Web operator | `web_sota/` | Vite UI on **10788** → proxy **10789** |
-| Desktop app | `web_sota/src-tauri/` | Tauri operator + PyInstaller sidecar |
-
-## Fleet position
-
-- **Hands, not brain** — see [MEMOPS_CUA.md](MEMOPS_CUA.md)
-- Isolation: [virtualization-mcp](https://github.com/sandraschi/virtualization-mcp)
-- Sibling: [autohotkey-mcp](https://github.com/sandraschi/autohotkey-mcp) (AHK scriptlets)
-- Browser DOM: use a Playwright MCP (orthogonal stack)
-
-## Packaging
-
-| Channel | Doc |
-|---------|-----|
-| `uv` / PyPI | [INSTALL.md](../INSTALL.md) |
-| MCPB | [mcpb/README.md](../mcpb/README.md) |
-| Desktop installer | [DESKTOP_APP.md](DESKTOP_APP.md) |