claude-inbox 0.6.0 → 0.7.0

data/mod/hooks/register.tsx

@@ -197,14 +197,20 @@ async function requestSwitch($: EngineInterface, host: Host, id: string): Promis
   })
 }
 
-// The pane keeps the keys while its session is away, so coming back to it would put the
-// person's typing in the pane. The engine has no blur: reopening without `focus` is the
-// one way to give the keys to the prompt.
+// Every key that takes the person out of the pane, to another session or the browser, calls
+// this first: the pane would keep the keys while they are away, and their typing on coming
+// back would land in it. The engine has no blur: reopening without `focus` is the one way
+// to give the keys to the prompt.
 async function handBackKeys($: EngineInterface): Promise<void> {
   await $.ui.close({ id: PANE })
   await $.ui.open(OPEN)
 }
 
+async function openInBrowser($: EngineInterface, url: string): Promise<void> {
+  await handBackKeys($)
+  await $.process.run(['open', url])
+}
+
 // Only while no inbox writes the snapshot; the gem's writer lock turns a second one away.
 function startHeadless($: EngineInterface, host: Host, now: number): void {
   if (host.isHeadlessRunning || now - host.headlessStartedAt < RESTART_S) return
@@ -410,8 +416,8 @@ export const register: Register = (on, options) => {
                     $.ui.toast(copied.isCopied ? `copied: claude attach ${row.id}` : 'no clipboard here'),
                   )
                 }),
-                key('o', () => (row?.pr ? void $.process.run(['open', row.pr.url]) : $.ui.toast('no pull request linked'))),
-                key('w', () => (row?.remote ? void $.process.run(['open', row.remote]) : $.ui.toast('no claude.ai/code page for this session'))),
+                key('o', () => (row?.pr ? void openInBrowser($, row.pr.url) : $.ui.toast('no pull request linked'))),
+                key('w', () => (row?.remote ? void openInBrowser($, row.remote) : $.ui.toast('no claude.ai/code page for this session'))),
                 key('q', () => void $.ui.close({ id: PANE })),
               ]
 

data/mod/tests/pane.test.tsx

@@ -220,6 +220,38 @@ 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) => {
+  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' }
+  on('state.get', ($, e, next) => (e.key === 'rows' ? seeded([remote]) : e.key === 'writtenAt' ? seeded(1_700_000_000) : next(e)))
+  const calls: string[] = []
+  on('ui.close', ($, e) => {
+    calls.push(`close ${e.id}`)
+    return { value: undefined }
+  })
+  on('ui.open', ($, e) => {
+    calls.push(`open ${e.id}${e.focus ? ' focused' : ''}`)
+    return { value: { isPlaced: true } }
+  })
+  on('process.run', ($, e) => {
+    calls.push(e.argv.join(' '))
+    return { value: { code: 0, stdout: '', stderr: '' } }
+  })
+  const ui = await $.ui.mount({ ...PANE, surface: 'terminal' })
+  await ui.press({ key: 'o' })
+  await ui.press({ key: 'w' })
+  expect(calls).toEqual([
+    'close inbox',
+    'open inbox',
+    'open https://github.com/o/r/pull/7',
+    'close inbox',
+    'open inbox',
+    'open https://claude.ai/code/session_b2',
+  ])
+  await ui.unmount()
+})
+
 test('a click on a row puts the cursor on it before it asks for the switch', async ($, on) => {
   mock.env(on, { HOME: '/Users/me' })
   mock.clock(on, { now: 1_700_000_000_000 })

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: claude-inbox
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Jordan Byron
@@ -108,21 +108,28 @@ files:
 - lib/claude_inbox/actions.rb
 - lib/claude_inbox/agents_client.rb
 - lib/claude_inbox/app.rb
+- lib/claude_inbox/app/attaching.rb
+- lib/claude_inbox/app/browsing.rb
+- lib/claude_inbox/app/filtering.rb
+- lib/claude_inbox/app/navigating.rb
+- lib/claude_inbox/app/spawning.rb
+- lib/claude_inbox/app/triaging.rb
+- lib/claude_inbox/app/updating.rb
+- lib/claude_inbox/cli.rb
 - lib/claude_inbox/config.rb
 - lib/claude_inbox/debug.rb
 - lib/claude_inbox/dialog.rb
 - lib/claude_inbox/headless.rb
 - lib/claude_inbox/images.rb
+- lib/claude_inbox/input.rb
 - lib/claude_inbox/job_state.rb
 - lib/claude_inbox/keymap.rb
-- lib/claude_inbox/logs.rb
 - lib/claude_inbox/mod.rb
 - lib/claude_inbox/mouse.rb
 - lib/claude_inbox/new_session_form.rb
 - lib/claude_inbox/painter.rb
 - lib/claude_inbox/palette.rb
 - lib/claude_inbox/paste.rb
-- lib/claude_inbox/peek.rb
 - lib/claude_inbox/poller.rb
 - lib/claude_inbox/pull_requests.rb
 - lib/claude_inbox/rate_limits.rb

data/lib/claude_inbox/logs.rb

@@ -1,68 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "vt_screen"
-
-module ClaudeInbox
-  # The `claude logs <id>` replay of each session as readable lines, fetched
-  # off the main thread, debounced and cached.
-  class Logs
-    DEBOUNCE = 0.25
-    TTL = 10
-    MAX_LINES = 400
-
-    def initialize(client, clock: -> { Time.now })
-      @client = client
-      @clock = clock
-      @cache = {}
-      @mutex = Mutex.new
-      @pending = nil
-      @pending_at = nil
-      @requests = Queue.new
-    end
-
-    def start
-      return if @thread&.alive?
-      @thread = Thread.new { worker }
-    end
-
-    def cached(id)
-      @mutex.synchronize { @cache[id]&.first }
-    end
-
-    def want(id)
-      @pending = id
-      @pending_at = @clock.call
-    end
-
-    # Promotes a request that has sat still for DEBOUNCE to the worker,
-    # unless the cache already has a fresh answer.
-    def tick
-      return unless @pending && @clock.call - @pending_at >= DEBOUNCE
-      id = @pending
-      @pending = nil
-      fresh = @mutex.synchronize { (e = @cache[id]) && @clock.call - e[1] < TTL }
-      @requests << id unless fresh
-    end
-
-    def stop = @thread&.kill
-
-    private
-
-    def worker
-      loop do
-        id = @requests.pop
-        id = @requests.pop until @requests.empty? # only the latest matters
-        lines = fetch(id)
-        @mutex.synchronize { @cache[id] = [lines, @clock.call] }
-      end
-    end
-
-    def fetch(id)
-      raw = @client.logs(id)
-      return ["(no output available — the session's process is not running)"] if raw.nil?
-      VtScreen.new.feed(raw).lines.last(MAX_LINES)
-    rescue => e
-      ["(logs failed: #{e.message})"]
-    end
-  end
-end

data/lib/claude_inbox/peek.rb

@@ -1,77 +0,0 @@
-# frozen_string_literal: true
-
-module ClaudeInbox
-  # The peek pane: whether it is open, which row it is on, how far back it
-  # is scrolled, and what to paint for that. The lines themselves come from
-  # Logs, which it asks on every selection change so that opening the pane
-  # lands on a warm cache. Main-thread state only.
-  class Peek
-    TERMINAL_NOTE = "This is a claude you opened in a terminal yourself. The daemon can't attach to it, read its output, or stop it from outside. Switch to that window."
-    REMOTE_NOTE = "This is a Remote Control session driven from claude.ai/code. The daemon can't attach to it or read its output from here. Enter pulls it into the daemon; w opens it in the web or mobile app."
-
-    # What Renderer paints: the body lines, the title bar, the dim line under it.
-    View = Struct.new(:lines, :title, :subtitle)
-
-    def initialize(logs)
-      @logs = logs
-      @open = false
-      @offset = 0
-      @session = nil
-    end
-
-    def open? = @open
-
-    def toggle
-      @open = !@open
-      @logs.want(@session.id) if @open && @session&.actionable?
-    end
-
-    def close = @open = false
-
-    # Called on every selection change; a fold has no session.
-    def select(session)
-      @session = session
-      @offset = 0
-      @logs.want(session.id) if session&.actionable?
-    end
-
-    # Positive scrolls back into history, negative towards the tail.
-    def scroll(delta)
-      @offset = [@offset + delta, 0].max
-    end
-
-    # `row` is the selected row as the frame shows it, or nil on a fold or
-    # with nothing selected. Answers nil when there is no pane to paint.
-    def view(row, body_h)
-      return nil unless @open && row
-      View.new(scrolled(body(row), body_h), row.label, subtitle(row))
-    end
-
-    private
-
-    def body(row)
-      return interactive_note(row.session) if row.session.interactive?
-      @logs.cached(row.session.id) || ["(loading…)"]
-    end
-
-    def interactive_note(s)
-      [s.remote? ? REMOTE_NOTE : TERMINAL_NOTE, "", "pid #{s.pid} · #{s.cwd}", "session #{s.session_id}", s.remote_url].compact
-    end
-
-    def subtitle(row)
-      s = row.session
-      parts = [s.effective_state, s.status, s.waiting_for, s.id, s.started_at&.strftime("started %b %-d %H:%M")].compact
-      parts += s.prs.map { |pr| "#{pr.short} #{pr.state&.downcase || "?"}" }
-      parts.join(" · ")
-    end
-
-    # Lines are shown tail-first; the offset scrolls back into history and is
-    # clamped to however much history there is.
-    def scrolled(lines, view_h)
-      return lines if @offset.zero?
-      max_off = [lines.size - view_h, 0].max
-      @offset = [@offset, max_off].min
-      lines[0, lines.size - @offset]
-    end
-  end
-end