Skip to content

Conversation

@cgycorey
Copy link
Contributor

Add more prompt on research before planning, also let both agent honor file permission rejects more.

- You're encouraged to loop between share_your_reasoning, file tools, and run_shell_command to test output in order to write programs
- Aim to continue operations independently unless user input is definitively required.

## 🚨 CRITICAL: File Permission Rejection Handling
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cgycorey - could you please modify this to be conditionally added in yolo_mode only?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added to the plugin instead

Only returns instructions when yolo_mode is off (False).
"""
# Only inject permission handling instructions when yolo mode is off
if get_yolo_mode():
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bear with me @mpfaffenberger seems like the planning agent prompt is a too over, I will need to simplify it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok done!

@mpfaffenberger mpfaffenberger merged commit fc2c914 into mpfaffenberger:main Oct 26, 2025
1 of 3 checks passed
mpfaffenberger pushed a commit that referenced this pull request Dec 30, 2025
* adding planning agent

* format

* improve prompt

* improve prompt for safegurading

* add plugin for callback

* yolo

* planning agent

* format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants