Skip to content

docs: fix drift in Queue Management - #307

Draft
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-cli-queue-20260814
Draft

docs: fix drift in Queue Management#307
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-cli-queue-20260814

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

Automated docs-accuracy audit of docs/cli/queue. Five proven discrepancies, capped at the per-PR limit. No open PR existed for this page.

Claim Current state Fix Evidence
queue --file /path/to/memory-queue.jsonll The file is JSONL; every other mention on the page (and the default path in runQueueCommand) uses .jsonl. jsonll is a typo in a copy-pasteable command Drop the extra l mcp-automem@92c2e33:src/cli/queue.ts#L116
Queue diagram: "Process each entry — POST /memory or PATCH /memory/{id}" The drain loop only calls client.storeMemory() (POST /memory). There is no PATCH path anywhere in queue.ts. Entries carrying relatesTo additionally get a POST /associate after the store pass Replace the PATCH branch with the association pass that actually exists mcp-automem@92c2e33:src/cli/queue.ts#L189, mcp-automem@92c2e33:src/cli/queue.ts#L219
Service-unavailable output is three lines: Checking AutoMem service at …, ❌ Service unavailable - skipping queue processing, Queue will be retried on next run The only thing printed on an unhealthy endpoint is AutoMem endpoint unavailable; skipping queue drain. None of the three documented lines exists in the source Show the real output mcp-automem@92c2e33:src/cli/queue.ts#L138
HealthStatus.statistics has four fields; falkordb and qdrant are typed string The interface declares ten optional fields — the four documented plus memory_count, vector_count, sync_status, vector_dimensions, enrichment — and falkordb/qdrant are any, not string Add the five missing rows, correct the two types, note that the fields are optional mcp-automem@92c2e33:src/types.ts#L148
UpdateMemoryArgs accepts ten parameters The interface also accepts t_valid and t_invalid, which are the documented way to express a fact's shelf life instead of date tags Add both rows mcp-automem@92c2e33:src/types.ts#L314

Verified against: mcp-automem@92c2e33cb932b71d0df35fdc250778c6a51257af

Checked and correct, left untouched: the endpoint priority chain (AUTOMEM_API_URL / AUTOMEM_ENDPOINT~/.claude.json mcpServershttp://127.0.0.1:8001), the --dry-run and --limit flags, the default queue path, the two-mode DeleteMemoryArgs table, destructiveHint: true, and the manual-recovery framing of the command.

Questions

  • The page says the retired capture/queue Stop hooks "were removed in v0.15". The removal itself is confirmed in the repo guide, but I could not tie it to that specific release from the checkout, so I left the version as written rather than guess at it.
  • statistics.backend is documented as "always automem". The interface types it as a bare string and the value is set server-side, so the "always" is not verifiable from the client repo. Left alone since it may well be true in practice.

Unverified

  • The "Update vs Delete Decision Matrix" and the two mermaid decision trees are editorial guidance with no code counterpart.
  • The Deletion Side Effects list describes backend behavior (DETACH DELETE, Qdrant point removal) that lives in automem, not mcp-automem; it matches what I saw while auditing the Memory Operations page in this same run, so I left it.

Follow-ups

  • The queue diagram puts the health check before reading the queue file. In runQueueCommand the file is read first, and a missing or empty file returns early with Queue file not found … / Memory queue empty. — the health check never runs in those cases.
  • The diagram's "Remove processed entries from queue file" step understates the behavior: when the queue fully drains, the file is unlinked rather than truncated.
  • "The handler validates the memory_id and delegates to the client" under Delete Implementation predates the two-mode delete the table above it already documents.
  • The three src/types.ts links are pinned at 538721c, several releases behind the audited SHA.

Generated by Claude Code

Correct the --file example extension, the queue processing diagram's
HTTP calls, the service-unavailable console output, the HealthStatus
statistics fields, and the UpdateMemoryArgs parameter table.

Verified against mcp-automem@92c2e33cb932b71d0df35fdc250778c6a51257af

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WdXojbwftVqDeiYvUDZ3JZ
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying automem-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1d307e0
Status: ✅  Deploy successful!
Preview URL: https://1584a600.automem-website.pages.dev
Branch Preview URL: https://docs-audit-cli-queue-2026081.automem-website.pages.dev

View logs

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.

2 participants