feat: v0.4.8 upgrade

[himanshu-dixit]

🔍 Review Summary

Release Summary

Purpose:

• Enhance existing features and prepare for potential new integrations.

Changes:

• Enhancement: Updated composio-core package to version 0.4.8 for better performance and stability.
• New Feature: Added ConnectionRequest in index.ts to support future integration possibilities.

Impact:

• Refines core functionalities and sets the stage for future enhancements, improving system stability and expanding potential functionalities.

Original Description

[!IMPORTANT]
Upgrade to version 0.4.8 and add ConnectionRequest export in index.ts.

• Version Upgrade:
  • Update version in package.json from 0.4.7 to 0.4.8.
• Exports:
  • Add ConnectionRequest export in index.ts from sdk/models/connectedAccounts.

^{This description was created by for 01649e2. 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	Dec 23, 2024 7:16am

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 01649e2 in 9 seconds

More details

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

1. js/src/index.ts:8

• Draft comment:
  Ensure that ConnectionRequest is used elsewhere in the codebase to justify its export. If not, consider removing it to keep the exports clean.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The import and export of ConnectionRequest is consistent and seems intentional. No issues here.

Workflow ID: wflow_ky8WpgVCzKDRfNcl

---

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

[entelligence-ai-pr-reviews]

Walkthrough

The pull request includes a version upgrade of the composio-core package from v0.4.7 to v0.4.8, which likely addresses minor improvements or bug fixes. Additionally, a new import statement for ConnectionRequest has been added to the index.ts file, suggesting potential new functionality or integration.

Changes

File(s)	Summary
js/package.json	Updated composio-core package version from 0.4.7 to 0.4.8.
js/src/index.ts	Added import for ConnectionRequest from sdk/models/connectedAccounts.

🔗 Related PRs

• chore: unpin pydantic #967: The pull request updates setup.py to unpin the pydantic version and modifies type annotations for compatibility across multiple files.
• fix: bring eslint + TS issues to zero #1013: The pull request aims to enhance code quality and type safety by enforcing stricter ESLint rules, refactoring code, and improving error handling, leading to a more maintainable and robust codebase.
• feat: Add lint and prettier check in workflow #955: The update introduces a GitHub Actions workflow for linting and formatting JavaScript and TypeScript files using ESLint and Prettier, along with configuration updates, script modifications, and file cleanup.

Instructions

Emoji Descriptions:

• ⚠️ Potential Issue - May require further investigation.
• 🔒 Security Vulnerability - Fix to ensure system safety.
• 💻 Code Improvement - Suggestions to enhance code quality.
• 🔨 Refactor Suggestion - Recommendations for restructuring code.
• ℹ️ Others - General comments and information.

Interact with the Bot:

• Send a message or request using the format:
  @bot + *your message*

Example: @bot Can you suggest improvements for this code?

• Help the Bot learn by providing feedback on its responses.
  @bot + *feedback*

Example: @bot Do not comment on `save_auth` function !

Execute a command using the format:

@bot + */command*

Example: @bot /updateCommit

Available Commands:

• /updateCommit ✨: Apply the suggested changes and commit them (or Click on the Github Action button to apply the changes !)
• /updateGuideline 🛠️: Modify an existing guideline.
• /addGuideline ➕: Introduce a new guideline.

Tips for Using @bot Effectively:

• Specific Queries: For the best results, be specific with your requests.
  🔍 Example: @bot summarize the changes in this PR.
• Focused Discussions: Tag @bot directly on specific code lines or files for detailed feedback.
  📑 Example: @bot review this line of code.
• Managing Reviews: Use review comments for targeted discussions on code snippets, and PR comments for broader queries about the entire PR.
  💬 Example: @bot comment on the entire PR.

Need More Help?

📚 Visit our documentation for detailed guides on using Entelligence.AI.
🌐 Join our community to connect with others, request features, and share feedback.
🔔 Follow us for updates on new features and improvements.

[shreysingla11]

Consider adding a brief comment above the import statement explaining the purpose of ConnectionRequest class and when it should be used. This would help SDK users understand its role in connection management.

[himanshu-dixit]

Dummy class that's returned. Used by user for testing. No other class are present.

[shreysingla11]

Code Review Summary

Changes Overview

• Version bump from 0.4.7 to 0.4.8
• Added ConnectionRequest class export for public SDK use
• Clean and focused changes for connection management functionality

Code Quality: ⭐⭐⭐⭐⭐ (5/5)

• Well-structured TypeScript code
• Proper error handling
• Good type safety
• Clear method signatures
• Follows existing code patterns

Suggestions

• Consider adding documentation comments to explain the purpose and usage of the ConnectionRequest class for SDK users
• A brief changelog entry in the PR description would be helpful

Overall, this is a solid PR that exposes important connection management functionality to SDK users in a clean and maintainable way.

[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-12462884018/coverage/index.html

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

[himanshu-dixit]

Merging as useCaseLimit test is failing in BE and not master.