claude-inbox 0.13.1 → 0.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36fa381f8d03173896c6b44dff47534ed54119c9ff529833a9783c7f5c524506
-  data.tar.gz: '0386da94a6ffa5259bbad165857821f44b082dc5a64d957beae352c46dc61100'
+  metadata.gz: 6f7809d0909d4391440edb2d01d0452b976b28ed59832d21a05518a11ef164b1
+  data.tar.gz: 106434e33d35e297d312b441bfa1ff333a16a7faddd54443c65ba006a6dcf95e
 SHA512:
-  metadata.gz: bb933d5723ee131285824d2faea2570762854a4695da2ecd98716d5b9d981b2b36aece7abbda8acf92fa063b47f52f40ac2931136ef5fecee194f0fda7cdc72c
-  data.tar.gz: 3593591f4192056035fdc13e3480a26824794be46df0d8aa5b72fdb5c84a4b6b039d658b4279ec64cdf35ec94c2d65533117321c01bdd9b3e06a4267571a83b6
+  metadata.gz: cf8d43ffd7d434977f4bcb18acb1d06817854e92973cad5e5e8d7d8efd8e611e82dc1820499f65c50c99f0b12a5b6d3d7d3a18235c1ad693c0efa2325be348a6
+  data.tar.gz: c0560e7bd6444fbf325618d8b2eb7172d74b0d63d331058cf9aca5afbcf88da1f0fd5a4d31c1576e808a4b022cf3933247bc61c5ba229f4543b171805e2aa3df

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")}"])
@@ -127,13 +144,13 @@ module ClaudeInbox
 
       def lines(width, _caret)
         case kind
-        when :stop then [ask("Stop", width), "", "  " + hint("y", "stop it"), "  " + hint("esc", "cancel")]
+        when :stop then [ask("Stop", width), *answers("stop it", "cancel")]
         when :delete
           [ask("Delete", width), "  Its worktree and conversation", "  go with it.",
-            "", "  " + hint("y", "delete it"), "  " + hint("esc", "keep it")]
+            *answers("delete it", "keep it")]
         when :adopt
           ["  Pull this session into the daemon?", "  Its phone session ends and a",
-            "  new one takes its place.", "", "  " + hint("y", "adopt it"), "  " + hint("esc", "cancel")]
+            "  new one takes its place.", *answers("adopt it", "cancel")]
         end
       end
 
@@ -147,6 +164,8 @@ module ClaudeInbox
 
       # One line, so a long name is cut rather than wrapped mid-word.
       def ask(verb, width) = "  #{verb} #{Text.truncate(@label.to_s, width - Text.width(verb) - 4)}?"
+
+      def answers(yes, no) = ["", "  " + hint("y", yes), "  " + hint("esc", no)]
     end
 
     # The first launch without the Claude Code pane asks once whether to add
@@ -170,9 +189,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/new_session_form.rb

@@ -333,7 +333,7 @@ module ClaudeInbox
 
     def prompt_box(f, w, h)
       on = f.equal?(focused)
-      edge = on ? ->(s) { @theme.cyan(s) } : ->(s) { @p.dim(s) }
+      edge = ->(s) { @theme.tint((:cyan if on), s) }
       rows, hidden = f.value.view(w - 4, h, cursor: (caret if on), chip: ->(s) { @theme.cyan_bold(s) })
       rows = [@p.dim("What should this session do?")] if f.value.empty? && !on
       rows += [""] * (h - rows.size)

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.1"
 end

data/mod/hooks/register.tsx

@@ -95,6 +95,11 @@ function itemKey(item: Item): string {
   return item.kind === 'fold' ? `fold:${item.section}` : `row:${item.row.id ?? item.row.label}`
 }
 
