agents/rules/quality-code-comments.md
Keep comments limited and avoid obvious ones. Comments should explain "why" not "what" - the code itself should be clear enough to explain what it does.
If none of these apply, skip the comment entirely. The function name, parameters, and return type should speak for themselves.
// ❌ Bad - Obvious comment
// Get the user
const user = await getUser(userId);
// ❌ Bad - Restating the code
// Loop through bookings
for (const booking of bookings) {
// Process booking
processBooking(booking);
}
// ✅ Good - Explains why, not what
// We need to fetch availability before slots because the timezone
// conversion depends on the user's configured availability rules
const availability = await getAvailability(userId);
const slots = convertToSlots(availability, timezone);
// ✅ Good - Documents a non-obvious constraint
// Google Calendar API has a 2500 event limit per sync request
const BATCH_SIZE = 2500;