grepfruit 1.1.0 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4375b6c22cf2b2323abd76e66e35c098a4f681fe24fd63a40df0a4fa90640ea3
-  data.tar.gz: 743a2141c660fa7aa90687f56de16e5d3c9b5172abe4b9d52f7aa7ad322a773c
+  metadata.gz: d6d72bac5375937fd7214c4d0f13de9f619695de3614202af1d877aaffdaf1f4
+  data.tar.gz: 7d1abd0f67111b48ccf0e988c8bfcab18fd90cf0510d02571292d29f2b4642b1
 SHA512:
-  metadata.gz: d2a124c7e449b5a098c229f07e2c392ee89c06391f1c75ae5666bb485ac28d10dec33e0f556eda2bd5bbcaf659b052ea013ae4bb2617efcb2a6a2ed7c3a89fa6
-  data.tar.gz: 1e4640be220f8a72b1592e229de4d8bf70bf06e1d6d461f2fd85d663b5bd67bc7d27ea0e513867a32e6a129ab07ad13dbe4b905ef023633c89c7c61fda81cdd8
+  metadata.gz: 18e978abd3bd6beeb29e14b88e8847f1410e3adfe8e1ad4b82910075cae099e25e5139c4ca8c4077b3666e7b2ad63911eb9a5cb637c8b3306ac42ad908f3caf1
+  data.tar.gz: 7d48ec2fd3f5f04baa4c69b9187675e0664c46d728907702aaa9a82f62b4689f999fc923bf5cc6035feb298d80350d9cd4f2906e006276b79b6e1a9768a6bc5b

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+## v1.1.2
+
+- Refactored code significantly for improved search efficiency and easier maintenance
+- Enhanced search result output for better readability
+
+## v1.1.1
+
+- Added test dataset to make testing and development easier
+- Updated gemspec file to include missing metadata
+
 ## v1.1.0
 
 - Added `--truncate` option to truncate the output of the search results

data/exe/grepfruit

@@ -21,7 +21,7 @@ OptionParser.new do |opts|
   end
 
   opts.on("-e", "--exclude x,y,z", Array, "Comma-separated list of files and directories to exclude") do |exclude|
-    options[:exclude] = exclude.map { |path| path.split("/") }
+    options[:exclude] = exclude
   end
 
   opts.on("-t", "--truncate N", Integer, "Truncate output to N characters") do |truncate|
@@ -40,4 +40,4 @@ end
 
 options[:dir] = ARGV[0] if ARGV[0]
 
-Grepfruit.run(**options)
+Grepfruit::Search.new(**options).run

data/grepfruit.gemspec

@@ -4,11 +4,14 @@ Gem::Specification.new do |spec|
   spec.name = "grepfruit"
   spec.version = Grepfruit::VERSION
   spec.authors = ["enjaku4"]
-  spec.metadata["rubygems_mfa_required"] = "true"
-  spec.summary = "A Ruby gem for searching text patterns in files with colorized output."
   spec.homepage = "https://github.com/enjaku4/grepfruit"
+  spec.metadata["homepage_uri"] = spec.homepage
+  spec.metadata["source_code_uri"] = spec.homepage
+  spec.metadata["changelog_uri"] = "#{spec.homepage}/blob/master/CHANGELOG.md"
+  spec.metadata["rubygems_mfa_required"] = "true"
+  spec.summary = "A Ruby gem for searching text patterns in files with colorized output"
   spec.license = "MIT"
