feat: add --accept-apple-changes flag, GIF/WEBP test fixtures

--accept-apple-changes: re-download files when iCloud reports a
different version_size. Without this flag, version changes are
logged as warnings but files are not re-downloaded. Metadata
updates from iCloud are always applied regardless.

Also adds test fixtures for GIF and WEBP formats with round-trip
and skip behaviour tests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: teh-hippo

src/icloudpd/base.py

@@ -454,6 +454,7 @@ def password_provider(_username: str) -> str | None:
                     dir_cache,
                     manifest,
                     collision_paths,
+                    user_config.accept_apple_changes,
                 )
                 if user_config.directory is not None
                 else (lambda _s, _c, _p: False)
@@ -784,6 +785,7 @@ def download_builder(
     dir_cache: DirCache,
     manifest: "ManifestDB | None",
     collision_paths: set[str],
+    accept_apple_changes: bool,
     icloud: PyiCloudService,
     counter: Counter,
     photo: PhotoAsset,
@@ -907,15 +909,24 @@ def download_builder(
                                 truncate_middle(download_path, 96),
                             )
                     elif manifest_row.version_size != version.size:
-                        # iCloud has a different version — re-download to the canonical path
-                        file_exists = False
-                        rel_path = os.path.relpath(download_path, directory)
-                        logger.debug(
-                            "%s version changed (manifest: %d, iCloud: %d), re-downloading",
-                            truncate_middle(download_path, 96),
-                            manifest_row.version_size,
-                            version.size,
-                        )
+                        # iCloud has a different version
+                        if accept_apple_changes:
+                            file_exists = False
+                            rel_path = os.path.relpath(download_path, directory)
+                            logger.info(
+                                "%s version changed (manifest: %d, iCloud: %d), re-downloading",
+                                truncate_middle(download_path, 96),
+                                manifest_row.version_size,
+                                version.size,
+                            )
+                        else:
+                            logger.warning(
+                                "%s version changed (manifest: %d, iCloud: %d), "
+                                "use --accept-apple-changes to re-download",
+                                truncate_middle(download_path, 96),
+                                manifest_row.version_size,
+                                version.size,
+                            )
                     else:
                         # Version matches — update metadata if needed
                         meta = _extract_manifest_metadata(photo, version)

src/icloudpd/cli.py

@@ -312,6 +312,15 @@ def add_options_for_user(parser: argparse.ArgumentParser) -> argparse.ArgumentPa
         help="Don't download any photos (default: download all photos and videos)",
         action="store_true",
     )
+    cloned.add_argument(
+        "--accept-apple-changes",
+        help=(
+            "Re-download files when iCloud reports a different version. "
+            "Without this flag, version changes are logged as warnings. "
+            "Metadata updates from iCloud are always applied regardless."
+        ),
+        action="store_true",
+    )
     return cloned
 
 
@@ -560,6 +569,7 @@ def map_to_config(user_ns: argparse.Namespace) -> UserConfig:
         skip_created_before=user_ns.skip_created_before,
         skip_created_after=user_ns.skip_created_after,
         skip_photos=user_ns.skip_photos,
+        accept_apple_changes=user_ns.accept_apple_changes,
     )
 
 

src/icloudpd/config.py

@@ -51,6 +51,7 @@ class _DefaultConfig:
     skip_created_before: datetime.datetime | datetime.timedelta | None
     skip_created_after: datetime.datetime | datetime.timedelta | None
     skip_photos: bool
+    accept_apple_changes: bool
 
 
 @dataclass(kw_only=True)

tests/data/test_image.gif

@@ -235,6 +235,7 @@ def test_cli_parser(self) -> None:
                         skip_created_before=None,
                         skip_created_after=None,
                         skip_photos=False,
+                        accept_apple_changes=False,
                     ),
                     UserConfig(
                         directory="def",
@@ -276,6 +277,7 @@ def test_cli_parser(self) -> None:
                         skip_created_before=None,
                         skip_created_after=None,
                         skip_photos=False,
+                        accept_apple_changes=False,
                     ),
                 ],
             ),
@@ -355,6 +357,7 @@ def test_cli_parser(self) -> None:
                         ).astimezone(get_localzone()),
                         skip_created_after=datetime.timedelta(days=2),
                         skip_photos=False,
+                        accept_apple_changes=False,
                     ),
                 ],
             ),

tests/data/test_image.webp

@@ -1412,3 +1412,55 @@ def test_build_args_orientation_zero_skipped(self) -> None:
         update = MetadataUpdate(orientation=0)
         args = build_exiftool_args(update, {"orientation"}, "/photos/test.JPG")
         self.assertFalse(any("Orientation" in a for a in args))
+
+
+class TestWebpRoundTrip(TestCase):
+    """Test WEBP metadata round-trip."""
+
+    def setUp(self) -> None:
+        self.tmp_dir = tempfile.mkdtemp()
+
+    def tearDown(self) -> None:
+        shutil.rmtree(self.tmp_dir)
+
+    def test_webp_rating_round_trip(self) -> None:
+        """WEBP should support Rating via EXIF."""
+        src = os.path.join(_DATA_DIR, "test_image.webp")
+        path = os.path.join(self.tmp_dir, "test.webp")
+        shutil.copy2(src, path)
+        update = MetadataUpdate(rating=5)
+        result = write_metadata(path, update, {"rating"})
+        self.assertTrue(result)
+        result2 = write_metadata(path, update, {"rating"})
+        self.assertFalse(result2, "Second write should be idempotent")
+
+    def test_webp_gps_round_trip(self) -> None:
+        """WEBP should support GPS via EXIF."""
+        src = os.path.join(_DATA_DIR, "test_image.webp")
+        path = os.path.join(self.tmp_dir, "test.webp")
+        shutil.copy2(src, path)
+        update = MetadataUpdate(gps_latitude=-33.86, gps_longitude=151.21, gps_altitude=50.0)
+        result = write_metadata(path, update, {"location"})
+        self.assertTrue(result)
+        result2 = write_metadata(path, update, {"location"})
+        self.assertFalse(result2, "Second write should be idempotent")
+
+
+class TestGifSkip(TestCase):
+    """Test that GIF files are skipped for EXIF writes."""
+
+    def test_gif_write_returns_false(self) -> None:
+        """GIF should not attempt any exiftool writes."""
+        update = MetadataUpdate(rating=5, gps_latitude=-33.86, gps_longitude=151.21, orientation=6)
+        args = build_exiftool_args(update, {"all"}, "/photos/test.gif")
+        self.assertEqual(args, [], "GIF should produce no exiftool args")
+
+    def test_gif_write_metadata_returns_false(self) -> None:
+        """write_metadata on GIF should return False (no args to write)."""
+        src = os.path.join(_DATA_DIR, "test_image.gif")
+        path = os.path.join(tempfile.mkdtemp(), "test.gif")
+        shutil.copy2(src, path)
+        update = MetadataUpdate(rating=5)
+        result = write_metadata(path, update, {"rating"})
+        self.assertFalse(result, "GIF writes should be skipped")
+        shutil.rmtree(os.path.dirname(path))