gopeak 2.3.4 → 2.3.6

package/README.md

@@ -39,6 +39,14 @@ npm install -g gopeak
 gopeak
 ```
 
+Optional shell hooks for update notifications are now **opt-in**:
+
+```bash
+gopeak setup
+```
+
+> `gopeak setup` only modifies supported bash/zsh rc files when you run it explicitly. `npm install` no longer installs shell hooks automatically.
+
 ### 2) Add MCP client config
 
 ```json
@@ -136,6 +144,20 @@ In `compact` mode, 78 additional tools are organized into **22 groups** that act
    > "Use `tool.groups` to reset all active groups."
 
 The server sends `notifications/tools/list_changed` so MCP clients (Claude Code, Claude Desktop) automatically refresh the tool list.
+If your MCP client caches tools aggressively and does not refresh after activation, reconnect the client or call the newly activated tool directly once to force a fresh `tools/list` round-trip.
+
+### Typed property values for scene tools
+
+Bridge-backed scene tools (`add_node`, `set_node_properties`) now coerce common vector payloads such as `{ "x": 100, "y": 200 }` and `[100, 200]` for typed properties like `position` and `scale`. Tagged values are still the safest cross-tool form:
+
+```json
+{
+  "position": { "type": "Vector2", "x": 100, "y": 200 },
+  "scale": { "type": "Vector2", "x": 2, "y": 2 }
+}
+```
+
+The internal headless serializer uses `_type`, but MCP callers should prefer `type` when they need an explicit cross-tool Godot value tag.
 
 ### Don't worry about tokens
 
@@ -168,6 +190,12 @@ npm install -g gopeak
 gopeak
 ```
 
+Optional shell hooks for update notifications remain available via:
+
+```bash
+gopeak setup
+```
+
 ### C) From source
 
 ```bash
@@ -201,10 +229,12 @@ GitHub Actions runs on push/PR and executes:
 2. `npx tsc --noEmit`
 3. `npm run smoke`
 
-Run the same checks locally:
+Run the same checks locally before opening a PR:
 
 ```bash
 npm run ci
+npm run test:dynamic-groups
+npm run test:integration
 ```
 
 ---
