gdharness 1.1.10 → 1.1.11

package/build/godot/addons/gdharness_editor/bridge_client.gd

@@ -243,7 +243,7 @@ func announced_url() -> String:
 	var file: FileAccess = FileAccess.open(ANNOUNCEMENT, FileAccess.READ)
 	if file == null:
 		return ""
-	var said: Variant = JSON.parse_string(file.get_as_text())
+	var said: Variant = Read.json_or_null(file.get_as_text())
 	file.close()
 	if not said is Dictionary:
 		return ""
@@ -481,7 +481,7 @@ func _on_reconnect_timer() -> void:
 
 
 func _handle_message(json_string: String) -> void:
-	var parsed: Variant = JSON.parse_string(json_string)
+	var parsed: Variant = Read.json_or_null(json_string)
 	if not parsed is Dictionary:
 		push_error("[gdharness] The server sent something that is not a message: %s" % json_string)
 		return

package/build/godot/addons/gdharness_editor/reading.gd

@@ -69,3 +69,16 @@ static func as_bool(value: Variant, fallback: bool = false) -> bool:
 		var text: String = value
 		return not text.is_empty()
 	return fallback
+
+
+## [param text] read as JSON, or null when it does not parse, without printing anything.
+##
+## [method JSON.parse_string] prints an engine error for text that does not parse, and that error
+## lands in the log of whatever is running it. A caller sending the word "five" where a number was
+## wanted was refused properly, and the game's own log then held an error it never made, so its
+## verdict read `clean: false`. Text that is not JSON is an answer here, not a fault.
+static func json_or_null(text: String) -> Variant:
+	var json: JSON = JSON.new()
+	if json.parse(text) != OK:
+		return null
+	return json.data

package/build/godot/addons/gdharness_editor/serialisation.gd

@@ -256,7 +256,7 @@ func fitted(value: Variant, type: int) -> Variant:
 
 	if rebuilt is String and type != TYPE_STRING:
 		var text: String = rebuilt
-		var parsed: Variant = JSON.parse_string(text)
+		var parsed: Variant = Read.json_or_null(text)
 		# A string that does not read as the type wanted is handed back as the string it is, so
 		# whoever asked can refuse it. type_convert answers 0 for "not an index" and true for any
 		# text at all, and a caller who sent a word got back a number they never sent: get_child

package/build/godot/addons/gdharness_editor/tools/animation_tools.gd

@@ -86,7 +86,7 @@ func _parse_json_maybe(value: Variant) -> Variant:
 	if typeof(value) != TYPE_STRING:
 		return value
 	var text: String = value
-	var parsed: Variant = JSON.parse_string(text)
+	var parsed: Variant = Read.json_or_null(text)
 	if parsed == null and text != "null":
 		return value
 	return parsed

package/build/godot/addons/gdharness_editor/tools/scene_tools.gd

@@ -127,7 +127,7 @@ func _parse_properties_arg(raw_properties: Variant) -> Dictionary:
 		var text: String = str(raw_properties)
 		if text.strip_edges().is_empty():
 			return {}
-		var parsed: Variant = JSON.parse_string(text)
+		var parsed: Variant = Read.json_or_null(text)
 		if typeof(parsed) == TYPE_DICTIONARY:
 			return parsed
 	return {}

package/build/godot/addons/gdharness_runtime/reading.gd

@@ -69,3 +69,16 @@ static func as_bool(value: Variant, fallback: bool = false) -> bool:
 		var text: String = value
 		return not text.is_empty()
 	return fallback
+
+
+## [param text] read as JSON, or null when it does not parse, without printing anything.
+##
+## [method JSON.parse_string] prints an engine error for text that does not parse, and that error
+## lands in the log of whatever is running it. A caller sending the word "five" where a number was
+## wanted was refused properly, and the game's own log then held an error it never made, so its
+## verdict read `clean: false`. Text that is not JSON is an answer here, not a fault.
+static func json_or_null(text: String) -> Variant:
+	var json: JSON = JSON.new()
+	if json.parse(text) != OK:
+		return null
+	return json.data

package/build/godot/addons/gdharness_runtime/serialisation.gd

@@ -256,7 +256,7 @@ func fitted(value: Variant, type: int) -> Variant:
 
 	if rebuilt is String and type != TYPE_STRING:
 		var text: String = rebuilt
-		var parsed: Variant = JSON.parse_string(text)
+		var parsed: Variant = Read.json_or_null(text)
 		# A string that does not read as the type wanted is handed back as the string it is, so
 		# whoever asked can refuse it. type_convert answers 0 for "not an index" and true for any
 		# text at all, and a caller who sent a word got back a number they never sent: get_child

package/build/godot/operations/import_pipeline.gd

@@ -645,7 +645,7 @@ static func _images_of(scene_path: String) -> Array[String]:
 				text = bytes.slice(20, mini(20 + length, bytes.size())).get_string_from_utf8()
 	if text.is_empty():
 		return images
-	var parsed: Variant = JSON.parse_string(text)
+	var parsed: Variant = Read.json_or_null(text)
 	if not parsed is Dictionary:
 		return images
 	var document: Dictionary = parsed

package/build/godot/operations/reading.gd

@@ -69,3 +69,16 @@ static func as_bool(value: Variant, fallback: bool = false) -> bool:
 		var text: String = value
 		return not text.is_empty()
 	return fallback
+
+
+## [param text] read as JSON, or null when it does not parse, without printing anything.
+##
+## [method JSON.parse_string] prints an engine error for text that does not parse, and that error
+## lands in the log of whatever is running it. A caller sending the word "five" where a number was
+## wanted was refused properly, and the game's own log then held an error it never made, so its
+## verdict read `clean: false`. Text that is not JSON is an answer here, not a fault.
+static func json_or_null(text: String) -> Variant:
+	var json: JSON = JSON.new()
+	if json.parse(text) != OK:
+		return null
+	return json.data

package/build/godot/operations/serialisation.gd

@@ -256,7 +256,7 @@ func fitted(value: Variant, type: int) -> Variant:
 
 	if rebuilt is String and type != TYPE_STRING:
 		var text: String = rebuilt
-		var parsed: Variant = JSON.parse_string(text)
+		var parsed: Variant = Read.json_or_null(text)
 		# A string that does not read as the type wanted is handed back as the string it is, so
 		# whoever asked can refuse it. type_convert answers 0 for "not an index" and true for any
 		# text at all, and a caller who sent a word got back a number they never sent: get_child

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gdharness",
-  "version": "1.1.10",
+  "version": "1.1.11",
   "mcpName": "io.github.Aureliolo/gdharness",
   "description": "A harness for driving a Godot 4 project from an agent: editor addons, a runtime bridge into the running game, an MCP server in front of them, and a CLI that installs and diagnoses the Godot side.",
   "type": "module",