datarobot-genai 0.2.31__tar.gz → 0.2.39__tar.gz

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: datarobot-genai
-Version: 0.2.31
+Version: 0.2.39
 Summary: Generic helpers for GenAI
 Project-URL: Homepage, https://github.com/datarobot-oss/datarobot-genai
 Author: DataRobot, Inc.

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "datarobot-genai"
-version = "0.2.31"
+version = "0.2.39"
 description = "Generic helpers for GenAI"
 readme = "README.md"
 requires-python = ">=3.10, <3.13"

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/core/cli/agent_kernel.py

@@ -107,7 +107,10 @@ class AgentKernel:
         stream: bool = False,
         config: Any | None = None,
     ) -> ChatCompletion | Stream[ChatCompletionChunk]:
-        chat_api_url = config.agent_endpoint if config else self.base_url
+        if config is not None:
+            chat_api_url = f"http://localhost:{config.local_dev_port}"
+        else:
+            chat_api_url = self.base_url
         print(chat_api_url)
 
         return self._do_chat_completion(chat_api_url, user_prompt, completion_json, stream=stream)

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/core/exceptions.py

@@ -19,7 +19,3 @@ class DynamicToolRegistrationError(Exception):
 
 class DynamicPromptRegistrationError(Exception):
     """Exception raised for errors in the dynamic prompt registration process."""
-
-
-class MCPError(Exception):
-    """Base class for MCP errors."""

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/core/logging.py

@@ -20,7 +20,7 @@ from collections.abc import Callable
 from typing import Any
 from typing import TypeVar
 
-from .exceptions import MCPError
+from fastmcp.exceptions import ToolError
 
 # Secret patterns to redact from logs
 SECRET_PATTERNS = [
@@ -93,6 +93,6 @@ def log_execution(func: F) -> F:
             return result
         except Exception as e:
             error_msg = _log_error(logger, func.__name__, e, args=args, kwargs=kwargs)
-            raise MCPError(error_msg)
+            raise ToolError(error_msg)
 
     return wrapper  # type: ignore[return-value]

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/test_utils/clients/base.py

@@ -27,7 +27,7 @@ from mcp.types import ListToolsResult
 from mcp.types import TextContent
 from openai.types.chat.chat_completion import ChatCompletion
 
-from ..utils import save_response_to_file
+from datarobot_genai.drmcp.test_utils.utils import save_response_to_file
 
 
 class ToolCall:

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/test_utils/mcp_utils_ete.py

@@ -68,6 +68,26 @@ def get_openai_llm_client_config() -> dict[str, str]:
     return config
 
 
+def get_dr_llm_gateway_client_config() -> dict[str, str]:
+    """Get DataRobot LLM Gateway client configuration."""
+    datarobot_api_token = os.environ.get("DATAROBOT_API_TOKEN")
+    datarobot_endpoint = os.environ.get("DATAROBOT_ENDPOINT")
+    save_llm_responses = os.environ.get("SAVE_LLM_RESPONSES", "false").lower() == "true"
+
+    if not datarobot_api_token:
+        raise ValueError("Missing required environment variable: DATAROBOT_API_TOKEN")
+
+    config: dict[str, str] = {
+        "datarobot_api_token": datarobot_api_token,
+        "save_llm_responses": str(save_llm_responses),
+    }
+
+    if datarobot_endpoint:
+        config["datarobot_endpoint"] = datarobot_endpoint
+
+    return config
+
+
 def get_headers() -> dict[str, str]:
     # When the MCP server is deployed in DataRobot, we have to include the API token in headers for
     # authentication.

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/tools/clients/gdrive.py

@@ -20,6 +20,7 @@ import logging
 import uuid
 from typing import Annotated
 from typing import Any
+from typing import Literal
 
 import httpx
 from datarobot.auth.datarobot.exceptions import OAuthServiceClientErr
@@ -821,6 +822,96 @@ class GoogleDriveClient:
             headers={"Content-Type": f"multipart/related; boundary={boundary}"},
         )
 
+    async def manage_access(
+        self,
+        *,
+        file_id: str,
+        action: Literal["add", "update", "remove"],
+        role: Literal["reader", "commenter", "writer", "fileOrganizer", "organizer", "owner"]
+        | None = None,
+        email_address: str | None = None,
+        permission_id: str | None = None,
+        transfer_ownership: bool = False,
+    ) -> str:
+        """Manage access permissions for a Google Drive file or folder.
+
+        Adds, updates, or removes sharing permissions on an existing Google Drive
+        file or folder using the Google Drive Permissions API.
+
+        This method supports granting access to users or groups, changing access
+        roles, and revoking permissions. Ownership transfer is supported for files
+        in "My Drive" when explicitly requested.
+
+        Args:
+            file_id: The ID of the Google Drive file or folder whose permissions
+                are being managed.
+            action: The permission operation to perform.
+            role: The access role to assign or update. Valid values include
+                Required for "add" and "update" actions.
+            email_address: The email address of the user or group to grant access to.
+                Required for the "add" action.
+            permission_id: The ID of the permission to update or remove.
+                Required for "update" and "remove" actions.
+            transfer_ownership: Whether to transfer ownership of the file.
+                Only applicable when action="update" and role="owner".
+
+        Returns
+        -------
+            Permission id.
+            For "add" its newly added permission.
+            For "update"/"remove" its previous permission.
+
+        Raises
+        ------
+            GoogleDriveError: If the permission operation fails (invalid arguments,
+                insufficient permissions, resource not found, ownership transfer
+                not allowed, rate limited, etc.).
+        """
+        if not file_id.strip():
+            raise GoogleDriveError("Argument validation error: 'file_id' cannot be empty.")
+
+        if action == "add" and not email_address:
+            raise GoogleDriveError("'email_address' is required for action 'add'.")
+
+        if action in ("update", "remove") and not permission_id:
+            raise GoogleDriveError("'permission_id' is required for action 'update' or 'remove'.")
+
+        if action != "remove" and not role:
+            raise GoogleDriveError("'role' is required for action 'add' or 'update'.")
+
+        if action == "add":
+            response = await self._client.post(
+                url=f"/{file_id}/permissions",
+                json={
+                    "type": "user",
+                    "role": role,
+                    "emailAddress": email_address,
+                },
+                params={"sendNotificationEmail": False, "supportsAllDrives": True},
+            )
+
+        elif action == "update":
+            response = await self._client.patch(
+                url=f"/{file_id}/permissions/{permission_id}",
+                json={"role": role},
+                params={"transferOwnership": transfer_ownership, "supportsAllDrives": True},
+            )
+
+        elif action == "remove":
+            response = await self._client.delete(url=f"/{file_id}/permissions/{permission_id}")
+
+        else:
+            raise GoogleDriveError(f"Invalid action '{action}'")
+
+        if response.status_code not in (200, 201, 204):
+            raise GoogleDriveError(f"Drive API error {response.status_code}: {response.text}")
+
+        if action == "add":
+            return response.json()["id"]
+
+        # Cannot be null here because of above validators
+        return permission_id  # type: ignore
+
     async def __aenter__(self) -> "GoogleDriveClient":
         """Async context manager entry."""
         return self

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/tools/clients/microsoft_graph.py

@@ -16,6 +16,8 @@
 
 import logging
 from typing import Any
+from typing import Literal
+from urllib.parse import quote
 
 import httpx
 from datarobot.auth.datarobot.exceptions import OAuthServiceClientErr
@@ -415,6 +417,175 @@ class MicrosoftGraphClient:
 
         return base_resource
 
+    async def get_personal_drive_id(self) -> str:
+        """Get the current user's personal OneDrive drive ID.
+
+        Returns
+        -------
+            The drive ID string for the user's personal OneDrive.
+
+        Raises
+        ------
+            MicrosoftGraphError: If the drive cannot be retrieved.
+        """
+        try:
+            response = await self._client.get(f"{GRAPH_API_BASE}/me/drive")
+            response.raise_for_status()
+            data = response.json()
+            return data["id"]
+        except httpx.HTTPStatusError as e:
+            status_code = e.response.status_code
+            if status_code == 401:
+                raise MicrosoftGraphError(
+                    "Authentication failed. Access token may be expired or invalid."
+                ) from e
+            if status_code == 403:
+                raise MicrosoftGraphError(
+                    "Permission denied: cannot access personal OneDrive. "
+                    "Requires Files.Read or Files.ReadWrite permission."
+                ) from e
+            raise MicrosoftGraphError(f"Failed to get personal OneDrive: HTTP {status_code}") from e
+
+    async def create_file(
+        self,
+        drive_id: str,
+        file_name: str,
+        content: str,
+        parent_folder_id: str = "root",
+        conflict_behavior: str = "rename",
+    ) -> MicrosoftGraphItem:
+        """Create a text file in a drive (SharePoint document library or OneDrive).
+
+        Uses Microsoft Graph's simple upload endpoint for files < 4MB.
+        Files are created as text/plain content.
+
+        Args:
+            drive_id: The ID of the drive (document library) where the file will be created.
+            file_name: The name of the file to create (e.g., 'report.txt').
+            content: The text content to store in the file.
+            parent_folder_id: ID of the parent folder. Defaults to "root" (drive root folder).
+            conflict_behavior: How to handle name conflicts. Options:
+                - "rename" (default): Auto-renames to 'filename (1).txt', etc.
+                - "fail": Returns 409 Conflict error
+                - "replace": Overwrites existing file
+
+        Returns
+        -------
+            MicrosoftGraphItem representing the created file.
+
+        Raises
+        ------
+            MicrosoftGraphError: If file creation fails.
+        """
+        if not drive_id or not drive_id.strip():
+            raise MicrosoftGraphError("drive_id cannot be empty")
+        if not file_name or not file_name.strip():
+            raise MicrosoftGraphError("file_name cannot be empty")
+
+        # URL encode the filename for path-based addressing
+        encoded_name = quote(file_name, safe="")
+
+        # Simple upload endpoint for files < 4MB
+        # Reference: https://learn.microsoft.com/en-us/graph/api/driveitem-put-content
+        upload_url = (
+            f"{GRAPH_API_BASE}/drives/{drive_id}/items/{parent_folder_id}:/{encoded_name}:/content"
+        )
+
+        try:
+            response = await self._client.put(
+                upload_url,
+                content=content.encode("utf-8"),
+                headers={"Content-Type": "text/plain"},
+                params={"@microsoft.graph.conflictBehavior": conflict_behavior},
+            )
+            response.raise_for_status()
+        except httpx.HTTPStatusError as e:
+            raise self._handle_create_file_error(e, drive_id, file_name, parent_folder_id) from e
+
+        return MicrosoftGraphItem.from_api_response(response.json())
+
+    def _handle_create_file_error(
+        self,
+        error: httpx.HTTPStatusError,
+        drive_id: str,
+        file_name: str,
+        parent_folder_id: str,
+    ) -> MicrosoftGraphError:
+        """Handle HTTP errors for file creation and return appropriate MicrosoftGraphError."""
+        status_code = error.response.status_code
+        error_msg = f"Failed to create file: HTTP {status_code}"
+
+        if status_code == 400:
+            try:
+                error_data = error.response.json()
+                api_message = error_data.get("error", {}).get("message", "Invalid request")
+                error_msg = f"Bad request creating file: {api_message}"
+            except Exception:
+                error_msg = "Bad request: invalid parameters for file creation."
+        elif status_code == 401:
+            error_msg = "Authentication failed. Access token may be expired or invalid."
+        elif status_code == 403:
+            error_msg = (
+                f"Permission denied: you don't have permission to create files in drive "
+                f"'{drive_id}'. Requires Files.ReadWrite.All permission."
+            )
+        elif status_code == 404:
+            error_msg = (
+                f"Parent folder '{parent_folder_id}' not found in drive '{drive_id}'."
+                if parent_folder_id != "root"
+                else f"Drive '{drive_id}' not found."
+            )
+        elif status_code == 409:
+            error_msg = f"File '{file_name}' already exists and conflict behavior is set to 'fail'."
+        elif status_code == 429:
+            error_msg = "Rate limit exceeded. Please try again later."
+
+        return MicrosoftGraphError(error_msg)
+
+    async def share_item(
+        self,
+        file_id: str,
+        document_library_id: str,
+        recipient_emails: list[str],
+        role: Literal["read", "write"],
+        send_invitation: bool,
+    ) -> None:
+        """
+        Share sharepoint / ondrive item using Microsoft Graph API.
+        Under the hood all resources in sharepoint/onedrive
+            in MS Graph API are treated as 'driveItem'.
+
+        Args:
+            file_id: The ID of the file or folder to share.
+            document_library_id: The ID of the document library containing the item.
+            recipient_emails: A list of email addresses to invite.
+            role: The role to assign.
+            send_invitation: Flag determining if recipients should be notified
+
+        Returns
+        -------
+            None
+
+        Raises
+        ------
+            MicrosoftGraphError: If sharing fails
+        """
+        graph_url = f"{GRAPH_API_BASE}/drives/{document_library_id}/items/{file_id}/invite"
+
+        payload = {
+            "recipients": [{"email": email} for email in recipient_emails],
+            "requireSignIn": True,
+            "sendInvitation": send_invitation,
+            "roles": [role],
+        }
+
+        response = await self._client.post(url=graph_url, json=payload)
+
+        if response.status_code not in (200, 201):
+            raise MicrosoftGraphError(
+                f"Microsoft Graph API error {response.status_code}: {response.text}"
+            )
+
     async def __aenter__(self) -> "MicrosoftGraphClient":
         """Async context manager entry."""
         return self

