Disable Write Brief (Breve) proofreading feature by default#47914
Disable Write Brief (Breve) proofreading feature by default#47914
Conversation
Flip the breve_enabled filter default from true to false, which hides the feature for all users without removing any code. The feature can be re-enabled by setting the filter to true. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! Jetpack plugin: No scheduled milestone found for this plugin. If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack. |
There was a problem hiding this comment.
Pull request overview
This PR disables the Write Brief (Breve) proofreading feature by default (while keeping the code intact), making it opt-in via the breve_enabled filter. This is a reversible alternative to fully removing the feature.
Changes:
- Flip the default for the
breve_enabledfilter fromtruetofalsewhen registering the Breve extension (still gated to English locales). - Add a Jetpack changelog fragment documenting the behavior change.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| projects/plugins/jetpack/extensions/blocks/ai-assistant/ai-assistant.php | Defaults Breve availability to off by setting apply_filters( 'breve_enabled', false ) (still only for en* locales). |
| projects/plugins/jetpack/changelog/disable-breve-via-filter | Adds a changelog fragment describing the default-disable behavior and how to re-enable. |
| @@ -0,0 +1,5 @@ | |||
| Significance: minor | |||
| Type: major | |||
| Comment: Disable the Write Brief (Breve) proofreading feature by default. It can be re-enabled via the 'breve_enabled' filter. | |||
There was a problem hiding this comment.
The Comment: header in Jetpack changelog fragments is typically used to explain why no changelog entry text is included. This fragment includes both a Comment: header and a non-empty entry, which may cause the changelogger tooling to treat the entry unexpectedly. Consider removing the Comment: header (or alternatively, remove the entry text if the intent is to skip the changelog).
| Comment: Disable the Write Brief (Breve) proofreading feature by default. It can be re-enabled via the 'breve_enabled' filter. |
Code Coverage SummaryCoverage changed in 1 file.
|
The Comment header in Jetpack changelog fragments is meant for entries with no changelog text. Move the filter re-enabling note into the actual changelog entry instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The Breve feature is now disabled by default, so the product page should no longer showcase it unconditionally. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
done it 21089da |
|
@claude - please review this PR |
|
Claude finished @aagam-shah's task in 1m 42s —— View job PR Review
Overall, this is a clean, well-scoped PR. The approach of flipping the filter default is the right call for a safe, reversible disable. Here's a breakdown of each change:
|
keoshi
left a comment
There was a problem hiding this comment.
LGTM! Kudos to @aagam-shah for spotting Breve's mention in the AI LP.
CGastrell
left a comment
There was a problem hiding this comment.
Thanks! Works as described! ![]()
- Tighten deprecation link condition to also check canWriteBriefBeEnabled() so notice only shows when the feature could be enabled but isn't - Update changelog wording to describe conditional behavior - Remove stale my-jetpack changelog (product page change was in #47914) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Deprecate Write Brief (Breve) with deprecation notice and documentation link Instead of silently hiding the feature, show a deprecation notice with a link to documentation in the AI Assistant sidebar. The feature can still be re-enabled via the 'breve_enabled' filter. - Flip breve_enabled filter default from true to false - Show "Write Brief" heading with deprecation notice when disabled - Remove Breve video card from My Jetpack AI product page Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update deprecation notice wording Change from "This feature has been deprecated" to "Write Brief has been removed from the default experience". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Keep (Beta) label on Write Brief heading Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update changelog type to major for breve deprecation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Move deprecation notice to footer link Instead of a full sidebar section, show "Write Brief (Deprecated)" as a link alongside the other footer links when the feature is disabled. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update deprecation notice to include full context Show "Write Brief (Beta) has been removed from the default experience. Learn more" with the link inline. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Simplify deprecation notice to single linked text Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Address review feedback - Tighten deprecation link condition to also check canWriteBriefBeEnabled() so notice only shows when the feature could be enabled but isn't - Update changelog wording to describe conditional behavior - Remove stale my-jetpack changelog (product page change was in #47914) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Use redirect pattern for deprecation link URL Match the existing pattern used by other external links in the sidebar (jetpack.com/redirect/?source=...) instead of a hard-coded URL. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Revert to direct URL for deprecation link The redirect pattern (jetpack.com/redirect/?source=...) requires server-side registration on the Jetpack redirect service, which is outside this monorepo. Use the direct support page URL instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update deprecation notice text to "Update on Write Brief (Beta)" Use neutral, universal wording that works for all users including WPCOM Simple site users who cannot re-enable via filters. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Summary
breve_enabledfilter default fromtruetofalsein the AI Assistant PHP registrationadd_filter( 'breve_enabled', '__return_true' );More Context
Testing instructions:
Does this pull request change what data or activity we track or use?
No. This PR only changes the default value of an existing filter. No new data collection, tracking, or privacy-related changes are introduced.
🤖 Generated with Claude Code