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(platform/frontend): Add behave as var #8364

Closed
wants to merge 17 commits into from

Conversation

aarushik93
Copy link
Contributor

Background

So the frontend behaves as the cloud not local

Changes 🏗️

Added NEXT_PUBLIC_BEHAVE_AS to both prod and dev as CLOUD

Testing 🔍

Note

Only for the new autogpt platform, currently in autogpt_platform/

  • Create from scratch and execute an agent with at least 3 blocks
  • Import an agent from file upload, and confirm it executes correctly
  • Upload agent to marketplace
  • Import an agent from marketplace and confirm it executes correctly
  • Edit an agent from monitor, and confirm it executes correctly

kcze and others added 17 commits October 14, 2024 17:50
Display existing credentials (OAuth and API keys) for all current providers: Google, Github, Notion and allow user to remove them. For providers that support it, we also revoke the tokens through the API: of the providers we currently have, Google and GitHub support it; Notion doesn't.

- Add credentials list and `Delete` button in `/profile`
- Add `revoke_tokens` abstract method to `BaseOAuthHandler` and implement it in each provider
- Revoke OAuth tokens for providers on `DELETE` `/{provider}/credentials/{cred_id}`, and return whether tokens could be revoked
   - Update `autogpt-server-api/baseClient.ts:deleteCredentials` with `CredentialsDeleteResponse` return type

Bonus:
- Update `autogpt-server-api/baseClient.ts:_request` to properly handle empty server responses
…#8331)

Change [graph]×[node] worker limit from 10×5 to 10×3

---------

Co-authored-by: Reinier van der Leer <[email protected]>
* update ci for dev

* update classic

* remove duplicate dev
- Apply consistent border styling to `Input`, `Select`, and `Textarea`
   - Remove `rounded-xl` from node input elements

- Add `whitespace-nowrap` to `CustomNode` header category tags

---------

Co-authored-by: Zamil Majdy <[email protected]>
* add sealed secrets

* add encrypted secrets

* remove extra space
* fix sealed secret names

* fix names and add annotation
* add health endpoint

* fix linting

---------

Co-authored-by: Nicholas Tindle <[email protected]>
* feat(platform): List and revoke credentials in user profile (#8207)

Display existing credentials (OAuth and API keys) for all current providers: Google, Github, Notion and allow user to remove them. For providers that support it, we also revoke the tokens through the API: of the providers we currently have, Google and GitHub support it; Notion doesn't.

- Add credentials list and `Delete` button in `/profile`
- Add `revoke_tokens` abstract method to `BaseOAuthHandler` and implement it in each provider
- Revoke OAuth tokens for providers on `DELETE` `/{provider}/credentials/{cred_id}`, and return whether tokens could be revoked
   - Update `autogpt-server-api/baseClient.ts:deleteCredentials` with `CredentialsDeleteResponse` return type

Bonus:
- Update `autogpt-server-api/baseClient.ts:_request` to properly handle empty server responses

* fix(backend): Lower the number of node workers to save DB connections (#8331)

Change [graph]×[node] worker limit from 10×5 to 10×3

---------

Co-authored-by: Reinier van der Leer <[email protected]>

* fix(ci,platform): Add dev branch trigger to all ci (#8339)

* update ci for dev

* update classic

* remove duplicate dev

* fix(frontend): Fix styling inconsistencies in input elements (#8337)

- Apply consistent border styling to `Input`, `Select`, and `Textarea`
   - Remove `rounded-xl` from node input elements

- Add `whitespace-nowrap` to `CustomNode` header category tags

---------

Co-authored-by: Zamil Majdy <[email protected]>

* feat(builder): Use configmap for builder (#8343)

use configmap in builder

* fix(platform,infra): Checkin non secret values  (#8344)

checkin non secrets

* security(platform): Add sealed secrets (#8342)

* add sealed secrets

* add encrypted secrets

* remove extra space

* Tf public media buckets (#8324)

* fix(infra): Fix sealed secret names  (#8350)

* fix sealed secret names

* fix names and add annotation

* feat(backend): Introduce executors shared DB connection (#8340)

* update health checkendpoint

---------

Co-authored-by: Krzysztof Czerwinski <[email protected]>
Co-authored-by: Reinier van der Leer <[email protected]>
Co-authored-by: Zamil Majdy <[email protected]>
Co-authored-by: Swifty <[email protected]>
* Initial upload of block docks

* add github + google block docs

* small tweak
* fix(market): agent pagination and search errors

* fix(frontend): search was not paginated

* fix: linting

* feat(market): linting ci

* fix(ci): branch limit name
@aarushik93 aarushik93 requested review from a team as code owners October 17, 2024 11:17
@aarushik93 aarushik93 requested review from kcze and majdyz and removed request for a team October 17, 2024 11:17
@aarushik93 aarushik93 closed this Oct 17, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end size/xl labels Oct 17, 2024
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Potential Performance Issue
The removal of asyncio and the change to synchronous execution may impact performance, especially for I/O-bound operations. This should be carefully evaluated to ensure it doesn't introduce bottlenecks.

Error Handling
The error handling in the execute_node function has been simplified. Ensure that this doesn't lead to loss of important error information or improper error handling.

API Changes
The changes to the AgentServer class, including the addition of the health endpoint and the removal of the send_execution_update method, may affect API consumers. Ensure that these changes are properly documented and communicated to users.

Model Changes
The introduction of the new ListResponse generic model and the removal of AgentListResponse may require updates to code that depends on these models. Verify that all references to the old model have been updated throughout the codebase.

Copy link

netlify bot commented Oct 17, 2024

Deploy Preview for auto-gpt-docs ready!

Name Link
🔨 Latest commit c568b77
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/6710f251c607210008db3504
😎 Deploy Preview https://deploy-preview-8364--auto-gpt-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codecov bot commented Oct 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.62%. Comparing base (ef7cfbb) to head (c568b77).
Report is 110 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8364   +/-   ##
=======================================
  Coverage   49.62%   49.62%           
=======================================
  Files         144      144           
  Lines        8916     8916           
  Branches     1240     1240           
=======================================
  Hits         4425     4425           
  Misses       4344     4344           
  Partials      147      147           
Flag Coverage Δ
Linux 49.43% <ø> (ø)
Windows 50.03% <ø> (ø)
agbenchmark 34.07% <ø> (ø)
autogpt-agent 33.94% <ø> (ø)
forge 58.03% <ø> (ø)
macOS 48.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aarushik93 aarushik93 deleted the aarushikansal/behav-as-cloud branch October 17, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation platform/backend AutoGPT Platform - Back end platform/frontend AutoGPT Platform - Front end Review effort [1-5]: 4 size/xl
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants