Skip to content

docs: expand architecture docs#525

Open
doorgan wants to merge 1 commit into
mainfrom
doorgan/architecture-docs
Open

docs: expand architecture docs#525
doorgan wants to merge 1 commit into
mainfrom
doorgan/architecture-docs

Conversation

@doorgan
Copy link
Copy Markdown
Contributor

@doorgan doorgan commented Mar 19, 2026

Expands the architecture docs with more information on how expert works

@doorgan doorgan force-pushed the doorgan/architecture-docs branch 3 times, most recently from 12f6aba to 29f4533 Compare March 24, 2026 20:10
@doorgan doorgan force-pushed the doorgan/architecture-docs branch from 29f4533 to d5a1ea9 Compare April 27, 2026 21:37
@doorgan doorgan marked this pull request as ready for review April 27, 2026 22:13
@doorgan doorgan force-pushed the doorgan/architecture-docs branch from d5a1ea9 to 7e2fd26 Compare April 27, 2026 22:14
Comment thread pages/architecture.md

Expert combines several sources of code-intelligence data:

- ElixirSense provides request-time suggestions, hover information, and fallback definition lookup.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For hover data, ElixirSense is also just a fallback IIRC, similar to definition

Comment thread pages/architecture.md

Expert starts the engine node as a separate OS process and communicates with it through distributed Erlang RPC.

Some distributed Erlang setups rely on the Erlang Port Mapper Daemon (EPMD), but Expert uses an EPMDless setup. `Forge.EPMD` avoids the system EPMD daemon, and `Forge.NodePortMapper` lets project nodes register their distribution ports with the manager node. This prevents stale EPMD entries and prevents nodes from different Expert instances from discovering each other accidentally.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I wonder if we should rename Forge.EPMD to Forge.FakeEPMD or something. But it's not really related to this PR.

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