fix(tui): detach reusable composer before clear() at remaining overlay open paths (jobs, tasks, oauth paste, /debug)

[Yeachan-Heo]

Composer terminal disposal remains reachable at four unguarded overlay open paths

Follow-up of #4604 / PR #4605. PR #4605 fixes the terminal-disposal defect at the five highest-traffic overlay paths (command palette / model selector / queued-message selector open and restore paths, plus pet-aware restores). Boundary review of that PR confirmed four pre-existing sibling paths with the identical bug still open on dev (27afb732b3):

Path	Reachable via	Site
Jobs overlay open	default alt+j keybinding, /monitors (also reachable from the command palette itself)	packages/coding-agent/src/modes/controllers/selector-controller.ts showJobsOverlay (~:3669)
Tasks pane open	default alt+t keybinding	selector-controller.ts showTasksPane (~:3699)
OAuth API-key paste login	API-key login for ~20 providers (packages/ai/src/utils/oauth/*)	selector-controller.ts onPrompt code input (~:3168)
/debug log + raw-SSE viewers	/debug	packages/coding-agent/src/debug/index.ts (~:348, :367)

Mechanism (identical to #4604): these opens call ctx.editorContainer.clear() while the live reusable CustomEditor is still attached. Container.clear() disposes children terminally, Editor.dispose() unregisters the tab-width change listener, and every later restore re-mounts a dead editor — runtime tab-width changes stop re-deriving composer layout after the first overlay round-trip. The close-path clears at ~:3163, ~:3657, ~:3685 self-heal once their opens detach.

Fix shape (same two lines per site): this.ctx.editorContainer.detachChild(this.ctx.editor); immediately before the open-path clear(), exactly as showSelector (:1344) and extension-ui-controller.ts (:1178) already do.

Not a regression from #4605 — all four sites predate it on the base commit; filed so the remaining surface is not lost. Regression coverage pattern to copy: the 4-cycle invalidation-count probe with red control in packages/coding-agent/test/gajae-pet-widget.test.ts and the repeated palette-cycle test in packages/coding-agent/test/command-palette-interactive-host.test.ts.

---

[repo owner's gaebal-gajae (clawdbot) 🦞]

[Yeachan-Heo]

Investigation and fix work has started in a dedicated current-dev lane. Scope is limited to the remaining overlay-open paths named in the issue (jobs, tasks, OAuth paste, and /debug), with focused reusable-composer lifecycle regressions and exact-head CI evidence before one PR is advanced.

—
[repo owner's gaebal-gajae (clawdbot) 🦞]

[Yeachan-Heo]

Fix lane is up: #4687 (branch fix/issue-4657-composer-detach, one commit f494d0671e on top of dev 416201eb5e).

Reproduced the defect on all four named paths with a real-CustomEditor/real-Container harness: each open-path clear() terminally disposed the composer and the tab-width invalidation probe went silent after the first round-trip. Verified red/green — 4/4 deterministic regressions fail on the unfixed tree, 4/4 pass with the fix:

• jobs overlay (alt+j / /monitors) → showJobsOverlay
• tasks pane (alt+t) → showTasksPane
• OAuth API-key paste → #handleOAuthLogin onPrompt code input (real production closure driven over a real Input)
• /debug log + raw-SSE viewers → real DebugSelectorComponent selection into the real DebugLogViewerComponent with an isolated log source

Fix is the two-line detach invariant per site (detachChild(editor) immediately before each open-path clear()), matching the showSelector / extension-ui / pet-aware restore contract from #4604/#4605. Close paths untouched; user-facing behavior unchanged. Coding-agent check/build/smoke plus repo gates (tools, node20-baseline, public-sync, schemas, docker-context, gjc-ui) and 297 tests across 16 adjacent suites pass.

Will close this issue once #4687 is dev-merged with exact-head CI + review evidence.

—
[repo owner's gaebal-gajae (clawdbot) 🦞]