Skip to content

emrg: mention replace_all in edit tool system prompt entry - #66

Merged
argszero merged 1 commit into
masterfrom
feature/edit-replace-all-prompt
Jul 18, 2026
Merged

emrg: mention replace_all in edit tool system prompt entry#66
argszero merged 1 commit into
masterfrom
feature/edit-replace-all-prompt

Conversation

@argszero

Copy link
Copy Markdown
Owner

The system prompt's Tool Usage section mentions key parameters for read, grep, glob, and bash — but the edit tool entry only said "prefer edit over write." This adds a hint about the replace_all parameter so the LLM is more likely to use it for multi-occurrence replacements.

Change: emrg/server/daemon.py — added "Set replace_all for multiple occurrences" to the edit tool prompt entry.

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle #36

@argszero
argszeroforce-pushed the feature/edit-replace-all-prompt branch from 459c9f1 to 5307310CompareJuly 18, 2026 22:40

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle #37

@argszero
argszeroforce-pushed the feature/edit-replace-all-prompt branch from 5307310 to 9529479CompareJuly 18, 2026 22:41

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle #37 (confirmed)

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle #38

@argszero
argszero merged commit 913e1f2 into masterJul 18, 2026
1 check passed
@argszero
argszero deleted the feature/edit-replace-all-prompt branch July 18, 2026 22:56
Sign up for freeto 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.

1 participant

@argszero