Feat(hub): add ui native setup wizard - #192
Merged
Merged
Conversation
Share setup phases and provider commands between CLI and Hub, run Claude Code and Codex population in owned background sessions, and stream bounded assistant text with compact tool activity. Add exact setup-file diff review and local commits that preserve unrelated staging, reject stale reviews, and promote the existing session into Hub. Render readable numbered additions/deletions with bounded raw fallback. Cover setup, streaming, Git transactions, browser flows and packaged assets; document the explicit local-commit policy and retain separate setup budgets.
…ze errors Hub setup populated scaffolds without grounding and then failed at finalize: - Headless Claude ran with acceptEdits only, so every `mex graph` call was denied at an approval prompt no one could answer. Pre-approve only the read-only graph, impact, and event-log commands, for both the Bash and PowerShell tools (Windows sessions use PowerShell). - The population prompt showed its anchor example as a live Markdown link and told agents to retain the rule in AGENTS.md. A verbatim copy left mex://<exact-node-id>, which grounding capture rejects. Show the example as inline code instead. - The Hub replaced every finalize failure with a generic message. Finalize now throws SetupFinalizationError with authored text naming the unverifiable reference, and the runner shows it.
…tory The headless allow-list comment claimed Git commands need a person. A probe with the original launch arguments showed `git log`, `git show`, and `git blame` already run; only `mex` commands were denied.
…ommitted The setup commit review returned every file's diff inline, capped at 128 Ki characters in total. A normally populated scaffold for a small app is about 185 KB, so the guided commit fell back to manual on ordinary projects. The preview now lists only per-file metadata (status, line counts, diff size). Each diff is served on expand by POST /api/v1/setup/commit/diff from the text retained with the reviewed snapshot; it runs no Git and answers only for the current, unexpired revision. Per-file review text is 128 Ki characters and the retained review total is 1 Mi. A truncated diff still blocks the in-app commit, and a blocked review stays readable to guide the manual commit. The file, count, and Git-output caps are unchanged.
The setup page's grid tracks defaulted to min-width auto, so an open diff with long unwrapped lines widened the whole card. The card clips overflow, which hid the right-aligned Refresh review and Commit setup and open Hub buttons until every diff was closed. Constrain the page, body, and review grids to minmax(0, 1fr) so wide rows scroll inside their own diff instead. The browser test now opens every diff with a long line in the fixture and checks that the card has no hidden overflow and both actions stay inside it.
theyashasvipandey
approved these changes
Sep 11, 2026
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
What
Why
Type of change
How to test
Checklist
npm test)Code-graph changes
mainLanguageExtractororFrameworkResolverinterfacecore / discuss-firstissue is linked above