fix(web): label pull request merge actions

[tarik02]

What Changed

• Label the primary and confirmation merge buttons from the selected merge method: Merge, Squash, or Rebase.
• Keep the confirmation action intact until the dialog's closing transition completes, then reset it.

Why

Selecting Squash or Rebase still left the primary and confirmation buttons labeled Merge, even though the selected method would be used.

Dismissing the confirmation dialog also reset its content as the closing transition began, causing parts of the dialog to disappear mid-animation.

UI Changes

Before

before.webm

After

after.webm

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Built with GPT-5.6-Sol using Codex in T3 Code.

---

Note

Low Risk
Localized UI and dialog state in the pull request detail panel; merge behavior still uses the existing perform path with selectedMergeMethod.

Overview
Merge controls in PullRequestDetailPanel now reflect the selected merge strategy (Merge, Squash, or Rebase) on the primary action button, the merge-method menu radios, and the confirmation dialog’s confirm label—instead of always saying Merge while selectedMergeMethod is passed to perform.

Confirmation flow is refactored from a nullable confirmAction to confirmation: { open, action }. onOpenChangeComplete clears the stored action only after the dialog close animation finishes, so title/copy/button text no longer flicker or vanish mid-dismiss.

^{Reviewed by Cursor Bugbot for commit d27d496. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Label pull request merge actions with human-readable method names

Adds a MERGE_METHOD_LABELS mapping in PullRequestDetailPanel.tsx to display "Merge", "Squash", or "Rebase" instead of raw method identifiers. The primary merge button, merge method radio options, and confirmation dialog button all now reflect the selected label. The confirmation state is refactored from a nullable string to an object with an explicit open flag, so the action type is preserved during the dialog close animation.

^{Macroscope summarized d27d496.}

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 005dea97-1772-4ca3-bf6b-66e320fcf0d3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Approved at d27d496

Macroscope's review found this PR approvable — This PR adds proper display labels for merge method buttons (showing 'Squash'/'Rebase' instead of always 'Merge') and refactors dialog state to prevent UI flickering during close animations. The changes are purely cosmetic and don't affect merge behavior.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate d27d496