prreview 0.4.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 714f7802004de97bfd9e4defb9f447ad5b445ea77e12b8f04123218a6684549b
-  data.tar.gz: ce62752c1d1d11c13f432e0c74df032873088d75018a13dec4dfc658b5ed6c73
+  metadata.gz: 28b817e16bf9a5637315d36b0dd477f851a80a03c1aa61988fdb521de6458dae
+  data.tar.gz: 44da1b71ce4de6cfe5ec280ee5eda07559ba28d2c5e9fcfda8a52fe30f9af899
 SHA512:
-  metadata.gz: 7526eaadf9570e7ad2967a2e003989b3126bff0ae2d0907d4d4056f92b90f8e2e7dff91a17cef9cd8a6cd81625d20a0cf27dcb378a348f6e1f7708124952011f
-  data.tar.gz: b9a446c787e170bb3998211173e198d5cb67480512446ddbb9f0a7620a8233ec692f8493bed7c1e1b4a181968c8e6b1ee07f1a4838ddd515c300ef60ef18bddf
+  metadata.gz: 30d0092f74f555b89c2bc43c3badc141219c66b7f95c36e7084994ad3ed979abd997935454b141feb5ee72b20a0a75949b0260d062f84a04669f088592529369
+  data.tar.gz: cf0b141c62a2142417a784b42a9c9f323ec10406b378c5dfb998d53bc27a6245e0e8a4556a009e85510ca546a07462629d28d10a136da871f68dfa9336e53c70

data/README.md

@@ -11,14 +11,14 @@ gem install prreview
 Run this command with a PR URL:
 
 ```sh
-prreview -u https://github.com/owner/repo/pull/123
+prreview https://github.com/owner/repo/pull/123
 ```
 
 Or use more options:
 
 ```
 prreview --help
-prreview --url https://github.com/owner/repo/pull/123 --all-content --prompt "Are there any security issues?"
+prreview https://github.com/owner/repo/pull/123 --all-content --prompt "Are there any security issues?"
 ```
 
 Now, just paste the PR details into ChatGPT, Claude, or any LLM for review.
@@ -57,7 +57,8 @@ However, in the future we might add some optional integrations.
 ## Requirements
 
 - Ruby
-- `GITHUB_TOKEN` environment variable
+- `GITHUB_TOKEN` environment
+  variable. [GitHub Docs](https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/managing-your-personal-access-tokens)
 
 ## License
 

data/lib/prreview/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Prreview
-  VERSION = '0.4.0'
+  VERSION = '0.5.1'
 end

data/lib/prreview.rb

@@ -37,8 +37,14 @@ module Prreview
 
     def process
       load_optional_files
-      fetch_pull_request
-      fetch_linked_issues
+      begin
+        fetch_pull_request
+        fetch_linked_issues
+      rescue Octokit::Unauthorized
+        abort 'Error: Invalid GITHUB_TOKEN.'
+      rescue Octokit::NotFound
+        abort 'Error: Pull request not found.'
+      end
       build_xml
       copy_result_to_clipboard
     end
@@ -54,9 +60,13 @@ module Prreview
       ARGV << '--help' if ARGV.empty?
 
       OptionParser.new do |parser|
-        parser.banner = "Usage: #{File.basename($PROGRAM_NAME)} -u URL [options]"
+        parser.banner = <<~BAN
+          Usage: #{File.basename($PROGRAM_NAME)} URL [options]
+
+          Pull request URL example: https://github.com/owner/repo/pull/1
+
+        BAN
 
-        parser.on('-u', '--url URL', 'Pull‑request URL (https://github.com/owner/repo/pull/1)') { |v| @url = v }
         parser.on('-p', '--prompt PROMPT', 'Custom LLM prompt') { |v| @prompt = v }
         parser.on('-a', '--all-content', 'Include full file contents') { @include_content = true }
         parser.on('-l', '--limit LIMIT', Integer, "Limit number of issues fetched (default: #{DEFAULT_LINKED_ISSUES_LIMIT})") { |v| @linked_issues_limit = v }
@@ -73,10 +83,12 @@ module Prreview
         end
         parser.parse!
       end
+
+      @url = ARGV.first
     end
 
     def parse_url!
-      abort 'Error: Pull-request URL missing. Use -u or --url.' if @url.to_s.empty?
+      abort 'Error: Pull request URL missing.' if @url.to_s.empty?
 
       match = @url.match(URL_REGEX)
       abort 'Error: Invalid URL format. See --help for usage.' unless match
@@ -92,8 +104,6 @@ module Prreview
       abort 'Error: GITHUB_TOKEN is not set.' if access_token.to_s.empty?
 
       @client = Octokit::Client.new(access_token:, auto_paginate: true)
-    rescue Octokit::Unauthorized
-      abort 'Error: Invalid GITHUB_TOKEN.'
     end
 
     def fetch_pull_request
@@ -106,7 +116,7 @@ module Prreview
         {
           filename: file.filename,
           patch: file.patch || '(no patch data)',
-          content: @include_content ? fetch_file_content(file.filename) : '(no content)'
+          content: @include_content && !skip_file?(file.filename) ? fetch_file_content(file.filename) : '(no content)'
         }
       end
     end
@@ -210,7 +220,7 @@ module Prreview
               x.file do
                 x.filename file[:filename]
                 x.content file[:content]
-                x.patch file[:patch]
+                x.patch extract_patch(file)
               end
             end
           end
@@ -244,6 +254,16 @@ module Prreview
       @xml = builder.doc.root.to_xml
     end
 
+    def extract_patch(file)
+      return if skip_file?(file[:filename])
+
+      file[:patch]
+    end
+
+    def skip_file?(filename)
+      File.extname(filename) == '.svg'
+    end
+
     def binary?(string)
       string.include?("\x00")
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: prreview
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Evgenii Morozov