PR #359 follow-ups: LabelShown has no probe reader (wrap gate unarmed) nf-hobp ← Beads

closed priority 2 task unassigned activesession:c

Twinned from https://github.com/cfsh/ninefold/issues/366 by tools/beads/import_github.py's reconcile pass.

Suggestions from ninefold-reviewer's review of #359.

1. RowLabelPennant.LabelShown has no reader — the wrap-vs-ellipsis gate is not armed.

#359 wires the #346 capture-from-draw contract into the row label (games/tonapse/ui/SongGrid.csLabelShown assigned from DrawPennant's out on both the single-line and wrapped paths, wrapped lines joined /). The capture is correct, but nothing in the probe harness can read it, so today it is a write-only property and the wrap can still silently regress to ellipsis with the suite green.

Two independent blockers:

Options, roughly in increasing cost:

A is the recommended default. This was explicitly kept out of #359's scope.

2. Below-gate, noted only: a sound name with a *trailing* hyphen would silently lose the hyphen on the single-line fallback path (shared/WaveletNames.cs only emits adj-noun, so nothing live hits this); and #359's screenshots landed under pr-new/ on the screenshots branch rather than pr<N>/ as CLAUDE.md specifies.

Dependencies

None.

Comments

No comments.

Add a comment