Refactor Dev modules for better code quality and fix post-install messages

## Code Quality Improvements:
- **Refactored ServerManager class**: Split large methods into smaller, focused methods
  - Extracted kill_processes into kill_running_processes, cleanup_socket_files, etc.
  - Broke down show_help into help_usage, help_commands, help_options, etc.
  - Simplified print_procfile_info by extracting box formatting helpers
- **Refactored FileManager class**: Reduced cyclomatic complexity of cleanup_stale_files
  - Split into cleanup_overmind_sockets and cleanup_rails_pid_file
  - Extracted helper methods for process detection and file cleanup
- **Removed most RuboCop overrides**: Code now naturally complies with style guidelines
- **Kept minimal justified overrides**: Only for legitimate cases (generator complexity, test patterns)

## User Experience Improvements:
- **Updated post-install message**: Added 'bin/dev prod' option as requested
- **Fixed customize message ordering**: Now shows customize tip before access URL

## Results:
- ✅ All RuboCop violations resolved (134 files, 0 offenses)
- ✅ Significantly improved code maintainability and readability
- ✅ Better separation of concerns in Dev modules
- ✅ Enhanced user messaging experience

🎨 Code quality improvement while maintaining full functionality

Author: justin808

.rubocop.yml

@@ -67,34 +67,25 @@ Lint/SuppressedException:
 
 Metrics/AbcSize:
   Max: 28
-  Exclude:
-    - 'lib/react_on_rails/dev/server_manager.rb'  # Process management complexity justified
 
 Metrics/CyclomaticComplexity:
   Max: 7
-  Exclude:
-    - 'lib/react_on_rails/dev/file_manager.rb'  # File cleanup logic complexity justified
-    - 'lib/react_on_rails/dev/server_manager.rb'  # Process management complexity justified
 
 Metrics/PerceivedComplexity:
   Max: 10
-  Exclude:
-    - 'lib/react_on_rails/dev/server_manager.rb'  # Process management complexity justified
 
 Metrics/ClassLength:
   Max: 150
   Exclude:
     - 'lib/generators/react_on_rails/base_generator.rb'  # Generator complexity justified
-    - 'lib/react_on_rails/dev/server_manager.rb'  # Dev tool complexity justified
+    - 'lib/react_on_rails/dev/server_manager.rb'  # Dev tool with multiple help methods
 
 Metrics/ParameterLists:
   Max: 5
   CountKeywordArgs: false
 
 Metrics/MethodLength:
   Max: 41
-  Exclude:
-    - 'lib/react_on_rails/dev/server_manager.rb'  # Process management methods justified
 
 Metrics/ModuleLength:
   Max: 180
@@ -159,3 +150,4 @@ RSpec/InstanceVariable:
 RSpec/StubbedMock:
   Exclude:
     - 'spec/react_on_rails/dev/**/*_spec.rb'  # Dev module tests use mixed stub/mock patterns
+

lib/generators/react_on_rails/generator_messages.rb

@@ -62,6 +62,7 @@ def helpful_message_after_installation(component_name: "HelloWorld")
                 1. Start the app:
                    ./bin/dev              # HMR (Hot Module Replacement) mode
                    ./bin/dev static       # Static bundles (no HMR, faster initial load)
+                   ./bin/dev prod         # Production-like mode for testing
                    ./bin/dev help         # See all available options
         #{process_manager_section}
 

lib/react_on_rails/dev/file_manager.rb

@@ -5,49 +5,56 @@ module Dev
     class FileManager
       class << self
         def cleanup_stale_files
-          cleaned_any = false
+          socket_cleanup = cleanup_overmind_sockets
+          pid_cleanup = cleanup_rails_pid_file
+
+          socket_cleanup || pid_cleanup
+        end
+
+        private
+
+        def cleanup_overmind_sockets
+          return false if overmind_running?
 
-          # Check for stale overmind socket files
           socket_files = [".overmind.sock", "tmp/sockets/overmind.sock"]
+          cleaned_any = false
 
-          # Only clean up if overmind is not actually running
-          overmind_pids = `pgrep -f "overmind" 2>/dev/null`.split("\n").map(&:to_i)
-
-          if overmind_pids.empty?
-            socket_files.each do |socket_file|
-              next unless File.exist?(socket_file)
-
-              puts "   🧹 Cleaning up stale socket: #{socket_file}"
-              begin
-                File.delete(socket_file)
-              rescue StandardError
-                nil
-              end
-              cleaned_any = true
-            end
+          socket_files.each do |socket_file|
+            cleaned_any = true if remove_file_if_exists(socket_file, "stale socket")
           end
 
-          # Check for stale Rails server.pid file
+          cleaned_any
+        end
+
+        def cleanup_rails_pid_file
           server_pid_file = "tmp/pids/server.pid"
-          if File.exist?(server_pid_file)
-            pid = File.read(server_pid_file).to_i
-            # Check if process is actually running
-            begin
-              Process.kill(0, pid)
-              # Process exists, don't clean up
-            rescue Errno::ESRCH
-              # Process doesn't exist, clean up stale pid file
-              puts "   🧹 Cleaning up stale Rails pid file: #{server_pid_file}"
-              begin
-                File.delete(server_pid_file)
-              rescue StandardError
-                nil
-              end
-              cleaned_any = true
-            end
-          end
+          return false unless File.exist?(server_pid_file)
 
-          cleaned_any
+          pid = File.read(server_pid_file).to_i
+          return false if process_running?(pid)
+
+          remove_file_if_exists(server_pid_file, "stale Rails pid file")
+        end
+
+        def overmind_running?
+          !`pgrep -f "overmind" 2>/dev/null`.split("\n").empty?
+        end
+
+        def process_running?(pid)
+          Process.kill(0, pid)
+          true
+        rescue Errno::ESRCH
+          false
+        end
+
+        def remove_file_if_exists(file_path, description)
+          return false unless File.exist?(file_path)
+
+          puts "   🧹 Cleaning up #{description}: #{file_path}"
+          File.delete(file_path)
+          true
+        rescue StandardError
+          false
         end
       end
     end