claude-inbox 0.13.1 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36fa381f8d03173896c6b44dff47534ed54119c9ff529833a9783c7f5c524506
-  data.tar.gz: '0386da94a6ffa5259bbad165857821f44b082dc5a64d957beae352c46dc61100'
+  metadata.gz: c346a215bf94ae3088c6ce5ddff23a2852e0294b77bbaa92a980d6baf4315cb7
+  data.tar.gz: 92e53f999cc3731ab9bf96c20e96048cd47a2261cb9640ed26bdafdc5032bbd6
 SHA512:
-  metadata.gz: bb933d5723ee131285824d2faea2570762854a4695da2ecd98716d5b9d981b2b36aece7abbda8acf92fa063b47f52f40ac2931136ef5fecee194f0fda7cdc72c
-  data.tar.gz: 3593591f4192056035fdc13e3480a26824794be46df0d8aa5b72fdb5c84a4b6b039d658b4279ec64cdf35ec94c2d65533117321c01bdd9b3e06a4267571a83b6
+  metadata.gz: cb9dc45979798d3e8c73534f06c1e6dc73a6bdfb5019c6df3f87ae7e3e3adb6cf0d67f370e61f29729f3a10a07a5728a58507dd1f961929843c2798e52f5cd18
+  data.tar.gz: 59558583519adb57a59ab15c9bad0d78bff87b8db105210c8bdd63370eaa40d3105a68bed312ec0972bfd6fe91bca53866c39decd734db64b09059d8071d4860

data/lib/claude_inbox/app/navigating.rb

@@ -7,12 +7,6 @@ module ClaudeInbox
     module Navigating
       private
 
-      # Remembers where the frame put the rows, so a click can find one.
-      def placed(frame)
-        @row_items = frame.items
-        @top = frame.top
-      end
-
       # The row to land on once a poll lists it, as after a spawn.
       def select_later(id)
         @pending_select = id
@@ -100,7 +94,7 @@ module ClaudeInbox
       # no item of its own; a click there resolves to the row above it.
       def row_item_at(row)
         idx = row - 1
-        @row_items[idx] || (@row_items[idx - 1] if idx > 0 && @row_items[idx - 1]&.row?)
+        @frame.items[idx] || (@frame.items[idx - 1] if idx > 0 && @frame.items[idx - 1]&.row?)
       end
     end
   end

data/lib/claude_inbox/app.rb

@@ -71,8 +71,8 @@ module ClaudeInbox
         reaper: reaper, queue: @queue, snapshot: snapshot, actions: actions, notifier: notifier)
       @listener = listen ? Remote::Listener.new(client: client, store: store, queue: @queue, **listen) : Remote::Listener.disabled
       @selected = nil
-      @row_items = []
-      @top = 0
+      @frame = Renderer::Frame.new([], [], 0)
+      @pointer_at = nil
       @expanded = Hash.new(false)
       @keymap = Keymap.new
       @tick = 0
@@ -190,6 +190,9 @@ module ClaudeInbox
     # the normal action table (see handle_key); mouse input defers to the
     # same rule rather than reaching past whatever has focus.
     def handle_mouse(event)
+      @pointer_at = [event.row, event.col]
+      link = link_under_pointer
+      return click_link(link) if event.kind == :click && link
       return if @modal || filter_editing?
       case event.kind
       when :click then click_row(event.row)
@@ -198,6 +201,15 @@ module ClaudeInbox
       end
     end
 
+    def click_link(link)
+      return open_in_browser(link.url, link.text) unless @modal
+      pr_picked if @modal.pick_by_url(link.url)
+    end
+
+    def link_under_pointer
+      @frame.link_at(*@pointer_at) if @pointer_at && !filter_editing?
+    end
+
     def handle_modal_key(name, key)
       return handle_form_key(name, key) if @modal.is_a?(NewSessionForm)
       case @modal.press(name, key)
@@ -274,15 +286,15 @@ module ClaudeInbox
       ensure_selection(sections)
       @tick += 1
       view = Renderer::View.new(
-        width: width, height: height, now: now, selected: @selected&.key, top: @top, expanded: @expanded,
+        width: width, height: height, now: now, selected: @selected&.key, top: @frame.top, expanded: @expanded,
         modal: modal_lines(width), screen: screen_lines(width, body_h),
         status: status_text(now), usage: @rate_limits.windows(now), filter: filter, filter_editing: filter_editing?,
         tick: @tick / 2, loading: loading_for, listening: @listener.snapshot
       )
