8000 feat: replace google drive tool bundle with Google Drive MCP server by tybalex · Pull Request #708 · obot-platform/tools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: replace google drive tool bundle with Google Drive MCP server #708

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Si 10000 gn up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

feat: replace google drive tool bundle with Google Drive MCP server #708

wants to merge 7 commits into from

Conversation

tybalex
Copy link
Contributor
@tybalex tybalex commented Jun 24, 2025

replace legacy google drive tool bundle with the Google Drive MCP server.

the mcp server:

  • assumes an access_token will be provided by Obot for each tool call.
  • supports streamable-HTTP and stdio
  • unit test coverage.
  • download/upload file not supported yet. Needs workspace support.

removed the legacy tool bundle.

@tybalex tybalex self-assigned this Jun 24, 2025
@tybalex tybalex mentioned this pull request Jun 24, 2025
Open
thedadams
thedadams requested changes Jun 26, 2025
View reviewed changes
Copy link
Contributor
@thedadams thedadams left a comment

Choose a reason for hiding this comment

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

Have we discussed how these are going to be launched? If we are implementing them as streamable HTTP servers, then we would need Docker images for them to work in Obot's Kubernetes deployments. But that wouldn't work for local stdio stuff.

google/drive/apis/workspace_file.py
@@ -1,8 +1,8 @@
import os
import gptscript
# import gptscript
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove if not needed.

google/drive/example_client.py Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this file? Seems like something that could be documentation.

google/drive/server.py
except Exception as error:
raise ToolError(f"Unexpected ToolError: {error}")

## TODO: this tool requires the support of workspace
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove code that is not needed.

@tybalex
Copy link
Contributor Author
tybalex commented Jul 1, 2025 •
edited
Loading

Have we discussed how these are going to be launched? If we are implementing them as streamable HTTP servers, then we would need Docker images for them to work in Obot's Kubernetes deployments. But that wouldn't work for local stdio stuff.

From what I've seen in Monday's standup, the google mcp servers will be launched as remote mcp servers for Obot or Darrent's chat client. I will add a Dockerfile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thedadams thedadams thedadams requested changes

@drpebcak drpebcak Awaiting requested review from drpebcak

@njhale njhale Awaiting requested review from njhale

@StrongMonkey StrongMonkey Awaiting requested review from StrongMonkey

@g-linville g-linville Awaiting requested review from g-linville

Requested changes must be addressed to merge this pull request.

Assignees

@tybalex tybalex

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tybalex @thedadams
0