Skip to content

Document the browser UI apps in AGENTS.md#4290

Open
gavande1 wants to merge 3 commits into
trunkfrom
document-browser-ui-apps
Open

Document the browser UI apps in AGENTS.md#4290
gavande1 wants to merge 3 commits into
trunkfrom
document-browser-ui-apps

Conversation

@gavande1

@gavande1 gavande1 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Related issues

  • None

How AI was used in this PR

Claude wrote the change, guided by the author. The author verified each claim against the repo and manually reviewed it before pushing.

Proposed Changes

I found this gap using the claude-md-management plugin, which I learned about at a recent meetup. It audits AGENTS.md and CLAUDE.md files for coverage and currency.

AGENTS.md described Studio as an Electron app only, so an agent session had no pointer to the browser UI that studio ui serves. That left apps/ui, apps/local, and apps/hosted undocumented, even though apps/ui is one of the most active areas of the repo and is already referenced by the theming rules further down the file.

This adds those three apps to the Architecture and Directory Structure sections, along with a note on the stack difference between apps/ui and apps/studio.

Testing Instructions

  1. Read the diff in AGENTS.md.
  2. Confirm the described layout matches the repo: ls apps should list cli, hosted, local, studio, and ui.
  3. Confirm the stack claim: grep -ril 'redux\|tailwind' apps/ui/src apps/ui/package.json should return nothing.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors? (Not applicable, documentation only.)

@gavande1
gavande1 marked this pull request as ready for review July 22, 2026 13:46
@gavande1
gavande1 requested a review from a team July 22, 2026 13:46

@sejas sejas left a comment

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.

LGTM!

Comment thread AGENTS.md
**`/apps/studio/src`**: Main (index.ts, ipc-handlers.ts, site-server.ts, storage/, lib/) | Renderer (components/, hooks/, stores/) | modules/ (sync, cli, user-settings, preview-site)
**`/apps/cli`**: index.ts, commands/ (auth, preview, site), lib/ (appdata, i18n, browser)
**`/apps/ui`**: Agentic browser UI (`@studio/ui`). app/ (providers, router), components/, data/, hooks/, lib/. **Different stack from `apps/studio`** — React 19, TanStack Query + Router, `@wordpress/ui` + `ThemeProvider`; no Redux, no Tailwind. Built per target: `build:local` / `build:hosted`.
**`/apps/local`**, **`/apps/hosted`**: HTTP/SSE backends for `apps/ui`. `local` is bundled into the CLI; `hosted` is experimental.

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.

In the future we could remove apps/hosted code and documentation.

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