ruby_raider 3.0.0 → 3.1.0

data/lib/scaffolding/scaffolding.rb

@@ -13,22 +13,10 @@ class Scaffolding < Thor::Group
 
   attr_writer :uses
 
-  OVERRIDE_DIR = '.ruby_raider/templates'
-
   def self.source_root
     "#{File.dirname(__FILE__)}/templates"
   end
 
-  # Check for user template override before using default
-  def template(source, *args, &block)
-    override = File.join(OVERRIDE_DIR, File.basename(source))
-    if File.exist?(override)
-      super(File.expand_path(override), *args, &block)
-    else
-      super
-    end
-  end
-
   # --- Generation methods ---
 
   def generate_page
@@ -60,68 +48,6 @@ class Scaffolding < Thor::Group
              default_path('config/config.yml', '.yml'))
   end
 
-  def generate_spec_from_page(source_file, ai: false) # rubocop:disable Naming/MethodParameterName
-    require_relative 'page_introspector'
-    @introspected = PageIntrospector.new(source_file)
-    enrich_with_ai_scenarios if ai
-    template('spec_from_page.tt', "spec/#{normalized_name}_spec.rb")
-  end
-
-  def generate_page_from_url(analysis)
-    @url_data = analysis
-    template('page_from_url.tt', "page_objects/pages/#{normalized_name}.rb")
-  end
-
-  def generate_spec_from_url(analysis)
-    @url_data = analysis
-    template('spec_from_url.tt', "spec/#{normalized_name}_spec.rb")
-  end
-
-  # --- Deletion methods ---
-
-  def delete_page
-    remove_file(default_path("page_objects/pages/#{normalized_name}.rb", '_page.rb'))
-  end
-
-  def delete_feature
-    remove_file(default_path("features/#{normalized_name}.feature", '.feature'))
-  end
-
-  def delete_spec
-    remove_file(default_path("spec/#{normalized_name}_page_spec.rb", '_spec.rb'))
-  end
-
-  def delete_helper
-    remove_file(default_path("helpers/#{normalized_name}_helper.rb", '_helper.rb'))
-  end
-
-  def delete_steps
-    remove_file(default_path("features/step_definitions/#{normalized_name}_steps.rb", '_steps.rb'))
-  end
-
-  def delete_component
-    remove_file(default_path("page_objects/components/#{normalized_name}.rb", '.rb'))
-  end
-
-  def delete_config
-    remove_file(default_path('config/config.yml', '.yml'))
-  end
-
-  # --- Path planning (for dry-run) ---
-
-  def self.planned_path(name, type, custom_path = nil)
-    n = NameNormalizer.normalize(name)
-    case type.to_s
-    when 'page' then custom_path ? "#{custom_path}/#{n}_page.rb" : "page_objects/pages/#{n}.rb"
-    when 'spec' then custom_path ? "#{custom_path}/#{n}_spec.rb" : "spec/#{n}_page_spec.rb"
-    when 'feature' then custom_path ? "#{custom_path}/#{n}.feature" : "features/#{n}.feature"
-    when 'steps' then custom_path ? "#{custom_path}/#{n}_steps.rb" : "features/step_definitions/#{n}_steps.rb"
-    when 'helper' then custom_path ? "#{custom_path}/#{n}_helper.rb" : "helpers/#{n}_helper.rb"
-    when 'component' then custom_path ? "#{custom_path}/#{n}.rb" : "page_objects/components/#{n}.rb"
-    when 'model' then "models/data/#{n}.yml"
-    end
-  end
-
   # --- Template helpers (available in .tt files) ---
 
   def class_name
@@ -164,10 +90,6 @@ class Scaffolding < Thor::Group
     automation_type == 'selenium'
   end
 
-  def capybara?
-    automation_type == 'capybara'
-  end
-
   def watir?
     automation_type == 'watir'
   end
@@ -176,39 +98,8 @@ class Scaffolding < Thor::Group
     Array(@uses).reject(&:empty?)
   end
 
-  attr_reader :introspected, :url_data
-
-  def ai_scenarios
-    @ai_scenarios || {}
-  end
-
   def default_path(standard_path, file_type)
     path ? "#{path}/#{normalized_name}#{file_type}" : standard_path
   end
 
-  private
-
-  def enrich_with_ai_scenarios
-    require_relative '../llm/client'
-    require_relative '../llm/prompts'
-    require_relative '../llm/response_parser'
-
-    return unless Llm::Client.available?
-
-    response = Llm::Client.complete(
-      Llm::Prompts.generate_test_scenarios(
-        @introspected.class_name,
-        @introspected.methods,
-        automation_type,
-        framework_type
-      ),
-      system_prompt: Llm::Prompts.system_prompt
-    )
-    scenarios = Llm::ResponseParser.extract_scenarios(response)
-    return unless scenarios
-
-    @ai_scenarios = scenarios.each_with_object({}) do |s, hash|
-      hash[s[:method]] = s
-    end
-  end
 end

data/lib/scaffolding/templates/component.tt

@@ -3,10 +3,7 @@
 require_relative '../abstract/component'
 
 class <%= class_name %> < Component
-<%- if capybara? -%>
-
-  # Example: find('.selector').text
-<%- elsif selenium? -%>
+<%- if selenium? -%>
 
   def content
     component.text

data/lib/scaffolding/templates/page_object.tt

@@ -16,10 +16,6 @@ require_relative '<%= dep %>'
 <%- end -%>
 
 <%= '  ' * module_parts.size %>  # Actions
-<%- if capybara? -%>
-
-<%= '  ' * module_parts.size %>  # Example: fill_in 'field_id', with: value
-<%- else -%>
 
 <%= '  ' * module_parts.size %>  private
 
@@ -29,7 +25,6 @@ require_relative '<%= dep %>'
 <%- elsif watir? -%>
 <%= '  ' * module_parts.size %>  # Example: browser.text_field(id: 'element_id')
 <%- end -%>
-<%- end -%>
 <%= '  ' * module_parts.size %>end
 <% module_parts.each_with_index do |_mod, i| -%>
 <%= '  ' * (module_parts.size - i - 1) %>end
@@ -43,10 +38,6 @@ class <%= page_class_name %> < Page
 <%- end -%>
 
   # Actions
-<%- if capybara? -%>
-
-  # Example: fill_in 'field_id', with: value
-<%- else -%>
 
   private
 
@@ -56,6 +47,5 @@ class <%= page_class_name %> < Page
 <%- elsif watir? -%>
   # Example: browser.text_field(id: 'element_id')
 <%- end -%>
-<%- end -%>
 end
 <% end -%>

data/lib/scaffolding/templates/spec.tt

@@ -10,18 +10,14 @@ describe '<%= page_class_name %>' do
   let(:page) { <%= page_class_name %>.new(driver) }
 <%- elsif watir? -%>
   let(:page) { <%= page_class_name %>.new(browser) }
-<%- elsif capybara? -%>
-  let(:page) { <%= page_class_name %>.new }
 <%- end -%>
 <%- uses_list.each do |dep| -%>
-  let(:<%= dep %>) { <%= NameNormalizer.to_page_class(dep) %>.new<%= selenium? ? '(driver)' : watir? ? '(browser)' : '' %> }
+  let(:<%= dep %>) { <%= NameNormalizer.to_page_class(dep) %>.new<%= selenium? ? '(driver)' : '(browser)' %> }
 <%- end -%>
 <%- if project_config['url'] -%>
 
   before do
-<%- if capybara? -%>
-    visit '<%= normalized_name %>'
-<%- elsif selenium? -%>
+<%- if selenium? -%>
     driver.navigate.to "\#{project_config['url']}/<%= normalized_name %>"
 <%- elsif watir? -%>
     browser.goto "\#{project_config['url']}/<%= normalized_name %>"

data/lib/scaffolding/templates/steps.tt

@@ -10,8 +10,6 @@ Given('I am on the <%= normalized_name.tr("_", " ") %> page') do
   @page = <%= page_class_name %>.new(driver)
 <%- elsif watir? -%>
   @page = <%= page_class_name %>.new(browser)
