Improve backup policy migration error handling

Author: premtsd-code

src/Migration/Destinations/Appwrite.php

@@ -1498,10 +1498,9 @@ public function importBackupResource(Resource $resource): Resource
                     $collection = match ($resource->getResourceType()) {
                         Resource::TYPE_DATABASE => 'databases',
                         Resource::TYPE_BUCKET => 'buckets',
-                        default => null, // Only databases and buckets support per-resource backup policies
+                        default => null,
                     };
 
-                    // Only pass resourceId for supported resource types
                     if ($collection !== null) {
                         $doc = $this->database->getDocument($collection, $resource->getResourceId());
                         if ($doc->isEmpty()) {
@@ -1514,6 +1513,12 @@ public function importBackupResource(Resource $resource): Resource
                         }
 
                         $params['resourceId'] = $resource->getResourceId();
+                    } elseif ($resource->getResourceType()) {
+                        // Per-resource backup policies only supported for databases and buckets
+                        $resource->setStatus(
+                            Resource::STATUS_WARNING,
+                            'Per-resource backup policies not supported for resource type: ' . $resource->getResourceType() . '. Created as service-level policy instead.'
+                        );
                     }
                 }
 

src/Migration/Sources/Appwrite.php

@@ -1651,8 +1651,16 @@ private function reportBackups(array $resources, array &$report, array $resource
             ]);
 
             $report[Resource::TYPE_BACKUP_POLICY] = $response['total'] ?? 0;
-        } catch (\Throwable) {
-            // Backup policies are Cloud-only, skip gracefully for self-hosted
+        } catch (\Throwable $e) {
+            $body = \json_decode($e->getMessage(), true);
+            $code = $body['code'] ?? 0;
+
+            // Re-throw permission errors as they indicate configuration issues
+            if ($code === 401 || $code === 403) {
+                throw new \Exception('Missing permission to access backup policies: ' . $e->getMessage(), previous: $e);
+            }
+
+            // Feature not available (404/501) - skip gracefully for self-hosted
             $report[Resource::TYPE_BACKUP_POLICY] = 0;
         }
     }