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

A run's summary is one escaped, clamped line in every row that shows it, landed from the work in pull #9964 #9979

Closed opened this
opened this

Replaces #73. Its work is written. Pulls #9946 and #9964 both went red only on tests they never touched (traffic-truth, fleet-ops, then a shard killed in week-pump). This issue lands that work once the flaky tests are fixed.

Done when:

  • [ ] A run row's note, an ask's says line and a decision-ledger entry each show the summary as one line: newlines and runs of spaces collapsed, escaped, never rendered as markdown, clamped by CSS (row-note), with the whole text in title. Who, the word and the time keep their place.
  • [ ] Any server-side cap is generous and named (#9964 used NOTE_MAX = 2000).
  • [ ] A test renders a run with a 600-character multi-line summary containing backticks, * and a link: one line, full text in title, the markdown characters literal. It fails on main and passes with the change.
  • [ ] bun scripts/design-lint.ts and bun test are green; the new class is named in docs/design/areas/<area>.md.

Files: src/web/studio/note.ts (new, as in #9964), the studio rows that show a run's summary, the CSS, the test.

Start from: pull #9964's diff; read it, rebase it on main, and don't rewrite it.

commented

Replaced by #10199: not on main; re-planned as ready work under P0 now that its parents were dropped.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs