Update plugin for Godot 4.3

Author: dalexeev

README.md

@@ -2,7 +2,7 @@
 
 An export plugin for stripping comments and "conditional compilation" of GDScript.
 
-Compatible with Godot **4.2.x**. Ready for production use. Please report if you find any bugs.
+Compatible with Godot **4.3.x**. Ready for production use. Please report if you find any bugs.
 
 ## How to use
 
@@ -14,6 +14,16 @@ The original scripts will not be changed, but in PCK/ZIP the scripts will be cha
 Use ZIP to check to check the result. If any errors occurred during the export,
 you will see them in the Output Log.
 
+> [!TIP]
+> Since 4.3, Godot supports exporting GDScript in binary mode out of the box. If your goal
+> is just to strip comments from the exported project, you may not need to use this plugin.
+
+> [!IMPORTANT]
+> If you are interested in features like conditional compilation and therefore want to use
+> this plugin, please note that it exports GDScript as text, regardless of the selected
+> export mode. But you should not lose much from this, the binary mode does not provide
+> significant obfuscation.
+
 ## Settings
 
 You can change the settings in the Options tab of the Export dialog:
@@ -25,14 +35,12 @@ You can change the settings in the Options tab of the Export dialog:
 
 </details>
 
-## Important note
-
-Each supported platform has certain standard feature tags (plus any custom tags you specify
-in the export preset). However, there are some standard tags that are not known in advance.
-See [godotengine/godot#76990](https://github.com/godotengine/godot/issues/76990) and
-[godotengine/godot#76996](https://github.com/godotengine/godot/pull/76996) for details.
-
-You can specify a list of feature tags that will be treated as dynamic in the export options.
+> [!NOTE]
+> Each supported platform has certain standard feature tags (plus any custom tags you specify
+> in the export preset). However, there are some standard tags that are not known in advance.
+> See [godotengine/godot#76990](https://github.com/godotengine/godot/issues/76990) and
+> [godotengine/godot#76996](https://github.com/godotengine/godot/pull/76996) for details.
+> You can specify a list of feature tags that will be treated as dynamic in the export options.
 
 ## Features
 

addons/gdscript_preprocessor/export_plugin.gd

@@ -1,14 +1,21 @@
 extends EditorExportPlugin
 
 
-@warning_ignore("inferred_declaration")
 const _Preprocessor = preload("./preprocessor.gd")
 
 var _preprocessor: _Preprocessor = _Preprocessor.new()
+var _addon_folder: String
+
+
+func _init() -> void:
+	var script: Script = get_script()
+	_addon_folder = script.resource_path.get_base_dir() + "/"
 
 
 func _get_name() -> String:
-	return "gdscript_preprocessor"
+	# Docs: "The plugins are sorted by name before exporting".
+	# See https://github.com/godotengine/godot/issues/93487.
+	return "AAA_gdscript_preprocessor"
 
 
 func _get_export_options(_platform: EditorExportPlatform) -> Array[Dictionary]:
@@ -72,7 +79,9 @@ func _export_file(path: String, type: String, _features: PackedStringArray) -> v
 	if type != "GDScript":
 		return
 
-	if _preprocessor.preprocess(FileAccess.get_file_as_string(path)):
+	if path.begins_with(_addon_folder):
+		skip()
+	elif _preprocessor.preprocess(FileAccess.get_file_as_string(path)):
 		skip()
 		add_file(path, _preprocessor.result.to_utf8_buffer(), false)
 	else:

addons/gdscript_preprocessor/plugin.gd

@@ -2,7 +2,6 @@
 extends EditorPlugin
 
 
-@warning_ignore("inferred_declaration")
 const _ExportPlugin = preload("./export_plugin.gd")
 
 var _export_plugin: _ExportPlugin = _ExportPlugin.new()

addons/gdscript_preprocessor/preprocessor.gd

@@ -84,6 +84,7 @@ const _VARIANT_DEFAULTS: Dictionary = {
 	"PackedVector2Array": "PackedVector2Array()",
 	"PackedVector3Array": "PackedVector3Array()",
 	"PackedColorArray":   "PackedColorArray()",
+	"PackedVector4Array": "PackedVector4Array()",
 }
 
 const _PARENS: Dictionary = {

project.godot

@@ -11,7 +11,7 @@ config_version=5
 [application]
 
 config/name="gdscript-preprocessor"
-config/features=PackedStringArray("4.2", "Forward Plus")
+config/features=PackedStringArray("4.3", "Forward Plus")
 config/icon="res://icon.png"
 
 [debug]

tests/features/directives.gd

@@ -11,9 +11,9 @@ var a2: int
 var a3: int
 #~endif # non_existent
 
-#~if OS.has_feature("linux")
+#~if OS.has_feature("pc")
 var a4: int
-#~endif # linux
+#~endif # pc
 
 #~endif # debug
 
@@ -26,13 +26,13 @@ var a7: int
 #~endif # debug
 #~endif # non_existent
 
-#~if OS.has_feature("non_existent") or OS.has_feature("linux")
+#~if OS.has_feature("non_existent") or OS.has_feature("pc")
 var a8: int
-#~endif # non_existent or linux
+#~endif # non_existent or pc
 
-#~if OS.has_feature("non_existent") and OS.has_feature("linux")
+#~if OS.has_feature("non_existent") and OS.has_feature("pc")
 var a9: int
-#~endif # non_existent and linux
+#~endif # non_existent and pc
 
 #~if OS.is_debug_build()
 var a10: int

tests/runner.gd

@@ -1,7 +1,6 @@
 extends SceneTree
 
 
-@warning_ignore("inferred_declaration")
 const _Preprocessor = preload("../addons/gdscript_preprocessor/preprocessor.gd")
 
 var _preprocessor: _Preprocessor = _Preprocessor.new()