fix: attach comments inside the body of an attributed function (#2517)

`getCommentChildNodes()` returns an explicit child list for functions
and methods that carry attributes, but it left out `name` and `body`.
Any comment inside the body of such a function was then resolved
against the return type instead of the body: with a return type it
crashed in `handleFunctionParameter` ("Cannot read properties of
undefined (reading 'kind')"), and without one the comment was hoisted
out of the body.

Return every child in source order. Including `name` keeps a comment
trailing the attribute (`#[T] // comment`) attached to the attribute
rather than being pushed into the body by
`handleLastFunctionArgComments`, so existing snapshots are unchanged.

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>

Author: mreiden

src/comments.mjs

@@ -938,10 +938,15 @@ function getCommentChildNodes(node) {
 
   if (node.attrGroups && node.attrGroups.length > 0) {
     if (node.kind === "method" || node.kind === "function") {
+      // Keep every child (in source order): dropping `body` makes comments
+      // inside the body resolve against the return type, dropping `name`
+      // makes a comment after the attribute look like it precedes the body.
       return [
         ...node.attrGroups,
+        ...(node.name && typeof node.name === "object" ? [node.name] : []),
         ...node.arguments,
         ...(node.type ? [node.type] : []),
+        ...(node.body ? [node.body] : []),
       ];
     }
 

tests/attributes/__snapshots__/jsfmt.spec.mjs.snap

@@ -243,6 +243,58 @@ class ParamCommentFunctionAnnotation
 ================================================================================
 `;
 
+exports[`attributes-body-comments.php 1`] = `
+====================================options=====================================
+parsers: ["php"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+<?php
+
+class A
+{
+    #[Override]
+    public function typed(): array
+    {
+        // leading comment in the body of an attributed method
+        $vals = [];
+
+        // between statements
+        return $vals;
+    }
+
+    #[Override] // trailing the attribute
+    public function untyped()
+    {
+        // only a comment
+    }
+}
+
+=====================================output=====================================
+<?php
+
+class A
+{
+    #[Override]
+    public function typed(): array
+    {
+        // leading comment in the body of an attributed method
+        $vals = [];
+
+        // between statements
+        return $vals;
+    }
+
+    #[Override] // trailing the attribute
+    public function untyped()
+    {
+        // only a comment
+    }
+}
+
+================================================================================
+`;
+
 exports[`attributes-per-cs.php 1`] = `
 ====================================options=====================================
 parsers: ["php"]

tests/attributes/attributes-body-comments.php

@@ -0,0 +1,20 @@
+<?php
+
+class A
+{
+    #[Override]
+    public function typed(): array
+    {
+        // leading comment in the body of an attributed method
+        $vals = [];
+
+        // between statements
+        return $vals;
+    }
+
+    #[Override] // trailing the attribute
+    public function untyped()
+    {
+        // only a comment
+    }
+}