13 KiB
Priorities
- Prioritise, in this order: stability, simplicity, performance.
- Before changing session or timeline code, record a production benchmark baseline and compare it after the change.
Debugging
- NEVER try to restart the app, or the server process, EVER.
Local Dev
opencode dev webproxieshttps://app.opencode.ai, so local UI/CSS changes will not show there.- For local UI changes, run the backend and app dev servers separately.
- Backend (from the repository root):
bun dev serve --port 4096 - App (from
packages/app):bun dev -- --port 4444 - Open
http://localhost:4444to verify UI changes (it targets the backend athttp://localhost:4096).
SolidJS
- Always prefer
createStoreover multiplecreateSignalcalls
Suspense
A flash, a blanked area, a dialog that resizes or a reset scroll on first open almost always means a read entered Suspense. Do not ship UI that reads async data without the guard below.
-
Reading
resource()suspends while it loads. So doesresource.latestuntil the resource has resolved once: before the first value,latestbehaves like a plain read.resource.loadingandresource.statenever suspend. -
Guard every render-time read on state, and show the data you already have until then:
const loaded = () => resource.state === "ready" || resource.state === "refreshing" const items = () => (loaded() ? resource.latest : cached) ?? []Precedent:
src/new-session/workspace/controller.ts(worktreesLoaded). -
When the source can change (a drilled project, a selected session), include its key in the fetched value and use the result only when the key matches. Otherwise
latestshows the previous key's data while the next loads. -
A
<Suspense>boundary only chooses what blanks; it does not prevent the suspension. Without a local boundary, the nearest one is often a whole dialog or route. -
Suspend only on purpose, on first load, with a comment that says why.
Typography
- Use
--line-height-compact(16px) for13pxcompact UI text and--line-height-base(20px) for body text. - Do not use
leading-none,line-height: 1, or a13pxline height for normal text. Inter descenders clip inside truncation and overflow containers. - Keep control and row heights explicit. Fix font metrics directly rather than using transforms, negative margins, or clip-padding compensation.
Fonts
Users choose three fonts in Settings > Appearance. Every surface uses one of these roles. This section is the single source for font rules across app, ui, session-ui, gui-extensions and desktop.
| Role | Use for | Use |
|---|---|---|
| UI font | Prose, labels, titles, controls | Inherit, or var(--font-family-sans), var(--v2-font-family-sans), font-sans |
| Code font | Shell commands, code, paths shown as code, diffs, logs, tool input | var(--font-family-mono), var(--v2-font-family-code), font-mono, or a <code> / <pre> element |
| Terminal font | Terminal output only | appearance.font("mono") from the extension SDK |
- Never write a literal family (
Inter,"IBM Plex Mono",monospace,Menlo,system-ui) in production CSS, classes, inline styles, canvasfontstrings or widget options. A literal ignores the user's setting. - Never add an
@font-face, font file, or font package outsidepackages/app/src/index.css. That file is the only place the app ships faces: Inter (variable), IBM Plex Mono Text (weights below 600), Bold (600 and up) and Text Italic, and JetBrainsMono Nerd Font Mono Regular and Bold. If a design needs a face that is not shipped, add it there and say why in the PR. - Show shell commands with the timeline shell metrics: code font,
13px, weight440,-0.04pxletter spacing. Do not invent another size for the same content. - Prefer weight tokens (
font-medium,var(--font-weight-medium)) over literal weights. Mono at600or more renders IBM Plex Mono Bold, so reserve it for real emphasis. - Do not read font settings directly.
src/settings/model.tsxwrites them to the CSS variables above, and the extension host passes the terminal font throughappearance.font.
Loading Indicators
This section is the single source for loading indicators across app, ui, session-ui, gui-extensions and desktop.
- The dot-matrix
Spinner(@opencode/ui/spinner, andSessionProgressIndicatorV2andActivityRing, which render it) means the AI is working: a running session, turn, subagent or tool call. Use it only for those states. - Every other loading state uses
Loader(@opencode/ui/loader): page, pane and file loads, browser reload, search, connecting, authenticating, sending, installing. Use a purpose-built indicator only when the state has one, such as the titlebar update ring. - Choose the indicator at each call site. Never restyle a shared indicator (
Loader,ActivityRing,Spinner) to look like another one; that silently changes every caller.
Scrolling
This section is the single source for scroll containers across app, ui, session-ui, gui-extensions and desktop.
- Scroll with a plain element:
overflow-y-auto(oroverflow-x-auto,overflow-auto) plusmin-h-0or a max height.packages/ui/src/styles/scrollbar.cssgives every native scrollbar the shared thumb. Do not addScrollViewto fix how a scrollbar looks. - Mark each scroll container with
data-scrollable. Nested-scroll handling (the timeline, Home,createAutoScroll) uses it to let the inner container scroll first. - Use one element when it both sizes and scrolls. Add an outer wrapper only for something that must stay put while the content scrolls, such as
::beforeand::afteredge fades: the wrapper isrelative flex flex-col min-h-0and owns the fades, and the inner element ismin-h-0 flex-1 overflow-y-auto. - A native scrollbar takes 8px from layout while the content overflows. Where text would visibly rewrap at that moment (the composer, settings search), add
scrollbar-gutter: stableand subtractvar(--scrollbar-size, 0px)from the end padding. - A ref runs before the element's children render and before it is in the document. Restore a scroll position from
onMountor a frame later, not in the ref. ScrollView(@opencode/ui/scroll-view) is only for the session timeline and Home, which draw the thumb in an external track (thumbContainer) and, in the timeline, correct it for virtualized rows. Use it again only for one of those needs.- Never set
scrollbar-width: thin,scrollbar-width: autoorscrollbar-color. Chromium then ignores the shared style and draws the native Windows scrollbar. Hide a scrollbar only withno-scrollbarorscrollbar-width: none.
Localization
- NEVER hardcode user-visible English strings in production code. ALWAYS use an i18n key for visible copy, placeholders, accessible labels, tooltips, menus, dialogs, toasts, empty states, and displayed errors.
- Feature work adds English source strings only. Leave non-English keys absent so the runtime English fallback applies; translations land separately after language review.
- Render count-sensitive copy only through
language.plural(baseKey, count, params). Never select or pass.zero,.one,.two,.few,.many, or.othervariants tolanguage.t(...). - When migrating existing copy to i18n, preserve the English text byte-for-byte unless the task explicitly requests a copy change.
- NEVER change existing English text or English keys to facilitate translation. English is intentional, designer-written source copy; adapt locale-specific translations and i18n mechanics around it.
- Keep locale complexity behind the shared typed i18n APIs. Feature and component code should use
language.t(...)for ordinary copy andlanguage.plural(baseKey, count, params)for count-sensitive copy. It must not inspect the locale, callIntl.PluralRules, construct or select plural-category keys such as.oneor.other, or branch on locale-specific grammar. - Do not compose translated count phrases or sentence fragments in feature components. Add a focused language-context operation that owns plural selection and composition when one UI concept needs coordinated keys.
- Prefer complete translated phrases. Do not concatenate grammatical fragments or make call sites assemble sentences. Keep placeholders to irreducible dynamic values such as names, paths, and counts.
- If a translation cannot be expressed by the current API, deepen the shared language/UI i18n module so one typed call owns locale selection, plural resolution, fallback, and interpolation. Do not leak that machinery into product code.
- Do not translate from model knowledge alone. Verify terminology and grammar with Unicode CLDR locale/plural data, Microsoft Localization Style Guides and terminology, Apple localization/style guidance and localized platform UI, Mozilla localization style guides, Mozilla Pontoon, and the Firefox localization corpus at
github.com/mozilla-l10n/firefox-l10n. - For developer-facing terminology, prefer the words already used by the target language's developer community over literal dictionary translations. Cross-check maintained localized developer products such as Firefox, KDE, and VS Code; use at least two independent corpora when they are available. If established practice keeps an English loanword or acronym, keep it rather than inventing a translation.
- Translate complete UI phrases in context. A glossary hit is evidence, not permission to translate word-by-word. Check terse labels such as session, prompt, agent, model, fork, shell, terminal, workspace, and worktree in the same grammatical role before choosing a term.
- Before a locale is ready, audit recurring concepts for one consistent translation and review every value that still equals English. Classify retained English as a product name, provider/tool name, URL, code token, keyboard legend, acronym, asset name, or established borrowing; translate unexplained leftovers.
- In translation review notes, name the corpora used and call out uncertain or region-specific terminology so native speakers can focus review where it matters.
- Also use the relevant language authority or official dictionary for the locale (for example RAE/Fundéu, FranceTerme, Duden, TDK, Kotus/Kielitoimiston sanakirja, Språkrådet/Bokmålsordboka, Rada Języka Polskiego/PWN, the Russian and Arabic language academies, the Ukrainian Orthography, Taiwan MOE dictionaries, or the Royal Society of Thailand). Treat the English dictionary as the semantic source of truth and preserve placeholders, code identifiers, product names, and keyboard labels.
Tests
A test must pay for its upkeep. Before adding one, answer three questions: which observable contract does it protect, which credible regression makes it fail, and why does no existing test already catch that regression. If any answer is missing, do not add the test.
- One contract, one owner. Test each behavior once, at the strongest boundary that observes it: the area's e2e keeper suite, or the owning module's unit test. Extend an existing table or keeper case before creating a file.
- Do not unit-test what an e2e keeper already proves, what a dependency does (for example
@pierre/trees), config that passes values through, or source text (import or string greps). Source text is fair game only when that text is the contract, such as a persisted key. - Never add production seams for tests: no test-only exports, parameters such as
now,delayorbudget, flags, globals, ordata-*attributes. Control time withsetSystemTimefrombun:testand drive the real entry point. - A test must be able to fail. Do not assert values the code under test produced, mock the behavior you assert, use soft assertions for the order the test guards, or leave
rejects/resolvesun-awaited. A regression test must fail on the code before the fix. - Do not repeat a case per theme, text direction, viewport or channel. Use one table, and only when the behavior differs.
- When you delete or move coverage, name the test that still proves the contract. When you move a contract, mutate its owner once and confirm the new test fails.
- A contract proven by a
packages/session-ui/component-teststest does not also get an app e2e copy.
Tool Calling
- ALWAYS USE PARALLEL TOOLS WHEN APPLICABLE.
Browser Automation
Use agent-browser for web automation. Run agent-browser --help for all commands.
Core workflow:
agent-browser open <url>- Navigate to pageagent-browser snapshot -i- Get interactive elements with refs (@e1, @e2)agent-browser click @e1/fill @e2 "text"- Interact using refs- Re-snapshot after page changes