Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addition of web search feature for better error resolution and new feature implementation #383

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

obliviousz
Copy link
Contributor

@obliviousz obliviousz commented Apr 6, 2024

Enhancements in this PR include:

  • Added web search functionality to the error resolution process in runner.py, triggered when the AI identifies unsolvable errors. This enhancement is crucial as error complexity varies dynamically, necessitating external resources for resolution.
  • Modified command_output = process.stdout.decode('utf-8') to capture error messages accurately by redirecting stdout to stderr. Previously, blank error strings were being returned which was not at all helpful for the AI and it had to guess from the code itself.

@obliviousz
Copy link
Contributor Author

Here is a video of the same (here although error was hardcoded and browser opening was hardcoded just to show)

screen-capture.webm

@obliviousz
Copy link
Contributor Author

Requesting your review @ARajgor

@Ahmet0691
Copy link
Contributor

thanks bro

…equired in both run and bug actions as bug also tries to patch
@obliviousz
Copy link
Contributor Author

I added an Error analyzer agent because for "bug" and "run" both we needed similar function error analyzer, plus we cannot import Agent into them due to circular dependency, hence it was best to create another Agent called Error Analyzer which analyzes the error and solves the error either by itself or through net what ever AI feels is best. This error analyzer could be used other places too I feel and hence an Error Analyzer agent was important

@obliviousz
Copy link
Contributor Author

Added web search functionality for feature addition and tested it too.

@obliviousz obliviousz changed the title Addition of web search feature for better error resolution Addition of web search feature for better error resolution and new feature implementation Apr 9, 2024
@ARajgor
Copy link
Collaborator

ARajgor commented Apr 12, 2024

Not sure about your changes...
But you should take browser thing from browser and search file instead re implementing in every files.

@obliviousz
Copy link
Contributor Author

Hey. Did I write any code that is repeating anything from the browser or search? I couldn't find out. Please do point out and I'll fix. Thanks.

@ARajgor
Copy link
Collaborator

ARajgor commented Apr 14, 2024

looks good to me.

@ARajgor ARajgor added the enhancement New feature or request label Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants