fix(web): pull request action menus no longer show duplicate dividers

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534

[RakshithBhat03]

Pull request action menus could show two dividers back to back when every permission-gated management action was unavailable. The empty section made the menu look like an action had failed to render.

Dividers around the optional management section now render only when it contains an action, and the merge-method divider only appears when another management action precedes it. This covers every combination of draft and ready-for-review actions, auto-merge, and merge-method choices without changing the surrounding environment, conflict, or close and reopen groups.

Verification

• vp test run apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts apps/web/src/components/pullRequest/PullRequestListFilters.test.tsx — 67 tests passed.
• vp run -F @t3tools/web typecheck
• vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts --report-unused-disable-directives
• Checked in local web and macOS development builds.

Before	After

---

Built by GPT-5.6 in T3 Code through the Codex harness.

---

Note

Low Risk
UI-only menu rendering change with covered logic; no API, auth, or data handling impact.

Overview
Fixes back-to-back menu dividers in the pull request “more actions” menu when draft, auto-merge, and merge-method entries are all hidden by permissions or state.

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to decide whether the optional management block should render at all and whether a separator should appear before the merge-method radio group (only when at least one other management action is visible). PullRequestDetailPanel pulls visibility into named flags and gates the whole management section and inner separator on that layout instead of always inserting a separator for open PRs.

Unit tests exhaust all combinations of the three visibility inputs so empty subsections never leave stray separators.

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

Note

[!NOTE]

Fix duplicate dividers in pull request action menus

Adds resolvePullRequestManagementMenuLayout in pullRequestDetail.logic.ts to compute whether the management menu group and its internal separator should render. In PullRequestDetailPanel, the entire management block is now conditional on at least one action being available, and the separator before merge methods only renders when a management action is also present.

^{Macroscope summarized 1ba4df9.}

[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: afdd0856-0c2b-40ba-9665-1ef683c5198b

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 65562f3

This is a straightforward UI bug fix that prevents duplicate menu dividers. The changes extract existing conditions into variables and add a well-tested helper function to control separator visibility. No functional behavior changes beyond the visual fix.

^{You can customize Macroscope's approvability policy. Learn more.}

[maria-rcks]

superseded by #6534