diff_gem 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73ae8c129ddec8ddc5fb106a8d34d9bf7bdc42cabf49a076e70918d006cf4e72
-  data.tar.gz: ceb5f41c3b67cfd6d3f7b58f5bdf9ed599e7cdc1ed122eb8536819cb395a7dc5
+  metadata.gz: 95cf2fa62f8822068d882ac6d935f71599b6565a15a67e5b61030eb4c0c6ee22
+  data.tar.gz: ca589c18ab6ee53f0d7be63eadf446befc6ed6e4ab17c70ab09936d07ac26526
 SHA512:
-  metadata.gz: ff5a6fe8679e51eacb93bf8de2d9a8de23101c4bc13963b171d8fd0bdbff0c9fab8b4e131a71cae08bfd07ed5128ff595286000456bf0cd9c39f89a2cf0efa2b
-  data.tar.gz: d6162df393f490ec848d6f80a67432bb11ee5d203ae4ebf0655f6aa5df866a31642297703a3d098513e1c27f608059b05b89e09cdb229706eb7683b2ee2108ab
+  metadata.gz: c25207a65f7d52745afbf6853d89831fcf5de8f0c98b06f70dadea82cf1adb44e139ed02e2baa6100cddd8632b4593b7b6080a4c82ef235ab2417af430f0c186
+  data.tar.gz: f79ac1773eb481b3441a10f5766cefd7e6a2b59b2960a76c90bdc35d906d5277a5521652c4b239d4b29e4269af13ba8a296e82e4f2b09bda90d0908aa046b074

data/README.md

@@ -1,4 +1,4 @@
-# DiffGem
+# Diff Gem
 
 Compare the source code and metadata between two versions of a Ruby gem.
 

data/lib/diff_gem/comparer.rb

@@ -1,4 +1,5 @@
 require 'fileutils'
+require 'English'
 
 module DiffGem
   class Comparer
@@ -29,22 +30,216 @@ module DiffGem
     private
 
     def generate_diff(old_path, new_path)
-      # Use system diff command to generate comprehensive diff
-      # The diff command will:
-      # - Compare directories recursively (-r)
-      # - Use unified format (-u) for better readability
-      # - Show files that exist in only one directory (-N)
-      system("diff -r -u -N '#{old_path}' '#{new_path}'")
+      status = if windows?
+                 run_windows_diff(old_path, new_path)
+               else
+                 run_unix_diff(old_path, new_path)
+               end
 
-      # diff returns non-zero exit status when there are differences,
-      # which is expected behavior
-      if $?.exitstatus > 1
-        raise Error, "Failed to generate diff (exit status: #{$?.exitstatus})"
-      end
+      handle_diff_exit(status)
     end
 
     def default_cache_dir
       ENV['DIFF_GEM_CACHE_DIR'] || File.expand_path('~/.diff_gem_cache')
     end
