Fixed Json Error

[AGI-CEO]

Fixed the json_parser.py error by making chatgpt fix itself

def fix_and_parse_json(json_str: str, try_to_fix_with_gpt: bool = True):
json_schema = """
{
"command": {
"name": "command name",
"args":{
"arg name": "value"
}
},
"thoughts":
{
"text": "thought",
"reasoning": "reasoning",
"plan": "- short bulleted\n- list that conveys\n- long-term plan",
"criticism": "constructive self-criticism",
"speak": "thoughts summary to say to user"
}
}
"""

def clean_input(json_str):
    brace_index = json_str.index("{")
    json_str = json_str[brace_index:]
    last_brace_index = json_str.rindex("}")
    json_str = json_str[:last_brace_index+1]
    return json_str

def attempt_parse(json_str):
    try:
        return json.loads(json_str)
    except Exception as e:
        return None

cleaned_json_str = clean_input(json_str)
parsed_json = attempt_parse(cleaned_json_str)

if parsed_json is not None:
    return parsed_json
elif try_to_fix_with_gpt:
    print("Warning: Failed to parse AI output, attempting to fix.")
    ai_fixed_json = fix_json(cleaned_json_str, json_schema, False)

    if ai_fixed_json != "failed":
        return json.loads(ai_fixed_json)
    else:
        print("Failed to fix ai output, telling the AI.")
        return json_str

raise ValueError("Failed to parse JSON: '{}'".format(json_str))

[sberney]

Why get rid of this message about confusing the AI?

[Torantulino]

Agreed, please add this back in.

[sberney]

Why not keep the try/except error handling paradigm?

[sberney]

Why not un-nest these functions while you're at it?

[Torantulino]

What are all these new .idea files?

[Torantulino]

I assume you added these by mistake?
Please remove them.

[ksylvan]

What are all these new .idea files?

@Torantulino Those are IntelliJ IDE files that people keep pushing by mistake. We can add those to .gitignore so we don't get them in PRs.

[nponeccop]

@ksylvan Add them in a separate PR. Also resolve the conflicts.

[ksylvan]

@nponeccop Not my PR, I was just commenting on it, since I'd seen a few different PRs with .idea/ files in them. @jetblaise is the original author here.

[nponeccop]

@jetblaise Could you resolve the conflicts?

[github-actions]

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

[github-actions]

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

[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

[Pwuts]

Closing this PR as it has conflicts and long-standing unresolved requests for changes