@hybridlabor-api/aos 4.3.0 → 4.4.1

package/mcps/unreal_mcp/plugins/UnrealAgent/Source/UnrealAgent/Private/Tests/UnrealAgentStudioKitAutomationChecks.cpp

@@ -1,155 +0,0 @@
-#if WITH_DEV_AUTOMATION_TESTS
-
-#include "Tests/UnrealAgentAutomationTestDelegates.h"
-
-#include "Acp/Context/UnrealAgentEditorContext.h"
-#include "Acp/Evidence/UnrealAgentEvidenceLedger.h"
-#include "Acp/StudioKit/UnrealAgentStudioKit.h"
-#include "Acp/Validation/UnrealAgentValidationRunner.h"
-#include "HAL/FileManager.h"
-#include "Misc/AutomationTest.h"
-#include "Misc/FileHelper.h"
-#include "Misc/Paths.h"
-
-namespace UnrealAgent::AutomationTests
-{
-    bool RunStudioKitAndContextTest(FAutomationTestBase& Test)
-    {
-        const FString TestDirectory = FPaths::ConvertRelativePathToFull(FPaths::Combine(FPaths::ProjectSavedDir(), TEXT("UnrealAgentStudioKitTest")));
-        IFileManager::Get().DeleteDirectory(*TestDirectory, false, true);
-        if (!IFileManager::Get().MakeDirectory(*TestDirectory, true))
-        {
-            Test.AddError(FString::Printf(TEXT("Failed to create Studio Kit test directory: %s"), *TestDirectory));
-            return false;
-        }
-
-        bool bPassed = true;
-        const FUnrealAgentStudioKitResult KitResult = FUnrealAgentStudioKit::EnsureForProject(TestDirectory);
-        bPassed &= Test.TestTrue(TEXT("Studio Kit generation succeeds"), KitResult.WasSuccessful());
-        bPassed &= Test.TestTrue(TEXT("Studio Kit writes multiple OpenCode files"), KitResult.FilesWritten >= 10);
-
-        FString PrimaryAgent;
-        const FString PrimaryAgentPath = FPaths::Combine(TestDirectory, TEXT(".opencode/agents/unreal-agent.md"));
-        bPassed &= Test.TestTrue(TEXT("Primary Unreal Agent file exists"), FPaths::FileExists(PrimaryAgentPath) && FFileHelper::LoadFileToString(PrimaryAgent, *PrimaryAgentPath));
-        bPassed &= Test.TestTrue(TEXT("Primary agent has prompt marker"), PrimaryAgent.Contains(FUnrealAgentStudioKit::GetPromptVersionMarker()));
-        bPassed &= Test.TestTrue(TEXT("Primary agent has Studio Kit marker"), PrimaryAgent.Contains(FUnrealAgentStudioKit::GetStudioKitVersionMarker()));
-        bPassed &= Test.TestTrue(TEXT("Primary agent includes specialist roles"), PrimaryAgent.Contains(TEXT("unreal-technical-director")) && PrimaryAgent.Contains(TEXT("unreal-qa-release")));
-        bPassed &= Test.TestTrue(TEXT("Primary agent includes MCP tool playbook"), PrimaryAgent.Contains(TEXT("MCP tool playbook")));
-        bPassed &= Test.TestTrue(TEXT("Primary agent includes release workflow"), PrimaryAgent.Contains(TEXT("Shipping: confirm packaging readiness")));
-
-        const FString SkillPath = FPaths::Combine(TestDirectory, TEXT(".opencode/skills/unreal-validation-loop/SKILL.md"));
-        const FString PluginPath = FPaths::Combine(TestDirectory, TEXT(".opencode/plugins/unreal-agent-guardrails.ts"));
-        const FString CommandPath = FPaths::Combine(TestDirectory, TEXT(".opencode/commands/unreal-ship-check.md"));
-        const FString ConfigPath = FPaths::Combine(TestDirectory, TEXT(".opencode/opencode.json"));
-        bPassed &= Test.TestTrue(TEXT("Validation skill is generated"), FPaths::FileExists(SkillPath));
-        bPassed &= Test.TestTrue(TEXT("Guardrails plugin is generated"), FPaths::FileExists(PluginPath));
-        bPassed &= Test.TestTrue(TEXT("Ship check command is generated"), FPaths::FileExists(CommandPath));
-        bPassed &= Test.TestTrue(TEXT("OpenCode config is generated"), FPaths::FileExists(ConfigPath));
-        FString OpenCodeConfigText;
-        bPassed &= Test.TestTrue(TEXT("OpenCode config is readable"), FFileHelper::LoadFileToString(OpenCodeConfigText, *ConfigPath));
-        bPassed &= Test.TestTrue(TEXT("OpenCode config has Studio Kit comment marker"), OpenCodeConfigText.Contains(FUnrealAgentStudioKit::GetStudioKitVersionMarker()));
-        bPassed &= Test.TestFalse(TEXT("OpenCode config does not contain unknown Studio Kit keys"), OpenCodeConfigText.Contains(TEXT("\"unreal_agent_studio_kit_version\"")));
-
-        const FString LegacyConfigDirectory = FPaths::ConvertRelativePathToFull(FPaths::Combine(FPaths::ProjectSavedDir(), TEXT("UnrealAgentStudioKitLegacyConfigTest")));
-        IFileManager::Get().DeleteDirectory(*LegacyConfigDirectory, false, true);
-        const FString LegacyConfigPath = FPaths::Combine(LegacyConfigDirectory, TEXT(".opencode/opencode.json"));
-        IFileManager::Get().MakeDirectory(*FPaths::GetPath(LegacyConfigPath), true);
-        const FString LegacyConfigText = FString()
-            + TEXT("{\n")
-            + TEXT("  \"$schema\": \"https://opencode.ai/config.json\",\n")
-            + TEXT("  \"permission\": {\n")
-            + TEXT("    \"read\": \"allow\",\n")
-            + TEXT("    \"glob\": \"allow\",\n")
-            + TEXT("    \"grep\": \"allow\",\n")
-            + TEXT("    \"list\": \"allow\",\n")
-            + TEXT("    \"edit\": \"ask\",\n")
-            + TEXT("    \"bash\": \"ask\",\n")
-            + TEXT("    \"skill\": {\n")
-            + TEXT("      \"unreal-*\": \"allow\"\n")
-            + TEXT("    }\n")
-            + TEXT("  }\n")
-            + TEXT("}\n");
-        bPassed &= Test.TestTrue(TEXT("Legacy generated OpenCode config is seeded"), FFileHelper::SaveStringToFile(LegacyConfigText, *LegacyConfigPath, FFileHelper::EEncodingOptions::ForceUTF8WithoutBOM));
-        FUnrealAgentStudioKit::EnsureForProject(LegacyConfigDirectory);
-        FString UpgradedLegacyConfigText;
-        bPassed &= Test.TestTrue(TEXT("Legacy generated OpenCode config remains readable"), FFileHelper::LoadFileToString(UpgradedLegacyConfigText, *LegacyConfigPath));
-        bPassed &= Test.TestTrue(TEXT("Legacy generated OpenCode config is upgraded with marker"), UpgradedLegacyConfigText.Contains(FUnrealAgentStudioKit::GetStudioKitVersionMarker()));
-
-        const FString Redacted = FUnrealAgentStudioKit::RedactSensitiveText(TEXT("Authorization: Bearer abc123\nX-MCP-Capability-Token: fake-capability-token\nsafe: value"));
-        bPassed &= Test.TestFalse(TEXT("Redaction removes bearer token"), Redacted.Contains(TEXT("abc123")));
-        bPassed &= Test.TestFalse(TEXT("Redaction removes capability token"), Redacted.Contains(TEXT("fake-capability-token")));
-        bPassed &= Test.TestTrue(TEXT("Redaction keeps safe lines"), Redacted.Contains(TEXT("safe: value")));
-
-        const FUnrealAgentEditorContextSnapshot Context = FUnrealAgentEditorContext::Capture(TestDirectory);
-        bPassed &= Test.TestTrue(TEXT("Editor context envelope is produced"), Context.Envelope.Contains(TEXT("<unreal_editor_context")));
-        bPassed &= Test.TestTrue(TEXT("Editor context has privacy guidance"), Context.Envelope.Contains(TEXT("Sensitive credential values")));
-        bPassed &= Test.TestFalse(TEXT("Editor context does not expose fake secrets"), Context.Envelope.Contains(TEXT("fake-capability-token")));
-        bPassed &= Test.TestFalse(TEXT("Editor context does not expose absolute project directory"), Context.Envelope.Contains(TestDirectory));
-        bPassed &= Test.TestTrue(TEXT("Editor context redacts project directory"), Context.Envelope.Contains(TEXT("projectDir: [redacted project root]")));
-
-        const FUnrealAgentValidationResult ValidationResult = FUnrealAgentValidationRunner::RunFastValidation(TestDirectory);
-        bPassed &= Test.TestTrue(TEXT("Fast validation passes after Studio Kit generation"), ValidationResult.bPassed);
-        bPassed &= Test.TestTrue(TEXT("Validation records evidence"), FPaths::FileExists(ValidationResult.EvidencePath));
-
-        FString FirstEvidencePath;
-        FString SecondEvidencePath;
-        bPassed &= Test.TestTrue(TEXT("First same-type evidence event records"), FUnrealAgentEvidenceLedger::RecordEvent(TestDirectory, TEXT("collision"), TEXT("passed"), TEXT("same summary"), TEXT("details"), &FirstEvidencePath));
-        bPassed &= Test.TestTrue(TEXT("Second same-type evidence event records"), FUnrealAgentEvidenceLedger::RecordEvent(TestDirectory, TEXT("collision"), TEXT("passed"), TEXT("same summary"), TEXT("details"), &SecondEvidencePath));
-        bPassed &= Test.TestTrue(TEXT("Same-second evidence events use distinct paths"), !FirstEvidencePath.IsEmpty() && !SecondEvidencePath.IsEmpty() && FirstEvidencePath != SecondEvidencePath && FPaths::FileExists(FirstEvidencePath) && FPaths::FileExists(SecondEvidencePath));
-
-        const FString MissingDirectory = FPaths::ConvertRelativePathToFull(FPaths::Combine(FPaths::ProjectSavedDir(), TEXT("UnrealAgentMissingValidationTest")));
-        IFileManager::Get().DeleteDirectory(*MissingDirectory, false, true);
-        const FUnrealAgentValidationResult MissingValidationResult = FUnrealAgentValidationRunner::RunFastValidation(MissingDirectory);
-        bPassed &= Test.TestFalse(TEXT("Validation fails for missing project directory"), MissingValidationResult.bPassed);
-        bPassed &= Test.TestTrue(TEXT("Validation reports missing project directory"), !MissingValidationResult.Errors.IsEmpty() && MissingValidationResult.Errors[0].Contains(TEXT("Project directory does not exist")));
-        bPassed &= Test.TestTrue(TEXT("Missing project validation does not write evidence"), MissingValidationResult.EvidencePath.IsEmpty() && !FPaths::DirectoryExists(FPaths::Combine(MissingDirectory, TEXT("Saved/UnrealAgent"))));
-
-        const FString BrokenDecisionsDirectory = FPaths::ConvertRelativePathToFull(FPaths::Combine(FPaths::ProjectSavedDir(), TEXT("UnrealAgentBrokenDecisionsTest")));
-        IFileManager::Get().DeleteDirectory(*BrokenDecisionsDirectory, false, true);
-        FUnrealAgentEvidenceLedger::EnsureLedger(BrokenDecisionsDirectory);
-        const FString BrokenDecisionsPath = FPaths::Combine(BrokenDecisionsDirectory, TEXT("Saved/UnrealAgent/decisions.md"));
-        IFileManager::Get().Delete(*BrokenDecisionsPath);
-        IFileManager::Get().MakeDirectory(*BrokenDecisionsPath, true);
-        Test.AddExpectedErrorPlain(TEXT("UnrealAgentBrokenDecisionsTest"), EAutomationExpectedErrorFlags::Contains, 2);
-        FString BrokenDecisionsEvidencePath = TEXT("stale-evidence-path");
-        bPassed &= Test.TestFalse(TEXT("Evidence recording fails when decisions ledger cannot be appended"), FUnrealAgentEvidenceLedger::RecordEvent(BrokenDecisionsDirectory, TEXT("broken-decisions"), TEXT("failed"), TEXT("summary"), TEXT("details"), &BrokenDecisionsEvidencePath));
-        bPassed &= Test.TestTrue(TEXT("Decision write failure clears stale success path"), BrokenDecisionsEvidencePath.IsEmpty());
-
-        const FString BrokenStateDirectory = FPaths::ConvertRelativePathToFull(FPaths::Combine(FPaths::ProjectSavedDir(), TEXT("UnrealAgentBrokenStateTest")));
-        IFileManager::Get().DeleteDirectory(*BrokenStateDirectory, false, true);
-        FUnrealAgentEvidenceLedger::EnsureLedger(BrokenStateDirectory);
-        const FString BrokenStatePath = FPaths::Combine(BrokenStateDirectory, TEXT("Saved/UnrealAgent/state.json"));
-        IFileManager::Get().Delete(*BrokenStatePath);
-        IFileManager::Get().MakeDirectory(*BrokenStatePath, true);
-        Test.AddExpectedErrorPlain(TEXT("UnrealAgentBrokenStateTest"), EAutomationExpectedErrorFlags::Contains, 2);
-        FString BrokenStateEvidencePath = TEXT("stale-evidence-path");
-        bPassed &= Test.TestFalse(TEXT("Evidence recording fails when state ledger cannot be written"), FUnrealAgentEvidenceLedger::RecordEvent(BrokenStateDirectory, TEXT("broken-state"), TEXT("failed"), TEXT("summary"), TEXT("details"), &BrokenStateEvidencePath));
-        bPassed &= Test.TestTrue(TEXT("State write failure clears stale success path"), BrokenStateEvidencePath.IsEmpty());
-
-        const FString CustomDirectory = FPaths::ConvertRelativePathToFull(FPaths::Combine(FPaths::ProjectSavedDir(), TEXT("UnrealAgentStudioKitCustomTest")));
-        IFileManager::Get().DeleteDirectory(*CustomDirectory, false, true);
-        const FString CustomAgentPath = FPaths::Combine(CustomDirectory, TEXT(".opencode/agents/unreal-agent.md"));
-        const FString CustomConfigPath = FPaths::Combine(CustomDirectory, TEXT(".opencode/opencode.json"));
-        IFileManager::Get().MakeDirectory(*FPaths::GetPath(CustomAgentPath), true);
-        const FString CustomAgentText = TEXT("custom user-owned Unreal Agent prompt");
-        const FString CustomConfigText = TEXT("{\n  \"$schema\": \"https://opencode.ai/config.json\",\n  \"permission\": {\n    \"read\": \"deny\"\n  }\n}\n");
-        FFileHelper::SaveStringToFile(CustomAgentText, *CustomAgentPath, FFileHelper::EEncodingOptions::ForceUTF8WithoutBOM);
-        FFileHelper::SaveStringToFile(CustomConfigText, *CustomConfigPath, FFileHelper::EEncodingOptions::ForceUTF8WithoutBOM);
-        FUnrealAgentStudioKit::EnsureForProject(CustomDirectory);
-        FString PreservedAgentText;
-        FString PreservedConfigText;
-        bPassed &= Test.TestTrue(TEXT("Custom prompt remains readable"), FFileHelper::LoadFileToString(PreservedAgentText, *CustomAgentPath));
-        bPassed &= Test.TestEqual(TEXT("Custom unmarked prompt is preserved"), PreservedAgentText, CustomAgentText);
-        bPassed &= Test.TestTrue(TEXT("Custom OpenCode config remains readable"), FFileHelper::LoadFileToString(PreservedConfigText, *CustomConfigPath));
-        bPassed &= Test.TestEqual(TEXT("Custom unmarked OpenCode config is preserved"), PreservedConfigText, CustomConfigText);
-
-        IFileManager::Get().DeleteDirectory(*TestDirectory, false, true);
-        IFileManager::Get().DeleteDirectory(*LegacyConfigDirectory, false, true);
-        IFileManager::Get().DeleteDirectory(*CustomDirectory, false, true);
-        IFileManager::Get().DeleteDirectory(*BrokenDecisionsDirectory, false, true);
-        IFileManager::Get().DeleteDirectory(*BrokenStateDirectory, false, true);
-        return bPassed;
-    }
-}
-
-#endif

package/mcps/unreal_mcp/tests/AGENTS.md

@@ -1,73 +0,0 @@
-# TEST KNOWLEDGE BASE
-
-Test infrastructure for the TypeScript MCP server and Unreal bridge contracts. Vitest tests run without Unreal; MCP integration suites normally require a built server plus a connected Unreal Editor.
-
-## STRUCTURE
-```text
-tests/
-|-- test-runner.mjs                 # stdio client, setup/cleanup, expectations, JSON reports
-|-- test-runner-response-utils.mjs  # response-path assertions and filtered capture selection
-|-- integration.mjs                 # compact representative parent-tool integration suite
-|-- expectation-utils.mjs           # shared expectation parsing
-|-- mcp-tools/
-|   |-- core/                       # actor, editor, assets, blueprints, levels, inspect, tools
-|   |-- gameplay/                   # animation, AI, character, combat, effects, GAS, inventory
-|   |-- utility/                    # audio, behavior trees, networking, sequences
-|   `-- world/                      # environment, geometry, level structure, PCG
-|-- native-mcp-parity-audit.mjs     # TS versus native canonical tool/action parity
-|-- parameter-combination-audit.mjs # audit CLI entrypoint
-|-- parameter-audit-*.mjs           # CLI, schema extraction, suite capture, coverage, context
-|-- unit/                           # Vitest behavior, security, routing, and source contracts
-`-- reports/                        # generated JSON only; never add AGENTS or hand-authored files
-```
-
-The mock smoke test is `scripts/smoke-test.ts`, not under this directory. It imports `dist/index.js`, uses linked in-memory transports, enables `MOCK_UNREAL_CONNECTION`, and checks the 23-tool surface plus `manage_tools`.
-
-## WHERE TO LOOK
-| Task | Location | Notes |
-|------|----------|-------|
-| Change live test execution | `test-runner.mjs`, `test-runner-response-utils.mjs` | Owns server spawn, response normalization, retries, assertions, captures, cleanup, reports |
-| Add broad integration coverage | `integration.mjs` | Canonical quick suite across parent tools |
-| Add tool/action coverage | `mcp-tools/<category>/*.test.mjs` | Import `runToolTests` directly and keep setup before dependent cases |
-| Change expectation parsing | `expectation-utils.mjs`, `unit/test_runner.test.ts` | Lock evaluator changes with focused unit cases |
-| Change native parity rules | `native-mcp-parity-audit.mjs` | Reads TS definitions and native canonical registry/tool C++ |
-| Change parameter audit | `parameter-audit-*.mjs` | Schema AST, suite capture, live/static evidence, reporting |
-| Change architectural contracts | `unit/plugin/`, `unit/source_structure.test.ts`, `unit/tools/handler_structure.test.ts` | These inspect source text and layout; they do not compile Unreal |
-
-## COMMANDS
-```bash
-npm run test:unit          # Vitest: src/**/*.test.ts and tests/unit/**/*.test.ts
-npm run test:smoke         # mock in-memory MCP check against built dist/
-npm test                   # tests/integration.mjs; Unreal-dependent
-npm run test:native-parity # canonical TS/native tool and action equality
-npm run test:params        # parity, then static + strict + optional-strict parameter audit
-```
-
-## INTEGRATION CASES
-- Use `{ scenario, toolName, arguments, expected }`; optional fields include `assertions`, `captureResult`, and `timeoutMs`.
-- Export no custom harness: end each suite with `runToolTests('<suite-name>', cases)`.
-- Keep the standard relative `runToolTests` import shape; the static audit replaces that import while evaluating suite definitions.
-- Use unique actor/asset names, usually timestamped, and add explicit cleanup for created state.
-- Captures use `{ key, fromField }`; array captures may add `where: { path, equals|includes }` and `selectField`. Later arguments reference `${captured:key}`.
-- Assertions address response paths such as `structuredContent.result.assetPath`; use `equals` for exact values, `includes` for string fragments, or `approximately` with a nonnegative `tolerance` for floating-point values.
-
-## EXPECTATION GRAMMAR
-- Strings split on literal ` or ` or `|`; the first token is the primary intent.
-- Put `success`, `error`, or `timeout` first according to what the case is proving.
-- Success-primary cases may name narrow state alternatives such as `already exists`, `not found`, `not loaded`, or `NOT_PARTITIONED`.
-- Do not use broad masks such as `success|error` or place `timeout` after `error`; crashes, bridge loss, and non-primary timeouts are infrastructure failures.
-- Object expectations support `condition`, `successPattern`, and `errorPattern`; prefer patterns for exact controlled fallback codes.
-- `structuredContent.success: false`, nested failures, or `isError: true` must not pass a success-primary case unless an explicit allowed alternative matches.
-
-## AUDIT CONTRACTS
-- Native parity compares canonical tool names and `definitions/shared/action-sets.ts` enums with native MCP registry and tool definitions.
-- Parameter schema extraction uses the TypeScript compiler API; suite coverage is captured from `mcp-tools/` plus `integration.mjs`.
-- Missing or extra actions always fail the parameter audit. `--strict` also fails undeclared test parameters.
-- `--optional-strict` fails optional schema parameters absent from static coverage; `npm run test:params` enables all strict static gates.
-- Live audit mode consumes the latest `<suite>-test-results-*.json`; only successful live responses prove optional-parameter coverage.
-- Treat `reports/` as disposable evidence. Diagnose failures from the newest JSON, but never edit reports to satisfy a gate.
-
-## SOURCE-CONTRACT TESTS
-- Many `unit/plugin/*contracts.test.ts` cases read C++/C# files and assert required or forbidden source patterns.
-- Structure tests enforce responsibility boundaries, resolvable local includes, naming rules, and the 250 pure-line ceiling.
-- When moving or renaming implementation files, update contract paths and assertions only when the intended invariant changed.

package/mcps/unreal_mcp/tests/expectation-utils.mjs

@@ -1,29 +0,0 @@
-export function expectedCondition(expectation) {
-  if (expectation && typeof expectation === 'object') {
-    if (typeof expectation.condition === 'string') return expectation.condition;
-
-    const conditions = [];
-    if (typeof expectation.successPattern === 'string') {
-      conditions.push('success', expectation.successPattern);
-    }
-    if (typeof expectation.errorPattern === 'string') {
-      conditions.push('error', expectation.errorPattern);
-    }
-    if (conditions.length > 0) return conditions.join('|');
-
-    try {
-      return JSON.stringify(expectation);
-    } catch {
-      return String(expectation);
-    }
-  }
-  return typeof expectation === 'string' ? expectation : String(expectation ?? '');
-}
-
-export function splitExpectedConditions(condition) {
-  if (typeof condition !== 'string') return [];
-  if (condition.includes(' or ')) return condition.split(' or ').map((entry) => entry.trim()).filter(Boolean);
-  if (condition.includes('|')) return condition.split('|').map((entry) => entry.trim()).filter(Boolean);
-  const trimmed = condition.trim();
-  return trimmed ? [trimmed] : [];
-}

package/mcps/unreal_mcp/tests/heartbeat-progress.test.mjs

