Skip to content

fix: unify game-of-throne-demo EVERMEMOS_URL default port to 1995#192

Merged
cyfyifanchen merged 1 commit into
EverMind-AI:mainfrom
pazyork:fix/game-of-throne-demo-port-mismatch
Apr 19, 2026
Merged

fix: unify game-of-throne-demo EVERMEMOS_URL default port to 1995#192
cyfyifanchen merged 1 commit into
EverMind-AI:mainfrom
pazyork:fix/game-of-throne-demo-port-mismatch

Conversation

@pazyork
Copy link
Copy Markdown
Contributor

@pazyork pazyork commented Apr 18, 2026

saw issue #28 was still open. the game-of-throne-demo server.ts had port 8001 as default but EverMemOS server actually runs on 1995. PR #181 fixed ports for the EverMemOS demos but game-of-throne-demo was added in #186 after and got missed.

tested:

  • typescript compiles clean
  • no other 8001 references left in the codebase
  • data flow checked end-to-end

cloud mode uses env override so only local dev affected. fixes #28

happy to adjust if needed, thanks.

server.ts used 8001 but the EverMemOS server default is 1995.
This broke local demo runs unless EVERMEMOS_URL was manually set.

Fixes EverMind-AI#28
@pazyork pazyork marked this pull request as draft April 18, 2026 05:31
@pazyork
Copy link
Copy Markdown
Contributor Author

pazyork commented Apr 18, 2026

@cyfyifanchen @gloryfromca
noticed issue #28 was still open. the game-of-throne-demo server.ts had port 8001 as default but EverMemOS server runs on 1995. PR #181 fixed most ports but this file was added later in #186 and got missed
just a one line fix, tested with tsc --noEmit. happy to adjust, thanks.

@pazyork pazyork marked this pull request as ready for review April 18, 2026 05:35
Copy link
Copy Markdown
Collaborator

@cyfyifanchen cyfyifanchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the PR.

@cyfyifanchen cyfyifanchen merged commit f06c303 into EverMind-AI:main Apr 19, 2026
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.

SimpleMemoryManager‘s default port is 1995, does not match 8001 in the README

3 participants