improvement(blocks): name the integration in every suggested-action template title#5775
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The fix is titles only in block metadata—no prompts, runtime logic, or display-layer prefixing. A few templates also use the block’s own icon instead of another product’s icon (e.g. GitHub, Gmail, Linear, Stripe), with matching unused icon imports removed. Reviewed by Cursor Bugbot for commit 2ba526f. Configure here. |
Greptile SummaryThis PR makes suggested-action titles identify their associated integrations. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (5): Last reviewed commit: "fix(blocks): align Greptile Slack Q&A bo..." | Re-trigger Greptile |
…emplate title Home-screen "Suggested actions" rows render only an icon + the block template title, so titles that did not name their integration (e.g. "Refund pattern monitor") were unidentifiable. Rewrite the 234 titles that omitted their integration so each names it naturally, matching each block's existing "Brand + phrase" style. Titles-only change; the catalog surfaces that already show the integration as page context are unaffected.
…n, not owner The Confluence-owned incident template uses a PagerDutyIcon, is triggered by PagerDuty, and is cross-listed to notion/pagerduty/datadog/slack, so prefixing the owner name misidentified it on those surfaces and fought its own icon. Name it after the icon + trigger integration instead.
f923fe9 to
ab92e2a
Compare
|
@cursor review |
…ecrawl names For rows where the owner-prefixed title named a different integration than the row's icon (github/Notion, stripe/Sheets, calendar/Twilio, gmail/Lemlist, slack/Linear, linear/Slack), swap the icon to the owner integration the title already names so the icon-only home row is coherent; drop the now-unused icon imports. Differentiate two Firecrawl titles that collided with existing near-duplicate templates.
|
@cursor review |
…llided Renamed a handful of titles whose brand prefix made them near-identical to a sibling template: google_translate (Intercom replier / ticket auto-reply), sentry (on-call triage agent), google_drive (personal notes assistant), sqs (alert enricher), typeform (survey summarizer). Swept every renamed block for the same near-duplicate class.
|
@cursor review |
The renamed title leads with Greptile but the row kept icon: SlackIcon, conflicting on the icon-only home surface. Swap to GreptileIcon (matching the block's sibling templates) and drop the orphaned SlackIcon import.
|
@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 2ba526f. Configure here.
Summary
/integrations/[slug]) already show the integration as page context. This keeps all three surfaces consistent.Type of Change
Testing
Tested manually — re-scanned all titles (0 genuinely-nameless remain),
tsc --noEmitclean, verified onlytitle:lines changed and every rewrite reads accurately against its prompt.Checklist