memory-lancedb: fix TypeBox Optional+enum schema typecheck on main#13897
Merged
memory-lancedb: fix TypeBox Optional+enum schema typecheck on main#13897
Conversation
zapabob
pushed a commit
to zapabob/clawdbot
that referenced
this pull request
Feb 11, 2026
shan-mx
pushed a commit
to shan-mx/openclaw
that referenced
this pull request
Feb 11, 2026
michaelleone
pushed a commit
to michaelleone/openclaw
that referenced
this pull request
Feb 11, 2026
Patrick3131
added a commit
to Patrick3131/openclaw
that referenced
this pull request
Feb 11, 2026
* main: (936 commits) fix: use configured base URL for Ollama model discovery (openclaw#14131) docs: modernize gateway configuration page (Phase 1) (openclaw#14111) docs(skills): update mintlify skill to reference docs/ directory (openclaw#14125) chore: update AGENTS.md and add mintlify skill (openclaw#14123) chore(irc): sync plugin version to 2026.2.10 fix(cli): drop logs --localTime alias noise fix(config): avoid redacting maxTokens-like fields (openclaw#14006) feat: Add --localTime option to logs command for local timezone display (openclaw#13818) fix: don't lowercase Slack channel IDs (openclaw#14055) fix(plugins): ignore install scripts during plugin/hook install fix(gateway): default-deny missing connect scopes fix(web-search): handle xAI Responses API format in Grok provider Feat/litellm provider (openclaw#12823) docs: start 2026.2.10 changelog section chore: bump version to 2026.2.10 docs: remove outdated pricing information docs: add Terraform IaC approach to Hetzner guide fix(ui): coerce form values to schema types before config.set (openclaw#13468) chore: make merge PR comment mandatory + skill name fix 🤖 memory-lancedb: avoid plugin-sdk enum helper in local TypeBox schema (openclaw#13897) ...
GoddessSerenity
pushed a commit
to GoddessSerenity/openclaw
that referenced
this pull request
Feb 12, 2026
Takhoffman
pushed a commit
to tomsun28/openclaw
that referenced
this pull request
Feb 12, 2026
sauerdaniel
pushed a commit
to sauerdaniel/openclaw
that referenced
this pull request
Feb 12, 2026
dbachelder
pushed a commit
to dbachelder/openclaw
that referenced
this pull request
Feb 13, 2026
Baukebrenninkmeijer
pushed a commit
to orq-ai/openclaw
that referenced
this pull request
Feb 13, 2026
skyhawk14
pushed a commit
to skyhawk14/openclaw
that referenced
this pull request
Feb 13, 2026
cloud-neutral
pushed a commit
to cloud-neutral-toolkit/openclawbot.svc.plus
that referenced
this pull request
Feb 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Human written summary:
The intent of this change is, as written by a human:
The rest of this PR was written by GPT-5-default, running in the pi harness. Full environment + prompt history appear at the end.
Changes
Type.Optional(stringEnum(MEMORY_CATEGORIES))inextensions/memory-lancedb/index.tswith a localType.Unsafe<MemoryCategory>enum schema.stringEnumimport fromopenclaw/plugin-sdkin that extension to avoid cross-module TypeBox schema type incompatibility.Tests
pnpm check✅Risks
type: "string", enum overMEMORY_CATEGORIES) and change is scoped to one extension tool parameter.Follow-ups
Prompt History
Environment
Harness: pi
Model: GPT-5
Thinking level: default
Terminal: zsh (pi harness)
System: Darwin 25.1.0 arm64
Prompts
- pnpm check ❌ (unrelated pre-existing TS error in extensions/memory-lancedb/index.ts:320) -> whats goin on on main?but where did it come from? lets just bundle in our PR. can you bisect or whatever to find where it originateD?Greptile Overview
Greptile Summary
Fixes TypeBox schema type incompatibility by inlining enum schema construction in the
memory-lancedbextension. The change removes thestringEnumimport fromopenclaw/plugin-sdkand replaces it with a localType.Unsafe<MemoryCategory>construction that uses the extension's own TypeBox instance.Root cause: Extensions with their own
@sinclair/typeboxdependency (even at the same version) create separate TypeBox instances, causing TypeScript type incompatibility when wrapping schemas from different instances.Changes:
stringEnumhelper import fromopenclaw/plugin-sdkType.Unsafe<MemoryCategory>({ type: "string", enum: [...MEMORY_CATEGORIES] })Confidence Score: 5/5
pnpm check✅), the change is scoped to a single tool parameter, and there are no other extensions usingstringEnum, eliminating any ripple effects