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

One list of issues and one of pulls: the studio's, for every repository #683

Merged opened this in answer to

Check · Review · Land · Live done — Landed and live in production.

Merged

Closes #627

shipped production is serving it for issue #627 — its issue is already done deploy 2fa0c24

opened this

Closes #627

In progress (steward build team). Description, removals and looks follow.

commented

Description (the pull body is fixed at open; this is it). Closes #627.

One list of issues and one of pulls. The studio's lists (tellsList, changesList in src/web/studio/lists.ts) now answer every repository, app or not, for every viewer, whatever the mode. /issues and /tells are two addresses for the same page, and so are /pulls and /changes. Nothing redirects and no old URL breaks: ?mode=dev and the mode cookie change nothing here.

Path. From a repository's tab: the words (Open, Told, Working, Look, Shipped, Stopped), then the label chips if the repository has labels, then rows. Primary action: open a row. On the issue list a signed-in person also gets one quiet verb, compose ("Tell the crew" on an app, "New issue" otherwise, key c).

Removed

  • Screens: dev's issue list (issuesList, src/web/pages/issues.ts) and dev's pull list (pullsList, verdictsFor, verdictMark, src/web/pages/pulls.ts). The "See dev's list" links on told and changes are gone too. The runs one stays until #629.
  • Redirects removed: /tells and /changes no longer bounce dev viewers or non-app repositories to /issues and /pulls. They answer the page.
  • Lines: src/ is 528 added and 743 deleted. About 230 of the added lines are dev's facet helpers (readQuery, filterBar, summaryRow…), moved unchanged into src/web/pages/browse.ts. Browsing a live origin is the only screen left using them: a repository we do not hold has no crew, runs or production to give a word to. Origins stay (#603).

What dev's list showed or did, and where it lives now

  • Open/Closed/All: these are the words now. An old ?state=closed|all link still answers with what it asked for.
  • Label chips on rows and the label filter: folded in. Rows wear their labels, and the repository's labels are chips under the words (?label=).
  • Search box (?q=) and ?author=: still honoured. Each one in force shows as a chip that clears itself. Search as a box lives at /search (#632).
  • ?assignee=: new, as the issue asks. It is a chip once set. The row already shows who has it.
  • Origin author on imported issues and pulls: folded into the row's meta line.
  • Closed issues with nothing merged: dev listed them, but the studio list dropped them (tellWord has no word for them). Now they read done (shipped) or cancelled (stopped), from the close reason.
  • #number, branch pair, review verdict, comment count: one step deeper, on the issue or change page.
  • "New pull request": a change starts from a branch (Branches → open a pull), which is the only path that knows its head.
  • Open/Closed counts: dropped. The tab already carries the open count.

Phone fix. The six-word bar pushed a 390px page 4px sideways. In .filters it now loses a little padding below 420px, and as a last resort it scrolls inside itself.

Tests (updated, not deleted, wherever the survivor keeps the behaviour): studio (one list in both modes and on non-apps; /tells and /changes never bounce; labels, assignee, author, q, ?state=, closed-as-done; compose only for signed-in people and never on pulls), studio-words (list titles), issues, web (the drawing-once check moved to a comment timeline, since list rows no longer repeat a glyph), web-ui.

Verified. tsc is clean. The touched suites pass, except the studio can change what an app says it is (#230), which fails the same way on main. The full gate's other two reds are also on main: check-list (two tracked tests in neither list) and crew assignment (a macOS /private path in the transcript). Looks at 390 and 1280: app told, plain issues, plain pulls, label filter, shipped. No sideways scroll at either width. The looks are with the steward and were taken against a seeded local forge.

🤖 Generated with Claude Code

commented

Update, rebased on main (81dab6e). #677 put "Report a problem" (for a stranger on a public repository) and "N from outside" (for its writers) on dev's issue list, which this pull deletes. Both now sit on the one list, next to compose, with a test. web-workflows' pull-list test now checks that the review standing is on the change, since list rows no longer carry review badges. Full gate: one red, crew assignment. It is the macOS /private path in the transcript, so it is this machine, not this change. tsc is clean.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs