gpt-pilot icon indicating copy to clipboard operation
gpt-pilot copied to clipboard

`Developer.set_up_environment()` should install & verify installation

Open nalbion opened this issue 1 year ago • 1 comments

Developer.set_up_environment() instructs the user

Please set up your local environment so that the technologies listed can be utilized. When you're done, write "DONE"

It's easy for the user to just type "DONE" without actually having installed the technologies.

Later, development/iteration.prompt relies on this being true

At development/define_user_review_goal.prompt I replied:

node is already running but MongoDB is not, I see "Cannot get /"

This feedback is included in development/iteration.prompt which includes:

IMPORTANT Do not tell me anything about setting up the database or anything OS related - only if some dependencies need to be installed.

so the LLM responds:

You do need to have MongoDB running on your machine on default port 27017 as described in the "server.js" file - however, you mentioned that doesn't need to be addressed.

Note that the LLM response also included:

There's no user-validation or message filtering based on receiver. You might want to improve this small chat app further by adding these features.

nalbion avatar Oct 09 '23 01:10 nalbion

Note that there are different types of "technologies" and installation should be treated separately:

  • build-managed dependencies (package.json, requirements.txt etc)
  • external dependencies - MongoDB, Node.js...

The prompt should use a richer JSON schema which allows the LLM to:

  • classify them
  • determine if they are installed
  • provide links/instructions on how to install
  • suggest versions

The user should be able to suggest changes to the list also.

Rather than just telling the user "You need to go and install Node.js & bootstrap etc" it could take ownership of installing bootstrap, and help the user with Node.js installation if it's not already installed.

See also #189

nalbion avatar Oct 14 '23 03:10 nalbion