sight 0.3.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e488680c7d4ecc42d8cd02b8d2731a315755b35df2a9cf319ed3f0d413e2809
-  data.tar.gz: 375af2bad773014e984f0b89c3333c839dd84bfc8e8cd610357c78bcfd052962
+  metadata.gz: 32b20a4a62fbca9fee67a4354538a8e033dc079b945c7aa5a6f6d23f63c0fac6
+  data.tar.gz: 4be26d309ed3cb0649bc093a78326d6f9a4a2ce2f2f3f3ec9943b713bac9f372
 SHA512:
-  metadata.gz: 76d83afcd3ac25b084fc22661d9561fd75bd16ef662237b5cf2e1d784225a442ac682740a312a711ae169e16eac1f8d997ff3e68b44fb1222a00a9fd3e077c6a
-  data.tar.gz: 934ab59d60fb58ac4029718705d1551fd6ede47ff0dfcadfefdf45fdd30eddc2060dcc815988060a796cf55cf993e0b9ba82b86426b1bc8043ed974631006099
+  metadata.gz: 7acf8bc51543269963f35953c5e7ffe7452483e655330bd5d9c314c394a575f6710233fe26b117eaa7ca00491c0d4f561c879e2c7ab160fcb32a6d3db15bf7e4
+  data.tar.gz: 696a3493aa44d9e7913913df11f9ef583de057e15bc210811027121a629534d486c7cb7c197180c146644aadba2eea331db8d49315c83da9197d2a4116f472ca

data/CHANGELOG.md

@@ -1,5 +1,33 @@
 # Changelog
 
+## [0.5.0] - 2026-03-07
+
+### Removed
+
+- Cursor hook support (use Cursor rules instead)
+
+### Changed
+
+- Refactored CLI dispatch to use case statement with extracted `open` and `print_help` methods
+- DRYed up install/uninstall hook agent lookup
+- Use `Set` for commented hunk lines lookup
+- Extracted `hunk_end_offset` to deduplicate boundary logic
+- Use `<<` for string building in `AnnotationFormatter`
+- Simplified prompt comment width clamping
+
+## [0.4.0] - 2026-03-06
+
+### Added
+
+- Untracked file support with file status badges (new, modified, deleted)
+- Color-coded status badges in file header
+- Ctrl-f/b/d/u scroll navigation
+- Scroll percentage indicator replacing line counter
+- Hook system for AI agent integration (`sight install-hook <agent>`)
+  - Claude Code hook via `UserPromptSubmit`
+  - Cursor hook via `beforeSubmitPrompt`
+- Highlighted commented hunks in gutter and status bar
+
 ## [0.3.0] - 2026-03-04
 
 ### Added

data/CLAUDE.md

@@ -13,14 +13,21 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 ## Architecture
 
-TUI for closing the loop on AI-generated code changes — browse diffs, jump between hunks, and annotate them. Entry point: `exe/sight` → `CLI.run` → `App.new(files).run`.
+TUI for closing the loop on AI-generated code changes — browse diffs, jump between hunks, and annotate them. Entry point: `exe/sight` → `CLI.run` → `CLI.open` → `App.new(files).run`.
 
-**Data flow**: `Git.diff` (raw string) → `DiffParser.parse` (returns `DiffFile[]`) → `App` (curses TUI)
+**Data flow**: `Git.diff` (raw string) → `DiffParser.parse` (returns `DiffFile[]`) → `App` (curses TUI).
+Untracked files are added via `Git.untracked_files` → `DiffParser.build_untracked`.
 
-**Key structs** (all in `diff_parser.rb`): `DiffFile(path, hunks)`, `Hunk(context, lines)`, `DiffLine(type, content, lineno, old_lineno)`. `DisplayLine(type, text, lineno)` is the render-side equivalent in `display_line.rb`.
+**Key structs** (all in `diff_parser.rb`): `DiffFile(path, hunks, status)`, `Hunk(context, lines)`, `DiffLine(type, content, lineno, old_lineno)`.
+`DisplayLine(type, text, lineno)` is the render-side equivalent in `display_line.rb`.
+`Annotation(file_path, type, hunk, comment)` in `annotation.rb` stores per-hunk comments; `AnnotationFormatter` serializes them for output.
 
 **App** renders per-file views with hunk-based navigation (j/k). Active hunk is highlighted; inactive hunks render in dark gray (color pair 5, color 240).
 
+**Hook system**: `sight install-hook claude` / `sight uninstall-hook claude`.
+`ClaudeHookInstaller` manages a Claude Code `UserPromptSubmit` hook in `~/.config/claude/settings.json`.
+Reads `.git/sight/pending-review`, outputs annotations, and deletes the file. Hidden subcommand: `hook-run`.
+
 ## Conventions
 
 - Ruby >= 3.2, uses `frozen_string_literal` in all files

