Skip to content

feat(blocks): add external-service url to block metadata#5032

Merged
waleedlatif1 merged 3 commits into
stagingfrom
feat/block-meta-tool-url
Jun 13, 2026
Merged

feat(blocks): add external-service url to block metadata#5032
waleedlatif1 merged 3 commits into
stagingfrom
feat/block-meta-tool-url

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Add an optional url field to BlockMeta — the integration's own homepage (e.g. https://exa.ai, https://www.salesforce.com), the "link back to the tool". It's distinct from docsLink, which points at Sim's own docs on docs.sim.ai.
  • Populate url on 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.
  • Document the field in the /add-block skill and command (example, rule, and checklist entries).
  • Backfill dspy's missing docsLink and add a few accurate tags entries.

Type of Change

  • New feature (catalog metadata)

Testing

  • tsc --noEmit passes (no new errors; the lone pre-existing tailwind.config.ts error is unrelated)
  • check:api-validation:strict passes
  • biome clean on all changed files
  • Every URL was fetch/search-verified against the live site

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)

- 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
@vercel

vercel Bot commented Jun 13, 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 Jun 13, 2026 7:18pm

Request Review

@cursor

cursor Bot commented Jun 13, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Catalog-only metadata on BlockMeta (not used by the executor); no auth, runtime, or API behavior changes beyond many static URL strings.

Overview
Adds an optional url on BlockMeta for the integration vendor’s canonical homepage (separate from docsLink on Sim docs). Integration catalog blocks are backfilled with verified homepages; first-party or protocol blocks without a single vendor site stay unset.

/add-block skill and command now document url in the meta template, rules, and checklist. dspy also gets a missing docsLink in this pass.

Reviewed by Cursor Bugbot for commit 864adb1. Configure here.

Comment thread apps/sim/blocks/blocks/agiloft.ts
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.
@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 37bc525. Configure here.

@greptile-apps

greptile-apps Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR introduces an optional url field to BlockMeta for linking each integration block back to its external service's homepage (distinct from docsLink, which points at Sim's own docs). It then backfills that field across 209 integration blocks, updates the add-block skill and command docs, and fixes a pre-existing missing docsLink on the dspy block.

  • Type change (types.ts): adds url?: string to BlockMeta with a JSDoc comment — non-breaking, clean addition.
  • Bulk metadata backfill (209 block files): each gets a verified homepage URL; V1/V2 block pairs both receive the same URL, and protocol/built-in blocks without a single vendor site are intentionally left without one.
  • Docs update (.agents/skills/add-block/SKILL.md, .claude/commands/add-block.md): new example, rule, and checklist entry for the url field are consistent with the type definition.

Confidence Score: 5/5

Safe 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

Filename Overview
apps/sim/blocks/types.ts Adds optional url?: string to BlockMeta interface with clear JSDoc distinguishing it from docsLink; clean, non-breaking change
apps/sim/blocks/blocks/sendgrid.ts URL contains /en-us/ locale segment (https://www.twilio.com/en-us/sendgrid) — same instability pattern fixed for Microsoft URLs elsewhere in this PR; sendgrid.com is the canonical domain
apps/sim/blocks/blocks/gitlab.ts Uses https://about.gitlab.com (marketing site) rather than https://gitlab.com (the actual service/tool root domain)
apps/sim/blocks/blocks/dspy.ts Backfills missing docsLink and adds url: 'https://dspy.ai'; both look correct
.agents/skills/add-block/SKILL.md Adds url field example, rule, and checklist entry to the add-block skill; documentation is clear and consistent with the type definition
.claude/commands/add-block.md Mirrors the SKILL.md changes — adds url to the template and checklist; no issues

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"
Loading

Reviews (2): Last reviewed commit: "fix(blocks): correct inaccurate tool URL..." | Re-trigger Greptile

Comment thread apps/sim/blocks/blocks/spotify.ts Outdated
Comment thread apps/sim/blocks/blocks/sts.ts Outdated
Comment thread apps/sim/blocks/blocks/onedrive.ts Outdated
- 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
@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 864adb1. Configure here.

@waleedlatif1 waleedlatif1 merged commit 1205730 into staging Jun 13, 2026
15 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/block-meta-tool-url branch June 13, 2026 19:37
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