sight 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e488680c7d4ecc42d8cd02b8d2731a315755b35df2a9cf319ed3f0d413e2809
-  data.tar.gz: 375af2bad773014e984f0b89c3333c839dd84bfc8e8cd610357c78bcfd052962
+  metadata.gz: ff89f4bccb463b21835b299946bcf7a54b43f9792eb0b262954812c79633da19
+  data.tar.gz: 7dab4e4911a9bf145a5b0542ac993c6b5d3ce25af54b345e537291c5567db681
 SHA512:
-  metadata.gz: 76d83afcd3ac25b084fc22661d9561fd75bd16ef662237b5cf2e1d784225a442ac682740a312a711ae169e16eac1f8d997ff3e68b44fb1222a00a9fd3e077c6a
-  data.tar.gz: 934ab59d60fb58ac4029718705d1551fd6ede47ff0dfcadfefdf45fdd30eddc2060dcc815988060a796cf55cf993e0b9ba82b86426b1bc8043ed974631006099
+  metadata.gz: c591ba467ae6d6a64d1d54c188513a2afe743c40e767bca6892f9f30e6ff45ecfb1f134fcef62c420b4e7fb8c13a77fcd9c69e8f830c6cf1a7c79f8d4d61b82f
+  data.tar.gz: 131fd548a595de6b14288b2ae536b2275eb66fb50f6e08182875866bdee44c0acfbb8194a0ee515e510472bb72f38eeae34fa7c64f64e0a97b2e4e4433867775

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## [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

@@ -15,12 +15,20 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 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`.
 
-**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**: Unified via `sight install-hook <agent>` / `sight uninstall-hook <agent>` (agent: `claude` or `cursor`).
+`ClaudeHookInstaller` manages a Claude Code `UserPromptSubmit` hook in `~/.config/claude/settings.json`.
+`CursorHookInstaller` manages a Cursor `beforeSubmitPrompt` hook in `~/.cursor/hooks.json`.
+Both read `.git/sight/pending-review`, output annotations, and delete the file. Hidden subcommands: `hook-run`, `cursor-hook-run`.
+
 ## Conventions
 
 - Ruby >= 3.2, uses `frozen_string_literal` in all files

data/README.md

@@ -24,10 +24,30 @@ 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:
+
+```bash
+sight install-hook claude   # Claude Code (~/.config/claude/settings.json)
+sight install-hook cursor   # Cursor (~/.cursor/hooks.json)
+```
+
+When you quit sight after annotating, the next message you send will include your annotations.
+
+To remove:
+
+```bash
+sight uninstall-hook claude
+sight uninstall-hook cursor
+```
+
 ## Development
 
 ```bash

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)
@@ -89,13 +98,21 @@ module Sight
         lines.size
       end
 
+      commented_lines = commented_hunk_lines
+
       scroll_height.times do |row|
         idx = offset + row
         break if idx >= lines.size
         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 +121,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 +146,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 +179,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 +192,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"],
@@ -271,6 +311,27 @@ module Sight
       end
     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
+      result = []
+      hunk_offsets.each_with_index do |offset, hunk_index|
+        next unless hunk_commented?(file_idx, hunk_index)
+        hunk_end = (hunk_index + 1 < hunk_offsets.size) ? hunk_offsets[hunk_index + 1] : lines.size
+        (offset...hunk_end).each { |i| result << i }
+      end
+      result
+    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

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "json"
+
 module Sight
   module CLI
     module_function
@@ -11,7 +13,9 @@ module Sight
         puts
         puts "Keys: j/k hunks, n/p files, c comment, ? help, q quit"
         puts
-        puts "Annotations are printed to stdout on quit."
+        puts "Subcommands:"
+        puts "  install-hook <agent>    Install hook (claude, cursor)"
+        puts "  uninstall-hook <agent>  Remove hook (claude, cursor)"
         return
       end
 
@@ -20,19 +24,103 @@ module Sight
         return
       end
 
-      raw = Git.diff
-      if raw.empty?
-        warn "No diff output"
+      if argv[0] == "install-hook"
+        install_hook(argv[1])
+        return
+      end
+
+      if argv[0] == "uninstall-hook"
+        uninstall_hook(argv[1])
         return
       end
 
+      if argv.include?("hook-run")
+        run_hook
+        return
+      end
+
+      if argv.include?("cursor-hook-run")
+        run_cursor_hook
+        return
+      end
+
+      Git.clear_pending_review
+
+      raw = Git.diff
       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
+
       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,
+      "cursor" => CursorHookInstaller
+    }.freeze
+
+    def install_hook(agent)
+      installer = AGENTS[agent]
+      unless installer
+        warn "Unknown agent: #{agent.inspect}. Use: claude, cursor"
+        return
+      end
+      installer.install
+    end
+
+    def uninstall_hook(agent)
+      installer = AGENTS[agent]
+      unless installer
+        warn "Unknown agent: #{agent.inspect}. Use: claude, cursor"
+        return
+      end
+      installer.uninstall
+    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
+
+    def run_cursor_hook
+      $stdin.read
+      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)
+      message = "The user has just finished reviewing your code changes in sight. Here are their annotations:\n\n#{content}"
+      puts JSON.generate(user_message: message)
+      File.delete(file)
+    end
   end
 end

data/lib/sight/cursor_hook_installer.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+require "json"
+require "fileutils"
+
+module Sight
+  module CursorHookInstaller
+    HOOK_COMMAND = "sight cursor-hook-run"
+
+    module_function
+
+    def hooks_path
+      File.join(Dir.home, ".cursor", "hooks.json")
+    end
+
+    def install(path: hooks_path)
+      config = if File.exist?(path)
+        JSON.parse(File.read(path))
+      else
+        {"version" => 1, "hooks" => {}}
+      end
+
+      config["version"] ||= 1
+      hooks = config["hooks"] ||= {}
+      prompt_hooks = hooks["beforeSubmitPrompt"] ||= []
+
+      if prompt_hooks.any? { |h| hook_is_sight?(h) }
+        puts "sight hook already installed"
+        return
+      end
+
+      prompt_hooks << {"command" => HOOK_COMMAND}
+
+      FileUtils.mkdir_p(File.dirname(path))
+      File.write(path, JSON.pretty_generate(config) + "\n")
+      puts "Installed sight hook into #{path}"
+    end
+
+    def uninstall(path: hooks_path)
+      unless File.exist?(path)
+        puts "No hooks file found"
+        return
+      end
+
+      config = JSON.parse(File.read(path))
+      prompt_hooks = config.dig("hooks", "beforeSubmitPrompt")
+
+      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(config) + "\n")
+      puts "Uninstalled sight hook"
+    end
+
+    def hook_is_sight?(entry)
+      entry["command"]&.include?("sight")
+    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.4.0"
 end

data/lib/sight.rb

@@ -6,6 +6,8 @@ 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/cursor_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.4.0
 platform: ruby
 authors:
 - Ariel Rzezak
@@ -43,7 +43,9 @@ files:
 - 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/cursor_hook_installer.rb
 - lib/sight/diff_parser.rb
 - lib/sight/display_line.rb
 - lib/sight/git.rb