Skip to content

fix(memory): support arbitrary Qdrant document ids - #1350

Merged
EmersonBraun merged 1 commit into
mainfrom
codex/fix-qdrant-point-ids
Jul 27, 2026
Merged

fix(memory): support arbitrary Qdrant document ids#1350
EmersonBraun merged 1 commit into
mainfrom
codex/fix-qdrant-point-ids

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Summary

  • map arbitrary vector document IDs to deterministic Qdrant-compatible UUIDs
  • preserve safe numeric IDs and valid UUIDs
  • restore original source IDs on search and use the same mapping on delete
  • keep adapter metadata internal
  • add the required patch changeset

Root cause

Qdrant accepts point IDs only as unsigned integers or UUIDs. AgentsKit RAG creates
chunk IDs such as doc-1_chunk_0, which the Qdrant adapter previously forwarded
unchanged and Qdrant rejected with HTTP 400.

Validation

  • pnpm --filter @agentskit/core build
  • pnpm --filter @agentskit/memory test — 209 passed
  • pnpm --filter @agentskit/memory lint
  • pnpm --filter @agentskit/memory build
  • live Qdrant 1.15.4 proof stored, restored, and deleted doc-1_chunk_0
  • git diff --check

This draft does not authorize merge, release, tag, npm publication, or the external
Qdrant example pull request.

@EmersonBraun
EmersonBraun marked this pull request as ready for review July 27, 2026 12:27
@EmersonBraun
EmersonBraun merged commit c6e3607 into main Jul 27, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant