Skip to content

Update STUDIO.md to CLI site start - #2889

Merged
sejas merged 1 commit into
trunkfrom
update/keep-updated-only-studio-md-when-site-starts
Mar 24, 2026
Merged

Update STUDIO.md to CLI site start#2889
sejas merged 1 commit into
trunkfrom
update/keep-updated-only-studio-md-when-site-starts

Conversation

@sejas

@sejas sejas commented Mar 24, 2026

Copy link
Copy Markdown
Member

Related issues

  • Related to the AI instructions feature flag removal effort

How AI was used in this PR

Claude Code assisted with identifying where updateManagedInstructionFiles was called and generating the code changes. All changes were reviewed manually.

Proposed Changes

  • Move updateManagedInstructionFiles from Electron main process (ipc-handlers.ts) to CLI site start command (start.ts), so CLI-managed sites also get updated STUDIO.md on every start
  • Remove CLAUDE.md from managed instruction files — only STUDIO.md is now auto-updated on site start (CLAUDE.md is user-editable)
  • Update tests to reflect the narrower managed files list

Testing Instructions

  • Build the CLI: npm run cli:build
  • Create a site: node apps/cli/dist/cli/main.js site create --path /tmp/test-site
  • Verify STUDIO.md exists in the site root
  • Modify STUDIO.md content manually
  • Stop and restart the site: node apps/cli/dist/cli/main.js site start --path /tmp/test-site
  • Modify manually STUDIO.md, keep the file but update its content.
  • Verify STUDIO.md was restored to the bundled version

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@sejas sejas self-assigned this Mar 24, 2026
@sejas
sejas requested a review from a team March 24, 2026 14:32
@sejas sejas changed the title Move AI instructions update to CLI site start Mar 24, 2026
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing a0702e5 vs trunk

app-size

Metric trunk a0702e5 Diff Change
App Size (Mac) 1246.65 MB 1246.65 MB 0.00 MB ⚪ 0.0%

site-editor

Metric trunk a0702e5 Diff Change
load 1943 ms 1928 ms 15 ms ⚪ 0.0%

site-startup

Metric trunk a0702e5 Diff Change
siteCreation 8170 ms 8177 ms +7 ms ⚪ 0.0%
siteStartup 4815 ms 4143 ms 672 ms 🟢 -14.0%

Results are median values from multiple test runs.

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

@fredrikekelund fredrikekelund 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.

Works as expected 👍

@sejas
sejas merged commit ca82284 into trunk Mar 24, 2026
12 of 13 checks passed
@sejas
sejas deleted the update/keep-updated-only-studio-md-when-site-starts branch March 24, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants