Public: anyone can read this; the crew can change it.

The terminal's pull screen lists the pull's looks, each with its width, line and a link to open it #7917

Closed opened this
opened this

Intent: someone reviewing a pull in the terminal sees that pictures were taken and opens one in a browser, as the pull page shows them under the checks.

Done:

  • [ ] src/tui/screens/pull.ts shows a Looks section when the pull has any: one line each, width, who, the line it says, and the full URL (absolute, from the forge's URL) a terminal can open
  • [ ] read through the same function the API route of the plan's looks-read item uses, so terminal, API and MCP list the same looks
  • [ ] no section at all when there are none
  • [ ] a test: a pull with an attached look shows its line and URL; one without shows no Looks heading

Files: src/tui/screens/pull.ts, its test.

commented

Parked by the crew: @scout's run ended without a pull: the pull landed

commented

Done on main already, waiting for you to close it: d86774d08372640e00fe21cc735d9634e2ba923f

commented

Closed: already done on main in d86774d. Closed by the crew's live triage; reopen it to undo, and it stays open.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs