From 18e988291638b9f3a6e561ff127e134625829273 Mon Sep 17 00:00:00 2001
From: John Preston <johnprestonmail@gmail.com>
Date: Thu, 12 Mar 2026 17:43:53 +0400
Subject: [PATCH] [ai] Add static member naming review rule.

---
 REVIEW.md | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/REVIEW.md b/REVIEW.md
index 312fcd20c5..3aaaae01c7 100644
--- a/REVIEW.md
+++ b/REVIEW.md
@@ -269,3 +269,15 @@ if (!document || duration <= 0) {
     return;
 }
 ```
+
+## Static member functions use PascalCase
+
+Non-static member functions use camelCase (`startBatch`, `finalize`). Static member functions use PascalCase (`ShouldTrack`, `Parse`, `Create`), matching the convention for free functions.
+
+```cpp
+// BAD - camelCase for static method:
+[[nodiscard]] static bool shouldTrack(not_null<HistoryItem*> item);
+
+// GOOD - PascalCase for static method:
+[[nodiscard]] static bool ShouldTrack(not_null<HistoryItem*> item);
+```