Skip to content

fix(openclaw): gate append retention on stored text - #2511

Merged
benfrank241 merged 1 commit into
vectorize-io:mainfrom
xmh1011:fix/issue-2505-openclaw-append-capability
Jul 15, 2026
Merged

fix(openclaw): gate append retention on stored text#2511
benfrank241 merged 1 commit into
vectorize-io:mainfrom
xmh1011:fix/issue-2505-openclaw-append-capability

Conversation

@xmh1011

@xmh1011 xmh1011 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • parse /version feature flags in the OpenClaw append capability probe
  • require both API version >= 0.5.0 and features.store_document_text=true before sending update_mode=append
  • fall back to per-turn document IDs when document text storage is disabled
  • add regression coverage for legacy payloads, enabled text storage, disabled text storage, and older API versions

Tests

  • npm test -- src/index.test.ts
  • npm run build
  • ./scripts/hooks/lint.sh

Closes #2505

@koriyoshi2041

Copy link
Copy Markdown
Contributor

I had a duplicate patch open for this path and closed it in favor of this one.

I compared this against #2505 and the current /version surface: FeaturesInfo exposes store_document_text as an explicit bool, and tests/test_store_document_text.py covers the disabled response. The stricter helper here, where append requires stored document text, matches that contract.

Local third-party check on this head:

cd hindsight-integrations/openclaw
npm test
# Test Files  9 passed (9)
# Tests       281 passed (281)

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

Labels

None yet

Projects

None yet

3 participants