claude-inbox 0.12.3 → 0.13.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 58507c555e4ab15701ccf14104a7d365abed00c4f3d4386afba2667b12a6afe1
-  data.tar.gz: 21cb5cc8697d843dae5c557b97872ccca09c0148c1c4655bcb8772b15e156d00
+  metadata.gz: 36fa381f8d03173896c6b44dff47534ed54119c9ff529833a9783c7f5c524506
+  data.tar.gz: '0386da94a6ffa5259bbad165857821f44b082dc5a64d957beae352c46dc61100'
 SHA512:
-  metadata.gz: 24352714cad8d1974cfe056ed606027aaeac4003e621abb6fc72c76fc9c18dfad052f86ba1bd157dec43cad09415f12db64374aae836d948687eabac645145ea
-  data.tar.gz: 7e426498a2aca8f7961c9a8e8e61b68a138b8c6f2d414ac0bb5b8334f61b923966bf1894c994a76bd8880e442c43d32b8f456116ab670a4a70fbfa347bc81262
+  metadata.gz: bb933d5723ee131285824d2faea2570762854a4695da2ecd98716d5b9d981b2b36aece7abbda8acf92fa063b47f52f40ac2931136ef5fecee194f0fda7cdc72c
+  data.tar.gz: 3593591f4192056035fdc13e3480a26824794be46df0d8aa5b72fdb5c84a4b6b039d658b4279ec64cdf35ec94c2d65533117321c01bdd9b3e06a4267571a83b6

data/README.md

@@ -49,7 +49,8 @@ It opens when a session starts. To stop that, set
 
 ## Sections
 
-1. **Pinned.** Rows you pinned with `t`.
+1. **Pinned.** Rows you pinned with `t`. Snoozing a pinned row moves it to
+   Snoozed, marked `★`, until it wakes back into Pinned.
 2. **Needs you.** Blocked or failed sessions, or ones whose pull request has
    failing checks or changes requested, that you haven't attached to yet.
 3. **Active.** Everything else that's running or not yet settled, including
@@ -58,6 +59,10 @@ It opens when a session starts. To stop that, set
 5. **Settled.** Settled with `x`, or every pull request is merged or closed.
    Collapsed.
 
+`--sort=activity` puts the session that spoke most recently first in every
+section but Snoozed, read from its transcript. Put it in the config file to
+keep it.
+
 Under each row is what the session is waiting on, what it produced, or its
 status line. A row with a pull request shows it as `#885 open`, `#885 merged`
 and so on, with `✗ checks`, `changes requested` or `conflicts` after it when
@@ -65,7 +70,9 @@ something is wrong, and `+2` after it when the session has two more.
 A steady `◌` with `idle · 1 shell` means the agent stopped
 and is waiting on a shell or sub-agent it started. A blue `●` marks a session you
 haven't attached to since it got blocked or finished; one you have attached
-to since is dimmed, unless `/color` tints it.
+to since is dimmed, unless `/color` tints it. The age after the badge is how
+long since the session last said anything, read from its transcript: a
+`working` row that spoke 30s ago is still talking, one silent for 2h is not.
 
 ## Keys
 
@@ -93,7 +100,7 @@ Vim bindings. Arrows and the mouse work too.
 | `P` | link a pull request by hand (empty clears) |
 | `X` | stop the session; `Enter` resumes it later |
 | `Ctrl-x` | delete the session, transcript and worktree for good |
-| `/` | filter; `Enter` keeps it, `Esc` clears |
+| `/` | filter by name, directory, prompt, status or pull request (`#885` or its title); `Enter` keeps it, `Esc` clears |
 | `R` | poll now |
 | `U` | update claude-inbox when the header says a newer one is out |
 | `q` | quit |
@@ -154,7 +161,8 @@ covers ssh, the JSON API and an iOS Shortcut.
 ## Pull requests
 
 Pull requests show up when a session's transcript links to them. Install
-`gh` to keep their state current. Use `P` to link one by hand.
+`gh` to keep their state current. Use `P` to link one by hand, and `/#885`
+to find the session behind a pull request.
 
 ## Notifications
 

data/exe/claude-inbox

@@ -1,6 +1,10 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
+# Claude's files are UTF-8 whatever the locale says; under LANG=C a read of
+# ~/.claude.json would otherwise come back US-ASCII and fail to parse.
+Encoding.default_external = Encoding::UTF_8
+
 require "claude_inbox"
 require "claude_inbox/cli"
 

data/lib/claude_inbox/actions.rb

@@ -39,7 +39,7 @@ module ClaudeInbox
     # Applies every request on disk to `store` and returns how many it took.
     def drain(store)
       return 0 unless @dir && File.directory?(@dir)
-      keys = store.sessions.select(&:actionable?).map(&:key)
+      keys = store.actionable_keys
       Dir.glob(File.join(@dir, "*.json")).sort.count { |path| take(path, store, keys) }
     end
 

data/lib/claude_inbox/agents_client.rb

@@ -27,7 +27,6 @@ module ClaudeInbox
       classify_origins(parse(r.out))
     end
 
-    # Poll interval for the agents-view watchdog below.
     WATCH_INTERVAL = 0.05
 
     # Hands the terminal to the child; caller must have restored cooked mode.
@@ -35,14 +34,11 @@ module ClaudeInbox
     # False when the attach failed on its own, as a session that cannot be
     # woken does: the CLI prints why and exits 1.
     #
-    # Pressing ← inside an attached session detaches it and then `claude
-    # attach` execs itself in place as `claude agents` (same pid). There is
-    # no flag or env var that suppresses only that relaunch: the one switch
-    # that exists disables attach too. So we watch the child's command line
-    # and, the moment it becomes the agents view, terminate it. The user then
-    # lands back in the inbox instead of native agent view.
-    # The block, polled with the command line, ends the attach early when it
-    # answers true: how a switch request from another front end is honoured.
+    # ← turns `claude attach` into `claude agents` in place, and nothing
+    # switches off only that (docs/cli-quirks.md), so the watchdog kills the
+    # child once it does. The block, polled alongside, ends the attach early
+    # when it answers true. That is how a switch request from another front
+    # end is honoured.
     def attach(id, &stop)
       pid = Process.spawn(*Subprocess.command(@bin, "attach", id))
       watchdog = Thread.new { kill_when_agents_view(pid, &stop) }
@@ -76,8 +72,7 @@ module ClaudeInbox
     # its path completion escapes them; a quoted path is not recognized.
     def self.mention(path) = "@" + path.gsub(" ", "\\ ")
 
-    # Pure so it can be tested: a setting left nil leaves its flag off. The
-    # prompt goes after "--", where a leading dash is text rather than an
+    # The prompt goes after "--", where a leading dash is text rather than an
     # unknown option, and --remote-control sits just before it: its optional
     # name would otherwise eat the prompt (both checked on 2.1.282).
     def self.spawn_args(bin, prompt:, model: nil, effort: nil, permission_mode: nil, worktree: false, name: nil, remote: false)
@@ -115,14 +110,9 @@ module ClaudeInbox
     # you type in: the headless print mode and the SDK's stream protocol.
     HEADLESS_FLAGS = %w[-p --print --input-format --output-format].freeze
 
-    # pid => origin, given `ps` for the sessions and for their parents. Pure.
-    #
-    #   :remote    a Remote Control worker: --sdk-url, or a `claude rc` parent
-    #   :headless  `claude -p "..."` or an SDK stream run. Its parent is
-    #              whatever shell spawned it, not the claude that asked for
-    #              it, so only its own command line gives it away
-    #   :subagent  parented by another claude process
-    #   :terminal  everything else, which is a claude you are sitting in
+    # pid => origin, given `ps` for the sessions and for their parents.
+    # A headless run's parent is whatever shell spawned it, not the claude
+    # that asked for it, so only its own command line gives it away.
     def self.origins(rows, parent_cmd)
       rows.to_h do |pid, ppid, cmd|
         parent = parent_cmd[ppid].to_s
@@ -143,7 +133,7 @@ module ClaudeInbox
 
     # pid => bridge session id, for the workers a `claude remote-control`
     # server spawned: each is started with `--session-id cse_…`, the same id
-    # a background session records as bridgeSessionId. Pure.
+    # a background session records as bridgeSessionId.
     def self.bridge_ids(rows)
       rows.filter_map { |pid, _, cmd| (m = cmd.match(/--session-id[= ](cse_\w+)/)) && [pid, m[1]] }.to_h
     end
@@ -185,10 +175,6 @@ module ClaudeInbox
       assign_origins(sessions, origins_from(rows), self.class.bridge_ids(rows))
     end
 
-    # Tags each interactive session with where it is driven from (pid =>
-    # origin, terminal when unlisted) and the bridge a remote one is served
-    # over, and drops the unattended ones, whose parent is the row worth
-    # showing.
     def assign_origins(sessions, origins, bridges)
       sessions
         .map { |s| s.interactive? ? s.with(origin: origins.fetch(s.pid, :terminal), bridge_id: bridges[s.pid]) : s }
@@ -259,9 +245,7 @@ module ClaudeInbox
       @bridges = bridges
     end
 
-    # Fixture rows carry no process tree; tag each interactive row from the
-    # pid => origin map the test hands in, otherwise as a terminal, then drop
-    # the unattended ones same as the real client does.
+    # Fixture rows carry no process tree.
     def list
       assign_origins(parse(File.read(@path)), @origins, @bridges)
     end

data/lib/claude_inbox/app/attaching.rb

@@ -35,7 +35,7 @@ module ClaudeInbox
           @terminal.release do
             @terminal.pause unless @client.attach(id) { @switch.requested_for?(from) }
           end
-          id = @switch.take(@store.sessions.select(&:actionable?).map(&:key))
+          id = @switch.take(@store.actionable_keys)
         end
       end
 

data/lib/claude_inbox/app.rb

@@ -47,7 +47,7 @@ module ClaudeInbox
 
     # The reaper deletes sessions and the listener lets other machines in,
     # so both are off unless `bin/claude-inbox` switches them on.
