sight 0.1.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05abdb72bec3313b413b826f36a5f5a5590374d7d61640460a41eddd75a57ab6
-  data.tar.gz: 27d5fb5d877eebf0152ffa070201e2bb0755256cb466027bc3f61c26bccd4a16
+  metadata.gz: 6e488680c7d4ecc42d8cd02b8d2731a315755b35df2a9cf319ed3f0d413e2809
+  data.tar.gz: 375af2bad773014e984f0b89c3333c839dd84bfc8e8cd610357c78bcfd052962
 SHA512:
-  metadata.gz: 41a607fda53ab9822d968c8d2af280b71db14bb3edf96752dfd8ce26450277d44e19ad85f1677940506b75f80503f6511201291795d031d42aab10beb598bbe5
-  data.tar.gz: 254d6c49a5abb2ead9130daad53fc874523a282b1b69c50b9df0f015280ea7fb47ff8a099f3b21badd10b074c0b39941749c7b200f913a358a7e6635029954dd
+  metadata.gz: 76d83afcd3ac25b084fc22661d9561fd75bd16ef662237b5cf2e1d784225a442ac682740a312a711ae169e16eac1f8d997ff3e68b44fb1222a00a9fd3e077c6a
+  data.tar.gz: 934ab59d60fb58ac4029718705d1551fd6ede47ff0dfcadfefdf45fdd30eddc2060dcc815988060a796cf55cf993e0b9ba82b86426b1bc8043ed974631006099

data/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [0.3.0] - 2026-03-04
+
+### Added
+
+- Comment on hunks with `c` — annotations are printed to stdout on quit as markdown
+
+### Changed
+
+- Removed initial-commit diff fallback from `Git.diff`
+
+## [0.2.0] - 2026-03-04
+
+### Changed
+
+- Navigation is now hunk-based: `j`/`k` jump between hunks
+- Active hunk is highlighted; inactive hunks are dimmed
+- Removed line-scrolling keys (`f`/`b`/`d`/`u`/`g`/`G`) and arrow keys
+
 ## [0.1.0] - 2026-03-04
 
 ### Added

data/CLAUDE.md

@@ -0,0 +1,29 @@
+# CLAUDE.md
+
+This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
+
+## Commands
+
+- `bundle exec rake test` — run all tests
+- `ruby -Ilib:test test/test_cli.rb` — run a single test file
+- `ruby -Ilib:test test/test_cli.rb -n test_help_flag` — run a single test method
+- `bundle exec standardrb` — lint
+- `bundle exec standardrb --fix` — lint and auto-fix
+- `bundle exec rake` — run tests + lint (default task)
+
+## 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`.
+
+**Data flow**: `Git.diff` (raw string) → `DiffParser.parse` (returns `DiffFile[]`) → `App` (curses TUI)
+
+**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`.
+
+**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).
+
+## Conventions
+
+- Ruby >= 3.2, uses `frozen_string_literal` in all files
+- Linter: StandardRB (ruby_version: 3.4 in `.standard.yml`)
+- Tests: Minitest with stubs/mocks, no test framework beyond minitest
+- Single runtime dependency: `curses ~1.4`

data/README.md

@@ -1,6 +1,6 @@
 # Sight
 
-Terminal UI for browsing git diffs interactively with colors, scrolling, and file navigation.
+TUI for closing the loop on AI-generated code changes.
 
 ## Installation
 
@@ -20,17 +20,12 @@ sight
 
 | Key | Action |
 |-----|--------|
-| `j` / `↓` | Scroll down |
-| `k` / `↑` | Scroll up |
-| `d` | Half page down |
-| `u` | Half page up |
-| `f` | Full page down |
-| `b` | Full page up |
-| `g` | Go to top |
-| `G` | Go to bottom |
-| `n` / `→` | Next file |
-| `p` / `←` | Previous file |
+| `j` | Next hunk |
+| `k` | Previous hunk |
+| `n` | Next file |
+| `p` | Previous file |
 | `?` | Toggle help |
+| `c` | Comment on hunk |
 | `q` / `Esc` | Quit |
 
 ## Development

data/lib/sight/annotation.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+module Sight
+  Annotation = Struct.new(:file_path, :type, :hunk, :comment, keyword_init: true)
+end

data/lib/sight/annotation_formatter.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Sight
+  module AnnotationFormatter
+    module_function
+
+    def format(annotations)
+      return "" if annotations.empty?
+
+      grouped = annotations.group_by(&:file_path)
+      grouped.map { |path, anns| format_file(path, anns) }.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|
+          next unless %i[add del].include?(line.type)
+          out += "#{line.content}\n"
+        end
+        out += "```\n"
+        out += "> #{ann.comment}\n\n"
+      end
+      out
+    end
+  end
+end

