indifferentketchup/boocode
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

batch3 T6 typing fix: inline narrowing for body.state guard

Browse Source 
Boolean indirection (const stateOnly = ...; if (stateOnly)) didn't carry
the narrowing to body.state inside the branch. Inline the check so
TypeScript narrows correctly and the explicit cast on pendingState
becomes unnecessary.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
indifferentketchup
2026-05-15 15:37:02 +00:00
parent 0a7e52326c
commit 2de67fe091
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
apps/web/src/hooks/usePanes.ts
View File

@@ -78,9 +78,8 @@ export function usePanes(sessionId: string | undefined): {
const update = useCallback(
async (id: string, body: PaneUpdateRequest): Promise<void> => {
const stateOnly = body.state !== undefined && body.position === undefined;
if (stateOnly) {
if (body.state !== undefined && body.position === undefined) {
const nextState = body.state;
// Optimistic local update
setPanes((prev) => {
if (!prev) return prev;
@@ -90,10 +89,10 @@ export function usePanes(sessionId: string | undefined): {
changed = true;
// Narrow via discriminated union to satisfy TypeScript
if (pane.kind === 'chat') {
return { ...pane, state: body.state as typeof pane.state };
return { ...pane, state: nextState as typeof pane.state };
}
if (pane.kind === 'file_browser') {
return { ...pane, state: body.state as typeof pane.state };
return { ...pane, state: nextState as typeof pane.state };
}
return pane;
});
@@ -101,7 +100,6 @@ export function usePanes(sessionId: string | undefined): {
});
// Coalesce: last state wins within debounce window
const nextState = body.state;
pendingState.current.set(id, nextState);
if (debounceTimer.current !== null) {