Skip to content

fix(ui): resolve all 13 TypeScript build errors#45

Open
dimakis wants to merge 3 commits intomainfrom
fix/ui-typescript-errors
Open

fix(ui): resolve all 13 TypeScript build errors#45
dimakis wants to merge 3 commits intomainfrom
fix/ui-typescript-errors

Conversation

@dimakis
Copy link
Owner

@dimakis dimakis commented Feb 23, 2026

Summary

Fix 13 TypeScript compilation errors that prevent the UI container from building (tsc -b fails during Containerfile.ui build stage).

Unused variables (5 fixes):

  • agent-topology.tsx — rename activeAgent to _activeAgent
  • useInlineAssistant.ts — remove unused abortStream
  • use-force-graph.ts — rename to _prefersReducedMotion
  • registry-store.test.ts — remove unused RegistryState import
  • EntityActionMenu.test.tsx — remove unused EntityAction import

Type fixes (6 fixes):

  • test/setup.ts — add @ts-expect-error for CJS require
  • registry-store.ts — add helpers to TabKey union
  • ChatHeader.tsx / useChatMessages.ts — align ModelInfo and Conversation types
  • proposals.ts — add yaml_content to create mutation type
  • useInlineAssistant.ts — widen setter type for DelegationMsg

Other fixes (2):

  • auth.ts — add challenge to PasskeyAuthOptions
  • dashboard-editor/index.tsx — add undefined argument to useRef (React 19)

Test plan

  • npx tsc --noEmit passes locally
  • No behavior changes — type fixes only

Made with Cursor

Remove unused variables/imports, widen delegation message setter types
to accept updater functions, add missing type fields (TabKey 'helpers',
PasskeyAuthOptions challenge, proposal yaml_content), and align prop
types across ModelPicker/ChatHeader/useChatMessages.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…okens

The distributed path was discarding the A2ARemoteClient.invoke() result,
yielding only trace events with no content. Now:

- Extracts response content from the result dict
- Streams it as OpenAI-format SSE token chunks
- Sends proper finish_reason=stop final chunk
- Commits session before [DONE]

This is the quick fix — full streaming proxy (SendStreamingMessage)
will replace this when per-token streaming is needed.

Co-authored-by: Cursor <cursoragent@cursor.com>
@dimakis dimakis force-pushed the fix/ui-typescript-errors branch from c7ac722 to 6464864 Compare February 23, 2026 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant