eslint-plugin-nextfriday 1.8.0 → 1.9.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # eslint-plugin-nextfriday
 
+## 1.9.0
+
+### Minor Changes
+
+- [#45](https://github.com/next-friday/eslint-plugin-nextfriday/pull/45) [`5a00484`](https://github.com/next-friday/eslint-plugin-nextfriday/commit/5a0048400d3a6b7640813b4ba718635c9117ea20) Thanks [@joetakara](https://github.com/joetakara)! - feat(rules): add no-inline-default-export rule
+
+  Disallow inline default exports. Prefer declaring functions/classes first, then exporting separately.
+  - `export default function foo() {}` → Error
+  - `function foo() {} export default foo;` → Valid
+
 ## 1.8.0
 
 ### Minor Changes

package/README.md

@@ -78,6 +78,7 @@ export default [
       "nextfriday/no-logic-in-params": "error",
       "nextfriday/enforce-sorted-destructuring": "error",
       "nextfriday/no-env-fallback": "error",
+      "nextfriday/no-inline-default-export": "error",
 
       // Import Optimization
       "nextfriday/prefer-import-type": "error",
@@ -157,6 +158,7 @@ module.exports = {
 | [no-logic-in-params](docs/rules/NO_LOGIC_IN_PARAMS.md)                     | Disallow logic/conditions in function parameters - extract to const    | ❌      |
 | [enforce-sorted-destructuring](docs/rules/ENFORCE_SORTED_DESTRUCTURING.md) | Enforce alphabetical sorting of destructured properties                | ❌      |
 | [no-env-fallback](docs/rules/NO_ENV_FALLBACK.md)                           | Disallow fallback values for environment variables                     | ❌      |
+| [no-inline-default-export](docs/rules/NO_INLINE_DEFAULT_EXPORT.md)         | Disallow inline default exports - declare first, then export           | ❌      |
 | [no-direct-date](docs/rules/NO_DIRECT_DATE.md)                             | Disallow direct usage of Date constructor and methods                  | ❌      |
 
 ### Import Optimization Rules
@@ -189,14 +191,14 @@ module.exports = {
 
 | Preset               | Severity | Base Rules | JSX Rules | Total Rules |
 | -------------------- | -------- | ---------- | --------- | ----------- |
-| `base`               | warn     | 17         | 0         | 17          |
-| `base/recommended`   | error    | 17         | 0         | 17          |
-| `react`              | warn     | 17         | 7         | 24          |
-| `react/recommended`  | error    | 17         | 7         | 24          |
-| `nextjs`             | warn     | 17         | 7         | 24          |
-| `nextjs/recommended` | error    | 17         | 7         | 24          |
+| `base`               | warn     | 18         | 0         | 18          |
+| `base/recommended`   | error    | 18         | 0         | 18          |
+| `react`              | warn     | 18         | 7         | 25          |
+| `react/recommended`  | error    | 18         | 7         | 25          |
+| `nextjs`             | warn     | 18         | 7         | 25          |
+| `nextjs/recommended` | error    | 18         | 7         | 25          |
 
-### Base Configuration Rules (17 rules)
+### Base Configuration Rules (18 rules)
 
 Included in `base`, `base/recommended`, and all other presets:
 
@@ -208,6 +210,7 @@ Included in `base`, `base/recommended`, and all other presets:
 - `nextfriday/no-direct-date`
 - `nextfriday/no-emoji`
 - `nextfriday/no-env-fallback`
+- `nextfriday/no-inline-default-export`
 - `nextfriday/no-lazy-identifiers`
 - `nextfriday/no-logic-in-params`
 - `nextfriday/no-single-char-variables`

package/docs/rules/NO_INLINE_DEFAULT_EXPORT.md

@@ -0,0 +1,78 @@
+# no-inline-default-export
+
+Disallow inline default exports. Prefer declaring first, then exporting separately.
+
+## Rule Details
+
+This rule enforces separating function/class declarations from their default exports. Instead of combining declaration and export in a single statement, declare the function or class first, then export it by reference.
+
+This pattern improves code readability and makes it easier to identify what a module exports at a glance.
+
+## Examples
+
+### Incorrect
+
+```typescript
+// Inline function default export
+export default function generator(plop: PlopTypes.NodePlopAPI): void {
+  // ...
+}
+
+// Inline class default export
+export default class MyService {
+  // ...
+}
+
+// Anonymous function default export
+export default function () {
+  return "anonymous";
+}
+
+// Arrow function default export
+export default () => "arrow";
+
+// Anonymous class default export
+export default class {
+  // ...
+}
+```
+
+### Correct
+
+```typescript
+// Separate function declaration and export
+function generator(plop: PlopTypes.NodePlopAPI): void {
+  // ...
+}
+
+export default generator;
+
+// Separate class declaration and export
+class MyService {
+  // ...
+}
+
+export default MyService;
+
+// Separate const arrow function and export
+const processData = (data: Data): Result => {
+  // ...
+};
+
+export default processData;
+
+// Exporting literals and objects is allowed
+export default "literal";
+export default { key: "value" };
+
+// Re-exports are allowed
+export { foo as default } from "./foo";
+```
+
+## When Not To Use
+
+If your project prefers inline default exports for brevity, or if you're working with frameworks that expect specific export patterns, you may want to disable this rule.
+
+## Related Rules
+
+- [prefer-function-declaration](./PREFER_FUNCTION_DECLARATION.md) - Prefer function declarations over expressions