feat: make compaction prompts configurable at runtime - #343
Conversation
|
I’m wondering whether this feature may not be necessary, as many of our users may not be familiar with this type of configuration. It could potentially cause some confusion and require additional explanation and support. For example, I’m considering simplifying the Provider even further: I still think this PR is very valuable, but perhaps we could avoid adding a configuration UI. Instead, if |
Co-authored-by: philau2512 <dplau25122002@gmail.com>
|
Thanks again for the contribution. I’ve prepared a much smaller implementation based on the core idea of this PR: This version: only checks ~/.cursor-byok-v3/prompts/compaction.md; git fetch https://github.com/leookun/cursor-byok.git fix/pr343-minimal-compaction-prompt |
aa07133 to
cfc333d
Compare
Feature Pull Request
User Story
Acceptance Criteria
~/.cursor-byok-v3/rules/compaction.mdfrom the bundled default prompt when the file does not exist./summarize) and automatic compaction reads the current file contents, so saved edits apply without restarting the application.Key Changes
{{FAKE_MODEL_NAME}}./summarizeand automatic context-window compaction now use the same compiled runtime compaction prompt.Demo
Testing
cargo test prompting::compiler::testscargo test --no-runcargo test --test compactioncargo check~/.cursor-byok-v3/rules/compaction.mdopens in the system default editor./summarizeand verify the subsequent model request uses the saved prompt.Impact & Reviewer Notes
/summarizeand automatic compaction use the same runtime prompt.