Annotation in inner class fails to parse causing broken formatting

[hannan-ahmad]

Hello there! I'm using the GDScript formatter with the Zed editor, and the formatter worked great for a few time, and did the auto-save perfectly without issues. However, soon, I began to notice it began formatting the code wrongly. Here's one video where I demonstrate the code formatter making the code buggy when I save my file in Zed.

Screencast_20260130_205252.webm

[NathanLovato]

Thank you for taking time to report this. When you encounter a formatter issue, could you please copy and paste a minimal code example that triggers the issue with before and after? You can but the before and after code in code fences. That makes it a bit easier for me to review triage and later address these things.

[hannan-ahmad]

Sure. Here's the before vs after the formatting code:

Before formatting

@abstract class State extends Resource:
    @warning_ignore("unused_signal")
    signal exited
	
	
    var object: Node


    @abstract func enter() -> void

After formatting

@abstract class State extends Resource:
    @warning_ignore("unused_signal") signal exitedvar object: Node

    @abstract func enter() -> void

Note

Also, apart from the buggy formatting, the function enter() now has one line above it instead of two. Before formatting, there were two lines before the enter() function.

Important

I've tried writing my code in the following way:

@abstract class State extends Resource:
    @warning_ignore("unused_signal") signal exited
  
    var object: Node


    @abstract func enter() -> void

but it still gives the same buggy formatting as the one given above.

[NathanLovato]

Thank you, Hannan. It's much appreciated.

[NathanLovato]

Just getting to testing this, it's another thing that needs a change in the tree-sitter parser upstream. In my tests it has to do with annotation + signal in inner class:

class State:
	@warning_ignore("unused_signal")
	signal exited

Parse output:

(source [0, 0] - [3, 0]
  (class_definition [0, 0] - [2, 14]
    name: (name [0, 6] - [0, 11])
    body: (class_body [0, 12] - [2, 14]
      (ERROR [1, 1] - [2, 14]
        (annotation [1, 1] - [1, 33]
          (identifier [1, 2] - [1, 16])
          arguments: (arguments [1, 16] - [1, 33]
            (string [1, 17] - [1, 32])))))))

---

With a following statement:

class State:
	@warning_ignore("unused_signal")
	signal exited
	var test

Parse output:

(source [0, 0] - [4, 0]
  (class_definition [0, 0] - [3, 9]
    name: (name [0, 6] - [0, 11])
    body: (class_body [0, 12] - [3, 9]
      (variable_statement [1, 1] - [3, 9]
        (annotations [1, 1] - [1, 33]
          (annotation [1, 1] - [1, 33]
            (identifier [1, 2] - [1, 16])
            arguments: (arguments [1, 16] - [1, 33]
              (string [1, 17] - [1, 32]))))
        (ERROR [2, 1] - [2, 14])
        name: (name [3, 5] - [3, 9])))))