Skip to content

fix(frontend): drop streaming status chrome from LocalNet create UI - #293

Merged
zheli merged 5 commits into
mainfrom
zzz/8d0a15ec
Jul 30, 2026
Merged

fix(frontend): drop streaming status chrome from LocalNet create UI#293
zheli merged 5 commits into
mainfrom
zzz/8d0a15ec

Conversation

@zheli

@zheli zheli commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove implementation-detail streaming status text and the running banner from the LocalNet create UI, and rename the in-flight cancel button to Cancel.

Changes

  • Replace the create-modal progress subtitle that exposed /api/instances/.../events
  • Remove the running-state banner box ("streaming step events")
  • Drop the footer · streaming suffix
  • Rename Cancel bring-up to Cancel in the create modal and creating panel

Test plan

  • make test
  • make lint
  • scripts/e2e/run-all.sh
  • Manual: open Create LocalNet, start bring-up, confirm no streaming banner/subtitle and Cancel label

Checklist

  • Tests pass locally
  • No coverage regression on touched code
  • Docs updated (and website/ synced if any mirrored docs/*.md changed)
  • CLI ↔ Web UI parity maintained (or follow-up issue + TODO(#issue) comment)

zheli added 4 commits July 29, 2026 21:10
Hide implementation-detail streaming copy and the running banner during bring-up, and shorten the cancel button to Cancel.
Vite's string proxy shorthand forced changeOrigin:true, so Host and Origin diverged and blocked LocalNet create from the frontend dev server. Keep changeOrigin false by default and add --insecure-skip-origin-check for other local-dev setups.
Drop the ready subtitle, green banner, instance footer, and elapsed timer so the finished bring-up view stays focused on the step list.
Clear terminal output and step state when the create-progress events URL changes so a second LocalNet create does not show logs from the prior instance.
@zheli
zheli marked this pull request as ready for review July 29, 2026 20:01
Unblocks CI typecheck: rerender({ url: null }) was rejected because initialProps narrowed url to string.
@zheli
zheli merged commit 43259e1 into main Jul 30, 2026
25 checks passed
@zheli
zheli deleted the zzz/8d0a15ec branch July 30, 2026 07:24
@bitdynamics-ab bitdynamics-ab deleted a comment from cursor Bot Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant