Skip to content

Surface waits and prioritize Workstream Board cards - #6277

Draft
loganj wants to merge 5 commits into
workstream-board/03a-presence-timer-retryfrom
workstream-board/04-waits-sort-polish
Draft

Surface waits and prioritize Workstream Board cards#6277
loganj wants to merge 5 commits into
workstream-board/03a-presence-timer-retryfrom
workstream-board/04-waits-sort-polish

Conversation

@loganj

@loganj loganj commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

🤖

Summary

Workstream coordinators need to see what each card is waiting on and which work deserves attention first without opening every linked pull request. This final slice adds manual and review-derived waits, deterministic owner-first ordering, and compact card polish to the experiment-gated Workstream Board.

This is PR 4/4 in the Workstream Board stack, based on the presence-retry regression prerequisite. The board remains off by default under Settings → Experiments. This PR is draft, and auto-merge is off.

What changes

  • Shows manual waits from the channel canvas together with waits derived from linked pull-request review state.
  • Lets a manual wait override matching derived wording without hiding unrelated waits; removing that manual override restores the derived wait until its underlying condition resolves.
  • Prioritizes cards owned by the authenticated user, then active work, with stable creation/name fallbacks. A display-name lookalike does not receive owner priority.
  • Keeps pull-request subscriptions shared and stable as cards remount, and improves compact wait/status presentation and accessibility labels.

Scope boundary: external canvas revisions refresh after ordinary route remount/refetch, NOT via live external subscription or in-place refresh.

Related issue

None found. This is an experiment-gated product slice.

Testing

  • Exact-head Blox integration validation at a39a72b918d5f290a01bf87e88a8cf604d5c851f: Workstream Board 52/52, presence reconciler 12/12, Desktop 5,090/5,090, typecheck and check passed.
  • Patch-equivalent pre-rebase matrix at 6e918ee1b806febe20e44cff7ce8dc05204e5b59: Desktop 5,011/5,011, focused tests 8/8, keyboard/accessibility 16/16, typecheck, check, production build, E2E build, and diff check passed.
  • Independent implementation review: 9.7/10, no material findings. Independent full-media reviews by Princess Donut, Carl, and Mongo: PASS.

The validated implementation was rebased from 6e918ee1b806febe20e44cff7ce8dc05204e5b59 to a39a72b918d5f290a01bf87e88a8cf604d5c851f. The B4 patch is byte-identical across that SHA migration; the new head was separately integration-validated after the stack rebase.

Demo

▶ Watch the full uninterrupted 369.920-second relay-backed demo — SHA-256 e6406b4e6b9c1db129d899035667bc87609dd0da032c8b9e3abd6b65312148df.

Derived reviewer wait Manual same-key override after remount
Derived pull-request reviewer wait Manual wait replacing matching derived wording
Manual card detail Removing override restores derived wait
Manual wait card detail Derived reviewer wait restored after manual removal
Approval clears derived wait Live card added
Approved pull request with derived wait cleared Fifth live board card added
Live card removed Stable after sustained remounts
Live board card removed and stable four-card order restored Stable board after sustained remount round six

Additional full-board frames: manual publication before remount, manual removal before remount, sustained round one, and healthy terminal UI.

The capture proves owner-first/lookalike-safe ordering, manual override/removal, approval clearing, live card add/remove, and six sustained rounds over 334.424 seconds with bounded memory. All 32/32 mechanical gates and 9/9 complete relay-pipeline readiness gates passed.

Evidence archives:

  • Source capture archive — SHA-256 2d62de4ed70256604eb79dea385c7ce963074f0a763750234bbcab38b91afdda; 696-entry manifest SHA-256 4b9308947d24bbbc2d38d28fab9740bfd9263cd8e11d64affdc1f0a106f6b946.
  • Additive immutable-transcript correction — SHA-256 87ac6ac4db0bc95b03bc0803175ea981417f2dac934691d692177280926fcec9; 13-entry manifest SHA-256 a316dcbaa13ec6d5edb7c34a5b347c1e366f36bc446dba16fc343e0dfc57beda.

The source capture preserved one terminal validator failure caused by treating an explicit no-event rate-limit closure as an event-order violation. The separate correction leaves the source immutable and classifies its 65 relevant requests as 32 ordered event-bearing, 32 successful-empty, and one terminal-closed request, with zero actual ordering violations; its self-tests passed 35/35.

Stack

  • Base: workstream-board/03a-presence-timer-retry
  • Head: workstream-board/04-waits-sort-polish at a39a72b918d5f290a01bf87e88a8cf604d5c851f
  • Position: PR 4/4
  • Discussion: Workstream Board thread

Draft only; no auto-merge.

loganj and others added 5 commits August 18, 2026 19:58
Signed-off-by: loganj <loganj@squareup.com>
Signed-off-by: loganj <loganj@squareup.com>
Signed-off-by: loganj <loganj@squareup.com>
Signed-off-by: loganj <loganj@squareup.com>
Co-authored-by: Amp <amp@ampcode.com>
Signed-off-by: loganj <loganj@squareup.com>
Signed-off-by: loganj <loganj@squareup.com>
Co-authored-by: Amp <amp@ampcode.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant