rubyn-code 0.1.0 → 0.2.0

data/lib/rubyn_code/tools/glob.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class Glob < Base
-      TOOL_NAME = "glob"
-      DESCRIPTION = "File pattern matching. Returns sorted list of file paths matching the glob pattern."
+      TOOL_NAME = 'glob'
+      DESCRIPTION = 'File pattern matching. Returns sorted list of file paths matching the glob pattern.'
       PARAMETERS = {
         pattern: { type: :string, required: true, description: "Glob pattern (e.g. '**/*.rb', 'app/**/*.erb')" },
-        path: { type: :string, required: false, description: "Directory to search in (defaults to project root)" }
+        path: { type: :string, required: false, description: 'Directory to search in (defaults to project root)' }
       }.freeze
       RISK_LEVEL = :read
       REQUIRES_CONFIRMATION = false
@@ -18,16 +18,14 @@ module RubynCode
       def execute(pattern:, path: nil)
         search_dir = path ? safe_path(path) : project_root
 
-        unless File.directory?(search_dir)
-          raise Error, "Directory not found: #{path || project_root}"
-        end
+        raise Error, "Directory not found: #{path || project_root}" unless File.directory?(search_dir)
 
         full_pattern = File.join(search_dir, pattern)
         matches = Dir.glob(full_pattern, File::FNM_DOTMATCH).sort
 
         matches
           .select { |f| File.file?(f) }
-          .reject { |f| File.basename(f).start_with?(".") && File.basename(f) == "." || File.basename(f) == ".." }
+          .reject { |f| (File.basename(f).start_with?('.') && File.basename(f) == '.') || File.basename(f) == '..' }
           .map { |f| relative_to_root(f) }
           .join("\n")
       end

data/lib/rubyn_code/tools/grep.rb

@@ -1,18 +1,20 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class Grep < Base
-      TOOL_NAME = "grep"
-      DESCRIPTION = "Searches file contents using regular expressions. Returns matching lines with file paths and line numbers."
+      TOOL_NAME = 'grep'
+      DESCRIPTION = 'Searches file contents using regular expressions. Returns matching lines with file paths and line numbers.'
       PARAMETERS = {
-        pattern: { type: :string, required: true, description: "Regular expression pattern to search for" },
-        path: { type: :string, required: false, description: "File or directory to search in (defaults to project root)" },
+        pattern: { type: :string, required: true, description: 'Regular expression pattern to search for' },
+        path: { type: :string, required: false,
+                description: 'File or directory to search in (defaults to project root)' },
         glob_filter: { type: :string, required: false, description: "Glob pattern to filter files (e.g. '*.rb')" },
-        max_results: { type: :integer, required: false, default: 50, description: "Maximum number of matching lines to return" }
+        max_results: { type: :integer, required: false, default: 50,
+                       description: 'Maximum number of matching lines to return' }
       }.freeze
       RISK_LEVEL = :read
       REQUIRES_CONFIRMATION = false
@@ -41,7 +43,7 @@ module RubynCode
         if File.file?(search_path)
           [search_path]
         elsif File.directory?(search_path)
-          glob_pattern = glob_filter || "**/*"
+          glob_pattern = glob_filter || '**/*'
           Dir.glob(File.join(search_path, glob_pattern))
              .select { |f| File.file?(f) }
              .reject { |f| binary_file?(f) }
@@ -70,7 +72,7 @@ module RubynCode
         sample = File.read(path, 512)
         return false if sample.nil?
 
-        sample.bytes.any? { |b| b.zero? }
+        sample.bytes.any?(&:zero?)
       rescue StandardError
         true
       end

data/lib/rubyn_code/tools/load_skill.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class LoadSkill < Base
-      TOOL_NAME = "load_skill"
-      DESCRIPTION = "Loads a skill document into the conversation context. Use /skill-name or provide the skill name."
+      TOOL_NAME = 'load_skill'
+      DESCRIPTION = 'Loads a skill document into the conversation context. Use /skill-name or provide the skill name.'
       PARAMETERS = {
-        name: { type: :string, required: true, description: "Name of the skill to load" }
+        name: { type: :string, required: true, description: 'Name of the skill to load' }
       }.freeze
       RISK_LEVEL = :read
       REQUIRES_CONFIRMATION = false
@@ -28,8 +28,8 @@ module RubynCode
 
       def default_loader
         skills_dirs = [
-          File.join(project_root, ".rubyn", "skills"),
-          File.join(Dir.home, ".rubyn", "skills")
+          File.join(project_root, '.rubyn', 'skills'),
+          File.join(Dir.home, '.rubyn', 'skills')
         ]
         catalog = Skills::Catalog.new(skills_dirs)
         Skills::Loader.new(catalog)

data/lib/rubyn_code/tools/memory_search.rb

@@ -1,20 +1,21 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class MemorySearch < Base
-      TOOL_NAME = "memory_search"
-      DESCRIPTION = "Searches project memories using full-text search. " \
-                    "Returns relevant memories including code patterns, user preferences, " \
-                    "project conventions, error resolutions, and past decisions."
+      TOOL_NAME = 'memory_search'
+      DESCRIPTION = 'Searches project memories using full-text search. ' \
+                    'Returns relevant memories including code patterns, user preferences, ' \
+                    'project conventions, error resolutions, and past decisions.'
       PARAMETERS = {
-        query: { type: :string, required: true, description: "Search query for finding relevant memories" },
-        tier: { type: :string, required: false, description: "Filter by memory tier: short, medium, or long" },
-        category: { type: :string, required: false, description: "Filter by category: code_pattern, user_preference, project_convention, error_resolution, or decision" },
-        limit: { type: :integer, required: false, description: "Maximum number of results to return (default 10)" }
+        query: { type: :string, required: true, description: 'Search query for finding relevant memories' },
+        tier: { type: :string, required: false, description: 'Filter by memory tier: short, medium, or long' },
+        category: { type: :string, required: false,
+                    description: 'Filter by category: code_pattern, user_preference, project_convention, error_resolution, or decision' },
+        limit: { type: :integer, required: false, description: 'Maximum number of results to return (default 10)' }
       }.freeze
       RISK_LEVEL = :read
       REQUIRES_CONFIRMATION = false
@@ -55,9 +56,9 @@ module RubynCode
           lines << "Tier: #{record.tier} | Category: #{record.category || 'none'}"
           lines << "Relevance: #{format('%.2f', record.relevance_score)} | Accessed: #{record.access_count} times"
           lines << "Created: #{record.created_at}"
-          lines << ""
+          lines << ''
           lines << record.content
-          lines << ""
+          lines << ''
         end
 
         lines.join("\n")

data/lib/rubyn_code/tools/memory_write.rb

@@ -1,21 +1,23 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class MemoryWrite < Base
-      TOOL_NAME = "memory_write"
-      DESCRIPTION = "Writes a new memory to the project memory store. " \
-                    "Use this to persist code patterns, user preferences, project conventions, " \
-                    "error resolutions, or architectural decisions for future reference."
+      TOOL_NAME = 'memory_write'
+      DESCRIPTION = 'Writes a new memory to the project memory store. ' \
+                    'Use this to persist code patterns, user preferences, project conventions, ' \
+                    'error resolutions, or architectural decisions for future reference.'
       PARAMETERS = {
-        content: { type: :string, required: true, description: "The memory content to store" },
-        tier: { type: :string, required: false, description: "Memory retention tier: short, medium (default), or long" },
-        category: { type: :string, required: false, description: "Category: code_pattern, user_preference, project_convention, error_resolution, or decision" }
+        content: { type: :string, required: true, description: 'The memory content to store' },
+        tier: { type: :string, required: false,
+                description: 'Memory retention tier: short, medium (default), or long' },
+        category: { type: :string, required: false,
+                    description: 'Category: code_pattern, user_preference, project_convention, error_resolution, or decision' }
       }.freeze
-      RISK_LEVEL = :read  # Memory is internal — no user approval needed
+      RISK_LEVEL = :read # Memory is internal — no user approval needed
 
       # @param project_root [String]
       # @param memory_store [Memory::Store] injected store instance
@@ -28,12 +30,12 @@ module RubynCode
       # @param tier [String] defaults to "medium"
       # @param category [String, nil]
       # @return [String] confirmation message
-      def execute(content:, tier: "medium", category: nil)
+      def execute(content:, tier: 'medium', category: nil)
         store = @memory_store || resolve_memory_store
         record = store.write(content: content, tier: tier, category: category)
 
         "Memory saved (ID: #{record.id}, tier: #{record.tier}" \