-<%- elsif capybara? -%>
-  visit '<%= normalized_name %>'
 <%- end -%>
 end
 

data/lib/utilities/utilities.rb

@@ -7,7 +7,7 @@ module Utilities
 
   class << self
     def browser=(browser)
-      set('browser', browser)
+      set('browser', browser.to_s.delete_prefix(':'))
     end
 
     def page_path=(path)
@@ -44,40 +44,36 @@ module Utilities
     end
 
     def browser_options=(opts)
-      set('browser_options', Array(opts).flatten)
+      reload_config
+      browser_name = @config['browser'] || 'chrome'
+      @config['browser_arguments'] ||= {}
+      @config['browser_arguments'][browser_name] = Array(opts).flatten
+      overwrite_yaml
     end
 
     def delete_browser_options
-      config.delete('browser_options')
+      reload_config
+      browser_name = @config['browser'] || 'chrome'
+      @config['browser_arguments']&.delete(browser_name)
       overwrite_yaml
     end
 
-    def llm_provider=(provider)
-      set('llm_provider', provider)
-    end
-
-    def llm_api_key=(key)
-      set('llm_api_key', key)
-    end
-
-    def llm_model=(model)
-      set('llm_model', model)
-    end
-
-    def llm_url=(url)
-      set('llm_url', url)
+    def headless=(enabled)
+      set('headless', enabled)
     end
 
     def debug=(enabled)
-      config['debug'] ||= {}
-      config['debug']['enabled'] = enabled
+      reload_config
+      @config['debug'] ||= {}
+      @config['debug']['enabled'] = enabled
       overwrite_yaml
     end
 
     # Set multiple config keys in a single YAML write.
     # Usage: Utilities.batch_update(browser: 'chrome', timeout: 30)
     def batch_update(**settings)
-      settings.each { |key, value| config[key.to_s] = value }
+      reload_config
+      settings.each { |key, value| @config[key.to_s] = value }
       overwrite_yaml
     end
 
@@ -93,18 +89,24 @@ module Utilities
 
     private
 
-    # Single-key setter: updates in-memory config and writes once
+    # Single-key setter: reloads from disk, updates, and writes back
     def set(key, value)
-      config[key] = value
+      reload_config
+      @config[key] = value
       overwrite_yaml
     end
 
     def overwrite_yaml
-      File.open(@path, 'w') { |file| YAML.dump(config, file) }
+      File.open(@path, 'w') { |file| YAML.dump(@config, file) }
+    end
+
+    # Always reload from disk so external changes (e.g. desktop app) are picked up
+    def reload_config
+      @config = File.exist?(@path) ? YAML.load_file(@path) : {}
     end
 
     def config
-      @config ||= File.exist?(@path) ? YAML.load_file(@path) : nil
+      @config || reload_config
     end
   end
 end

data/lib/version

@@ -1 +1 @@
-3.0.0
+3.1.0

data/sig/commands/scaffolding_commands.rbs

@@ -7,22 +7,10 @@ class ScaffoldingCommands < Thor
   def steps: (String name) -> void
   def component: (String name) -> void
   def scaffold: (*String names) -> void
-  def destroy: (*String names) -> void
-  def from_url: (String url) -> void
 
   private
 
   def load_config_path: (String type) -> String?
-  def delete_scaffolding: (String name, String type) -> void
   def generate_scaffolding: (String name, String type, String? path, ?uses: Array[String]?) -> void
-  def dry_run_preview: (String name, String type) -> void
   def generate_default_scaffold: (String name) -> void
-  def generate_selected_components: (String name, Array[String] components) -> void
-  def generate_crud: (String name) -> void
-  def generate_spec_from_page: (String name, String source_file, ?ai: bool) -> void
-  def generate_model_data: (String name) -> void
-  def model_template: (String name) -> String
-  def destroy_scaffold: (String name, ?Array[String]? components) -> void
-  def detect_scaffold_types: () -> Array[String]
-  def interactive_scaffold: () -> void
 end

