From 049061a68b10ec7f225a04d7616eb8c2ff840b54 Mon Sep 17 00:00:00 2001 From: Joshua Liebow-Feeser Date: Mon, 2 Feb 2026 09:54:10 -0500 Subject: [PATCH] [agents] Update review guidelines for TODOs Addresses https://github.com/google/zerocopy/pull/2966#discussion_r2754618840 --- agent_docs/reviewing.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/agent_docs/reviewing.md b/agent_docs/reviewing.md index 7af780e880..7ca4d2cb0c 100644 --- a/agent_docs/reviewing.md +++ b/agent_docs/reviewing.md @@ -92,6 +92,14 @@ resolved. When you encounter a `TODO` comment: be resolved*. 2. **Critique** only if the `TODO` is insufficient (i.e., the code would still be problematic *even if* the `TODO` were resolved). +3. **Safety Placeholders:** A `// SAFETY: TODO` comment is a valid placeholder + for a safety comment, and a `/// TODO` comment in a `/// # Safety` doc + section is a valid placeholder for safety documentation. **DO NOT** flag + the first as a missing safety justification or a critical issue, and **DO + NOT** flag the second as missing safety documentation. You must assume the + author will write a sound justification or accurate safety documentation + before merging. + ## 4. Anti-Patterns (NEVER Do This)