@@ -1,143 +0,0 @@
-#!/usr/bin/env node
-/**
- * Heartbeat/Progress Protocol Integration Tests
- *
- * Tests the progress_update message protocol that keeps long-running
- * operations alive by extending their timeout on the server side.
- *
- * Prerequisites:
- * - UE Editor must be running with the McpAutomationBridge plugin loaded
- * - MCP server must be built (npm run build:core)
- *
- * Usage:
- *   node tests/heartbeat-progress.test.mjs
- */
-
-import { runToolTests } from './test-runner.mjs';
-
-const testCases = [
-  // Test 1: Basic progress protocol test - short operation with progress
-  {
-    scenario: 'Progress: Short operation with progress updates (5 steps, 200ms each)',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 5,
-      stepDurationMs: 200,
-      sendProgress: true
-    },
-    expected: 'success'
-  },
-
-  // Test 2: Progress protocol disabled - should still work but no progress updates
-  {
-    scenario: 'Progress: Operation without progress updates',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 3,
-      stepDurationMs: 100,
-      sendProgress: false
-    },
-    expected: 'success'
-  },
-
-  // Test 3: Longer operation - tests max extension limit
-  // 10 progress updates hits MAX_PROGRESS_EXTENSIONS=10 limit and gets rejected
-  // This is CORRECT behavior - safety mechanism prevents deadlock
-  {
-    scenario: 'Progress: Longer operation hits max extensions limit (10 steps = 11 updates)',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 10,
-      stepDurationMs: 500,
-      sendProgress: true
-    },
-    expected: { condition: 'error', errorPattern: 'max progress extensions' }
-  },
-
-  // Test 4: Edge case - minimum steps
-  {
-    scenario: 'Progress: Minimum steps (1 step)',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 1,
-      stepDurationMs: 100,
-      sendProgress: true
-    },
-    expected: 'success'
-  },
-
-  // Test 5: Within max extensions limit (9 steps = 10 updates, just under limit)
-  {
-    scenario: 'Progress: Within max extensions limit (9 steps)',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 9,
-      stepDurationMs: 100,
-      sendProgress: true
-    },
-    expected: 'success'
-  },
-
-  // Test 6: Stale progress test - sends same percent repeatedly
-  // This SHOULD FAIL because stale detection triggers (correct safety behavior)
-  {
-    scenario: 'Progress: Stale detection triggers correctly (same percent 5 times)',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_stale_progress',
-      staleCount: 5
-    },
-    expected: { condition: 'error', errorPattern: 'stalled' }
-  },
-
-  // Test 7: Verify progress fields in response
-  {
-    scenario: 'Progress: Verify response contains progress test results',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 3,
-      stepDurationMs: 100,
-      sendProgress: true
-    },
-    expected: { condition: 'success', successPattern: 'progressSent' }
-  },
-
-  // Test 8: Very short operation - should complete within base timeout even without progress
-  {
-    scenario: 'Progress: Quick operation completes fast',
-    toolName: 'system_control',
-    arguments: {
-      action: 'test_progress_protocol',
-      steps: 2,
-      stepDurationMs: 50,
-      sendProgress: true
-    },
-    expected: 'success'
-  }
-];
-
-console.log('='.repeat(60));
-console.log('Heartbeat/Progress Protocol Integration Tests');
-console.log('='.repeat(60));
-console.log('');
-console.log('Testing:');
-console.log('  - progress_update message handling');
-console.log('  - Timeout extension during long operations');
-console.log('  - Stale progress detection');
-console.log('  - Deadlock prevention mechanisms');
-console.log('');
-console.log('Configuration:');
-console.log('  - DEFAULT_OPERATION_TIMEOUT_MS = 30000 (30s)');
-console.log('  - PROGRESS_EXTENSION_MS = 30000 (30s per update)');
-console.log('  - MAX_PROGRESS_EXTENSIONS = 10');
-console.log('  - PROGRESS_STALE_THRESHOLD = 3');
-console.log('  - ABSOLUTE_MAX_TIMEOUT_MS = 300000 (5 min)');
-console.log('');
-
-runToolTests('heartbeat-progress', testCases);

