docs: fix drift in ChatGPT - #266
Draft
jack-arturo wants to merge 1 commit into
Draft
Conversation
Remove the false "Flask defaults to port 5000" claim from the Railway caution and rewrite the ECONNREFUSED troubleshooting row that was premised on it. The memory service defaults to port 8001. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Bh4cvNW2mR2vCoyg3FDQL3
Deploying automem-website with
|
| Latest commit: |
a25c8c4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3ddb2ae5.automem-website.pages.dev |
| Branch Preview URL: | https://docs-audit-platforms-chatgpt.automem-website.pages.dev |
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.
Routine docs-accuracy audit of
src/content/docs/docs/platforms/chatgpt.md(never previously reviewed — no entry inscripts/file-doc-map.json).One finding, stated in two places. It is the last unswept instance of the "Flask defaults to 5000" myth: the same false claim was removed from
overview,operations/troubleshooting, anddeployment/railway(#247), and fromplatforms/claude-webin the companion PR from this run (#265).:::caution— "The memory service must havePORT=8001set — without it, Flask defaults to port 5000 and connections will fail."port = int(os.environ.get("PORT", "8001")). There is no code path that produces 5000. LeavingPORTunset is the working configuration, so this instructs users to set a variable to avoid a failure that cannot happen.8001unlessPORTis overridden, and the port inAUTOMEM_API_URLhas to match whatever it is actually bound to.PORT=8001in memory service" → "AddPORT=8001to memory service environment variables"ECONNREFUSEDon:8001will add a variable whose value already applies, conclude the fix didn't work, and still not know the real cause.PORTon the service against the port inAUTOMEM_API_URL.Spot-checked and correct, left untouched:
mcp-sse-server/server.js:/mcp(L1039),/mcp/sse(L1092),/mcp/messages(L1126),/health(L857).PORTdefault8080(const port = process.env.PORT || 8080), and bothAUTOMEM_API_URLandAUTOMEM_API_TOKENare read as documented.setInterval(…, 20000)).toolsarray inmcp-automem'ssrc/index.ts.curl …/healthin Troubleshooting — the sidecar's/healthroute takes no request token; it probes upstream with its ownAUTOMEM_API_TOKEN.${{...}}Railway variable-resolution section and the "Security Notes" bullets about URL tokens appearing in logs.Verified against: verygoodplugins/automem@8ff266e
Questions
None.
Unverified
Follow-ups
Below the severity bar for this PR:
/mcpas methodPOST; it is registered withapp.all('/mcp', …), soGETandDELETEalso reach it.POSTis the right thing to document — the table is just narrower than the route. Same note applies to the identical table onplatforms/claude-web.Generated by Claude Code as part of the recurring automem-docs-review routine.
Generated by Claude Code