Added hide comments checkbox to disable commenting dialog

[rob-ghost]

Summary

When disabling a member from commenting, staff may also want to hide all the member's previous comments. This adds an optional "Hide all previous comments" checkbox to the disable commenting confirmation dialog.

The checkbox is gated behind a separate disableMemberCommentingHideComments labs flag since the backend support for the hide_comments API parameter will be deployed separately from the frontend.

ref https://linear.app/ghost/issue/BER-3184

Requires #25906

---

Note

Medium Risk
Moderate risk because it changes comment-moderation behavior and request payloads (new hide_comments parameter) behind a feature flag; failures could incorrectly hide or not hide comments when disabling members.

Overview
Adds a labs-gated “Hide all previous comments” checkbox to the disable-commenting confirmation dialog in Comments; when enabled it passes a new hide_comments parameter when calling POST /members/:id/commenting/disable.

Introduces the new disableMemberCommentingHideComments labs flag (surfaced in Labs settings), threads hideCommentsEnabled through the comments UI, and extends the members API hook/types to support the optional hideComments field. Updates Playwright page objects and adds e2e coverage for checkbox visibility and hide-vs-keep behavior.

^{Written by Cursor Bugbot for commit b091c8b. This will update automatically on new commits. Configure here.}

Summary by CodeRabbit

Release Notes

• New Features
  • Added member commenting management—disable or enable commenting for individual members.
  • Added optional "Hide all previous comments" feature when disabling member commenting.
  • New private lab feature "Hide Comments When Disabling" to control the hide comments option.

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request adds functionality to hide all previous comments when disabling a member's commenting ability. It introduces a new lab feature flag to gate the "Hide all previous comments" UI option, updates the member API with related mutations, and adds corresponding test coverage.

Changes

Cohort / File(s)	Summary
API Mutations apps/admin-x-framework/src/api/members.ts	Added commenting field to Member type with disable metadata. Expanded useDisableMemberCommenting to accept optional hideComments parameter. Added new useEnableMemberCommenting mutation for re-enabling member commenting.
Comments UI Components apps/posts/src/views/comments/comments.tsx, apps/posts/src/views/comments/components/comments-list.tsx	Introduced hideCommentsEnabled flag from lab config and passed to CommentsList. Added internal state and checkbox UI to CommentsList for "Hide all previous comments" option, gated by feature flag. Updated mutation payload to include hide flag when applicable.
E2E Test Support e2e/helpers/pages/admin/comments/comments-page.ts	Added hideCommentsCheckbox locator property to CommentsPage class for test interaction.
E2E Test Coverage e2e/tests/admin/comments/disable-commenting.test.ts	Added new test group validating hide comments checkbox visibility and behavior when feature is enabled, including scenarios with and without the checkbox selected.
Feature Flag Configuration apps/admin-x-settings/src/components/settings/advanced/labs/private-features.tsx	Registered new "Hide Comments When Disabling" lab feature flag (disableMemberCommentingHideComments) in private features list.

Poem

🐰 A hop, a skip, to hide the words,
When members speak no more,
Their comments vanish, clean as birds,
Behind a lab-flag door!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: adding a hide comments checkbox to the disable commenting dialog, which is the central feature of this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/comment-moderation-disable-comments-ui-hide-comments

---

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 and usage tips.}

[github-actions]

React E2E Tests Failed

To view the Playwright test report locally, run:

REPORT_DIR=$(mktemp -d) && gh run download 21171924876 -n playwright-report-react -D "$REPORT_DIR" && npx playwright show-report "$REPORT_DIR"

[github-actions]

E2E Tests Failed

To view the Playwright test report locally, run:

REPORT_DIR=$(mktemp -d) && gh run download 21593990831 -n playwright-report -D "$REPORT_DIR" && npx playwright show-report "$REPORT_DIR"

[github-actions]

E2E Tests Failed

To view the Playwright test report locally, run:

REPORT_DIR=$(mktemp -d) && gh run download 21593990831 -n playwright-report -D "$REPORT_DIR" && npx playwright show-report "$REPORT_DIR"

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@e2e/tests/admin/comments/disable-commenting.test.ts`:
- Around line 98-165: Update the three test titles in disable-commenting.test.ts
to follow the required "what is tested - expected outcome" lowercase format with
a " - " separator by changing the first argument strings passed to test():
rename 'hide comments checkbox appears in modal' to e.g. 'disable commenting
modal - shows hide comments checkbox', 'disabling with hide comments checked
marks comments as hidden' to 'disable commenting action - marks comments hidden
when hide checked', and 'disabling without hide comments checked keeps comments
visible' to 'disable commenting action - keeps comments visible when hide
unchecked'; leave test bodies and references to CommentsPage methods
(getCommentRowByText, openMoreMenu, clickDisableCommenting,
confirmDisableCommenting, hideCommentsCheckbox, commentingDisabledIndicator)
unchanged.

[weylandswart]

Looks good to me and works.

I thought the checkbox should probably use the same checkbox styles we use elsewhere (I found an example under Portal settings) but it looks like that comes from the old Admin X design system. Shade doesn't seem to have a checkbox for now. I'll make a note for design to add it.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}