Skip to content

Implement bot sdk#2

Merged
pksunkara merged 1 commit intomasterfrom
pavan/pro-230-implement-python-sdk
May 4, 2025
Merged

Implement bot sdk#2
pksunkara merged 1 commit intomasterfrom
pavan/pro-230-implement-python-sdk

Conversation

@pksunkara
Copy link
Member

No description provided.

@linear
Copy link

linear bot commented May 1, 2025

@pksunkara pksunkara requested a review from Copilot May 1, 2025 14:51

This comment was marked as outdated.

@pksunkara pksunkara requested a review from Copilot May 1, 2025 15:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a new bot SDK by updating the webhook signature verification to accept payloads as strings, updating tests accordingly, and adding new code resources and client implementations.

  • Update verify_webhook and generate_webhook_signature to use string payloads instead of JSON objects
  • Add code resource endpoints for download, propose, and cleanup operations
  • Update project metadata and add supporting types and client updates

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/bot/tests/test_webhook.py Updated tests to pass payload as a string
packages/bot/src/automa/bot/webhook.py Modified function signatures to accept payload as strings
packages/bot/src/automa/bot/resources/shared/task.py Added new Task and TaskWithToken definitions
packages/bot/src/automa/bot/resources/code.py New implementations for synchronous and asynchronous code resource methods
packages/bot/src/automa/bot/resources/init.py Updated module exports
packages/bot/src/automa/bot/_types.py Added new type definitions
packages/bot/src/automa/bot/_resource.py New resource base classes
packages/bot/src/automa/bot/_client.py New client implementations
packages/bot/src/automa/bot/_base_client.py Base client implementations for sync/async HTTP operations
packages/bot/src/automa/bot/init.py Updated module attributes for exported symbols
packages/bot/pyproject.toml Updated project name and added dependency constraints
Comments suppressed due to low confidence (1)

packages/bot/pyproject.toml:2

  • Ensure that the updated project name 'automa-bot' is consistent with the repository and package naming conventions throughout the project.
name    = "automa-bot"

@pksunkara pksunkara marked this pull request as draft May 2, 2025 02:49
@pksunkara pksunkara force-pushed the pavan/pro-230-implement-python-sdk branch 4 times, most recently from 8aa2854 to 83acc7d Compare May 4, 2025 12:37
@pksunkara pksunkara marked this pull request as ready for review May 4, 2025 12:39
@pksunkara pksunkara force-pushed the pavan/pro-230-implement-python-sdk branch from 83acc7d to 19c35f9 Compare May 4, 2025 12:41
@pksunkara pksunkara enabled auto-merge May 4, 2025 12:41
@pksunkara pksunkara merged commit b089d84 into master May 4, 2025
3 of 4 checks passed
@pksunkara pksunkara deleted the pavan/pro-230-implement-python-sdk branch May 4, 2025 12:41
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