+
+    def windows?
+      (/mswin|mingw|cygwin|bccwin/i).match?(RUBY_PLATFORM)
+    end
+
+    def run_unix_diff(old_path, new_path)
+      # Use unified diff to closely match traditional diff output on Unix-like systems
+      system('diff', '-r', '-u', '-N', old_path.to_s, new_path.to_s)
+      $CHILD_STATUS
+    end
+
+    def run_windows_diff(old_path, new_path)
+      script = windows_compare_script(old_path, new_path)
+      system('powershell.exe', '-NoProfile', '-Command', script)
+      $CHILD_STATUS
+    end
+
+    def handle_diff_exit(status)
+      raise Error, 'Failed to execute diff command' if status.nil?
+      return if status.exitstatus <= 1
+
+      raise Error, "Failed to generate diff (exit status: #{status.exitstatus})"
+    end
+
+    def windows_compare_script(old_path, new_path)
+      escaped_old = escape_for_powershell(old_path)
+      escaped_new = escape_for_powershell(new_path)
+
+      <<~POWERSHELL
+        $ErrorActionPreference = 'Stop'
+        $old = '#{escaped_old}'
+        $new = '#{escaped_new}'
+        $hasDifferences = $false
+
+        function Write-Line {
+          param([string]$text)
+          [Console]::Out.WriteLine($text)
+        }
+
+        function Get-RelativePath {
+          param(
+            [string]$root,
+            [string]$full
+          )
+
+          $normalizedRoot = [System.IO.Path]::GetFullPath($root)
+          if (-not $normalizedRoot.EndsWith([System.IO.Path]::DirectorySeparatorChar)) {
+            $normalizedRoot += [System.IO.Path]::DirectorySeparatorChar
+          }
+
+          $normalizedFull = [System.IO.Path]::GetFullPath($full)
+          return $normalizedFull.Substring($normalizedRoot.Length).Replace('\\', '/')
+        }
+
+        function Get-FileMap {
+          param([string]$root)
+
+          $map = @{}
+          if (-not (Test-Path -LiteralPath $root)) {
+            return $map
+          }
+
+          Get-ChildItem -LiteralPath $root -File -Recurse | ForEach-Object {
+            $relative = Get-RelativePath $root $_.FullName
+            $map[$relative] = $_.FullName
+          }
+
+          return $map
+        }
+
+        function Read-FileLines {
+          param([string]$path)
+
+          try {
+            return Get-Content -LiteralPath $path -ErrorAction Stop
+          } catch {
+            return $null
+          }
+        }
+
+        function Write-DiffHeader {
+          param(
+            [Parameter(Mandatory = $true)][string]$relative,
+            [string]$oldTarget,
+            [string]$newTarget
+          )
+
+          if (-not $oldTarget) { $oldTarget = "a/$relative" }
+          if (-not $newTarget) { $newTarget = "b/$relative" }
+
+          Write-Line "diff --git a/$relative b/$relative"
+          Write-Line "--- $oldTarget"
+          Write-Line "+++ $newTarget"
+        }
+
+        function Write-LineDiff {
+          param(
+            [string]$oldPath,
+            [string]$newPath,
+            [string]$relative
+          )
+
+          $oldLines = Read-FileLines $oldPath
+          $newLines = Read-FileLines $newPath
+
+          if ($null -eq $oldLines -or $null -eq $newLines) {
+            Write-Line "Binary files differ: $relative"
+            Write-Line ''
+            return $true
+          }
+
+          $diff = Compare-Object -ReferenceObject $oldLines -DifferenceObject $newLines -IncludeEqual:$false
+
+          if (-not $diff) {
+            return $false
+          }
+
+          Write-DiffHeader -relative $relative
+
+          foreach ($entry in $diff) {
+            $prefix = if ($entry.SideIndicator -eq '<=') { '-' } else { '+' }
+            $line = [string]$entry.InputObject
+            Write-Line "$prefix$line"
+          }
+
+          Write-Line ''
+          return $true
+        }
+
+        $oldFiles = Get-FileMap $old
+        $newFiles = Get-FileMap $new
+
+        $allPaths = (@($oldFiles.Keys) + @($newFiles.Keys)) | Sort-Object -Unique
+
+        foreach ($path in $allPaths) {
+          $oldExists = $oldFiles.ContainsKey($path)
+          $newExists = $newFiles.ContainsKey($path)
+
+          if (-not $oldExists -and $newExists) {
+            $newPathFull = $newFiles[$path]
+            $newLines = Read-FileLines $newPathFull
+
+            if ($null -eq $newLines) {
+              Write-Line "Binary file added: $path"
+              Write-Line ''
+            } else {
+              Write-DiffHeader -relative $path -oldTarget '/dev/null' -newTarget "b/$path"
+
+              foreach ($line in $newLines) {
+                Write-Line "+$line"
+              }
+
+              Write-Line ''
+            }
+
+            $hasDifferences = $true
+            continue
+          }
+
+          if ($oldExists -and -not $newExists) {
+            $oldPathFull = $oldFiles[$path]
+            $oldLines = Read-FileLines $oldPathFull
+
+            if ($null -eq $oldLines) {
+              Write-Line "Binary file removed: $path"
+              Write-Line ''
+            } else {
+              Write-DiffHeader -relative $path -oldTarget "a/$path" -newTarget '/dev/null'
+
+              foreach ($line in $oldLines) {
+                Write-Line "-$line"
+              }
+
+              Write-Line ''
+            }
+
+            $hasDifferences = $true
+            continue
+          }
+
+          $oldPathFull = $oldFiles[$path]
+          $newPathFull = $newFiles[$path]
+
+          if (Write-LineDiff -oldPath $oldPathFull -newPath $newPathFull -relative $path) {
+            $hasDifferences = $true
+          }
+        }
+
+        if ($hasDifferences) {
+          exit 1
+        } else {
+          exit 0
+        }
+      POWERSHELL
+    end
+
+    def escape_for_powershell(path)
+      path.to_s.gsub("'", "''")
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: diff_gem
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
-- Developer
+- Thomas Powell
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-11-04 00:00:00.000000000 Z
+date: 2025-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -55,7 +55,7 @@ dependencies:
 description: A tool to download and compare the source trees of two different versions
   of a Ruby gem, displaying differences in diff format
 email:
-- developer@example.com
+- twilliampowell@gmail.com
 executables:
 - diff_gem
 extensions: []
@@ -72,13 +72,12 @@ files:
 - lib/diff_gem/gem_extractor.rb
 - lib/diff_gem/metadata_comparer.rb
 - lib/diff_gem/version.rb
-homepage: https://github.com/example/diff_gem
+homepage: https://github.com/stringsn88keys/diff_gem
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/example/diff_gem
-  source_code_uri: https://github.com/example/diff_gem
-  changelog_uri: https://github.com/example/diff_gem/CHANGELOG.md
+  homepage_uri: https://github.com/stringsn88keys/diff_gem
+  changelog_uri: https://github.com/stringsn88keys/diff_gem/CHANGELOG.md
 post_install_message:
 rdoc_options: []
 require_paths: