Skip to content

docs(integrations): add missing manual intros; fix light brand tiles rendering white glyphs - #6774

Merged
waleedlatif1 merged 3 commits into
stagingfrom
docs/integration-manual-intros
Aug 17, 2026
Merged

docs(integrations): add missing manual intros; fix light brand tiles rendering white glyphs#6774
waleedlatif1 merged 3 commits into
stagingfrom
docs/integration-manual-intros

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

Two related integration-surface fixes.

1. Missing manual intro sections (docs)

Every generated integration page carries a hand-written MANUAL-CONTENT:intro block between the BlockInfoCard and ## 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-account credential guides, index.mdx, and hubspot-setup.mdx are hand-written pages with no backing block (they're in the generator's HANDWRITTEN_INTEGRATION_DOCS set), 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() in blocks/icon-color.ts returns the literal 'text-black' for a light tile, and the icons paint with fill='currentColor'. The problem is that .text-black was never generated: blocks/ was not in the Tailwind content globs, so the string literal in icon-color.ts was never scanned — and no scanned file contains a bare text-black candidate, only dark:text-black and text-black/75, which compile to different selectors. With no rule to match, currentColor fell back to the inherited ambient color, which is near-white in dark mode. Dark tiles looked fine only because text-white is 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) to content, 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-whiteblocks/icon-color.ts (the tile bug, including the canvas block tiles that pass important)
  • font-boldblocks/block-tile.tsx
  • max-h-44, ml-6, w-[380px] — workspace-forking picker, file tree, sync view
  • min-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 detail

Note: 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

  • Bug fix
  • Documentation

Testing

bun run docs:check reports "Generated integration docs are in sync" — the new intro blocks round-trip through scripts/generate-docs.ts unchanged. 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), and check-block-registry.ts origin/staging all pass.

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)

@vercel

vercel Bot commented Aug 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 17, 2026 7:52am

Request Review

@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation and Tailwind content-path changes only; no runtime auth or data logic. Slightly broader CSS output from expanded globs, including one harmless false-positive rule from Neo4j help text.

Overview
Adds hand-written MANUAL-CONTENT:intro sections to eight integration docs (Buffer, ClickUp, Microsoft SQL Server, Rocketlane, Smartlead, Snowflake, Splunk, TikTok) so they match other generated integration pages: service overview, capability bullets, and an agent-use closing paragraph.

Fixes light-background brand tiles showing near-white icons. getTileIconColorClass() already returns text-black for light tiles, but Tailwind never emitted .text-black because blocks/ (and related dirs) were missing from the content scan globs. The config now includes blocks, ee, providers, tools, and content, which also restores several other utilities that were never generated from those trees.

Reviewed by Cursor Bugbot for commit 74b824a. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The 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.

  • Adds generated-doc-compatible manual introduction blocks for Buffer, ClickUp, SQL Server, Rocketlane, Smartlead, Snowflake, Splunk, and TikTok.
  • Adds block, enterprise, provider, tool, and content directories to Tailwind class discovery.
  • Revises Snowflake unload guidance to require materialization before export.

Confidence Score: 4/5

The 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

Important Files Changed

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

Comment thread apps/docs/content/docs/en/integrations/snowflake.mdx Outdated
@waleedlatif1 waleedlatif1 changed the title docs(integrations): add manual intro sections to eight integration pages docs(integrations): add missing manual intros; fix light brand tiles rendering white glyphs Aug 17, 2026
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

Comment thread apps/docs/content/docs/en/integrations/snowflake.mdx
@waleedlatif1
waleedlatif1 merged commit b38e4e2 into staging Aug 17, 2026
23 of 24 checks passed
@waleedlatif1
waleedlatif1 deleted the docs/integration-manual-intros branch August 17, 2026 07:49

@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 74b824a. Configure here.

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