-  spec.required_ruby_version = ">= 3.0.0", "< 3.4"
+  spec.required_ruby_version = ">= 3.0", "< 3.4"
 
   spec.files = [
     "grepfruit.gemspec", "README.md", "CHANGELOG.md", "LICENSE.txt"

data/lib/grepfruit/version.rb

@@ -1,3 +1,3 @@
 module Grepfruit
-  VERSION = "1.1.0"
+  VERSION = "1.1.2"
 end

data/lib/grepfruit.rb

@@ -1,53 +1,6 @@
 require_relative "grepfruit/version"
-require "pathname"
-require "find"
+require_relative "search"
 
 module Grepfruit
-  def self.run(dir:, regex:, exclude:, truncate:, search_hidden:)
-    lines = []
-    files = 0
-    excluded_lines = exclude.select { |e| e.any? { |s| s.include?(":") } }
-    excluded_paths = exclude - excluded_lines
-
-    puts "Searching for #{regex.inspect} in #{dir.inspect}...\n\n"
-
-    Find.find(dir) do |path|
-      Find.prune if excluded_paths.any? { |e| path.split("/").last(e.length) == e } || !search_hidden && File.basename(path).start_with?(".")
-
-      next if File.directory?(path) || File.symlink?(path)
-
-      files += 1
-
-      match = false
-
-      File.foreach(path).with_index do |line, line_num|
-        next unless line.valid_encoding?
-
-        if line.match?(regex)
-          path_with_line = "#{Pathname.new(path).relative_path_from(Pathname.new(dir))}:#{line_num + 1}"
-
-          next if excluded_lines.any? { |e| path_with_line.split("/").last(e.length) == e }
-
-          processed_line = line.strip
-          processed_line = "#{processed_line[0..truncate - 1]}..." if truncate && processed_line.length > truncate
-          lines << "\e[36m#{path_with_line}\e[0m: #{processed_line}"
-          match = true
-        end
-      end
-
-      print match ? "\e[31mF\e[0m" : "\e[32m.\e[0m"
-    end
-
-    puts "\n\n"
-
-    if lines.empty?
-      puts "#{files} file#{'s' if files > 1} checked, \e[32mno matches found\e[0m"
-      exit(0)
-    else
-      puts "Matches:\n\n"
-      puts "#{lines.join("\n")}\n\n"
-      puts "#{files} file#{'s' if files > 1} checked, \e[31m#{lines.size} match#{'es' if lines.size > 1} found\e[0m"
-      exit(1)
-    end
-  end
+  class Error < StandardError; end
 end

data/lib/search.rb

@@ -0,0 +1,101 @@
+require "pathname"
+require "find"
+
+module Grepfruit
+  class Search
+    CYAN = "\e[36m"
+    RED = "\e[31m"
+    GREEN = "\e[32m"
+    RESET = "\e[0m"
+
+    attr_reader :dir, :regex, :excluded_paths, :excluded_lines, :truncate, :search_hidden
+
+    def initialize(dir:, regex:, exclude:, truncate:, search_hidden:)
+      @dir = dir
+      @regex = regex
+      @excluded_lines, @excluded_paths = exclude.map { |e| e.split("/") }.partition { |e| e.last.include?(":") }
+      @truncate = truncate
+      @search_hidden = search_hidden
+    end
+
+    def run
+      lines, files = [], 0
+
+      puts "Searching for #{regex.inspect} in #{dir.inspect}...\n\n"
+
+      Find.find(dir) do |path|
+        Find.prune if excluded_path?(path) || !search_hidden && hidden?(path)
+
+        next if File.directory?(path) || File.symlink?(path)
+
+        files += 1
+
+        match = false
+
+        File.foreach(path).with_index do |line, line_num|
+          next unless line.valid_encoding?
+
+          if line.match?(regex)
+            next if excluded_line?(path, line_num)
+
+            lines << "#{CYAN}#{relative_path_with_line_num(path, line_num)}#{RESET}: #{processed_line(line)}"
+            match = true
+          end
+        end
+
+        print match ? "#{RED}M#{RESET}" : "#{GREEN}.#{RESET}"
+      end
+
+      puts "\n\n" if files.positive?
+
+      if lines.empty?
+        puts "#{number_of_files(files)} checked, #{GREEN}no matches found#{RESET}"
+        exit(0)
+      else
+        puts "Matches:\n\n"
+        puts "#{lines.join("\n")}\n\n"
+        puts "#{number_of_files(files)} checked, #{RED}#{number_of_matches(lines.size)} found#{RESET}"
+        exit(1)
+      end
+    end
+
+    private
+
+    def excluded_path?(path)
+      excluded?(excluded_paths, relative_path(path))
+    end
+
+    def excluded_line?(path, line_num)
+      excluded?(excluded_lines, relative_path_with_line_num(path, line_num))
+    end
+
+    def excluded?(list, path)
+      list.any? { |e| path.split("/").last(e.length) == e }
+    end
+
+    def relative_path(path)
+      Pathname.new(path).relative_path_from(Pathname.new(dir)).to_s
+    end
+
+    def relative_path_with_line_num(path, line_num)
+      "#{relative_path(path)}:#{line_num + 1}"
+    end
+
+    def processed_line(line)
+      stripped_line = line.strip
+      truncate && stripped_line.length > truncate ? "#{stripped_line[0..truncate - 1]}..." : stripped_line
+    end
+
+    def hidden?(path)
+      File.basename(path).start_with?(".")
+    end
+
+    def number_of_files(num)
+      "#{num} file#{'s' if num > 1}"
+    end
+
+    def number_of_matches(num)
+      "#{num} match#{'es' if num > 1}"
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grepfruit
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.2
 platform: ruby
 authors:
 - enjaku4
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-31 00:00:00.000000000 Z
+date: 2024-08-15 00:00:00.000000000 Z
 dependencies: []
 description:
 email:
@@ -24,10 +24,14 @@ files:
 - grepfruit.gemspec
 - lib/grepfruit.rb
 - lib/grepfruit/version.rb
+- lib/search.rb
 homepage: https://github.com/enjaku4/grepfruit
 licenses:
 - MIT
 metadata:
+  homepage_uri: https://github.com/enjaku4/grepfruit
+  source_code_uri: https://github.com/enjaku4/grepfruit
+  changelog_uri: https://github.com/enjaku4/grepfruit/blob/master/CHANGELOG.md
   rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
@@ -37,7 +41,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.0.0
+      version: '3.0'
   - - "<"
     - !ruby/object:Gem::Version
       version: '3.4'
@@ -47,8 +51,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.33
+rubygems_version: 3.5.11
 signing_key:
 specification_version: 4
-summary: A Ruby gem for searching text patterns in files with colorized output.
+summary: A Ruby gem for searching text patterns in files with colorized output
 test_files: []