Fix missing Gemfile.development_dependencies and update linter config

The monorepo restructure inadvertently lost the Gemfile.development_dependencies
file which is required by both the root Gemfile and the dummy app's Gemfile.

Changes:
- File already exists on master at root - it was just missing from this branch
- Updated dummy app Gemfile to reference ../../../Gemfile.development_dependencies
  (three levels up to monorepo root instead of two)
- Updated .rubocop.yml to exclude all Gemfile* files from filename conventions
- Updated eslint.config.ts to handle both old and new directory structures
- Applied formatting fixes to generator templates

Fixes the CI failure where react_component helper was undefined due to
gem not loading properly because of missing development dependencies.

Author: justin808

.rubocop.yml

@@ -37,7 +37,7 @@ AllCops:
 
 Naming/FileName:
   Exclude:
-    - '**/Gemfile'
+    - '**/Gemfile*'
     - '**/Rakefile'
     - '**/Steepfile'
 

eslint.config.ts

@@ -26,13 +26,20 @@ const config = tsEslint.config([
     'react_on_rails_pro/',
     // used for tests only
     'spec/react_on_rails/dummy-for-generators',
+    'react_on_rails/spec/react_on_rails/dummy-for-generators',
     // temporary and generated files
     'spec/dummy/.yalc',
+    'react_on_rails/spec/dummy/.yalc',
     'spec/dummy/public',
+    'react_on_rails/spec/dummy/public',
     'spec/dummy/vendor',
+    'react_on_rails/spec/dummy/vendor',
     'spec/dummy/tmp',
+    'react_on_rails/spec/dummy/tmp',
     'spec/dummy/app/assets/config/manifest.js',
+    'react_on_rails/spec/dummy/app/assets/config/manifest.js',
     'spec/dummy/client/app/packs/server-bundle.js',
+    'react_on_rails/spec/dummy/client/app/packs/server-bundle.js',
     '**/*.res.js',
     '**/coverage',
     '**/assets/webpack/',
@@ -82,7 +89,10 @@ const config = tsEslint.config([
       'import/core-modules': ['react-redux'],
 
       'import/resolver': {
-        alias: [['Assets', './spec/dummy/client/app/assets']],
+        alias: [
+          ['Assets', './spec/dummy/client/app/assets'],
+          ['Assets', './react_on_rails/spec/dummy/client/app/assets'],
+        ],
 
         node: {
           extensions: ['.js', '.jsx', '.ts', '.tsx', '.d.ts'],
@@ -177,7 +187,7 @@ const config = tsEslint.config([
     },
   },
   {
-    files: ['lib/generators/react_on_rails/templates/**/*'],
+    files: ['lib/generators/react_on_rails/templates/**/*', 'react_on_rails/lib/generators/react_on_rails/templates/**/*'],
     rules: {
       // It doesn't use package.json from the template
       'import/no-unresolved': 'off',
@@ -188,12 +198,19 @@ const config = tsEslint.config([
     },
   },
   {
-    files: ['spec/dummy/**/*'],
+    files: ['spec/dummy/**/*', 'react_on_rails/spec/dummy/**/*'],
     rules: {
       // The dummy app dependencies are managed separately and may not be installed
       'import/no-unresolved': 'off',
     },
   },
+  {
+    files: ['**/e2e/playwright/**/*', '**/playwright/**/*.spec.{js,ts}'],
+    rules: {
+      // Playwright dependencies may not be installed during linting
+      'import/no-unresolved': 'off',
+    },
+  },
   {
     files: ['**/*.ts{x,}', '**/*.[cm]ts'],
 

react_on_rails/lib/generators/react_on_rails/templates/base/base/app/javascript/bundles/HelloWorld/components/HelloWorld.jsx

@@ -1,8 +1,9 @@
 import React, { useState } from 'react';
+import PropTypes from 'prop-types';
 import * as style from './HelloWorld.module.css';
 
-const HelloWorld = (props) => {
-  const [name, setName] = useState(props.name);
+const HelloWorld = ({ name: initialName }) => {
+  const [name, setName] = useState(initialName);
 
   return (
     <div>
@@ -18,4 +19,8 @@ const HelloWorld = (props) => {
   );
 };
 
+HelloWorld.propTypes = {
+  name: PropTypes.string.isRequired,
+};
+
 export default HelloWorld;

react_on_rails/lib/generators/react_on_rails/templates/base/base/app/javascript/src/HelloWorld/ror_components/HelloWorld.client.jsx

@@ -1,8 +1,9 @@
 import React, { useState } from 'react';
+import PropTypes from 'prop-types';
 import * as style from './HelloWorld.module.css';
 
-const HelloWorld = (props) => {
-  const [name, setName] = useState(props.name);
+const HelloWorld = ({ name: initialName }) => {
+  const [name, setName] = useState(initialName);
 
   return (
     <div>
@@ -18,4 +19,8 @@ const HelloWorld = (props) => {
   );
 };
 
+HelloWorld.propTypes = {
+  name: PropTypes.string.isRequired,
+};
+
 export default HelloWorld;

react_on_rails/lib/generators/react_on_rails/templates/redux/base/app/javascript/bundles/HelloWorld/components/HelloWorld.jsx

@@ -1,4 +1,5 @@
 import React from 'react';
+import PropTypes from 'prop-types';
 import * as style from './HelloWorld.module.css';
 
 const HelloWorld = ({ name, updateName }) => (
@@ -17,4 +18,9 @@ const HelloWorld = ({ name, updateName }) => (
   </div>
 );
 
+HelloWorld.propTypes = {
+  name: PropTypes.string.isRequired,
+  updateName: PropTypes.func.isRequired,
+};
+
 export default HelloWorld;

react_on_rails/spec/dummy/app/assets/config/manifest.js

@@ -1,2 +1 @@
-{
-}
+// This file is intentionally empty - webpack handles all asset compilation

react_on_rails/spec/react_on_rails/binstubs/dev_spec.rb

@@ -3,8 +3,8 @@
 require "react_on_rails/dev"
 
 RSpec.describe "bin/dev script" do
-  let(:script_path) { "lib/generators/react_on_rails/templates/base/base/bin/dev" }
-  let(:dummy_dev_path) { "spec/dummy/bin/dev" }
+  let(:script_path) { "react_on_rails/lib/generators/react_on_rails/templates/base/base/bin/dev" }
+  let(:dummy_dev_path) { "react_on_rails/spec/dummy/bin/dev" }
 
   # To suppress stdout during tests
   original_stderr = $stderr
@@ -68,9 +68,9 @@ def setup_script_execution_for_tool_tests
       expect(File.exist?(dummy_dev_path)).to be true
       expect(File.exist?(script_path)).to be true
 
-      # Verify the dummy script references the template
+      # Verify the dummy script references the template (using relative path from dummy/bin)
       dummy_content = File.read(dummy_dev_path)
-      expect(dummy_content).to include(script_path)
+      expect(dummy_content).to include("../../../lib/generators/react_on_rails/templates/base/base/bin/dev")
     end
 
     it "can require react_on_rails/dev and access all necessary modules" do

react_on_rails/spec/react_on_rails/rake_tasks_spec.rb

@@ -4,8 +4,8 @@
 
 RSpec.describe "RBS Rake Tasks" do
   before do
-    # Load the rake tasks file
-    load File.expand_path("../../rakelib/rbs.rake", __dir__)
+    # Load the rake tasks file from monorepo root (3 levels up from react_on_rails/spec/react_on_rails/)
+    load File.expand_path("../../../rakelib/rbs.rake", __dir__)
   end
 
   describe "rake rbs:validate" do