Skip to content
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.

By clicking “Sign 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

Update release #1191

Merged
merged 2 commits into from
Jan 14, 2025
Merged

Update release #1191

merged 2 commits into from
Jan 14, 2025

Conversation

kaavee315
Copy link
Contributor

@kaavee315 kaavee315 commented Jan 14, 2025

Important

Update version to 0.6.14 across core, plugins, and Dockerfile; update SWE Kit to 0.3.15.

  • Version Updates:
    • Update version to 0.6.14 in __version__.py, setup.py, and multiple plugin setup.py files.
    • Update swekit version to 0.3.15 in swe/setup.py.
    • Update Dockerfile to install composio-core[all]==0.6.14.

This description was created by Ellipsis for 287b33a. It will automatically update as commits are pushed.

Copy link

vercel bot commented Jan 14, 2025

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 Jan 14, 2025 7:18pm

@@ -23,7 +23,7 @@
],
python_requires=">=3.9,<4",
install_requires=[
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.4"

@@ -23,7 +23,7 @@
],
python_requires=">=3.9,<4",
install_requires=[
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.4"

@@ -22,6 +22,6 @@
"Operating System :: OS Independent",
],
python_requires=">=3.9,<4",
install_requires=["composio_core>=0.6.11,<0.7.0", "griptape>=0.24.2"],
install_requires=["composio_core>=0.6.0,<0.7.0", "griptape>=0.24.2"],

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
install_requires=["composio_core>=0.6.0,<0.7.0", "griptape>=0.24.2"],
install_requires=[

@@ -27,7 +27,7 @@
"langchain-openai>=0.0.2.post1",
"pydantic>=2.6.4",
"langchainhub>=0.1.15",
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.4"

@@ -24,7 +24,7 @@
python_requires=">=3.9,<4",
install_requires=[
"llama_index>=0.10.43",
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.4"

@@ -25,7 +25,7 @@
install_requires=[
"lyzr-automata>=0.1.3",
"pydantic>=2.6.4",
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.0"

@@ -22,6 +22,6 @@
"Operating System :: OS Independent",
],
python_requires=">=3.9,<4",
install_requires=["composio_core>=0.6.11,<0.7.0", "openai"],
install_requires=["composio_core>=0.6.0,<0.7.0", "openai"],

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
install_requires=["composio_core>=0.6.0,<0.7.0", "openai"],
install_requires=["composio_core==0.6.0", "openai"]

@@ -23,7 +23,7 @@
],
python_requires=">=3.9,<4",
install_requires=[
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.4"

@@ -22,6 +22,6 @@
"Operating System :: OS Independent",
],
python_requires=">=3.9",
install_requires=["composio_core>=0.6.11,<0.7.0", "PraisonAI>=0.0.2"],
install_requires=["composio_core>=0.6.0,<0.7.0", "PraisonAI>=0.0.2"],

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
install_requires=["composio_core>=0.6.0,<0.7.0", "PraisonAI>=0.0.2"],
install_requires=[

@@ -66,7 +66,7 @@ def scan_for_package_data(
"swebench==2.1.0",
"datasets>=2.20.0",
"gitpython>=3.1.43",
"composio_core>=0.6.11,<0.7.0",
"composio_core>=0.6.0,<0.7.0",

Choose a reason for hiding this comment

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

The version bump for composio_core is too broad. It should be pinned to a specific minor version to avoid unexpected breaking changes from future 0.6.x releases.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"composio_core>=0.6.0,<0.7.0",
"composio_core==0.6.4"

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to cc4f2b0 in 1 minute and 16 seconds

More details
  • Looked at 324 lines of code in 18 files
  • Skipped 0 files when reviewing.
  • Skipped posting 8 drafted comments based on config settings.
1. python/plugins/autogen/setup.py:26
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Comment did not seem useful.
2. python/plugins/camel/setup.py:26
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.
3. python/plugins/google/setup.py:26
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.
4. python/plugins/langchain/setup.py:30
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.
5. python/plugins/llamaindex/setup.py:27
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.
6. python/plugins/lyzr/setup.py:28
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.
7. python/plugins/phidata/setup.py:26
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.
8. python/swe/setup.py:69
  • Draft comment:
    The version constraint for composio_core was changed from >=0.6.11 to >=0.6.0. This is a downgrade in the minimum version requirement and could introduce compatibility issues. Please ensure this change is intentional.
  • Reason this comment was not posted:
    Marked as duplicate.

Workflow ID: wflow_yLp3s7DVcPcnBA3n


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

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Incremental review on 287b33a in 12 seconds

More details
  • Looked at 140 lines of code in 11 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. python/plugins/autogen/setup.py:26
  • Draft comment:
    Ensure that the updated version constraint for composio_core (>=0.6.11,<0.7.0) is compatible with all other dependencies in this setup file. This comment applies to all other setup files in this PR where the same change is made.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The PR updates the version constraint for the composio_core package across multiple setup files. This is a straightforward change, but it's important to ensure consistency and correctness across all files.

Workflow ID: wflow_3gHnzUp5qV45IECi


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

@kaavee315 kaavee315 merged commit 29d8861 into master Jan 14, 2025
20 of 24 checks passed
@shreysingla11
Copy link
Collaborator

Code Review Summary

Changes Overview

  • Core package version bump from 0.6.13 to 0.6.14
  • All plugin packages version bump to match core version
  • Relaxed composio_core dependency constraint from >=0.6.11,<0.7.0 to >=0.6.0,<0.7.0
  • SWE kit version bump from 0.3.14 to 0.3.15

Analysis

  1. Version Management:

    • Consistent version updates across all packages ✅
    • Proper version synchronization between core and plugins ✅
  2. Dependency Management:

    • Relaxed version constraint appears safe given the stable plugin interface
    • No breaking changes identified in core functionality between 0.6.0 and 0.6.11
  3. Build System:

    • Dockerfile updated correctly with new version ✅
    • All setup.py files properly updated ✅

Recommendations

  1. Consider adding a CHANGELOG.md entry to document:

    • The version updates
    • The rationale for relaxing the version constraint
    • Any potential impacts on existing installations
  2. Consider automating version updates using tools like bump2version to ensure consistency

Overall Assessment

LGTM (Looks Good To Me)

  • Clean and consistent version updates
  • No functional changes or risks identified
  • Safe to merge

@kaavee315 kaavee315 deleted the kaavee/release branch January 14, 2025 19:18
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