mirror of
https://github.com/Kilo-Org/kilocode.git
synced 2026-08-29 03:44:06 +08:00
test(tui): trim verbose backstory from slot reactivity test header
Remove implementation history paragraph from the test file's doc comment, keeping only the concise description of what the test validates.
This commit is contained in:
@@ -1,18 +1,6 @@
|
||||
/**
|
||||
* Regression test for the Slot wrapper in plugin/slots.tsx.
|
||||
*
|
||||
* PR #9425 introduced a wrapper around the opentui Slot that memoizes the
|
||||
* fallback children so stateful defaults (like the session prompt) aren't
|
||||
* recreated when props change. The first implementation used an object spread
|
||||
* `{...props}` to forward props, which *breaks SolidJS prop reactivity*: every
|
||||
* prop except the getter-defined `children` freezes at mount time.
|
||||
*
|
||||
* Downstream consequence: `ref`, `visible`, `disabled`, `on_submit` stop
|
||||
* updating on the session_prompt slot once the outer <Show> no longer gates
|
||||
* mounting. That's what made Enter stop submitting after a blocking overlay
|
||||
* closed — the prompt ref callback and submit handler were captured against
|
||||
* stale closures.
|
||||
*
|
||||
* This test locks in two things:
|
||||
* 1. A static invariant: the wrapper does NOT spread raw props (`...props`),
|
||||
* which would silently reintroduce the regression on refactors.
|
||||
|
||||
Reference in New Issue
Block a user