clarify agent instructions (#5655)
This commit is contained in:
@@ -43,7 +43,8 @@
|
|||||||
- Suggestions to add comments for self-documenting code
|
- Suggestions to add comments for self-documenting code
|
||||||
- Refactoring suggestions unless there's a clear bug or maintainability issue
|
- Refactoring suggestions unless there's a clear bug or maintainability issue
|
||||||
- Listing multiple potential issues in one comment (choose the single most critical issue)
|
- Listing multiple potential issues in one comment (choose the single most critical issue)
|
||||||
- Suggestions to add more logging (the codebase needs less logging, not more)
|
- Suggestions to add logging statements, unless for errors or security events (the codebase needs less logging, not more)
|
||||||
|
|
||||||
|
|
||||||
## Response Format
|
## Response Format
|
||||||
|
|
||||||
|
|||||||
@@ -85,7 +85,7 @@ Simplicity: Don't make things optional that don't need to be - the compiler will
|
|||||||
Simplicity: Booleans should default to false, not be optional
|
Simplicity: Booleans should default to false, not be optional
|
||||||
Errors: Don't add error context that doesn't add useful information (e.g., `.context("Failed to X")` when error already says it failed)
|
Errors: Don't add error context that doesn't add useful information (e.g., `.context("Failed to X")` when error already says it failed)
|
||||||
Simplicity: Avoid overly defensive code - trust Rust's type system
|
Simplicity: Avoid overly defensive code - trust Rust's type system
|
||||||
Logging: Clean up existing logs, don't add more
|
Logging: Clean up existing logs, don't add more unless for errors or security events
|
||||||
|
|
||||||
## Never
|
## Never
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user