chore: small cleanups

[tushar-composio]

Important

This pull request cleans up code, updates URLs, and removes unnecessary files to improve maintainability.

• Code Cleanups:
  • Remove unused imports in toolset.py and composio_langchain/toolset.py.
  • Simplify query parameter handling in endpoints.py.
  • Remove deprecated V1BaseModel handling in toolset.py.
  • Remove pydantic.v1.error_wrappers usage in utils/pydantic.py and composio_langchain/toolset.py.
• URL Updates:
  • Update GitHub URLs in setup.py and swe/setup.py.
• File Removals:
  • Delete package.json, playwright.config.ts, session.json, and tools.py from tests directory.

^{This description was created by for d26d62f. It will automatically update as commits are pushed.}

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
composio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 28, 2024 3:16pm

[sweep-ai]

Hey @tushar-composio, here is an example of how you can ask me to improve this pull request:

@Sweep Add unit tests for the new `Endpoint` query parameter handling in `endpoints.py` to verify that URL encoding and query string construction works correctly with:
- Empty query parameters
- Single query parameter
- Multiple query parameters
- Query parameters containing special characters that need encoding

📖 For more information on how to use Sweep, please read our documentation.

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 5b95a52 in 43 seconds

More details

• Looked at 484 lines of code in 11 files
• Skipped 1 files when reviewing.
• Skipped posting 7 drafted comments based on config settings.

1. python/composio/client/__init__.py:1

• Draft comment:
  The removal of the # fmt: off comment is appropriate as it is not needed here. The code is already formatted properly.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The removal of the 'fmt: off' comment is appropriate as it is not needed here. The code is already formatted properly.

2. python/composio/client/__init__.py:64

• Draft comment:
  Adding a trailing comma to the runtime parameter in the constructor is a good practice for future additions.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The trailing comma added to the runtime parameter in the constructor is a good practice for future additions.

3. python/composio/client/endpoints.py:59

• Draft comment:
  The refactoring of the URL query parameter construction is more efficient and cleaner.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The refactoring of the URL query parameter construction is more efficient and cleaner.

4. python/composio/tools/toolset.py:578

• Draft comment:
  The removal of the V1BaseModel check is appropriate since it is not used anymore.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The removal of the V1BaseModel check is appropriate since it is not used anymore.

5. python/composio/utils/pydantic.py:1

• Draft comment:
  The removal of the pydantic.v1.error_wrappers import is correct since it's not used anymore.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The removal of the pydantic.v1.error_wrappers import is correct since it's not used anymore.

6. python/setup.py:99

• Draft comment:
  The update to the GitHub URL is correct and reflects the new repository location.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The update to the GitHub URL is correct and reflects the new repository location.

7. python/swe/setup.py:44

• Draft comment:
  The update to the GitHub URL is correct and reflects the new repository location.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The update to the GitHub URL is correct and reflects the new repository location.

Workflow ID: wflow_St4QzPVxOg9yDlQ1

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[github-actions]

This comment was generated by github-actions[bot]!

JS SDK Coverage Report

📊 Coverage report for JS SDK can be found at the following URL:
https://pub-92e668239ab84bfd80ee07d61e9d2f40.r2.dev/coverage-11557022805/coverage/index.html

📁 Test report folder can be found at the following URL:
https://pub-92e668239ab84bfd80ee07d61e9d2f40.r2.dev/html-report-11557022805/html-report/report.html

[ellipsis-dev]

👍 Looks good to me! Incremental review on d26d62f in 16 seconds

More details

• Looked at 22 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. python/composio/client/__init__.py:261

• Draft comment:
  Unnecessary trailing comma added. Remove it for consistency. This applies to line 276 as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR description mentions removing unused imports, but there are no changes in this file related to imports. However, there are trailing commas added in the execute method, which are unnecessary and not mentioned in the PR description.

Workflow ID: wflow_AHuz4xGncqioKsJd

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.