nitpicker-code-review 1.0.0 → 1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b493182d8ba98bc4e102c13f0a379eaf545f920f9cb4f72ce4b4a0db4d164d51
-  data.tar.gz: 285e6dd3c76d4a2a7d4df310d3618f058fd6e39335dd5d965b1155205376a399
+  metadata.gz: f0515786e9e4de04268cb3972eb9a88c72f6fbe004feba7f99c57f9cac8f7af3
+  data.tar.gz: 855c02aa8daeb96b26ea6d9e4ef65a339a60b20be6f4caaedf7c4c3f80a2091e
 SHA512:
-  metadata.gz: eb1d96af03c6add163aca48519af15bbccc4ebc685d9d212b3cf52f925b345ff1bbd908c807f686e2e240e5d7075ee12f9e78dbf5ea45abb69d1da9b8ca91ad3
-  data.tar.gz: 54a486f87f322170dd2c0edfb41a225f5723f1507e8280f45d74dfd5a16026ed5a1a529a36eac3d08255c953ee2f076b760fc433e75d2272cc9c0a3c68d68fc8
+  metadata.gz: be5e14b952451be9208553b2066132a81ea4585e64a2de3bf231ff8c0e2591608ba4ecbfa71a0e8090485c9423d9cd2771e08587a01297b7c82686f3133782af
+  data.tar.gz: 988e0441dbe173d17aaede678d76d9c635fe0f527ec3eab997e1b334af410ec5e55f4ee1013b913550600685bed6199f6275979f6e8e9edfc1fa205583620613

data/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2025 Your Name
+Copyright (c) 2025 Justin Bishop
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -1,14 +1,14 @@
 # NitPicker
 
-NitPicker is a command-line tool that provides AI-powered code reviews for your Git changes. It analyzes your staged changes and offers constructive feedback to help improve code quality, security, performance, and maintainability.
+NitPicker is a command-line tool that provides AI-powered code reviews for your Git changes. It analyzes your staged changes by default, but can also review any diff piped to it. It offers constructive feedback to help improve code quality, security, performance, and maintainability.
 
 ## Features
 
-- AI-powered code review of staged Git changes
+- AI-powered code review of staged Git changes or any piped diff
 - Comprehensive analysis covering code quality, security, performance, and best practices
 - Customizable AI prompts for tailored review feedback
 - Support for repository-specific prompts
-- Choose any AI model (Google Gemini 2.5 Flash by default)
+- Choose any AI model (OpenAI GPT-4.1-mini by default)
 - Easy integration into your Git workflow
 
 ## Installation
@@ -81,7 +81,7 @@ Optionally, you can specify a different model by setting:
 export GIT_REVIEW_MODEL="openai/gpt-4o"
 ```
 
-The default model is `google/gemini-2.5-flash-preview-05-20` if not specified.
+The default model is `openai/gpt-4.1-mini` if not specified.
 
 ### Prompt Configuration
 
@@ -100,18 +100,31 @@ The special string `{{DIFF}}` in your prompt will be replaced with the current g
 
 ## Usage
 
+### Review Staged Changes (Default)
+
 Navigate to your Git repository, stage your changes, and run:
 
 ```bash
 nitpicker [options]
 ```
 
+### Review Any Diff via Pipe
+
+You can pipe any diff into nitpicker for review:
+
+```bash
+git show | nitpicker              # Review a specific commit
+git diff HEAD~1 | nitpicker       # Review changes from previous commit
+git diff main..feature | nitpicker # Review differences between branches
+```
+
 Options:
 - `--version`: Show version information
 - `-h, --help`: Show help message
 
 ## Examples
 
+### Review Staged Changes
 ```bash
 # Stage some changes first
 git add .
@@ -120,6 +133,24 @@ git add .
 nitpicker
 ```
 
+### Review Specific Commits or Diffs
+```bash
+# Review the last commit
+git show | nitpicker
+
+# Review changes from 2 commits ago
+git show HEAD~2 | nitpicker
+
+# Review uncommitted changes (both staged and unstaged)
+git diff HEAD | nitpicker
+
+# Review differences between branches
+git diff main..feature-branch | nitpicker
+
+# Review a range of commits
+git diff HEAD~3..HEAD | nitpicker
+```
+
 Example output:
 ```
 ## Code Review Summary

data/config/prompt

@@ -1,4 +1,8 @@
-You are an expert software engineer conducting a thorough code review. Analyze the following git diff and provide constructive feedback focusing on:
+You are an expert software engineer conducting a thorough code review.
+
+IMPORTANT: Do NOT reproduce, echo, or reformat the code.  Your job is to ANALYZE and provide FEEDBACK only.  Do NOT simply summarize what was done.
+
+Analyze the following git diff and provide constructive feedback focusing on:
 
 **Code Quality & Best Practices:**
 - Code readability, maintainability, and organization
@@ -35,5 +39,7 @@ Provide your feedback in a clear, constructive manner. For each issue identified
 
 If the code looks good overall, highlight what was done well and provide any minor suggestions for improvement.
 
+Start your response with "## Code Review" and provide analysis in prose, NOT code blocks or diffs.
+
 DIFF:
-{{DIFF}}
+{{DIFF}}

data/lib/nitpicker/version.rb

@@ -1,3 +1,3 @@
 module NitPicker
-  VERSION = '1.0.0'
-end
+  VERSION = '1.2'
+end

data/lib/nitpicker.rb

@@ -20,7 +20,10 @@ module NitPicker
       require 'optparse'
       
       OptionParser.new do |opts|
