Skip to content

Upgrades the agent connectors docs#455

Open
saif-at-scalekit wants to merge 3 commits intomainfrom
super-upgrade-connectors-mds
Open

Upgrades the agent connectors docs#455
saif-at-scalekit wants to merge 3 commits intomainfrom
super-upgrade-connectors-mds

Conversation

@saif-at-scalekit
Copy link
Collaborator

@saif-at-scalekit saif-at-scalekit commented Mar 1, 2026

  • scripts that automates generation of agent connector documetation
  • templatised the setup instructions
  • adds gifts and images necessary

Summary by CodeRabbit

Release Notes

  • New Features
    • Added comprehensive setup guides for 35+ agent connectors including Airtable, Asana, BigQuery, ClickUp, Confluence, Dropbox, GitHub, Gmail, Google services (Docs, Drive, Forms, Sheets, Calendar, Meet), HubSpot, Jira, Linear, Microsoft services (Teams, Excel, Word), Notion, Salesforce, Slack, Snowflake, Zendesk, Zoom, and more. Each guide provides step-by-step instructions, visual aids, and links to relevant documentation for easy configuration.

…rious connectors

- Introduced setup sections for Airtable, Asana, BigQuery, ClickUp, Confluence, Dropbox, GitHub, Gmail, Gong, Google Ads, Google Calendar, HubSpot, Intercom, Jira, Linear, Monday, Notion, OneDrive, OneNote, Outlook, Salesforce, ServiceNow, SharePoint, Slack, Snowflake, Zendesk, and Zoom.
- Each connector now includes a dedicated setup component to streamline the integration process.
- Added backticks around JSON examples in property tables for better readability.
- Improved formatting of example structures to ensure consistency across the documentation.
- Clarified runtime notes and descriptions for better understanding of the properties and their usage.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 1, 2026

Walkthrough

This PR adds 35 setup documentation templates for agent connectors and enhances the sync script to automatically discover, organize, and inject these setup components into generated MDX documentation. A new documentation file describes the sync workflow.

Changes

Cohort / File(s) Summary
Documentation & Sync Script
project-docs/SYNC_AGENT_CONNECTORS.md, scripts/sync-agent-connectors.js
Introduces setup section discovery mechanism (buildSetupStemMap, getSetupComponent), template index regeneration (syncTemplateIndex), and dynamic setup component injection into MDX generation. Includes comprehensive guide on running the sync script and managing connector setup templates.
Setup Templates (Connectors)
src/components/templates/agent-connectors/_setup-*.mdx (35 files: airtable, asana, bigquery, clickup, confluence, dropbox, github, gmail, gong, google-ads, google-docs, google-drive, google-forms, google-meets, google-sheets, googlecalendar, hubspot, intercom, jira, linear, microsoft-excel, microsoft-teams, microsoft-word, monday, notion, onedrive, onenote, outlook, salesforce, servicenow, sharepoint, slack, snowflake, zendesk, zoom)
Step-by-step setup guides for 35 agent connectors, each importing UI components (Steps, Aside) and image assets, providing consistent multi-step OAuth/credential configuration flows tailored to each provider's requirements.
Component Exports
src/components/templates/agent-connectors/index.ts, src/components/templates/index.ts
Re-exports all 35 SetupXxxSection components from setup MDX templates; aggregates agent-connectors exports into parent templates module for unified API surface.
Reference Documentation Updates
src/content/docs/reference/agent-connectors/*.mdx (35 files: airtable, asana, bigquery, clickup, confluence, dropbox, github, gmail, gong, google-ads, googlecalendar, hubspot, intercom, jira, linear, monday, notion, onedrive, onenote, outlook, salesforce, servicenow, sharepoint, slack, snowflake, zendesk, zoom)
Each reference page now imports its corresponding SetupXxxSection component and injects a "Set up the agent connector" section to render the setup guide, with minor consolidation of documentation content in some pages.

Sequence Diagram

sequenceDiagram
    participant Build as Build/Sync Process
    participant FS as File System
    participant Script as sync-agent-connectors.js
    participant Gen as MDX Generator
    participant Export as Export Index

    Build->>Script: Invoke sync script
    Script->>FS: Scan src/components/templates/agent-connectors
    FS-->>Script: List _setup-*.mdx files
    Script->>Script: buildSetupStemMap() - create stem to component mapping
    Script->>Export: syncTemplateIndex(SETUP_STEM_MAP)
    Export->>FS: Regenerate index.ts with all Setup exports
    FS-->>Export: Confirm write
    
    Script->>Script: For each provider API call
    Script->>Script: getSetupComponent(provider_slug)
    Script-->>Script: Resolve to setup component name
    Script->>Gen: generateMdxContent() with setup component
    Gen->>Gen: Import dynamic setup component
    Gen->>Gen: Render "Set up the agent connector" section
    Gen->>FS: Write generated MDX to src/content/docs/reference
    FS-->>Gen: Confirm write
    
    Script->>Build: Log templates synced count
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

The PR spans 40+ files but exhibits strong pattern homogeneity: 35 setup templates follow an identical structure, 35 documentation updates use the same import-and-inject pattern, and component exports are straightforward re-exports. The primary complexity concentrates in the sync script logic (template discovery, stem mapping, dynamic component injection), which requires verification of correct slug-to-component mapping and MDX generation integration. The consistency and repetitive nature reduce per-file review burden.

Possibly related PRs

Suggested reviewers

  • ravibits
  • amitash1912
  • Avinash-Kamath
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Upgrades the agent connectors docs' accurately reflects the primary change of enhancing agent connector documentation with new setup templates, automation scripts, and improved content structure.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch super-upgrade-connectors-mds

Comment @coderabbitai help to get the list of available commands and usage tips.

@saif-at-scalekit saif-at-scalekit changed the title Upgrades the agent connectors documetnation Upgrades the agent connectors docs Mar 1, 2026
@netlify
Copy link

netlify bot commented Mar 1, 2026

Deploy Preview for scalekit-starlight ready!

Name Link
🔨 Latest commit 1a1b11c
🔍 Latest deploy log https://app.netlify.com/projects/scalekit-starlight/deploys/69a485fbb9ac160008c8d06c
😎 Deploy Preview https://deploy-preview-455--scalekit-starlight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 34

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (3)
src/content/docs/reference/agent-connectors/github.mdx (1)

2-2: ⚠️ Potential issue | 🟡 Minor

Use correct brand capitalization: "GitHub"

The title uses "Github" but the official brand name is "GitHub" with a capital H. This appears in the frontmatter title and affects both SEO and user-facing display.

📝 Suggested fix
 ---
-title: Github
+title: GitHub
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/github.mdx` at line 2, The
frontmatter title uses incorrect brand capitalization ("title: Github"); update
the frontmatter title value to use the official "GitHub" capitalization so the
line reads title: GitHub, ensuring all occurrences of the frontmatter title in
this document are corrected.
src/content/docs/reference/agent-connectors/hubspot.mdx (1)

83-83: ⚠️ Potential issue | 🟠 Major

Update CDN URL from cdn.scalekit.com to cdn.scalekit.cloud.

The HubSpot connector uses https://cdn.scalekit.com/sk-connect/assets/provider-icons/hub_spot.svg while the preferred CDN pattern across most other connector files is https://cdn.scalekit.cloud/sk-connect/assets/provider-icons/. Update this URL to use cdn.scalekit.cloud for consistency with documented guidelines.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/hubspot.mdx` at line 83, Replace
the img tag's src URL that currently points to
"https://cdn.scalekit.com/sk-connect/assets/provider-icons/hub_spot.svg" with
the preferred CDN host so it becomes
"https://cdn.scalekit.cloud/sk-connect/assets/provider-icons/hub_spot.svg";
locate the <img ... alt="HubSpot logo" /> element in the HubSpot connector doc
and update only the domain from cdn.scalekit.com to cdn.scalekit.cloud for
consistency.
src/content/docs/reference/agent-connectors/confluence.mdx (1)

83-83: ⚠️ Potential issue | 🟡 Minor

Update CDN URL to use cdn.scalekit.cloud domain.

Change line 83 from cdn.scalekit.com to cdn.scalekit.cloud to match the standard used across most agent connector documentation files.

Current code
  <img src="https://cdn.scalekit.com/sk-connect/assets/provider-icons/confluence.svg" width="64" height="64" alt="Confluence logo" />
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/confluence.mdx` at line 83,
Update the img tag's src URL domain in the Confluence connector doc: locate the
<img ...
src="https://cdn.scalekit.com/sk-connect/assets/provider-icons/confluence.svg"
/> element (the src attribute on the img tag) and change the domain from
cdn.scalekit.com to cdn.scalekit.cloud so the src becomes
https://cdn.scalekit.cloud/sk-connect/assets/provider-icons/confluence.svg.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@project-docs/SYNC_AGENT_CONNECTORS.md`:
- Line 1: Change the top-level heading "# Agent Connectors Sync Script" to
sentence case by updating it to "# Agent connectors sync script" so it follows
the docs style (use sentence case for all titles/headings); locate the existing
heading string "Agent Connectors Sync Script" and replace it with the
sentence-case version.

In `@scripts/sync-agent-connectors.js`:
- Around line 281-302: In buildSetupStemMap, guard against empty string segments
when converting stem to PascalCase: after computing stem (in buildSetupStemMap),
filter out empty segments from stem.split('-') before mapping to ensure w[0] is
never accessed on an empty string (e.g., use .split('-').filter(...)). Update
the mapping logic that builds the value for map[stem] so it only processes
non-empty words and still returns the same 'Setup' + PascalCased + 'Section'
format.
- Around line 304-307: The getSetupComponent function currently only checks
stemMap for providerSlug with underscores replaced by hyphens and with
underscores removed, which can miss keys that match the original slug; update
getSetupComponent to also try the original providerSlug as a fallback (i.e.,
check stemMap[providerSlug] before or between the existing replacements) so
lookups like "google_ads" will match "google_ads" or "google-ads" in stemMap;
modify the lookup order inside getSetupComponent to attempt providerSlug,
providerSlug.replace(/_/g, '-'), providerSlug.replace(/_/g, '') and return null
if none match.

In `@src/components/templates/agent-connectors/_setup-asana.mdx`:
- Line 35: Fix the indentation of the Step 4 list item ("Click **Save**.") so it
matches the preceding list items' indentation level in the MDX template; update
the leading spaces for the "Click **Save**." line to use the same 6-space
indentation as the previous steps (keeping the content unchanged) to ensure
consistent MDX/Steps parsing.
- Line 1: Remove the unused Aside import from the import statement that
currently pulls in Steps, Aside, Badge from '@astrojs/starlight/components';
update the import to only include the used symbols (Steps and Badge) so the
declaration no longer references Aside and eliminates the unused import.

In `@src/components/templates/agent-connectors/_setup-bigquery.mdx`:
- Around line 2-4: The file imports oauthWebAppImg and addRedirectUriImg but the
MDX uses raw markdown image paths for those images while enableApiImg.src is
used correctly; fix by choosing one approach: either remove the unused imports
(oauthWebAppImg, addRedirectUriImg) and keep the markdown image paths as-is, or
replace the markdown image references at the locations that currently use
'@/...' with JSX <img> tags that reference the imported variables
(oauthWebAppImg and addRedirectUriImg) for consistency with enableApiImg.src;
ensure no duplicate unused imports remain and image usage is consistent across
the document.

In `@src/components/templates/agent-connectors/_setup-dropbox.mdx`:
- Around line 13-15: The Aside block contains an incomplete sentence ("go
directly to the  section"); update the Aside content in this template so the
missing section name is inserted (e.g., "go directly to the testing section" or
the canonical "Test connections" section used in other templates), and ensure
the sentence matches wording used in other connector templates for consistency;
edit the Aside component text to include the correct section name and verify the
same fix is applied to other templates with the same issue.

In `@src/components/templates/agent-connectors/_setup-github.mdx`:
- Around line 14-16: The Aside component contains an incomplete sentence "go
directly to the  section"; update that phrase to include the missing section
name (e.g., mirror the Gmail template's wording) so the sentence reads clearly
(for example "go directly to the Testing section" or the exact section title
used elsewhere). Locate the Aside block in _setup-github.mdx and replace the
blank in "go directly to the  section" with the correct section header text used
across the docs to ensure consistency.

In `@src/components/templates/agent-connectors/_setup-gmail.mdx`:
- Around line 20-22: The Aside block contains a missing section reference ("go
directly to the  section"); update the Aside text in the _setup-gmail.mdx file
(the Aside component) to include the correct target section name or heading
(e.g., "Testing" or the exact heading used elsewhere in this doc such as
"Testing the connector"), so the sentence reads "go directly to the Testing
section" (or the exact heading) to give users a clear navigation target.

In `@src/components/templates/agent-connectors/_setup-gong.mdx`:
- Around line 13-15: The Aside text contains a missing section reference ("go
directly to the  section"); open
src/components/templates/agent-connectors/_setup-gong.mdx, locate the Aside
component and replace the blank with the correct target section title or anchor
(e.g., the "Testing" or "Quickstart" section name used elsewhere in this
document), and, if possible, add an internal link/anchor to that section so the
phrase reads like "go directly to the <SectionName> section" and navigates
correctly.

In `@src/components/templates/agent-connectors/_setup-google-ads.mdx`:
- Around line 19-21: The Aside block in _setup-google-ads.mdx contains an
incomplete phrase "go directly to the  section"; edit the Aside content to
replace that fragment with the correct section name (e.g., "go directly to the
Testing section" or the actual target section title used elsewhere) so the
sentence reads naturally; update the Aside component text in the file to
reference the exact section heading used in the document.

In `@src/components/templates/agent-connectors/_setup-google-sheets.mdx`:
- Around line 19-21: The Aside text contains an incomplete sentence ("go
directly to the  section") inside the Aside component; update that string to
include the missing section name (for example: "go directly to the 'Testing'
section" or the specific section title used elsewhere) and remove the extra
space so the sentence reads correctly; locate and edit the Aside element's
content in _setup-google-sheets.mdx to replace the broken fragment with the
complete section reference.

In `@src/components/templates/agent-connectors/_setup-intercom.mdx`:
- Line 1: Remove the unused Aside import from the module imports: in the import
statement that currently reads "import { Steps, Aside } from
'@astrojs/starlight/components'", delete Aside so only used symbols (e.g.,
Steps) are imported to avoid unused import warnings.

In `@src/components/templates/agent-connectors/_setup-jira.mdx`:
- Around line 7-37: The Steps block uses "1. ###" headings and 4-space
indentation which violates the repo MDX standard; update each step header inside
the <Steps> block (e.g., the "Set up auth redirects", "Get client credentials",
"Add credentials in Scalekit" sections) to use "1. ##" and adjust the content
lines to use exactly 3-space indentation (not 4) so the Steps component renders
correctly; keep the existing step order and ensure the closing </Steps> tag
remains unchanged.

In `@src/components/templates/agent-connectors/_setup-linear.mdx`:
- Line 1: The import currently brings in an unused symbol Aside; remove Aside
from the import statement so only the used Steps component is imported (i.e.,
change the import that references Steps and Aside to import only Steps) to
eliminate the unused-import warning.

In `@src/components/templates/agent-connectors/_setup-microsoft-excel.mdx`:
- Line 1: Remove the unused Aside import from the module import list: update the
import statement that currently reads "import { Steps, Aside } from
'@astrojs/starlight/components'" to only import used symbols (e.g., "Steps") so
Aside is not imported; ensure no other references to Aside exist in the file
(search for "Aside" symbol) before committing the change.

In `@src/components/templates/agent-connectors/_setup-microsoft-teams.mdx`:
- Line 1: The import statement currently imports Aside but that symbol is
unused; remove Aside from the named import so only used components (e.g., Steps)
are imported from '@astrojs/starlight/components' to eliminate the unused import
(remove "Aside" from the import list in the top-level import).

In `@src/components/templates/agent-connectors/_setup-microsoft-word.mdx`:
- Line 1: The import statement includes an unused symbol "Aside" which should be
removed to avoid dead imports; update the import from
'@astrojs/starlight/components' by deleting "Aside" so only the used "Steps"
symbol remains (ensure any usages of Aside are not present elsewhere in this
template, e.g., references around Steps in the _setup-microsoft-word.mdx
content).
- Line 10: Update the Azure terminology in the instruction: replace the text
"Azure Active Directory" with "Microsoft Entra ID" in the Sign-in step (the
sentence starting "Sign into [portal.azure.com]... → **Azure Active Directory**
→ **App registrations** → **New registration**") so the guidance reads "...go to
**Microsoft Entra ID** → **App registrations** → **New registration**".

In `@src/components/templates/agent-connectors/_setup-outlook.mdx`:
- Line 10: Replace the outdated term "Azure Active Directory" with "Microsoft
Entra ID" in the sentence that reads "Sign into portal.azure.com and go to
**Azure Active Directory** → **App registrations** → **New registration**;
specifically update the bolded navigation label to **Microsoft Entra ID** while
keeping the rest of the instruction (portal link, App registrations, New
registration) unchanged.

In `@src/components/templates/agent-connectors/_setup-servicenow.mdx`:
- Around line 14-16: The Aside text contains a missing section name ("go
directly to the  section"); update the string inside the Aside component in
src/components/templates/agent-connectors/_setup-servicenow.mdx so it matches
the same phrasing used in other templates (e.g., "go directly to the Testing
section" or "go directly to the Test connection section"), replacing the blank
with the correct section title and keeping the surrounding sentence unchanged.

In `@src/components/templates/agent-connectors/_setup-sharepoint.mdx`:
- Line 10: Update the outdated "Azure Active Directory" wording in the step that
currently reads "Sign into https://portal.azure.com and go to **Azure Active
Directory** → **App registrations** → **New registration**: replace the text to
reference "Microsoft Entra ID" and point users to the Microsoft Entra admin
center by using the URL https://entra.microsoft.com (e.g., "Sign into
https://entra.microsoft.com and go to **Microsoft Entra ID** → **App
registrations** → **New registration**"), ensuring all occurrences of the string
"Azure Active Directory" in this file are changed to "Microsoft Entra ID".

In `@src/components/templates/agent-connectors/_setup-slack.mdx`:
- Line 1: The import list contains an unused symbol Aside; remove Aside from the
named import so only used components (e.g., Steps) are imported from
'@astrojs/starlight/components' to eliminate the unused import warning and keep
imports minimal.

In `@src/components/templates/agent-connectors/_setup-zendesk.mdx`:
- Line 1: The file imports Aside but never uses it; remove the unused symbol
from the import statement so only Steps is imported (i.e., change the import
line that currently reads "import { Steps, Aside } from
'@astrojs/starlight/components'" to import only Steps). This eliminates the
unused-import warning while keeping the Steps import intact.
- Line 8: Update the external "Scalekit dashboard" anchor links in the
_setup-zendesk.mdx content (the links that point to https://app.scalekit.com and
any other external Scalekit URL referenced, e.g., the one instructing to go to
Agent Auth → Create Connection) to include target="_blank" and rel="noopener"
attributes; find the anchor elements whose link text contains "Scalekit
dashboard" and add these attributes to both occurrences to ensure they open in a
new tab securely.

In `@src/components/templates/agent-connectors/_setup-zoom.mdx`:
- Line 1: The import list includes an unused symbol Aside; remove Aside from the
import statement (leave Steps) in the module so the line imports only Steps from
'@astrojs/starlight/components' and eliminate the unused import warning (i.e.,
update the import that currently reads "import { Steps, Aside } ..." to only
include the used symbol Steps).

In `@src/components/templates/agent-connectors/index.ts`:
- Line 16: The exported identifier names use inconsistent PascalCase: rename
exports in src/components/templates/agent-connectors/index.ts to use
SetupGoogleCalendarSection, SetupOneDriveSection, and SetupOneNoteSection
(update the current exports SetupGooglecalendarSection, SetupOnedriveSection,
SetupOnenoteSection respectively), and update all imports that reference them in
src/content/docs/reference/agent-connectors/googlecalendar.mdx, onedrive.mdx,
and onenote.mdx to use the new identifiers so references to the exported
components match the corrected PascalCase names.

In `@src/content/docs/reference/agent-connectors/confluence.mdx`:
- Around line 74-76: The import statement in this file brings in unused
components (Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code, Accordion,
AccordionItem) which should be removed to clean up the module; update the import
lines so only SetupConfluenceSection and any actually used components (e.g.,
Badge if used) remain, keeping the import from '@astrojs/starlight/components'
and 'accessible-astro-components' minimal and removing references to unused
symbols (Card, CardGrid, Tabs, TabItem, Steps, Aside, Code, Accordion,
AccordionItem) to eliminate dead imports.

In `@src/content/docs/reference/agent-connectors/google_ads.mdx`:
- Line 74: Remove the unused Code import from the import list so the file only
imports components that are actually used; update the import statement that
currently reads "import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside,
Code } from '@astrojs/starlight/components'" to exclude Code, leaving the other
symbols (Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside) intact.

In `@src/content/docs/reference/agent-connectors/hubspot.mdx`:
- Line 74: Remove the unused Code import from the component import list: locate
the import statement that reads "import { Card, CardGrid, Tabs, TabItem, Badge,
Steps, Aside, Code } from '@astrojs/starlight/components'" and delete "Code"
from the destructured imports so only used components remain (Card, CardGrid,
Tabs, TabItem, Badge, Steps, Aside).

In `@src/content/docs/reference/agent-connectors/jira.mdx`:
- Line 74: Remove the unused Code import from the import list so only used
components are imported; edit the import statement that currently reads "import
{ Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from
'@astrojs/starlight/components'" and delete the "Code" symbol, leaving the
remaining components intact (e.g., Card, CardGrid, Tabs, TabItem, Badge, Steps,
Aside).

In `@src/content/docs/reference/agent-connectors/linear.mdx`:
- Around line 74-76: The import line pulls in many unused components—remove the
unused symbols Card, CardGrid, Tabs, TabItem, Steps, Aside, and Code from the
import from '@astrojs/starlight/components' and only keep the actual used
symbol(s) (e.g., Badge); leave the other imports that are used (e.g.,
SetupLinearSection, Accordion, AccordionItem) intact so only referenced
components remain in the file.

In `@src/content/docs/reference/agent-connectors/outlook.mdx`:
- Line 74: Remove the unused Code component from the component import list:
update the import statement that currently includes "Card, CardGrid, Tabs,
TabItem, Badge, Steps, Aside, Code" to omit "Code" so only used components
remain; modify the import in the top of the MDX file (the import containing
Code) and run your linter/build to confirm no remaining references to Code.

In `@src/content/docs/reference/agent-connectors/salesforce.mdx`:
- Line 74: Remove the unused Code import from the import list that currently
reads "import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } ..."
by deleting "Code" from the destructured imports (leaving Card, CardGrid, Tabs,
TabItem, Badge, Steps, Aside) and verify there are no remaining references to
the Code symbol in this MDX so the import cleanup is safe.

---

Outside diff comments:
In `@src/content/docs/reference/agent-connectors/confluence.mdx`:
- Line 83: Update the img tag's src URL domain in the Confluence connector doc:
locate the <img ...
src="https://cdn.scalekit.com/sk-connect/assets/provider-icons/confluence.svg"
/> element (the src attribute on the img tag) and change the domain from
cdn.scalekit.com to cdn.scalekit.cloud so the src becomes
https://cdn.scalekit.cloud/sk-connect/assets/provider-icons/confluence.svg.

In `@src/content/docs/reference/agent-connectors/github.mdx`:
- Line 2: The frontmatter title uses incorrect brand capitalization ("title:
Github"); update the frontmatter title value to use the official "GitHub"
capitalization so the line reads title: GitHub, ensuring all occurrences of the
frontmatter title in this document are corrected.

In `@src/content/docs/reference/agent-connectors/hubspot.mdx`:
- Line 83: Replace the img tag's src URL that currently points to
"https://cdn.scalekit.com/sk-connect/assets/provider-icons/hub_spot.svg" with
the preferred CDN host so it becomes
"https://cdn.scalekit.cloud/sk-connect/assets/provider-icons/hub_spot.svg";
locate the <img ... alt="HubSpot logo" /> element in the HubSpot connector doc
and update only the domain from cdn.scalekit.com to cdn.scalekit.cloud for
consistency.

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 287f811 and 1a1b11c.

⛔ Files ignored due to path filters (61)
  • src/assets/docs/agent-connectors/airtable/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/airtable/add-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/airtable/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/asana/add-redirect-url.gif is excluded by !**/*.gif
  • src/assets/docs/agent-connectors/asana/enable-distribution.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/asana/get-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/bigquery/enable-bigquery-api.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/bigquery/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/clickup/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/clickup/add-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/clickup/get-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/clickup/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/confluence/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/confluence/add-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/confluence/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/dropbox/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/dropbox/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/github/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/github/add-redirect-uri.gif is excluded by !**/*.gif
  • src/assets/docs/agent-connectors/github/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/add-redirect-uri.gif is excluded by !**/*.gif
  • src/assets/docs/agent-connectors/gmail/add-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/connect-google-app.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/enable-gmail-api.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/oauth-web-app.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gmail/use-own-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gong/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/gong/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/google_ads/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/google_docs/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/google_drive/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/google_forms/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/google_meets/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/google_sheets/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/googlecalendar/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/hubspot/add-redirect-url.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/hubspot/add-scopes.gif is excluded by !**/*.gif
  • src/assets/docs/agent-connectors/intercom/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/intercom/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/jira/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/jira/add-redirect-uri.gif is excluded by !**/*.gif
  • src/assets/docs/agent-connectors/jira/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/microsoft_teams/add-redirect-url.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/microsoft_teams/bot-setup.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/monday/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/monday/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/servicenow/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/servicenow/manage-instance.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/servicenow/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/slack/add-redirect-url.gif is excluded by !**/*.gif
  • src/assets/docs/agent-connectors/slack/enable-distribution.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/snowflake/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/snowflake/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/zendesk/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/zendesk/use-own-credentials-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/zoom/add-credentials.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/zoom/add-redirect-uri.png is excluded by !**/*.png
  • src/assets/docs/agent-connectors/zoom/use-own-credentials-redirect-uri.png is excluded by !**/*.png
📒 Files selected for processing (66)
  • project-docs/SYNC_AGENT_CONNECTORS.md
  • scripts/sync-agent-connectors.js
  • src/components/templates/agent-connectors/_setup-airtable.mdx
  • src/components/templates/agent-connectors/_setup-asana.mdx
  • src/components/templates/agent-connectors/_setup-bigquery.mdx
  • src/components/templates/agent-connectors/_setup-clickup.mdx
  • src/components/templates/agent-connectors/_setup-confluence.mdx
  • src/components/templates/agent-connectors/_setup-dropbox.mdx
  • src/components/templates/agent-connectors/_setup-github.mdx
  • src/components/templates/agent-connectors/_setup-gmail.mdx
  • src/components/templates/agent-connectors/_setup-gong.mdx
  • src/components/templates/agent-connectors/_setup-google-ads.mdx
  • src/components/templates/agent-connectors/_setup-google-docs.mdx
  • src/components/templates/agent-connectors/_setup-google-drive.mdx
  • src/components/templates/agent-connectors/_setup-google-forms.mdx
  • src/components/templates/agent-connectors/_setup-google-meets.mdx
  • src/components/templates/agent-connectors/_setup-google-sheets.mdx
  • src/components/templates/agent-connectors/_setup-googlecalendar.mdx
  • src/components/templates/agent-connectors/_setup-hubspot.mdx
  • src/components/templates/agent-connectors/_setup-intercom.mdx
  • src/components/templates/agent-connectors/_setup-jira.mdx
  • src/components/templates/agent-connectors/_setup-linear.mdx
  • src/components/templates/agent-connectors/_setup-microsoft-excel.mdx
  • src/components/templates/agent-connectors/_setup-microsoft-teams.mdx
  • src/components/templates/agent-connectors/_setup-microsoft-word.mdx
  • src/components/templates/agent-connectors/_setup-monday.mdx
  • src/components/templates/agent-connectors/_setup-notion.mdx
  • src/components/templates/agent-connectors/_setup-onedrive.mdx
  • src/components/templates/agent-connectors/_setup-onenote.mdx
  • src/components/templates/agent-connectors/_setup-outlook.mdx
  • src/components/templates/agent-connectors/_setup-salesforce.mdx
  • src/components/templates/agent-connectors/_setup-servicenow.mdx
  • src/components/templates/agent-connectors/_setup-sharepoint.mdx
  • src/components/templates/agent-connectors/_setup-slack.mdx
  • src/components/templates/agent-connectors/_setup-snowflake.mdx
  • src/components/templates/agent-connectors/_setup-zendesk.mdx
  • src/components/templates/agent-connectors/_setup-zoom.mdx
  • src/components/templates/agent-connectors/index.ts
  • src/components/templates/index.ts
  • src/content/docs/reference/agent-connectors/airtable.mdx
  • src/content/docs/reference/agent-connectors/asana.mdx
  • src/content/docs/reference/agent-connectors/bigquery.mdx
  • src/content/docs/reference/agent-connectors/clickup.mdx
  • src/content/docs/reference/agent-connectors/confluence.mdx
  • src/content/docs/reference/agent-connectors/dropbox.mdx
  • src/content/docs/reference/agent-connectors/github.mdx
  • src/content/docs/reference/agent-connectors/gmail.mdx
  • src/content/docs/reference/agent-connectors/gong.mdx
  • src/content/docs/reference/agent-connectors/google_ads.mdx
  • src/content/docs/reference/agent-connectors/googlecalendar.mdx
  • src/content/docs/reference/agent-connectors/hubspot.mdx
  • src/content/docs/reference/agent-connectors/intercom.mdx
  • src/content/docs/reference/agent-connectors/jira.mdx
  • src/content/docs/reference/agent-connectors/linear.mdx
  • src/content/docs/reference/agent-connectors/monday.mdx
  • src/content/docs/reference/agent-connectors/notion.mdx
  • src/content/docs/reference/agent-connectors/onedrive.mdx
  • src/content/docs/reference/agent-connectors/onenote.mdx
  • src/content/docs/reference/agent-connectors/outlook.mdx
  • src/content/docs/reference/agent-connectors/salesforce.mdx
  • src/content/docs/reference/agent-connectors/servicenow.mdx
  • src/content/docs/reference/agent-connectors/sharepoint.mdx
  • src/content/docs/reference/agent-connectors/slack.mdx
  • src/content/docs/reference/agent-connectors/snowflake.mdx
  • src/content/docs/reference/agent-connectors/zendesk.mdx
  • src/content/docs/reference/agent-connectors/zoom.mdx

@@ -0,0 +1,56 @@
# Agent Connectors Sync Script
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Use sentence case for the top-level heading.

Line 1 is in title case; standardize it to sentence case for consistency with docs style.

✏️ Suggested edit
-# Agent Connectors Sync Script
+# Sync agent connectors
As per coding guidelines, "Use sentence case for all titles and headings in MD/MDX documentation."
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# Agent Connectors Sync Script
# Sync agent connectors
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@project-docs/SYNC_AGENT_CONNECTORS.md` at line 1, Change the top-level
heading "# Agent Connectors Sync Script" to sentence case by updating it to "#
Agent connectors sync script" so it follows the docs style (use sentence case
for all titles/headings); locate the existing heading string "Agent Connectors
Sync Script" and replace it with the sentence-case version.

Comment on lines +281 to +302
function buildSetupStemMap() {
const templatesDir = path.join(__dirname, '../src/components/templates/agent-connectors')
let files
try {
files = fs.readdirSync(templatesDir)
} catch {
return {}
}
const map = {}
for (const file of files) {
if (!file.startsWith('_setup-') || !file.endsWith('.mdx')) continue
const stem = file.replace('_setup-', '').replace('.mdx', '')
map[stem] =
'Setup' +
stem
.split('-')
.map((w) => w[0].toUpperCase() + w.slice(1))
.join('') +
'Section'
}
return map
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Guard against empty string segments in stem conversion.

If a stem contains consecutive hyphens (e.g., _setup-google--ads.mdx), split('-') produces empty strings, and w[0].toUpperCase() would throw on an empty string.

🛡️ Defensive fix
 function buildSetupStemMap() {
   const templatesDir = path.join(__dirname, '../src/components/templates/agent-connectors')
   let files
   try {
     files = fs.readdirSync(templatesDir)
   } catch {
     return {}
   }
   const map = {}
   for (const file of files) {
     if (!file.startsWith('_setup-') || !file.endsWith('.mdx')) continue
     const stem = file.replace('_setup-', '').replace('.mdx', '')
     map[stem] =
       'Setup' +
       stem
         .split('-')
+        .filter((w) => w.length > 0)
         .map((w) => w[0].toUpperCase() + w.slice(1))
         .join('') +
       'Section'
   }
   return map
 }
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
function buildSetupStemMap() {
const templatesDir = path.join(__dirname, '../src/components/templates/agent-connectors')
let files
try {
files = fs.readdirSync(templatesDir)
} catch {
return {}
}
const map = {}
for (const file of files) {
if (!file.startsWith('_setup-') || !file.endsWith('.mdx')) continue
const stem = file.replace('_setup-', '').replace('.mdx', '')
map[stem] =
'Setup' +
stem
.split('-')
.map((w) => w[0].toUpperCase() + w.slice(1))
.join('') +
'Section'
}
return map
}
function buildSetupStemMap() {
const templatesDir = path.join(__dirname, '../src/components/templates/agent-connectors')
let files
try {
files = fs.readdirSync(templatesDir)
} catch {
return {}
}
const map = {}
for (const file of files) {
if (!file.startsWith('_setup-') || !file.endsWith('.mdx')) continue
const stem = file.replace('_setup-', '').replace('.mdx', '')
map[stem] =
'Setup' +
stem
.split('-')
.filter((w) => w.length > 0)
.map((w) => w[0].toUpperCase() + w.slice(1))
.join('') +
'Section'
}
return map
}
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@scripts/sync-agent-connectors.js` around lines 281 - 302, In
buildSetupStemMap, guard against empty string segments when converting stem to
PascalCase: after computing stem (in buildSetupStemMap), filter out empty
segments from stem.split('-') before mapping to ensure w[0] is never accessed on
an empty string (e.g., use .split('-').filter(...)). Update the mapping logic
that builds the value for map[stem] so it only processes non-empty words and
still returns the same 'Setup' + PascalCased + 'Section' format.

Comment on lines +304 to +307
function getSetupComponent(stemMap, providerSlug) {
if (!providerSlug) return null
return stemMap[providerSlug.replace(/_/g, '-')] || stemMap[providerSlug.replace(/_/g, '')] || null
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Slug lookup may miss some provider identifiers.

getSetupComponent tries providerSlug.replace(/_/g, '-') and providerSlug.replace(/_/g, ''), but provider identifiers like google_ads need to match google-ads in the stem map. The first replacement handles this, but consider adding a fallback for the original slug as well.

♻️ Add original slug fallback
 function getSetupComponent(stemMap, providerSlug) {
   if (!providerSlug) return null
-  return stemMap[providerSlug.replace(/_/g, '-')] || stemMap[providerSlug.replace(/_/g, '')] || null
+  return (
+    stemMap[providerSlug.replace(/_/g, '-')] ||
+    stemMap[providerSlug.replace(/_/g, '')] ||
+    stemMap[providerSlug] ||
+    null
+  )
 }
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@scripts/sync-agent-connectors.js` around lines 304 - 307, The
getSetupComponent function currently only checks stemMap for providerSlug with
underscores replaced by hyphens and with underscores removed, which can miss
keys that match the original slug; update getSetupComponent to also try the
original providerSlug as a fallback (i.e., check stemMap[providerSlug] before or
between the existing replacements) so lookups like "google_ads" will match
"google_ads" or "google-ads" in stemMap; modify the lookup order inside
getSetupComponent to attempt providerSlug, providerSlug.replace(/_/g, '-'),
providerSlug.replace(/_/g, '') and return null if none match.

@@ -0,0 +1,36 @@
import { Steps, Aside, Badge } from '@astrojs/starlight/components'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Remove unused import: Aside

The Aside component is imported but never used in this file. Remove it to keep imports minimal.

-import { Steps, Aside, Badge } from '@astrojs/starlight/components'
+import { Steps, Badge } from '@astrojs/starlight/components'
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
import { Steps, Aside, Badge } from '@astrojs/starlight/components'
import { Steps, Badge } from '@astrojs/starlight/components'
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/components/templates/agent-connectors/_setup-asana.mdx` at line 1, Remove
the unused Aside import from the import statement that currently pulls in Steps,
Aside, Badge from '@astrojs/starlight/components'; update the import to only
include the used symbols (Steps and Badge) so the declaration no longer
references Aside and eliminates the unused import.

- Client ID (from above)
- Client Secret (from above)
- Permissions (scopes — see [Asana OAuth scopes reference](https://developers.asana.com/docs/oauth#scopes))
- Click **Save**.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix indentation inconsistency in Step 4

Line 35 uses only 3 spaces of indentation, but the preceding list items (lines 31-34) use 6 spaces. This inconsistency may cause MDX/Steps parsing issues.

-   - Click **Save**.
+      - Click **Save**.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/components/templates/agent-connectors/_setup-asana.mdx` at line 35, Fix
the indentation of the Step 4 list item ("Click **Save**.") so it matches the
preceding list items' indentation level in the MDX template; update the leading
spaces for the "Click **Save**." line to use the same 6-space indentation as the
previous steps (keeping the content unchanged) to ensure consistent MDX/Steps
parsing.

@@ -73,6 +73,7 @@ head:

import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Remove unused Code import.

The Code component is imported but not used in this file.

♻️ Proposed fix
-import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
+import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside } from '@astrojs/starlight/components'

Based on learnings: "In MDX files, import { Code } from 'astrojs/starlight/components' only if the MDX content actually uses the component."

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside } from '@astrojs/starlight/components'
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/hubspot.mdx` at line 74, Remove
the unused Code import from the component import list: locate the import
statement that reads "import { Card, CardGrid, Tabs, TabItem, Badge, Steps,
Aside, Code } from '@astrojs/starlight/components'" and delete "Code" from the
destructured imports so only used components remain (Card, CardGrid, Tabs,
TabItem, Badge, Steps, Aside).

@@ -73,6 +73,7 @@ head:

import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Remove unused Code import.

The Code component is imported but not used in this file.

♻️ Proposed fix
-import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
+import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside } from '@astrojs/starlight/components'

Based on learnings: "In MDX files, import { Code } from 'astrojs/starlight/components' only if the MDX content actually uses the component."

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/jira.mdx` at line 74, Remove the
unused Code import from the import list so only used components are imported;
edit the import statement that currently reads "import { Card, CardGrid, Tabs,
TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'" and
delete the "Code" symbol, leaving the remaining components intact (e.g., Card,
CardGrid, Tabs, TabItem, Badge, Steps, Aside).

Comment on lines 74 to +76
import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
import { Accordion, AccordionItem } from 'accessible-astro-components'
import { SetupLinearSection } from '@components/templates'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Remove unused component imports.

Several imported components appear unused in this file: Card, CardGrid, Tabs, TabItem, Steps, Aside, Code. Only Badge is used in the visible content. Unused imports increase bundle size.

Proposed fix
-import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
-import { Accordion, AccordionItem } from 'accessible-astro-components'
+import { Badge } from '@astrojs/starlight/components'
 import { SetupLinearSection } from '@components/templates'

Based on learnings: "In MDX files, import { Code } from 'astrojs/starlight/components' only if the MDX content actually uses the component."

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
import { Accordion, AccordionItem } from 'accessible-astro-components'
import { SetupLinearSection } from '@components/templates'
import { Badge } from '@astrojs/starlight/components'
import { Accordion, AccordionItem } from 'accessible-astro-components'
import { SetupLinearSection } from '@components/templates'
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/linear.mdx` around lines 74 - 76,
The import line pulls in many unused components—remove the unused symbols Card,
CardGrid, Tabs, TabItem, Steps, Aside, and Code from the import from
'@astrojs/starlight/components' and only keep the actual used symbol(s) (e.g.,
Badge); leave the other imports that are used (e.g., SetupLinearSection,
Accordion, AccordionItem) intact so only referenced components remain in the
file.

@@ -73,6 +73,7 @@ head:

import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Remove unused Code import.

The Code component is imported but never used in this file. Per learnings, only import { Code } if the MDX content actually uses the <Code> component to avoid unnecessary imports and reduce bundle size.

♻️ Proposed fix
-import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
+import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside } from '@astrojs/starlight/components'

Based on learnings: "In MDX files, import { Code } from 'astrojs/starlight/components' only if the MDX content actually uses the component."

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/outlook.mdx` at line 74, Remove
the unused Code component from the component import list: update the import
statement that currently includes "Card, CardGrid, Tabs, TabItem, Badge, Steps,
Aside, Code" to omit "Code" so only used components remain; modify the import in
the top of the MDX file (the import containing Code) and run your linter/build
to confirm no remaining references to Code.

@@ -73,6 +73,7 @@ head:

import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Remove unused Code import.

The Code component is imported but not used in this file.

♻️ Proposed fix
-import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
+import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside } from '@astrojs/starlight/components'

Based on learnings: "In MDX files, import { Code } from 'astrojs/starlight/components' only if the MDX content actually uses the component."

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } from '@astrojs/starlight/components'
import { Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside } from '@astrojs/starlight/components'
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/content/docs/reference/agent-connectors/salesforce.mdx` at line 74,
Remove the unused Code import from the import list that currently reads "import
{ Card, CardGrid, Tabs, TabItem, Badge, Steps, Aside, Code } ..." by deleting
"Code" from the destructured imports (leaving Card, CardGrid, Tabs, TabItem,
Badge, Steps, Aside) and verify there are no remaining references to the Code
symbol in this MDX so the import cleanup is safe.

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.

3 participants