Skip to content

feat(chat): natural-language tool titles, past-tense completion, and smooth agent-group narration#5660

Merged
waleedlatif1 merged 7 commits into
stagingfrom
feat/chat-agent-group-polish
Jul 14, 2026
Merged

feat(chat): natural-language tool titles, past-tense completion, and smooth agent-group narration#5660
waleedlatif1 merged 7 commits into
stagingfrom
feat/chat-agent-group-polish

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Show past-tense tool titles once a tool call completes successfully ("Querying logs" → "Queried logs") via a leading-gerund verb map applied to the resolved title; failed/cancelled calls keep the attempted-action wording, and the workspace_file live-title path gets the same rewrite
  • Add natural gerund titles for ~50 tools that previously fell through to robotic Title Case ("Deploy Api" → "Deploying API"), and make humanizeToolName keep acronym casing (MCP, API, OAuth, URL, ID, AI)
  • Smooth agent-group narration rows with useSmoothText (word-by-word paced reveal on the live tail row), matching top-level text
  • Render inline markdown (bold, italic, code, label as label) in narration rows instead of leaking raw syntax
  • Tone narration rows with --text-muted (drop the opacity-60 stack on --text-secondary)
  • Insert a space at the seam when consecutive thinking/text blocks glue together ("triggered it.The failing" → "triggered it. The failing")

Type of Change

  • Improvement

Testing

Unit tests added for past-tense rewrites, acronym casing, gerund title coverage, inline markdown rendering, and text-seam joins; full message-content suite passing (50 tests), typecheck clean

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 Jul 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 (UTC)
docs Skipped Skipped Jul 14, 2026 1:56am

Request Review

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Presentation-layer changes in chat message parsing and rendering with broad unit test coverage; no auth, data, or API contract changes.

Overview
Improves copilot chat presentation: tool labels get gerund titles for many tools, acronym-aware humanization, and past-tense rewrites on success (via getToolCompletedTitle in toToolData and the workspace_file live-title path). Agent-group narration uses useSmoothText on the live tail, a lightweight renderInlineMarkdown helper, and --text-muted styling.

Parsing merges thinking/text with a guarded space at channel seams only. Tool rows in groups drop per-tool status icons and show text-only titles with shimmer while executing.

Tests cover title rewrites, inline markdown, and seam joining.

Reviewed by Cursor Bugbot for commit 74babad. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR improves chat narration and tool-call labels. The main changes are:

  • Past-tense titles for successful tool calls.
  • Natural gerund labels and acronym casing for tool names.
  • Smooth word-by-word reveal for live agent-group narration.
  • Inline markdown rendering in narration rows.
  • Safer spacing at thinking-to-text seams.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/home/components/message-content/message-content.tsx Adds completed tool-title rewriting and channel-aware spacing for agent narration.
apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/agent-group.tsx Adds smooth reveal and inline markdown rendering for narration rows.
apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/inline-markdown.tsx Adds a focused inline markdown renderer for narration text.
apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/tool-call-item.tsx Applies completed-title wording to live workspace-file tool rows.
apps/sim/lib/copilot/tools/tool-display.ts Adds natural tool titles, acronym casing, and completed-title rewrites.

Reviews (5): Last reviewed commit: "fix(chat): classify inline-markdown toke..." | Re-trigger Greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@waleedlatif1 waleedlatif1 merged commit 8d7d590 into staging Jul 14, 2026
18 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/chat-agent-group-polish branch July 14, 2026 02:02
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 74babad. Configure here.

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