-    def initialize(client: AgentsClient.new, store: Store.new, pull_requests: PullRequests.new,
+    def initialize(client: AgentsClient.new, store: Store.new, pull_requests: PullRequests.new, transcripts: Transcript.new,
       rate_limits: RateLimits.new, reaper: Reaper.disabled, snapshot: Snapshot.disabled, switch: Switch.disabled,
       actions: Actions.disabled, notifier: Notifier.disabled, listen: nil, updater: nil, argv: [], out: $stdout, input: $stdin, color: true,
       trust: Trust.method(:projects), terminal: Terminal.new(out, input), queue: Queue.new, pane_offer: Mod::Offer.disabled,
@@ -67,7 +67,7 @@ module ClaudeInbox
       @renderer = Renderer.new
       @input = Input.new(input, out)
       @queue = queue
-      @poller = Poller.new(client: client, store: store, pull_requests: pull_requests,
+      @poller = Poller.new(client: client, store: store, pull_requests: pull_requests, transcripts: transcripts,
         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
@@ -87,7 +87,7 @@ module ClaudeInbox
     end
 
     def run
-      @booted_at = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+      @booted_at = monotonic
       install_traps
       @terminal.enter
       @poller.start
@@ -264,7 +264,7 @@ module ClaudeInbox
     end
 
     def render
-      t0 = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+      t0 = monotonic
       now = Time.now
       all = @store.sections(now)
       @snapshot.write(all, now)
@@ -283,7 +283,7 @@ module ClaudeInbox
       frame = frame.plain unless @color # --no-color: decided here and nowhere else
       placed(frame)
       @terminal.paint(frame.lines)
-      dt = Process.clock_gettime(Process::CLOCK_MONOTONIC) - t0
+      dt = monotonic - t0
       Debug.log("render #{(dt * 1000).round}ms") if dt > 0.05
     end
 
@@ -309,11 +309,13 @@ module ClaudeInbox
     # already says how to retry.
     def loading_for
       return nil if @last_poll || @error || !@booted_at
-      Process.clock_gettime(Process::CLOCK_MONOTONIC) - @booted_at
+      monotonic - @booted_at
     end
 
     def notice(msg)
       @notice = [msg, Time.now + 4]
     end
+
+    def monotonic = Process.clock_gettime(Process::CLOCK_MONOTONIC)
   end
 end

data/lib/claude_inbox/cli.rb

@@ -59,7 +59,7 @@ module ClaudeInbox
       opts[:color] = options[:color]
       opts[:notifier] = notifier
       return opts if options[:fixture] || writer_lock.take(role: "inbox")
-      opts.merge(store: Store.new(writable: false, relay: Actions.new), reaper: Reaper.disabled,
+      opts.merge(store: Store.new(writable: false, relay: Actions.new, sort: options[:sort]), reaper: Reaper.disabled,
         snapshot: Snapshot.disabled, actions: Actions.disabled, pane_offer: Mod::Offer.disabled, notifier: Notifier.disabled)
     end
 
@@ -113,7 +113,7 @@ module ClaudeInbox
 
     def headless
       abort "claude-inbox: --headless takes no --fixture" if options[:fixture]
-      Headless.new(client: AgentsClient.new, store: Store.new, updater: updater, notifier: notifier).run
+      Headless.new(client: AgentsClient.new, store: Store.new(sort: options[:sort]), updater: updater, notifier: notifier).run
     end
 
     # Only what a check within the day already found: a cold launch never waits
@@ -132,7 +132,7 @@ module ClaudeInbox
       # token phones are paired with nor hold the real inbox's lock.
       dir = File.join(Dir.tmpdir, "claude-inbox-fixture")
       {client: FixtureClient.new(options[:fixture]),
-       store: Store.new(path: nil),
+       store: Store.new(path: nil, sort: options[:sort]),
        pull_requests: PullRequests.new(cache_path: File.join(fixtures, "gh-pr-status-cache.json"), resolved_path: nil, gh: nil),
        listen: options[:listen]&.merge(fixture: true, pairing: Remote::Pairing.new(path: File.join(dir, "listen.json")),
          lock_path: File.join(dir, "listen.lock"), images_dir: File.join(dir, "images"))}
@@ -144,7 +144,7 @@ module ClaudeInbox
     # comes only from here, and only when asked for.
     def live_options
       client = AgentsClient.new
-      store = Store.new
+      store = Store.new(sort: options[:sort])
       {client: client, store: store, reaper: options[:reap] ? Reaper.new(client, store) : Reaper.disabled,
        snapshot: Snapshot.new, switch: Switch.new, actions: Actions.new, pane_offer: Mod::Offer.new,
        listen: options[:listen], updater: updater,
@@ -162,7 +162,7 @@ module ClaudeInbox
     def writer_lock = (@lock ||= WriterLock.new)
 
     def parse(argv)
-      opts = {color: true, update_check: true}
+      opts = {color: true, update_check: true, sort: "default"}
       parser = OptionParser.new(USAGE) do |p|
         p.on("--version", "Print the version") {
           @out.puts VERSION
@@ -176,6 +176,7 @@ module ClaudeInbox
         p.on("--headless", "Keep the pane's snapshot fresh without a screen") { opts[:headless] = true }
         p.on("--reap", "Delete background sessions idle 14 days") { opts[:reap] = true }
         p.on("--no-color", "Draw without color") { opts[:color] = false }
+        p.on("--sort=ORDER", Store::SORTS, "Order every section by activity, newest message first") { |order| opts[:sort] = order }
         p.on("--notify", "Send a desktop notification when a session needs you or finishes") { opts[:notify] = true }
         p.on("--auto-update", "Install a release found today before launching") { opts[:auto_update] = true }
         p.on("--no-update-check", "Never look for a newer release") { opts[:update_check] = false }

data/lib/claude_inbox/dialog.rb

@@ -28,11 +28,11 @@ module ClaudeInbox
         .split("\n")
     end
 
+    private
+
     # How wide the box wants to be; the frame caps it at the terminal.
     def fit = self.class::WIDTH
 
-    private
-
     def key(name) = @theme.cyan_bold(name)
 
     # A key and what it does.
@@ -83,10 +83,6 @@ module ClaudeInbox
 
       def title = " #{kind.to_s.capitalize} pull request "
 
-      # Titles vary, so the box grows to the longest line plus some air,
-      # and never shrinks below a plain dialog.
-      def fit = [WIDTH, lines(Float::INFINITY, nil).map { |l| Text.width(l) }.max + MARGIN].max
-
       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")}"])
@@ -102,11 +98,16 @@ module ClaudeInbox
 
       private
 
+      # Titles vary, so the box grows to the longest line plus some air,
+      # and never shrinks below a plain dialog.
+      def fit = [WIDTH, lines(Float::INFINITY, nil).map { |l| Text.width(l) }.max + MARGIN].max
+
       # The list's badge, then the title. A bare #190 is ambiguous only
       # once the list spans repos.
       def label(pr)
         name = (@prs.map(&:repo).uniq.size > 1) ? pr.qualified : pr.short
-        pr.title ? @theme.pr_badge(pr, name) + @theme.dim(" · ") + pr.title : @theme.pr_badge(pr, name)
+        badge = @theme.pr_badge(pr, name)
+        pr.title ? badge + @theme.dim(" · ") + pr.title : badge
       end
     end
 

data/lib/claude_inbox/headless.rb

@@ -17,7 +17,7 @@ module ClaudeInbox
   class Headless
     RECHECK_EVERY = 60
 
-    def initialize(client:, store:, pull_requests: PullRequests.new, actions: Actions.new, snapshot: Snapshot.new,
+    def initialize(client:, store:, pull_requests: PullRequests.new, transcripts: Transcript.new, actions: Actions.new, snapshot: Snapshot.new,
       lock: WriterLock.new, updater: nil, notifier: Notifier.disabled, recheck_every: RECHECK_EVERY)
       @lock = lock
       @snapshot = snapshot
@@ -25,7 +25,7 @@ module ClaudeInbox
       @recheck_every = recheck_every
       @notice = nil
       @queue = Queue.new
-      @poller = Poller.new(client: client, store: store, pull_requests: pull_requests, reaper: Reaper.disabled,
+      @poller = Poller.new(client: client, store: store, pull_requests: pull_requests, transcripts: transcripts, reaper: Reaper.disabled,
         queue: @queue, snapshot: snapshot, actions: actions, notifier: notifier)
     end
 
@@ -33,8 +33,11 @@ module ClaudeInbox
     def run
       return false unless @lock.take(role: "headless")
       @poller.start
-      Thread.new { watch_for_updates } if @updater
+      updates = Thread.new { watch_for_updates } if @updater
       loop { @queue.pop }
+    ensure
+      @poller.stop
+      updates&.kill
     end
 
     private

data/lib/claude_inbox/images.rb

@@ -42,13 +42,12 @@ module ClaudeInbox
     # two requests saving in the same millisecond apart. Readable by the
     # owner only: a photo can be anything.
     def self.save(bytes, dir:, index:, now: Time.now)
-      data = bytes.b
-      ext = extension(data)
+      ext = extension(bytes)
       raise Unsupported, "not a PNG, JPEG, GIF or WebP image" unless ext
       FileUtils.mkdir_p(dir)
       prune(dir, now)
       path = File.join(dir, "#{now.strftime("%Y%m%d-%H%M%S-%L")}-#{index}-#{SecureRandom.hex(3)}#{ext}")
-      File.open(path, File::WRONLY | File::CREAT | File::EXCL | File::BINARY, 0o600) { |f| f.write(data) }
+      File.open(path, File::WRONLY | File::CREAT | File::EXCL | File::BINARY, 0o600) { |f| f.write(bytes) }
       path
     end
 

data/lib/claude_inbox/job_state.rb

@@ -8,17 +8,9 @@ module ClaudeInbox
   # ~/.claude/jobs/<id>/state.json. The only reader of that file, and the
   # only writer: a new session's color.
   #
-  # `claude agents --json` reports one word, `state`, and "working" covers two
-  # different situations: the agent is thinking, or the agent is idle and
-  # something it started earlier — a watch shell, a sub-agent — is still open.
-  # The file tells them apart: `tempo` is the agent's own pulse, `fan` names
-  # each outstanding piece of work, and `detail` is the session's own status
-  # line, with `needs` naming what it is waiting on while blocked and
-  # `output.result` what it produced once done. It also carries the PR links
-  # the daemon scanned out of the transcript and the transcript's path, which
-  # PullRequests reads through here, the color `/color` set on the session,
-  # which `claude agents --json` drops, and `intent`, the prompt the session
-  # was started from, and `createdAt`, when it was.
+  # `claude agents --json` says "working" both while the agent thinks and
+  # while it sits idle on a shell or sub-agent it started. This file tells
+  # the two apart, and holds what the JSON drops (docs/cli-quirks.md).
   #
   # Never cached. A color changes the moment you type `/color`, so every poll
   # asks the file again.
@@ -40,7 +32,7 @@ module ClaudeInbox
 
     # Each background session with its job file read onto `job_state`, or nil
     # when there is none: interactive sessions have no job file, and neither
-    # does one the daemon has already forgotten. Sessions.load calls this.
+    # does one the daemon has already forgotten.
     def self.enrich(sessions, jobs_dir:)
       sessions.map { |s| s.background? ? s.with(job_state: read(s.id, jobs_dir: jobs_dir)) : s }
     end
@@ -91,7 +83,6 @@ module ClaudeInbox
     # a wake, are the only record of Remote Control being on it.
     def remote_control? = flags.include?("--remote-control")
 
-    # True when the agent has stopped and is only waiting on what it started.
     def waiting_on_work? = agent_idle? && in_flight?
 
     # What the session needs while blocked, what it produced once done,

data/lib/claude_inbox/new_session_form.rb

@@ -12,12 +12,11 @@ require_relative "text_buffer"
 require_relative "theme"
 
 module ClaudeInbox
-  # State and key handling for the "new session" modal. Pure: returns what
-  # happened so the App can act on it. Rendering returns plain lines; the
-  # App wraps them in a box.
+  # The `n` form a new session is composed in. Pure: it says what happened
+  # and the App acts on it.
   class NewSessionForm
     # `value` is a TextBuffer on :text and :multiline fields and the picked
-    # string on :choice ones — `kind` says which.
+    # string on :choice ones.
     Field = Struct.new(:key, :label, :kind, :value, :choices)
 
     # A directory the Directory field offers, shown by its shortest
@@ -42,7 +41,6 @@ module ClaudeInbox
     def initialize(cwd:, dirs: [], home: Dir.home, clipboard: Images.method(:from_clipboard))
       @p = Pastel.new(enabled: true)
       @theme = Theme.new
-      @palette = Palette.new
       @home = home
       @clipboard = clipboard
       @places = dirs.map { |path| Place.new(path, SessionRequest.label(path, dirs), path.sub(/\A#{Regexp.escape(home)}(?=\/|\z)/, "~")) }
@@ -72,11 +70,9 @@ module ClaudeInbox
 
     # => :cancel | :start | :start_and_attach | :changed
     def press(name, raw)
-      # `:start`/`:start_and_attach` leave the form up until the spawn is
-      # known to have worked, so a failure (untrusted directory, `claude`
-      # missing, ...) can hand the composed prompt back instead of losing
-      # it; there is no way to cancel a spawn already under way, so every
-      # key is swallowed until the App reports back.
+      # A start leaves the form up until the spawn has worked, so a failure
+      # can hand the prompt back. A spawn under way can't be cancelled, so
+      # every key is swallowed until the App reports back.
       return :changed if @busy
       return confirm_discard_press(name, raw) if @confirm_discard
       @error = nil
@@ -150,8 +146,7 @@ module ClaudeInbox
       @defaults = Settings.defaults(cwd, home: @home)
     end
 
-    # Full-screen body: a tall prompt editor, then one row per setting with
-    # every choice visible. Exactly `height` lines.
+    # The whole body, exactly `height` lines.
     def screen(width, height)
       inner_w = width - 4
       fixed = 3 + 2 + 1 + (@fields.size - 1) + 1
@@ -190,11 +185,8 @@ module ClaudeInbox
       @theme.hints(keys)
     end
 
-    # The App's `claude --bg` call failed after `:start`/`:start_and_attach`
-    # already handed the form off — an untrusted directory, a stale binary,
-    # a permissions error. The composed prompt is still here, so the form
-    # reopens on it with the reason shown instead of the App just logging
-    # the failure and losing the draft.
+    # The App's `claude --bg` call failed after a start. The form shows why,
+    # over the prompt it kept.
     def submission_failed(message)
       @busy = false
       @error = message
@@ -326,7 +318,7 @@ module ClaudeInbox
         f.choices.map { |c|
           text = (c == DEFAULT) ? default_text(f) : c
           if c == f.value then (on ? @theme.pill(" #{text} ", :cyan) : @theme.cyan_bold(" #{text} "))
-          elsif f.key == :color then @palette.paint(" #{text} ", c)
+          elsif f.key == :color then Palette.paint(" #{text} ", c)
           else @p.dim(" #{text} ")
           end
         }.join(" ")

data/lib/claude_inbox/palette.rb

@@ -9,7 +9,7 @@ module ClaudeInbox
   # Closing with 39 (default foreground) rather than 0 (reset everything)
   # matters: the label is often already bold or italic and a full reset
   # would strip that back off.
-  class Palette
+  module Palette
     ANSI = {
       "red" => 31, "green" => 32, "yellow" => 33,
       "blue" => 34, "purple" => 35, "cyan" => 36
@@ -22,7 +22,7 @@ module ClaudeInbox
 
     RESET = "\e[39m"
 
-    def paint(text, name)
+    def self.paint(text, name)
       seq = SEQUENCES[name]
       seq ? seq + text + RESET : text
     end

data/lib/claude_inbox/poller.rb

@@ -16,12 +16,13 @@ module ClaudeInbox
     INTERVAL = 4
     ACTIONS_INTERVAL = 0.5
 
-    def initialize(client:, store:, pull_requests:, reaper:, queue:, snapshot:, actions:,
+    def initialize(client:, store:, pull_requests:, transcripts:, reaper:, queue:, snapshot:, actions:,
       notifier: Notifier.disabled, interval: INTERVAL, clock: -> { Time.now })
       @client = client
       @clock = clock
       @store = store
       @pull_requests = pull_requests
+      @transcripts = transcripts
       @reaper = reaper
       @queue = queue
       @snapshot = snapshot
@@ -54,7 +55,7 @@ module ClaudeInbox
     # state moved.
     def once
       now = @clock.call
-      sessions = Sessions.load(client: @client, pull_requests: @pull_requests, overrides: @store.pr_overrides)
+      sessions = Sessions.load(client: @client, pull_requests: @pull_requests, transcripts: @transcripts, overrides: @store.pr_overrides)
       doomed = @reaper.due(sessions, now)
       @store.hide(doomed)
       publish(sessions)

data/lib/claude_inbox/pull_requests.rb

@@ -54,25 +54,18 @@ module ClaudeInbox
 
   # Finds the PRs a session is tied to and keeps their state fresh.
   #
-  # Claude Code already does the hard part: the daemon scans each background
-  # session's transcript for links and writes them to the session's job state
-  # file, which JobState reads onto `job_state`; `claude agents --json` does
-  # not expose them.
-  # It is a link scan, so a session that merely mentions a PR gets it too.
-  # The daemon keeps one PR per session, so StackLinks adds the rest of a
-  # stack the session opened.
-  # Interactive sessions have no job file; for those the store's `pr` override
-  # is the only source.
+  # The links come from the daemon's scan of each background session's
+  # transcript, so a session that merely mentions a PR gets it too. The
+  # daemon keeps one PR per session, so StackLinks adds the rest of a stack
+  # the session opened. Interactive sessions have no job file; for those the
+  # store's `pr` override is the only source.
   #
-  # State comes first from our own record of resolved PRs, then from
-  # ~/.claude/gh-pr-status-cache.json (whatever Claude Code last saw), then
-  # from `gh pr view` for PRs that are still open, at most once per
-  # REFRESH_AFTER. A merged or closed PR never changes again, so once gh has
-  # said so, and when, it is written to RESOLVED_PATH and never asked about
-  # again. Claude Code's cache never says when, and says nothing at all about
-  # most of a link scan's PRs: it only covers PRs its own sessions opened.
+  # A merged or closed PR never changes again, so once gh has said so, and
+  # when, it goes in RESOLVED_PATH and gh is never asked about it again.
+  # Claude Code's cache never says when, and only covers PRs its own
+  # sessions opened.
   #
-  # `enrich` never touches gh; it is what stands between `claude agents` and
+  # `enrich` never touches gh, since it stands between `claude agents` and
   # the first frame. `refresh` is the slow half, one network round trip per
   # open PR, and the poller calls it after the list has already gone up.
   class PullRequests
@@ -93,11 +86,10 @@ module ClaudeInbox
       @stacks = StackLinks.new
     end
 
-    # Each session with its `prs` set from what is already known, asking
-    # nobody. The scanned links come off `job_state`, so a session without
-    # one (interactive, or forgotten) has none; Sessions.load sees to the
-    # order. `overrides` maps session key => url for links set by hand; an
-    # override replaces the scanned list.
+    # Each session with its `prs` set from what is already known. The scanned
+    # links come off `job_state`, so Sessions.load reads that first.
+    # `overrides` maps session key => url for links set by hand; an override
+    # replaces the scanned list.
     def enrich(sessions, overrides)
       sessions.map do |s|
         urls = overrides[s.key] ? [overrides[s.key]] : scanned(s.job_state)
@@ -127,7 +119,7 @@ module ClaudeInbox
       end
     end
 
-    # Parse `gh pr view --json` output. Pure so it can be tested.
+    # Parse `gh pr view --json` output.
     def self.parse(url, json)
       h = JSON.parse(json)
       state = (h["state"] == "OPEN" && h["isDraft"]) ? "DRAFT" : h["state"]

data/lib/claude_inbox/reaper.rb

@@ -8,14 +8,13 @@ module ClaudeInbox
   # Reaps sessions that have sat idle past Store::REAP_AFTER.
   #
   # The one thing in the inbox that destroys anything without being asked
-  # first, so the whole of it lives here rather than spread through the poll
-  # loop, along with the append-only log that is the last record a session
-  # ever existed once `claude rm` has taken its transcript.
+  # first, so all of it lives here, along with the append-only log that is
+  # the last record a session ever existed once `claude rm` has taken its
+  # transcript.
   #
-  # Unpushed work is safe by construction. `claude rm` refuses a worktree
-  # holding commits that aren't pushed and reports a --discard-unpushed token
-  # to override it; nothing here ever passes that token, so a refusal is the
-  # end of it. Refusals are logged and retried at most daily.
+  # Unpushed work is safe: `claude rm` refuses it, and nothing here
+  # overrides that (docs/cli-quirks.md). Refusals are logged and retried at
+  # most daily.
   class Reaper
     RETRY_AFTER = 24 * 3600
     DEFAULT_LOG = File.join(Dir.home, ".config", "claude-inbox", "reaped.log")
@@ -27,14 +26,13 @@ module ClaudeInbox
       @enabled = enabled
     end
 
-    # For --fixture runs and tests: selects nothing, deletes nothing, and
-    # needs neither a client nor a store to do it.
+    # For --fixture runs and tests: selects nothing, deletes nothing.
     def self.disabled = new(nil, nil, enabled: false)
 
-    # Reaps everything due, returning the keys it actually deleted so the
-    # caller can drop those rows before they reach the store. Refusals come
-    # back as survivors rather than exceptions: one worktree with unpushed
-    # commits must not stop the rest of the sweep.
+    # Returns the keys it deleted, so the caller can drop those rows before
+    # they reach the store. Refusals come back as survivors rather than
+    # exceptions: one worktree with unpushed commits must not stop the rest
+    # of the sweep.
     #
     # Raises if the log cannot be opened, before anything is deleted. No
     # audit trail, no reaping.