Skip to content

Remove ENABLE_STUDIO_AI feature flag and release studio code command - #3021

Merged
sejas merged 1 commit into
trunkfrom
stu-1445-remove-studio-ai-feature-flag
Apr 9, 2026
Merged

Remove ENABLE_STUDIO_AI feature flag and release studio code command#3021
sejas merged 1 commit into
trunkfrom
stu-1445-remove-studio-ai-feature-flag

Conversation

@sejas

@sejas sejas commented Apr 9, 2026

Copy link
Copy Markdown
Member

Related issues

How AI was used in this PR

Claude removed the feature flag wrapper and associated build config entries. All changes were reviewed before committing.

Proposed Changes

  • Remove the __ENABLE_STUDIO_AI__ feature flag guard from apps/cli/index.ts, making the studio code command always available
  • Remove the __ENABLE_STUDIO_AI__ define from vite.config.base.ts (production) and vite.config.dev.ts (dev)
  • Remove the __ENABLE_STUDIO_AI__ type declaration from globals.d.ts

Testing Instructions

  1. Build the CLI: npm run cli:build:prod
  2. Run node apps/cli/dist/cli/main.mjs --help and verify the command code is in the list
  3. Run node apps/cli/dist/cli/main.mjs code --help and verify the command is registered

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sejas sejas self-assigned this Apr 9, 2026
@sejas sejas changed the title Remove ENABLE_STUDIO_AI feature flag Apr 9, 2026
@sejas
sejas requested a review from a team April 9, 2026 08:17
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing a87ecc7 vs trunk

app-size

Metric trunk a87ecc7 Diff Change
App Size (Mac) 1261.00 MB 1261.14 MB +0.14 MB ⚪ 0.0%

site-editor

Metric trunk a87ecc7 Diff Change
load 1831 ms 1834 ms +3 ms ⚪ 0.0%

site-startup

Metric trunk a87ecc7 Diff Change
siteCreation 8119 ms 8134 ms +15 ms ⚪ 0.0%
siteStartup 4155 ms 4167 ms +12 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

@sejas
sejas enabled auto-merge (squash) April 9, 2026 09:53

@epeicher epeicher left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @sejas! I have tested it, and I see the command is listed and it works as expected. LGTM!

Image
@sejas
sejas merged commit f38bf62 into trunk Apr 9, 2026
12 checks passed
@sejas
sejas deleted the stu-1445-remove-studio-ai-feature-flag branch April 9, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants