Skip to content

fix(openclaw-plugin): gate low-score resource recall#2546

Open
Mijamind719 wants to merge 1 commit into
volcengine:mainfrom
Mijamind719:codex/openclaw-resource-recall-gating
Open

fix(openclaw-plugin): gate low-score resource recall#2546
Mijamind719 wants to merge 1 commit into
volcengine:mainfrom
Mijamind719:codex/openclaw-resource-recall-gating

Conversation

@Mijamind719

Copy link
Copy Markdown
Collaborator

Summary

Replaces #2084 with a smaller, current-main-based fix for noisy OpenClaw resource recall.

  • add resource-aware recall score gating for viking://resources/ results
  • keep user/agent memory threshold behavior unchanged
  • make recall query tokenization Unicode-aware so Cyrillic query terms can contribute lexical overlap
  • document the focused design in docs/plans/2026-06-10-openclaw-resource-recall-gating-design.md

Verification

  • npm test -- tests/ut/memory-ranking.test.ts tests/ut/context-engine-assemble.test.ts
  • npm run build
  • git diff --check

@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

2084 - Fully compliant

Compliant requirements:

  • Added resource-aware recall score gating for viking://resources/ results
  • Kept user/agent memory threshold behavior unchanged
  • Made recall query tokenization Unicode-aware for Cyrillic support
  • Added design documentation
  • Added unit tests for the new functionality
⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🏅 Score: 92
🧪 PR contains tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant