jacob-ai-bot / jacob

Just Another Coding Bot
https://jacb.ai
Apache License 2.0
131 stars 20 forks source link

Bug: Plan Generation Includes Folders Instead of File Names Causing Errors #293

Closed kleneway closed 1 week ago

kleneway commented 1 week ago

Description

There is a bug when creating a plan where, for actions such as "create new file" or "edit existing file," the plan sometimes includes a folder path instead of an actual file name. When the system attempts to create or edit a folder as if it were a file, it throws an error and breaks the entire process.

Additionally, we need to ensure that for every file listed in the plan:

It is believed that there is code already in place to verify the existence of files, but we need to double-check to ensure this validation is working correctly.

Expected Outcome

If the plan has a step that is to create a new folder, just ignore that step and definitely don't save that step to the database. @jacob-ai-bot --skip-build

jacob-ai-bot[bot] commented 1 week ago

JACoB here...

You mentioned me on this issue and I am busy taking a look at it.

I'll continue to comment on this issue with status as I make progress.

jacob-ai-bot[bot] commented 1 week ago

Update

I've completed my work on this issue and have created a pull request: JACoB PR for Issue Bug: Plan Generation Includes Folders Instead of File Names Causing Errors.

Please review my changes there.

jacob-ai-bot[bot] commented 1 week ago

Update

I've updated this pull request: JACoB PR for Issue Bug: Plan Generation Includes Folders Instead of File Names Causing Errors.

The changes currently result in an error, so I'll be making some additional changes before it is ready to merge.

jacob-ai-bot[bot] commented 1 week ago

Update

I've updated this pull request: JACoB PR for Issue Bug: Plan Generation Includes Folders Instead of File Names Causing Errors.

The changes currently result in an error, so I'll be making some additional changes before it is ready to merge.

jacob-ai-bot[bot] commented 1 week ago

Update

I've updated this pull request: JACoB PR for Issue Bug: Plan Generation Includes Folders Instead of File Names Causing Errors.

The changes currently result in an error, so I'll be making some additional changes before it is ready to merge.

jacob-ai-bot[bot] commented 1 week ago

Update

I've updated this pull request: JACoB PR for Issue Bug: Plan Generation Includes Folders Instead of File Names Causing Errors.

The changes currently result in an error, so I'll be making some additional changes before it is ready to merge.