Skip to content

feat(integrations): add amplitude, google pagespeed insights, and pagerduty integrations#3385

Merged
waleedlatif1 merged 6 commits intostagingfrom
waleedlatif1/add-amplitude
Mar 1, 2026
Merged

feat(integrations): add amplitude, google pagespeed insights, and pagerduty integrations#3385
waleedlatif1 merged 6 commits intostagingfrom
waleedlatif1/add-amplitude

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • Add Amplitude integration with 11 tools: send events, identify users, group identify, event segmentation, active users, revenue, realtime active users, user search, user activity, user profile, list events
  • Add Google PageSpeed Insights integration with analyze tool supporting performance, accessibility, SEO, and best practices scoring with Core Web Vitals metrics (both display and numeric values)
  • Register blocks, tools, icons, and docs for both integrations

Type of Change

  • New feature

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 Feb 28, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 1, 2026 2:55am

Request Review

@waleedlatif1 waleedlatif1 force-pushed the waleedlatif1/add-amplitude branch from 7092ac0 to 24702e7 Compare March 1, 2026 00:19
@waleedlatif1 waleedlatif1 changed the title feat(integrations): add amplitude and google pagespeed insights integrations feat(integrations): add amplitud, google pagespeed insights, and pagerduty integrations Mar 1, 2026
@waleedlatif1 waleedlatif1 marked this pull request as ready for review March 1, 2026 00:23
@waleedlatif1 waleedlatif1 changed the title feat(integrations): add amplitud, google pagespeed insights, and pagerduty integrations feat(integrations): add amplitude, google pagespeed insights, and pagerduty integrations Mar 1, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 1, 2026

Greptile Summary

This PR successfully adds three new integrations to Sim: Amplitude (11 tools), Google PageSpeed Insights (1 tool), and PagerDuty (6 tools). All integrations are fully implemented with proper registration, comprehensive documentation, and appropriate icons.

Key additions:

  • Amplitude integration: Provides event tracking, user identification, analytics queries, and revenue metrics using HTTP V2 and Dashboard REST APIs
  • Google PageSpeed integration: Enables webpage performance analysis with Lighthouse metrics including Core Web Vitals
  • PagerDuty integration: Supports incident management, service listing, and on-call schedule queries

Implementation quality:

  • All blocks and tools are properly registered in their respective registries
  • API keys correctly use user-only visibility following security best practices
  • Error handling is implemented across all tools
  • TypeScript types are well-defined with proper interfaces
  • Documentation is comprehensive with detailed parameter descriptions and use cases
  • Icons are properly added to both the main app and documentation

All previous review comments about missing PagerDuty registrations have been addressed. The code follows established patterns and conventions throughout the codebase.

Confidence Score: 5/5

  • This PR is safe to merge with no identified issues
  • All three integrations are fully implemented with proper registration, comprehensive documentation, appropriate security practices (user-only API keys), error handling, and TypeScript typing. Previous review feedback has been addressed. No bugs, security issues, or architectural problems identified.
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/blocks/registry.ts Added imports and registered Amplitude, GooglePagespeed, and PagerDuty blocks successfully
apps/sim/tools/registry.ts Registered all 18 tools (11 Amplitude, 1 Google PageSpeed, 6 PagerDuty) with proper imports
apps/sim/blocks/blocks/amplitude.ts Comprehensive block config with 11 operations, proper auth mode, and detailed field mappings
apps/sim/blocks/blocks/google_pagespeed.ts Clean block config for PageSpeed analysis with proper strategy and category options
apps/sim/blocks/blocks/pagerduty.ts Well-structured block with 6 operations for incident management and on-call schedules
apps/sim/tools/amplitude/send_event.ts Event tracking tool with proper JSON parsing, error handling, and complete parameter support
apps/sim/tools/google_pagespeed/analyze.ts Comprehensive PageSpeed analysis with Core Web Vitals metrics and category scores
apps/sim/tools/pagerduty/create_incident.ts Incident creation tool with proper request structure and error handling
apps/sim/components/icons.tsx Added AmplitudeIcon, PagerDutyIcon, and GooglePagespeedIcon with proper SVG formatting
apps/docs/content/docs/en/tools/amplitude.mdx Comprehensive documentation covering all 11 tools with detailed parameters and use cases

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[New Integrations] --> B[Amplitude]
    A --> C[Google PageSpeed]
    A --> D[PagerDuty]
    
    B --> B1[11 Tools]
    B1 --> B2[Event Tracking]
    B1 --> B3[User Analytics]
    B1 --> B4[Revenue Metrics]
    
    C --> C1[1 Tool]
    C1 --> C2[Performance Analysis]
    C2 --> C3[Core Web Vitals]
    C2 --> C4[Lighthouse Scores]
    
    D --> D1[6 Tools]
    D1 --> D2[Incident Management]
    D1 --> D3[Service Listing]
    D1 --> D4[On-Call Schedules]
    
    B --> E[Block Registry]
    C --> E
    D --> E
    
    B2 --> F[Tool Registry]
    B3 --> F
    B4 --> F
    C2 --> F
    D2 --> F
    D3 --> F
    D4 --> F
    
    E --> G[UI Components]
    F --> G
    
    G --> H[Icons]
    G --> I[Documentation]
Loading

Last reviewed commit: 84f5b11

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.

36 files reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

waleedlatif1 and others added 2 commits February 28, 2026 16:49
Re-add registry entries that were reverted after initial commit.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@waleedlatif1
Copy link
Collaborator Author

@cursor review

@waleedlatif1
Copy link
Collaborator Author

@greptile

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is kicking off a free cloud agent to fix this issue. This run is complimentary, but you can enable autofix for all future PRs in the Cursor dashboard.

@waleedlatif1 waleedlatif1 merged commit ee20e11 into staging Mar 1, 2026
11 checks passed
@waleedlatif1 waleedlatif1 deleted the waleedlatif1/add-amplitude branch March 1, 2026 02:56
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