flowent 0.0.6 → 0.0.7

package/README.md

@@ -8,7 +8,7 @@
   <a href="https://github.com/ImFeH2/flowent/blob/main/LICENSE"><img src="https://img.shields.io/npm/l/flowent.svg?style=flat-square&label=License" alt="License" /></a>
   <a href="https://github.com/ImFeH2/flowent/actions/workflows/ci.yml"><img src="https://github.com/ImFeH2/flowent/workflows/CI/badge.svg" alt="CI" /></a>
   <a href="https://github.com/ImFeH2/flowent/actions/workflows/release.yml"><img src="https://github.com/ImFeH2/flowent/workflows/Release/badge.svg" alt="Release" /></a>
-  <a href="https://github.com/ImFeH2/flowent/pkgs/container/flowent"><img src="https://github.com/ImFeH2/flowent/workflows/Publish%20Docker%20image/badge.svg" alt="Docker image" /></a>
+  <a href="https://github.com/ImFeH2/flowent/pkgs/container/flowent"><img src="https://github.com/ImFeH2/flowent/workflows/Docker/badge.svg" alt="Docker" /></a>
 </p>
 
 # Flowent

package/backend/README.md

@@ -8,7 +8,7 @@
   <a href="https://github.com/ImFeH2/flowent/blob/main/LICENSE"><img src="https://img.shields.io/npm/l/flowent.svg?style=flat-square&label=License" alt="License" /></a>
   <a href="https://github.com/ImFeH2/flowent/actions/workflows/ci.yml"><img src="https://github.com/ImFeH2/flowent/workflows/CI/badge.svg" alt="CI" /></a>
   <a href="https://github.com/ImFeH2/flowent/actions/workflows/release.yml"><img src="https://github.com/ImFeH2/flowent/workflows/Release/badge.svg" alt="Release" /></a>
-  <a href="https://github.com/ImFeH2/flowent/pkgs/container/flowent"><img src="https://github.com/ImFeH2/flowent/workflows/Publish%20Docker%20image/badge.svg" alt="Docker image" /></a>
+  <a href="https://github.com/ImFeH2/flowent/pkgs/container/flowent"><img src="https://github.com/ImFeH2/flowent/workflows/Docker/badge.svg" alt="Docker" /></a>
 </p>
 
 # Flowent