package/mcps/unreal_mcp/tests/integration/get_ai_info_characterization.mjs

@@ -1,216 +0,0 @@
-#!/usr/bin/env node
-/**
- * Characterization test for `manage_ai.get_ai_info(blackboardPath)`.
- *
- * Pins the current Blackboard key serialization shape so that future refactors
- * of the BB serialization path in `McpAutomationBridge_AIHandlers.cpp` cannot
- * silently regress the response contract that downstream callers depend on.
- *
- * Run:
- *   npm run build:core
- *   node tests/integration/get_ai_info_characterization.mjs
- *
- * Requires UE editor running with the McpAutomationBridge plugin loaded.
- *
- * Pinned facts (observed against UE 5.7 + current dev):
- *   - Response data lives at `response.structuredContent.result.aiInfo`.
- *   - UE auto-inserts a `SelfActor` key inherited from BlackboardData base, so
- *     `keyCount` is (added keys + 1). This inheritance must be preserved.
- *   - Type strings carry the `BlackboardKeyType_*` prefix (the C++ class name),
- *     not the bare authoring enum (`Bool`/`Object`/`Int`).
- */
-
-import { Client } from '@modelcontextprotocol/sdk/client/index.js';
-import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-import assert from 'node:assert/strict';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const repoRoot = path.resolve(__dirname, '..', '..');
-
-const TEST_FOLDER = '/Game/IntegrationTest';
-// Each run lives inside its own temp folder under TEST_FOLDER. Sweeping that
-// folder in `finally` removes every artifact this run created, even ones an
-// individual asset-level delete might miss (and keeps prior runs that crashed
-// before cleanup isolated to their own tag rather than scattering siblings).
-const RUN_TAG = `${Date.now()}_${Math.random().toString(36).slice(2, 8)}`;
-const RUN_FOLDER = `${TEST_FOLDER}/PR0a_run_${RUN_TAG}`;
-const BB_NAME = 'BB_GetAiInfoCharacterization';
-
-const transport = new StdioClientTransport({
-  command: 'node',
-  args: [path.join(repoRoot, 'dist', 'cli.js')],
-  cwd: repoRoot,
-});
-
-const client = new Client({ name: 'pr0a-characterization', version: '0.0.1' }, { capabilities: {} });
-
-/**
- * Invoke an MCP tool and return its `structuredContent` payload.
- *
- * Reads from `structuredContent` (the canonical JSON envelope this MCP server
- * emits for automation responses); the sibling `content[0].text` is a flat
- * human-readable summary, not JSON, and is intentionally ignored except for
- * surfacing detail on transport-level `isError`. Throws on any of:
- *   - transport `isError: true`
- *   - missing `structuredContent`
- *   - `structuredContent.success === false`
- *
- * @param {string} toolName  Tool identifier (e.g. `manage_ai`, `manage_asset`).
- * @param {object} args      Tool arguments forwarded as `arguments`.
- * @returns {Promise<object>} The parsed `structuredContent` object.
- */
-async function call(toolName, args) {
-  const res = await client.callTool({ name: toolName, arguments: args });
-  if (res?.isError) {
-    const detail = (res?.content?.[0]?.text ?? '').slice(0, 500);
-    throw new Error(`Tool ${toolName} returned isError; detail: ${detail}`);
-  }
-  const sc = res?.structuredContent;
-  if (!sc) throw new Error(`No structuredContent from ${toolName}`);
-  // Reject anything that is not an explicit success — strict `!== true` so a
-  // response that omits the field (which can happen on some MCP servers
-  // depending on result-class) doesn't slip through and let downstream code
-  // operate on an unverified payload.
-  if (sc.success !== true) {
-    throw new Error(`Tool ${toolName} did not report success:true; payload: ${JSON.stringify(sc).slice(0, 500)}`);
-  }
-  return sc;
-}
-
-/**
- * Call `add_blackboard_key` with a short retry. The first add after
- * `create_blackboard_asset` can race the UE asset registry's visibility
- * propagation; subsequent adds run after the BB is already registered, so
- * only the first invocation needs this guard.
- *
- * @param {object} args  Arguments forwarded to `add_blackboard_key`.
- * @param {number} attempts  Total tries including the first.
- * @param {number} delayMs   Backoff between tries (linear, kept small).
- * @returns {Promise<object>} The `call()` result of the successful attempt.
- */
-async function addKeyWithRetry(args, attempts = 3, delayMs = 100) {
-  let lastErr;
-  for (let i = 0; i < attempts; i++) {
-    try {
-      return await call('manage_ai', { action: 'add_blackboard_key', ...args });
-    } catch (e) {
-      lastErr = e;
-      if (i < attempts - 1) await new Promise((r) => setTimeout(r, delayMs));
-    }
-  }
-  throw lastErr;
-}
-
-let exitCode = 0;
-// Hoisted so `finally` can target the created BB explicitly. Without this, the
-// folder-delete code path enumerates assets via McpSafeDeleteFolder which, for
-// some asset classes (BlackboardData observed in UE 5.7), triggers an
-// interactive "Delete Assets" modal that wedges the editor main thread and
-// times out the request at 30s. Deleting the asset directly first uses a
-// quieter UEditorAssetLibrary::DeleteAsset path; the subsequent folder delete
-// then takes the fast empty-folder branch.
-let bbPath = '';
-try {
-  // Connect inside try so a failed connect (transport spawn / MCP handshake) still hits the finally cleanup
-  await client.connect(transport);
-
-  // Safety belt: if a prior process somehow used the same RUN_TAG (essentially
-  // impossible with timestamp + random suffix), best-effort sweep its folder.
-  try { await call('manage_asset', { action: 'delete', path: RUN_FOLDER, force: true }); } catch {}
-
-  // Setup — create_folder is best-effort for both the shared parent (may already
-  // exist from tests/integration.mjs) and the per-run subfolder.
-  try { await call('manage_asset', { action: 'create_folder', path: TEST_FOLDER }); } catch {}
-  try { await call('manage_asset', { action: 'create_folder', path: RUN_FOLDER }); } catch {}
-
-  // Use the bbPath returned by create_blackboard_asset (the canonical UE object
-  // path, e.g. "/Game/.../BB_Name.BB_Name") for every follow-up call rather than
-  // a client-side string concat. Insulates the test against any future server-
-  // side path normalization changes.
-  const createResp = await call('manage_ai', { action: 'create_blackboard_asset', path: RUN_FOLDER, name: BB_NAME });
-  bbPath = createResp.result?.blackboardPath ?? '';
-  assert.ok(bbPath, 'create_blackboard_asset must return result.blackboardPath');
-
-  // First add retries the asset-registry visibility race; later adds don't need it.
-  await addKeyWithRetry({ blackboardPath: bbPath, keyName: 'Spotted', keyType: 'Bool' });
-  await call('manage_ai', { action: 'add_blackboard_key', blackboardPath: bbPath, keyName: 'TargetActor',   keyType: 'Object', baseObjectClass: 'Actor' });
-  await call('manage_ai', { action: 'add_blackboard_key', blackboardPath: bbPath, keyName: 'WaypointIndex', keyType: 'Int' });
-  await call('manage_ai', { action: 'set_key_instance_synced', blackboardPath: bbPath, keyName: 'WaypointIndex', isInstanceSynced: true });
-
-  // Action under test
-  const resp = await call('manage_ai', { action: 'get_ai_info', blackboardPath: bbPath });
-
-  // Envelope
-  assert.equal(resp.success, true, 'structuredContent.success must be true');
-
-  // Access path: structuredContent.result.aiInfo
-  const aiInfo = resp.result?.aiInfo;
-  assert.ok(aiInfo, 'response must include result.aiInfo for blackboardPath input');
-
-  // Existing top-level fields (must remain bit-identical across PR1a refactor)
-  assert.equal(aiInfo.assignedBlackboard, BB_NAME,
-    'assignedBlackboard must equal the BB asset name');
-
-  // 3 keys we added + 1 SelfActor inherited from BlackboardData base = 4
-  assert.equal(aiInfo.keyCount, 4,
-    'keyCount = added keys + 1 SelfActor (inherited from BlackboardData)');
-
-  assert.ok(Array.isArray(aiInfo.blackboardKeys),
-    'blackboardKeys must be an array');
-  assert.equal(aiInfo.blackboardKeys.length, 4,
-    'blackboardKeys array length matches keyCount');
-
-  const byName = (n) => aiInfo.blackboardKeys.find((k) => k.name === n);
-
-  // Per-key shape: name, type ("BlackboardKeyType_*" — the UE C++ class name),
-  // instanceSynced. PR1a may add fields (baseClass, entryCategory,
-  // parentBlackboard, ...); per-key extra fields are tolerated, the existing
-  // three fields must remain unchanged.
-
-  const selfActor = byName('SelfActor');
-  assert.ok(selfActor, 'SelfActor key must be present (auto-inserted by UE BlackboardData base)');
-  assert.equal(selfActor.name, 'SelfActor');
-  assert.equal(selfActor.type, 'BlackboardKeyType_Object');
-  assert.equal(selfActor.instanceSynced, false);
-
-  const spotted = byName('Spotted');
-  assert.ok(spotted, 'Spotted key must be present');
-  assert.equal(spotted.name, 'Spotted');
-  assert.equal(spotted.type, 'BlackboardKeyType_Bool');
-  assert.equal(spotted.instanceSynced, false);
-
-  const targetActor = byName('TargetActor');
-  assert.ok(targetActor, 'TargetActor key must be present');
-  assert.equal(targetActor.name, 'TargetActor');
-  assert.equal(targetActor.type, 'BlackboardKeyType_Object');
-  assert.equal(targetActor.instanceSynced, false);
-
-  const waypointIndex = byName('WaypointIndex');
-  assert.ok(waypointIndex, 'WaypointIndex key must be present');
-  assert.equal(waypointIndex.name, 'WaypointIndex');
-  assert.equal(waypointIndex.type, 'BlackboardKeyType_Int');
-  assert.equal(waypointIndex.instanceSynced, true,
-    'set_key_instance_synced must have flipped this to true');
-
-  console.log('PASS: get_ai_info BB shape characterization');
-} catch (err) {
-  console.error('FAIL:', err?.message ?? err);
-  if (err?.stack) console.error(err.stack);
-  exitCode = 1;
-} finally {
-  // Two-step teardown, best-effort. Step 1 deletes the BB asset by its exact
-  // path so the plugin takes the single-asset code path (silent
-  // UEditorAssetLibrary::DeleteAsset). Step 2 then sweeps the now-empty
-  // RUN_FOLDER; the empty-folder fast path skips asset enumeration entirely.
-  // This avoids the interactive "Delete Assets" modal that the folder-delete
-  // code path can raise for some asset classes in UE 5.7.
-  if (bbPath) {
-    try { await call('manage_asset', { action: 'delete', path: bbPath, force: true }); } catch {}
-  }
-  try { await call('manage_asset', { action: 'delete', path: RUN_FOLDER, force: true }); } catch {}
-  try { await client.close(); } catch {}
-}
-
-process.exit(exitCode);