Skip to content

Refuse breeze start dev mode when UI development ports are already in use - #71241

Open
Andrushika wants to merge 1 commit into
apache:mainfrom
Andrushika:fix-breeze-dev-ui-port-conflicts
Open

Refuse breeze start dev mode when UI development ports are already in use#71241
Andrushika wants to merge 1 commit into
apache:mainfrom
Andrushika:fix-breeze-dev-ui-port-conflicts

Conversation

@Andrushika

Copy link
Copy Markdown
Contributor

Why

I was using Git worktrees and accidentally ran breeze start-airflow --dev-mode from a second worktree. Breeze showed no warning, but the browser only showed a blank page.
Root cause: the old Vite process still owned port 5173 or 5174, so the new Vite process exited silently because of --strictPort while Breeze continued starting Airflow and the browser kept loading assets from the old Vite server.

What

Check both ports before starting the background UI dev hook, and fail with a clear message when either port is in use. This only applies to dev asset compilation; regular UI asset builds are unchanged.

This PR only handles port conflicts that already exist when run_compile_ui_assets starts. compile_ui_assets_dev.py still does not keep the Popen handles, so it cannot detect a Vite process that exits later. I plan to handle process supervision in a follow-up PR.


Was generative AI tooling used to co-author this PR?
  • Yes — Codex (GPT-5) and Claude Code (Fable 5)

Generated-by: Codex (GPT-5) and Claude Code (Fable 5) following the guidelines

Starting dev mode while an older Vite server owns the required ports can leave Breeze serving assets from another worktree. Failing early makes the conflict visible instead of starting a mismatched environment.
@Andrushika Andrushika changed the title Breeze: Fail when UI development ports are already in use Fail in breeze start dev mode when UI development ports are already in use Aug 6, 2026
@Andrushika Andrushika changed the title Fail in breeze start dev mode when UI development ports are already in use Refuse breeze start dev mode when UI development ports are already in use Aug 6, 2026
@Andrushika
Andrushika marked this pull request as draft August 6, 2026 17:42
@Andrushika
Andrushika marked this pull request as ready for review August 6, 2026 17:42
@Andrushika Andrushika closed this Aug 6, 2026
@Andrushika Andrushika reopened this Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant