Skip to content

docs: fix drift in architecture/embeddings#178

Merged
jack-arturo merged 1 commit into
mainfrom
docs/audit-architecture-embeddings-20260616
Jun 22, 2026
Merged

docs: fix drift in architecture/embeddings#178
jack-arturo merged 1 commit into
mainfrom
docs/audit-architecture-embeddings-20260616

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

Automated docs audit — 2026-06-16.

Verified against: automem@ebcf5f16d8a0eecc9400957be1503efaf97fa530


Fixes

Claim Current state Fix
12 source-note SHA permalinks at ed36b98e Current HEAD is ebcf5f16 (June 12 2026); old SHA links resolve but point to stale code Updated all 12 to ebcf5f16
"Not configurable via environment variable — set in the provider constructor at initialization." (Voyage model) At ebcf5f16, provider_init.py reads os.getenv("VOYAGE_MODEL", "voyage-4") — the env var is real and functional Changed to "Configurable via VOYAGE_MODEL env var."
VOYAGE_MODEL absent from Provider-Specific Configuration table Confirmed in provider_init.py@ebcf5f16 Added VOYAGE_MODEL row with default voyage-4 and supported models

Open question (not fixed — needs maintainer input)

Job Structure table (Queue-Based Processing section) lists three fields: memory_id, content, attempt: int. Code inspection of runtime_pipeline.py@ebcf5f16 suggests jobs may be stored as (memory_id, content) tuples, not dicts with an attempt field — the attempt logic in the Error Handling table (re-queue if attempt < 3) may be tracked as a worker-local counter rather than a job field. Could not confirm definitively from the WebFetch response. Recommend a maintainer review the Job Structure table and the "Increment attempt counter, re-queue if attempt < 3" error row for accuracy.


Generated by Claude Code

- Update 12 source-note SHA permalinks ed36b98e → ebcf5f16
- Fix Voyage model env var claim: VOYAGE_MODEL is configurable via env var
- Add VOYAGE_MODEL to provider-specific configuration table

Verified against: automem@ebcf5f16d8a0eecc9400957be1503efaf97fa530
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying automem-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: fb02909
Status: ✅  Deploy successful!
Preview URL: https://e282e41c.automem-website.pages.dev
Branch Preview URL: https://docs-audit-architecture-embe-zm3z.automem-website.pages.dev

View logs

@jack-arturo

Copy link
Copy Markdown
Member Author

Verified fixes merging now. Open questions from this audit are tracked in #190 so they aren't lost.

@jack-arturo jack-arturo marked this pull request as ready for review June 22, 2026 16:36
@jack-arturo jack-arturo added this pull request to the merge queue Jun 22, 2026
Merged via the queue into main with commit 2eab999 Jun 22, 2026
5 checks passed
@jack-arturo jack-arturo deleted the docs/audit-architecture-embeddings-20260616 branch June 22, 2026 16:50
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