refactor generators: eliminate wasteful cleanup

- Clean Base: Make BaseGenerator conditional based on Redux option
	- Only create HelloWorld structure for non-Redux components
	- Only copy HelloWorld.module.css for non-Redux components
- Simplify Redux: Remove cleanup_base_generator_files method entirely
- Result: Eliminates wasteful create-then-delete pattern
- Verified: Both non-Redux (HelloWorld) and Redux (HelloWorldApp) work correctly

This approach is much cleaner since auto-registration is now the default behavior.

Author: ihabadham

lib/generators/react_on_rails/base_generator.rb

@@ -22,8 +22,11 @@ def add_hello_world_route
       end
 
       def create_react_directories
-        # Create auto-registration directory structure
-        empty_directory("app/javascript/src/HelloWorld/ror_components")
+        # Create auto-registration directory structure for non-Redux components only
+        # Redux components handle their own directory structure
+        unless options.redux?
+          empty_directory("app/javascript/src/HelloWorld/ror_components")
+        end
       end
 
       def copy_base_files
@@ -42,8 +45,14 @@ def copy_base_files
 
       def copy_js_bundle_files
         base_path = "base/base/"
-        base_files = %w[app/javascript/packs/server-bundle.js
-                        app/javascript/src/HelloWorld/HelloWorld.module.css]
+        base_files = %w[app/javascript/packs/server-bundle.js]
+        
+        # Only copy HelloWorld.module.css for non-Redux components
+        # Redux components handle their own CSS files
+        unless options.redux?
+          base_files << "app/javascript/src/HelloWorld/HelloWorld.module.css"
+        end
+        
         base_files.each { |file| copy_file("#{base_path}#{file}", file) }
       end
 

lib/generators/react_on_rails/react_with_redux_generator.rb

@@ -62,13 +62,6 @@ def add_redux_yarn_dependencies
         run "yarn add redux react-redux"
       end
 
-      def cleanup_base_generator_files
-        # Remove files created by base generator since Redux uses different structure
-        remove_file "app/javascript/src/HelloWorld/HelloWorld.module.css"
-        remove_dir "app/javascript/src/HelloWorld/ror_components" if Dir.exist?("app/javascript/src/HelloWorld/ror_components") && Dir.empty?("app/javascript/src/HelloWorld/ror_components")
-        remove_dir "app/javascript/src/HelloWorld" if Dir.exist?("app/javascript/src/HelloWorld") && Dir.entries("app/javascript/src/HelloWorld").size <= 2 # only . and ..
-      end
-
       def add_redux_specific_messages
         # Override the generic messages with Redux-specific instructions
         require_relative "generator_messages"