Skip to content

cli: Don't clear system prompt when using '/clear'#20067

Merged
danbev merged 2 commits intoggml-org:masterfrom
roj234:dont-clear-system-prompt
Mar 6, 2026
Merged

cli: Don't clear system prompt when using '/clear'#20067
danbev merged 2 commits intoggml-org:masterfrom
roj234:dont-clear-system-prompt

Conversation

@roj234
Copy link
Copy Markdown
Contributor

@roj234 roj234 commented Mar 3, 2026

Add system prompt to messages when clearing chat history.

Add system prompt to messages when clearing chat history.
@roj234 roj234 requested a review from ngxson as a code owner March 3, 2026 11:30
@roj234
Copy link
Copy Markdown
Contributor Author

roj234 commented Mar 3, 2026

Currently system prompt is specified by command line argument, so /clear remove it is pretty weird

Comment thread tools/cli/cli.cpp Outdated
@danbev danbev merged commit f7db3f3 into ggml-org:master Mar 6, 2026
78 checks passed
@roj234 roj234 deleted the dont-clear-system-prompt branch March 7, 2026 19:12
bartowski1182 pushed a commit to bartowski1182/llama.cpp that referenced this pull request Mar 10, 2026
* Enhance /clear command to include system prompt

Add system prompt to messages when clearing chat history.

* Use lambda
Ethan-a2 pushed a commit to Ethan-a2/llama.cpp that referenced this pull request Mar 20, 2026
* Enhance /clear command to include system prompt

Add system prompt to messages when clearing chat history.

* Use lambda
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
* Enhance /clear command to include system prompt

Add system prompt to messages when clearing chat history.

* Use lambda
rsenthilkumar6 pushed a commit to rsenthilkumar6/llama.cpp that referenced this pull request May 1, 2026
* Enhance /clear command to include system prompt

Add system prompt to messages when clearing chat history.

* Use lambda
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants