fix(files): guide retries after consumed edit intents - #6797
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview A shared Tests cover missing patch metadata, compile failures, thrown errors, and the unchanged no-intent path. Reviewed by Cursor Bugbot for commit fb7b2b8. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Greptile SummaryThe PR improves
Confidence Score: 5/5The PR appears safe to merge with no actionable defects identified. The recovery suffix is applied only after successful intent consumption, pre-consumption guidance remains unchanged, and the lockfile metadata aligns with the workspace manifest.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/copilot/tools/server/files/edit-content.ts | Consistently appends accurate recovery guidance to failures occurring after the one-shot file intent has been consumed. |
| apps/sim/lib/copilot/tools/server/files/edit-content.test.ts | Adds targeted coverage for consumed-intent validation and compilation failures, thrown errors, and the unchanged no-intent response. |
| bun.lock | Refreshes valid Bun lockfile metadata to match the existing terminal-protocol package manifest. |
Reviews (1): Last reviewed commit: "chore(deps): refresh workspace lockfile" | Re-trigger Greptile
851b48a to
fb7b2b8
Compare
Summary
edit_contentconsumes its one-shotworkspace_fileintent before it compiles or applies an edit. When later validation, compilation, or persistence failed, the agent received the immediate error but no instruction that retryingedit_contentalone could never work.Every failure after intent consumption now tells the agent to call
workspace_fileagain before retrying. The existing first-use guidance remains unchanged when no intent was consumed.Companion: simstudioai/mothership#438
Type of Change
Testing
cd apps/sim && bun x vitest run lib/copilot/tools/server/files/edit-content.test.ts— 5 tests passed.Checklist
Screenshots/Videos
Not applicable; this changes server-tool recovery guidance and tests only.