{datarobot_genai-0.2.31 → datarobot_genai-0.2.39}/src/datarobot_genai/drmcp/tools/confluence/tools.py

@@ -55,26 +55,16 @@ async def confluence_get_page(
     if isinstance(access_token, ToolError):
         raise access_token
 
-    try:
-        async with ConfluenceClient(access_token) as client:
-            if page_id_or_title.isdigit():
-                page_response = await client.get_page_by_id(page_id_or_title)
-            else:
-                if not space_key:
-                    raise ToolError(
-                        "Argument validation error: "
-                        "'space_key' is required when identifying a page by title."
-                    )
-                page_response = await client.get_page_by_title(page_id_or_title, space_key)
-    except ConfluenceError as e:
-        logger.error(f"Confluence error getting page: {e}")
-        raise ToolError(str(e))
-    except Exception as e:
-        logger.error(f"Unexpected error getting Confluence page: {e}")
-        raise ToolError(
-            f"An unexpected error occurred while getting Confluence page "
-            f"'{page_id_or_title}': {str(e)}"
-        )
+    async with ConfluenceClient(access_token) as client:
+        if page_id_or_title.isdigit():
+            page_response = await client.get_page_by_id(page_id_or_title)
+        else:
+            if not space_key:
+                raise ToolError(
+                    "Argument validation error: "
+                    "'space_key' is required when identifying a page by title."
+                )
+            page_response = await client.get_page_by_title(page_id_or_title, space_key)
 
     return ToolResult(
         content=f"Successfully retrieved page '{page_response.title}'.",
@@ -116,22 +106,12 @@ async def confluence_create_page(
     if isinstance(access_token, ToolError):
         raise access_token
 
-    try:
-        async with ConfluenceClient(access_token) as client:
-            page_response = await client.create_page(
-                space_key=space_key,
-                title=title,
-                body_content=body_content,
-                parent_id=parent_id,
-            )
-    except ConfluenceError as e:
-        logger.error(f"Confluence error creating page: {e}")
-        raise ToolError(str(e))
-    except Exception as e:
-        logger.error(f"Unexpected error creating Confluence page: {e}")
-        raise ToolError(
-            f"An unexpected error occurred while creating Confluence page "
-            f"'{title}' in space '{space_key}': {str(e)}"
+    async with ConfluenceClient(access_token) as client:
+        page_response = await client.create_page(
+            space_key=space_key,
+            title=title,
+            body_content=body_content,
+            parent_id=parent_id,
         )
 
     return ToolResult(
@@ -164,19 +144,10 @@ async def confluence_add_comment(
     if isinstance(access_token, ToolError):
         raise access_token
 
-    try:
-        async with ConfluenceClient(access_token) as client:
-            comment_response = await client.add_comment(
-                page_id=page_id,
-                comment_body=comment_body,
-            )
-    except ConfluenceError as e:
-        logger.error(f"Confluence error adding comment: {e}")
-        raise ToolError(str(e))
-    except Exception as e:
-        logger.error(f"Unexpected error adding comment to Confluence page: {e}")
-        raise ToolError(
-            f"An unexpected error occurred while adding comment to page '{page_id}': {str(e)}"
+    async with ConfluenceClient(access_token) as client:
+        comment_response = await client.add_comment(
+            page_id=page_id,
+            comment_body=comment_body,
         )
 
     return ToolResult(
@@ -220,32 +191,24 @@ async def confluence_search(
     if isinstance(access_token, ToolError):
         raise access_token
 
-    try:
-        async with ConfluenceClient(access_token) as client:
-            results = await client.search_confluence_content(
-                cql_query=cql_query, max_results=max_results
-            )
-
-            # If include_body is True, fetch full content for each page
-            if include_body and results:
-                data = []
-                for result in results:
-                    flat = result.as_flat_dict()
-                    try:
-                        page = await client.get_page_by_id(result.id)
-                        flat["body"] = page.body
-                    except ConfluenceError:
-                        flat["body"] = None  # Keep excerpt if page fetch fails
-                    data.append(flat)
-            else:
-                data = [result.as_flat_dict() for result in results]
-
-    except ConfluenceError as e:
-        logger.error(f"Confluence error searching content: {e}")
-        raise ToolError(str(e))
-    except Exception as e:
-        logger.error(f"Unexpected error searching Confluence content: {e}")
-        raise ToolError(f"An unexpected error occurred while searching Confluence: {str(e)}")
+    async with ConfluenceClient(access_token) as client:
+        results = await client.search_confluence_content(
+            cql_query=cql_query, max_results=max_results
+        )
+
+        # If include_body is True, fetch full content for each page
+        if include_body and results:
+            data = []
+            for result in results:
+                flat = result.as_flat_dict()
+                try:
+                    page = await client.get_page_by_id(result.id)
+                    flat["body"] = page.body
+                except ConfluenceError:
+                    flat["body"] = None  # Keep excerpt if page fetch fails
+                data.append(flat)
+        else:
+            data = [result.as_flat_dict() for result in results]
 
     n = len(results)
     return ToolResult(
@@ -296,20 +259,11 @@ async def confluence_update_page(
     if isinstance(access_token, ToolError):
         raise access_token
 
-    try:
-        async with ConfluenceClient(access_token) as client:
-            page_response = await client.update_page(
-                page_id=page_id,
-                new_body_content=new_body_content,
-                version_number=version_number,
-            )
-    except ConfluenceError as e:
-        logger.error(f"Confluence error updating page: {e}")
-        raise ToolError(str(e))
-    except Exception as e:
-        logger.error(f"Unexpected error updating Confluence page: {e}")
-        raise ToolError(
-            f"An unexpected error occurred while updating Confluence page '{page_id}': {str(e)}"
+    async with ConfluenceClient(access_token) as client:
+        page_response = await client.update_page(
+            page_id=page_id,
+            new_body_content=new_body_content,
+            version_number=version_number,
         )
 
     return ToolResult(