-          "#{record.category ? ", category: #{record.category}" : ''})."
+          "#{", category: #{record.category}" if record.category})."
       end
 
       private

data/lib/rubyn_code/tools/rails_generate.rb

@@ -1,17 +1,19 @@
 # frozen_string_literal: true
 
-require "open3"
-require_relative "base"
-require_relative "registry"
+require 'open3'
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class RailsGenerate < Base
-      TOOL_NAME = "rails_generate"
-      DESCRIPTION = "Runs a Rails generator command. Validates that the project is a Rails application."
+      TOOL_NAME = 'rails_generate'
+      DESCRIPTION = 'Runs a Rails generator command. Validates that the project is a Rails application.'
       PARAMETERS = {
-        generator: { type: :string, required: true, description: "Generator name (e.g. 'model', 'controller', 'migration')" },
-        args: { type: :string, required: true, description: "Arguments for the generator (e.g. 'User name:string email:string')" }
+        generator: { type: :string, required: true,
+                     description: "Generator name (e.g. 'model', 'controller', 'migration')" },
+        args: { type: :string, required: true,
+                description: "Arguments for the generator (e.g. 'User name:string email:string')" }
       }.freeze
       RISK_LEVEL = :execute
       REQUIRES_CONFIRMATION = false
@@ -20,7 +22,7 @@ module RubynCode
         validate_rails_project!
 
         command = "bundle exec rails generate #{generator} #{args}"
-        stdout, stderr, status = Open3.capture3(command, chdir: project_root)
+        stdout, stderr, status = safe_capture3(command, chdir: project_root)
 
         build_output(stdout, stderr, status)
       end
@@ -28,16 +30,14 @@ module RubynCode
       private
 
       def validate_rails_project!
-        gemfile_path = File.join(project_root, "Gemfile")
+        gemfile_path = File.join(project_root, 'Gemfile')
 
-        unless File.exist?(gemfile_path)
-          raise Error, "No Gemfile found. This does not appear to be a Ruby project."
-        end
+        raise Error, 'No Gemfile found. This does not appear to be a Ruby project.' unless File.exist?(gemfile_path)
 
         gemfile_content = File.read(gemfile_path)
