test(memory): establish semantic retrieval no-go baseline - #1304
Merged
XingYu-Zhong merged 7 commits intoSep 9, 2026
Merged
Conversation
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
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
This PR intentionally does not change production Memory retrieval. SQLite FTS5 and filesystem fallback remain the supported path.
Why a no-go result is useful
The decision thresholds were frozen before holdout execution. The selected candidate improved holdout Recall@5 from 0.500 to 0.625 (+0.125), below the required +0.150, and holdout MRR stayed at 0.500, below the required +0.100 improvement. Lowering the thresholds or tuning against the revealed holdout would invalidate the experiment, so P2-B is not proposed from this candidate.
Changes
npm --prefix kun run eval:memory-retrievalcommand.Frozen candidate and result
multilingual-e5-small-q8-hybrid-0.8-1761b726dd34fb83930e26aab4e9ac3899aa1fa78f80102d3f2a1229f387d3c81909990d8945513e347b0eab049f7de3c6f98c1934bee9d01b3bd48fe3da18917bd009b3fdb8b105681d653e377c6427dc608eadaWindows x64 resource evidence
All frozen resource limits passed. Model assets and the generated stress index remain outside the repository.
Tests
npm --prefix kun run eval:memory-retrieval— 7 files, 38 tests passednpm run build:kun— passednpm run typecheck— passednpm run build— passednpm run check:file-lines— 6,947 tracked text files passednpm run lint— 0 errors; 30 existing warningsnpm exec -- openspec validate add-kun-semantic-memory-retrieval --strict— passedgit diff --check— passedThe complete
npm testrun finished with 5,751 passed, 27 skipped, and 22 failures in seven files outside this change. Seven Chromium-cookie failures were caused by the isolated runner PATH omitting the installedsqlite3.exe; both cookie files passed on focused rerun after restoring the normal user PATH (22 tests). Eleven failures remain reproducible in five unchanged upstream files: eight file-session index assertions, two Windows/POSIX path expectations, and one POSIX permission-bit expectation. No failed file or its production module is changed here.Non-goals