Skip to content

docs: fix drift in Railway Deployment#247

Draft
jack-arturo wants to merge 1 commit into
mainfrom
claude/dreamy-wright-2894kr
Draft

docs: fix drift in Railway Deployment#247
jack-arturo wants to merge 1 commit into
mainfrom
claude/dreamy-wright-2894kr

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

Routine docs-accuracy audit of deployment/railway.md (never previously reviewed).

Claim Current state Fix
Caution box + Troubleshooting: "Without PORT, Flask defaults to port 5000, causing connection failures" automem/runtime_wiring.py does int(os.environ.get("PORT", "8001")) — an unset PORT already resolves to 8001, not Flask's own 5000 default (same class of already-fixed myth as overview PR #239 / operations/troubleshooting PR #225) Reworded both the caution box and the troubleshooting "Solution 1" to state the real default, while keeping the operational advice (set PORT=8001 explicitly so other Railway services can reach it reliably)
Environment Variables Reference → PORT row lists default 5000 Same myth, in table form Corrected default to 8001 with a note on why source
memory-service Environment Variables Reference table is missing 7 real vars docker-compose.yml's flask-api.environment also defines VOYAGE_API_KEY, EMBEDDING_PROVIDER (default auto), VECTOR_SIZE (default 1024), MEMORY_CONTENT_HARD_LIMIT (default 2000), MEMORY_AUTO_SUMMARIZE (default true), QDRANT_TIMEOUT_SECONDS (unset by default), QDRANT_ENSURE_PAYLOAD_INDEXES (default true) — the same 6 vars already added to deployment/docker in PR #246, plus EMBEDDING_PROVIDER which was also missing here Added rows for all 7 variables with their real defaults
Intro links to INSTALLATION.md and docs/MCP_SSE.md used /blob/main/ branch links Constraint requires permalinks over branch links Converted both to commit-SHA permalinks

Confirmed correct, no changes needed: the mcp-sse-server directory and its Railway build config (mcp-sse-server/railway.json, Dockerfile Path: mcp-sse-server/Dockerfile, default PORT=8080) genuinely exist in the automem repo (this is distinct from the mcp-automem package, where PR #245 found mcp-sse-server does not exist — both are true, just about different repos); IPv6 dual-stack host="::" binding in runtime_wiring.py; scripts/recover_from_qdrant.py path; railway.json build config (DOCKERFILE builder, Dockerfile path) at repo root; FalkorDB required vars and --requirepass caution; health check path/timeout.

Verified against: automem@4b5eaafd2602c9eba39bbfe38e4120e3654c67e9 (runtime_wiring.py, docker-compose.yml, scripts/recover_from_qdrant.py, mcp-sse-server/, railway.json, docs/MCP_SSE.md), mcp-automem@946f9e5ed1385b632efd2e5b250d064bcc4295e8 (CLI flags cross-check, no drift found)

No open question / follow-ups for this page.


Generated by Claude Code

Routine docs-accuracy audit of deployment/railway.md (never previously reviewed).

- PORT env var: corrected the "Flask defaults to 5000" myth (automem/runtime_wiring.py
  already defaults to 8001) in the caution box, the env var table, and the
  troubleshooting section
- Added 7 real memory-service env vars missing from the reference table:
  VOYAGE_API_KEY, EMBEDDING_PROVIDER, VECTOR_SIZE, MEMORY_CONTENT_HARD_LIMIT,
  MEMORY_AUTO_SUMMARIZE, QDRANT_TIMEOUT_SECONDS, QDRANT_ENSURE_PAYLOAD_INDEXES
- Converted two branch-pinned links (INSTALLATION.md, docs/MCP_SSE.md) to
  commit-SHA permalinks

Verified against: automem@4b5eaafd2602c9eba39bbfe38e4120e3654c67e9
(runtime_wiring.py, docker-compose.yml, scripts/recover_from_qdrant.py,
mcp-sse-server/, railway.json)

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

Copy link
Copy Markdown

Deploying automem-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: e927bfa
Status: ✅  Deploy successful!
Preview URL: https://267bebe2.automem-website.pages.dev
Branch Preview URL: https://claude-dreamy-wright-2894kr.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