#@teknium Hi there, great product, thx
1 messages · Page 1 of 1 (latest)
yes can you list the PRs here
first three, reviewed and ok to launch
-
#7250 no harm, and I agent+human reviewed.
-
#7762 lgtm, key issue is openviking actually requires session.commit to make memory extraction
-
#9150 this happens to setup, screenshot for reproducing issues in PR desc.
draft (pending preview merge and rebase): #9167 planning to rebase all prev commits then resposible for ut & e2e test further.
#9167 should be able to include feat on #9103 #5764 #8707 #9104 and closes them.
the rest of PRs: #8362 #5775 #5780 #8474 #8235 are not urgent, i will review in future when prev 4 is merged.
What does this PR do?
This PR fixes result ordering in the OpenViking memory provider's _tool_search() path.
Before this change, results were grouped by bucket and concatenated in implicit ...
Fixes #7759
Problem
The /new and /reset commands were not calling shutdown_memory_provider() on the cached agent before eviction. This caused OpenViking (and any memory provider that relies on sess...
bug fixes on setup:
Non-secret fields with env_var (like OPENVIKING_ENDPOINT) were never written to .env - Fixed by adding them to env_writes
Status output only showed secret fields as missing - F...
merged all that are not drafts!
added this few-lines fix #9807, extracted from #9167 with clear reproduce steps
pls help to review. After this core features of openviking would work!
test failed, irrelevant
hi @mild aurora pls help to review the fix. #9807 It’s trivial and easy to reproduce. Thx