src/Appwrite/Event/Event.php (1)

537-539: Use a root-prefix check for database remapping.

Line 537 currently uses str_contains($pattern, 'databases.'). Consider str_starts_with($pattern, 'databases.') to avoid remapping non-root patterns that merely contain that token.

💡 Suggested refinement

-        if ((str_contains($pattern, 'databases.') && $database && $database->getAttribute('type') !== 'legacy')) {
+        if ((str_starts_with($pattern, 'databases.') && $database && $database->getAttribute('type') !== 'legacy')) {
             $parsed = self::getDatabaseTypeEvents($database, $parsed);
         }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/Appwrite/Event/Event.php` around lines 537 - 539, The conditional that
decides to remap database events currently checks str_contains($pattern,
'databases.') which can match non-root patterns; update the check in Event::
(the if using $pattern and $database) to use str_starts_with($pattern,
'databases.') so only root-prefixed patterns trigger
getDatabaseTypeEvents($database, $parsed); keep the rest of the condition and
call to self::getDatabaseTypeEvents unchanged.

tests/e2e/Services/Realtime/RealtimeCustomClientTest.php (1)

2674-2676: Assert tablesdb.* event names explicitly in this E2E case.

At Line 2675 the test only validates the databases-prefixed compatibility event. Add a tablesdb.{db}.tables.{table}.rows.{row}.create assertion so regressions in primary tablesdb event generation are caught directly.

💡 Suggested assertion addition

         // Primary event should still be present
         $this->assertContains("databases.{$databaseId}.tables.{$tableId}.rows.{$rowId}.create", $response['data']['events']);
+        $this->assertContains("tablesdb.{$databaseId}.tables.{$tableId}.rows.{$rowId}.create", $response['data']['events']);
         $this->assertNotEmpty($response['data']['payload']);

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@tests/e2e/Services/Realtime/RealtimeCustomClientTest.php` around lines 2674 -
2676, Add an explicit assertion for the tablesdb-prefixed primary event in the
RealtimeCustomClientTest so regressions are caught: in the test method where
$response, $databaseId, $tableId, and $rowId are used (around the existing
assertions that check
"databases.{$databaseId}.tables.{$tableId}.rows.{$rowId}.create" and
$response['data']['payload']), add an assertContains for the string
"tablesdb.{$databaseId}.tables.{$tableId}.rows.{$rowId}.create" against
$response['data']['events'] to ensure the
tablesdb.{db}.tables.{table}.rows.{row}.create event is present.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro