Enhance ESLint configuration and TypeScript support for react-on-rails packages

- Added specific ESLint configurations for the react-on-rails and react-on-rails-pro packages to handle import resolution issues in a monorepo setup.
- Introduced new tsconfig.eslint.json files for both packages to extend the base TypeScript configuration and include relevant source and test files.
- Updated the ESLint config to disable certain rules for the react-on-rails-pro package, ensuring compatibility with TypeScript's validation of imports.
- Made minor adjustments to the RSCRoute component to remove unnecessary TypeScript directives, improving code clarity.

These changes improve linting accuracy and TypeScript support across the project.

Author: AbanoubGhadban

eslint.config.ts

@@ -16,6 +16,7 @@ const compat = new FlatCompat({
 });
 
 const config = tsEslint.config([
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
   includeIgnoreFile(path.resolve(__dirname, '.gitignore')),
   globalIgnores([
     // compiled code
@@ -156,6 +157,14 @@ const config = tsEslint.config([
       'import/extensions': ['error', 'ignorePackages'],
     },
   },
+  {
+    files: ['packages/react-on-rails-pro/**/*'],
+    rules: {
+      // Disable import/named for pro package - can't resolve monorepo workspace imports
+      // TypeScript compiler validates these imports
+      'import/named': 'off',
+    },
+  },
   {
     files: ['lib/generators/react_on_rails/templates/**/*'],
     rules: {
@@ -181,12 +190,7 @@ const config = tsEslint.config([
 
     languageOptions: {
       parserOptions: {
-        projectService: {
-          allowDefaultProject: ['eslint.config.ts', 'knip.ts', 'packages/*/tests/*.test.{ts,tsx}'],
-          // Needed because `import * as ... from` instead of `import ... from` doesn't work in this file
-          // for some imports.
-          defaultProject: 'tsconfig.eslint.json',
-        },
+        projectService: true,
       },
     },
 

packages/react-on-rails-pro/src/RSCRoute.tsx

@@ -12,6 +12,8 @@
  * https://github.com/shakacode/react_on_rails/blob/master/REACT-ON-RAILS-PRO-LICENSE.md
  */
 
+/// <reference types="react/experimental" />
+
 import * as React from 'react';
 import { useRSC } from './RSCProvider.tsx';
 import { ServerComponentFetchError } from './ServerComponentFetchError.ts';
@@ -73,8 +75,7 @@ export type RSCRouteProps = {
 
 const PromiseWrapper = ({ promise }: { promise: Promise<React.ReactNode> }) => {
   // React.use is available in React 18.3+
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  return (React as any).use(promise);
+  return React.use(promise);
 };
 
 const RSCRoute = ({ componentName, componentProps }: RSCRouteProps): React.ReactNode => {

packages/react-on-rails-pro/tsconfig.eslint.json

@@ -0,0 +1,4 @@
+{
+  "extends": "./tsconfig.json",
+  "include": ["src/**/*", "tests/**/*"]
+}

packages/react-on-rails/tests/serverRenderReactComponent.test.ts

@@ -27,6 +27,7 @@ describe('serverRenderReactComponent', () => {
   beforeEach(() => {
     ComponentRegistry.components().clear();
     // Setup globalThis.ReactOnRails for serverRenderReactComponent
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/unbound-method, @typescript-eslint/no-explicit-any
     globalThis.ReactOnRails = { getComponent: ComponentRegistry.get } as any;
   });
 

packages/react-on-rails/tsconfig.eslint.json

@@ -0,0 +1,4 @@
+{
+  "extends": "./tsconfig.json",
+  "include": ["src/**/*", "tests/**/*"]
+}