Skip to content

Web/CLI: Remove cache_prompt key.#12

Merged
SloMR merged 2 commits into
mainfrom
fix/RemoveFlag
Apr 21, 2026
Merged

Web/CLI: Remove cache_prompt key.#12
SloMR merged 2 commits into
mainfrom
fix/RemoveFlag

Conversation

@SloMR
Copy link
Copy Markdown
Owner

@SloMR SloMR commented Apr 21, 2026

  • cache_prompt key is just used for llama.cpp.

- cache_prompt key is just used for llama.cpp.
@SloMR SloMR self-assigned this Apr 21, 2026
Copilot AI review requested due to automatic review settings April 21, 2026 18:36
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the non-standard cache_prompt request field from both the Web and CLI chat request payloads (it appears to be llama.cpp-specific and is not used elsewhere in the repo).

Changes:

  • Remove cache_prompt from the Web app chat request body.
  • Remove cache_prompt from the CLI chat request body.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
web/src/app/core/translation.service.ts Drops cache_prompt from the JSON body sent by the web translator.
cli/core/batch_runner.py Drops cache_prompt from the JSON body sent by the CLI batch runner.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SloMR SloMR merged commit cf7071e into main Apr 21, 2026
3 checks passed
@SloMR SloMR deleted the fix/RemoveFlag branch April 21, 2026 18:40
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