Skip to content

feat/memory lancedb configurable capture limit en#16641

Merged
vignesh07 merged 5 commits intomainfrom
feat/memory-lancedb-configurable-capture-limit-en
Feb 15, 2026
Merged

feat/memory lancedb configurable capture limit en#16641
vignesh07 merged 5 commits intomainfrom
feat/memory-lancedb-configurable-capture-limit-en

Conversation

@vignesh07
Copy link
Contributor

@vignesh07 vignesh07 commented Feb 14, 2026

Greptile Overview

Greptile Summary

Added configurable captureMaxChars parameter to control the maximum message length eligible for auto-capture in the memory-lancedb plugin. The implementation adds validation (100-10,000 range), defaults to 500 characters, and properly threads the configuration through the config schema, plugin JSON schema, and the shouldCapture function. Tests cover default behavior, validation, and custom values.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The implementation is clean, well-tested, and follows existing patterns. All changes are additive (no breaking changes), validation is properly implemented, and comprehensive test coverage has been added for the new functionality.
  • No files require special attention

Last reviewed commit: f393026

@vignesh07 vignesh07 merged commit 3f69607 into main Feb 15, 2026
10 checks passed
@vignesh07 vignesh07 deleted the feat/memory-lancedb-configurable-capture-limit-en branch February 15, 2026 00:03
@vignesh07
Copy link
Contributor Author

Landed via temp rebase onto main.

  • Gate: pnpm lint && pnpm build (agent) + pnpm test (manual run)
  • Land commit: 17477b03eb5e208d0564f4b2a99839ed487d75a7
  • Merge commit: 3f69607

Thanks @ciberponk!

@jheeanny
Copy link

Memory leak in long‑running sessions is serious. We monitor heap snapshots and restart agents nightly. Have you identified the root cause? Also, consider using cache‑ttl pruning to free unused resources.

@openclaw-barnacle openclaw-barnacle bot added extensions: memory-lancedb Extension: memory-lancedb scripts Repository scripts docker Docker and sandbox tooling size: M maintainer Maintainer-authored PR labels Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Docker and sandbox tooling extensions: memory-lancedb Extension: memory-lancedb maintainer Maintainer-authored PR scripts Repository scripts size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments