appkit-assistant 0.16.3__py3-none-any.whl → 0.17.1__py3-none-any.whl

appkit_assistant/backend/processors/lorem_ipsum_processor.py

@@ -38,6 +38,8 @@ LOREM_MODELS = {
         icon="codesandbox",
         model="lorem-short",
         stream=True,
+        supports_attachments=True,
+        supports_tools=True,
     )
 }
 

appkit_assistant/backend/processors/openai_base.py

@@ -33,7 +33,7 @@ GPT_4o: Final = AIModel(
     icon="openai",
     model="gpt-4o",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
 )
 
@@ -43,7 +43,7 @@ GPT_4_1: Final = AIModel(
     icon="openai",
     model="gpt-4.1",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
 )
 
@@ -54,7 +54,7 @@ O3: Final = AIModel(
     model="o3",
     temperature=1,
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
 )
 
@@ -64,7 +64,7 @@ O4_MINI: Final = AIModel(
     icon="openai",
     model="o4-mini",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )
@@ -75,7 +75,7 @@ GPT_5: Final = AIModel(
     icon="openai",
     model="gpt-5",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )
@@ -86,7 +86,7 @@ GPT_5_1: Final = AIModel(
     icon="openai",
     model="gpt-5.1",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )
@@ -97,7 +97,7 @@ GPT_5_2: Final = AIModel(
     icon="openai",
     model="gpt-5.2",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )
@@ -108,7 +108,7 @@ GPT_5_MINI: Final = AIModel(
     icon="openai",
     model="gpt-5-mini",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )
@@ -119,7 +119,7 @@ GPT_5_1_MINI: Final = AIModel(
     icon="openai",
     model="gpt-5.1-mini",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )
@@ -130,7 +130,7 @@ GPT_5_NANO: Final = AIModel(
     icon="openai",
     model="gpt-5-nano",
     stream=True,
-    supports_attachments=True,
+    supports_attachments=False,
     supports_tools=True,
     temperature=1,
 )

appkit_assistant/backend/processors/openai_chat_completion_processor.py

@@ -66,23 +66,40 @@ class OpenAIChatCompletionsProcessor(BaseOpenAIProcessor):
                     if event.choices and event.choices[0].delta:
                         content = event.choices[0].delta.content
                         if content:
-                            yield self._create_chunk(content, model.model, stream=True)
+                            yield self._create_chunk(
+                                content,
+                                model.model,
+                                stream=True,
+                                message_id=event.id,
+                            )
             else:
                 content = session.choices[0].message.content
                 if content:
-                    yield self._create_chunk(content, model.model)
+                    yield self._create_chunk(
+                        content, model.model, message_id=session.id
+                    )
         except Exception as e:
             raise e
 
-    def _create_chunk(self, content: str, model: str, stream: bool = False) -> Chunk:
+    def _create_chunk(
+        self,
+        content: str,
+        model: str,
+        stream: bool = False,
+        message_id: str | None = None,
+    ) -> Chunk:
+        metadata = {
+            "source": "chat_completions",
+            "streaming": str(stream),
+            "model": model,
+        }
+        if message_id:
+            metadata["message_id"] = message_id
+
         return Chunk(
             type=ChunkType.TEXT,
             text=content,
-            chunk_metadata={
-                "source": "chat_completions",
-                "streaming": str(stream),
-                "model": model,
-            },
+            chunk_metadata=metadata,
         )
 
     def _convert_messages_to_openai_format(

appkit_assistant/backend/processors/openai_responses_processor.py

@@ -206,6 +206,12 @@ class OpenAIResponsesProcessor(BaseOpenAIProcessor):
             tool_name = getattr(item, "name", "unknown_tool")
             tool_id = getattr(item, "id", "unknown_id")
             server_label = getattr(item, "server_label", "unknown_server")
+            logger.debug(
+                "MCP call started: %s.%s (id=%s)",
+                server_label,
+                tool_name,
+                tool_id,
+            )
             return self._create_chunk(
                 ChunkType.TOOL_CALL,
                 f"Benutze Werkzeug: {server_label}.{tool_name}",
@@ -369,27 +375,28 @@ class OpenAIResponsesProcessor(BaseOpenAIProcessor):
 
         if event_type == "response.mcp_call.in_progress":
             tool_id = getattr(event, "item_id", "unknown_id")
-            return self._create_chunk(
-                ChunkType.TOOL_CALL,
-                "Tool call in progress...",
-                {"tool_id": tool_id, "status": "in_progress"},
-            )
+            # This event doesn't have tool details, just acknowledge it
+            logger.debug("MCP call in progress: %s", tool_id)
+            return None
+
+        if event_type == "response.mcp_call.completed":
+            # MCP call completed successfully - handled via response.output_item.done
+            # but we can log for debugging
+            tool_id = getattr(event, "item_id", "unknown_id")
+            logger.debug("MCP call completed: %s", tool_id)
+            return None
 
         if event_type == "response.mcp_list_tools.in_progress":
+            # This is a setup event, not a tool call - just log and return None
             tool_id = getattr(event, "item_id", "unknown_id")
-            return self._create_chunk(
-                ChunkType.TOOL_CALL,
-                "Lade verfügbare Werkzeuge...",
-                {"tool_id": tool_id, "status": "listing_tools"},
-            )
+            logger.debug("MCP list_tools in progress: %s", tool_id)
+            return None
 
         if event_type == "response.mcp_list_tools.completed":
+            # This is a setup event, not a tool call - just log and return None
             tool_id = getattr(event, "item_id", "unknown_id")
-            return self._create_chunk(
-                ChunkType.TOOL_RESULT,
-                "Verfügbare Werkzeuge geladen.",
-                {"tool_id": tool_id, "status": "tools_listed"},
-            )
+            logger.debug("MCP list_tools completed: %s", tool_id)
+            return None
 
         if event_type == "response.mcp_list_tools.failed":
             tool_id = getattr(event, "item_id", "unknown_id")

appkit_assistant/{logic → backend}/response_accumulator.py

@@ -52,6 +52,14 @@ class ResponseAccumulator:
 
     def process_chunk(self, chunk: Chunk) -> None:
         """Process a single chunk and update internal state."""
+        # Update message ID if provided in metadata
+        if (
+            self.messages
+            and self.messages[-1].type == MessageType.ASSISTANT
+            and "message_id" in chunk.chunk_metadata
+        ):
+            self.messages[-1].id = chunk.chunk_metadata["message_id"]
+
         if chunk.type == ChunkType.TEXT:
             if self.messages and self.messages[-1].type == MessageType.ASSISTANT:
                 self.messages[-1].text += chunk.text
@@ -128,8 +136,17 @@ class ResponseAccumulator:
         )
 
         if chunk.type == ChunkType.THINKING:
-            if item.text and item.text != text:
+            # Check if this is a streaming delta (has "delta" in metadata)
+            is_delta = chunk.chunk_metadata.get("delta") is not None
+            if is_delta:
+                # Streaming delta - append directly without separator
+                item.text = (item.text or "") + chunk.text
+            elif item.text and item.text != text:
+                # Non-delta chunk with different text - append with newline
                 item.text += f"\n{chunk.text}"
+            else:
+                # Initial text
+                item.text = text
         elif chunk.type == ChunkType.THINKING_RESULT:
             item.status = ThinkingStatus.COMPLETED
             if chunk.text:
@@ -161,8 +178,30 @@ class ResponseAccumulator:
         tool_id = self._get_or_create_tool_session(chunk)
 
         tool_name = chunk.chunk_metadata.get("tool_name", "Unknown")
-        if chunk.type == ChunkType.TOOL_CALL:
-            self.current_activity = f"Nutze Werkzeug: {tool_name}..."
+        server_label = chunk.chunk_metadata.get("server_label", "")
+        # Use server_label.tool_name format if both available
+        if server_label and tool_name and tool_name != "Unknown":
+            display_name = f"{server_label}.{tool_name}"
+        else:
+            display_name = tool_name
+
+        logger.debug(
+            "Tool chunk received: type=%s, tool_id=%s, tool_name=%s, "
+            "server_label=%s, display_name=%s",
+            chunk.type,
+            tool_id,
+            tool_name,
+            server_label,
+            display_name,
+        )
+
+        # Only update activity display if we have a real tool name
+        if (
+            chunk.type == ChunkType.TOOL_CALL
+            and display_name
+            and display_name != "Unknown"
+        ):
+            self.current_activity = f"Nutze Werkzeug: {display_name}..."
 
         status = ThinkingStatus.IN_PROGRESS
         text = ""
@@ -174,11 +213,11 @@ class ResponseAccumulator:
             parameters = chunk.chunk_metadata.get("parameters", chunk.text)
             text = chunk.chunk_metadata.get("description", "")
         elif chunk.type == ChunkType.TOOL_RESULT:
-            is_error = (
-                "error" in chunk.text.lower()
-                or "failed" in chunk.text.lower()
-                or chunk.chunk_metadata.get("error")
-            )
+            # Check error flag from metadata - don't rely on text content
+            # as valid results may contain words like "error" in data
+            # Note: metadata values may be strings, so check for "True" string
+            error_value = chunk.chunk_metadata.get("error")
+            is_error = error_value is True or error_value == "True"
             status = ThinkingStatus.ERROR if is_error else ThinkingStatus.COMPLETED
             result = chunk.text
             if is_error:
@@ -186,12 +225,15 @@ class ResponseAccumulator:
         else:
             text = chunk.text
 
+        # Only pass tool_name if we have a real value
+        effective_tool_name = display_name if display_name != "Unknown" else None
+
         item = self._get_or_create_thinking_item(
             tool_id,
             ThinkingType.TOOL_CALL,
             text=text,
             status=status,
-            tool_name=tool_name,
+            tool_name=effective_tool_name,
             parameters=parameters,
             result=result,
             error=error,
@@ -200,8 +242,13 @@ class ResponseAccumulator:
         if chunk.type == ChunkType.TOOL_CALL:
             item.parameters = parameters
             item.text = text
-            if not item.tool_name or item.tool_name == "Unknown":
-                item.tool_name = tool_name
+            # Only update tool_name if we have a better value and item needs it
+            if (
+                display_name
+                and display_name != "Unknown"
+                and (not item.tool_name or item.tool_name == "Unknown")
+            ):
+                item.tool_name = display_name
             item.status = ThinkingStatus.IN_PROGRESS
         elif chunk.type == ChunkType.TOOL_RESULT:
             item.status = status

appkit_assistant/components/__init__.py

@@ -11,6 +11,7 @@ from appkit_assistant.backend.models import (
     MessageType,
     ThreadModel,
     ThreadStatus,
+    UploadedFile,
 )
 from appkit_assistant.state.thread_list_state import ThreadListState
 from appkit_assistant.state.thread_state import ThreadState
@@ -31,6 +32,7 @@ __all__ = [
     "ThreadModel",
     "ThreadState",
     "ThreadStatus",
+    "UploadedFile",
     "composer",
     "mcp_servers_table",
 ]

appkit_assistant/components/composer.py

@@ -3,6 +3,7 @@ from collections.abc import Callable
 import reflex as rx
 
 import appkit_mantine as mn
+from appkit_assistant.backend.models import UploadedFile
 from appkit_assistant.components.tools_modal import tools_popover
 from appkit_assistant.state.thread_state import ThreadState
 
@@ -63,23 +64,107 @@ def submit() -> rx.Component:
     )
 
 
-def add_attachment(show: bool = False) -> rx.Component | None:
-    if not show:
-        return None
+def _uploaded_file_thumbnail(file: UploadedFile) -> rx.Component:
+    """Render a thumbnail for an uploaded file with a remove button."""
+    return rx.box(
+        rx.hstack(
+            rx.icon("file", size=16, color=rx.color("gray", 9)),
+            rx.text(
+                file.filename,
+                size="1",
+                max_width="100px",
+                overflow="hidden",
+                text_overflow="ellipsis",
+                white_space="nowrap",
+            ),
+            spacing="1",
+            align="center",
+            padding="4px 8px",
+            background=rx.color("gray", 3),
+            border_radius="6px",
+        ),
+        rx.icon_button(
+            rx.icon("x", size=10),
+            width="16px",
+            height="16px",
+            variant="solid",
+            color_scheme="gray",
+            position="absolute",
+            top="-6px",
+            right="-6px",
+            border_radius="12px",
+            padding="0px",
+            cursor="pointer",
+            on_click=lambda: ThreadState.remove_file_from_prompt(file.file_path),
+        ),
+        position="relative",
+    )
 
-    return rx.tooltip(
-        rx.button(
-            rx.icon("paperclip", size=18),
-            rx.text("2 files", size="1", color="gray.2"),
-            id="composer-attachment",
-            variant="ghost",
-            padding="8px",
-            access_key="s",
+
+def selected_files_row() -> rx.Component:
+    """Render the row of selected file thumbnails (only visible when files exist)."""
+    return rx.cond(
+        ThreadState.uploaded_files.length() > 0,
+        rx.hstack(
+            rx.foreach(
+                ThreadState.uploaded_files,
+                _uploaded_file_thumbnail,
+            ),
+            spacing="2",
+            flex_wrap="wrap",
+            margin_top="6px",
+            margin_left="12px",
         ),
-        content="Manage Attachments…",
+        rx.fragment(),
     )
 
 
+def file_upload(show: bool = False) -> rx.Component:
+    """File upload button with drag-and-drop support."""
+    return rx.cond(
+        show & ThreadState.selected_model_supports_attachments,
+        rx.tooltip(
+            rx.upload.root(
+                rx.box(
+                    rx.icon("paperclip", size=18, color=rx.color("gray", 9)),
+                    cursor="pointer",
+                    padding="8px",
+                    border_radius="8px",
+                    _hover={"background": rx.color("gray", 3)},
+                ),
+                id="composer_file_upload",
+                accept={
+                    #                "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet": [
+                    #                    ".xlsx"
+                    #                ],
+                    #                "text/csv": [".csv"],
+                    #                "application/vnd.openxmlformats-officedocument."
+                    #                "wordprocessingml.document": [".docx"],
+                    #                "application/vnd.openxmlformats-officedocument."
+                    #                "presentationml.presentation": [".pptx"],
+                    #                "text/markdown": [".md"],
+                    "application/pdf": [".pdf"],
+                    "image/png": [".png"],
+                    "image/jpeg": [".jpg", ".jpeg"],
+                },
+                multiple=True,
+                max_files=5,
+                max_size=5 * 1024 * 1024,
+                on_drop=ThreadState.handle_upload(
+                    rx.upload_files(upload_id="composer_file_upload")
+                ),
+            ),
+            content="Dateien hochladen (max. 5, 5MB pro Datei)",
+        ),
+        rx.fragment(),
+    )
+
+
+def add_attachment(show: bool = False) -> rx.Component:
+    """Legacy attachment function - now wraps file_upload."""
+    return file_upload(show=show)
+
+
 def choose_model(show: bool = False) -> rx.Component | None:
     if not show:
         return None
@@ -146,7 +231,9 @@ class ComposerComponent(rx.ComponentNamespace):
     add_attachment = staticmethod(add_attachment)
     choose_model = staticmethod(choose_model)
     clear = staticmethod(clear)
+    file_upload = staticmethod(file_upload)
     input = staticmethod(composer_input)
+    selected_files_row = staticmethod(selected_files_row)
     submit = staticmethod(submit)
     tools = staticmethod(tools)