ruby_workspace_manager 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e8c287aa2d28cf7ed4c21f9ce567eb25f5a2164e0d064619aa2110676461ae0
-  data.tar.gz: b2d3ca1d17eecdf4b9233469bdaf24c4f43a9d7ec3cb6bd99fdb9fb2a527b419
+  metadata.gz: fbe69d99300fafa6359944aa4a1cb16a3ed9cbb0e1e544b3a428a4e04af4dc66
+  data.tar.gz: 114e54d2ac5872ae866ecf0503b8ebdeb9c72b0db892a564e0b3889b4ff1e0fd
 SHA512:
-  metadata.gz: 9b1c5e6dac68401d8ec61e70323af0bea421370354a5bfa36d66917bcd08222026b9ac535e800c1aa5f8405ffde6759381c53fecea629972f799e42e1176f4b2
-  data.tar.gz: a535d8642e4c5c3736a4606954e4a21a2d32e8926293d89ca2622fc3c8c0ced4b3667e0d5375b51c5b23cece660d76b1aa9e062cf270d73c001977b2e4a5ab7d
+  metadata.gz: bad333bc09182d824ef4c5b4e50fc290a7922e81a8e616a991d54cf994cca8340038fc5bb4b2dfa1612292ac7fc3f6008089ea6ff7a95b702ef65c0335ee97b4
+  data.tar.gz: bdfb899d956de2f9aa4d694d83b74cd79cb9936c0f38cfa1f64c78931846436be644510e0fce9dcfb8774af61b74d5cc24e09769aa0b2547c6a9ffba45d1dc95

data/lib/rwm/affected_detector.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "open3"
+
 module Rwm
   class AffectedDetector
     attr_reader :workspace, :graph, :base_branch
@@ -41,16 +43,22 @@ module Rwm
 
     def detect_base_branch
       # Try to read the remote's default branch
-      ref = `git -C #{workspace.root} symbolic-ref refs/remotes/origin/HEAD 2>/dev/null`.chomp
-      unless ref.empty?
-        # refs/remotes/origin/main → main
-        return ref.sub(%r{^refs/remotes/origin/}, "")
+      ref, _, status = Open3.capture3("git", "-C", workspace.root, "symbolic-ref", "refs/remotes/origin/HEAD")
+      if status.success?
+        ref = ref.chomp
+        unless ref.empty?
+          # refs/remotes/origin/main → main
+          return ref.sub(%r{^refs/remotes/origin/}, "")
+        end
       end
 
       # Fall back: check if main or master exists
-      branches = `git -C #{workspace.root} branch --list main master 2>/dev/null`.lines.map(&:strip)
-      return "main" if branches.include?("main")
-      return "master" if branches.include?("master")
+      out, _, status = Open3.capture3("git", "-C", workspace.root, "branch", "--list", "main", "master")
+      if status.success?
+        branches = out.lines.map(&:strip)
+        return "main" if branches.include?("main")
+        return "master" if branches.include?("master")
+      end
 
       # Last resort
       "main"
@@ -60,17 +68,17 @@ module Rwm
       files = Set.new
 
       # 1. Committed changes: base branch vs HEAD
-      committed = `git -C #{workspace.root} diff --name-only #{base_branch}...HEAD 2>/dev/null`.chomp
-      committed.lines.each { |l| files << l.chomp }
+      committed, _, status = Open3.capture3("git", "-C", workspace.root, "diff", "--name-only", "#{base_branch}...HEAD")
+      committed.lines.each { |l| files << l.chomp } if status.success?
 
       unless @committed_only
         # 2. Staged changes (not yet committed)
-        staged = `git -C #{workspace.root} diff --name-only --cached 2>/dev/null`.chomp
-        staged.lines.each { |l| files << l.chomp }
+        staged, _, status = Open3.capture3("git", "-C", workspace.root, "diff", "--name-only", "--cached")
+        staged.lines.each { |l| files << l.chomp } if status.success?
 
         # 3. Unstaged working directory changes