data/sig/commands/utility_commands.rbs

@@ -12,10 +12,4 @@ class UtilityCommands < Thor
   def debug: (String toggle) -> void
   def start_appium: () -> void
   def desktop: () -> void
-  def llm: (?String? provider) -> void
-
-  private
-
-  def configure_llm: (String provider) -> void
-  def show_llm_status: () -> void
 end

data/sig/generators/cucumber/cucumber_generator.rbs

@@ -6,11 +6,7 @@ class CucumberGenerator < Generator
   def generate_env_file: () -> void
   def generate_world: () -> void
   def generate_cucumber_file: () -> void
-  def generate_visual_feature: () -> void
-  def generate_visual_steps: () -> void
   def generate_accessibility_feature: () -> void
   def generate_accessibility_steps: () -> void
-  def generate_performance_feature: () -> void
-  def generate_performance_steps: () -> void
   def template_name: () -> String
 end

data/sig/generators/generator.rbs

@@ -15,24 +15,13 @@ class Generator < Thor::Group
   def ios?: () -> bool
   def android?: () -> bool
   def rspec?: () -> bool
-  def minitest?: () -> bool
   def selenium?: () -> bool
-  def capybara?: () -> bool
   def watir?: () -> bool
   def web?: () -> bool
   def selenium_based?: () -> bool
 
   # Add-on predicates
   def axe_addon?: () -> bool
-  def visual_addon?: () -> bool
-  def lighthouse_addon?: () -> bool
-
-  # Skip/reporter predicates
-  def skip_allure?: () -> bool
-  def skip_video?: () -> bool
-  def allure_reporter?: () -> bool
-  def junit_reporter?: () -> bool
-  def json_reporter?: () -> bool
 
   private
 

data/sig/generators/helper_generator.rbs

@@ -5,14 +5,10 @@ class HelpersGenerator < Generator
   private
 
   def generate_allure_helper: () -> void
-  def generate_video_helper: () -> void
+
   def generate_browser_helper: () -> void
   def generate_spec_helper: () -> void
   def generate_driver_helper: () -> void
-  def generate_capybara_helper: () -> void
   def generate_appium_helper: () -> void
-  def generate_test_helper: () -> void
   def generate_debug_helper: () -> void
-  def generate_visual_helper: () -> void
-  def generate_performance_helper: () -> void
 end

data/sig/generators/invoke_generators.rbs

@@ -3,8 +3,7 @@ module InvokeGenerators
   def self?.generate_framework: (?Hash[Symbol, untyped] structure) -> void
   def self?.add_generator: (Array[String] generators, *String gens) -> void
   def self?.invoke_generator: (?Hash[Symbol, untyped] structure) -> void
-  def self?.collect_skip_flags: (Hash[Symbol, untyped] structure) -> Array[String]
-  def self?.reporter_flags: (String? reporter) -> Array[String]
+  def self?.collect_flags: (Hash[Symbol, untyped] structure) -> Array[String]
   def self?.mobile_automation?: (String automation) -> bool
   def self?.to_bool: (String? string) -> bool?
 end

data/sig/generators/menu_generator.rbs

@@ -16,14 +16,10 @@ class MenuGenerator
   def offer_desktop_download: () -> void
   def select_test_framework: (String automation) -> void
   def create_framework_options: (Hash[Symbol, untyped] params) -> untyped
-  def select_reporter: (String framework, String automation_type, ?String? ci_platform) -> void
-  def select_accessibility: (String framework, String automation_type, ?String? ci_platform, ?String? reporter) -> void
-  def select_visual: (String framework, String automation_type, ?String? ci_platform, ?String? reporter, ?bool accessibility) -> void
-  def select_performance: (String framework, String automation_type, ?String? ci_platform, ?String? reporter, ?bool accessibility, ?bool visual) -> void
+  def select_accessibility: (String framework, String automation_type) -> void
   def mobile_automation?: (String automation_type) -> bool
