Update main from bpsa#58
Merged
Merged
Conversation
…ore last message Previously, knowledge was inserted as a USER message just before the last message, which disrupted the natural conversation flow. Now it is injected as a SYSTEM message immediately after the system prompt, treating it as foundational background context. Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
…d of before last message" This reverts commit b57b106.
The knowledge block is still injected just before the last message, but now uses SYSTEM role instead of USER role, better signalling it as background context rather than a conversational turn. Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
New context order: 1. All memory steps (TaskStep, PlanningStep, ActionStep) 2. System prompt (SYSTEM role) 3. Knowledge (USER message, just before last message) 4. Last message Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
The knowledge role was changed to SYSTEM in a previous commit but was inadvertently reverted to USER during the context reordering. Now restored. Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
…ence note, fix merge output format - Remove 'To your own benefit' phrase (unnecessary justification) - Add explicit note that original steps are permanently removed after compression, so the compressor writes more carefully - Add structured <summary>/<knowledge_updates> output format to merge prompt, consistent with the main compression prompt Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
- Add MCP server integration feature bullet to README.md features list - Add --mcp usage examples to README.md CLI usage block - Add new MCP Server Integration section to CLI.md - Add --mcp usage examples to CLI.md quick-start block Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
- Read BPSA_MCP env var (newline-separated) and merge with --mcp CLI flags - Document BPSA_MCP in CLI.md env vars table - Document BPSA_MCP usage in MCP Server Integration section Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
Co-Authored-By: bpsa2 <241537330+bpsa2@users.noreply.github.com>
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
1 participant
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.
No description provided.