fix(binder): bind symbol for declare function

Author: Song Gao

crates/oxc_semantic/src/binder.rs

@@ -161,7 +161,9 @@ impl<'a> Binder<'a> for Function<'a> {
                     builder.current_node_id,
                 );
                 ident.symbol_id.set(Some(symbol_id));
-            } else if self.r#type == FunctionType::FunctionDeclaration {
+            } else if self.r#type == FunctionType::FunctionDeclaration
+                || self.r#type == FunctionType::TSDeclareFunction
+            {
                 // The visitor is already inside the function scope,
                 // retrieve the parent scope for the function id to bind to.
 

crates/oxc_semantic/tests/fixtures/typescript-eslint/decorators/class-deco-with-object-param.snap

@@ -1,5 +1,6 @@
 ---
 source: crates/oxc_semantic/tests/main.rs
+assertion_line: 147
 input_file: crates/oxc_semantic/tests/fixtures/typescript-eslint/decorators/class-deco-with-object-param.ts
 snapshot_kind: text
 ---
@@ -14,7 +15,7 @@ snapshot_kind: text
         "symbols": [
           {
             "flags": "SymbolFlags(FunctionScopedVariable)",
-            "id": 0,
+            "id": 1,
             "name": "param",
             "node": "BindingRestElement",
             "references": []
@@ -33,9 +34,23 @@ snapshot_kind: text
     "id": 0,
     "node": "Program",
     "symbols": [
+      {
+        "flags": "SymbolFlags(BlockScopedVariable | Function)",
+        "id": 0,
+        "name": "deco",
+        "node": "Function(deco)",
+        "references": [
+          {
+            "flags": "ReferenceFlags(Read)",
+            "id": 0,
+            "name": "deco",
+            "node_id": 19
+          }
+        ]
+      },
       {
         "flags": "SymbolFlags(Class)",
-        "id": 1,
+        "id": 2,
         "name": "Foo",
         "node": "Class(Foo)",
         "references": []

crates/oxc_semantic/tests/fixtures/typescript-eslint/functions/function-declaration/declare-function.snap

@@ -0,0 +1,45 @@
+---
+source: crates/oxc_semantic/tests/main.rs
+assertion_line: 147
+input_file: crates/oxc_semantic/tests/fixtures/typescript-eslint/functions/function-declaration/declare-function.ts
+snapshot_kind: text
+---
+[
+  {
+    "children": [
+      {
+        "children": [],
+        "flags": "ScopeFlags(StrictMode | Function)",
+        "id": 1,
+        "node": "Function(A)",
+        "symbols": []
+      },
+      {
+        "children": [],
+        "flags": "ScopeFlags(StrictMode | Function)",
+        "id": 2,
+        "node": "Function(B)",
+        "symbols": []
+      }
+    ],
+    "flags": "ScopeFlags(StrictMode | Top)",
+    "id": 0,
+    "node": "Program",
+    "symbols": [
+      {
+        "flags": "SymbolFlags(BlockScopedVariable | Function)",
+        "id": 0,
+        "name": "A",
+        "node": "Function(A)",
+        "references": []
+      },
+      {
+        "flags": "SymbolFlags(BlockScopedVariable | Function)",
+        "id": 1,
+        "name": "B",
+        "node": "Function(B)",
+        "references": []
+      }
+    ]
+  }
+]

crates/oxc_semantic/tests/fixtures/typescript-eslint/functions/function-declaration/declare-function.ts

@@ -0,0 +1,2 @@
+declare function A(): void;
+function B(): void { };

crates/oxc_semantic/tests/fixtures/typescript-eslint/functions/function-declaration/overload.snap

@@ -1,5 +1,6 @@
 ---
 source: crates/oxc_semantic/tests/main.rs
+assertion_line: 147
 input_file: crates/oxc_semantic/tests/fixtures/typescript-eslint/functions/function-declaration/overload.ts
 snapshot_kind: text
 ---
@@ -14,14 +15,14 @@ snapshot_kind: text
         "symbols": [
           {
             "flags": "SymbolFlags(FunctionScopedVariable)",
-            "id": 0,
+            "id": 1,
             "name": "a",
             "node": "FormalParameter(a)",
             "references": []
           },
           {
             "flags": "SymbolFlags(FunctionScopedVariable)",
-            "id": 1,
+            "id": 2,
             "name": "b",
             "node": "FormalParameter(b)",
             "references": []
@@ -64,7 +65,7 @@ snapshot_kind: text
     "symbols": [
       {
         "flags": "SymbolFlags(BlockScopedVariable | Function)",
-        "id": 2,
+        "id": 0,
         "name": "foo",
         "node": "Function(foo)",
         "references": []

crates/oxc_semantic/tests/fixtures/typescript-eslint/type-declaration/literal-type3.snap

@@ -1,5 +1,6 @@
 ---
 source: crates/oxc_semantic/tests/main.rs
+assertion_line: 147
 input_file: crates/oxc_semantic/tests/fixtures/typescript-eslint/type-declaration/literal-type3.ts
 snapshot_kind: text
 ---
@@ -28,7 +29,7 @@ snapshot_kind: text
         "symbols": [
           {
             "flags": "SymbolFlags(FunctionScopedVariable)",
-            "id": 2,
+            "id": 3,
             "name": "value",
             "node": "FormalParameter(value)",
             "references": []
@@ -67,6 +68,13 @@ snapshot_kind: text
             "node_id": 31
           }
         ]
+      },
+      {
+        "flags": "SymbolFlags(BlockScopedVariable | Function)",
+        "id": 2,
+        "name": "setAlignment",
+        "node": "Function(setAlignment)",
+        "references": []
       }
     ]
   }