Remove duplicated 'Able to be Merged' headline from the MERGE section (keep the sidebar one) #592

New issue

Closed

opened 2026-09-15 19:33:35 +00:00 by crueber · 1 comment

crueber commented

2026-09-15 19:33:35 +00:00

Owner

What's observed

The PR sidebar renders "Able to be Merged" twice (screenshot): once under MERGEABILITY at the top (the Pull.jsx:1384 sidebar value via mergeabilityDisplay, #588) and once under MERGE (the MergeBox state line at MergeBox.jsx:164, also #588). Keep only the top one.

What's requested

Remove the state headline from MergeBox — the disp() line rendering the mergeabilityDisplay text — so the sidebar MERGEABILITY value is the single status headline. The amber "blocking merge: …" reasons line (MergeBox.jsx:194-197) stays: it is not duplicated and carries remediation detail. When mergeable, the MERGE section shows Strategy + buttons with no status line.

Acceptance criteria

MERGE section shows no Able-to-be-Merged/blocked headline; the sidebar headline is unchanged.
The blocked-reasons line still renders in MergeBox when blocked.
mergeabilityDisplay helper retained for the sidebar (no per-callsite fork); dead MergeBox plumbing (disp memo, zeroChecks prop if now unused) removed cleanly.
#588 test pins updated from MergeBox-usage to MergeBox-absence pins (justified update, not weakening); full suite green.

## What's observed The PR sidebar renders "Able to be Merged" twice (screenshot): once under MERGEABILITY at the top (the Pull.jsx:1384 sidebar value via mergeabilityDisplay, #588) and once under MERGE (the MergeBox state line at MergeBox.jsx:164, also #588). Keep only the top one. ## What's requested Remove the state headline from MergeBox — the disp() line rendering the mergeabilityDisplay text — so the sidebar MERGEABILITY value is the single status headline. The amber "blocking merge: …" reasons line (MergeBox.jsx:194-197) stays: it is not duplicated and carries remediation detail. When mergeable, the MERGE section shows Strategy + buttons with no status line. ## Acceptance criteria - [ ] MERGE section shows no Able-to-be-Merged/blocked headline; the sidebar headline is unchanged. - [ ] The blocked-reasons line still renders in MergeBox when blocked. - [ ] mergeabilityDisplay helper retained for the sidebar (no per-callsite fork); dead MergeBox plumbing (disp memo, zeroChecks prop if now unused) removed cleanly. - [ ] #588 test pins updated from MergeBox-usage to MergeBox-absence pins (justified update, not weakening); full suite green.

crueber added this to the v1 milestone

2026-09-15 19:33:35 +00:00

crueber added the

bug

issues

pull requests

labels

2026-09-15 19:33:35 +00:00

crueber referenced this issue from a commit

2026-09-15 19:36:10 +00:00

web: drop the MergeBox duplicate mergeability headline (Forgejo #592)

crueber referenced this issue

2026-09-15 19:36:11 +00:00

Drop the MergeBox duplicate mergeability headline (Forgejo #592) #593

crueber referenced this issue

2026-09-15 19:37:50 +00:00

Drop the MergeBox duplicate mergeability headline (Forgejo #592) #593

crueber commented

2026-09-15 19:38:03 +00:00

Author

Owner

Fixed by #593 (merged): MergeBox state headline removed — the sidebar MERGEABILITY value is now the single status headline; blocked-reasons line, tooltip, gate, and strategy/buttons untouched; dead disp/zeroChecks plumbing removed; helper retained for the sidebar. Verified: 1486 unit tests green (smoke excluded, pre-existing), vite/esbuild green, independent review APPROVE.

crueber closed this issue

2026-09-15 19:38:03 +00:00

crueber referenced this issue from a commit

2026-09-15 19:38:03 +00:00

Merge pull request 'Drop the MergeBox duplicate mergeability headline (Forgejo #592)' (#593) from fix/issue-592 into main

crueber referenced this issue

2026-09-15 23:20:10 +00:00

Gate client CHANGES_REQUESTED merge-block on required-reviews rule (fixes #612) #615

crueber referenced this issue

2026-09-15 23:44:30 +00:00

Draft PRs: open-as-draft / mark-ready transitions (fixes #613) #616