diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index deb8d64ca4e3..173fec0dc7c8 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -43,7 +43,8 @@ - Suggestions to add comments for self-documenting code - Refactoring suggestions unless there's a clear bug or maintainability 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 diff --git a/AGENTS.md b/AGENTS.md index 35e0cc08c69f..75a4c5f0c478 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -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 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 -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