From 2b3795224d707756059b3c58eb3abfcc3d5c1e90 Mon Sep 17 00:00:00 2001
From: Jake Barnby <jakeb994@gmail.com>
Date: Fri, 15 Aug 2025 01:33:22 +1200
Subject: [PATCH] Check for documentId on ops that require it

---
 src/Appwrite/Utopia/Database/Validator/Operation.php | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/Appwrite/Utopia/Database/Validator/Operation.php b/src/Appwrite/Utopia/Database/Validator/Operation.php
index 8ef3817668..ac585e26bc 100644
--- a/src/Appwrite/Utopia/Database/Validator/Operation.php
+++ b/src/Appwrite/Utopia/Database/Validator/Operation.php
@@ -71,6 +71,15 @@ class Operation extends Validator
             return false;
         }
 
+        // If action requires documentId, it must be present
+        if (
+            isset($this->requiresDocumentId[$value['action']]) &&
+            !\array_key_exists('documentId', $value)
+        ) {
+            $this->description = "Key 'documentId' is required for action '{$value['action']}'";
+            return false;
+        }
+
         // Data must be array (can be empty)
         if (!\is_array($value['data'])) {
             $this->description = "Key 'data' must be an array";