fix(workspace): keep unmatched focus labels from selling a no-op - #882
fix(workspace): keep unmatched focus labels from selling a no-op#882cursor[bot] wants to merge 8 commits into
Conversation
Replace the stub rehearsal-priorities card with role-and-section pairs so a player can see what to lock in before the room starts. Prefer high priority, then medium, then focus sections, and never turn blank or none sentinels into instructions. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Repeated verse labels no longer consume the third rehearsal-priority slot, and empty copy now points at the section roadmap instead of a no-op role click. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Clicking a named role-and-section pair selects that role and focuses the matching section so players can jump from the priority card into the part they need to lock. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Fallback focus labels now jump to the first matching section so players can still act when analysis only names a section, not a role pair. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Omit focus labels that do not match a roadmap card so a missing bridge cannot clear verse focus. Scroll the named section into view when a lock-in pair or fallback label is opened. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
There was a problem hiding this comment.
Stale comment
Review at
43cb9438This draft is the same tree as #880
a0c2e612(1e75a423). #880 is already ondevelopand ready for review.Do not merge #882. Landing vehicle is #880. Keep this draft closed or idle so the residual-fix slice has one PR.
The unmatched-label omit,
nonefirst-section walk, and roadmapscrollIntoView/aria-currentbehavior is sound. Distinct from #776. Do not mix stem-lane work onto this slice.Sent by Cursor Automation: Fix Issues
There was a problem hiding this comment.
Defer to #880
#880 is now retargeted to develop@acdbea63 at a0c2e612. That is the lock-in landing vehicle.
This draft (43cb9438) is the same residual opened while #880 still showed main. Keep it draft and unmerged. Do not land it in parallel with #880 or #874. Do not open a third residual-fix PR. Do not mix stem-lane work here.
This comment is not approval.
Sent by Cursor Automation: Fix Issues


Closed as proven duplicate of canonical #874
Exact evidence before closure:
43cb9438d6338890a218f2d3b7c4ba74199adb00is a direct child of canonical feat(workspace): name the parts to lock in first #874 predecessorc646037d390ed92dfe12c4416e8eb9874cc03b6b.nonefallback labels are skipped, and the matched roadmap card is focused/scrolled witharia-current.cursor/bc-587d0b33-7e84-4562-916c-0929fcedc42e-ae0cwas fast-forwarded without force to this exact43cb9438…head and refetched successfully.No predecessor or duplicate PR checks/reviews transfer. #874 must regenerate exact-current-head CI/coverage/security/SBOM/supply-chain/review evidence on
43cb9438…and remain gated by #783/#861/central prerequisites plus independent approval.