Skip to content

rfc: 0001 agent architecture v1#497

Draft
Enriquefft wants to merge 2 commits intosiddharthvaddem:mainfrom
Enriquefft:rfc/0001-agent-architecture
Draft

rfc: 0001 agent architecture v1#497
Enriquefft wants to merge 2 commits intosiddharthvaddem:mainfrom
Enriquefft:rfc/0001-agent-architecture

Conversation

@Enriquefft
Copy link
Copy Markdown
Contributor

@Enriquefft Enriquefft commented Apr 27, 2026

Summary

Draft RFC proposing a short planning window for the OpenScreen agent surface before more verbs land on top of #350.

Establishes a lightweight RFC convention for the project under docs/rfc/, with this as 0001.

What's in this RFC

  • Index (0001-agent-architecture.md) — one designed concern, two noticed-but-not-planned concerns, explicit acknowledgement that more may exist.
  • State coordination (companion) — full design: 12 GUI/CLI desync surfaces collapsing to one decision, editor-as-server + file-as-persistence + operation-log proposal.

The index is intentionally honest about asymmetry: state coordination has a companion doc; the two noticed items are starting points, not designs. The RFC's central proposal is a short planning window, not the items themselves.

Status

Draft. Not asking for merge yet — discussion first. Companion to #349 and #350; review notes on those issues point here.

Test plan

  • No code changes — docs-only PR.
  • Maintainer / author review of the proposed RFC convention and the index framing.
  • Surface any architectural concerns the noticed list misses.

Establishes a lightweight RFC convention for the project under docs/rfc/
and lands RFC 0001 as the first entry. Draft, opened for discussion.

The RFC proposes the architectural shape for OpenScreen's agent surface
(CLI + MCP + future SDK/IPC) before more verbs land on top of siddharthvaddem#350.
Nine architectural concerns are catalogued with file:line citations,
and a verb-registry shape (defineVerb / executeVerb / StateSource /
CommandResult) is sketched as the SSOT for transport fan-out.

Companion docs cover (a) the twelve GUI/CLI desync surfaces and an
editor-as-server + file-as-persistence + op-log model, and (b) an
audit of PR siddharthvaddem#350 grounding the RFC's claims in concrete code.

Tracking: siddharthvaddem#349
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 27, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bcf39c32-e9ce-40a9-af39-6a1089f02a0e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

…ed concerns

The index now distinguishes "designed" (state coordination, with companion
doc) from "noticed, not yet planned" (two concerns surfaced via filtered
review of PR siddharthvaddem#350). Audit doc dropped — PR-review observations belong on
PR siddharthvaddem#350 itself, not in the RFC.
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