Skip to content

Studio Code: Keep the agent's identity in SOUL.md - #5257

Closed
chubes4 wants to merge 1 commit into
trunkfrom
studio-code/soul-md
Closed

chubes4 wants to merge 1 commit into
trunkfrom
studio-code/soul-md

Conversation

@chubes4

@chubes4 chubes4 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Related issues

  • Continues the Studio Code system prompt work.

How AI was used in this PR

Claude (via the Pi coding agent in a Roadie session) wrote the change. I reviewed the diff.

Proposed Changes

  • Moved Studio Code's identity from a constant in system-prompt.ts into apps/cli/ai/SOUL.md.

Closed in favor of #5260.

Studio Code's identity moves out of system-prompt.ts into apps/cli/ai/SOUL.md,
shipped next to the bundle like the skills and read when the prompt is built.
The prompt is byte-identical. A test pins the md5 of its first 423 bytes, which
the WordPress.com AI proxy allow-lists, so a change to the opening fails in
Studio before the proxy refuses it.
@chubes4
chubes4 marked this pull request as draft October 21, 2026 12:53
@chubes4

chubes4 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor Author

Closing: a separate SOUL.md file adds a mechanism without simplifying the prompt. Studio Code becomes extensible through pi extensions instead (#5260).

Comment drafted by Claude (via the Pi coding agent in a Roadie session); reviewed by me.

@chubes4 chubes4 closed this Oct 21, 2026
@chubes4
chubes4 deleted the studio-code/soul-md branch October 21, 2026 13:04
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 3d4763d vs trunk

app-size

Metric trunk 3d4763d Diff Change
App Size (Mac) 944.37 MB 944.37 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 3d4763d Diff Change
load 1211 ms 1213 ms +2 ms ⚪ 0.0%

site-startup

Metric trunk 3d4763d Diff Change
siteCreation 7061 ms 7066 ms +5 ms ⚪ 0.0%
siteStartup 4370 ms 4372 ms +2 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

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.

2 participants