gitcommitshow / autogpt-node

AutoGPT for node.js

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Some early findings

Nojahhh opened this issue · comments

Hello @gitcommitshow,

Nice to see AutoGPT is coming to Node.js!

I haven't been successful in getting this to run yet but I'm looking forward to it.

A couple of things I've encountered during a short test-session:

  • lib/browser.js needs to be rewritten as your currently exporting a function and the main script requires a constructor.
  • There is no "on" method built in to your binary file.
  • Your different functions for generating Though, Reasoning, Plan and Critisism under lib/openai.js is all the same function repeated with a new name. I'm guessing this is just something that you haven't completed yet.
  • Trying to run bin/autogpt.js after adressing these issues will result in ERR_REQUIRE_ESM which states that inquirer.js must be imported differently but that's impossible since we can't use import outside of a module. Don't know why this happens and haven't looked deep enough to conclude anything yet.
  • There is a missing index.js file under bin/ as you re loading "const autogpt = require('./index');" on row 6 in main autogpt.js binary.

Since this library isn't fully functional yet it shouldn't be available for installation via npm imo but either way you should state that on the top of your README.md file for future users to see and it will also help developers noticing your project.

I will continue to look in to this as I have more time. In the meantime I will be looking forward seeing the development of this library!

Thanks a lot for the feedback. Useful.
Will to resolve all the issues this week. Do you have any other feedback on the api structure? Want to lock on this today.

  • Fixed ERR_REQUIRE_ESM with inquirer.js, requires fresh npm install (inquirer v9 does not support non-esm usage)
  • Fixed missing on observer method
  • Fixed missing bin/index.js

The project still won't work until, as I need to complete openai.js code (adding prompts). I worked on it today but spent too much time reading prompt engg. research papers, probably more than required. Will finish it in the next release. After that, it should be ready to use.