Uh oh!
There was an error while loading. Please reload this page.
improvement(integrations): add Managed Agents templates and guard docs links - #6608
Conversation
…s links The Claude Managed Agents block shipped without templates or suggested skills, so its integration detail page had nothing to offer and the "Add to Sim" chat handoff was the only affordance. Add nine templates and seven skills, each grounded in an operation the block actually exposes. Its docsLink also pointed at integrations/managed-agent while the page is managed_agent, so the link 404'd. Five more blocks had the same class of bug via a stale tools/ prefix. Nothing validated these, because the catalog check only compares deployment fields. Add that validation, and collapse the three copies of the docs-URL contract onto one exported helper so the checker and the generator cannot drift apart.
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Fixes 6 broken Adds a CI check in Reviewed by Cursor Bugbot for commit 12dc12b. Configure here. |
Greptile SummaryThe PR adds Managed Agents templates and skills, corrects six integration documentation URLs, and centralizes generated documentation URL handling.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previously reported omitted-link stale-allowlist bypass is fixed by tracking vendor links from each block’s resolved documentation URL.
|
| Filename | Overview |
|---|---|
| scripts/check-integration-catalog.ts | Adds documentation-link validation and correctly records vendor-linked types from resolved URLs, fixing the previously reported stale-allowlist bypass. |
| scripts/generate-docs.ts | Exports the documentation origin, output path, and default integration URL helper so generation and validation share one contract. |
| apps/sim/blocks/blocks/managed_agent.ts | Corrects the documentation URL and adds nine templates and seven skills to the Managed Agents integration metadata. |
| apps/sim/lib/integrations/integrations.json | Updates generated catalog documentation URLs to match the corrected integration routes. |
Reviews (2): Last reviewed commit: "fix(integrations): resolve docsLink once..." | Re-trigger Greptile
Uh oh!
There was an error while loading. Please reload this page.
…owlist The stale-allowlist predicate read block.docsLink directly while the main loop read the resolved link, so an allowlisted block that dropped its explicit docsLink produced undefined from the optional chain, negated to true, and was treated as still vendor-linked — the stale entry went undetected. Record vendor-linked types during the single pass that already resolves each link, so both checks agree by construction.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
waleedlatif1
commented
Aug 12, 2026
waleedlatif1
commented
Aug 12, 2026
@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 12dc12b. Configure here.
…ions claims alsoIntegrations is documented as the blocks a template's prompt references, and it drives which catalog pages the template cross-lists on plus the icon cluster on the detail page. Three prompts named a service only implicitly, or not at all: the runaway stopper claimed Slack without mentioning it, the PR reviewer said "pull request" rather than GitHub, and the weekly report said "emails" rather than Gmail. Name the service in each prompt so the field is accurate and the templates surface on the right pages.
Six blocks had a vendor documentation URL in docsLink — cursor, enrich, enrow, google_groups, qdrant, and similarweb — which was accidental rather than deliberate. Each already has a generated Sim page, and each already carries the vendor's homepage on BlockMeta.url, so the vendor link in docsLink only sent readers away from our own documentation. Point all six at their Sim page and drop the allowlist that had been tolerating them. Every visible integration gets a generated page, so a docsLink outside docs.sim.ai is now always an error.
Uh oh!
There was an error while loading. Please reload this page.
Summary
docsLinkvalues.managed_agentpointed atintegrations/managed-agent(hyphen) while the page ismanaged_agent;datagma,dropcontact,icypeas,leadmagic, andthriveused a staletools/prefix. All six 404'd.check-integration-catalog.tsnow resolves every visible integration'sdocsLinkto a real.mdx, and the 6 blocks that deliberately link to vendor docs are an explicit allowlist rather than an unchecked pass — a newly pasted vendor URL, or a stale allowlist entry, now fails.generate-docs.tshad two copies of the default and the checker was adding a third; it now ownsdefaultIntegrationDocsUrl()and both consumers use it, so the checker can't validate a different rule than the generator emits.Type of Change
Testing
Verified the guard fails on each case it exists to catch — a broken Sim link, an un-allowlisted vendor link, and a stale allowlist entry — and returns green when restored. Confirmed the 9 templates resolve as owned with valid
alsoIntegrations, and the 7 skill names are unique.bun run scripts/check-integration-catalog.tspasses (237 integrations),type-checkand biome are clean, andblocks/tests pass (510).Checklist