Fix annotations with parameters always wrapping in variables

Author: NathanLovato

src/formatter.rs

@@ -2297,6 +2297,14 @@ fn emit_inter_child_separator(
         return;
     }
 
+    // Variable annotations stay inline even when they have arguments. Keeping
+    // them inline also ensures that sibling and nested annotation AST shapes
+    // produce the same output.
+    if previous_kind == GDScriptNodeKind::Annotations && parent_kind == GDScriptNodeKind::Variable {
+        render_elements.push(RenderElement::Space);
+        return;
+    }
+
     // If the previous child is an annotations node and any of the annotations
     // has arguments (e.g. @rpc("any_peer")), keep the annotation on its own line.
     if previous_kind == GDScriptNodeKind::Annotations {

src/reorder.rs

@@ -59,7 +59,7 @@ pub enum DeclarationKind {
     OnReadyVariable, // 10
     Method,          // 11: functions (sub-sorted by MethodType)
     InnerClass,      // 12
-    Unknown,         // 255
+    Unknown = 255,   // 255
 }
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord)]

tests/expected/variable_annotations.gd

@@ -2,6 +2,8 @@
 @export var max_health := 5
 @onready var health := max_health
 @onready var health := max_health
+@export_range(10.0, 200.0) var jump_height := 50.0
+@export_range(0.1, 1.5) var jump_time_to_peak := 0.37
 
 @export_group("my group")
 @export var v = 1

tests/input/variable_annotations.gd

@@ -3,6 +3,9 @@
 @onready var health:=max_health
 @onready
 var health:=max_health
+@export_range(10.0, 200.0) var jump_height:=50.0
+@export_range(0.1, 1.5)
+var jump_time_to_peak:=0.37
 
 @export_group("my group")
 @export var v = 1