Skip to content

fix(memory-lancedb): register runtime for doctor and status#2339

Open
BingqingLyu wants to merge 2 commits into
mainfrom
fork-pr-65066-codex-fix-memory-lancedb-runtime-registration
Open

fix(memory-lancedb): register runtime for doctor and status#2339
BingqingLyu wants to merge 2 commits into
mainfrom
fork-pr-65066-codex-fix-memory-lancedb-runtime-registration

Conversation

@BingqingLyu

@BingqingLyu BingqingLyu commented Apr 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • register a memory capability runtime for memory-lancedb so core doctor/status surfaces can discover it as the active memory plugin
  • expose the minimal runtime hooks doctor/status need: backend config resolution, search-manager status, embedding probe, and vector probe
  • add regression coverage proving the plugin registers a visible runtime instead of only tools/hooks

Problem

memory-lancedb is a kind: "memory" plugin, but it did not register a memory runtime/capability. That meant the core memory-runtime path had nothing to resolve, so doctor/status could incorrectly report that no active memory plugin was registered even when the plugin was installed and working.

Validation

  • pnpm vitest run extensions/memory-lancedb/index.test.ts
  • pnpm lint extensions/memory-lancedb/index.ts extensions/memory-lancedb/index.test.ts
  • pnpm build --filter memory-lancedb

Fixes openclaw#60177

@BingqingLyu BingqingLyu added the independent-review Non-trivial PR with no function-level conflict with other open PRs label May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

independent-review Non-trivial PR with no function-level conflict with other open PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: memory-lancedb-pro loaded and working, but doctor/status reports no active memory plugin

2 participants