fix(copilot): show custom block names in read tool rows - #7044
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview
Tests cover path resolution, display text, and jsdom hydration scenarios for icons and read titles. Reviewed by Cursor Bugbot for commit 913dd20. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR resolves organization custom-block read paths through the client block registry and refreshes existing Copilot read rows when custom-block metadata hydrates.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previously reported stale metadata path now subscribes to overlay changes and re-resolves both labels and icons after hydration.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/copilot/tools/client/read-block.ts | Extends read-target resolution to organization custom-block schemas through the overlay-aware block registry. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/message-content.tsx | Recomputes parsed tool-call display titles whenever the custom-block overlay version changes. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/tool-call-item.tsx | Re-resolves read-target icons on overlay updates so rows mounted before hydration refresh correctly. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/message-content-overlay.test.tsx | Verifies that an already-mounted read title refreshes after custom-block registry hydration. |
Reviews (2): Last reviewed commit: "fix(copilot): refresh custom block metad..." | Re-trigger Greptile
|
@cursor review |
There was a problem hiding this comment.
✅ 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 913dd20. Configure here.
Summary
Type of Change
Testing
bunx vitest run(71 focused tests)bun run type-checkbun run lintbun run apps/sim/scripts/check-block-registry.ts origin/stagingbun run check:auditsbun run docs-manifest:checkChecklist