Repository navigation
Studio Code: Point to the remote-site skill from the active-site line instead of the system prompt - #5191
Studio Code: Point to the remote-site skill from the active-site line instead of the system prompt#5191chubes4 wants to merge 1 commit into
Conversation
…ive-site line instead of the system prompt
|
Closing. The eval in the description shows the trunk prompt line is what makes the agent load Claude (via the Pi coding agent in a Roadie session) ran the evals and wrote this note. |
📊 Performance Test ResultsComparing a3edfc3 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
Related issues
How AI was used in this PR
Claude (via the Pi coding agent in a Roadie session) found the rule, ran an eval showing a plain deletion regresses, wrote the replacement and its tests, and ran the checks and evals below.
Proposed Changes
edit-manage-remote-sitefirst for any WordPress.com site. That line is removed.[Active site: "X" (ID: 123) at https://… (WordPress.com, live: load the edit-manage-remote-site skill before working on it)]Testing Instructions
npx vitest run apps/cli/ai apps/cli/commands/ai: the newsite-selection.test.tschecks both active-site lines; the prompt test checks the skill's description carries the routing and the prompt no longer does.edit-manage-remote-sitebefore its firstwpcom_request, makes only GET requests, and touches no local site.wpcom_requestwithout the skill. Answers were still right for these read-only questions, but a change would have skipped the skill's plan check, size limits and verification.Pre-merge Checklist