From 0958c5a2996ecd9781ecc900c814e6162d74524a Mon Sep 17 00:00:00 2001
From: Hugo Locurcio <hugo.locurcio@hugo.pro>
Date: Wed, 3 Nov 2021 15:39:32 +0100
Subject: [PATCH] Remove support for built-in scripts in the editor

Built-in scripts have many limitations that users tend to discover
late in their project. This causes a lot of pain as scripts must
be replaced by their non-built-in variants one by one.

Note that this only unexposes the ability to create and edit built-in
scripts from the editor. It's still possible to save a scene that contains
executable built-in scripts using PackedScene or external tools.

This is a compatibility-breaking change, as built-in scripts need
to be turned into non-built-in-scripts before upgrading a project
to Godot 4.0.
---
 core/object/script_language.h                 |   1 -
 editor/editor_data.cpp                        |   4 -
 editor/editor_inspector.cpp                   |   2 +-
 editor/filesystem_dock.cpp                    |   2 +-
 editor/plugins/script_editor_plugin.cpp       |  75 +------------
 editor/plugins/script_editor_plugin.h         |   2 -
 editor/script_create_dialog.cpp               | 105 +++---------------
 editor/script_create_dialog.h                 |   8 +-
 .../include/pluginscript/godot_pluginscript.h |   1 -
 .../gdnative/nativescript/nativescript.cpp    |   4 -
 modules/gdnative/nativescript/nativescript.h  |   1 -
 .../pluginscript/pluginscript_language.cpp    |   4 -
 .../pluginscript/pluginscript_language.h      |   1 -
 modules/gdscript/gdscript.cpp                 |   6 +-
 modules/mono/csharp_script.cpp                |   4 -
 modules/mono/csharp_script.h                  |   3 -
 modules/visual_script/visual_script.cpp       |   4 -
 modules/visual_script/visual_script.h         |   1 -
 18 files changed, 29 insertions(+), 199 deletions(-)

diff --git a/core/object/script_language.h b/core/object/script_language.h
index 8d76cbf47956..0b49a6f37b2b 100644
--- a/core/object/script_language.h
+++ b/core/object/script_language.h
@@ -286,7 +286,6 @@ class ScriptLanguage {
 	virtual String validate_path(const String &p_path) const { return ""; }
 	virtual Script *create_script() const = 0;
 	virtual bool has_named_classes() const = 0;
-	virtual bool supports_builtin_mode() const = 0;
 	virtual bool supports_documentation() const { return false; }
 	virtual bool can_inherit_from_file() const { return false; }
 	virtual int find_function(const String &p_function, const String &p_code) const = 0;
diff --git a/editor/editor_data.cpp b/editor/editor_data.cpp
index aee9c2100733..f0948e506dbf 100644
--- a/editor/editor_data.cpp
+++ b/editor/editor_data.cpp
@@ -566,10 +566,6 @@ void EditorData::remove_scene(int p_idx) {
 		current_edited_scene--;
 	}
 
-	if (edited_scene[p_idx].path != String()) {
-		ScriptEditor::get_singleton()->close_builtin_scripts_from_scene(edited_scene[p_idx].path);
-	}
-
 	edited_scene.remove(p_idx);
 }
 
diff --git a/editor/editor_inspector.cpp b/editor/editor_inspector.cpp
index 9f049a0e5820..852439ad232b 100644
--- a/editor/editor_inspector.cpp
+++ b/editor/editor_inspector.cpp
@@ -2506,7 +2506,7 @@ void EditorInspector::update_tree() {
 				}
 			}
 			if (category->icon.is_null()) {
-				if (type != String()) { // Can happen for built-in scripts.
+				if (type != String()) {
 					category->icon = EditorNode::get_singleton()->get_class_icon(type, "Object");
 				}
 			}
diff --git a/editor/filesystem_dock.cpp b/editor/filesystem_dock.cpp
index f2a3aa3b4445..39d8d4c453c3 100644
--- a/editor/filesystem_dock.cpp
+++ b/editor/filesystem_dock.cpp
@@ -1942,7 +1942,7 @@ void FileSystemDock::_file_option(int p_option, const Vector<String> &p_selected
 			if (!fpath.ends_with("/")) {
 				fpath = fpath.get_base_dir();
 			}
-			make_script_dialog->config("Node", fpath.plus_file("new_script.gd"), false, false);
+			make_script_dialog->config("Node", fpath.plus_file("new_script.gd"), false);
 			make_script_dialog->popup_centered();
 		} break;
 
diff --git a/editor/plugins/script_editor_plugin.cpp b/editor/plugins/script_editor_plugin.cpp
index 3f98560a2f0d..e984a92ef4cf 100644
--- a/editor/plugins/script_editor_plugin.cpp
+++ b/editor/plugins/script_editor_plugin.cpp
@@ -227,12 +227,6 @@ void ScriptEditorBase::_bind_methods() {
 	ADD_SIGNAL(MethodInfo("replace_in_files_requested", PropertyInfo(Variant::STRING, "text")));
 }
 
-static bool _is_built_in_script(Script *p_script) {
-	String path = p_script->get_path();
-
-	return path.find("::") != -1;
-}
-
 class EditorScriptCodeCompletionCache : public ScriptCodeCompletionCache {
 	struct Cache {
 		uint64_t time_loaded = 0;
@@ -1159,7 +1153,7 @@ void ScriptEditor::_menu_option(int p_option) {
 	ScriptEditorBase *current = _get_current_editor();
 	switch (p_option) {
 		case FILE_NEW: {
-			script_create_dialog->config("Node", "new_script", false, false);
+			script_create_dialog->config("Node", "new_script", false);
 			script_create_dialog->popup_centered();
 		} break;
 		case FILE_NEW_TEXTFILE: {
@@ -1205,23 +1199,8 @@ void ScriptEditor::_menu_option(int p_option) {
 
 			List<String> extensions;
 			ResourceLoader::get_recognized_extensions_for_type("Script", &extensions);
-			bool built_in = !path.is_resource_file();
-
-			if (extensions.find(path.get_extension()) || built_in) {
-				if (built_in) {
-					String res_path = path.get_slice("::", 0);
-					if (ResourceLoader::get_resource_type(res_path) == "PackedScene") {
-						if (!EditorNode::get_singleton()->is_scene_open(res_path)) {
-							EditorNode::get_singleton()->load_scene(res_path);
-							script_editor->call_deferred(SNAME("_menu_option"), p_option);
-							previous_scripts.push_back(path); //repeat the operation
-							return;
-						}
-					} else {
-						EditorNode::get_singleton()->load_resource(res_path);
-					}
-				}
 
+			if (extensions.find(path.get_extension())) {
 				Ref<Script> scr = ResourceLoader::load(path);
 				if (!scr.is_valid()) {
 					editor->show_warning(TTR("Could not load file at:") + "\n\n" + path, TTR("Error!"));
@@ -1614,24 +1593,6 @@ bool ScriptEditor::can_take_away_focus() const {
 	}
 }
 
-void ScriptEditor::close_builtin_scripts_from_scene(const String &p_scene) {
-	for (int i = 0; i < tab_container->get_child_count(); i++) {
-		ScriptEditorBase *se = Object::cast_to<ScriptEditorBase>(tab_container->get_child(i));
-
-		if (se) {
-			Ref<Script> script = se->get_edited_resource();
-			if (script == nullptr || !script.is_valid()) {
-				continue;
-			}
-
-			if (script->get_path().find("::") != -1 && script->get_path().begins_with(p_scene)) { //is an internal script and belongs to scene being closed
-				_close_tab(i);
-				i--;
-			}
-		}
-	}
-}
-
 void ScriptEditor::edited_scene_changed() {
 	_update_modified_scripts_for_external_editor();
 }
@@ -1818,9 +1779,7 @@ void ScriptEditor::_update_members_overview() {
 	}
 
 	String path = se->get_edited_resource()->get_path();
-	bool built_in = !path.is_resource_file();
-	String name = built_in ? path.get_file() : se->get_name();
-	filename->set_text(name);
+	filename->set_text(se->get_name());
 }
 
 void ScriptEditor::_update_help_overview_visibility() {
@@ -1932,20 +1891,7 @@ void ScriptEditor::_update_script_names() {
 				// to update original path to previously edited resource.
 				se->set_meta("_edit_res_path", path);
 			}
-			bool built_in = !path.is_resource_file();
-			String name;
-
-			if (built_in) {
-				name = path.get_file();
-				const String &resource_name = se->get_edited_resource()->get_name();
-				if (resource_name != "") {
-					// If the built-in script has a custom resource name defined,
-					// display the built-in script name as follows: `ResourceName (scene_file.tscn)`
-					name = vformat("%s (%s)", resource_name, name.substr(0, name.find("::", 0)));
-				}
-			} else {
-				name = se->get_name();
-			}
+			const String name = se->get_name();
 
 			_ScriptEditorItemData sd;
 			sd.icon = icon;
@@ -2570,9 +2516,7 @@ void ScriptEditor::_add_callback(Object *p_obj, const String &p_function, const
 		script_list->select(script_list->find_metadata(i));
 
 		// Save the current script so the changes can be picked up by an external editor.
-		if (!_is_built_in_script(script.ptr())) { // But only if it's not built-in script.
-			save_current_script();
-		}
+		save_current_script();
 
 		break;
 	}
@@ -3858,15 +3802,6 @@ void ScriptEditorPlugin::edit(Object *p_object) {
 		Script *p_script = Object::cast_to<Script>(p_object);
 		String res_path = p_script->get_path().get_slice("::", 0);
 
-		if (_is_built_in_script(p_script)) {
-			if (ResourceLoader::get_resource_type(res_path) == "PackedScene") {
-				if (!EditorNode::get_singleton()->is_scene_open(res_path)) {
-					EditorNode::get_singleton()->load_scene(res_path);
-				}
-			} else {
-				EditorNode::get_singleton()->load_resource(res_path);
-			}
-		}
 		script_editor->edit(p_script);
 	} else if (Object::cast_to<TextFile>(p_object)) {
 		script_editor->edit(Object::cast_to<TextFile>(p_object));
diff --git a/editor/plugins/script_editor_plugin.h b/editor/plugins/script_editor_plugin.h
index 2b0bdfd109bb..2b2a42580d28 100644
--- a/editor/plugins/script_editor_plugin.h
+++ b/editor/plugins/script_editor_plugin.h
@@ -500,8 +500,6 @@ class ScriptEditor : public PanelContainer {
 	void notify_script_close(const Ref<Script> &p_script);
 	void notify_script_changed(const Ref<Script> &p_script);
 
-	void close_builtin_scripts_from_scene(const String &p_scene);
-
 	void goto_help(const String &p_desc) { _help_class_goto(p_desc); }
 	void update_doc(const String &p_name);
 
diff --git a/editor/script_create_dialog.cpp b/editor/script_create_dialog.cpp
index 1e19d9bd47e1..063e04de46f6 100644
--- a/editor/script_create_dialog.cpp
+++ b/editor/script_create_dialog.cpp
@@ -78,9 +78,7 @@ void ScriptCreateDialog::_path_hbox_sorted() {
 		int filename_start_pos = initial_bp.rfind("/") + 1;
 		int filename_end_pos = initial_bp.length();
 
-		if (!is_built_in) {
-			file_path->select(filename_start_pos, filename_end_pos);
-		}
+		file_path->select(filename_start_pos, filename_end_pos);
 
 		// First set cursor to the end of line to scroll LineEdit view
 		// to the right and then set the actual cursor position.
@@ -91,11 +89,7 @@ void ScriptCreateDialog::_path_hbox_sorted() {
 	}
 }
 
-bool ScriptCreateDialog::_can_be_built_in() {
-	return (supports_built_in && built_in_enabled);
-}
-
-void ScriptCreateDialog::config(const String &p_base_name, const String &p_base_path, bool p_built_in_enabled, bool p_load_enabled) {
+void ScriptCreateDialog::config(const String &p_base_name, const String &p_base_path, bool p_load_enabled) {
 	class_name->set_text("");
 	class_name->deselect();
 	parent_name->set_text(p_base_name);
@@ -111,7 +105,6 @@ void ScriptCreateDialog::config(const String &p_base_name, const String &p_base_
 	}
 	file_path->deselect();
 
-	built_in_enabled = p_built_in_enabled;
 	load_enabled = p_load_enabled;
 
 	_lang_changed(current_language);
@@ -315,15 +308,13 @@ void ScriptCreateDialog::_create_new() {
 		}
 	}
 
-	if (!is_built_in) {
-		String lpath = ProjectSettings::get_singleton()->localize_path(file_path->get_text());
-		scr->set_path(lpath);
-		Error err = ResourceSaver::save(lpath, scr, ResourceSaver::FLAG_CHANGE_PATH);
-		if (err != OK) {
-			alert->set_text(TTR("Error - Could not create script in filesystem."));
-			alert->popup_centered();
-			return;
-		}
+	String lpath = ProjectSettings::get_singleton()->localize_path(file_path->get_text());
+	scr->set_path(lpath);
+	Error err = ResourceSaver::save(lpath, scr, ResourceSaver::FLAG_CHANGE_PATH);
+	if (err != OK) {
+		alert->set_text(TTR("Error - Could not create script in filesystem."));
+		alert->popup_centered();
+		return;
 	}
 
 	emit_signal(SNAME("script_created"), scr);
@@ -348,10 +339,6 @@ void ScriptCreateDialog::_lang_changed(int l) {
 
 	has_named_classes = language->has_named_classes();
 	can_inherit_from_file = language->can_inherit_from_file();
-	supports_built_in = language->supports_builtin_mode();
-	if (!supports_built_in) {
-		is_built_in = false;
-	}
 
 	String selected_ext = "." + language->get_extension();
 	String path = file_path->get_text();
@@ -504,17 +491,6 @@ void ScriptCreateDialog::_update_script_templates(const String &p_extension) {
 	}
 }
 
-void ScriptCreateDialog::_built_in_pressed() {
-	if (internal->is_pressed()) {
-		is_built_in = true;
-		is_new_script_created = true;
-	} else {
-		is_built_in = false;
-		_path_changed(file_path->get_text());
-	}
-	_update_dialog();
-}
-
 void ScriptCreateDialog::_browse_path(bool browse_parent, bool p_save) {
 	is_browsing_parent = browse_parent;
 
@@ -572,10 +548,6 @@ void ScriptCreateDialog::_browse_class_in_tree() {
 }
 
 void ScriptCreateDialog::_path_changed(const String &p_path) {
-	if (is_built_in) {
-		return;
-	}
-
 	is_path_valid = false;
 	is_new_script_created = true;
 
@@ -628,7 +600,7 @@ void ScriptCreateDialog::_update_dialog() {
 
 	// Is script path/name valid (order from top to bottom)?
 
-	if (!is_built_in && !is_path_valid) {
+	if (!is_path_valid) {
 		_msg_script_valid(false, TTR("Invalid path."));
 		script_ok = false;
 	}
@@ -662,41 +634,18 @@ void ScriptCreateDialog::_update_dialog() {
 		class_name->set_text("");
 	}
 
-	// Is script Built-in?
-
-	if (is_built_in) {
-		file_path->set_editable(false);
-		path_button->set_disabled(true);
-		re_check_path = true;
-	} else {
-		file_path->set_editable(true);
-		path_button->set_disabled(false);
-		if (re_check_path) {
-			re_check_path = false;
-			_path_changed(file_path->get_text());
-		}
-	}
-
-	if (!_can_be_built_in()) {
-		internal->set_pressed(false);
+	file_path->set_editable(true);
+	path_button->set_disabled(false);
+	if (re_check_path) {
+		re_check_path = false;
+		_path_changed(file_path->get_text());
 	}
-	internal->set_disabled(!_can_be_built_in());
 
 	// Is Script created or loaded from existing file?
-
-	builtin_warning_label->set_visible(is_built_in);
-
 	// Check if the script name is the same as the parent class.
-	// This warning isn't relevant if the script is built-in.
-	script_name_warning_label->set_visible(!is_built_in && _get_class_name() == parent_name->get_text());
+	script_name_warning_label->set_visible(_get_class_name() == parent_name->get_text());
 
-	if (is_built_in) {
-		get_ok_button()->set_text(TTR("Create"));
-		parent_name->set_editable(true);
-		parent_search_button->set_disabled(false);
-		parent_browse_button->set_disabled(!can_inherit_from_file);
-		_msg_path_valid(true, TTR("Built-in script (into scene file)."));
-	} else if (is_new_script_created) {
+	if (is_new_script_created) {
 		// New script created.
 
 		get_ok_button()->set_text(TTR("Create"));
@@ -739,7 +688,7 @@ void ScriptCreateDialog::_update_dialog() {
 }
 
 void ScriptCreateDialog::_bind_methods() {
-	ClassDB::bind_method(D_METHOD("config", "inherits", "path", "built_in_enabled", "load_enabled"), &ScriptCreateDialog::config, DEFVAL(true), DEFVAL(true));
+	ClassDB::bind_method(D_METHOD("config", "inherits", "path", "load_enabled"), &ScriptCreateDialog::config, DEFVAL(true));
 
 	ADD_SIGNAL(MethodInfo("script_created", PropertyInfo(Variant::OBJECT, "script", PROPERTY_HINT_RESOURCE_TYPE, "Script")));
 }
@@ -760,13 +709,6 @@ ScriptCreateDialog::ScriptCreateDialog() {
 	path_error_label = memnew(Label);
 	vb->add_child(path_error_label);
 
-	builtin_warning_label = memnew(Label);
-	builtin_warning_label->set_text(
-			TTR("Note: Built-in scripts have some limitations and can't be edited using an external editor."));
-	vb->add_child(builtin_warning_label);
-	builtin_warning_label->set_autowrap_mode(Label::AUTOWRAP_WORD_SMART);
-	builtin_warning_label->hide();
-
 	script_name_warning_label = memnew(Label);
 	script_name_warning_label->set_text(
 			TTR("Warning: Having the script name be the same as a built-in type is usually not desired."));
@@ -849,14 +791,6 @@ ScriptCreateDialog::ScriptCreateDialog() {
 	gc->add_child(template_menu);
 	template_menu->connect("item_selected", callable_mp(this, &ScriptCreateDialog::_template_changed));
 
-	/* Built-in Script */
-
-	internal = memnew(CheckBox);
-	internal->set_text(TTR("On"));
-	internal->connect("pressed", callable_mp(this, &ScriptCreateDialog::_built_in_pressed));
-	gc->add_child(memnew(Label(TTR("Built-in Script:"))));
-	gc->add_child(internal);
-
 	/* Path */
 
 	hb = memnew(HBoxContainer);
@@ -898,10 +832,7 @@ ScriptCreateDialog::ScriptCreateDialog() {
 	is_path_valid = false;
 
 	has_named_classes = false;
-	supports_built_in = false;
 	can_inherit_from_file = false;
-	is_built_in = false;
-	built_in_enabled = true;
 	load_enabled = true;
 
 	is_new_script_created = true;
diff --git a/editor/script_create_dialog.h b/editor/script_create_dialog.h
index 7c2ef1e15066..10cd075e9efc 100644
--- a/editor/script_create_dialog.h
+++ b/editor/script_create_dialog.h
@@ -48,7 +48,6 @@ class ScriptCreateDialog : public ConfirmationDialog {
 	LineEdit *class_name;
 	Label *error_label;
 	Label *path_error_label;
-	Label *builtin_warning_label;
 	Label *script_name_warning_label;
 	PanelContainer *status_panel;
 	LineEdit *parent_name;
@@ -70,12 +69,9 @@ class ScriptCreateDialog : public ConfirmationDialog {
 	bool is_new_script_created;
 	bool is_path_valid;
 	bool has_named_classes;
-	bool supports_built_in;
 	bool can_inherit_from_file;
 	bool is_parent_name_valid;
 	bool is_class_name_valid;
-	bool is_built_in;
-	bool built_in_enabled;
 	bool load_enabled;
 	int current_language;
 	int default_language;
@@ -102,11 +98,9 @@ class ScriptCreateDialog : public ConfirmationDialog {
 	String base_type;
 
 	void _path_hbox_sorted();
-	bool _can_be_built_in();
 	void _path_changed(const String &p_path = String());
 	void _path_submitted(const String &p_path = String());
 	void _lang_changed(int l = 0);
-	void _built_in_pressed();
 	bool _validate_parent(const String &p_string);
 	bool _validate_class(const String &p_string);
 	String _validate_path(const String &p_path, bool p_file_must_exist);
@@ -130,7 +124,7 @@ class ScriptCreateDialog : public ConfirmationDialog {
 	static void _bind_methods();
 
 public:
-	void config(const String &p_base_name, const String &p_base_path, bool p_built_in_enabled = true, bool p_load_enabled = true);
+	void config(const String &p_base_name, const String &p_base_path, bool p_load_enabled = true);
 	void set_inheritance_base_type(const String &p_base);
 	ScriptCreateDialog();
 };
diff --git a/modules/gdnative/include/pluginscript/godot_pluginscript.h b/modules/gdnative/include/pluginscript/godot_pluginscript.h
index 02ee4066d08d..ec709a6a679b 100644
--- a/modules/gdnative/include/pluginscript/godot_pluginscript.h
+++ b/modules/gdnative/include/pluginscript/godot_pluginscript.h
@@ -128,7 +128,6 @@ typedef struct {
 	const char **comment_delimiters; // nullptr terminated array
 	const char **string_delimiters; // nullptr terminated array
 	godot_bool has_named_classes;
-	godot_bool supports_builtin_mode;
 	godot_bool can_inherit_from_file;
 
 	godot_string (*get_template_source_code)(godot_pluginscript_language_data *p_data, const godot_string *p_class_name, const godot_string *p_base_class_name);
diff --git a/modules/gdnative/nativescript/nativescript.cpp b/modules/gdnative/nativescript/nativescript.cpp
index 368eb67fa6cb..98b0a5bf6b2d 100644
--- a/modules/gdnative/nativescript/nativescript.cpp
+++ b/modules/gdnative/nativescript/nativescript.cpp
@@ -1064,10 +1064,6 @@ bool NativeScriptLanguage::has_named_classes() const {
 	return true;
 }
 
-bool NativeScriptLanguage::supports_builtin_mode() const {
-	return true;
-}
-
 int NativeScriptLanguage::find_function(const String &p_function, const String &p_code) const {
 	return -1;
 }
diff --git a/modules/gdnative/nativescript/nativescript.h b/modules/gdnative/nativescript/nativescript.h
index 2364c6c0f63c..2c78803501c3 100644
--- a/modules/gdnative/nativescript/nativescript.h
+++ b/modules/gdnative/nativescript/nativescript.h
@@ -317,7 +317,6 @@ class NativeScriptLanguage : public ScriptLanguage {
 	virtual bool validate(const String &p_script, const String &p_path, List<String> *r_functions, List<ScriptLanguage::ScriptError> *r_errors = nullptr, List<ScriptLanguage::Warning> *r_warnings = nullptr, Set<int> *r_safe_lines = nullptr) const;
 	virtual Script *create_script() const;
 	virtual bool has_named_classes() const;
-	virtual bool supports_builtin_mode() const;
 	virtual int find_function(const String &p_function, const String &p_code) const;
 	virtual String make_function(const String &p_class, const String &p_name, const PackedStringArray &p_args) const;
 	virtual void auto_indent_code(String &p_code, int p_from_line, int p_to_line) const;
diff --git a/modules/gdnative/pluginscript/pluginscript_language.cpp b/modules/gdnative/pluginscript/pluginscript_language.cpp
index 79aba342c90c..204875117cf3 100644
--- a/modules/gdnative/pluginscript/pluginscript_language.cpp
+++ b/modules/gdnative/pluginscript/pluginscript_language.cpp
@@ -151,10 +151,6 @@ bool PluginScriptLanguage::has_named_classes() const {
 	return _desc.has_named_classes;
 }
 
-bool PluginScriptLanguage::supports_builtin_mode() const {
-	return _desc.supports_builtin_mode;
-}
-
 bool PluginScriptLanguage::can_inherit_from_file() const {
 	return _desc.can_inherit_from_file;
 }
diff --git a/modules/gdnative/pluginscript/pluginscript_language.h b/modules/gdnative/pluginscript/pluginscript_language.h
index 26ab4a95e34b..a48040529d3e 100644
--- a/modules/gdnative/pluginscript/pluginscript_language.h
+++ b/modules/gdnative/pluginscript/pluginscript_language.h
@@ -78,7 +78,6 @@ class PluginScriptLanguage : public ScriptLanguage {
 	virtual bool validate(const String &p_script, const String &p_path = "", List<String> *r_functions = nullptr, List<ScriptLanguage::ScriptError> *r_errors = nullptr, List<ScriptLanguage::Warning> *r_warnings = nullptr, Set<int> *r_safe_lines = nullptr) const;
 	virtual Script *create_script() const;
 	virtual bool has_named_classes() const;
-	virtual bool supports_builtin_mode() const;
 	virtual bool can_inherit_from_file() const;
 	virtual int find_function(const String &p_function, const String &p_code) const;
 	virtual String make_function(const String &p_class, const String &p_name, const PackedStringArray &p_args) const;
diff --git a/modules/gdscript/gdscript.cpp b/modules/gdscript/gdscript.cpp
index 2e55506fd470..a7a3c2cd2d5a 100644
--- a/modules/gdscript/gdscript.cpp
+++ b/modules/gdscript/gdscript.cpp
@@ -835,7 +835,7 @@ Error GDScript::reload(bool p_keep_state) {
 			GDScriptLanguage::get_singleton()->debug_break_parse(get_path(), parser.get_errors().front()->get().line, "Parser Error: " + parser.get_errors().front()->get().message);
 		}
 		// TODO: Show all error messages.
-		_err_print_error("GDScript::reload", path.is_empty() ? "built-in" : (const char *)path.utf8().get_data(), parser.get_errors().front()->get().line, ("Parse Error: " + parser.get_errors().front()->get().message).utf8().get_data(), false, ERR_HANDLER_SCRIPT);
+		_err_print_error("GDScript::reload", (const char *)path.utf8().get_data(), parser.get_errors().front()->get().line, ("Parse Error: " + parser.get_errors().front()->get().message).utf8().get_data(), false, ERR_HANDLER_SCRIPT);
 		ERR_FAIL_V(ERR_PARSE_ERROR);
 	}
 
@@ -849,7 +849,7 @@ Error GDScript::reload(bool p_keep_state) {
 
 		const List<GDScriptParser::ParserError>::Element *e = parser.get_errors().front();
 		while (e != nullptr) {
-			_err_print_error("GDScript::reload", path.is_empty() ? "built-in" : (const char *)path.utf8().get_data(), e->get().line, ("Parse Error: " + e->get().message).utf8().get_data(), false, ERR_HANDLER_SCRIPT);
+			_err_print_error("GDScript::reload", (const char *)path.utf8().get_data(), e->get().line, ("Parse Error: " + e->get().message).utf8().get_data(), false, ERR_HANDLER_SCRIPT);
 			e = e->next();
 		}
 		ERR_FAIL_V(ERR_PARSE_ERROR);
@@ -869,7 +869,7 @@ Error GDScript::reload(bool p_keep_state) {
 			if (EngineDebugger::is_active()) {
 				GDScriptLanguage::get_singleton()->debug_break_parse(get_path(), compiler.get_error_line(), "Parser Error: " + compiler.get_error());
 			}
-			_err_print_error("GDScript::reload", path.is_empty() ? "built-in" : (const char *)path.utf8().get_data(), compiler.get_error_line(), ("Compile Error: " + compiler.get_error()).utf8().get_data(), false, ERR_HANDLER_SCRIPT);
+			_err_print_error("GDScript::reload", (const char *)path.utf8().get_data(), compiler.get_error_line(), ("Compile Error: " + compiler.get_error()).utf8().get_data(), false, ERR_HANDLER_SCRIPT);
 			ERR_FAIL_V(ERR_COMPILATION_FAILED);
 		} else {
 			return err;
diff --git a/modules/mono/csharp_script.cpp b/modules/mono/csharp_script.cpp
index 6970349414af..fee8f394d621 100644
--- a/modules/mono/csharp_script.cpp
+++ b/modules/mono/csharp_script.cpp
@@ -422,10 +422,6 @@ bool CSharpLanguage::has_named_classes() const {
 	return false;
 }
 
-bool CSharpLanguage::supports_builtin_mode() const {
-	return false;
-}
-
 #ifdef TOOLS_ENABLED
 static String variant_type_to_managed_name(const String &p_var_type_name) {
 	if (p_var_type_name.is_empty()) {
diff --git a/modules/mono/csharp_script.h b/modules/mono/csharp_script.h
index c998d9c1e4e4..f066ddcc9e89 100644
--- a/modules/mono/csharp_script.h
+++ b/modules/mono/csharp_script.h
@@ -102,8 +102,6 @@ class CSharpScript : public Script {
 	bool tool = false;
 	bool valid = false;
 
-	bool builtin;
-
 	GDMonoClass *base = nullptr;
 	GDMonoClass *native = nullptr;
 	GDMonoClass *script_class = nullptr;
@@ -473,7 +471,6 @@ class CSharpLanguage : public ScriptLanguage {
 	String validate_path(const String &p_path) const override;
 	Script *create_script() const override;
 	bool has_named_classes() const override;
-	bool supports_builtin_mode() const override;
 	/* TODO? */ int find_function(const String &p_function, const String &p_code) const override { return -1; }
 	String make_function(const String &p_class, const String &p_name, const PackedStringArray &p_args) const override;
 	virtual String _get_indentation() const;
diff --git a/modules/visual_script/visual_script.cpp b/modules/visual_script/visual_script.cpp
index 01fc7cfca020..8479ffadcaf6 100644
--- a/modules/visual_script/visual_script.cpp
+++ b/modules/visual_script/visual_script.cpp
@@ -2269,10 +2269,6 @@ bool VisualScriptLanguage::has_named_classes() const {
 	return false;
 }
 
-bool VisualScriptLanguage::supports_builtin_mode() const {
-	return true;
-}
-
 int VisualScriptLanguage::find_function(const String &p_function, const String &p_code) const {
 	return -1;
 }
diff --git a/modules/visual_script/visual_script.h b/modules/visual_script/visual_script.h
index 39cef8f68b3c..eb51794f96d6 100644
--- a/modules/visual_script/visual_script.h
+++ b/modules/visual_script/visual_script.h
@@ -574,7 +574,6 @@ class VisualScriptLanguage : public ScriptLanguage {
 	virtual bool validate(const String &p_script, const String &p_path = "", List<String> *r_functions = nullptr, List<ScriptLanguage::ScriptError> *r_errors = nullptr, List<ScriptLanguage::Warning> *r_warnings = nullptr, Set<int> *r_safe_lines = nullptr) const;
 	virtual Script *create_script() const;
 	virtual bool has_named_classes() const;
-	virtual bool supports_builtin_mode() const;
 	virtual int find_function(const String &p_function, const String &p_code) const;
 	virtual String make_function(const String &p_class, const String &p_name, const PackedStringArray &p_args) const;
 	virtual void auto_indent_code(String &p_code, int p_from_line, int p_to_line) const;