feature: interactive setup wizard by l0gicgate · Pull Request #14 · anomalyco/opencode

l0gicgate · 2025-05-15T15:30:18Z

Description

On first start, when .opencode.json does not exist, we prompt the user with the setup wizard:

Step 1: Select Provider

Step 2: Select Model

Step 3: Enter API Key

Demo

Remove .opencode and .opencode.json from your home directory
Run the app

Closes #7

adamdotdevin

this is really great, thank you so much! left some feedback/questions!

adamdotdevin · 2025-05-15T20:22:44Z

sorry for the delay @l0gicgate, just wanted to pull this down and run through it myself before merging; should have it merged first thing tomorrow!

l0gicgate · 2025-05-15T21:34:32Z

No worries @adamdottv, happy to address any other feedback you have before we merge!

adamdotdevin

pulled this down this morning and ran through it, a few changes needed:

if you hit ctrl+c you're able to exit the setup dialog and get into a weird state where there's nothing to do but close the app and restart
the list of providers/models should use the new simple-list component (it has vim navigation with j/k as well)
it's not clear to me what "API key" would be for some of the providers like Bedrock and Azure where you have to have AWS/Azure credentials configured in your environment for these to be setup correctly; not sure what we should do here, but we could just display a message when you choose one of those providers (instead of the API key form) that basically tells the user they need to ensure they have AWS/Azure creds setup and then restart the app
had an inline note about repurposing the provider friendly names

l0gicgate · 2025-05-16T20:31:56Z

@adamdottv I've applied all of your suggestions via 455e5c1, 71b017c, 4e47ad6

Can I follow up in a different PR to add support for providers that use credentials? The scope of this PR is already massive.

adamdotdevin · 2025-05-16T22:35:44Z

can we just remove bedrock and azure from that dialog in the interim?

adamdotdevin · 2025-05-17T15:22:48Z

i actually forgot to delete the existing folder/file, so didn't hit the setup wizard, just went straight into a previously saved session and saw the "thinking..." indicator

l0gicgate · 2025-05-17T17:55:50Z

@adamdottv okay I was able to repro and find the bug!

Fixed via 9e274d0

l0gicgate · 2025-05-20T15:49:32Z

@adamdottv is it safe to assume that this PR will be superseded by #38? I will close if so

thdxr · 2025-05-21T20:10:47Z

@l0gicgate that PR is overhauling the core of opencode to be powered by bun. the TUI is still going to be in go - simple server client model.

this PR will probably have to change but we'll keep the TUI parts

l0gicgate · 2025-05-21T23:13:50Z

Okay cool @thdxr, thanks for the update! Let me know what changes need to be done and I’ll address right away.

thdxr · 2025-06-14T03:28:45Z

closing as we rewrote the codebase - appreciate it though!

- Issue anomalyco#14: SDK cache now limited to 50 entries with LRU eviction - Issue anomalyco#15: Languages cache now limited to 100 entries with LRU eviction - Issue anomalyco#16: Instance cache now limited to 20 entries with LRU eviction Creates new createLruCache utility in src/util/cache.ts with: - maxEntries limit for bounded cache size - LRU eviction policy when limit is reached - Optional onEvict callback for cleanup

add refresh gesture to chat window

The previous flow took the three ORT sessions out of self, ran the preprocess/encode/decode pipeline in a closure, and restored them back only on normal return. Any .unwrap() panic mid-pipeline left self with three None sessions, so the next transcribe() hit "Not loaded" and the user had to wait ~1 s for ONNX to reload ~700 MB from disk. Wrap the body in std::panic::catch_unwind(AssertUnwindSafe(...)) and keep the sessions in local Options that we restore unconditionally before returning — even on unwind. The panic payload is stringified so the caller gets a meaningful error instead of a hard abort. Closes anomalyco#14/21.