-        opts.banner = 'Usage: nitpicker [options]'
+        opts.banner = 'Usage: nitpicker [options]
+       git show | nitpicker              # Review a specific commit
+       git diff HEAD~1 | nitpicker       # Review changes from previous commit
+       nitpicker                         # Review staged changes (default)'
         opts.version = VERSION
 
         opts.on('-h', '--help', 'Show this help message') do
@@ -33,12 +36,25 @@ module NitPicker
     def run
       parse_options
       
-      # Get git diff of staged changes
-      diff = `git diff --staged`
-      
-      if diff.empty?
-        puts "No changes staged for review."
-        exit 1
+      # Check if we have piped input
+      if STDIN.tty?
+        # Interactive terminal - use git diff of staged changes
+        diff = `git diff --staged`
+        if diff.empty?
+          puts "No changes staged for review."
+          exit 1
+        end
+      else
+        # STDIN is redirected - try to read from it
+        diff = STDIN.read
+        if diff.empty?
+          # Empty piped input, fall back to git diff
+          diff = `git diff --staged`
+          if diff.empty?
+            puts "No changes staged for review."
+            exit 1
+          end
+        end
       end
 
       # Get AI-generated code review
@@ -85,13 +101,24 @@ module NitPicker
     def generate_code_review(diff)
       api_key = ENV['OPENROUTER_API_KEY']
       if api_key.nil? || api_key.empty?
-        puts "Error: OPENROUTER_API_KEY environment variable not set"
-        exit 1
+        puts "❌ Error: OPENROUTER_API_KEY environment variable not set"
+        puts ""
+        puts "To get AI code reviews, you need an OpenRouter API key:"
+        puts "1. Sign up at https://openrouter.ai/"
+        puts "2. Get your API key from the dashboard"
+        puts "3. Set it as an environment variable:"
+        puts "   export OPENROUTER_API_KEY=\"sk-or-v1-your-key-here\""
+        puts ""
+        puts "For now, showing the raw diff instead:"
+        puts "=" * 50
+        return diff
       end
 
-      model = ENV['GIT_REVIEW_MODEL'] || 'google/gemini-2.5-flash-preview-05-20'
+      model = ENV['GIT_REVIEW_MODEL'] || 'openai/gpt-4.1-mini'
+      puts "Using model: #{model}" if ENV['DEBUG']
       
       prompt = get_prompt.gsub('{{DIFF}}', diff)
+      puts "Prompt length: #{prompt.length} characters" if ENV['DEBUG']
       
       uri = URI.parse('https://openrouter.ai/api/v1/chat/completions')
       http = Net::HTTP.new(uri.host, uri.port)
@@ -102,28 +129,80 @@ module NitPicker
       request['Authorization'] = "Bearer #{api_key}"
       request['HTTP-Referer'] = 'https://github.com/nitpicker-code-review'
       
-      request.body = {
+      request_payload = {
         model: model,
         messages: [
           { role: 'user', content: prompt }
         ],
         max_tokens: 2000
-      }.to_json
+      }
+      
+      request.body = request_payload.to_json
+      puts "Making API request to OpenRouter..." if ENV['DEBUG']
       
       begin
         response = http.request(request)
+        puts "API Response code: #{response.code}" if ENV['DEBUG']
         
         if response.code == '200'
           result = JSON.parse(response.body)
-          review = result['choices'][0]['message']['content'].strip
-          return review
+          puts "API Response parsed successfully" if ENV['DEBUG']
+          
+          if result['choices'] && result['choices'][0] && result['choices'][0]['message']
+            review = result['choices'][0]['message']['content'].strip
+            puts "Review length: #{review.length} characters" if ENV['DEBUG']
+            puts "Review content preview: #{review[0..200]}..." if ENV['DEBUG']
+            return review
+          else
+            puts "❌ Error: Unexpected API response structure"
+            puts "Response: #{response.body}" if ENV['DEBUG']
+            puts ""
+            puts "Showing raw diff instead:"
+            puts "=" * 50
+            return diff
+          end
         else
-          puts "API Error (#{response.code}): #{response.body}"
-          exit 1
+          error_body = JSON.parse(response.body) rescue response.body
+          error_message = error_body.is_a?(Hash) && error_body['error'] ? error_body['error']['message'] || error_body['error'] : response.body
+          
+          puts "❌ API Error (#{response.code}): #{error_message}"
+          
+          if response.code == '401'
+            puts ""
+            puts "This usually means:"
+            puts "• Your API key is invalid or expired"
+            puts "• You haven't set the OPENROUTER_API_KEY environment variable"
+            puts "• Your API key doesn't have sufficient credits"
+            puts ""
+            puts "Please check your API key at https://openrouter.ai/"
+          elsif response.code == '429'
+            puts ""
+            puts "You've hit the rate limit. Please wait a moment and try again."
+          end
+          
+          puts ""
+          puts "Showing raw diff instead:"
+          puts "=" * 50
+          return diff
         end
+      rescue JSON::ParserError => e
+        puts "❌ Error parsing API response: #{e.message}"
+        puts "Raw response: #{response.body}" if ENV['DEBUG']
+        puts ""
+        puts "Showing raw diff instead:"
+        puts "=" * 50
+        return diff
       rescue => e
-        puts "Error: #{e.message}"
-        exit 1
+        puts "❌ Error making API request: #{e.message}"
+        puts ""
+        puts "This could be due to:"
+        puts "• Network connectivity issues"
+        puts "• OpenRouter API service being temporarily unavailable"
+        puts "• Invalid API configuration"
+        puts ""
+        puts "Showing raw diff instead:"
+        puts "=" * 50
+        return diff
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nitpicker-code-review
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: '1.2'
 platform: ruby
 authors:
 - Justin Bishop