+// The Button drawn for a row's pull request, a click on which opens it.
+function prKey(item: Item): string | undefined {
+  return item.kind === 'row' && item.row.pr ? `pr:${itemKey(item)}` : undefined
+}
+
 // The visible box drawn for an item, what the pane scrolls to.
 function viewKey(item: Item): string {
   return `view:${itemKey(item)}`
@@ -313,14 +318,18 @@ export const register: Register = (on, options) => {
   // the cursor's item, the arrows move it to the next, and the cursor follows.
   on('ui.focus', { requestId: PANE }, async ($, e, next) => {
     const flat = await listed($)
+    const was = onList(flat, await read($, cursor))
     if (e.element === undefined) {
-      const item = flat[onList(flat, await read($, cursor))]
+      const item = flat[was]
       return item ? next({ ...e, element: itemKey(item) }) : next(e)
     }
-    const at = flat.findIndex(item => itemKey(item) === e.element)
+    // A PR chip's Button sits in the ring between its row and the next: the ring arrives on it going down
+    // from its own row, so it passes on to the next; going up, or by a click, it lands on its own row.
+    const chipOf = flat.findIndex(item => prKey(item) === e.element)
+    const at = chipOf < 0 ? flat.findIndex(item => itemKey(item) === e.element) : chipOf === was ? Math.min(chipOf + 1, flat.length - 1) : chipOf
     if (at < 0) return {}
     await update($, cursor, () => at)
-    return next(e)
+    return next({ ...e, element: itemKey(flat[at]!) })
   })
 
   // An arrow in a focused pane scrolls its body a row while the tree is taller than the
@@ -384,6 +393,12 @@ export const register: Register = (on, options) => {
       if (!item.row.id || !item.row.actionable) return $.ui.toast('this session cannot be attached')
       void requestSwitch($, host, item.row.id)
     }
+    // The click's focus may have walked the ring past the row, so the cursor is put back on it.
+    const openPr = (item: Item) => () => {
+      if (item.kind !== 'row' || !item.row.pr) return
+      const url = item.row.pr.url
+      void moveTo($, flat, flat.indexOf(item)).then(() => openInBrowser($, url))
+    }
 
     // Only the keys a chord allows are mounted while it waits, so any other key is a no-op.
     const keys =
@@ -475,10 +490,13 @@ export const register: Register = (on, options) => {
               return (
                 <Box key={viewKey(item)} flexDirection="column">
                   <Box>
-                    <Text color={isHere ? 'yellow' : 'blue'}>{isHere ? '› ' : dot ? '● ' : '  '}</Text>
-                    <Text color={COLOR[one.state]} dimColor={isQuiet || one.state === 'done'}>
-                      {GLYPH[one.state] ?? '·'}{' '}
-                    </Text>
+                    {/* Only the label gives way to a long row: a squeezed glyph wraps under itself. */}
+                    <Box flexShrink={0}>
+                      <Text color={isHere ? 'yellow' : 'blue'}>{isHere ? '› ' : dot ? '● ' : '  '}</Text>
+                      <Text color={COLOR[one.state]} dimColor={isQuiet || one.state === 'done'}>
+                        {GLYPH[one.state] ?? '·'}{' '}
+                      </Text>
+                    </Box>
                     {/* A Button cannot truncate, so its Box clips a long label to the row. A Button takes no
                         color either, so a session's /color is the Box's background rather than the label's ink. */}
                     <Box key={`label:${itemKey(item)}`} height={1} flexShrink={1} overflow="hidden" backgroundColor={tint}>
@@ -494,12 +512,23 @@ export const register: Register = (on, options) => {
                         {one.isUnread && !isQuiet ? <Text bold>{text}</Text> : text}
                       </Button>
                     </Box>
-                    {/* The label gives up its cells to the PR chip, which would otherwise wrap under it. */}
+                    {/* Takes what a short label leaves, so the PR and age sit at the right edge on every row;
+                        the label's own box would stretch a /color tint across it. */}
+                    <Box flexGrow={1} />
+                    {/* The label gives up its cells to the PR chip, which would otherwise wrap under it. The chip is
+                        a Button rather than a Link: a terminal without OSC 8 cannot click a Link, and prints its URL. */}
                     {one.pr ? (
                       <Box flexShrink={0}>
+                        <Box marginLeft={1}>
+                          <Button key={prKey(item)} plain label={one.pr.short} onPress={openPr(item)}>
+                            <Text underline color={isQuiet ? undefined : PR_COLOR[one.pr.state]} dimColor={isQuiet}>
+                              {one.pr.short}
+                            </Text>
+                          </Button>
+                        </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: 'Button', key: 'pr:row:b2' }))?.props.label).toBe('#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' })
@@ -249,7 +250,7 @@ test('a switch hands the keys back to the prompt before it asks, so a return lan
   await ui.unmount()
 })
 
-test('o and w hand the keys back to the prompt before they open the browser', async ($, on) => {
+test('o, w and a click on the PR hand the keys back to the prompt before they open the browser', async ($, on) => {
   mock.env(on, { HOME: '/Users/me' })
   mock.clock(on, { now: 1_700_000_000_000 })
   const remote = { ...ROWS[1], remote: 'https://claude.ai/code/session_b2' }
@@ -270,6 +271,7 @@ test('o and w hand the keys back to the prompt before they open the browser', as
   const ui = await $.ui.mount({ ...PANE, surface: 'terminal' })
   await ui.press({ key: 'o' })
   await ui.press({ key: 'w' })
+  await ui.press({ key: 'pr:row:b2' })
   expect(calls).toEqual([
     'close inbox',
     'open inbox',
@@ -277,6 +279,9 @@ test('o and w hand the keys back to the prompt before they open the browser', as
     'close inbox',
     'open inbox',
     'open https://claude.ai/code/session_b2',
+    'close inbox',
+    'open inbox',
+    'open https://github.com/o/r/pull/7',
   ])
   await ui.unmount()
 })
@@ -359,3 +364,23 @@ test('with openOnStart off only /inbox opens it', { options: { openOnStart: fals
   await polled
   expect(opened).toEqual([])
 })
+
+test("the ring steps over a row's PR chip: down to the next row, up onto its own", async ($, on) => {
+  mock.env(on, { HOME: '/Users/me' })
+  mock.clock(on, { now: 1_700_000_000_000 })
+  on('state.get', ($, e, next) => (e.key === 'rows' ? seeded(ROWS) : e.key === 'writtenAt' ? seeded(1_700_000_000) : next(e)))
+  const landed: (string | undefined)[] = []
+  on('ui.focus', ($, e) => {
+    landed.push(e.element)
+    return {}
+  })
+  const ui = await $.ui.mount({ ...PANE, surface: 'terminal' })
+  // The harness has no ring to walk, so the test raises ui.focus as the engine does for an arrow.
+  const arrowOnto = (element: string) =>
+    $.ui.focus({ component: 'Pane', requestId: 'inbox', plugin: 'inbox-pane', element, origin: { kind: 'person' } } as never)
+  await arrowOnto('row:b2')
+  await arrowOnto('pr:row:b2')
+  await arrowOnto('pr:row:b2')
+  expect(landed).toEqual(['row:b2', 'row:uuid-term', 'row:b2'])
+  await ui.unmount()
+})

metadata

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