-        unstaged = `git -C #{workspace.root} diff --name-only 2>/dev/null`.chomp
-        unstaged.lines.each { |l| files << l.chomp }
+        unstaged, _, status = Open3.capture3("git", "-C", workspace.root, "diff", "--name-only")
+        unstaged.lines.each { |l| files << l.chomp } if status.success?
       end
 
       files.reject(&:empty?).to_a

data/lib/rwm/cli.rb

@@ -40,6 +40,8 @@ module Rwm
         return 0
       end
 
+      check_required_tools
+
       # Expand task shortcuts: `rwm test` → `rwm run test`
       if TASK_SHORTCUTS.include?(command_name)
         @argv.unshift(command_name)
@@ -64,6 +66,15 @@ module Rwm
 
     private
 
+    def check_required_tools
+      %w[git bundle].each do |tool|
+        unless system("which", tool, out: File::NULL, err: File::NULL)
+          $stderr.puts "Error: #{tool} is not installed or not in PATH."
+          exit 1
+        end
+      end
+    end
+
     def print_help
       puts <<~HELP
         rwm #{Rwm::VERSION} — Ruby Workspace Manager

data/lib/rwm/gemfile.rb

@@ -12,12 +12,16 @@
 #   rwm_lib "auth"
 
 require "bundler"
+require "open3"
 
 module Rwm
   module GemfileDsl
     def rwm_workspace_root
-      @rwm_workspace_root ||= `git rev-parse --show-toplevel 2>/dev/null`.strip.tap do |root|
+      @rwm_workspace_root ||= begin
+        out, _, status = Open3.capture3("git", "rev-parse", "--show-toplevel")
+        root = status.success? ? out.strip : ""
         raise "rwm: not inside a git repository" if root.empty?
+        root
       end
     end
 

data/lib/rwm/task_cache.rb

@@ -3,7 +3,7 @@
 require "digest"
 require "fileutils"
 require "json"
-require "shellwords"
+require "open3"
 
 module Rwm
   class TaskCache
@@ -63,12 +63,12 @@ module Rwm
         digest.update(File.read(file))
       end
 
-      # Include dependency content hashes (transitive invalidation)
+      # Include dependency content hashes (transitive invalidation).
+      # If a dependency is missing, let it raise — a stale graph should
+      # not silently produce incorrect cache hits.
       @graph.dependencies(package.name).sort.each do |dep_name|
         dep_pkg = @workspace.find_package(dep_name)
         digest.update(content_hash(dep_pkg))
-      rescue PackageNotFoundError
-        next
       end
 
       @content_hashes[package.name] = digest.hexdigest
@@ -78,8 +78,8 @@ module Rwm
     def cache_declarations(package)
       return @cache_declarations[package.name] if @cache_declarations.key?(package.name)
 
-      output = `cd #{Shellwords.escape(package.path)} && bundle exec rake rwm:cache_config 2>/dev/null`
-      @cache_declarations[package.name] = if $?.success? && !output.strip.empty?
+      output, _, status = Open3.capture3("bundle", "exec", "rake", "rwm:cache_config", chdir: package.path)
+      @cache_declarations[package.name] = if status.success? && !output.strip.empty?
                                              JSON.parse(output.strip)
                                            else
                                              {}

data/lib/rwm/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rwm
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/lib/rwm/workspace.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "open3"
+
 module Rwm
   class Workspace
     RWM_DIR = ".rwm"
@@ -15,7 +17,8 @@ module Rwm
     # Find the workspace root via git
     def self.find(start_dir = Dir.pwd)
       dir = File.expand_path(start_dir)
-      git_root = `git -C #{dir} rev-parse --show-toplevel 2>/dev/null`.chomp
+      git_root, _, status = Open3.capture3("git", "-C", dir, "rev-parse", "--show-toplevel")
+      git_root = status.success? ? git_root.chomp : ""
 
       raise WorkspaceNotFoundError if git_root.empty?
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby_workspace_manager
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Siddharth Bhatt