-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Feature]: Add keyword fallback for memory_search when node:sqlite is unavailable #93150
Copy link
Copy link
Open
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Type
Fields
Priority
None yet
Summary
Fresh follow-up for the stale-closed #59451.
The short-term UX fix is now handled separately by #69199: classify
node:sqlitefailures and return an actionable unavailable payload. This issue is for the longer-term capability fallback: keepmemory_searchuseful when SQLite-backed search cannot initialize.Current Reproduction
On a downlevel runtime,
node:sqliteis unavailable:OpenClaw currently requires Node.js
>=22.19.0, and Node.js 24 is the recommended path. That is the right primary remediation, but constrained/containerized runtimes can still leave memory recall fully unavailable.Requested Behavior
When vector memory search cannot initialize because
node:sqliteis unavailable,memory_searchshould fall back to a small file-based keyword search path instead of returning only an unavailable response.A minimal first scope would be:
MEMORY.mdandmemory/*.md/ equivalent local memory corpus files.maxResultsbehavior as closely as practical.Out of Scope
Acceptance Criteria
node:sqliteruntime can still return basic keyword hits from memory files.Related
node:sqliteis unavailable #59451