Skip to content

(AgentChat) - realtime tool calls fix , (OpenAI) browser login gpt 4.1 option removed - #386

Open
lucifer78907 wants to merge 2 commits into
initializ:mainfrom
lucifer78907:fix/chat-progress-and-gpt41
Open

(AgentChat) - realtime tool calls fix , (OpenAI) browser login gpt 4.1 option removed#386
lucifer78907 wants to merge 2 commits into
initializ:mainfrom
lucifer78907:fix/chat-progress-and-gpt41

Conversation

@lucifer78907

Copy link
Copy Markdown

Type of Change

  • Bug fix
  • New feature
  • Enhancement / refactor
  • New skill
  • Documentation
  • CI / build

Description

Two independent fixes:

1. fix(chat) — tool execution progress never rendered in the web UI
(forge-ui/static/app.js)

The progress SSE handler looked for tool info in a data part of
status.message.parts, but the agent doesn't send it that way: the tool name
and phase arrive in task metadata (progress_tool / progress_phase) and
the human-readable line arrives as a text part. The part.kind === 'data'
branch therefore never matched and no tool card was ever appended.

The handler now:

  • reads progress_tool / progress_phase from parsed.metadata
  • maps the backend phases tool_start / tool_end onto the bare
    start / end values ToolCard expects
  • takes the text part as the card's message
  • keeps the existing replace-the-pending-start behaviour, now keyed off the
    metadata tool name (tools execute sequentially, so start/end never interleave)

2. fix(providers) — drop GPT 4.1 from the OpenAI model list
(forge-core/catalog/providers.go)

Comments out the GPT 4.1 / gpt-4.1 entry so it no longer appears in the
provider model picker. Left commented rather than deleted for easy restore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant