revert: rename retrieved_memory tag back to user_context#278
Closed
revert: rename retrieved_memory tag back to user_context#278
Conversation
notgitika
temporarily deployed
to
auto-approve
GitHub Actions
Inactive
notgitika
temporarily deployed
to
auto-approve
GitHub Actions
Inactive
notgitika
force-pushed
the
revert/retrieved-memory-tag-rename
branch
from
d23dfdd to
7df8315
Compare
notgitika
temporarily deployed
to
auto-approve
GitHub Actions
Inactive
jariy17
approved these changes
Feb 23, 2026
Contributor
jariy17
left a comment
jariy17
left a comment
There was a problem hiding this comment.
LGTM, just check what failed the workflow linter.
Contributor
Author
|
raised another PR makign it configurable: #279 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reverts the
<user_context>→<retrieved_memory>tag rename introduced in #271. This tag rename is a breaking change for users who reference<user_context>in system prompts or parsing logic.The core prefill fix from #271 (inserting memory before the last user message) is not affected — only the tag name is reverted.
A configurable
context_tagparameter is proposed in #277 so users can opt in to a custom tag name without breaking existing setups.Test plan
<retrieved_memory>references replaced with<user_context>in source + tests