feat(filename): enhance default excepts patterns and improve validation

- Add default excepts patterns for common framework files:
  - Files starting with underscore (_app.tsx, _layout.tsx)
  - Numeric files (404.tsx)
  - Square bracket files ([slug].tsx)
- Improve validation logic to filter non-word/dash characters
- Update documentation with excepts option details
- Add comprehensive test cases for new exception patterns

This enhances the filename rule to be more framework-agnostic and
handles edge cases with special characters in filenames.

Author: Rel1cx

.github/instructions/nx.instructions.md

@@ -1,5 +1,5 @@
 ---
-applyTo: "**"
+applyTo: '**'
 ---
 
 // This file is automatically generated by Nx Console
@@ -9,14 +9,12 @@ You are in an nx workspace using Nx 21.4.0 and pnpm as the package manager.
 You have access to the Nx MCP server and the tools it provides. Use them. Follow these guidelines in order to best help the user:
 
 # General Guidelines
-
 - When answering questions, use the nx_workspace tool first to gain an understanding of the workspace architecture
 - For questions around nx configuration, best practices or if you're unsure, use the nx_docs tool to get relevant, up-to-date docs!! Always use this instead of assuming things about nx configuration
 - If the user needs help with an Nx configuration or project graph error, use the 'nx_workspace' tool to get any errors
 - To help answer questions about the workspace structure or simply help with demonstrating how tasks depend on each other, use the 'nx_visualize_graph' tool
 
 # Generation Guidelines
-
 If the user wants to generate something, use the following flow:
 
 - learn about the nx workspace and any specifics the user needs by using the 'nx_workspace' tool and the 'nx_project_details' tool if applicable
@@ -31,11 +29,12 @@ If the user wants to generate something, use the following flow:
 - use the information provided in the log file to answer the user's question or continue with what they were doing
 
 # Running Tasks Guidelines
-
 If the user wants help with tasks or commands (which include keywords like "test", "build", "lint", or other similar actions), use the following flow:
-
 - Use the 'nx_current_running_tasks_details' tool to get the list of tasks (this can include tasks that were completed, stopped or failed).
 - If there are any tasks, ask the user if they would like help with a specific task then use the 'nx_current_running_task_output' tool to get the terminal output for that task/command
 - Use the terminal output from 'nx_current_running_task_output' to see what's wrong and help the user fix their problem. Use the appropriate tools if necessary
 - If the user would like to rerun the task or command, always use `nx run <taskId>` to rerun in the terminal. This will ensure that the task will run in the nx context and will be run the same way it originally executed
-- If the task was marked as "continuous" do not offer to rerun the task. This task is already running and the user can see the output in the terminal. You can use 'nx_current_running_task_output' to get the output of the task to verify the output.
+- If the task was marked as "continuous" do not offer to rerun the task. This task is already running and the user can see the output in the terminal. You can use 'nx_current_running_task_output' to get the output of the task to verify the output. 
+
+
+

packages/plugins/eslint-plugin-react-naming-convention/src/rules/filename.md

@@ -69,6 +69,12 @@ src/components/example-component.tsx
   2. `camelCase`: camelCase
   3. `kebab-case`: kebab-case
   4. `snake_case`: snake_case
+- `excepts`: An array of regular expressions to ignore specific file names. Default is:
+  - `"$"`: Ignore `$` files (e.g., `$.tsx`).
+  - `"index"`: Ignore `index` files (e.g., `index.tsx`).
+  - `"/^_/"`: Ignore files starting with an underscore (e.g., `_app.tsx`, `_layout.tsx`).
+  - `"/^[0-9]+$/"`: Ignore files with only numbers (e.g., `404.tsx`).
+  - `"/^\[[^\]]+\]$/"`: Ignore files with square brackets (e.g., `[slug].tsx`).
 
 ## Rule Options Examples
 
@@ -103,23 +109,36 @@ export default [
 export default [
   // ...
   {
-    files: ["src/**/*.{ts,tsx}"],
-    ignore: ["**/index.{ts,tsx}"],
+    files: ["src/components/**/*.{ts,tsx}"],
     rules: {
       "@eslint-react/naming-convention/filename": ["warn", "PascalCase"],
     },
   },
   {
-    files: ["src/pages/**/*.{ts,tsx}"],
-    ignore: ["**/index.{ts,tsx}"],
+    files: ["src/hooks/**/use*.{ts,tsx}"],
+    rules: {
+      "@eslint-react/naming-convention/filename": ["warn", "camelCase"],
+    },
+  },
+];
+```
+
+### Opting out of the rule for framework-specific files
+
+```js title="eslint.config.js"
+// ...
+export default [
+  // ...
+  {
+    files: ["**/*.{ts,tsx}"],
     rules: {
       "@eslint-react/naming-convention/filename": ["warn", "kebab-case"],
     },
   },
   {
-    files: ["src/hooks/**/use*.{ts,tsx}"],
+    files: ["app/**/*.{ts,tsx}"], // Opting out of the rule for framework-specific files
     rules: {
-      "@eslint-react/naming-convention/filename": ["warn", "camelCase"],
+      "@eslint-react/naming-convention/filename": "off",
     },
   },
 ];

packages/plugins/eslint-plugin-react-naming-convention/src/rules/filename.spec.ts

@@ -151,5 +151,25 @@ ruleTester.run(RULE_NAME, rule, {
       filename: "snake_case.tsx",
       options: [{ rule: "snake_case" }],
     },
+    {
+      code,
+      filename: "404.tsx",
+      options: [{ rule: "PascalCase" }],
+    },
+    {
+      code,
+      filename: "$.tsx",
+      options: [{ rule: "PascalCase" }],
+    },
+    {
+      code,
+      filename: "_app.tsx",
+      options: [{ rule: "PascalCase" }],
+    },
+    {
+      code,
+      filename: "[slug].tsx",
+      options: [{ rule: "PascalCase" }],
+    },
   ],
 });

packages/plugins/eslint-plugin-react-naming-convention/src/rules/filename.ts

@@ -32,7 +32,13 @@ type Options = readonly [
 
 const defaultOptions = [
   {
-    excepts: ["^index$"],
+    excepts: [
+      "$",
+      "index",
+      "/^_/",
+      "/^[0-9]+$/",
+      "/^\\[[^\\]]+\\]$/",
+    ],
     rule: "PascalCase",
   },
 ] as const satisfies Options;
@@ -95,15 +101,14 @@ export function create(context: RuleContext<MessageID, Options>): RuleListener {
     : (options.excepts ?? []).map((s) => RE.toRegExp(s));
 
   function validate(name: string, casing: Case = rule, ignores = excepts) {
-    const shouldIgnore = ignores
-      .some((pattern) => pattern.test(name));
-    if (shouldIgnore) return true;
-
+    if (ignores.some((pattern) => pattern.test(name))) return true;
+    const filteredName = name.match(/[\w-]/gu)?.join("") ?? "";
+    if (filteredName.length === 0) return false;
     return match(casing)
-      .with("PascalCase", () => RE.PASCAL_CASE.test(name))
-      .with("camelCase", () => RE.CAMEL_CASE.test(name))
-      .with("kebab-case", () => RE.KEBAB_CASE.test(name))
-      .with("snake_case", () => RE.SNAKE_CASE.test(name))
+      .with("PascalCase", () => RE.PASCAL_CASE.test(filteredName))
+      .with("camelCase", () => RE.CAMEL_CASE.test(filteredName))
+      .with("kebab-case", () => RE.KEBAB_CASE.test(filteredName))
+      .with("snake_case", () => RE.SNAKE_CASE.test(filteredName))
       .exhaustive();
   }