data/lib/sight/app.rb

@@ -4,14 +4,17 @@ require "curses"
 
 module Sight
   class App
-    attr_reader :files, :file_lines
-    attr_accessor :file_idx, :offset
+    attr_reader :files, :file_lines, :annotations
+    attr_accessor :file_idx, :offset, :hunk_idx
 
     def initialize(files)
       @files = files
       @file_lines = files.map { build_file_lines(it) }
       @file_idx = 0
       @offset = 0
+      @hunk_idx = 0
+      @hunk_offsets_cache = {}
+      @annotations = []
     end
 
     def run
@@ -49,6 +52,7 @@ module Sight
       Curses.init_pair(2, Curses::COLOR_RED, -1)
       Curses.init_pair(3, Curses::COLOR_CYAN, -1)
       Curses.init_pair(4, Curses::COLOR_YELLOW, -1)
+      Curses.init_pair(5, 240, -1)
     end
 
     def lines
@@ -78,20 +82,29 @@ module Sight
       dim = Curses.color_pair(0) | Curses::A_DIM
       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
+
       scroll_height.times do |row|
         idx = offset + row
         break if idx >= lines.size
         line = lines[idx]
         win.setpos(row + 2, 0)
-        win.attron(dim) { win.addstr("#{format_gutter(line.type, line.lineno, gutter)} │ ") }
-        win.attron(color_for(line.type)) { win.addstr(line.text[0, content_width]) }
+        active = idx >= selected_start && idx < selected_end
+        win.attron(dim) { win.addstr("#{format_gutter(line.type, line.lineno, gutter)} \u2502 ") }
+        attr = active ? color_for(line.type) : Curses.color_pair(5)
+        win.attron(attr) { win.addstr(line.text[0, content_width]) }
       end
     end
 
     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} | Line #{offset + 1}/#{lines.size} "
+        status = " File #{file_idx + 1}/#{files.size} | Hunk #{hunk_idx + 1}/#{hunk_offsets.size} | Line #{offset + 1}/#{lines.size} "
         win.addstr(status.ljust(width))
       end
     end
@@ -126,42 +139,27 @@ module Sight
       end
     end
 
-    def scroll_to(delta)
-      max = [0, lines.size - scroll_height].max
-      self.offset = (offset + delta).clamp(0, max)
-    end
-
     def handle_input
       key = Curses.getch
       case key
       when "q", 27 then return false
-      when "j", Curses::KEY_DOWN then scroll_to(1)
-      when "k", Curses::KEY_UP then scroll_to(-1)
-      when "f" then scroll_to(scroll_height)
-      when "b" then scroll_to(-scroll_height)
-      when "d" then scroll_to(scroll_height / 2)
-      when "u" then scroll_to(-scroll_height / 2)
-      when "g" then self.offset = 0
-      when "G" then scroll_to(lines.size)
-      when "n", Curses::KEY_RIGHT then jump_file(1)
-      when "p", Curses::KEY_LEFT then jump_file(-1)
+      when "j" then jump_hunk(1)
+      when "k" then jump_hunk(-1)
+      when "n" then jump_file(1)
+      when "p" then jump_file(-1)
+      when "c" then annotate_hunk
       when "?" then show_help
       end
       true
     end
 
     HELP_KEYS = [
-      ["j / ↓", "Scroll down"],
-      ["k / ↑", "Scroll up"],
-      ["d", "Half Page down"],
-      ["u", "Half page up"],
-      ["f", "Full page down"],
-      ["b", "Full page up"],
-      ["g", "Go to top"],
-      ["G", "Go to bottom"],
-      ["n / →", "Next file"],
-      ["p / ←", "Previous file"],
+      ["j", "Next hunk"],
+      ["k", "Previous hunk"],
+      ["n", "Next file"],
+      ["p", "Previous file"],
       ["q / Esc", "Quit"],
+      ["c", "Comment on hunk"],
       ["?", "Toggle this help"]
     ].freeze
 
@@ -201,11 +199,93 @@ module Sight
       end
     end
 
+    def annotate_hunk
+      hunk = files[file_idx].hunks[hunk_idx]
+      return unless hunk
+      comment = prompt_comment("Comment on hunk")
+      return unless comment
+      @annotations << Annotation.new(
+        file_path: files[file_idx].path,
+        type: :hunk,
+        hunk: hunk,
+        comment: comment
+      )
+    end
+
+    def prompt_comment(title)
+      win = Curses.stdscr
+      max_width = 80
+      width = [Curses.cols * 2 / 3, 50].max
+      width = [width, max_width].min
+      height = 5
+      top = (Curses.lines - height) / 2
+      left = (Curses.cols - width) / 2
+
+      draw_box(win, top, left, width, height, title, [""])
+      win.setpos(top + 3, left + 3)
+      win.refresh
+
+      Curses.curs_set(1)
+      text = ""
+      field_width = width - 6
+      redraw_field = -> {
+        win.setpos(top + 3, left + 3)
+        win.addstr(" " * field_width)
+        visible = (text.length > field_width) ? text[-field_width..] : text
+        win.setpos(top + 3, left + 3)
+        win.addstr(visible)
+      }
+      loop do
+        ch = Curses.getch
+        case ch
+        when 10, 13, Curses::KEY_ENTER
+          break
+        when 27
+          text = nil
+          break
+        when Curses::KEY_BACKSPACE, 127, 8
+          unless text.empty?
+            text = text[0..-2]
+            redraw_field.call
+          end
+        else
+          text += ch.chr if ch.is_a?(Integer) && ch >= 32 && ch < 127
+          text += ch if ch.is_a?(String) && ch.length == 1
+          redraw_field.call
+        end
+      end
+      Curses.curs_set(0)
+
+      text&.strip&.empty? ? nil : text&.strip
+    end
+
+    def hunk_offsets
+      @hunk_offsets_cache[file_idx] ||= begin
+        offsets = []
+        line_idx = 0
+        files[file_idx].hunks.each do |hunk|
+          offsets << line_idx
+          line_idx += hunk.lines.size
+        end
+        offsets
+      end
+    end
+
+    def jump_hunk(delta)
+      return if hunk_offsets.empty?
+      self.hunk_idx = (hunk_idx + delta).clamp(0, hunk_offsets.size - 1)
+      target = hunk_offsets[hunk_idx]
+      margin = [2, scroll_height / 4].min
+      max = [0, lines.size - scroll_height].max
+      self.offset = [target - margin, 0].max.clamp(0, max)
+    end
+
     def jump_file(direction)
       new_idx = (file_idx + direction).clamp(0, files.size - 1)
       return if new_idx == file_idx
       self.file_idx = new_idx
       self.offset = 0
+      self.hunk_idx = 0
     end
   end
 end

data/lib/sight/cli.rb

@@ -9,7 +9,9 @@ module Sight
         puts "Usage: sight"
         puts "Interactive git diff viewer (staged + unstaged)"
         puts
-        puts "Keys: j/k/↑/↓ scroll, f/b page, d/u half-page, g/G top/bottom, n/p/→/← next/prev file, ? help, q quit"
+        puts "Keys: j/k hunks, n/p files, c comment, ? help, q quit"
+        puts
+        puts "Annotations are printed to stdout on quit."
         return
       end
 
@@ -20,12 +22,17 @@ module Sight
 
       raw = Git.diff
       if raw.empty?
-        warn "No diff output."
+        warn "No diff output"
         return
       end
 
       files = DiffParser.parse(raw)
-      App.new(files).run
+      app = App.new(files)
+      app.run
+
+      unless app.annotations.empty?
+        puts AnnotationFormatter.format(app.annotations)
+      end
     end
   end
 end

data/lib/sight/git.rb

@@ -6,10 +6,7 @@ module Sight
 
     def diff
       output, success = run_cmd(["git", "diff", "--no-color", "HEAD"])
-      unless success
-        output, success = run_cmd(["git", "diff", "--no-color", "--cached"], err: [:child, :out])
-        raise Error, "git diff failed: #{output}" unless success
-      end
+      raise Error, "git diff failed" unless success
       output
     end
 

data/lib/sight/version.rb

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

data/lib/sight.rb

@@ -4,6 +4,8 @@ require_relative "sight/version"
 require_relative "sight/display_line"
 require_relative "sight/diff_parser"
 require_relative "sight/git"
+require_relative "sight/annotation"
+require_relative "sight/annotation_formatter"
 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.1.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Ariel Rzezak
@@ -23,8 +23,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
-description: A TUI tool to browse git diffs interactively with colors, file navigation,
-  and scrolling.
+description: Browse diffs by hunk, comment on changes, and close the feedback loop
+  with AI agents.
 email:
 - arzezak@gmail.com
 executables:
@@ -33,12 +33,15 @@ extensions: []
 extra_rdoc_files: []
 files:
 - CHANGELOG.md
+- CLAUDE.md
 - CODE_OF_CONDUCT.md
 - LICENSE.txt
 - README.md
 - Rakefile
 - exe/sight
 - lib/sight.rb
+- lib/sight/annotation.rb
+- lib/sight/annotation_formatter.rb
 - lib/sight/app.rb
 - lib/sight/cli.rb
 - lib/sight/diff_parser.rb
@@ -70,5 +73,5 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 rubygems_version: 4.0.7
 specification_version: 4
-summary: Interactive git diff viewer for the terminal
+summary: TUI for closing the loop on AI-generated code changes
 test_files: []