Skip to content

fix(chat): stabilize inline tool activity updates - #7806

Merged
waleedlatif1 merged 3 commits into
stagingfrom
codex/activity-stream-ux-review
Sep 14, 2026
Merged

waleedlatif1 merged 3 commits into
stagingfrom
codex/activity-stream-ux-review

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Sep 14, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Pace inline activity updates across chat and subagents while keeping history, completion, and requests for input immediate.
  • Remove agent prefixes, preserve icons and disclosure animations, and compact related summaries such as “Navigated, read pages.”

Type of Change

  • Bug fix

Testing

678 targeted tests passed, including timing, parallel failures, keyboard disclosure, browser favicon retention, and integration icon transitions. Type-check, lint, all 46 audits, block registry, and docs manifest checks passed. Manually checked the browser comparison with a real favicon in light and dark themes, including stable parallel-agent spacing from the first action through completion.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 14, 2026 4:26am UTC

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@greptile-apps

greptile-apps Bot commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge.

Summary

  • Active labels coalesce updates over a one-second cadence.
  • Completion, failures, input requests, and expanded history remain immediate.
  • Related actions use shorter shared-object summaries such as “Navigated, read pages.”

Diagram

sequenceDiagram
  participant Stream as Tool stream
  participant Group as AgentGroup
  participant Activity as ActivityStream
  participant User as User
  Stream->>Group: Tool call starts or changes
  Group->>Activity: Label, icon, activity key
  alt Cosmetic active-label change
    Activity-->>User: Keep current label and icon
    Activity-->>User: Show next label after 1 second
  else Failure, stop, skip, completion, or input needed
    Activity-->>User: Update immediately
  end
  User->>Activity: Open disclosure
  Activity-->>User: Show full history immediately
Loading

Reviews (3) · Last reviewed commit: "fix(chat): retain parallel activity and ..."

Comment thread apps/sim/lib/copilot/tools/tool-activity.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1
waleedlatif1 merged commit 85a5938 into staging Sep 14, 2026
34 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/activity-stream-ux-review branch September 14, 2026 04:32

This branch was previously deployed

1 inactive deployment
Preview — 09861094 Deployed Sep 14, 2026 by vercel[bot]
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