From 737e2f86241f255c9f42ad151767a28da99176c2 Mon Sep 17 00:00:00 2001
From: Steven Nguyen <stnguyen90@users.noreply.github.com>
Date: Tue, 26 Aug 2025 15:10:23 -0700
Subject: [PATCH] fix: Remove $sequence attribute from documents on create and
 update

Including attribute can cause a invalid structure error as the incoming data may be an int but the database expects a string.
---
 .../Databases/Http/Databases/Collections/Documents/Action.php  | 3 +--
 .../Http/Databases/Collections/Documents/Bulk/Update.php       | 3 +++
 .../Databases/Http/Databases/Collections/Documents/Create.php  | 3 +++
 .../Databases/Http/Databases/Collections/Documents/Update.php  | 3 +++
 4 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Action.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Action.php
index 541d8edece..78df15b0c1 100644
--- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Action.php
+++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Action.php
@@ -29,7 +29,7 @@ abstract class Action extends AppwriteAction
 
         // Use the same helper method to ensure consistency
         $contextId = '$' . $this->getCollectionsEventsContext() . 'Id';
-        $this->removableAttributes = ['$databaseId', $contextId];
+        $this->removableAttributes = ['$databaseId', $contextId, '$sequence'];
 
         return parent::setHttpPath($path);
     }
@@ -232,7 +232,6 @@ abstract class Action extends AppwriteAction
 
         $collectionId = $collection->getId();
         $document->removeAttribute('$collection');
-        $document->removeAttribute('$tenant');
         $document->setAttribute('$databaseId', $database->getId());
         $document->setAttribute('$' . $this->getCollectionsEventsContext() . 'Id', $collectionId);
 
diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Bulk/Update.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Bulk/Update.php
index b5a4750c4f..a9f9c3f76d 100644
--- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Bulk/Update.php
+++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Bulk/Update.php
@@ -127,6 +127,9 @@ class Update extends Action
             }
         }
 
+        // Remove sequence if set
+        unset($document['$sequence']);
+
         $documents = [];
 
         try {
diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Create.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Create.php
index f2d678f411..04c90c4ec1 100644
--- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Create.php
+++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Create.php
@@ -351,6 +351,9 @@ class Create extends Action
                 }
             }
 
+            // Remove sequence if set
+            unset($document['$sequence']);
+
             // Assign a unique ID if needed, otherwise use the provided ID.
             $document['$id'] = $sourceId === 'unique()' ? ID::unique() : $sourceId;
 
diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Update.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Update.php
index 4f5055abc4..334bcb8448 100644
--- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Update.php
+++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Update.php
@@ -159,6 +159,9 @@ class Update extends Action
             $permissions = $document->getPermissions() ?? [];
         }
 
+        // Remove sequence if set
+        unset($document['$sequence']);
+
         $data['$id'] = $documentId;
         $data['$permissions'] = $permissions;
         $newDocument = new Document($data);