Repository navigation
docs: add a source-verified GraphAgent architecture overview - #702
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 057b52f7ee
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <g data-component="api"> | ||
| <rect class="node" x="280" y="264" width="184" height="120" rx="6" /> | ||
| <text class="node-name" x="300" y="292">HTTP API 与事件</text> | ||
| <text class="sublabel" x="300" y="316">Hono / Effect HttpApi</text> |
There was a problem hiding this comment.
Remove Hono from the runtime API label
The diagram places Hono inside the opencode service process, but the reviewed baseline constructs this server with Effect's HttpRouter and HttpApiApp (packages/opencode/src/server/server.ts:3-12,55-67,99-113), and packages/opencode has no Hono dependency or import; the repository's Hono usages belong to the unrelated packages/function and packages/enterprise services. This makes the source-verified overview direct readers toward a nonexistent runtime layer, so the label should describe only the Effect HTTP stack.
Useful? React with 👍 / 👎.
Why
The Docs entry has no single overview of the current GraphAgent runtime. Readers need a map of the client transports, service composition, DAG execution, model calls, tools, and storage.
What changed
docs/architecture.mdwith a Mermaid overview, component responsibilities, and source links.docs/architecture.htmlwith the project's black-and-white architecture diagram.Evidence
Verify the overview against
mainat10806c9ad7. Show AppLayer as service composition, distinguish HTTP requests from SSE events, and explain the default AI SDK path and opt-in native model path. Describe SQLite state separately from workspace files, Memory YAML, and managed artifacts.Checklist
docs.Closes #701