This repository was archived by the owner on Aug 1, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 4
This repository was archived by the owner on Aug 1, 2024. It is now read-only.
Some early findings #1
Copy link
Copy link
Open
Description
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!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels