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

[IDEA] Support plugin allowedlist in Array format not just a comma-delineated list of names in a string #88

Open
zudsniper opened this issue Jun 30, 2023 · 1 comment
Assignees

Comments

@zudsniper
Copy link

zudsniper commented Jun 30, 2023

Summary

The plugin named AutoGPTApiTools causes an error to be thrown, as shown in the following image

kurtosis startup error

The kurtosis call is

kurtosis run github.com/kurtosis-tech/autogpt-package@$AUTOGPT_VERSION --enclave autogpt "$(cat $ENV_JSON_FILE)"

Notably, I am using AUTOGPT_VERSION="0.3.1". Could this be the issue? If it is, I believe supporting plugins made for current 0.4.x versions of Auto-GPT wouldn't actually break anything when used with 0.3.1, unless those plugins use something specifically for 0.4.x.

Let me know if anyone has any ideas on this one, as I'm confused. Thanks guys


Epilogue

Before posting this I realized what the problem is. Essentially, I am storing my ALLOWLISTED_PLUGINS like this in the json file:

 "ALLOWLISTED_PLUGINS":
    [
        "AutoGPTApiTools",
        "AutoGPTEmailPlugin",
        "PlannerPlugin",
        "autogpt-random-values",
        "AutoGPTTelegram",
        "wikipedia-search",
        "autogpt-wolframalpha-search",
        "AutoGPTDiscord",
        "AutoGPTMessagesPlugin",
        "AutoGPTWebInteraction",
        "AutoGPTWolframAlpha"
    ]

when of course it is expected, as it is in the Auto-GPT .env file, as a comma separated list of plugin names within a single string. Personally, I prefer this structure, as I think it is cleaner and clearer, and would appreciate seeing it supported in some capacity. It is however a very semantic issue, so I understand if you guys don't want to stray from the way that Auto-GPT is currently doing it. I would appreciate supporting both versions, and simply checking the JSON to determine if the key ALLOWLISTED_PLUGINS is of type Array or of type String. But I can also just, ya know, flatten it myself... I guess...

@Peeeekay Peeeekay self-assigned this Jul 5, 2023
@Peeeekay
Copy link
Contributor

Peeeekay commented Jul 5, 2023

Hi, thanks for creating the ticket and great suggestion - let me bring this forward to the team and will prioritize it accordingly.

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

No branches or pull requests

2 participants