Only take subclasses of AutoGPTPluginTemplate as plugins

[ppetermann]

this fixes #4215, and is a re-submission of #4217 where i managed to screw up the history of the PR.

Background

limit plugin detection to classes that implement AutoGPTPluginTemplate (have it as parent),
solving #4215

Changes

added check to scan_plugin

Test Plan

create empty plugin, add class triggering old detection -> run = only real plugins detected

PR Quality Checklist

• My pull request is atomic and focuses on a single change.
• I have thoroughly tested my changes with multiple different prompts.
• I have considered potential risks and mitigations for my changes.
• I have documented my changes clearly and comprehensively.
• I have not snuck in any "extra" small tweaks changes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 10, 2023 3:40am

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (cfdb24e) 70.77% compared to head (a12d5d9) 70.77%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4345   +/-   ##
=======================================
  Coverage   70.77%   70.77%           
=======================================
  Files          68       68           
  Lines        3326     3326           
  Branches      532      532           
=======================================
  Hits         2354     2354           
  Misses        795      795           
  Partials      177      177           

Impacted Files	Coverage Δ
autogpt/plugins/__init__.py	73.54% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[vercel]

Deployment failed with the following error:

Resource is limited - try again in 19 minutes (more than 100, code: "api-deployments-free-per-day").

[vercel]

Deployment failed with the following error:

Resource is limited - try again in 9 hours (more than 100, code: "api-deployments-free-per-day").

[vercel]

Deployment failed with the following error:

Resource is limited - try again in 8 hours (more than 100, code: "api-deployments-free-per-day").

[vercel]

Deployment failed with the following error:

Resource is limited - try again in 7 hours (more than 100, code: "api-deployments-free-per-day").

[github-actions]

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

[netlify]

✅ Deploy Preview for auto-gpt-docs canceled.

Name	Link
🔨 Latest commit	a12d5d9
🔍 Latest deploy log	https://app.netlify.com/sites/auto-gpt-docs/deploys/6499e273126218000870d98b

[github-actions]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

[lc0rp]

Changes made in subsequent commits.

The changes were made in a subsequent commit

[ppetermann]

my python might mislead me here, but doesn't this mean the class needs to exist as such? the class is currently a dependency of the plugins not a part of autogpt itself, (currently its listed in the dev-dependencies in requirements.txt, this change makes it a runtime dependency if i'm not mistaken)