ultimate-unreal-engine-mcp 0.1.7 → 0.1.9

package/dist/plugin-bridge/client.js

@@ -177,32 +177,53 @@ export class PluginBridgeClient {
     connect() {
         return new Promise((resolve, reject) => {
             const socket = net.createConnection({ port: this.port, host: '127.0.0.1' });
-            // JSON-newline response parser — accumulates partial chunks, splits on \n,
-            // and resolves the correlated pending promise.
-            // Threat T-07-09 mitigation: try/catch around JSON.parse; unknown correlationId
-            // is silently dropped — no state corruption.
+            // JSON response parser — handles UE's pretty-printed multi-line JSON.
+            // UE's FJsonSerializer outputs JSON with \r\n and tabs inside the object,
+            // so we can't split on \n (that would break mid-object). Instead we track
+            // brace depth: when depth returns to 0, we have a complete JSON object.
             socket.on('data', (chunk) => {
                 this.receiveBuffer += chunk.toString('utf8');
-                let newlineIdx;
-                while ((newlineIdx = this.receiveBuffer.indexOf('\n')) !== -1) {
-                    const line = this.receiveBuffer.slice(0, newlineIdx).trim();
-                    this.receiveBuffer = this.receiveBuffer.slice(newlineIdx + 1);
-                    if (line.length === 0) {
-                        continue;
-                    }
-                    try {
-                        const response = JSON.parse(line);
-                        const pending = this.pendingCommands.get(response.correlationId);
-                        if (pending) {
-                            this.pendingCommands.delete(response.correlationId);
-                            pending.resolve(response);
+                let startIdx = -1;
+                let depth = 0;
+                for (let i = 0; i < this.receiveBuffer.length; i++) {
+                    const ch = this.receiveBuffer[i];
+                    if (ch === '{') {
+                        if (depth === 0) {
+                            startIdx = i;
                         }
-                        // Unknown correlationId — silently drop (T-07-09 mitigation)
+                        depth++;
                     }
-                    catch {
-                        // Non-JSON line — ignore (T-07-09 mitigation)
+                    else if (ch === '}') {
+                        depth--;
+                        if (depth === 0 && startIdx >= 0) {
+                            const jsonStr = this.receiveBuffer.slice(startIdx, i + 1);
+                            // Remove everything up to and including this object
+                            this.receiveBuffer = this.receiveBuffer.slice(i + 1);
+                            try {
+                                const response = JSON.parse(jsonStr);
+                                const pending = this.pendingCommands.get(response.correlationId);
+                                if (pending) {
+                                    this.pendingCommands.delete(response.correlationId);
+                                    pending.resolve(response);
+                                }
+                            }
+                            catch {
+                                // Malformed JSON — ignore (T-07-09 mitigation)
+                            }
+                            // Reset loop to scan remaining buffer from the start
+                            i = -1;
+                            startIdx = -1;
+                        }
                     }
                 }
+                // Keep only unprocessed data (partial object) in the buffer
+                if (startIdx > 0) {
+                    this.receiveBuffer = this.receiveBuffer.slice(startIdx);
+                }
+                else if (depth === 0) {
+                    // No open braces — discard any whitespace/newlines between objects
+                    this.receiveBuffer = '';
+                }
             });
             socket.once('connect', () => {
                 this.socket = socket;

package/dist/tools/editor/index.js

@@ -93,15 +93,20 @@ export function registerEditorTools(server, bridge) {
                 y: z.number().describe('Y coordinate in Unreal units (cm)'),
                 z: z.number().describe('Z coordinate in Unreal units (cm)'),
             }).describe('World-space location to spawn the actor at'),
+            label: z.string().optional().describe('Optional editor display label for the spawned actor'),
         }),
         annotations: {
             readOnlyHint: false,
             destructiveHint: false,
         },
     }, withKnownIssues('ue_spawn_actor', async (args) => {
+        const payload = { class_name: args.class_name, location: args.location };
+        if (args.label) {
+            payload['label'] = args.label;
+        }
         return sendOrDisconnect(_bridge, {
             type: 'actor.spawn',
-            payload: { class_name: args.class_name, location: args.location },
+            payload,
         });
     }));
     // --------------------------------------------------------------------------

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ultimate-unreal-engine-mcp",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "MCP server giving AI assistants full access to Unreal Engine 5.7 projects",
   "type": "module",
   "engines": {

package/unreal-plugin/MCPBridge/Source/MCPBridgeEditor/Private/MCPActorCommands.cpp

@@ -201,6 +201,13 @@ void RegisterActorCommands(FMCPCommandRouter& Router)
 		const FVector   Location(X, Y, Z);
 		const FRotator  Rotation(0.0, 0.0, 0.0);
 
+		// Optional label — sets the editor display name after spawn.
+		FString Label;
+		if (Payload->TryGetStringField(TEXT("label"), Label) && !Label.IsEmpty())
+		{
+			Params.Name = FName(*Label);
+		}
+
 		AActor* Spawned = World->SpawnActor<AActor>(ActorClass, Location, Rotation, Params);
 		if (!Spawned)
 		{
@@ -208,6 +215,12 @@ void RegisterActorCommands(FMCPCommandRouter& Router)
 			return;
 		}
 
+		// Apply explicit label if provided (overrides auto-generated label).
+		if (!Label.IsEmpty())
+		{
+			Spawned->SetActorLabel(Label);
+		}
+
 		TSharedPtr<FJsonObject> Data = MakeShared<FJsonObject>();
 		Data->SetStringField(TEXT("label"), Spawned->GetActorLabel());
 		Data->SetStringField(TEXT("id"),    Spawned->GetName());