Remove PropTypes and fix webpack CSS configuration

- Remove deprecated PropTypes from all HelloWorld component templates
- Fix testing setup indentation to match other sections
- Revert commonWebpackConfig.js.tt to original simple version
- Let Shakapacker handle CSS modules through built-in configuration

This modernizes generated components for React 19 compatibility
and fixes build issues with static/prod modes.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: justin808

lib/generators/react_on_rails/generator_messages.rb

@@ -112,12 +112,12 @@ def build_testing_section
       <<~TESTING
 
 
-🧪 TESTING SETUP (Optional):
-─────────────────────────────────────────────────────────────────────────
-For JavaScript testing with asset compilation, add this to your RSpec config:
+        🧪 TESTING SETUP (Optional):
+        ─────────────────────────────────────────────────────────────────────────
+        For JavaScript testing with asset compilation, add this to your RSpec config:
 
-# In spec/rails_helper.rb or spec/spec_helper.rb:
-ReactOnRails::TestHelper.configure_rspec_to_compile_assets(config)
+        # In spec/rails_helper.rb or spec/spec_helper.rb:
+        ReactOnRails::TestHelper.configure_rspec_to_compile_assets(config)
       TESTING
     end
 

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

@@ -1,4 +1,3 @@
-import PropTypes from 'prop-types';
 import React, { useState } from 'react';
 import * as style from './HelloWorld.module.css';
 
@@ -19,8 +18,4 @@ const HelloWorld = (props) => {
   );
 };
 
-HelloWorld.propTypes = {
-  name: PropTypes.string.isRequired, // this is passed from the Rails view
-};
-
 export default HelloWorld;

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

@@ -1,4 +1,3 @@
-import PropTypes from 'prop-types';
 import React, { useState } from 'react';
 import * as style from './HelloWorld.module.css';
 
@@ -19,8 +18,4 @@ const HelloWorld = (props) => {
   );
 };
 
-HelloWorld.propTypes = {
-  name: PropTypes.string.isRequired, // this is passed from the Rails view
-};
-
 export default HelloWorld;

lib/generators/react_on_rails/templates/base/base/config/webpack/commonWebpackConfig.js.tt

@@ -2,32 +2,13 @@
 
 // Common configuration applying to client and server configuration
 const { generateWebpackConfig, merge } = require('shakapacker');
-const MiniCssExtractPlugin = require('mini-css-extract-plugin');
 
 const baseClientWebpackConfig = generateWebpackConfig();
 
 const commonOptions = {
   resolve: {
     extensions: ['.css', '.ts', '.tsx'],
   },
-  module: {
-    rules: [
-      {
-        test: /\.module\.css$/i,
-        use: [
-          process.env.NODE_ENV === 'development' ? 'style-loader' : MiniCssExtractPlugin.loader,
-          {
-            loader: 'css-loader',
-            options: {
-              modules: {
-                localIdentName: '[name]__[local]__[hash:base64:5]',
-              },
-            },
-          },
-        ],
-      },
-    ],
-  },
 };
 
 // Copy the object using merge b/c the baseClientWebpackConfig and commonOptions are mutable globals

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

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