docs(integrations): add missing manual intros; fix light brand tiles rendering white glyphs - #6774
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Fixes light-background brand tiles showing near-white icons. Reviewed by Cursor Bugbot for commit 74b824a. Configure here. |
Greptile SummaryThe PR adds manual introductions to eight integration pages and expands Tailwind content scanning so utility classes emitted outside the previously scanned directories are generated. The Snowflake wording attempts to correct the previously reported query-result export instructions, but one unsupported alternative remains.
Confidence Score: 4/5The PR should not merge until the Snowflake introduction stops recommending views as unload sources. The reply says the prior issue was fixed in 74b824a, but the revised bullet still directs readers to materialize results as a view while the unload implementation only generates a qualified table-source statement; retaining only the working table-materialization alternative resolves the residual failure. Files Needing Attention: apps/docs/content/docs/en/integrations/snowflake.mdx
|
| Filename | Overview |
|---|---|
| apps/docs/content/docs/en/integrations/snowflake.mdx | Adds the integration introduction and attempts to correct unload guidance, but still recommends an unsupported view-source workflow. |
| apps/sim/tailwind.config.ts | Expands Tailwind scanning to application directories that emit utility-class strings. |
| apps/docs/content/docs/en/integrations/buffer.mdx | Adds a manual introduction describing supported Buffer actions. |
| apps/docs/content/docs/en/integrations/clickup.mdx | Adds a manual introduction describing supported ClickUp actions and triggers. |
| apps/docs/content/docs/en/integrations/mssql.mdx | Adds a manual introduction describing SQL Server operations. |
| apps/docs/content/docs/en/integrations/rocketlane.mdx | Adds a manual introduction describing Rocketlane project-delivery operations. |
| apps/docs/content/docs/en/integrations/smartlead.mdx | Adds a manual introduction describing Smartlead campaign operations. |
| apps/docs/content/docs/en/integrations/splunk.mdx | Adds a manual introduction describing Splunk searches, jobs, and alerts. |
| apps/docs/content/docs/en/integrations/tiktok.mdx | Adds a manual introduction describing TikTok profile, video, draft, and event capabilities. |
Reviews (2): Last reviewed commit: "docs(snowflake): correct the unload-data..." | 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 74b824a. Configure here.
Summary
Two related integration-surface fixes.
1. Missing manual intro sections (docs)
Every generated integration page carries a hand-written
MANUAL-CONTENT:introblock between theBlockInfoCardand## Usage Instructions. Eight were missing one: Buffer, ClickUp, Microsoft SQL Server, Rocketlane, Smartlead, Snowflake, Splunk, TikTok.Each got an intro in the existing shape — a linked paragraph on the service, a "With the X integration in Sim, you can:" list derived from that block's real actions and triggers, and a closing paragraph on what agents do with it. The 22
*-service-accountcredential guides,index.mdx, andhubspot-setup.mdxare hand-written pages with no backing block (they're in the generator'sHANDWRITTEN_INTEGRATION_DOCSset), so they intentionally keep no markers.2. Light-background brand tiles rendered a white glyph
Infisical (
#F7FE62), Intercom, Notion, and Daytona (#FFFFFF) showed a white icon on a light tile — near-invisible on the white ones.The component logic was correct the whole time.
getTileIconColorClass()inblocks/icon-color.tsreturns the literal'text-black'for a light tile, and the icons paint withfill='currentColor'. The problem is that.text-blackwas never generated:blocks/was not in the Tailwindcontentglobs, so the string literal inicon-color.tswas never scanned — and no scanned file contains a baretext-blackcandidate, onlydark:text-blackandtext-black/75, which compile to different selectors. With no rule to match,currentColorfell back to the inherited ambient color, which is near-white in dark mode. Dark tiles looked fine only becausetext-whiteis emitted elsewhere, which is why this hid for so long.Building the utility layer against the config confirms it —
.text-black{appears 0 times before the change and once after.Fixed by adding the unscanned directories that emit classes (
blocks,ee,providers,tools,content) tocontent, matching the reasoning the config already documents for the./lib/**glob. Diffing emitted classes before/after shows 14 that were silently dead, all now live:text-black,!text-black,!text-white—blocks/icon-color.ts(the tile bug, including the canvas block tiles that passimportant)font-bold—blocks/block-tile.tsxmax-h-44,ml-6,w-[380px]— workspace-forking picker, file tree, sync viewmin-h-[60px],hover:bg-transparent— SSO settings;pl-[46px]— verified domains!size-[9px]— access-control group detail;hover:bg-[var(--surface-3)]— whitelabeling and custom-block detailNote: scanning
blocks/also makes Tailwind extract one false-positive candidate from a Cypher example in the Neo4j block's help text, emitting a harmless 30-byte.\[r\:KNOWS\]{r:KNOWS}rule that no element carries. Narrowing the glob to dodge it would recreate exactly the bug class this PR fixes, so the broad glob stays.Type of Change
Testing
bun run docs:checkreports "Generated integration docs are in sync" — the new intro blocks round-trip throughscripts/generate-docs.tsunchanged. The tile fix was verified by building the utility layer with the Tailwind CLI against the real config and diffing emitted class names before and after.bun run lint,bun run check:audits(29 audits), andcheck-block-registry.ts origin/stagingall pass.Checklist