fix(lint/noRedeclare): allow overloads

CHANGELOG.md

@@ -45,6 +45,18 @@ Read our [guidelines for writing a good changelog entry](https://github.com/biom
   }
   ```
 
+- Fix [#1061](https://github.com/biomejs/biome/issues/1061). [noRedeclare](https://biomejs.dev/linter/rules/no-redeclare) no longer reports overloads of `export default function`. Contributed by @Conaclos
+
+  The following code is no longer reported:
+
+  ```ts
+  export default function(a: boolean): boolean;
+  export default function(a: number): number;
+  export default function(a: number | boolean): number | boolean {
+  	return a;
+  }
+  ```
+
 ### Parser
 
 

crates/biome_js_analyze/tests/specs/suspicious/noRedeclare/valid-declaration-merging.ts

@@ -108,3 +108,16 @@ export enum Orientation2 {
 	East = 2,
 	West = 3,
 }
+
+// overloads
+function f(a: boolean): boolean;
+function f(a: number): number;
+function f(a: number | boolean): number | boolean {
+	return a;
+}
+
+export default function(a: boolean): boolean;
+export default function(a: number): number;
+export default function(a: number | boolean): number | boolean {
+	return a;
+}

crates/biome_js_analyze/tests/specs/suspicious/noRedeclare/valid-declaration-merging.ts.snap

@@ -1,6 +1,5 @@
 ---
 source: crates/biome_js_analyze/tests/spec_tests.rs
-assertion_line: 80
 expression: valid-declaration-merging.ts
 ---
 # Input
@@ -116,6 +115,19 @@ export enum Orientation2 {
 	West = 3,
 }
 
+// overloads
+function f(a: boolean): boolean;
+function f(a: number): number;
+function f(a: number | boolean): number | boolean {
+	return a;
+}
+
+export default function(a: boolean): boolean;
+export default function(a: number): number;
+export default function(a: number | boolean): number | boolean {
+	return a;
+}
+
 ```
 
 

crates/biome_js_syntax/src/binding_ext.rs

@@ -114,6 +114,11 @@ impl AnyJsBindingDeclaration {
                 AnyJsBindingDeclaration::JsFunctionDeclaration(_)
                 | AnyJsBindingDeclaration::TsDeclareFunctionDeclaration(_),
             ) => true,
+            (
+                AnyJsBindingDeclaration::TsDeclareFunctionExportDefaultDeclaration(_),
+                AnyJsBindingDeclaration::JsFunctionExportDefaultDeclaration(_)
+                | AnyJsBindingDeclaration::TsDeclareFunctionExportDefaultDeclaration(_),
+            ) => true,
             (
                 AnyJsBindingDeclaration::TsEnumDeclaration(_),
                 AnyJsBindingDeclaration::TsEnumDeclaration(_),

website/src/content/docs/internals/changelog.mdx

@@ -51,6 +51,18 @@ Read our [guidelines for writing a good changelog entry](https://github.com/biom
   }
   ```
 
+- Fix [#1061](https://github.com/biomejs/biome/issues/1061). [noRedeclare](https://biomejs.dev/linter/rules/no-redeclare) no longer reports overloads of `export default function`. Contributed by @Conaclos
+
+  The following code is no longer reported:
+
+  ```ts
+  export default function(a: boolean): boolean;
+  export default function(a: number): number;
+  export default function(a: number | boolean): number | boolean {
+  	return a;
+  }
+  ```
+
 ### Parser