refactor: drop compound databaseId:tableId resourceId

[abnegate]

Summary

• CSV/JSON sources and destinations now take string \$databaseId, string \$tableId directly. The colon-splitting on \$resourceId is gone.
• Target::run / Source::run / Destination::run / Transfer::run gain a new optional \$rootResourceChildId parameter. For database roots, this scopes the transfer to a specific table within the root database. \$rootResourceId keeps its existing semantics (always matches \$rootResourceType).
• Sources/Appwrite no longer inspects `rootResourceId` for ":" or splits it; uses `$this->rootResourceChildId` for per-table filtering.

Test plan

• composer lint passes locally
• composer check (PHPStan level 3) passes locally
• CSV/JSON unit tests updated to pass separate IDs

Linked

• Consumer: refactor(migrations): split composite resourceId into parent/leaf appwrite/appwrite#12193

🤖 Generated with Claude Code

[greptile-apps]

Greptile Summary

This PR replaces the implicit databaseId:tableId compound string with explicit, typed $databaseId / $tableId constructor parameters in the CSV and JSON adapters, and introduces a first-class $rootResourceChildId parameter on the run() pipeline to scope transfers to a specific table within a root database — removing all str_contains/explode(':') logic from Sources/Appwrite.php.

• CSV/JSON sources and destinations now accept separate $databaseId and $tableId constructor arguments, eliminating the fragile colon-split that would panic at runtime if a caller omitted the delimiter.
• Target::run / Source::run / Destination::run / Transfer::run gain an optional $rootResourceChildId parameter (default ''), fully backward-compatible; all signatures match the abstract declaration.
• Sources/Appwrite table-filter logic is simplified to use $this->rootResourceChildId directly instead of parsing $rootResourceId.

Confidence Score: 5/5

All changed code paths are backward-compatible additions or straightforward string-parsing removals; no logic was dropped that wasn't replaced by equivalent, cleaner code.

The refactor is mechanically consistent across every layer of the pipeline: abstract signature, concrete overrides, forwarding calls, and tests all align. The previous abstract-signature mismatch flagged in an earlier review is confirmed fixed. No pre-existing colon-split edge case was dropped without replacement, and the $rootResourceChildId default of '' keeps all existing callers working without change.

No files require special attention.

Important Files Changed

Filename	Overview
src/Migration/Target.php	Adds $rootResourceChildId protected property and updates the abstract run() signature with matching parameter order across all overrides.
src/Migration/Source.php	Adds $rootResourceChildId parameter to run() and assigns it to the new property; signature matches the abstract declaration.
src/Migration/Destination.php	Adds $rootResourceChildId parameter and forwards it to the source's run() call; signatures are consistent.
src/Migration/Transfer.php	Adds nullable $rootResourceChildId parameter (null-coalesced to '' before forwarding), consistent with the existing $rootResourceType pattern.
src/Migration/Destinations/Appwrite.php	Adds $rootResourceChildId to the overriding run() and passes it to parent::run(); no logic changes.
src/Migration/Destinations/CSV.php	Replaces single $resourceId constructor param with explicit $databaseId + $tableId; error messages and warning context updated accordingly.
src/Migration/Destinations/JSON.php	Same split-param refactor as CSV destination; consistent error messaging.
src/Migration/Sources/Appwrite.php	Removes all str_contains/explode logic on $rootResourceId; uses $rootResourceChildId for per-table filtering in exportEntities.
src/Migration/Sources/CSV.php	Replaces $resourceId with $databaseId + $tableId; eliminates the explode(':') call that would panic on inputs without a colon.
src/Migration/Sources/JSON.php	Same split-param refactor as CSV source; exportRows now builds Database/Table objects directly from the typed properties.
tests/Migration/Unit/General/CSVTest.php	All six test instantiations of TestCSV updated to pass separate 'test_db' and 'test_table_id' arguments.
tests/Migration/Unit/General/JSONTest.php	Both DestinationJSON constructor calls updated to pass separate database and table IDs.

_{Reviews (3): Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." | Re-trigger Greptile}

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.