Add "Update on Write Brief (Beta)" link to AI Assistant sidebar#47947
Add "Update on Write Brief (Beta)" link to AI Assistant sidebar#47947aagam-shah merged 12 commits intotrunkfrom
Conversation
…on 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>
|
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. |
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>
There was a problem hiding this comment.
Pull request overview
Adds a user-facing deprecation notice link in the Jetpack AI Assistant sidebar when Write Brief (Breve) isn’t available, along with changelog fragments documenting the change.
Changes:
- Add a footer
ExternalLinkto the Write Brief deprecation/support doc when Breve is unavailable. - Add a Jetpack plugin changelog fragment describing the deprecation notice behavior.
- Add a My Jetpack package changelog fragment referencing removal of the Write Brief video showcase section.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| projects/plugins/jetpack/extensions/plugins/ai-assistant-plugin/components/ai-assistant-plugin-sidebar/index.tsx | Shows a new footer documentation link when Breve is unavailable. |
| projects/plugins/jetpack/changelog/deprecate-breve-with-notice | Records the sidebar deprecation notice behavior for the Jetpack plugin changelog. |
| projects/packages/my-jetpack/changelog/deprecate-breve-with-notice | Adds a My Jetpack changelog note about the product page’s Write Brief section removal. |
| { ! isBreveAvailable && ( | ||
| <PanelRow className="jetpack-ai-sidebar__external-link"> | ||
| <ExternalLink href="https://jetpack.com/support/publish-better-content-with-write-brief-with-ai/"> | ||
| { __( 'Write Brief (Beta) has been removed from the default experience', 'jetpack' ) } | ||
| </ExternalLink> | ||
| </PanelRow> | ||
| ) } |
There was a problem hiding this comment.
This PR introduces new conditional UI (the Write Brief deprecation footer link) but the existing sidebar test suite doesn’t assert that the link appears when Breve is unavailable and is hidden when Breve is enabled. Adding a couple of focused tests here would prevent regressions in the sidebar footer behavior.
- 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>
|
@claude please review |
|
Claude finished @saroshaga's task in 1m 35s —— View job PR Review
The PR is clean and focused. Here are my findings: Hard-coded URL (minor)
// New link — direct URL
<ExternalLink href="https://jetpack.com/support/publish-better-content-with-write-brief-with-ai/">
// Existing pattern in the same file
<ExternalLink href="https://jetpack.com/redirect/?source=jetpack-ai-feedback">
<ExternalLink href="https://jetpack.com/redirect/?source=ai-guidelines">Using a redirect source (e.g. Missing test coverage
Condition logic (edge case, low priority)The condition ChangelogThe entry wording is clear and accurate. No issues. What looks good
· Branch |
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>
|
|
||
| { canWriteBriefBeEnabled() && ! isBreveAvailable && ( | ||
| <PanelRow className="jetpack-ai-sidebar__external-link"> | ||
| <ExternalLink href="https://jetpack.com/redirect/?source=write-brief-deprecation"> |
There was a problem hiding this comment.
Is there going to be a new documentation for this or should this be https://jetpack.com/support/publish-better-content-with-write-brief-with-ai/?
There was a problem hiding this comment.
We'll need to add this redirect rule. Since this is a temporary link, I've reverted it for now, and added back the actual docs page link which is to be updated soon (work in progress)
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>
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>
Summary
Follow-up to #47914 which disabled Write Brief by default. This adds an informational link to the AI Assistant sidebar so users can learn about the change.
When Breve is disabled (the new default), users see a link at the bottom of the sidebar:
If the feature is re-enabled via
add_filter( 'breve_enabled', '__return_true' ), the full Write Brief UI shows instead and the link is hidden.More Context
Changes
ai-assistant-plugin-sidebar/index.tsx: Add an "Update on Write Brief (Beta)" footer link to the Breve documentation page when the feature is disabledTesting instructions:
pnpm jetpack build plugins/jetpack)add_filter( 'breve_enabled', '__return_true' );to your theme'sfunctions.phpDoes this pull request change what data or activity we track or use?
No. This PR only adds a static link to the sidebar. No new data collection, tracking, or privacy-related changes are introduced.
🤖 Generated with Claude Code