fix(dev): command-palette-interactive-host failures from #4591 pet integration

[Yeachan-Heo]

Reproduction (local, deterministic)

At dev heads 05d33c9984 (#4591 pet stabilization) and later, three tests fail in packages/coding-agent/test/command-palette-interactive-host.test.ts:

(fail) keeps a draft when the palette action is unavailable and does not leak palette components
(fail) blocks the palette while a draft or palette command is active without leaking a modal
(fail) reports rejected extension commands, clears the modal, and recovers for later dispatch

Assertions at lines 425/454/481 (editorContainer.children equality: expected [editor], received a leaked CommandPaletteComponent with DynamicBorder/CommandPaletteList children).

Bisection

• da648897e8 (pre-fix(pet): stabilize iTerm2 Gajae Pet placement, drag handling, and animation rendering #4591): 0 fail
• 05d33c9984 (fix(pet): stabilize iTerm2 Gajae Pet placement, drag handling, and animation rendering #4591): 3 fail
• 60a92c7b, e1849e676b (current dev): still 3 fail

git diff da648897..05d33c99 -- packages/coding-agent/src/modes/interactive-mode.ts shows #4591 changed composer remounting (restoreComposer now routes through petWidget.remountComposer() when a pet widget exists) and palette-adjacent editor container wiring. The palette host tests construct the real InteractiveMode; a pet widget is now present in those tests, and the cancelled-palette restore path appears to leave the palette component mounted instead of restoring [editor].

Dev CI runs at 05d33c99/60a92c7b/e1849e6 (shard-8: command-palette-interactive-host.test.ts: exit=1) confirm this is blocking dev.

Owner lane being attached.

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

[Yeachan-Heo]

Bounded blocker report — lane complete, merge blocked by governance gate only

PR: #4605 (fix/issue-4604-command-palette-pet-restore, head c194e79453bb80a711a44eebb12eda09bebd942a, base dev @ e1849e676b)

Deliverables 1–3: complete

• Root cause: fix(pet): stabilize iTerm2 Gajae Pet placement, drag handling, and animation rendering #4591 gated GajaePetWidget.remountComposer() behind #canMutateSharedUi(). InteractiveMode.restoreComposer() always routes through it once init() creates a widget (even with pet.mode: "off"), but a never-activated widget (epoch 0, never an overlay owner) fails both clauses of #canMutateSharedUi() → remountComposer() silently no-ops → the palette close/cancel paths never unmount CommandPaletteComponent from editorContainer (the exact leak in the 3 failing tests).
• Fix (8 lines, one method): composer-mount authority separated from overlay ownership — never-activated widgets remount the plain editor exactly like the no-pet fallback; only disposal or a live successor revokes the mount. fix(pet): stabilize iTerm2 Gajae Pet placement, drag handling, and animation rendering #4591's successor-takeover isolation and all dispose()/#applyMode() gating untouched. 2 regression tests added pinning both ownership states.
• Validation: command-palette-interactive-host 8/8 (was 3 fail), gajae-pet-widget 62 pass, tui gajae-pet 19 pass, --cwd packages/tui check clean, --cwd packages/coding-agent check clean, 6 adjacent suites 104 pass.

Deliverable 4–5: blocked at exactly one gate

CI on the exact PR head: every code job green — affected-path plan, native-build, test:gajae-pet-widget, ts-build, gjc-state-gates ×4, Local public surfaces, Virtual integration validation.

Sole red: PR contract bootstrap (.github/workflows/dev-ci.yml job pr-contract-bootstrap). It requires the PR body to contain exactly one gajae.pr-review-verdict.v1 merge-approved sha256:<digest> line whose reviewer-id is not the PR author, holds an APPROVED review on the exact head SHA, and has admin/maintain/write permission. The only credential in this lane is Yeachan-Heo, which authored the PR — satisfying the gate from here would mean fabricating an independent review, so I did not.

Action needed (maintainer): review + approve #4605 at head c194e79453 and post the verdict line with the exact-head diff digest, e.g.:

gajae.pr-review-verdict.v1 merge-approved sha256:<digest> reviewer:human reviewer-id:<your-login> evidence:<link to review>

Digest is sha256 of git diff --binary --full-index --no-ext-diff e1849e676bbccf4975abb356ac5daa3460af7cff...c194e79453bb80a711a44eebb12eda09bebd942a. Nothing speculative was merged.

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

[Yeachan-Heo]

Exact-head digest for the verdict line (sha256 of git diff --binary --full-index --no-ext-diff e1849e676bbccf4975abb356ac5daa3460af7cff...c194e79453bb80a711a44eebb12eda09bebd942a):

gajae.pr-review-verdict.v1 merge-approved sha256:519b3872487f2bfe637dad926294b1a9f4844c589e8f0343bde85e96f76adfad reviewer:human reviewer-id:<maintainer-login> evidence:<review link>

Maintainer fills in <maintainer-login> (≠ PR author, write+ permission) and <review link> after approving #4605 at head c194e79453, then edits that line into the PR body.

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

[Yeachan-Heo]

형님 직접 수정으로 dev에 반영 완료: 52dad458 fix(pet): allow composer remount when no overlay owner is registered. dev head에서 command-palette-interactive-host 8/8 로컬 검증 완료. 내 lane의 #4605는 이 커밋과 동일 근본 원인을 다른 지점(never-activated widget remount)에서 수정한 중복 수정이라 클린업 대상으로 남겨둠 — 형님 판단으로 #4605 close 또는 merge.

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