data/README.md

@@ -24,10 +24,43 @@ sight
 | `k` | Previous hunk |
 | `n` | Next file |
 | `p` | Previous file |
+| `Ctrl-F` / `Ctrl-B` | Scroll full page down / up |
+| `Ctrl-D` / `Ctrl-U` | Scroll half page down / up |
 | `?` | Toggle help |
 | `c` | Comment on hunk |
 | `q` / `Esc` | Quit |
 
+### Agent Integration
+
+Install a hook so annotations are automatically fed as context in your next message:
+
+**Claude Code** (~/.config/claude/settings.json):
+
+```bash
+sight install-hook claude
+```
+
+When you quit sight after annotating, the next message you send will include your annotations.
+
+The hook runs on every prompt but is a no-op when there's no pending review — it produces no output and adds nothing to the agent's context.
+
+To remove:
+
+```bash
+sight uninstall-hook claude
+```
+
+**Cursor** — Create a rule in `.cursor/rules/sight.mdc`:
+
+```markdown
+---
+description: When the user asks to review their sight annotations or code review comments
+alwaysApply: false
+---
+
+Read `.git/sight/pending-review`, address the annotations, then delete the file.
+```
+
 ## Development
 
 ```bash

data/RELEASING.md

@@ -0,0 +1,7 @@
+# Releasing
+
+1. Update `lib/sight/version.rb` and `CHANGELOG.md`
+2. Run `bundle install` to update `Gemfile.lock`
+3. Run `bundle exec rake` to verify tests and lint pass
+4. Commit: `git commit -am "Release vX.Y.Z"`
+5. Run `bundle exec rake release` — tags, builds, and pushes to RubyGems

data/lib/sight/annotation_formatter.rb

@@ -8,21 +8,21 @@ module Sight
       return "" if annotations.empty?
 
       grouped = annotations.group_by(&:file_path)
-      grouped.map { |path, anns| format_file(path, anns) }.join("\n")
+      grouped.map { |path, file_annotations| format_file(path, file_annotations) }.join("\n")
     end
 
     def format_file(path, annotations)
       out = "## File: #{path}\n\n"
-      annotations.each do |ann|
-        context = ann.hunk.context ? " #{ann.hunk.context}" : ""
-        out += "Hunk (@@#{context}):\n"
-        out += "```diff\n"
-        ann.hunk.lines.each do |line|
+      annotations.each do |annotation|
+        context = annotation.hunk.context ? " #{annotation.hunk.context}" : ""
+        out << "Hunk (@@#{context}):\n"
+        out << "```diff\n"
+        annotation.hunk.lines.each do |line|
           next unless %i[add del].include?(line.type)
-          out += "#{line.content}\n"
+          out << "#{line.content}\n"
         end
-        out += "```\n"
-        out += "> #{ann.comment}\n\n"
+        out << "```\n"
+        out << "> #{annotation.comment}\n\n"
       end
       out
     end

data/lib/sight/app.rb

@@ -53,6 +53,7 @@ module Sight
       Curses.init_pair(3, Curses::COLOR_CYAN, -1)
       Curses.init_pair(4, Curses::COLOR_YELLOW, -1)
       Curses.init_pair(5, 240, -1)
+      Curses.init_pair(6, Curses::COLOR_MAGENTA, -1)
     end
 
     def lines
@@ -70,11 +71,19 @@ module Sight
     end
 
     def render_header(win, width)
-      path = files[file_idx].path
+      file = files[file_idx]
+      badge = "[#{file.status || :modified}]"
+      path = file.path
+      gap = width - path.length - badge.length
       win.setpos(0, 0)
-      win.attron(color_for(:header)) { win.addstr(path[0, width]) }
+      if gap >= 1
+        win.attron(color_for(:header)) { win.addstr("#{path}#{" " * gap}") }
+        win.attron(badge_color(file.status)) { win.addstr(badge) }
+      else
+        win.attron(color_for(:header)) { win.addstr(path[0, width]) }
+      end
       win.setpos(1, 0)
-      win.attron(color_for(:header)) { win.addstr("\u2500" * width) }
+      win.attron(Curses.color_pair(0) | Curses::A_BOLD) { win.addstr("\u2500" * width) }
     end
 
     def render_content(win, width)
@@ -83,11 +92,9 @@ module Sight
       content_width = width - gutter - 3
 
       selected_start = hunk_offsets[hunk_idx]
-      selected_end = if hunk_idx + 1 < hunk_offsets.size
-        hunk_offsets[hunk_idx + 1]
-      else
-        lines.size
-      end
+      selected_end = hunk_end_offset(hunk_idx)
+
+      commented_lines = commented_hunk_lines
 
       scroll_height.times do |row|
         idx = offset + row
@@ -95,7 +102,13 @@ module Sight
         line = lines[idx]
         win.setpos(row + 2, 0)
         active = idx >= selected_start && idx < selected_end
-        win.attron(dim) { win.addstr("#{format_gutter(line.type, line.lineno, gutter)} \u2502 ") }
+        gutter_str = format_gutter(line.type, line.lineno, gutter)
+        commented = commented_lines.include?(idx)
+        separator = commented ? "\u2503" : "\u2502"
+        sep_attr = commented ? Curses.color_pair(4) : dim
+        win.attron(dim) { win.addstr("#{gutter_str} ") }
+        win.attron(sep_attr) { win.addstr(separator) }
+        win.attron(dim) { win.addstr(" ") }
         attr = active ? color_for(line.type) : Curses.color_pair(5)
         win.attron(attr) { win.addstr(line.text[0, content_width]) }
       end
@@ -104,7 +117,13 @@ module Sight
     def render_status_bar(win, width)
       win.setpos(Curses.lines - 1, 0)
       win.attron(Curses.color_pair(4) | Curses::A_REVERSE) do
-        status = " File #{file_idx + 1}/#{files.size} | Hunk #{hunk_idx + 1}/#{hunk_offsets.size} | Line #{offset + 1}/#{lines.size} "
+        percent = if lines.size <= scroll_height
+          100
+        else
+          ((offset + scroll_height) * 100.0 / lines.size).ceil.clamp(0, 100)
+        end
+        commented = hunk_commented?(file_idx, hunk_idx) ? " [commented]" : ""
+        status = " File #{file_idx + 1}/#{files.size} | Hunk #{hunk_idx + 1}/#{hunk_offsets.size}#{commented} | #{percent}% "
         win.addstr(status.ljust(width))
       end
     end
@@ -123,11 +142,20 @@ module Sight
       case type
       when :add then Curses.color_pair(1)
       when :del then Curses.color_pair(2)
-      when :header then Curses.color_pair(4) | Curses::A_BOLD
+      when :header then Curses.color_pair(0) | Curses::A_BOLD
       else Curses.color_pair(0)
       end
     end
 
+    def badge_color(status)
+      case status
+      when :added then Curses.color_pair(1) | Curses::A_BOLD
+      when :deleted then Curses.color_pair(2) | Curses::A_BOLD
+      when :untracked then Curses.color_pair(6) | Curses::A_BOLD
+      else Curses.color_pair(4) | Curses::A_BOLD
+      end
+    end
+
     def scroll_height
       Curses.lines - 3
     end
@@ -147,6 +175,10 @@ module Sight
       when "k" then jump_hunk(-1)
       when "n" then jump_file(1)
       when "p" then jump_file(-1)
+      when 6 then scroll(scroll_height)
+      when 2 then scroll(-scroll_height)
+      when 4 then scroll(scroll_height / 2)
+      when 21 then scroll(-scroll_height / 2)
       when "c" then annotate_hunk
       when "?" then show_help
       end
