feat(blocks): add external-service url to block metadata#5032
Conversation
- Add optional `url` field to BlockMeta for the integration's own homepage (e.g. exa.ai, salesforce.com), distinct from docsLink which points at Sim's docs - Populate `url` on all 209 integration blocks with verified homepages (protocol/internal blocks without a single vendor site are left without one) - Document the field in the add-block skill and command, with rule + checklist entries - Backfill missing docsLink on dspy and add a few accurate tag entries
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview
Reviewed by Cursor Bugbot for commit 864adb1. Configure here. |
Revert the agiloft/exa/tailscale tag additions; integrations.json (used by landing/SEO) is not regenerated here, so the expanded tags would diverge from getAllBlockMeta(). Keep the url additions, which are not projected into integrations.json.
|
@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 37bc525. Configure here.
Greptile SummaryThis PR introduces an optional
Confidence Score: 5/5Safe to merge — this is a pure catalog metadata addition with no runtime or executor impact; all changes are confined to BlockMeta objects never read by the workflow engine. The only changed runtime surface is the new optional field on a presentation-only interface. All 209 URL values are string literals in static objects. The two nits found (a locale segment in the SendGrid URL and about.gitlab.com vs gitlab.com) are style-level metadata inaccuracies with no execution consequences. apps/sim/blocks/blocks/sendgrid.ts (locale segment in URL) and apps/sim/blocks/blocks/gitlab.ts (marketing subdomain vs service root) are worth a quick glance before merge. Important Files Changed
Entity Relationship Diagram%%{init: {'theme': 'neutral'}}%%
erDiagram
BlockMeta {
readonly_IntegrationTag[] tags
string url "optional — external service homepage"
readonly_BlockTemplate[] templates "optional"
readonly_SuggestedSkill[] skills "optional"
}
BlockConfig {
string docsLink "points to docs.sim.ai"
string type
string name
string description
}
BlockMeta ||--o{ BlockConfig : "catalog metadata for"
Reviews (2): Last reviewed commit: "fix(blocks): correct inaccurate tool URL..." | Re-trigger Greptile |
- spotify: open.spotify.com (web player) -> www.spotify.com brand homepage - sts: point to the STS API reference (aws.amazon.com/sts 404s; STS has no standalone product page) instead of the shared IAM page - microsoft: drop locale-specific /en-us/ segments across Excel, Teams, OneDrive, SharePoint, Outlook, Dataverse, Planner, Entra ID, and shorten OneDrive/Outlook to stable product roots
|
@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 864adb1. Configure here.
Summary
urlfield toBlockMeta— the integration's own homepage (e.g.https://exa.ai,https://www.salesforce.com), the "link back to the tool". It's distinct fromdocsLink, which points at Sim's own docs ondocs.sim.ai.urlon all 209 integration blocks with homepages that were verified to resolve to the right service. Protocol/internal blocks with no single vendor site (IMAP, RSS, the built-in enrichment cascade) are intentionally left without one./add-blockskill and command (example, rule, and checklist entries).dspy's missingdocsLinkand add a few accuratetagsentries.Type of Change
Testing
tsc --noEmitpasses (no new errors; the lone pre-existingtailwind.config.tserror is unrelated)check:api-validation:strictpassesChecklist