fix(ui): mount ConfigGeneratorYamlPreview in the config generator panel - #6371
Conversation
Wire the already-tested YAML preview into ConfigGeneratorPanel so live formState is visible as operators fill the form. Closes JSONbored#6183. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Tip ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-16 05:17:21 UTC
Review summary Nits — 4 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionAddressed Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. Visual preview
Click any thumbnail to open the full-size screenshot. Before = production · After = this PR's preview deploy. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by LoopOver, a quiet PR intelligence layer for OSS maintainers.
|


Summary
ConfigGeneratorYamlPreviewinsideConfigGeneratorPanel, wired to the same liveformStatethe field groups already update.ConfigGeneratorYamlPreviewitself — mount-only fix for fix(ui): config generator never mounts ConfigGeneratorYamlPreview, so it never shows the user the YAML it generates #6183.Scope
apps/loopover-ui/Note: this issue carries the
visuallabel (owner-led). Expect the loopover gate to hold for owner review rather than auto-merge.Validation
npm --workspace @loopover/ui run test -- src/components/site/app-panels/config-generator-panel.test.tsx— 2 passedapps/**Safety
UI Evidence
combine: consensus)Closes #6183