Fixes all 17 Copilot comments: Telemetry opt-out (#1, #2, #3, #9, #16): - tracker.ts now reads FINNY_TELEMETRY=0 / OPENCODE_TELEMETRY=0 at module load, so the opt-out applies in every entrypoint (TUI worker, server, CLI), not just the bootstrap path. - Email dialog auto-prompt and /subscribe slash command also gated on Analytics.isEnabled() — email POST goes to the same Convex deployment. Tracker correctness (#5, #14, #17): - drain() now also waits for the DeviceProfile.userId() resolution promise, not just the post-send promise. - worker rpc.shutdown() and main-thread thread.ts both call Analytics.drain() before tearing down — neither beforeExit nor SIGTERM fire on worker.terminate() / process.exit(0). - beforeExit handler runs at most once per process, preventing an infinite drain loop when a hung telemetry request later resolves and re-empties the loop. Privacy (#10, #11, #12): - Drop process.argv from tui.worker.booted (could include --prompt text and project paths). - error.surfaced now sends only error.name; error messages frequently embed user prompt content and local file paths. - Don't persist the submitted email address to local KV (no reader, just unnecessary PII retention). Email dialog state machine (#4, #6, #8): - Wait for kv.ready before evaluating the prompt gate so we don't re-show the dialog to users who already submitted/skipped. - Conditional dismiss handler — only marks "skipped" when no terminal status has been set, so a successful submit isn't overwritten by the dismiss callback that fires on dialog.clear() and isn't suppressed by another dialog replacing this one. Server-side validation (#15): - subscribe mutation now rejects empty / oversized / regex-failing emails with a ConvexError, so the public Convex client can't bypass the TUI regex. Command coverage (#13): - Hook Analytics.track() in DialogCommand.trigger() instead of the bus event handler. trigger() is the central activation point for palette, slash, keybind, plugin, and bus paths — single hook, full coverage, no double-counting. Comment wording (#7): - bootstrap.ts: "opt-in by default" → "enabled by default" (correct description of opt-out semantics). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

The previous flow took the three ORT sessions out of self, ran the preprocess/encode/decode pipeline in a closure, and restored them back only on normal return. Any .unwrap() panic mid-pipeline left self with three None sessions, so the next transcribe() hit "Not loaded" and the user had to wait ~1 s for ONNX to reload ~700 MB from disk. Wrap the body in std::panic::catch_unwind(AssertUnwindSafe(...)) and keep the sessions in local Options that we restore unconditionally before returning — even on unwind. The panic payload is stringified so the caller gets a meaningful error instead of a hard abort. Closes anomalyco#14/21.

feat: add non-invasive in-app updater

…as/refiner UI fixes: - anomalyco#15 Remove the gear-icon "Tweaks" popover (theme + collapse) — user reported it as useless chrome. Theme falls back to saved pref / system, rail collapse stays on the rail's own chevron toggle. - anomalyco#7 Task names now follow each session's title (auto-generated or user- renamed). Root cause: `newWorkflowTask` was pre-setting `title: "Workflow"` on session.create, which blocked the backend's title agent from auto-generating (the agent only fires when title matches the default ISO placeholder). New sessions now leave title unset → ensureTitle fires → each task gets a distinct name. Rail and Tasks drawer show "Untitled" for sessions that still have the default placeholder; node-tab labels prefer the bound child session's title. - Rail's sub-task list capped at 8 visible rows with internal scroll (was squeezing all tasks into the visible rail height). - anomalyco#14 Per-slave model picker: clear `local.model` before opening the picker so picking the SAME model as master still routes (previously the on(key) effect required a key change to fire, so same-model picks silently no-op'd — the user's "选其他不生效" complaint). - anomalyco#10 Canvas bottom-half occluded: `.wf-canvas-host` was `position: relative` only, so its `flex: 1` child `.wf-r2-canvas` had no flex parent to grow into. Made the host a flex column so the canvas actually fills its allotted height. - anomalyco#11 React production error anomalyco#409 fixed by un-nesting the drill-in button: the outer node card was a `<button>` wrapping an inner drill-in `<button>` — invalid HTML that React surfaces as anomalyco#409. Switched the outer to `div role="button"` with Enter/Space handling. Refiner: - anomalyco#13 Refiner no longer observes user messages addressed to the orchestrator agent (workflow planning commands aren't generalisable experiences). Gate added in `extractUserMessage`. - anomalyco#5 Archived experiences are now discoverable: new "已归档 (N)" chip in the Knowledge substrip when N > 0, toggles `includeArchived`. Archived rows already had an "Unarchive" button in the card menu — this just makes finding them obvious. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

l0gicgate added 6 commits May 15, 2025 01:55

fix: typo

6fad2f6

fix: add initialization logic for setup dialog

25bac47

fix: split out bedrock models

92fac03

feat: add setup complete and update logic

1a99269

feat: add setup dialog

eff2322

fix: sort models by alphabetical order

d4648c2

l0gicgate commented May 15, 2025

View reviewed changes

Comment thread internal/tui/components/chat/editor.go

l0gicgate commented May 15, 2025

View reviewed changes

Comment thread internal/llm/agent/setup-agent.go

adamdotdevin requested changes May 15, 2025

View reviewed changes

Comment thread internal/tui/tui.go Outdated

Comment thread internal/tui/tui.go Outdated

Comment thread internal/llm/models/models.go Outdated

Comment thread internal/config/config.go Outdated

l0gicgate added 2 commits May 15, 2025 11:02

fix: add api key validation & blinking

735ddc5

refactor: extract setup in module

98de5ba

l0gicgate requested a review from adamdotdevin May 15, 2025 17:32

fix: tweak styles

08df5ec

l0gicgate mentioned this pull request May 15, 2025

remove: Initialize Project Dialog #21

Closed

l0gicgate added 2 commits May 15, 2025 17:31

refactor: render help

63d45c4

fix: q -> s

3a68df5

l0gicgate changed the title ~~feature: Setup Wizard~~ feature: interactive setup wizard May 16, 2025

adamdotdevin requested changes May 16, 2025

View reviewed changes

Comment thread internal/tui/components/dialog/setup.go Outdated

l0gicgate and others added 5 commits May 16, 2025 10:49

Merge branch 'sst:dev' into feature/setup-dialog

cdd2299

fix: move model/provider utils

455e5c1

fix: show setup dialog again after quit attempt

71b017c

refactor: use simple-list component

4e47ad6

Merge branch 'dev' into feature/setup-dialog

90808bb

l0gicgate requested a review from adamdotdevin May 16, 2025 20:32

fix: show setup dialog when no selected in quit dialog

6110a0c

fix: remove debug

d18e33b

l0gicgate and others added 2 commits May 17, 2025 09:38

Merge branch 'sst:dev' into feature/setup-dialog

7d53155

fix: move setup initialization

9e274d0

l0gicgate added 2 commits May 18, 2025 17:02

Merge branch 'sst:dev' into feature/setup-dialog

f417494

Merge branch 'dev' into feature/setup-dialog

d55d7af

l0gicgate added 2 commits May 24, 2025 00:26

fix: auto initialize project

fabf639

fix: auto initialize project

9b81d7b

thdxr closed this Jun 14, 2025

github-actions Bot mentioned this pull request Nov 4, 2025

feature: uninstall command #3900

Closed

1 task

ramses-lopez mentioned this pull request Feb 28, 2026

opentui: fatal: The "path" property must be of type string, got number #15465

Closed

BillSteinUNB pushed a commit to BillSteinUNB/whispercode that referenced this pull request Mar 7, 2026

Merge pull request anomalyco#14 from DNGriffin/dg/pull-to-refresh

cedb90b

add refresh gesture to chat window

coolbossov mentioned this pull request Mar 19, 2026

"Extra usage is required for long context requests" error still occurs after Anthropic removed long-context premium (March 13) #18242

Open

ramses-lopez mentioned this pull request Apr 10, 2026

opentui: fatal: The "path" property must be of type string, got number #14279

Open

kitlangton mentioned this pull request Apr 30, 2026

docs(httpapi): update migration spec to current state #25173

Merged

1 task

This was referenced May 12, 2026

feat: recover auto-review feature (supervisor + cross-review) OpenCodeEngineer/opencode#2

Closed

feat: recover auto-review feature (supervisor + cross-review) OpenCodeEngineer/opencode#3

Closed

bussard76 pushed a commit to bussard76/openwork that referenced this pull request May 12, 2026

Merge pull request anomalyco#14 from different-ai/feat/live-update

632e4ee

feat: add non-invasive in-app updater

Conversation

l0gicgate commented May 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Demo

Uh oh!

Uh oh!

Uh oh!

adamdotdevin left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

adamdotdevin commented May 15, 2025

Uh oh!

l0gicgate commented May 15, 2025

Uh oh!

adamdotdevin left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

l0gicgate commented May 16, 2025

Uh oh!

adamdotdevin commented May 16, 2025

Uh oh!

adamdotdevin commented May 17, 2025

Uh oh!

l0gicgate commented May 17, 2025

Uh oh!

l0gicgate commented May 20, 2025

Uh oh!

thdxr commented May 21, 2025

Uh oh!

l0gicgate commented May 21, 2025

Uh oh!

thdxr commented Jun 14, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

l0gicgate commented May 15, 2025 •

edited

Loading