From d763db6bf97ddfa00cfb7e165ccc1e8dd7fe8b60 Mon Sep 17 00:00:00 2001
From: Neil <4138956+nwparker@users.noreply.github.com>
Date: Thu, 6 Aug 2026 08:24:22 -0700
Subject: [PATCH] docs(native-chat): name the corrected commit and the
 degraded-jamo consequence
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Records in the file itself that 61977d45177 is pushed and wrong, quoting
the two claims that are false, so a reader who finds it in git log reaches
the correction from the file that replaced it.

Also states the residual as a consequence rather than a curiosity: a lone
leading jamo returns as a standalone compatibility jamo (U+3131), which is
not a composable state — the user cannot resume the syllable, only delete
and retype. Preserved, but degraded into something unusable. That is the
note to find if a reporter ever describes exactly that.

The invariant these tests pin is not "the composer commits on unmount" but
"composition input events must reach React" — which is what a future IME
change would break, and is not visible from the swap site at all.
---
 ...uestion-card-composition-survival.test.tsx | 30 +++++++++++++------
 1 file changed, 21 insertions(+), 9 deletions(-)

diff --git a/src/renderer/src/components/native-chat/native-chat-question-card-composition-survival.test.tsx b/src/renderer/src/components/native-chat/native-chat-question-card-composition-survival.test.tsx
index e17b0bcd95e..fa44e2aee1b 100644
--- a/src/renderer/src/components/native-chat/native-chat-question-card-composition-survival.test.tsx
+++ b/src/renderer/src/components/native-chat/native-chat-question-card-composition-survival.test.tsx
@@ -16,10 +16,15 @@
  *  via `defaultValue={draft}`. `compositionend` is a reconciliation fallback,
  *  not the commit path.
  *
- *  An earlier revision of this file characterized the preedit as LOST. That was
- *  a simulation artifact: it assigned `textarea.value` directly, with no `input`
- *  event, which no IME does. Drive composition through the frames below or the
- *  loss reappears as an artifact rather than a defect.
+ *  CORRECTION TO A PUSHED COMMIT. `61977d45177` ("characterize preedit loss
+ *  when a question card replaces the composer") asserted this loss as a real
+ *  data-loss path — "the preedit is never committed to the draft", "the 가 is
+ *  gone". That is false, and since it is pushed the history cannot be quietly
+ *  corrected, so the correction lives here: the loss was an artifact of that
+ *  file's own harness, which assigned `textarea.value` directly with no `input`
+ *  event — something no IME does. Production never behaved that way. The
+ *  surviving contract below is what actually holds. Drive composition through
+ *  the frames below or the artifact reappears and looks like a defect again.
  *
  *  THIS OWNS NO REPORTED ROW. It is not a regression guard for #12118 or
  *  STA-3219. Those reporters describe continuous flicker keyed to streaming
@@ -28,11 +33,18 @@
  *  across 120 such rerenders. An AskUserQuestion card arrives once per question,
  *  which does not match that cadence.
  *
- *  Two residual facts are deliberately unasserted because no JS owns them: the
- *  OS aborts the composition when the field disappears, so the surviving text
- *  returns committed (a lone jamo comes back as a compatibility jamo the user
- *  cannot compose onto), and the remounted composer is unfocused because the
- *  card owned focus. Geometry too: happy-dom has no layout engine. */
+ *  RESIDUAL, USER-FACING, AND NOT JS-REACHABLE. The text survives but the
+ *  composition does not: the OS aborts it when the field disappears, so what
+ *  returns is committed text. Mid-syllable Hangul is the bad case — a lone
+ *  leading jamo comes back as a standalone compatibility jamo (U+3131 ㄱ), which
+ *  is not a composable state. The user cannot resume the syllable; they must
+ *  delete it and retype. The text is preserved but degraded into something
+ *  unusable. No JS owns this — the IME's internal state is gone before any
+ *  handler could run — so it is recorded rather than asserted. If a reporter
+ *  ever describes exactly this, it is the note they are looking for.
+ *
+ *  Also unasserted: the remounted composer is unfocused, because the card owned
+ *  focus. Geometry too — happy-dom has no layout engine. */
 
 import { act, cleanup, fireEvent, render } from '@testing-library/react'
 import { afterEach, beforeAll, describe, expect, it, vi } from 'vitest'