(fix): CI — restore JSON decode for indexes in SQLite deleteAttribute/renameIndex

deleteAttribute and renameIndex call $this->getDocument(metadata, ...)
on the adapter directly, which doesn't apply the metadata 'json' filter
— so $collection->getAttribute('indexes', []) returns the raw JSON
string from the column. Commit 59c363c removed the json_decode under
the assumption the value was always pre-decoded; that's only true when
loaded via Database::getCollection().

The TypeError thrown from foreach() over a string surfaced as
testPurgeCollectionCache failing, and worse, testEvents threw before
its $database->on(..., null) cleanup ran — leaking the EVENT_ALL
listener into every later test that triggered an event, which is why
306 tests on SQLite started asserting '' matches null at
CollectionTests.php:1532.

Decode only when the value is still a string so both call paths work.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: 2 people

src/Database/Adapter/SQLite.php

@@ -612,6 +612,9 @@ public function deleteAttribute(string $collection, string $id, bool $array = fa
         }
 
         $indexes = $collection->getAttribute('indexes', []);
+        if (\is_string($indexes)) {
+            $indexes = \json_decode($indexes, true) ?? [];
+        }
 
         foreach ($indexes as $index) {
             $attributes = $index['attributes'];
@@ -662,6 +665,9 @@ public function renameIndex(string $collection, string $old, string $new): bool
         $old = $this->filter($old);
         $new = $this->filter($new);
         $indexes = $collection->getAttribute('indexes', []);
+        if (\is_string($indexes)) {
+            $indexes = \json_decode($indexes, true) ?? [];
+        }
         $index = null;
 
         foreach ($indexes as $node) {