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

fix: Composio component #5834

Open
wants to merge 67 commits into
base: main
Choose a base branch
from

Conversation

abhishekpatil4
Copy link
Contributor

  • Updated component to display only OAuth-based authentication method apps.
  • Bumped composio-core and composio-langchain versions from 0.6.13 to 0.6.16.

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Jan 21, 2025
@github-actions github-actions bot added the bug Something isn't working label Jan 21, 2025
@ogabrielluiz ogabrielluiz changed the base branch from fix/composio-component to main January 21, 2025 15:22
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Jan 21, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 21, 2025
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Jan 21, 2025
Copy link
Collaborator

@edwinjosechittilappilly edwinjosechittilappilly left a comment

Choose a reason for hiding this comment

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

lgtm

@edwinjosechittilappilly
Copy link
Collaborator

User should refresh the page each time an app is selected. Consider implementing real-time refresh to minimize manual refreshes. Also, please ensure that the values are persisted.

@ogabrielluiz what do you think?

@ogabrielluiz ogabrielluiz added lgtm This PR has been approved by a maintainer and removed lgtm This PR has been approved by a maintainer labels Jan 21, 2025
@ogabrielluiz
Copy link
Contributor

User should refresh the page each time an app is selected. Consider implementing real-time refresh to minimize manual refreshes. Also, please ensure that the values are persisted.

@ogabrielluiz what do you think?

I agree.

@ogabrielluiz
Copy link
Contributor

Hey @abhishekpatil4

I can't commit to your branch. Could you fix the lint issues, please?

@abhishekpatil4
Copy link
Contributor Author

User should refresh the page each time an app is selected. Consider implementing real-time refresh to minimize manual refreshes. Also, please ensure that the values are persisted.

@ogabrielluiz what do you think?

@edwinjosechittilappilly
Right now, users need to click the refresh button after selecting an app. This helps us check account connections and manage server requests smoothly. While a real-time refresh could be great, it might cause delays that could confuse users expecting instant updates. We think the refresh button gives a clearer indication that something is happening.

We really appreciate your suggestions and will definitely look into ways to improve the experience in future updates!

@abhishekpatil4
Copy link
Contributor Author

Hey @abhishekpatil4

I can't commit to your branch. Could you fix the lint issues, please?

Hi @ogabrielluiz,

I followed the instructions in the DEVELOPMENT.md file and ran make lint. However, I don’t see the "Allow edits by maintainers" option, nor am I able to grant you permission to make changes to my forked repository.

Could you please guide me on how to resolve this? Any help would be greatly appreciated!

@ogabrielluiz
Copy link
Contributor

hey @abhishekpatil4

Just merge the PR with main and it should be good to go.

@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Jan 24, 2025
@abhishekpatil4
Copy link
Contributor Author

@ogabrielluiz I don't think I have access to merge the PR

erichare and others added 20 commits January 24, 2025 23:24
… permissions (langflow-ai#5879)

* fix: Allow direct specification of api_endpoint on oss

* Update astradb.py

* Update Vector Store RAG.json

* Add vector store rag template updates

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
* Added onBuildError to error case on buildVertices

* add onBuildError just if there's no source

* refactor: Update markdown rendering in alert components

* Fixed list on error

* Fixed md exibition of error

* Fixed exibition for component erorrs

* Made errors be generated with the correct map

* Removed markdown from unused places

* Update src/frontend/src/utils/buildUtils.ts

Co-authored-by: Cristhian Zanforlin Lousa <[email protected]>

* Update src/frontend/src/utils/buildUtils.ts

Co-authored-by: Cristhian Zanforlin Lousa <[email protected]>

---------

Co-authored-by: anovazzi1 <[email protected]>
Co-authored-by: Cristhian Zanforlin Lousa <[email protected]>
…nput in FileComponent (langflow-ai#5901)

* fix: Update 'advanced' flag for concurrency multithreading and data input in file components

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
…#5886)

* fix: Expose API Endpoint when no DB list access in Astra

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Update starter template

* [autofix.ci] apply automated fixes

* Update Vector Store RAG.json

* Update astradb.py

* [autofix.ci] apply automated fixes

* Update astradb.py

* [autofix.ci] apply automated fixes

* Update astradb.py

* [autofix.ci] apply automated fixes

* Small update to dropdown for OSS

* [autofix.ci] apply automated fixes

* fix: Don't delete value when refreshing db list

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
…ase locks (langflow-ai#5896)

* refactor: Update _log_transaction_async to be asynchronous and improve task management

- Changed _log_transaction_async method from synchronous to asynchronous to allow proper handling of transaction logging.
- Added error handling for task cancellation and ensured that pending tasks are managed correctly.
- Updated calls to _log_transaction_async in the ComponentVertex class to await the asynchronous method, ensuring proper flow execution and error handling.

* refactor: Simplify _log_transaction_async method by removing redundant error handling

- Streamlined the _log_transaction_async method to enhance readability and maintainability.
- Removed unnecessary try-except blocks for task cancellation, as the async context already handles task management effectively.
- Ensured that completed tasks are awaited and cleared properly, improving overall task management.

* fix: Correctly handle vertex build response in asynchronous flow construction

- Moved the retrieval of the vertex build response to occur after awaiting the build task, ensuring proper handling of task completion.
- Improved error handling by maintaining the cancellation logic while ensuring the response is only accessed after the task is completed.
- This change enhances the reliability of the flow construction process in the chat API.

* fix: Improve task management in Vertex class by refining log transaction handling

- Updated the log transaction handling in the Vertex class to await a single task instead of gathering all tasks, enhancing efficiency.
- Removed the clearing of the task list, ensuring that only the most recent task is processed, which simplifies the task management logic.
- This change aims to improve the reliability and performance of asynchronous logging in the flow execution.

* refactor: Comment out log transaction handling in Vertex class for future review
* feat: Add ScrapeGraph icon component

This commit adds a new icon component called ScrapeGraph. It is a forwardRef component that renders the ScrapeGraphAI icon. The component is added to the list of sidebar bundles and can be used to display the ScrapeGraph icon in the UI.

* refactor: Update ScrapeGraph bundle name

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]>
)

📝 (NodeOutputfield/index.tsx): Add styleClasses property with value "z-40" to improve styling
📝 (outputModal/index.tsx): Add className property with value "z-50" to improve styling
📝 (textModal/index.tsx): Add handleEscapeKeyDown function to handle escape key event and close modal
…5907)

Bumps [undici](https://github.com/nodejs/undici) from 6.21.0 to 6.21.1.
- [Release notes](https://github.com/nodejs/undici/releases)
- [Commits](nodejs/undici@v6.21.0...v6.21.1)

---
updated-dependencies:
- dependency-name: undici
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]>
* docs:add-changelog-to-nav

* docs: add OpenRouter component documentation with detailed inputs and outputs

* docs: add Outputs section to components-models documentation for Cohere and Ollama

* docs: update references from configuration-objects to concepts-objects across multiple components and documentation files

* feat: Add DataFrame operations section to components-processing documentation

* title-case-in-nav

* fix-memories-tab-in-chat-memory

* tool-calling-agent-update

* feat: enhance documentation with icon imports and improved instructions for OpenAI component

* material-icon

* fix: update documentation for tool mode input connection in agent component

* add-loop-component

* add-img-for-loop-summary

* feat: add documentation for using logic components in a flow with examples

* fix: enhance documentation for Loop component with detailed data flow explanation

* redirect-for-config-objects-page

* fix: improve error handling in data processing module

* fix: update documentation for Data objects in Loop component and add import statement in memory chatbot tutorial

* quickstart-screenshots

* docs: update starter flow images

* update-agent-screenshots

* move-repl-agent

* docs: enhance global variables documentation and clarify prerequisites for vector store RAG flow

* docs: update Simple Agent to use URL component

* docs: enhance memory chatbot tutorial with example conversation and clarify session ID terminology

* docs: update visibility icon description in concepts-components.md

* Apply suggestions from code review

Co-authored-by: brian-f <[email protected]>

* correct-playground-sequence-and-typo

---------

Co-authored-by: brian-f <[email protected]>
auto-merge was automatically disabled January 24, 2025 17:55

Head branch was pushed to by a user without write access

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jan 24, 2025
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Jan 24, 2025
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jan 24, 2025
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.