Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • fix salesforce docs & update styling

Type of Change

  • Bug fix
  • Documentation

Testing

Tested manually

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
Copy link

vercel bot commented Dec 8, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Dec 8, 2025 8:05pm

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 8, 2025

Greptile Overview

Greptile Summary

Refactored Salesforce tools from monolithic files into modular, single-purpose tools and updated documentation to match. Also updated styling for Fumadocs v16 compatibility and fixed sidebar active state detection for internationalized routes.

  • Salesforce tools refactoring: Broke down large files (contacts.ts, leads.ts, opportunities.ts, cases.ts, tasks.ts, reports.ts, dashboards.ts) into individual operation files (e.g., create_contact.ts, update_contact.ts, delete_contact.ts), improving maintainability and following single responsibility principle
  • Documentation updates: Expanded salesforce.mdx with comprehensive tool descriptions, parameters, and examples for all 30+ Salesforce operations; added intro section to cursor.mdx and fixed parameter name from promptText to followupPromptText
  • Styling improvements: Updated global.css for Fumadocs v16 compatibility by targeting new data attributes, improved responsive design with mobile-specific margins, and enhanced sidebar styling with higher CSS specificity
  • Sidebar fixes: Added language prefix stripping logic to correctly detect active sidebar items in internationalized routes (e.g., /en/tools/salesforce now properly matches /tools/salesforce)

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Well-executed refactoring that improves code organization without changing functionality. Documentation accurately reflects the code changes, CSS updates properly handle framework migration, and sidebar logic correctly handles i18n routing. No security issues, breaking changes, or logical errors detected.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
apps/docs/content/docs/en/tools/salesforce.mdx 5/5 Expanded documentation with comprehensive tool descriptions and examples
apps/docs/content/docs/en/tools/cursor.mdx 4/5 Added detailed intro section and fixed parameter name in docs
apps/docs/app/global.css 5/5 Updated CSS for Fumadocs v16 compatibility and improved responsive design
apps/docs/components/docs-layout/sidebar-components.tsx 5/5 Added language prefix handling for proper active state detection
apps/sim/tools/salesforce/index.ts 5/5 Refactored to export individual tool modules instead of monolithic file

Sequence Diagram

sequenceDiagram
    participant User
    participant Docs
    participant SalesforceIndex
    participant ModularTools
    participant SalesforceAPI

    User->>Docs: View Salesforce documentation
    Docs->>Docs: Render updated MDX with tool descriptions
    
    User->>SalesforceIndex: Import Salesforce tool
    SalesforceIndex->>ModularTools: Export specific tool (e.g., create_contact)
    ModularTools->>SalesforceAPI: Execute API request with typed params
    SalesforceAPI-->>ModularTools: Return typed response
    ModularTools-->>User: Return success/error with metadata
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

50 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@waleedlatif1 waleedlatif1 merged commit feb5918 into staging Dec 8, 2025
5 checks passed
@waleedlatif1 waleedlatif1 deleted the fix/docss branch December 8, 2025 20:08
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.

2 participants