Uh oh!
There was an error while loading. Please reload this page.
fix(onboarding): UX polish — build mode, domain-neutral descriptions, editable handoff - #512
Merged
Merged
Conversation
…rminal handoff, split research questions
- default_agent: plan → build (onboarding needs tool execution from the start)
- Add choice_descriptions to score schema + compiler; intent options now carry
explicit domain-neutral descriptions so the LLM doesn't invent quantum-specific ones
- Handoff stage is now a terminal message ('Onboarding finished! Please start a
new session to begin.') with no choices
- Split research_area into two back-to-back questions: 'What research areas?'
and 'What kind of experiments?'
- Regenerated golden files…andoff The handoff stage now presents the auto-generated description in an editable text field (using the new question tool `default` field from the opencode fork) so the user can review and modify before it's saved. Requires: harmoniqs/opencode@question-text-default (adds `default` field to question schema for pre-filled text inputs).
Uh oh!
There was an error while loading. Please reload this page.
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThe PR changes new-session defaults from ChangesOnboarding updates
Estimated code review effort: 2 (Simple) | ~10 minutes Sequence Diagram(s)sequenceDiagram
participant Overture onboarding
participant Score compiler
participant OpenCode session
Overture onboarding->>Score compiler: submit intent and research answers
Score compiler->>OpenCode session: generate interview instructions
OpenCode session->>OpenCode session: save edited description and completion marker
OpenCode session->>OpenCode session: start a new build session
Suggested reviewers: ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
aarontrowbridge added a commit
that referenced
this pull request
Aug 23, 2026
…pest domain (#515) The template opened 'You are Amico — Amicode's pulse-design copilot' while the same prompt's error-corrected-loop section said 'research studio, not a single-purpose copilot' and the overture had already been rewritten to 'coding and research companion' (tests assert the pulse phrasing is gone). The identity block was the straggler — and it mis-framed every session for users whose field isn't pulses. New identity: autoresearch copilot — the loop first (campaigns, hypotheses, spec gates, experiments, mechanical verdicts) plus the dev work it needs (issues, PRs, skills, fleet ops), adapting to the user's field from recorded state (profile, mounts, memory). Quantum control stays as the deepest domain (Piccolo pulses, Run Inspector) explicitly as the first domain pack, not the boundary — QEC, other physics, anything modelable, same loop. Voice section follows ('done research with this user', not 'pulse design with this researcher'). Also fix-forwards main's typecheck (red from #503/#512): removes the orphaned armOnboardingSession (its caller died with the splash transition), an unused writeWelcomeShown import, and a dead enterGroup const in the splash animation. Version 0.2.5 -> 0.2.6 (the vsix ships this AGENTS.md). Closes#515.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Three onboarding UX fixes plus one new capability:
1. Sessions start in build mode (not plan)
Onboarding needs tool execution from the start — the interview calls
amicode_profile,question, etc. Opening in plan mode forced users to manually switch before anything worked.2. Domain-neutral option descriptions on intent choices
Added
choice_descriptionsto the score schema so the compiler renders explicit descriptions alongside option labels. The LLM uses these verbatim instead of inventing quantum-specific ones from the surrounding context.3. Handoff: editable description confirmation (not a dead-end question)
The handoff stage now presents the auto-generated "About you" description in a pre-filled editable text field (via the new
defaultfield on the question tool). The user can accept or edit before submitting.Old choices ("Let's dive into my first task" / "Show me around first") are removed — they had no downstream behavior.
4. Split research_area into two questions
"What research areas?" and "What kind of experiments?" are now asked back-to-back instead of as one combined question.
Depends on
defaultfield to the question tool schema)Verification
Summary by CodeRabbit