Skip to content

docs: update intentd architecture documentation - #3406

Open
wookiehangover wants to merge 1 commit into
mainfrom
codex/update-intentd-architecture
Open

docs: update intentd architecture documentation#3406
wookiehangover wants to merge 1 commit into
mainfrom
codex/update-intentd-architecture

Conversation

@wookiehangover

Copy link
Copy Markdown
Contributor

Summary

  • Audit the architecture document against the current daemon implementation, canonical protocol documentation, and frontend integration.
  • Document all 17 workspace crates, their production dependency graph, daemon supervision, remote transports, subscriptions, workspace transfers, harness versioning, and voice transcription.
  • Correct stale claims about service-module imports, transient events, search dependencies, ACP dependencies, and installed-daemon behavior.

Validation

  • Compared every documented workspace crate and production dependency against Cargo metadata.
  • Verified all 13 relative documentation links.
  • Confirmed the documentation diff passes whitespace validation.

@augmentcode

augmentcode Bot commented Aug 25, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR refreshes the intentd architecture reference against the current daemon and protocol.

Changes:

  • Updates the workspace crate inventory, responsibilities, and production dependency graph.
  • Documents local/WSS transports, pairing, reverse RPC ownership, and loopback tunneling.
  • Explains durable versus transient events and typed snapshot-plus-delta subscriptions.
  • Adds installed-daemon supervision, update, and restart behavior for `intentd-sitter`.
  • Describes SQLite state, CRDT note merging, workspace transfer, and versioned harness behavior.
  • Documents daemon-owned voice transcription, provider credentials, and workspace vocabulary.

Technical Notes: The document now points readers to canonical protocol and harness documentation, corrects stale service/dependency claims, and preserves the daemon binary as the composition root.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread docs/ARCHITECTURE.md
targets the first connected live client and fails over in connection order.
Reverse requests use the `rev-<n>` identifier namespace and bounded
timeouts.
- **Remote loopback tunneling.** The authenticated `/tunnel` WebSocket shares

@augmentcode augmentcode Bot Aug 25, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

docs/ARCHITECTURE.md:176 describes /tunnel as authenticated, but serve --insecure serves that endpoint over plain ws:// with TLS and bearer-token enforcement skipped. This leaves an absolute security claim that is false in the documented development mode.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Deep Code Review Agent🐛

Review completed with 0 suggestions.

Reviewed commit: 460fcec

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.

1 participant