-      frame = @renderer.frame(sections, view)
-      frame = frame.plain unless @color # --no-color: decided here and nowhere else
-      placed(frame)
-      @terminal.paint(frame.lines)
+      @frame = @renderer.frame(sections, view)
+      @frame = @frame.plain unless @color # --no-color: decided here and nowhere else
+      @terminal.paint(@frame.lines)
+      @terminal.pointer(!link_under_pointer.nil?)
       dt = monotonic - t0
       Debug.log("render #{(dt * 1000).round}ms") if dt > 0.05
     end

data/lib/claude_inbox/dialog.rb

@@ -33,6 +33,11 @@ module ClaudeInbox
     # How wide the box wants to be; the frame caps it at the terminal.
     def fit = self.class::WIDTH
 
+    def bordered(rows, inner)
+      top = Text.truncate(title, inner)
+      ["┌#{top}#{"─" * (inner - Text.width(top))}┐", *rows.map { |row| "│#{row}│" }, "└#{"─" * inner}┘"]
+    end
+
     def key(name) = @theme.cyan_bold(name)
 
     # A key and what it does.
@@ -83,6 +88,18 @@ module ClaudeInbox
 
       def title = " #{kind.to_s.capitalize} pull request "
 
+      # Drawn by hand: TTY::Box counts a link's hidden characters toward the
+      # line's width, so it would clip every line that has a link.
+      def frame(width, _caret = nil)
+        inner = [width - 4, fit].min - 2
+        bordered(lines(inner - 2, nil).map { |l| " #{Text.pad(l, inner - 2)} " }, inner)
+      end
+
+      def pick_by_url(url)
+        @choice = @prs.find { |pr| pr.url == url }
+        @choice && :pick
+      end
+
       def lines(width, _caret)
         (@prs.each_with_index.map { |pr, i| "  #{key((i + 1).to_s)}  #{label(pr)}" } +
           ["", "  #{hint(KEYS.fetch(kind), "#{kind} #{@prs.first.short}")} · #{key("esc")} #{@theme.dim("cancel")}"])
@@ -170,9 +187,7 @@ module ClaudeInbox
       # pushes the right border out on every line that has one.
       def frame(width, _caret = nil)
         inner = [width - 4, WIDTH].min - 2
-        ["┌#{title}#{"─" * (inner - Text.width(title))}┐",
-          *lines(inner, nil).map { |l| "│#{Text.pad(Text.truncate(l, inner), inner)}│" },
-          "└#{"─" * inner}┘"]
+        bordered(lines(inner, nil).map { |l| Text.pad(Text.truncate(l, inner), inner) }, inner)
       end
 
       def lines(_width, _caret)

data/lib/claude_inbox/mouse.rb

@@ -2,31 +2,32 @@
 
 module ClaudeInbox
   # Parses SGR mouse reports ("\e[<Cb;Cx;Cy(M|m)", emitted once App turns on
-  # \e[?1000h\e[?1006h) into clicks and wheel ticks. Pure: no terminal, no
-  # IO. Row is the terminal's own 1-based row, so callers can index straight
-  # into what they just painted.
+  # \e[?1003h\e[?1006h) into clicks, pointer moves and wheel ticks. Pure: no
+  # terminal, no IO. Row and column are the terminal's own 1-based ones, so
+  # callers can index straight into what they just painted.
   module Mouse
     SEQUENCE = /\e\[<(\d+);(\d+);(\d+)([Mm])/
 
-    Event = Struct.new(:kind, :row)
+    Event = Struct.new(:kind, :row, :col)
 
     # A raw keypress can glue several reports together the same way a fast
     # "esc gg" glues onto one read (see Input#split_keys), so this scans
     # rather than matching once.
     def self.events(raw)
-      raw.scan(SEQUENCE).filter_map do |cb, _col, row, type|
+      raw.scan(SEQUENCE).filter_map do |cb, col, row, type|
         kind = kind_for(cb.to_i, type)
-        Event.new(kind, row.to_i) if kind
+        Event.new(kind, row.to_i, col.to_i) if kind
       end
     end
 
-    # Bit 6 (0x40) marks a wheel tick, direction in bit 0. Otherwise this is
-    # a button press/release: bits 0-1 give the button (0 = left) and bit 5
-    # (0x20) marks a drag, so 0x23 masks both at once. Only a left click,
-    # released of any drag, counts; middle/right buttons and drags are left
-    # for the terminal's own handling.
+    # Bit 6 (0x40) marks a wheel tick, direction in bit 0. Otherwise bits
+    # 0-1 give the button (0 = left, 3 = none) and bit 5 (0x20) marks motion,
+    # so 0x23 masks both at once. Only a left click, released of any drag,
+    # counts, and motion with no button held is the pointer moving; middle
+    # and right buttons and drags are left for the terminal's own handling.
     def self.kind_for(cb, type)
       return (cb.even? ? :scroll_up : :scroll_down) if cb & 0x40 != 0
+      return :move if cb & 0x23 == 0x23
       :click if type == "M" && cb & 0x23 == 0
     end
     private_class_method :kind_for

data/lib/claude_inbox/pull_requests.rb

@@ -169,7 +169,7 @@ module ClaudeInbox
     end
 
     def seed(url)
-      number = url[%r{/pull/(\d+)}, 1]&.to_i
+      number = url[%r{/(?:pull|-/merge_requests)/(\d+)}, 1]&.to_i
       cached = resolved[url] || claude_cache[url]
       PullRequest.new(number: cached&.dig("number") || number, url: url, state: cached&.dig("state"),
         resolved_at: cached&.dig("resolved_at"), title: cached&.dig("title"))

data/lib/claude_inbox/renderer.rb

@@ -12,7 +12,12 @@ module ClaudeInbox
   class Renderer
     Frame = Struct.new(:lines, :items, :top) do
       # The same frame for a terminal drawn without color.
-      def plain = self.class.new(lines.map { |l| Text.strip_ansi(l) }, items, top)
+      def plain = self.class.new(lines.map { |l| l.gsub(Text::CSI, "") }, items, top)
+
+      def link_at(row, col)
+        @links ||= lines.map { |l| Text.links(l) }
+        @links.fetch(row - 1, []).find { |link| link.columns.cover?(col - 1) }
+      end
     end
 
     CHROME_ROWS = 2 # header + footer
@@ -396,7 +401,7 @@ module ClaudeInbox
     def pr_badge(s, section)
       pr = s.pr
       return nil unless pr
-      badge = (section == :settled) ? @p.dim("#{pr.short} #{pr.state&.downcase}".strip) : @theme.pr_badge(pr)
+      badge = @theme.pr_badge(pr, quiet: section == :settled)
       more = (s.prs.size > 1) ? @p.dim(" · +#{s.prs.size - 1}") : ""
       badge + more
     end
@@ -434,7 +439,8 @@ module ClaudeInbox
       block_w = block.map { |l| Text.width(l) }.max || 0
       left = [(width - block_w) / 2, 0].max
       top = [(lines.size - block.size) / 2, 0].max
-      out = lines.dup
+      # Nothing behind a pop-up is clickable.
+      out = lines.map { |l| l.gsub(Text::LINK, "") }
       block.each_with_index do |bl, i|
         y = top + i
         next if y >= out.size

data/lib/claude_inbox/terminal.rb

@@ -17,10 +17,14 @@ module ClaudeInbox
     # know the mode ignore it and keep scrolling their own history.
     WHEEL_KEYS_ON = "\e[?1007h"
     WHEEL_KEYS_OFF = "\e[?1007l"
-    # Button events in the SGR encoding, which Mouse parses. A terminal that
-    # knows neither mode falls back to WHEEL_KEYS_ON, or its own scrollback.
-    MOUSE_ON = "\e[?1000h\e[?1006h"
-    MOUSE_OFF = "\e[?1006l\e[?1000l"
+    # Any-motion events in the SGR encoding, which Mouse parses: the pointer
+    # moving too, so a link can take a hand pointer. A terminal that knows
+    # neither mode falls back to WHEEL_KEYS_ON, or its own scrollback.
+    MOUSE_ON = "\e[?1003h\e[?1006h"
+    MOUSE_OFF = "\e[?1006l\e[?1003l"
+    # OSC 22 sets the pointer's shape. Terminals without it ignore it.
+    POINTER = "\e]22;pointer\e\\"
+    POINTER_OFF = "\e]22;default\e\\"
     # Bracketed paste: what is pasted arrives fenced off from what is typed
     # (Paste), and a pasted image, which has no text, arrives as an empty
     # fence rather than not at all.
@@ -33,6 +37,7 @@ module ClaudeInbox
       @painter = Painter.new(out)
       @restored = true
       @size = nil
+      @over_link = false
     end
 
     # The tty's mode is kept as found, not assumed: `cooked!` sets a stock
@@ -53,8 +58,9 @@ module ClaudeInbox
     def restore
       return if @restored
       @restored = true
+      @over_link = false
       @input.console_mode = @mode if @mode && @input.tty?
-      @out.print TTY::Cursor.show, PASTE_OFF, MOUSE_OFF, WHEEL_KEYS_OFF, ALT_OFF
+      @out.print TTY::Cursor.show, POINTER_OFF, PASTE_OFF, MOUSE_OFF, WHEEL_KEYS_OFF, ALT_OFF
       @out.flush
     rescue
       nil
@@ -93,6 +99,13 @@ module ClaudeInbox
 
     def paint(lines) = @painter.paint(lines)
 
+    def pointer(over_link)
+      return if over_link == @over_link
+      @over_link = over_link
+      @out.print(over_link ? POINTER : POINTER_OFF)
+      @out.flush
+    end
+
     private
 
     def measure

data/lib/claude_inbox/text.rb

@@ -6,15 +6,42 @@ module ClaudeInbox
   # Width-aware string helpers. Never use String#[] on user-facing text:
   # session names carry emoji and glyphs that break byte/char slicing.
   module Text
-    ANSI = /\e\[[0-9;?]*[A-Za-z]/
+    # An OSC 8 hyperlink's opening or, with an empty url, its closing.
+    LINK = /\e\]8;[^;\e]*;[^\e]*\e\\/
+    CSI = /\e\[[0-9;?]*[A-Za-z]/
+    ANSI = /#{CSI}|#{LINK}/
     # A grapheme, or an escape that takes no column.
     GLYPH = /#{ANSI}|\X/
     ELLIPSIS = "…"
+    LINK_END = "\e]8;;\e\\"
 
     module_function
 
     def strip_ansi(s) = s.gsub(ANSI, "")
 
+    # The url comes from a transcript, so a control character in it is dropped.
+    def link(s, url) = "\e]8;;#{url.delete("\x00-\x1f\x7f")}\e\\#{s}#{LINK_END}"
+
+    Link = Data.define(:columns, :url, :text)
+
+    def links(s)
+      found = []
+      col = 0
+      start = url = text = nil
+      s.scan(GLYPH) do |g|
+        if g.start_with?("\e]8;")
+          found << Link.new(start...col, url, text) if url
+          url = g[/\A\e\]8;[^;\e]*;([^\e]+)/, 1]
+          start = col
+          text = +""
+        elsif !g.start_with?("\e")
+          text << g if url
+          col += Unicode::DisplayWidth.of(g)
+        end
+      end
+      found
+    end
+
     # Bytes that aren't UTF-8 become U+FFFD.
     def utf8(s) = String.new(s.to_s, encoding: Encoding::UTF_8).scrub("\uFFFD")
 
@@ -32,8 +59,8 @@ module ClaudeInbox
       take(s, w - width(ELLIPSIS)) << ELLIPSIS
     end
 
-    # First `n` columns, no ellipsis. Colors survive the cut, and a reset
-    # closes whichever one the cut left open.
+    # First `n` columns, no ellipsis. Colors and links survive the cut, and
+    # the cut closes whichever one it left open.
     def take(s, n)
       out = +""
       used = 0
@@ -43,6 +70,8 @@ module ClaudeInbox
         out << g
         used += gw
       end
+      last_link = out.scan(LINK).last
+      out << LINK_END if last_link && last_link != LINK_END
       (out.include?("\e") && !out.end_with?("\e[0m")) ? out << "\e[0m" : out
     end
 

data/lib/claude_inbox/theme.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative "text"
+
 module ClaudeInbox
   # Chrome hues tuned to Atom One Dark rather than the terminal profile's
   # own ANSI reds and greens. 256-color indices, since those render the
@@ -28,12 +30,16 @@ module ClaudeInbox
 
     # "#885 open", or "#885 open · ✗ checks" when something is wrong.
     # `name` replaces the bare number where that would be ambiguous.
-    def pr_badge(pr, name = pr.short)
-      return dim(name) unless PR_HUES.key?(pr.state)
-      badge = tint(PR_HUES[pr.state], "#{name} #{pr.state.downcase}")
-      pr.trouble ? badge + dim(" · ") + tint(PR_TROUBLE_HUES[pr.health], pr.trouble) : badge
+    def pr_badge(pr, name = pr.short, quiet: false)
+      hue = PR_HUES[pr.state] unless quiet
+      linked = Text.link(underline(tint(hue, name)), pr.url)
+      return linked unless PR_HUES.key?(pr.state)
+      badge = linked + tint(hue, " #{pr.state.downcase}")
+      (pr.trouble && !quiet) ? badge + dim(" · ") + tint(PR_TROUBLE_HUES[pr.health], pr.trouble) : badge
     end
 
+    def underline(text) = "\e[4m#{text}\e[24m"
+
     # A filled pill: editor-background text on the hue, for a selected row
     # or choice — the inverse of `color`, which puts the hue on the text.
     def pill(text, hue) = "\e[38;5;#{BG};48;5;#{HUES.fetch(hue)}m#{text}\e[0m"

data/lib/claude_inbox.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ClaudeInbox
-  VERSION = "0.13.1"
+  VERSION = "0.14.0"
 end

data/mod/hooks/register.tsx

@@ -335,7 +335,7 @@ export const register: Register = (on, options) => {
   })
 
   on('ui.render', { component: 'Pane', requestId: PANE }, async ($, e) => {
-    const { Box, Text, Button } = $.ui.resolve(e)
+    const { Box, Text, Button, Link } = $.ui.resolve(e)
     const list = await read($, rows)
     const viewing = await read($, current)
     const folded = await read($, folds)
@@ -497,9 +497,16 @@ export const register: Register = (on, options) => {
                     {/* The label gives up its cells to the PR chip, which would otherwise wrap under it. */}
                     {one.pr ? (
                       <Box flexShrink={0}>
+                        <Box key={`pr:${itemKey(item)}`} marginLeft={1} hover={{ backgroundColor: 'subtle' }}>
+                          <Link href={one.pr.url}>
+                            <Text underline color={isQuiet ? undefined : PR_COLOR[one.pr.state]} dimColor={isQuiet}>
+                              {one.pr.short}
+                            </Text>
+                          </Link>
+                        </Box>
                         <Text color={isQuiet ? undefined : PR_COLOR[one.pr.state]} dimColor={isQuiet}>
                           {' '}
-                          {one.pr.short} {one.pr.state}
+                          {one.pr.state}
                         </Text>
                       </Box>
                     ) : null}

data/mod/tests/pane.test.tsx

@@ -129,7 +129,8 @@ test("rows sit under the gem's sections, Settled folded until za opens it", asyn
     expect((await ui.findAll({ type: 'Text', text: '● ' })).filter(dot => dot.props.color === 'blue')).toHaveLength(1)
     expect((await ui.find({ type: 'Text', text: 'finished unseen' }))?.props.bold).toBe(true)
     expect(await ui.find({ type: 'Text', text: 'still going' })).toBeUndefined()
-    expect(await ui.find({ type: 'Text', text: /#7 open/ })).toBeDefined()
+    expect((await ui.find({ type: 'Link' }))?.props.href).toBe('https://github.com/o/r/pull/7')
+    expect((await ui.find({ type: 'Text', text: '#7' }))?.props.underline).toBe(true)
     expect(await ui.find({ text: /1 folded/ })).toBeDefined()
     expect(await ui.find({ text: /put away/ })).toBeUndefined()
     await ui.press({ key: 'g' })

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: claude-inbox
 version: !ruby/object:Gem::Version
-  version: 0.13.1
+  version: 0.14.0
 platform: ruby
 authors:
 - Jordan Byron