-  def create_framework: (String framework, String automation_type, ?String? ci_platform, ?String? reporter, ?bool accessibility, ?bool visual, ?bool performance) -> void
+  def create_framework: (String framework, String automation_type, ?accessibility: bool) -> void
   def yes_no_menu_choices: () -> Hash[Symbol, Proc]
   def select_automation_framework: (untyped menu) -> void
   def automation_options: (untyped menu) -> void
-  def select_ci_platform: (String framework, String automation) -> void
 end

data/sig/generators/rspec/rspec_generator.rbs

@@ -2,7 +2,5 @@
 class RspecGenerator < Generator
   def generate_login_spec: () -> void
   def generate_pdp_spec: () -> void
-  def generate_visual_spec: () -> void
   def generate_accessibility_spec: () -> void
-  def generate_performance_spec: () -> void
 end

data/sig/ruby_raider.rbs

@@ -1,15 +1,13 @@
 # Type definitions for main RubyRaider entry point
 module RubyRaider
   class Raider < Thor
-    def self.plugin_commands?: () -> bool
     def current_version: () -> String
 
     def new: (String project_name) -> void
     def version: () -> void
-    def adopt: () -> void
 
     private
 
-    def merge_skip_flags: (Hash[Symbol, untyped] params) -> void
+    def merge_flags: (Hash[Symbol, untyped] params) -> void
   end
 end

data/sig/scaffolding/project_detector.rbs

@@ -8,7 +8,6 @@ module ScaffoldProjectDetector
   def self?.detect_framework: (?String? gemfile) -> String
   def self?.read_gemfile: () -> String
   def self?.selenium?: () -> bool
-  def self?.capybara?: () -> bool
   def self?.watir?: () -> bool
   def self?.config: () -> Hash[String, untyped]
 end

data/sig/scaffolding/scaffolding.rbs

@@ -2,12 +2,9 @@
 class Scaffolding < Thor::Group
   include Thor::Actions
 
-  OVERRIDE_DIR: String
-
   attr_writer uses: Array[String]
 
   def self.source_root: () -> String
-  def self.planned_path: (String name, String type, ?String? custom_path) -> String?
 
   # Generation methods
   def generate_page: () -> void
@@ -17,18 +14,6 @@ class Scaffolding < Thor::Group
   def generate_steps: () -> void
   def generate_component: () -> void
   def generate_config: () -> void
-  def generate_spec_from_page: (String source_file, ?ai: bool) -> void
-  def generate_page_from_url: (Hash[Symbol, untyped] analysis) -> void
-  def generate_spec_from_url: (Hash[Symbol, untyped] analysis) -> void
-
-  # Deletion methods
-  def delete_page: () -> void
-  def delete_feature: () -> void
-  def delete_spec: () -> void
-  def delete_helper: () -> void
-  def delete_steps: () -> void
-  def delete_component: () -> void
-  def delete_config: () -> void
 
   # Template helpers
   def class_name: () -> String?
@@ -41,15 +26,7 @@ class Scaffolding < Thor::Group
   def framework_type: () -> String
   def project_config: () -> Hash[String, untyped]
   def selenium?: () -> bool
-  def capybara?: () -> bool
   def watir?: () -> bool
   def uses_list: () -> Array[String]
-  def introspected: () -> PageIntrospector?
-  def url_data: () -> Hash[Symbol, untyped]?
-  def ai_scenarios: () -> Hash[Symbol, untyped]
   def default_path: (String standard_path, String file_type) -> String
-
-  private
-
-  def enrich_with_ai_scenarios: () -> void
 end

data/sig/utilities/utilities.rbs

@@ -13,10 +13,6 @@ module Utilities
   def self?.platform=: (String platform) -> void
   def self?.browser_options=: (Array[String] opts) -> void
   def self?.delete_browser_options: () -> void
-  def self?.llm_provider=: (String provider) -> void
-  def self?.llm_api_key=: (String key) -> void
-  def self?.llm_model=: (String model) -> void
-  def self?.llm_url=: (String url) -> void
   def self?.debug=: (bool enabled) -> void
   def self?.batch_update: (**untyped settings) -> void
   def self?.run: (?String? opts) -> void

