Skip to content

Add "Update on Write Brief (Beta)" link to AI Assistant sidebar#47947

Merged
aagam-shah merged 12 commits intotrunkfrom
update/deprecate-breve-with-notice
Apr 7, 2026
Merged

Add "Update on Write Brief (Beta)" link to AI Assistant sidebar#47947
aagam-shah merged 12 commits intotrunkfrom
update/deprecate-breve-with-notice

Conversation

@saroshaga
Copy link
Copy Markdown
Contributor

@saroshaga saroshaga commented Apr 6, 2026

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:

Update on Write Brief (Beta) ↗

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

  • pgLAn3-iK-p2
  • p1HpG7-xUC-p2

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 disabled

Testing instructions:

  1. Build the Jetpack plugin (pnpm jetpack build plugins/jetpack)
  2. Open the WordPress block editor (create or edit a post)
  3. Open the Jetpack AI Assistant sidebar panel
  4. Scroll to the bottom of the panel — verify a link reading "Update on Write Brief (Beta)" appears below "AI guidelines"
  5. Click the link and verify it opens https://jetpack.com/support/publish-better-content-with-write-brief-with-ai/
  6. Add add_filter( 'breve_enabled', '__return_true' ); to your theme's functions.php
  7. Reload the editor — verify the link is gone and the full Write Brief UI is shown instead
  8. Verify other AI features (title optimization, featured image, feedback) still work normally

Does 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

…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>
@saroshaga saroshaga marked this pull request as draft April 6, 2026 03:51
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2026

Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.

  • To test on WoA, go to the Plugins menu on a WoA dev site. Click on the "Upload" button and follow the upgrade flow to be able to upload, install, and activate the Jetpack Beta plugin. Once the plugin is active, go to Jetpack > Jetpack Beta, select your plugin (Jetpack), and enable the update/deprecate-breve-with-notice branch.
  • To test on Simple, run the following command on your sandbox:
bin/jetpack-downloader test jetpack update/deprecate-breve-with-notice

Interested in more tips and information?

  • In your local development environment, use the jetpack rsync command to sync your changes to a WoA dev blog.
  • Read more about our development workflow here: PCYsg-eg0-p2
  • Figure out when your changes will be shipped to customers here: PCYsg-eg5-p2

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2026

Thank you for your PR!

When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:

  • ✅ Include a description of your PR changes.
  • ✅ Add a "[Status]" label (In Progress, Needs Review, ...).
  • ✅ Add testing instructions.
  • ✅ Specify whether this PR includes any changes to data or privacy.
  • ✅ Add changelog entries to affected projects

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:

  1. Ensure all required checks appearing at the bottom of this PR are passing.
  2. Make sure to test your changes on all platforms that it applies to. You're responsible for the quality of the code you ship.
  3. You can use GitHub's Reviewers functionality to request a review.
  4. When it's reviewed and merged, you will be pinged in Slack to deploy the changes to WordPress.com simple once the build is done.

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.

@github-actions github-actions Bot added the [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. label Apr 6, 2026
@saroshaga saroshaga changed the base branch from update/disable-breve-via-filter to trunk April 6, 2026 03:52
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>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 ExternalLink to 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.

Comment on lines +237 to +243
{ ! 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>
) }
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
Comment thread projects/plugins/jetpack/changelog/deprecate-breve-with-notice Outdated
Comment thread projects/packages/my-jetpack/changelog/deprecate-breve-with-notice Outdated
- 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>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@saroshaga
Copy link
Copy Markdown
Contributor Author

@claude please review

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 6, 2026

Claude finished @saroshaga's task in 1m 35s —— View job


PR Review

  • Read changed files
  • Review sidebar component logic
  • Review changelog entries
  • Check Copilot's comments and existing review feedback
  • Post review

The PR is clean and focused. Here are my findings:

Hard-coded URL (minor)

index.tsx:239 uses a direct support URL while all other external links in the file use Jetpack's redirect pattern:

// 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. ?source=write-brief-deprecation) keeps links manageable server-side if the destination URL changes, and is consistent with the rest of the file. Fix this →

Missing test coverage

test/index.test.tsx mocks canWriteBriefBeEnabled: () => false, so the deprecation link is never exercised. Two simple cases would cover the new branch:

  1. canWriteBriefBeEnabled=true, isBreveAvailable=false → link renders
  2. canWriteBriefBeEnabled=true, isBreveAvailable=true → link is absent

Fix this →

Condition logic (edge case, low priority)

The condition canWriteBriefBeEnabled() && !isBreveAvailable correctly excludes the remote-disable case. However, getBreveAvailability() also returns false when the AI Assistant block is hidden (hiddenBlocks.includes('jetpack/ai-assistant')), which would surface the deprecation notice in that unrelated situation. Since a user who explicitly hides the AI Assistant block likely won't be looking at this sidebar anyway, this is low priority — but worth keeping in mind if the condition ever needs to be tightened.

Changelog

The entry wording is clear and accurate. No issues.

What looks good

  • The canWriteBriefBeEnabled() guard cleanly avoids showing the notice when remotely disabled — that's the right call.
  • Reusing the existing jetpack-ai-sidebar__external-link class keeps the styling consistent.
  • The change is small, self-contained, and matches the PR description.

· 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>
@saroshaga saroshaga marked this pull request as ready for review April 6, 2026 06:40
@saroshaga saroshaga added [Status] Needs Review This PR is ready for review. and removed [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. [Status] In Progress [Block] AI Assistant labels Apr 6, 2026

{ canWriteBriefBeEnabled() && ! isBreveAvailable && (
<PanelRow className="jetpack-ai-sidebar__external-link">
<ExternalLink href="https://jetpack.com/redirect/?source=write-brief-deprecation">
Copy link
Copy Markdown
Contributor

@kat3samsin kat3samsin Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/?

Copy link
Copy Markdown
Contributor Author

@saroshaga saroshaga Apr 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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>
t-wright
t-wright previously approved these changes Apr 7, 2026
Copy link
Copy Markdown
Contributor

@t-wright t-wright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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>
@saroshaga saroshaga changed the title Add Write Brief deprecation notice link to AI Assistant sidebar Add "Update on Write Brief (Beta)" link to AI Assistant sidebar Apr 7, 2026
@saroshaga saroshaga requested a review from aagam-shah April 7, 2026 08:23
Copy link
Copy Markdown
Contributor

@aagam-shah aagam-shah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@aagam-shah aagam-shah merged commit b96215f into trunk Apr 7, 2026
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Extension] AI Assistant Plugin [Package] My Jetpack [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Status] UI Changes Add this to PRs that change the UI so documentation can be updated.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants