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>

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