data/spec/commands/raider_commands_spec.rb

@@ -45,10 +45,6 @@ RSpec.describe RubyRaider::Raider do
       expect(described_class.map['v']).to eq('version')
     end
 
-    it 'maps a to adopt' do
-      expect(described_class.map['a']).to eq('adopt')
-    end
-
     it 'maps g to generate' do
       expect(described_class.map['g']).to eq('generate')
     end
@@ -56,19 +52,11 @@ RSpec.describe RubyRaider::Raider do
     it 'maps u to utility' do
       expect(described_class.map['u']).to eq('utility')
     end
-
-    it 'maps pm to plugin_manager' do
-      expect(described_class.map['pm']).to eq('plugin_manager')
-    end
   end
 
   describe 'registered subcommands' do
     let(:subcommands) { described_class.subcommands }
 
-    it 'includes adopt' do
-      expect(subcommands).to include('adopt')
-    end
-
     it 'includes generate' do
       expect(subcommands).to include('generate')
     end
@@ -76,54 +64,5 @@ RSpec.describe RubyRaider::Raider do
     it 'includes utility' do
       expect(subcommands).to include('utility')
     end
-
-    it 'includes plugin_manager' do
-      expect(subcommands).to include('plugin_manager')
-    end
-  end
-end
-
-RSpec.describe AdoptCommands do
-  let(:source_dir) { 'tmp_cmd_adopt_source' }
-  let(:output_dir) { 'tmp_cmd_adopt_output' }
-
-  before do
-    FileUtils.mkdir_p(source_dir)
-    File.write("#{source_dir}/Gemfile", "gem 'rspec'\ngem 'selenium-webdriver'\n")
-    FileUtils.mkdir_p("#{source_dir}/spec")
-  end
-
-  after do
-    FileUtils.rm_rf(source_dir)
-    FileUtils.rm_rf(output_dir)
-  end
-
-  describe 'project command with parameters' do
-    it 'calls AdoptMenu.adopt with parsed params' do
-      result = { plan: double(warnings: [], output_path: output_dir), results: { pages: 0, tests: 0, features: 0, steps: 0, errors: [] } } # rubocop:disable RSpec/VerifiedDoubles
-      allow(Adopter::AdoptMenu).to receive(:adopt).and_return(result)
-
-      expect do
-        described_class.new.invoke(:project, nil, [source_dir, '-p',
-                                                   "output_path:#{output_dir}",
-                                                   'target_automation:selenium',
-                                                   'target_framework:rspec'])
-      end.to output(/Adoption complete/).to_stdout
-
-      expect(Adopter::AdoptMenu).to have_received(:adopt).with(
-        hash_including(source_path: source_dir, target_automation: 'selenium', target_framework: 'rspec')
-      )
-    end
-  end
-
-  describe 'project command without parameters' do
-    it 'calls AdoptMenu#run' do
-      menu = instance_double(Adopter::AdoptMenu, run: nil)
-      allow(Adopter::AdoptMenu).to receive(:new).and_return(menu)
-
-      described_class.new.invoke(:project, nil, [source_dir])
-
-      expect(menu).to have_received(:run)
-    end
   end
 end

data/spec/commands/scaffolding_commands_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'open3'
+
+RSpec.describe 'ScaffoldingCommands' do
+  describe 'dependencies' do
+    it 'loads without relying on transitive requires for Pathname' do
+      # Loads the file in a clean Ruby process where nothing else has required 'pathname'.
+      # This catches the exact bug where Pathname is used but never explicitly required,
+      # which works in test suites (rspec loads pathname transitively) but crashes at runtime.
+      script = <<~RUBY
+        require_relative '../../lib/commands/scaffolding_commands'
+        puts 'OK'
+      RUBY
+
+      stdout, stderr, status = Open3.capture3('ruby', '-e', script, chdir: __dir__)
+      expect(status.success?).to be(true),
+                                 "ScaffoldingCommands failed to load in isolation:\n#{stderr}"
+      expect(stdout.strip).to eq('OK')
+    end
+  end
+end