Skip to content

Show skill tool calls as "Reading <name> skill"#2440

Merged
rafaeelaudibert merged 2 commits into
mainfrom
posthog-code/skill-tool-display
Jun 1, 2026
Merged

Show skill tool calls as "Reading <name> skill"#2440
rafaeelaudibert merged 2 commits into
mainfrom
posthog-code/skill-tool-display

Conversation

@rafaeelaudibert
Copy link
Copy Markdown
Member

Summary

  • The current display for Skill tool calls dumps the raw input as JSON, e.g. Skill {"skill":"implementing-mcp-tools"}.
  • This change renders it as Reading <skill-name> skill with the skill name in the accent color, matching how we already style input previews.

Test plan

  • Trigger a session that invokes a skill (e.g. ask the agent to use the implementing-mcp-tools skill) and confirm the tool row reads Reading implementing-mcp-tools skill with the skill name highlighted.
  • Confirm non-Skill tool calls (Read, Bash, Grep, MCP tools, etc.) are unchanged.
  • Confirm the row is still expandable and the expanded view still shows the full raw input.

Replace the raw JSON dump for Skill tool calls (e.g. `Skill {"skill":"implementing-mcp-tools"}`) with a friendlier display: `Reading <skill-name> skill`, with the skill name in the accent color.

Generated-By: PostHog Code
Task-Id: 8188a460-b85f-4aa0-b936-d65982aaba74
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 29, 2026

Reviews (1): Last reviewed commit: "Show skill tool calls as "Reading `<name..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@k11kirky k11kirky left a comment

Choose a reason for hiding this comment

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

Image

Looks good, would be nice for tool search too

Apply the same friendly display to ToolSearch calls as Skill calls, per PR review feedback. Generalize the special-casing into a small lookup table mapping tool name to prefix/suffix/highlighted input field.

Generated-By: PostHog Code
Task-Id: 8188a460-b85f-4aa0-b936-d65982aaba74
Copy link
Copy Markdown
Member Author

@k11kirky added tool search too!

@rafaeelaudibert rafaeelaudibert merged commit 35f8549 into main Jun 1, 2026
18 checks passed
@rafaeelaudibert rafaeelaudibert deleted the posthog-code/skill-tool-display branch June 1, 2026 18:47
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.

2 participants