Utilize environment variables for all agent key bindings

[scottschluer]

Background

Support for using environment variables to store key bindings was incomplete. Hard-coded references to y, n and s keys were still included in console output. Additionally, there was no environment variable declared for the new SELF_FEEDBACK_KEY.

Changes

Added self_feedback_key to config.py and test_plugins.py. Updated agent.py to use configuration values for all references to "yes", "no" and "self-feedback" keys.

Documentation

No documentation updates required.

Test Plan

Tested locally over several runs. Utilized all keys and verified output and behavior matched expectations.

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 ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Ignored (Inspect)			May 6, 2023 2:11pm

[p-i-]

This is a mass message from the AutoGPT core team.
Our apologies for the ongoing delay in processing PRs.
This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to:
https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

[github-actions]

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

[github-actions]

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

[github-actions]

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

[vercel]

Deployment failed with the following error:

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

[github-actions]

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

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (e088295) 50.05% compared to head (59087e9) 50.05%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3774   +/-   ##
=======================================
  Coverage   50.05%   50.05%           
=======================================
  Files         116      116           
  Lines        4799     4799           
  Branches      643      643           
=======================================
  Hits         2402     2402           
  Misses       2218     2218           
  Partials      179      179           

Impacted Files	Coverage Δ
autogpt/agent/agent.py	59.18% <ø> (ø)

☔ 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 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.

[github-actions]

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

[Pwuts]

Self feedback mode was removed due to breakage, but your other fixes are still good to have :)

[netlify]

✅ Deploy Preview for auto-gpt-docs ready!

Name	Link
🔨 Latest commit	59087e9
🔍 Latest deploy log	https://app.netlify.com/sites/auto-gpt-docs/deploys/64a72499ba06c90008543147
😎 Deploy Preview	https://deploy-preview-3774--auto-gpt-docs.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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