@@ -337,6 +367,7 @@ Visualize your entire project architecture with `visualizer.map` (`map_project`
 - **Project path invalid** → confirm `project.godot` exists
 - **Runtime tools not working** → install/enable runtime addon plugin
 - **Need a tool that is not visible** → run `tool.catalog` to search and auto-activate matching groups, or use `tool.groups` to activate a specific group
+- **`get_editor_status` says disconnected while the Godot editor shows connected** → check whether another `gopeak`/MCP server instance already owns bridge port `6505`; the status payload now reports the startup error and suggests stopping duplicate servers
 
 ---
 

package/build/addon/godot_mcp_editor/tools/scene_tools.gd

@@ -81,49 +81,92 @@ func _find_node(root: Node, path: String) -> Node:
 	return root.get_node_or_null(path)
 
 
-func _parse_value(value):
-	if typeof(value) == TYPE_DICTIONARY and value.has("type"):
-		match value["type"]:
-			"Vector2":
-				return Vector2(value.get("x", 0), value.get("y", 0))
-			"Vector3":
-				return Vector3(value.get("x", 0), value.get("y", 0), value.get("z", 0))
-			"Color":
-				return Color(value.get("r", 1), value.get("g", 1), value.get("b", 1), value.get("a", 1))
-			"Vector2i":
-				return Vector2i(value.get("x", 0), value.get("y", 0))
-			"Vector3i":
-				return Vector3i(value.get("x", 0), value.get("y", 0), value.get("z", 0))
-			"Rect2":
-				return Rect2(value.get("x", 0), value.get("y", 0), value.get("width", 0), value.get("height", 0))
-			"Transform2D":
-				if value.has("x") and value.has("y") and value.has("origin"):
-					var xx: Dictionary = value["x"]
-					var yy: Dictionary = value["y"]
-					var oo: Dictionary = value["origin"]
-					return Transform2D(
-						Vector2(xx.get("x", 1), xx.get("y", 0)),
-						Vector2(yy.get("x", 0), yy.get("y", 1)),
-						Vector2(oo.get("x", 0), oo.get("y", 0))
-					)
-			"Transform3D":
-				if value.has("basis") and value.has("origin"):
-					var b: Dictionary = value["basis"]
-					var o: Dictionary = value["origin"]
-					var basis := Basis(
-						Vector3(b.get("x", {}).get("x", 1), b.get("x", {}).get("y", 0), b.get("x", {}).get("z", 0)),
-						Vector3(b.get("y", {}).get("x", 0), b.get("y", {}).get("y", 1), b.get("y", {}).get("z", 0)),
-						Vector3(b.get("z", {}).get("x", 0), b.get("z", {}).get("y", 0), b.get("z", {}).get("z", 1))
-					)
-					return Transform3D(basis, Vector3(o.get("x", 0), o.get("y", 0), o.get("z", 0)))
-			"NodePath":
-				return NodePath(value.get("path", ""))
-			"Resource":
-				var resource_path: String = str(value.get("path", ""))
-				if resource_path.is_empty():
-					return null
-				return load(resource_path)
+func _parse_value(value, expected_type: int = TYPE_NIL):
+	if typeof(value) == TYPE_DICTIONARY:
+		var type_tag := ""
+		if value.has("type"):
+			type_tag = str(value["type"])
+		elif value.has("_type"):
+			type_tag = str(value["_type"])
+
+		if not type_tag.is_empty():
+			match type_tag:
+				"Vector2":
+					return Vector2(value.get("x", 0), value.get("y", 0))
+				"Vector3":
+					return Vector3(value.get("x", 0), value.get("y", 0), value.get("z", 0))
+				"Color":
+					return Color(value.get("r", 1), value.get("g", 1), value.get("b", 1), value.get("a", 1))
+				"Vector2i":
+					return Vector2i(value.get("x", 0), value.get("y", 0))
+				"Vector3i":
+					return Vector3i(value.get("x", 0), value.get("y", 0), value.get("z", 0))
+				"Rect2":
+					return Rect2(value.get("x", 0), value.get("y", 0), value.get("width", 0), value.get("height", 0))
+				"Transform2D":
+					if value.has("x") and value.has("y") and value.has("origin"):
+						var xx: Dictionary = value["x"]
+						var yy: Dictionary = value["y"]
+						var oo: Dictionary = value["origin"]
+						return Transform2D(
+							Vector2(xx.get("x", 1), xx.get("y", 0)),
+							Vector2(yy.get("x", 0), yy.get("y", 1)),
+							Vector2(oo.get("x", 0), oo.get("y", 0))
+						)
+				"Transform3D":
+					if value.has("basis") and value.has("origin"):
+						var b: Dictionary = value["basis"]
+						var o: Dictionary = value["origin"]
+						var basis := Basis(
+							Vector3(b.get("x", {}).get("x", 1), b.get("x", {}).get("y", 0), b.get("x", {}).get("z", 0)),
+							Vector3(b.get("y", {}).get("x", 0), b.get("y", {}).get("y", 1), b.get("y", {}).get("z", 0)),
+							Vector3(b.get("z", {}).get("x", 0), b.get("z", {}).get("y", 0), b.get("z", {}).get("z", 1))
+						)
+						return Transform3D(basis, Vector3(o.get("x", 0), o.get("y", 0), o.get("z", 0)))
+				"NodePath":
+					return NodePath(value.get("path", ""))
+				"Resource":
+					var resource_path: String = str(value.get("path", ""))
+					if resource_path.is_empty():
+						return null
+					return load(resource_path)
+
+		match expected_type:
+			TYPE_VECTOR2:
+				if value.has("x") and value.has("y"):
+					return Vector2(value.get("x", 0), value.get("y", 0))
+			TYPE_VECTOR2I:
+				if value.has("x") and value.has("y"):
+					return Vector2i(value.get("x", 0), value.get("y", 0))
+			TYPE_VECTOR3:
+				if value.has("x") and value.has("y") and value.has("z"):
+					return Vector3(value.get("x", 0), value.get("y", 0), value.get("z", 0))
+			TYPE_VECTOR3I:
+				if value.has("x") and value.has("y") and value.has("z"):
+					return Vector3i(value.get("x", 0), value.get("y", 0), value.get("z", 0))
+			TYPE_COLOR:
+				if value.has("r") and value.has("g") and value.has("b"):
+					return Color(value.get("r", 1), value.get("g", 1), value.get("b", 1), value.get("a", 1))
+			TYPE_RECT2:
+				if value.has("x") and value.has("y") and value.has("width") and value.has("height"):
+					return Rect2(value.get("x", 0), value.get("y", 0), value.get("width", 0), value.get("height", 0))
+			TYPE_NODE_PATH:
+				if value.has("path"):
+					return NodePath(value.get("path", ""))
 	if typeof(value) == TYPE_ARRAY:
+		match expected_type:
+			TYPE_VECTOR2:
+				if value.size() >= 2:
+					return Vector2(value[0], value[1])
+			TYPE_VECTOR2I:
+				if value.size() >= 2:
+					return Vector2i(value[0], value[1])
+			TYPE_VECTOR3:
+				if value.size() >= 3:
+					return Vector3(value[0], value[1], value[2])
+			TYPE_VECTOR3I:
+				if value.size() >= 3:
+					return Vector3i(value[0], value[1], value[2])
 		var result: Array = []
 		for item in value:
 			result.append(_parse_value(item))
@@ -131,6 +174,13 @@ func _parse_value(value):
 	return value
 
 
+func _get_property_type(node: Node, prop_name: String) -> int:
+	for prop in node.get_property_list():
+		if str(prop.get("name", "")) == prop_name:
+			return int(prop.get("type", TYPE_NIL))
+	return TYPE_NIL
+
+
 func _serialize_value(value) -> Variant:
 	match typeof(value):
 		TYPE_VECTOR2:
@@ -174,7 +224,8 @@ func _serialize_value(value) -> Variant:
 
 func _set_node_properties(node: Node, properties: Dictionary) -> void:
 	for prop_name in properties:
-		var val = _parse_value(properties[prop_name])
+		var expected_type := _get_property_type(node, str(prop_name))
+		var val = _parse_value(properties[prop_name], expected_type)
 		node.set(prop_name, val)
 
 

package/build/addon/godot_mcp_runtime/mcp_runtime_autoload.gd

@@ -45,6 +45,9 @@ func _process(_delta: float) -> void:
 			continue
 		
 		client.poll()
+		if client.get_status() != StreamPeerTCP.STATUS_CONNECTED:
+			clients_to_remove.append(client)
+			continue
 		var available = client.get_available_bytes()
 		if available > 0:
 			var data = client.get_utf8_string(available)

package/build/godot-bridge.js

@@ -335,10 +335,13 @@ export class GodotBridge extends EventEmitter {
         nextSocket.on('close', (code, reasonBuffer) => {
             const reason = reasonBuffer.toString();
             this.log('warn', `Godot disconnected (code=${code}, reason=${reason || 'none'})`);
-            this.handleDisconnect(new Error('Godot disconnected during request'));
+            this.handleDisconnect(nextSocket, new Error('Godot disconnected during request'));
         });
         nextSocket.on('error', (error) => {
             this.log('error', `WebSocket error: ${error.message}`);
+            if (nextSocket.readyState === WebSocket.CLOSED || nextSocket.readyState === WebSocket.CLOSING) {
+                this.handleDisconnect(nextSocket, error);
+            }
         });
     }
     handleRawMessage(data) {
@@ -463,7 +466,11 @@ export class GodotBridge extends EventEmitter {
         clearInterval(this.pingInterval);
         this.pingInterval = null;
     }
-    handleDisconnect(reason) {
+    handleDisconnect(disconnectedSocket, reason) {
+        if (disconnectedSocket && this.socket && disconnectedSocket !== this.socket) {
+            this.log('debug', 'Ignoring stale Godot socket disconnect event');
+            return;
+        }
         this.stopKeepalive();
         this.socket = null;
         this.connectionInfo = null;