@@ -156,6 +188,10 @@ module Sight
     HELP_KEYS = [
       ["j", "Next hunk"],
       ["k", "Previous hunk"],
+      ["C-f", "Page down"],
+      ["C-b", "Page up"],
+      ["C-d", "Half page down"],
+      ["C-u", "Half page up"],
       ["n", "Next file"],
       ["p", "Previous file"],
       ["q / Esc", "Quit"],
@@ -214,9 +250,7 @@ module Sight
 
     def prompt_comment(title)
       win = Curses.stdscr
-      max_width = 80
-      width = [Curses.cols * 2 / 3, 50].max
-      width = [width, max_width].min
+      width = (Curses.cols * 2 / 3).clamp(50, 80)
       height = 5
       top = (Curses.lines - height) / 2
       left = (Curses.cols - width) / 2
@@ -271,6 +305,28 @@ module Sight
       end
     end
 
+    def hunk_end_offset(idx)
+      (idx + 1 < hunk_offsets.size) ? hunk_offsets[idx + 1] : lines.size
+    end
+
+    def hunk_commented?(file_index, hunk_index)
+      path = files[file_index].path
+      hunk = files[file_index].hunks[hunk_index]
+      annotations.any? { |a| a.file_path == path && a.hunk.equal?(hunk) }
+    end
+
+    def commented_hunk_lines
+      hunk_offsets.each_with_index.each_with_object(Set.new) do |(offset, hunk_index), set|
+        next unless hunk_commented?(file_idx, hunk_index)
+        (offset...hunk_end_offset(hunk_index)).each { |i| set << i }
+      end
+    end
+
+    def scroll(delta)
+      max = [0, lines.size - scroll_height].max
+      self.offset = (offset + delta).clamp(0, max)
+    end
+
     def jump_hunk(delta)
       return if hunk_offsets.empty?
       self.hunk_idx = (hunk_idx + delta).clamp(0, hunk_offsets.size - 1)

data/lib/sight/claude_hook_installer.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+require "json"
+require "fileutils"
+
+module Sight
+  module ClaudeHookInstaller
+    HOOK_COMMAND = "sight hook-run"
+
+    module_function
+
+    def settings_path
+      File.join(Dir.home, ".config", "claude", "settings.json")
+    end
+
+    def install(path: settings_path)
+      settings = if File.exist?(path)
+        JSON.parse(File.read(path))
+      else
+        {}
+      end
+
+      hooks = settings["hooks"] ||= {}
+      prompt_hooks = hooks["UserPromptSubmit"] ||= []
+
+      if prompt_hooks.any? { |h| hook_is_sight?(h) }
+        puts "sight hook already installed"
+        return
+      end
+
+      prompt_hooks << {
+        "matcher" => "*",
+        "hooks" => [{"type" => "command", "command" => HOOK_COMMAND}]
+      }
+
+      FileUtils.mkdir_p(File.dirname(path))
+      File.write(path, JSON.pretty_generate(settings) + "\n")
+      puts "Installed sight hook into #{path}"
+    end
+
+    def uninstall(path: settings_path)
+      unless File.exist?(path)
+        puts "No settings file found"
+        return
+      end
+
+      settings = JSON.parse(File.read(path))
+      prompt_hooks = settings.dig("hooks", "UserPromptSubmit")
+
+      unless prompt_hooks&.any? { |h| hook_is_sight?(h) }
+        puts "No sight hook found"
+        return
+      end
+
+      prompt_hooks.reject! { |h| hook_is_sight?(h) }
+      File.write(path, JSON.pretty_generate(settings) + "\n")
+      puts "Uninstalled sight hook"
+    end
+
+    def hook_is_sight?(entry)
+      Array(entry["hooks"]).any? { |h| h["command"]&.include?("sight") }
+    end
+  end
+end

data/lib/sight/cli.rb

@@ -5,34 +5,84 @@ module Sight
     module_function
 
     def run(argv)
-      if argv.include?("--help") || argv.include?("-h")
-        puts "Usage: sight"
-        puts "Interactive git diff viewer (staged + unstaged)"
-        puts
-        puts "Keys: j/k hunks, n/p files, c comment, ? help, q quit"
-        puts
-        puts "Annotations are printed to stdout on quit."
-        return
-      end
+      return print_help if argv.include?("--help") || argv.include?("-h")
+      return puts("sight #{VERSION}") if argv.include?("--version") || argv.include?("-v")
 
-      if argv.include?("--version") || argv.include?("-v")
-        puts "sight #{VERSION}"
-        return
+      case argv[0]
+      when "install-hook" then install_hook(argv[1])
+      when "uninstall-hook" then uninstall_hook(argv[1])
+      when "hook-run" then run_hook
+      else open
       end
+    end
+
+    def print_help
+      puts "Usage: sight"
+      puts "Interactive git diff viewer (staged + unstaged)"
+      puts
+      puts "Keys: j/k hunks, n/p files, c comment, ? help, q quit"
+      puts
+      puts "Subcommands:"
+      puts "  install-hook <agent>    Install hook (claude)"
+      puts "  uninstall-hook <agent>  Remove hook (claude)"
+    end
+
+    def open
+      Git.clear_pending_review
 
       raw = Git.diff
-      if raw.empty?
-        warn "No diff output"
+      files = DiffParser.parse(raw)
+
+      Git.untracked_files.each do |path|
+        content = Git.file_content(path)
+        next unless content.valid_encoding? && !content.include?("\x00")
+        files << DiffParser.build_untracked(path, content)
+      end
+
+      if files.empty?
+        warn "No changes"
         return
       end
 
-      files = DiffParser.parse(raw)
       app = App.new(files)
       app.run
 
       unless app.annotations.empty?
-        puts AnnotationFormatter.format(app.annotations)
+        formatted = AnnotationFormatter.format(app.annotations)
+        puts formatted
+        Git.save_pending_review(formatted)
       end
     end
+
+    AGENTS = {
+      "claude" => ClaudeHookInstaller
+    }.freeze
+
+    def install_hook(agent)
+      resolve_installer(agent)&.install
+    end
+
+    def uninstall_hook(agent)
+      resolve_installer(agent)&.uninstall
+    end
+
+    def resolve_installer(agent)
+      AGENTS.fetch(agent) { warn "Unknown agent: #{agent.inspect}. Use: claude" }
+    end
+
+    def run_hook
+      git_dir = Git.repo_dir
+    rescue Error
+      nil
+    else
+      file = File.join(git_dir, "sight", "pending-review")
+      return unless File.exist?(file)
+
+      content = File.read(file)
+      puts "The user has just finished reviewing your code changes in sight. Here are their annotations:"
+      puts
+      puts content
+      File.delete(file)
+    end
   end
 end

data/lib/sight/diff_parser.rb

@@ -3,7 +3,7 @@
 module Sight
   DiffLine = Struct.new(:type, :content, :lineno, :old_lineno, keyword_init: true)
   Hunk = Struct.new(:context, :lines, keyword_init: true)
-  DiffFile = Struct.new(:path, :hunks, keyword_init: true)
+  DiffFile = Struct.new(:path, :hunks, :status, keyword_init: true)
 
   module DiffParser
     module_function
@@ -19,10 +19,14 @@ module Sight
         if line.start_with?("diff --git ")
           current_hunk = nil
           path = line.split(" b/", 2).last
-          current_file = DiffFile.new(path: path, hunks: [])
+          current_file = DiffFile.new(path: path, hunks: [], status: :modified)
           files << current_file
         elsif current_file.nil?
           next
+        elsif line.start_with?("new file mode")
+          current_file.status = :added
+        elsif line.start_with?("deleted file mode")
+          current_file.status = :deleted
         elsif line.start_with?("@@ ")
           context, new_start, old_start = parse_hunk_header(line)
           new_lineno = new_start
@@ -48,6 +52,14 @@ module Sight
       files
     end
 
+    def build_untracked(path, content)
+      lines = content.lines(chomp: true).each_with_index.map do |text, i|
+        DiffLine.new(type: :add, content: "+#{text}", lineno: i + 1, old_lineno: nil)
+      end
+      hunk = Hunk.new(context: nil, lines: lines)
+      DiffFile.new(path: path, hunks: [hunk], status: :untracked)
+    end
+
     def parse_hunk_header(line)
       match = line.match(/@@ -(\d+)(?:,\d+)? \+(\d+)(?:,\d+)? @@(.*)/)
       return [nil, 1, 1] unless match

data/lib/sight/git.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "fileutils"
+
 module Sight
   module Git
     module_function
@@ -10,6 +12,33 @@ module Sight
       output
     end
 
+    def untracked_files
+      output, success = run_cmd(["git", "ls-files", "--others", "--exclude-standard"])
+      return [] unless success
+      output.lines(chomp: true).reject(&:empty?)
+    end
+
+    def file_content(path)
+      File.read(path, mode: "rb")
+    end
+
+    def repo_dir
+      output, success = run_cmd(["git", "rev-parse", "--git-dir"])
+      raise Error, "not a git repository" unless success
+      output.strip
+    end
+
+    def save_pending_review(content)
+      dir = File.join(repo_dir, "sight")
+      FileUtils.mkdir_p(dir)
+      File.write(File.join(dir, "pending-review"), content)
+    end
+
+    def clear_pending_review
+      path = File.join(repo_dir, "sight", "pending-review")
+      File.delete(path) if File.exist?(path)
+    end
+
     def run_cmd(cmd, err: IO::NULL)
       output = IO.popen(cmd, err: err, &:read)
       [output, $?.success?]

data/lib/sight/version.rb

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

data/lib/sight.rb

@@ -6,6 +6,7 @@ require_relative "sight/diff_parser"
 require_relative "sight/git"
 require_relative "sight/annotation"
 require_relative "sight/annotation_formatter"
+require_relative "sight/claude_hook_installer"
 require_relative "sight/cli"
 require_relative "sight/app"
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sight
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Ariel Rzezak
@@ -37,12 +37,14 @@ files:
 - CODE_OF_CONDUCT.md
 - LICENSE.txt
 - README.md
+- RELEASING.md
 - Rakefile
 - exe/sight
 - lib/sight.rb
 - lib/sight/annotation.rb
 - lib/sight/annotation_formatter.rb
 - lib/sight/app.rb
+- lib/sight/claude_hook_installer.rb
 - lib/sight/cli.rb
 - lib/sight/diff_parser.rb
 - lib/sight/display_line.rb