package/backend/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "flowent"
-version = "0.0.6"
+version = "0.0.7"
 description = "A workflow orchestration platform for multi-agent collaboration."
 readme = "README.md"
 authors = [

package/backend/src/flowent/agent.py

@@ -1,9 +1,9 @@
 from __future__ import annotations
 
 import json
+import re
 import threading
 import time as _time
-import traceback
 import uuid as _uuid
 from collections.abc import Callable
 from dataclasses import dataclass, field
@@ -132,7 +132,91 @@ class ResolvedModelSource:
 
 
 def build_error_context(content: str) -> str:
-    return f"<system>Previous runtime error:\n{content}</system>"
+    return (
+        f"<system>Previous runtime error:\n{normalize_error_context(content)}</system>"
+    )
+
+
+_RAW_HTML_DOCUMENT_PATTERN = re.compile(
+    r"<!doctype\s+html|</?html\b|</?head\b|</?body\b|<title\b",
+    re.IGNORECASE,
+)
+_RAW_HTML_FRAGMENT_LINE_PATTERN = re.compile(
+    r"^\s*</?(?:div|span|pre|code|p|br|h[1-6]|script|style)\b",
+    re.IGNORECASE,
+)
+_ERROR_RESPONSE_TEXT_PATTERN = re.compile(
+    r"\b(?:bad gateway|gateway timeout|internal server error|service unavailable|"
+    r"not found|forbidden|unauthorized|upstream|exception|runtimeerror|error)\b",
+    re.IGNORECASE,
+)
+_TRACEBACK_PATTERN = re.compile(r"(^|\n)\s*traceback\b", re.IGNORECASE)
+
+
+def contains_raw_error_response(content: str) -> bool:
+    if _RAW_HTML_DOCUMENT_PATTERN.search(content):
+        return True
+    if not _ERROR_RESPONSE_TEXT_PATTERN.search(content):
+        return False
+    return any(
+        _RAW_HTML_FRAGMENT_LINE_PATTERN.search(line) for line in content.splitlines()
+    )
+
+
+def normalize_error_context(content: str) -> str:
+    normalized = content.strip()
+    if not normalized:
+        return "Runtime error"
+
+    if contains_raw_error_response(normalized):
+        return "Upstream returned an unexpected error response"
+
+    traceback_match = _TRACEBACK_PATTERN.search(normalized)
+    if traceback_match is None:
+        return normalized
+
+    prefix = normalized[: traceback_match.start()].strip()
+    if prefix:
+        return prefix
+
+    for line in reversed(normalized.splitlines()):
+        candidate = line.strip()
+        if not candidate:
+            continue
+        if candidate.lower().startswith("traceback"):
+            continue
+        if candidate.startswith(("File ", "^")):
+            continue
+        if candidate.startswith("The above exception") or candidate.startswith(
+            "During handling"
+        ):
+            continue
+        return candidate
+    return "Runtime error"
+
+
+def normalize_compacted_execution_context(content: str) -> str:
+    normalized = content.strip()
+    if not normalized:
+        return ""
+    if contains_raw_error_response(normalized):
+        return "Previous compacted execution context contained an unexpected error response."
+
+    traceback_match = _TRACEBACK_PATTERN.search(normalized)
+    if traceback_match is None:
+        return normalized
+
+    prefix = normalized[: traceback_match.start()].strip()
+    if prefix:
+        return prefix
+    return normalize_error_context(normalized)
+
+
+def build_runtime_error_summary(exc: Exception) -> str:
+    message = str(exc).strip()
+    if not message:
+        return type(exc).__name__
+    return f"{type(exc).__name__}: {message}"
 
 
 class Agent:
@@ -1139,11 +1223,9 @@ class Agent:
                     self._interrupt_requested.clear()
                     self.set_interrupt_callback(None)
                     self._log.exception("Agent error")
-                    tb_str = traceback.format_exc()
-                    self._append_history(
-                        ErrorEntry(content=f"{type(exc).__name__}: {exc}\n\n{tb_str}"),
-                    )
-                    self.set_state(AgentState.ERROR, f"{type(exc).__name__}: {exc}")
+                    error_summary = build_runtime_error_summary(exc)
+                    self._append_history(ErrorEntry(content=error_summary))
+                    self.set_state(AgentState.ERROR, error_summary)
                     self._wait_for_input()
                     if self._terminate.is_set():
                         break
@@ -2050,7 +2132,8 @@ class Agent:
         if summary:
             messages.append(
                 self._build_runtime_system_message(
-                    f"Compacted execution context:\n{summary}"
+                    "Compacted execution context:\n"
+                    f"{normalize_compacted_execution_context(summary)}"
                 )
             )
         messages.extend(self._build_history_messages(history_snapshot[history_cutoff:]))

package/backend/src/flowent/graph_service.py

@@ -51,7 +51,6 @@ from flowent.settings import (
 )
 from flowent.tools import (
     MINIMUM_TOOLS,
-    is_assistant_only_mcp_tool_name,
     is_assistant_only_tool_name,
 )
 from flowent.workspace_store import workspace_store
@@ -117,10 +116,9 @@ def build_tools_for_role(
     for tool_name in [*MINIMUM_TOOLS, *included_tools, *(requested_tools or [])]:
         if tool_name in seen_tools:
             continue
-        if not assistant_boundary and (
-            is_assistant_only_tool_name(tool_name)
-            or is_assistant_only_mcp_tool_name(tool_name)
-        ):
+        if tool_name.startswith("mcp__"):
+            continue
+        if not assistant_boundary and is_assistant_only_tool_name(tool_name):
             continue
         if tool_name in excluded_tools and tool_name not in MINIMUM_TOOLS:
             continue
@@ -1081,111 +1079,6 @@ def create_tab(
     return tab
 
 
-def duplicate_tab(
-    *,
-    tab_id: str,
-) -> tuple[Tab | None, str | None]:
-    source_tab = workspace_store.get_tab(tab_id)
-    if source_tab is None:
-        return None, f"Tab '{tab_id}' not found"
-
-    _sync_tab_permissions_from_legacy_leader(source_tab)
-    duplicated_definition = WorkflowDefinition.from_mapping(
-        source_tab.definition.serialize()
-    )
-    id_map: dict[str, str] = {}
-    duplicated_nodes: list[WorkflowNodeDefinition] = []
-
-    for node in duplicated_definition.nodes:
-        new_node_id = str(uuid.uuid4())
-        id_map[node.id] = new_node_id
-        duplicated_node = build_workflow_node_definition(
-            node_id=new_node_id,
-            node_kind=node.type,
-            config=node.config,
-        )
-        duplicated_nodes.append(duplicated_node)
-
-    duplicated_edges = [
-        GraphEdge(
-            id=str(uuid.uuid4()),
-            from_node_id=id_map.get(edge.from_node_id, edge.from_node_id),
-            from_port_key=edge.from_port_key,
-            to_node_id=id_map.get(edge.to_node_id, edge.to_node_id),
-            to_port_key=edge.to_port_key,
-            kind=edge.kind,
-        )
-        for edge in duplicated_definition.edges
-    ]
-    duplicated_view_positions = {
-        id_map.get(node_id, node_id): position
-        for node_id, position in duplicated_definition.view.positions.items()
-        if id_map.get(node_id, node_id) in id_map.values()
-    }
-
-    settings = settings_module.get_settings()
-    new_tab = Tab(
-        id=str(uuid.uuid4()),
-        title=f"{source_tab.title} Copy",
-        leader_id=str(uuid.uuid4()),
-        allow_network=source_tab.allow_network,
-        write_dirs=list(source_tab.write_dirs),
-        permissions_initialized=True,
-        definition=WorkflowDefinition(
-            version=duplicated_definition.version,
-            nodes=duplicated_nodes,
-            edges=duplicated_edges,
-            view=duplicated_definition.view.__class__(
-                positions=duplicated_view_positions
-            ),
-        ),
-    )
-    assert new_tab.leader_id is not None
-    workspace_store.upsert_tab(new_tab)
-    workspace_store.upsert_node_record(
-        _build_leader_record(
-            tab_id=new_tab.id,
-            leader_id=new_tab.leader_id,
-            settings=settings,
-        )
-    )
-
-    for node in source_tab.definition.nodes:
-        if node.type != WorkflowNodeKind.AGENT:
-            continue
-        duplicated_node_id = id_map.get(node.id)
-        if duplicated_node_id is None:
-            continue
-        config, error = build_node_config(
-            role_name=str(node.config.get("role_name", "")),
-            tab_id=new_tab.id,
-            name=str(node.config["name"])
-            if isinstance(node.config.get("name"), str)
-            else None,
-        )
-        if error is not None or config is None:
-            return None, error or "Failed to duplicate workflow"
-        workspace_store.upsert_node_record(
-            GraphNodeRecord(
-                id=duplicated_node_id,
-                config=config,
-                state=AgentState.INITIALIZING,
-                position=duplicated_view_positions.get(duplicated_node_id),
-            )
-        )
-
-    if registry.get_all():
-        _start_tab_runtime(new_tab.id)
-    event_bus.emit(
-        Event(
-            type=EventType.TAB_CREATED,
-            agent_id="assistant",
-            data=serialize_tab_summary(new_tab),
-        )
-    )
-    return new_tab, None
-
-
 def _is_path_within_boundary(path: str, boundary_dirs: list[str]) -> bool:
     resolved_path = resolve_path(path)
     return any(

package/backend/src/flowent/role_management.py

@@ -67,10 +67,7 @@ def resolve_role_tool_config(
     included_tools: list[str] | None,
     excluded_tools: list[str] | None,
 ) -> tuple[list[str], list[str]]:
-    from flowent.tools import (
-        is_assistant_only_mcp_tool_name,
-        is_assistant_only_tool_name,
-    )
+    from flowent.tools import is_assistant_only_tool_name
 
     next_included = normalize_tool_names(
         included_tools
@@ -83,8 +80,8 @@ def resolve_role_tool_config(
         next_included = [
             tool_name
             for tool_name in next_included
-            if not is_assistant_only_tool_name(tool_name)
-            and not is_assistant_only_mcp_tool_name(tool_name)
+            if not tool_name.startswith("mcp__")
+            and not is_assistant_only_tool_name(tool_name)
         ]
     next_excluded = normalize_tool_names(
         excluded_tools
@@ -93,6 +90,12 @@ def resolve_role_tool_config(
         if current
         else []
     )
+    if excluded_tools is not None:
+        next_excluded = [
+            tool_name
+            for tool_name in next_excluded
+            if not tool_name.startswith("mcp__")
+        ]
     validate_role_tool_config(next_included, next_excluded)
     return next_included, next_excluded
 

package/backend/src/flowent/routes/__init__.py

@@ -3,7 +3,6 @@ from fastapi import APIRouter
 from flowent.routes.access import router as access_router
 from flowent.routes.assistant import router as assistant_router
 from flowent.routes.image_assets import router as image_assets_router
-from flowent.routes.mcp import router as mcp_router
 from flowent.routes.meta import router as meta_router
 from flowent.routes.nodes import router as nodes_router
 from flowent.routes.prompts import router as prompts_router
@@ -18,7 +17,6 @@ router.include_router(access_router)
 router.include_router(nodes_router)
 router.include_router(assistant_router)
 router.include_router(image_assets_router)
-router.include_router(mcp_router)
 router.include_router(roles_router)
 router.include_router(providers_router)
 router.include_router(prompts_router)

package/backend/src/flowent/routes/assistant.py

@@ -12,6 +12,7 @@ from flowent.assistant_commands import (
 )
 from flowent.image_assets import require_image_asset
 from flowent.models import (
+    AgentState,
     Message,
     content_parts_to_text,
     has_image_parts,
@@ -90,6 +91,8 @@ async def send_assistant_message(req: AssistantMessageRequest) -> dict:
     assistant = _get_assistant()
     if assistant is None:
         raise HTTPException(status_code=404, detail="Assistant not found")
+    if assistant.state == AgentState.TERMINATED:
+        raise HTTPException(status_code=409, detail="Assistant is no longer available")
 
     try:
         parts = _parse_request_parts(req)

package/backend/src/flowent/routes/nodes.py

@@ -344,7 +344,12 @@ async def clear_node_chat(node_id: str) -> dict:
 @router.post("/api/nodes/{node_id}/messages")
 async def dispatch_node_message(node_id: str, req: DispatchNodeMessageRequest) -> dict:
     from flowent.graph_service import dispatch_node_message
-    from flowent.models import NodeType, content_parts_to_text, has_image_parts
+    from flowent.models import (
+        AgentState,
+        NodeType,
+        content_parts_to_text,
+        has_image_parts,
+    )
     from flowent.models import TextPart as ModelTextPart
 
     node = registry.get(node_id)
@@ -367,6 +372,11 @@ async def dispatch_node_message(node_id: str, req: DispatchNodeMessageRequest) -
             status_code=400,
             detail="Human input can only target Assistant or a Workflow Leader",
         )
+    if node.state == AgentState.TERMINATED:
+        raise HTTPException(
+            status_code=409,
+            detail="This workflow chat is no longer available",
+        )
 
     try:
         parts = _parse_dispatch_parts(req)