refactor(api-docs): remove redux and split into docs, playground and shared packages

.changeset/config.json

@@ -7,5 +7,9 @@
   "access": "public",
   "baseBranch": "main",
   "updateInternalDependencies": "patch",
-  "ignore": []
+  "bumpVersionsWithWorkspaceProtocolOnly": true,
+  "ignore": [
+    "@usebruno/api-shared",
+    "@usebruno/api-playground"
+  ]
 }

.changeset/playground-edits-reach-docs-variables.md

@@ -0,0 +1,5 @@
+---
+"@usebruno/api-docs": minor
+---
+
+Docs pages now show edits made in the playground until the page is reloaded.

.changeset/playground-loads-only-when-opened.md

@@ -0,0 +1,5 @@
+---
+"@usebruno/api-docs": patch
+---
+
+The playground now loads only when opened, so docs pages load less code.

.coderabbit.yaml

@@ -0,0 +1,160 @@
+# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
+
+language: 'en-US'
+early_access: false
+tone_instructions: 'Expert reviewer in TypeScript, React, Emotion, Vite, Vitest and Playwright. Be concise. Favour the simplest code that works; flag abstractions without a second user. Explain more only when asked.'
+
+knowledge_base:
+  opt_out: false
+  code_guidelines:
+    enabled: true
+    filePatterns:
+      - '**/CODING_STANDARDS.md'
+  learnings:
+    scope: 'local'
+  jira:
+    usage: 'disabled'
+
+reviews:
+  profile: 'chill'
+  high_level_summary: true
+  poem: false
+  review_status: true
+  collapse_walkthrough: false
+  auto_review:
+    enabled: true
+    drafts: false
+    base_branches: ['main', 'release/.*']
+  path_filters:
+    - '!**/dist/**'
+    - '!**/dist-standalone/**'
+    - '!**/dist-cdn/**'
+    - '!**/playwright-report/**'
+    - '!**/test-results/**'
+    - '!**/package-lock.json'
+    - '!packages/bruno-shared/src/styles/theme.generated.css'
+  path_instructions:
+    - path: '**/*'
+      instructions: |
+        This repo renders a Bruno API collection as docs with a playground that sends real
+        requests. Reason as an API client would (requests, responses, auth, headers, bodies,
+        environments), not as a document viewer. CODING_STANDARDS.md is the source of truth
+        and ARCHITECTURE.md says who owns what. Its core rule: the simplest code that works,
+        abstractions only when required. Don't restate ESLint style; the pre-commit hook fixes
+        it. Focus on correctness, root cause over symptom, the simplicity smells, missing
+        tests, dead code left behind and comments that narrate the change.
+        Three packages: bruno-shared, bruno-api-playground, bruno-api-docs. Boundary: shared
+        imports neither of the others; playground imports only shared; docs only type-imports
+        the playground, apart from the one lazy import() in
+        packages/bruno-api-docs/src/components/Playground/load.ts. Cross-package imports use
+        package names; `@/` exists only in docs. Flag code placed in the wrong package: if
+        both need it, shared; anything that runs requests or scripts, playground.
+    - path: 'packages/bruno-shared/**'
+      instructions: |
+        Shared code used by docs and the playground (session, document, routing, theme, ui,
+        shared components, test utils). Flag:
+        - Any import from @usebruno/api-docs or @usebruno/api-playground, or a relative path
+          into their folders.
+        - An `@/` import; shared uses relative imports or its own package name.
+        - Code only one package uses; it belongs in that package.
+        - A new React context or module-level state the playground reads, without adding it to
+          the externals in packages/bruno-api-playground/vite.config.cdn.ts and Bruno.__rt in
+          packages/bruno-api-docs/src/components/Playground/loadCdn.ts. Without that the CDN
+          build gets a second copy and the playground reads a different context.
+        - A single barrel that re-exports everything; it makes Monaco load eagerly.
+    - path: 'packages/bruno-api-playground/**'
+      instructions: |
+        The playground: PlaygroundBody and below, the runner and QuickJS scripting. It is
+        loaded lazily, as an ES module in dev and as the classic script
+        api-playground-<hash>.js in the CDN build. Flag:
+        - Any import from @usebruno/api-docs or a relative path into packages/bruno-api-docs.
+        - An `@/` import; the playground uses relative imports or package names.
+        - A change to the externals in vite.config.cdn.ts without the matching Bruno.__rt
+          entry in packages/bruno-api-docs/src/components/Playground/loadCdn.ts, or the other
+          way round.
+        - State that must outlive the playground kept inside it instead of in the shared
+          session or workspace.
+    - path: 'packages/bruno-api-docs/**'
+      instructions: |
+        Docs view, the playground dock shell, the standalone CDN entry and all e2e. Flag:
+        - A value import of @usebruno/api-playground. Only types may be imported; the one
+          value import is the lazy import() in src/components/Playground/load.ts.
+        - Anything that makes the playground load before Try or a deep link.
+        - Changes to the public Bruno.apiDocs API or the bundle file names.
+        - Docs code that runs requests or scripts; that belongs in the playground.
+    - path: 'packages/bruno-shared/src/{session,observable,document}/**'
+      instructions: |
+        State rules from CODING_STANDARDS.md §State. Flag:
+        - A fact with two owners, or a copy of state that could be derived during render.
+        - An updater that mutates, rebuilds untouched branches, or returns a new object on a
+          no-op. Updaters are pure `(state, payload) => state`.
+        - A storage key written outside the owner that persists it, or a renamed key.
+        - Anything that makes the document writable after load.
+        - A new store library, generic store layer or option nobody passes.
+    - path: 'packages/*/src/components/**'
+      instructions: |
+        React rules from CODING_STANDARDS.md §React and §Styling. Flag:
+        - A leaf reading the session or workspace directly instead of taking props and callbacks.
+        - An effect that computes something derivable during render.
+        - A handler reading live state at click time instead of a value from render.
+        - A context hook that doesn't throw outside its provider.
+        - A component doing two jobs, or a near-duplicate of something in a ui or components
+          folder of any package.
+        - Compound components where there are no real slots.
+        - Colours or fonts not from `--oc-*` CSS variables.
+    - path: 'packages/bruno-api-docs/e2e/**'
+      instructions: |
+        Follow packages/bruno-api-docs/e2e/README.md and CODING_STANDARDS.md §E2E. Flag:
+        - Locators not by data-testid, or selectors inlined in specs instead of page objects.
+        - `new` in specs; expects inside page objects or components.
+        - `page.waitForTimeout`, `page.pause()`, `test.only`.
+        - Serial mode without a stated reason.
+        - Assertions that are conditional or check a substring the fixture already has.
+    - path: 'packages/bruno-api-playground/src/{runner,scripting}/**'
+      instructions: |
+        This code builds real HTTP requests and runs user scripts in a QuickJS sandbox. Check
+        it against what the Bruno desktop app does for the same request. Remember:
+        - Interpolation and auth are separate stages; a change to one doesn't cover the other.
+        - Only basic, bearer, apikey and digest reach the wire. awsv4, ntlm and wsse are
+          silently dropped. Flag code or tests that assume otherwise.
+        - HTTP is plain browser fetch; every network failure is one TypeError.
+        - Treat interpolated values, headers, URLs and script output as untrusted: header or
+          URL injection, new host capabilities exposed to scripts, secrets in logs or errors.
+        - `x || default` where an empty token, header or param is a real state.
+  pre_merge_checks:
+    custom_checks:
+      - name: 'No abstraction without a second user'
+        mode: 'warning'
+        instructions: |
+          Fail if the PR adds an interface or factory with one implementation, a wrapper that
+          only forwards, a helper with one caller, an option no caller passes, or an export
+          nothing imports. Pass if each new abstraction has at least two real users in the
+          code after this PR.
+      - name: 'Package change has a changeset'
+        mode: 'warning'
+        instructions: |
+          If the PR changes files under packages/*/src, it must add a markdown file under
+          .changeset/ for @usebruno/api-docs. Fail if it doesn't. Pass for PRs that only touch docs, tests,
+          CI or tooling.
+      - name: 'Behaviour change has a test'
+        mode: 'warning'
+        instructions: |
+          If the PR changes what the app does (new branch, default, UI state, bug fix), it must
+          add or change a unit spec or an e2e spec that would fail without the change. Fail if
+          a behaviour change has no test. Pass for pure refactors with existing coverage.
+      - name: 'No renamed storage keys or public names'
+        mode: 'warning'
+        instructions: |
+          Fail if the PR renames or removes the storage keys oc-docs.theme, oc-docs.env,
+          oc-docs:docsSidebarWidth, oc-docs:playgroundSidebarWidth or any other oc-docs key,
+          the .oc-playground class, the dist bundle file names, or the Bruno.apiDocs API
+          (updateCollection, updateOptions, destroy).
+      - name: 'E2E conventions'
+        mode: 'warning'
+        instructions: |
+          For changed files under packages/bruno-api-docs/e2e: fail on page.waitForTimeout,
+          test.only, page.pause, locators not by data-testid, expects outside specs, or serial
+          mode without a comment saying why.
+
+chat:
+  auto_reply: true

.github/workflows/ci.yml

@@ -38,7 +38,9 @@ jobs:
 
       - run: npm run lint
 
-      - run: npm run test:run -w packages/bruno-api-docs
+      - run: npm run typecheck --workspaces --if-present
+
+      - run: npm run test:run --workspaces --if-present
 
       # `draft` is null on push events, so that arm is required.
       - name: Build

.gitignore

@@ -2,11 +2,13 @@ node_modules
 dist
 .DS_Store
 dist-standalone
+dist-cdn
 test.js
 
 # Claude Code
 .claude/
 CLAUDE.md
+!/CLAUDE.md
 test-results
 !integrations/contract-tests/fixtures/walk-safety/node_modules/
 integrations/dist-tarballs

AGENTS.md

@@ -0,0 +1,55 @@
+# AGENTS.md
+
+`@usebruno/api-docs` renders a Bruno OpenCollection as API docs with a playground that sends
+real requests. It ships as a CDN bundle (`Bruno.apiDocs(target, { content })`), not on npm.
+The code lives in three workspaces under `packages/`; the repo root holds lint, changesets and CI.
+
+**Read `ARCHITECTURE.md` before changing code. Follow `CODING_STANDARDS.md`.** Its core rule:
+the simplest code that works, abstractions only when required.
+
+## Where code goes
+
+- `packages/bruno-api-docs` (`@usebruno/api-docs`): docs pages, sidebar, search, the playground
+  dock shell, the standalone and dev entries, all e2e.
+- `packages/bruno-api-playground` (`@usebruno/api-playground`, private): `PlaygroundBody` and below,
+  the runner, QuickJS scripting. Anything that runs requests or scripts goes here.
+- `packages/bruno-shared` (`@usebruno/api-shared`, private): session, document, routing, theme,
+  styles, ui primitives, shared components, test utils. If docs and playground both need it, it goes here.
+
+Boundary (lint): shared imports neither; playground imports only shared; docs only type-imports
+the playground, apart from the one lazy `import()` in
+`packages/bruno-api-docs/src/components/Playground/load.ts`. Cross-package imports use package
+names. `@/` exists only in docs.
+
+## Commands
+
+From the repo root:
+
+| Task | Command |
+|---|---|
+| Unit tests, all packages | `npm run test:run` |
+| Unit tests, one package | `npm run test:run -w @usebruno/api-shared` (or `@usebruno/api-playground`, `@usebruno/api-docs`) |
+| Lint changed lines | `npm run lint` (whole tree: `npm run lint:all`) |
+| Changeset | `npm run changeset` |
+
+From `packages/bruno-api-docs`:
+
+| Task | Command |
+|---|---|
+| Dev server | `npm run dev` |
+| E2E, main + standalone | `npm run test:e2e` |
+| E2E, standalone only | `npm run test:e2e:standalone` |
+| One e2e spec | `npx playwright test e2e/tests/<area>/<name>.spec.ts` |
+| Build | `npm run build`, `npm run build:standalone` (builds the playground file first) |
+
+## Rules agents break most
+
+1. No new abstraction (interface, factory, wrapper, option, helper) without a second real use.
+2. PRs target the active `release/vX.Y.0` branch, never `main`.
+3. A change to any package needs a changeset (`npm run changeset` at the root).
+4. Locally run only the e2e specs your change touches. CI runs the full suite.
+5. Never rename storage keys (`oc-docs.theme`, `oc-docs.env`, `oc-docs:*`), the
+   `.oc-playground` class or bundle names.
+6. Zero comments by default. Only what the code cannot say.
+7. New files: components `PascalCase.tsx`, everything else in `src` `camelCase.ts`, e2e `kebab-case`.
+   Don't copy a neighbouring file's naming.