fix(deploy): resolve infinite re-render in DeployControl

[jidechao]

Summary

• Fix "Maximum update depth exceeded" crash caused by unstable [] fallback in the deployments Zustand selector in DeployControl
• Replace with a module-level stable empty array constant (EMPTY_DEPLOYMENTS), matching the existing pattern used for EMPTY_LOG and EMPTY_STATS in preview-pane.tsx

Root Cause

useStore((s) => selectActiveTask(s)?.deployments ?? []) creates a new [] reference on every render. Zustand detects the reference change, re-renders the component, which creates another new [], causing an infinite loop.

Test plan

• Run pnpm dev and open the app
• Verify no "getSnapshot should be cached" or "Maximum update depth exceeded" errors in the browser console
• Verify DeployControl still renders correctly after a Convert completes
• Verify deployment history dropdown still works

[lefarcen]

Heads-up: PR #33 is already open against the same DeployControl selector loop. Both PRs touch src/components/deploy-control.tsx and replace the freshly allocated deployments ?? [] fallback with a shared empty deployments array.

You and @cszhouwei may want to compare approaches; the maintainer team will decide which one lands. Sharing this so neither effort gets wasted.

[mrcfps]

@jidechao Reviewed the DeployControl selector fix and the new EMPTY_DEPLOYMENTS fallback; this matches the stable selector pattern already used for EMPTY_LOG and EMPTY_STATS, so the changed range looks correct for stopping the render loop when a task has no deployments. Thanks for jumping on this bugfix so quickly 🙌

_{🔁 Powered by Looper · runner=reviewer · agent=opencode · An autonomous AI dev team for your GitHub repos.}

[lefarcen]

Adding one more coordination note: PR #42 is now also open against this same DeployControl selector loop. It touches src/components/deploy-control.tsx and uses the same stable empty deployments fallback direction as #33 and this PR.

The maintainer team will choose which focused fix lands; noting the overlap so everyone has the latest picture.

[joeylee12629-star]

Superseded by #33 (squash-merged). Same fix landed via @cszhouwei's PR with the cleanest implementation (readonly module-scope constant). Thanks for the contribution!