solidarity 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11cfca9758e874328f377cf0657fbd8632f00302a3dd9f2ac7ee2f2d7fe5e4a9
-  data.tar.gz: b23fff6c7471189cabad4c86ad1a2e043d9fbf606054efdaeef7cf28a0483f4b
+  metadata.gz: 9da1a43ee95a6830a9cf80311f4e4f3ba226e5fa07c1e47de8af77006906db9b
+  data.tar.gz: e09c1317b1da612d1c325fcabb781408d2ad77a779bacb942d03eff81ca2c273
 SHA512:
-  metadata.gz: eb1fc0758c2b738d5eda80f9f31e74d715172d01f6d198e18a659aea0ae8d7178f52802be66a1cb24ca9d6585e1c2a56f741ca86e0a119012afb545378848403
-  data.tar.gz: bab76cdf46d6727eda7505ccc057a7c733cd8109d462ce50484effead92dec31b60a522c710b8bed351277db831b9e26c8dc87520152b5bfc01689e97734e06f
+  metadata.gz: 53284a3396d78311d11e86fa771dd4fd10630da1df562022a6f1a028b860cfb06406c0ee4cc7dafff9dfcc3ad3d09a2763904a87729414c82be4012028836ec1
+  data.tar.gz: d5caf90cca1bb333bb30dd6849359c85fea19a99404e9ab2744dcfca3dcf0189f1e8038e83586f85739d92e48d021b3f6a9e0ab4eb65757c23fae782b1025c68

data/exe/solidarity

@@ -1,20 +1,37 @@
 #!/usr/bin/env ruby
 
 require "solidarity"
+require 'optparse'
 
-# TODO: Add option parsing (e.g., using OptionParser)
+options = {}
+OptionParser.new do |opts|
+  opts.banner = "Usage: #{File.basename($PROGRAM_NAME)} [options] <path_to_ruby_project>"
+
+  opts.on("--with-specs", "Include spec/test files in the analysis") do
+    options[:with_specs] = true
+  end
+
+  opts.on("--with-srp-details", "Include detailed SRP scores in the report") do
+    options[:with_srp_details] = true
+  end
+
+  opts.on("-h", "--help", "Prints this help") do
+    puts opts
+    exit
+  end
+end.parse!
 
 project_path = ARGV[0]
 
 unless project_path
-  warn "Usage: #{File.basename($PROGRAM_NAME)} <path_to_ruby_project>"
+  warn "Usage: #{File.basename($PROGRAM_NAME)} [options] <path_to_ruby_project>"
   exit 1
 end
 
 begin
-  graph = Solidarity::RailRoadyRunner.run(project_path)
+  graph = Solidarity::RailRoadyRunner.run(project_path, with_specs: options[:with_specs])
   solid_results = Solidarity::SolidEvaluator.new(graph).evaluate_all
-  report = Solidarity::Reporter.generate_report(solid_results)
+  report = Solidarity::Reporter.generate_report(solid_results, with_srp_details: options[:with_srp_details])
   puts report
 rescue => e
   warn "Error: #{e.message}"

data/lib/solidarity/ast_processor.rb

@@ -14,9 +14,27 @@ module Solidarity
     end
 
     def process_file(file_path)
-      content = File.read(file_path)
-      ast = Parser::CurrentRuby.parse(content)
-      traverse_ast(ast, nil) # Pass initial context as nil
+      content = File.read(file_path, encoding: 'UTF-8')
+      begin
+        ast = Parser::CurrentRuby.parse(content)
+        traverse_ast(ast, nil) # Pass initial context as nil
+      rescue Parser::SyntaxError => e
+        warn "Warning: Could not parse #{file_path} due to syntax error: #{e.message}"
+        nil # Return nil to indicate parsing failed
+      rescue EncodingError => e
+        warn "Warning: Could not read #{file_path} due to encoding error: #{e.message}. Attempting to force encoding to UTF-8."
+        begin
+          content.force_encoding('UTF-8')
+          ast = Parser::CurrentRuby.parse(content)
+          traverse_ast(ast, nil)
+        rescue Parser::SyntaxError => e
+          warn "Warning: Could not parse #{file_path} after forcing encoding due to syntax error: #{e.message}"
+          nil
+        rescue EncodingError => e
+          warn "Warning: Could not read #{file_path} after forcing encoding due to encoding error: #{e.message}"
+          nil
+        end
+      end
     end
 
     private

data/lib/solidarity/railroady_runner.rb

@@ -4,19 +4,21 @@ require_relative 'ast_processor' # Add this line
 
 module Solidarity
   class RailRoadyRunner
-    def self.run(project_path)
+    def self.run(project_path, with_specs: false)
       unless File.directory?(project_path)
         raise ArgumentError, "Project path '#{project_path}' is not a valid directory."
       end
 
-      # Remove Railroady's Rails-specific environment loading
-      # options = OptionsStruct.new(root: project_path, verbose: false, output: nil)
-      # diagram = ModelsDiagram.new(options)
-      # diagram.process
-      # diagram.generate
-      # diagram.graph
+      all_ruby_files = Dir.glob(File.join(project_path, "**", "*.rb"))
+
+      ruby_files = if with_specs
+                     all_ruby_files
+                   else
+                     all_ruby_files.reject do |file|
+                       file.include?("/test/") || file.include?("/spec/") || file.end_with?("_test.rb") || file.end_with?("_spec.rb")
+                     end
+                   end
 
-      ruby_files = Dir.glob(File.join(project_path, "**", "*.rb"))
       ast_data = process_ruby_files_with_ast(ruby_files)
       Solidarity::Graph.from_ast_data(ast_data)
     end

data/lib/solidarity/reporter.rb

@@ -1,12 +1,22 @@
 module Solidarity
   class Reporter
-    def self.generate_report(solid_results)
+    def self.generate_report(solid_results, with_srp_details: false)
       report = "# Ruby SOLID Principles Analysis Report\n\n"
 
       solid_results.each do |principle, result|
         report += "## #{principle.to_s.upcase} - #{full_principle_name(principle)}\n"
         report += "Score: #{result[:score].round(2)}/100\n"
-        report += "Details: #{result[:details]}\n\n"
+        if principle == :srp && with_srp_details
+          report += "Details:\n"
+          result[:details].each do |class_name, score|
+            report += "  - #{class_name}: #{score}/100\n"
+          end
+        elsif principle == :srp && !with_srp_details
+          report += "Details: Omitted for brevity. Use --with-srp-details to see per-class scores.\n"
+        else
+          report += "Details: #{result[:details]}\n"
+        end
+        report += "\n"
       end
 
       report

data/lib/solidarity/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Solidarity
-  VERSION = "1.0.1"
+  VERSION = "1.0.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: solidarity
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Your Name