-        unless gemfile_content.match?(/['"]rails['"]/)
-          raise Error, "Gemfile does not include Rails. This does not appear to be a Rails project."
-        end
+        return if gemfile_content.match?(/['"]rails['"]/)
+
+        raise Error, 'Gemfile does not include Rails. This does not appear to be a Rails project.'
       end
 
       def build_output(stdout, stderr, status)
@@ -45,7 +45,7 @@ module RubynCode
         parts << stdout unless stdout.empty?
         parts << "STDERR:\n#{stderr}" unless stderr.empty?
         parts << "Exit code: #{status.exitstatus}" unless status.success?
-        parts.empty? ? "(no output)" : parts.join("\n")
+        parts.empty? ? '(no output)' : parts.join("\n")
       end
     end
 

data/lib/rubyn_code/tools/read_file.rb

@@ -1,17 +1,18 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class ReadFile < Base
-      TOOL_NAME = "read_file"
-      DESCRIPTION = "Reads a file from the filesystem. Returns file content with line numbers prepended."
+      TOOL_NAME = 'read_file'
+      DESCRIPTION = 'Reads a file from the filesystem. Returns file content with line numbers prepended.'
       PARAMETERS = {
-        path: { type: :string, required: true, description: "Path to the file to read (relative to project root or absolute)" },
-        offset: { type: :integer, required: false, description: "Line number to start reading from (1-based)" },
-        limit: { type: :integer, required: false, description: "Number of lines to read" }
+        path: { type: :string, required: true,
+                description: 'Path to the file to read (relative to project root or absolute)' },
+        offset: { type: :integer, required: false, description: 'Line number to start reading from (1-based)' },
+        limit: { type: :integer, required: false, description: 'Number of lines to read' }
       }.freeze
       RISK_LEVEL = :read
       REQUIRES_CONFIRMATION = false

data/lib/rubyn_code/tools/read_inbox.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     # Tool for reading unread messages from a teammate's inbox.
     class ReadInbox < Base
-      TOOL_NAME = "read_inbox"
+      TOOL_NAME = 'read_inbox'
       DESCRIPTION = "Reads all unread messages from the agent's inbox and marks them as read."
       PARAMETERS = {
-        name: { type: :string, required: true, description: "The agent name whose inbox to read" }
+        name: { type: :string, required: true, description: 'The agent name whose inbox to read' }
       }.freeze
       RISK_LEVEL = :read
       REQUIRES_CONFIRMATION = false
@@ -27,7 +27,7 @@ module RubynCode
       # @param name [String] the reader's agent name
       # @return [String] formatted messages or a notice if the inbox is empty
       def execute(name:)
-        raise Error, "Agent name is required" if name.nil? || name.strip.empty?
+        raise Error, 'Agent name is required' if name.nil? || name.strip.empty?
 
         messages = @mailbox.read_inbox(name)
 

data/lib/rubyn_code/tools/registry.rb

@@ -34,9 +34,9 @@ module RubynCode
         end
 
         def load_all!
-          tool_files = Dir[File.join(__dir__, "*.rb")]
+          tool_files = Dir[File.join(__dir__, '*.rb')]
           tool_files.each do |file|
-            basename = File.basename(file, ".rb")
+            basename = File.basename(file, '.rb')
             next if %w[base registry schema executor].include?(basename)
 
             require_relative basename

data/lib/rubyn_code/tools/review_pr.rb

@@ -1,19 +1,19 @@
 # frozen_string_literal: true
 
-require_relative "base"
-require_relative "registry"
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class ReviewPr < Base
-      TOOL_NAME = "review_pr"
-      DESCRIPTION = "Review current branch changes against Ruby/Rails best practices. " \
-                    "Gets the diff of the current branch vs the base branch, analyzes each changed file, " \
-                    "and provides actionable suggestions with explanations."
+      TOOL_NAME = 'review_pr'
+      DESCRIPTION = 'Review current branch changes against Ruby/Rails best practices. ' \
+                    'Gets the diff of the current branch vs the base branch, analyzes each changed file, ' \
+                    'and provides actionable suggestions with explanations.'
       PARAMETERS = {
         base_branch: {
           type: :string,
-          description: "Base branch to diff against (default: main)",
+          description: 'Base branch to diff against (default: main)',
           required: false
         },
         focus: {
@@ -24,25 +24,25 @@ module RubynCode
       }.freeze
       RISK_LEVEL = :read
 
-      def execute(base_branch: "main", focus: "all")
+      def execute(base_branch: 'main', focus: 'all')
         # Check git is available
-        unless system("git rev-parse --is-inside-work-tree > /dev/null 2>&1", chdir: project_root)
-          return "Error: Not a git repository or git is not installed."
+        unless system('git rev-parse --is-inside-work-tree > /dev/null 2>&1', chdir: project_root)
+          return 'Error: Not a git repository or git is not installed.'
         end
 
         # Get current branch
-        current = run_git("rev-parse --abbrev-ref HEAD").strip
-        return "Error: Could not determine current branch." if current.empty?
+        current = run_git('rev-parse --abbrev-ref HEAD').strip
+        return 'Error: Could not determine current branch.' if current.empty?
 
         if current == base_branch
           return "You're on #{base_branch}. Switch to a feature branch first, or specify a different base: review_pr(base_branch: 'develop')"
         end
 
         # Check base branch exists
-        unless run_git("rev-parse --verify #{base_branch} 2>/dev/null").strip.length > 0
+        unless run_git("rev-parse --verify #{base_branch} 2>/dev/null").strip.length.positive?
           # Try origin/main
           base_branch = "origin/#{base_branch}"
-          unless run_git("rev-parse --verify #{base_branch} 2>/dev/null").strip.length > 0
+          unless run_git("rev-parse --verify #{base_branch} 2>/dev/null").strip.length.positive?
             return "Error: Base branch '#{base_branch}' not found."
           end
         end
@@ -57,33 +57,33 @@ module RubynCode
         commit_log = run_git("log #{base_branch}..HEAD --oneline")
 
         # Build the review context
-        ruby_files = files_changed.select { |f| f.match?(/\.(rb|rake|gemspec|ru)$/) }
-        erb_files = files_changed.select { |f| f.match?(/\.(erb|haml|slim)$/) }
-        spec_files = files_changed.select { |f| f.match?(/_spec\.rb$|_test\.rb$/) }
-        migration_files = files_changed.select { |f| f.include?("db/migrate") }
-        config_files = files_changed.select { |f| f.match?(/config\/|\.yml$|\.yaml$/) }
+        ruby_files = files_changed.grep(/\.(rb|rake|gemspec|ru)$/)
+        erb_files = files_changed.grep(/\.(erb|haml|slim)$/)
+        spec_files = files_changed.grep(/_spec\.rb$|_test\.rb$/)
+        migration_files = files_changed.select { |f| f.include?('db/migrate') }
+        config_files = files_changed.grep(%r{config/|\.yml$|\.yaml$})
 
         review = []
         review << "# PR Review: #{current} → #{base_branch}"
-        review << ""
-        review << "## Summary"
+        review << ''
+        review << '## Summary'
         review << stat
-        review << ""
-        review << "## Commits"
+        review << ''
+        review << '## Commits'
         review << commit_log
-        review << ""
-        review << "## Files by Category"
+        review << ''
+        review << '## Files by Category'
         review << "- Ruby: #{ruby_files.length} files" unless ruby_files.empty?
         review << "- Templates: #{erb_files.length} files" unless erb_files.empty?
         review << "- Specs: #{spec_files.length} files" unless spec_files.empty?
         review << "- Migrations: #{migration_files.length} files" unless migration_files.empty?
         review << "- Config: #{config_files.length} files" unless config_files.empty?
-        review << ""
+        review << ''
 
         # Add focus-specific review instructions
         review << "## Review Focus: #{focus.upcase}"
         review << review_instructions(focus)
-        review << ""
+        review << ''
 
         # Add the diff (truncated if too large)
         if diff.length > 40_000
@@ -91,24 +91,24 @@ module RubynCode
           review << diff[0...40_000]
           review << "\n... [truncated #{diff.length - 40_000} chars]"
         else
-          review << "## Full Diff"
+          review << '## Full Diff'
           review << diff
         end
 
-        review << ""
-        review << "---"
-        review << "Review this diff against Ruby/Rails best practices. For each issue found:"
-        review << "1. Quote the specific code"
+        review << ''
+        review << '---'
+        review << 'Review this diff against Ruby/Rails best practices. For each issue found:'
+        review << '1. Quote the specific code'
         review << "2. Explain what's wrong and WHY it matters"
-        review << "3. Show the suggested fix"
-        review << "4. Rate severity: [critical] [warning] [suggestion] [nitpick]"
-        review << ""
-        review << "Also check for:"
-        review << "- Missing tests for new code"
-        review << "- N+1 queries in ActiveRecord changes"
-        review << "- Security issues (SQL injection, XSS, mass assignment)"
-        review << "- Missing database indexes for new associations"
-        review << "- Proper error handling"
+        review << '3. Show the suggested fix'
+        review << '4. Rate severity: [critical] [warning] [suggestion] [nitpick]'
+        review << ''
+        review << 'Also check for:'
+        review << '- Missing tests for new code'
+        review << '- N+1 queries in ActiveRecord changes'
+        review << '- Security issues (SQL injection, XSS, mass assignment)'
+        review << '- Missing database indexes for new associations'
+        review << '- Proper error handling'
 
         truncate(review.join("\n"))
       end
@@ -121,21 +121,21 @@ module RubynCode
 
       def review_instructions(focus)
         case focus.to_s.downcase
-        when "security"
-          "Focus on: SQL injection, XSS, CSRF, mass assignment, authentication/authorization gaps, " \
-          "sensitive data exposure, insecure dependencies, command injection, path traversal."
-        when "performance"
-          "Focus on: N+1 queries, missing indexes, eager loading, caching opportunities, " \
-          "unnecessary database calls, memory bloat, slow iterations, missing pagination."
-        when "style"
-          "Focus on: Ruby idioms, naming conventions, method length, class organization, " \
-          "frozen string literals, guard clauses, DRY violations, dead code."
-        when "testing"
-          "Focus on: Missing test coverage, test quality, factory usage, assertion quality, " \
-          "test isolation, flaky test risks, edge cases, integration vs unit test balance."
+        when 'security'
+          'Focus on: SQL injection, XSS, CSRF, mass assignment, authentication/authorization gaps, ' \
+          'sensitive data exposure, insecure dependencies, command injection, path traversal.'
+        when 'performance'
+          'Focus on: N+1 queries, missing indexes, eager loading, caching opportunities, ' \
+          'unnecessary database calls, memory bloat, slow iterations, missing pagination.'
+        when 'style'
+          'Focus on: Ruby idioms, naming conventions, method length, class organization, ' \
+          'frozen string literals, guard clauses, DRY violations, dead code.'
+        when 'testing'
+          'Focus on: Missing test coverage, test quality, factory usage, assertion quality, ' \
+          'test isolation, flaky test risks, edge cases, integration vs unit test balance.'
         else
-          "Review all aspects: code quality, security, performance, testing, Rails conventions, " \
-          "Ruby idioms, and architectural patterns."
+          'Review all aspects: code quality, security, performance, testing, Rails conventions, ' \
+          'Ruby idioms, and architectural patterns.'
         end
       end
     end

data/lib/rubyn_code/tools/run_specs.rb

@@ -1,27 +1,28 @@
 # frozen_string_literal: true
 
-require "open3"
-require_relative "base"
-require_relative "registry"
+require 'open3'
+require_relative 'base'
+require_relative 'registry'
 
 module RubynCode
   module Tools
     class RunSpecs < Base
-      TOOL_NAME = "run_specs"
-      DESCRIPTION = "Runs RSpec or Minitest specs. Auto-detects which test framework is in use."
+      TOOL_NAME = 'run_specs'
+      DESCRIPTION = 'Runs RSpec or Minitest specs. Auto-detects which test framework is in use.'
       PARAMETERS = {
-        path: { type: :string, required: false, description: "Specific test file or directory to run" },
-        format: { type: :string, required: false, default: "documentation", description: "Output format (default: 'documentation')" },
-        fail_fast: { type: :boolean, required: false, description: "Stop on first failure" }
+        path: { type: :string, required: false, description: 'Specific test file or directory to run' },
+        format: { type: :string, required: false, default: 'documentation',
+                  description: "Output format (default: 'documentation')" },
+        fail_fast: { type: :boolean, required: false, description: 'Stop on first failure' }
       }.freeze
       RISK_LEVEL = :execute
       REQUIRES_CONFIRMATION = false
 
-      def execute(path: nil, format: "documentation", fail_fast: false)
+      def execute(path: nil, format: 'documentation', fail_fast: false)
         framework = detect_framework
 
         command = build_command(framework, path, format, fail_fast)
-        stdout, stderr, status = Open3.capture3(command, chdir: project_root)
+        stdout, stderr, status = safe_capture3(command, chdir: project_root)
 
         build_output(stdout, stderr, status)
       end
@@ -29,7 +30,7 @@ module RubynCode
       private
 
       def detect_framework
-        gemfile_path = File.join(project_root, "Gemfile")
+        gemfile_path = File.join(project_root, 'Gemfile')
 
         if File.exist?(gemfile_path)
           content = File.read(gemfile_path)
@@ -37,26 +38,26 @@ module RubynCode
           return :minitest if content.match?(/['"]minitest['"]/)
         end
 
-        return :rspec if File.exist?(File.join(project_root, ".rspec"))
-        return :rspec if File.directory?(File.join(project_root, "spec"))
-        return :minitest if File.directory?(File.join(project_root, "test"))
+        return :rspec if File.exist?(File.join(project_root, '.rspec'))
+        return :rspec if File.directory?(File.join(project_root, 'spec'))
+        return :minitest if File.directory?(File.join(project_root, 'test'))
 
-        raise Error, "Could not detect test framework. Ensure RSpec or Minitest is configured."
+        raise Error, 'Could not detect test framework. Ensure RSpec or Minitest is configured.'
       end
 
       def build_command(framework, path, format, fail_fast)
         case framework
         when :rspec
-          cmd = "bundle exec rspec"
+          cmd = 'bundle exec rspec'
           cmd += " --format #{format}" if format
-          cmd += " --fail-fast" if fail_fast
+          cmd += ' --fail-fast' if fail_fast
           cmd += " #{path}" if path
           cmd
         when :minitest
           if path
             "bundle exec ruby -Itest #{path}"
           else
-            "bundle exec rails test"
+            'bundle exec rails test'
           end
         end
       end
@@ -66,7 +67,7 @@ module RubynCode
         parts << stdout unless stdout.empty?
         parts << "STDERR:\n#{stderr}" unless stderr.empty?
         parts << "Exit code: #{status.exitstatus}" unless status.success?
-        parts.empty? ? "(no output)" : parts.join("\n")
+        parts.empty? ? '(no output)' : parts.join("\n")
       end
     end