fix(memory): source_id short ID + recall/remember help schemas + decay default + score normalization#440
Merged
Merged
Conversation
…c + score normalization Five Wave-1 fixes from /tmp/v023-usage-audit-consolidated.md UE3 + ADR-A: 1. remember(source_id=...) accepts 8-char short IDs (Critical) — same prefix resolution as get/link/update. Closes recon #291 and the chain create->remember workflow. 2. recall(help=true) + remember(help=true) HandlerDef params updated to include all PR #406/#421 args (top_k, score_floor, fusion_strategy, embedding_model + presentation for verbose breakdown). 3. decay_factor default doc corrected: 0.1 -> 0.01 (10x off). 4. compute_score normalizes RRF-fused scores by (k+1) to comparable range with weighted/union so score_floor is portable across fusion_strategy. 5. presentation="verbose" on recall includes per-component score breakdown without changing agent-mode response shape. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.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.
Closes UE3 Crit + 4 Major. (1) remember(source_id=) accepts 8-char short IDs via runtime.resolve_prefix. (2) recall/remember help=true now lists top_k/score_floor/fusion_strategy/embedding_model. (3) decay_factor default doc corrected 0.1→0.01. (4) compute_score normalizes RRF to comparable range so score_floor is portable across fusion strategies. 107 tests pass.