Skip to content

feat(microsoft-tools): added planner, onedrive, and sharepoint#840

Merged
waleedlatif1 merged 23 commits intostagingfrom
feat/microsoft-tools
Aug 6, 2025
Merged

feat(microsoft-tools): added planner, onedrive, and sharepoint#840
waleedlatif1 merged 23 commits intostagingfrom
feat/microsoft-tools

Conversation

@aadamgough
Copy link
Copy Markdown
Contributor

Description

Added planner, onedrive, and sharepoint with the Microsoft graph API

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please see the video for testing

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally and in CI (bun run test)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have updated version numbers as needed (if needed)
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • My changes do not introduce any new security vulnerabilities
  • I have considered the security implications of my changes
Screen.Recording.2025-08-01.at.10.33.16.AM.mov

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 6, 2025 3:39am
sim ❌ Failed (Inspect) Aug 6, 2025 3:39am

aadamgough pushed a commit that referenced this pull request Aug 1, 2025
@vercel vercel bot temporarily deployed to Preview – docs August 1, 2025 17:42 Inactive
@aadamgough aadamgough marked this pull request as ready for review August 1, 2025 17:45
Copy link
Copy Markdown
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.

Greptile Summary

This PR introduces a comprehensive Microsoft Graph API integration that adds three new productivity tools to the platform: Microsoft Planner for task management, OneDrive for cloud file storage, and SharePoint for site and page management. The implementation extends the existing Microsoft OAuth infrastructure to support these new services with appropriate scopes and authentication flows.

The changes follow established architectural patterns throughout the codebase:

  • Tool Layer: Each service includes complete CRUD operations (create, read, list) implemented as individual tools following the ToolConfig pattern used by existing integrations like Google Drive and GitHub
  • Block Layer: New block configurations provide user-friendly interfaces with operation-based conditional fields, file selectors, and OAuth credential management
  • API Layer: REST endpoints handle authentication, token refresh, and Microsoft Graph API communication using the existing OAuth infrastructure
  • Type Safety: Comprehensive TypeScript interfaces define request/response structures, maintaining consistency with other tool integrations

The Microsoft Graph API integration leverages existing Microsoft OAuth providers (established for Teams and Excel) while adding service-specific scopes: Group and Tasks permissions for Planner, Files permissions for OneDrive, and Sites permissions for SharePoint. The implementation reuses established patterns like dual parameter approaches (UI selectors + programmatic IDs), standardized response transformation, and consistent error handling.

Key architectural decisions include treating these as separate OAuth services under the Microsoft provider umbrella, extending the unified Microsoft file selector component to handle all three services, and maintaining backward compatibility with existing Microsoft integrations. The tools integrate seamlessly with the platform's workflow engine, providing users with expanded automation capabilities across Microsoft's productivity suite.

Confidence score: 2/5

  • This PR contains several critical implementation issues including duplicate field IDs, missing implementations, and type safety concerns that could cause runtime failures
  • Score reflects serious code quality issues including incomplete functionality (empty get_content file), duplicate field definitions that will cause conflicts, and excessive use of any types
  • Pay close attention to apps/sim/blocks/blocks/onedrive.ts for duplicate fileName field IDs, apps/sim/tools/onedrive/get_content.ts which is empty, and type safety issues throughout the OneDrive and Planner implementations

30 files reviewed, 35 comments

Edit Code Review Bot Settings | Greptile

aadamgough pushed a commit that referenced this pull request Aug 1, 2025
@aadamgough aadamgough force-pushed the feat/microsoft-tools branch from a155074 to 7e322bc Compare August 1, 2025 18:10
@vercel vercel bot temporarily deployed to Preview – docs August 1, 2025 18:10 Inactive
@aadamgough aadamgough changed the base branch from main to staging August 1, 2025 19:45
@vercel vercel bot temporarily deployed to Preview – docs August 1, 2025 20:26 Inactive
@vercel vercel bot temporarily deployed to Preview – docs August 1, 2025 20:27 Inactive
aadamgough pushed a commit that referenced this pull request Aug 1, 2025
@vercel vercel bot temporarily deployed to Preview – docs August 1, 2025 21:13 Inactive
aadamgough pushed a commit that referenced this pull request Aug 1, 2025
@vercel vercel bot temporarily deployed to Preview – sim August 1, 2025 23:47 Inactive
aadamgough pushed a commit that referenced this pull request Aug 1, 2025
@vercel vercel bot temporarily deployed to Preview – sim August 1, 2025 23:49 Inactive
@icecrasher321 icecrasher321 self-assigned this Aug 1, 2025
aadamgough pushed a commit that referenced this pull request Aug 2, 2025
@vercel vercel bot temporarily deployed to Preview – docs August 6, 2025 03:30 Inactive
@waleedlatif1 waleedlatif1 merged commit a3a5bf1 into staging Aug 6, 2025
4 of 5 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/microsoft-tools branch August 6, 2025 17:27
arenadeveloper02 pushed a commit to arenadeveloper02/p2-sim that referenced this pull request Sep 19, 2025
…udioai#840)

* first push

* feat: finished onedrive tool

* added refresh

* added sharepoint with create page

* finished sharepoint and onedrive

* planner working

* fixed create task tool

* made read task better

* cleaned up read task

* bun run lint

* cleaned up simstudioai#840

* greptile changes and clean up

* bun run lint

* fix simstudioai#840

* added docs simstudioai#840

* bun run lint simstudioai#840

* removed unnecessary logic simstudioai#840

* removed page token simstudioai#840

* fixed docs and descriptions, added advanced mode simstudioai#840

* remove unused types, cleaned up a lot, fixed docs

* readded file upload and changed docs

* bun run lint

* added folder name

---------

Co-authored-by: Adam Gough <[email protected]>
Co-authored-by: Adam Gough <[email protected]>
Co-authored-by: waleedlatif1 <[email protected]>
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