Skip to content

Annotate still available if agentic features are disabled - #4455

Open
katinthehatsite wants to merge 1 commit into
trunkfrom
fix/annotate-available-with-agentic-off
Open

Annotate still available if agentic features are disabled#4455
katinthehatsite wants to merge 1 commit into
trunkfrom
fix/annotate-available-with-agentic-off

Conversation

@katinthehatsite

Copy link
Copy Markdown
Contributor

Related issues

Fixes STU-2192

How AI was used in this PR

It was used to solve the issue

Proposed Changes

This PR ensures that when the agentic features are off, the annotate option in the built in preview is not available.

Testing Instructions

  • Pull the changes from this branch
  • Start Studio with npm start
  • Disable Agentic features in Settings > AI
  • Confirm that the annotate option is no longer available in the preview

With agentic features:

Screenshot 2026-08-05 at 2 56 24 PM

Without agentic features:

Screenshot 2026-08-05 at 2 56 01 PM

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@katinthehatsite katinthehatsite self-assigned this Aug 5, 2026
@katinthehatsite
katinthehatsite requested a review from a team August 5, 2026 12:57
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 6c38c42 vs trunk

app-size

Metric trunk 6c38c42 Diff Change
App Size (Mac) 1402.86 MB 1402.84 MB 0.02 MB ⚪ 0.0%

site-editor

Metric trunk 6c38c42 Diff Change
load 1074 ms 1062 ms 12 ms ⚪ 0.0%

site-startup

Metric trunk 6c38c42 Diff Change
siteCreation 7538 ms 7522 ms 16 ms ⚪ 0.0%
siteStartup 3393 ms 3352 ms 41 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@sejas sejas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch! It makes sense to disable the annotate for this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants