Skip to content

Conversation

@Tarquinen
Copy link
Collaborator

Summary

  • Refactor prunable-tools injection to use assistant role instead of system role for better model compatibility
  • Add reasoning model support for synthetic message injection
  • Update prompts to use first-person language appropriate for assistant messages

Switch from user role to assistant role message for injecting
prunable-tools context. This reduces the likelihood of the model
directly addressing the injected content as if responding to
user input, improving the conversational experience.
Reasoning models expect their encrypted reasoning parts in assistant messages,
which we cannot generate. This adds detection via chat.params hook and appends
a synthetic user message (<system-context-injection/>) to close the assistant
turn properly when a reasoning model is active.
…nt messages

- Nudge prompts now use 'I must/I will' instead of 'You must/You should'
- Prunable tools wrapper and cooldown message use first-person
- System prompts updated to correctly describe injection as assistant message
…olete injected_context_handling instructions
@Tarquinen Tarquinen merged commit 8f8a5c6 into dev Dec 23, 2025
1 check passed
@Tarquinen Tarquinen deleted the feat/assistant-role-injection branch December 23, 2025 04:16
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