Significant-Gravitas / AutoGPT

AutoGPT is the vision of accessible AI for everyone, to use and to build on. Our mission is to provide the tools, so that you can focus on what matters.
https://agpt.co
Other
168.53k stars 44.47k forks source link

dx: Reduce number of files needed for OAuth and/or Webhook-triggered blocks #8735

Open Pwuts opened 1 day ago

Pwuts commented 1 day ago

The number of files you need to add/edit for new blocks with auth, and webhook-triggered blocks, is quite large. For DX we should try to reduce/simplify this.

File you have to add/edit for a new block:

Files where references have to be added for a new provider (in the credentials system):

Extra files for a new OAuth provider:

Extra files for a new WebhooksManager:

I'm sure we can put the front end stuff together in one file. Not so sure about the back end. As long as it's clearly documented, I don't see an issue with having to add (references in) multiple files, but if we can reasonably avoid it that's still good.

_Originally posted by @Pwuts in https://github.com/Significant-Gravitas/AutoGPT/pull/8358#discussion_r1843982423_

Pwuts commented 1 day ago

I wonder if we can do reflection to handle some of this via enumerators like we do enumerating the blocks in the directory

\~ @ntindle

Pwuts commented 1 day ago

I wonder if we can do reflection to handle some of this via enumerators like we do enumerating the blocks in the directory

\~ @ntindle