pushframe 5.1.8__tar.gz → 5.1.9__tar.gz

{pushframe-5.1.8 → pushframe-5.1.9}/CHANGELOG.md

@@ -12,6 +12,21 @@ every channel.
 `pushframe` is an unofficial community CLI for Aura Frames digital photo
 frames; it is not affiliated with Aura Frames Inc.
 
+## [5.1.9] - 2026-10-01
+
+### Fixed
+
+- **`google-sync`/`google-album` no longer fail on albums whose final page is
+  short** (2026-10-01 live drift, the 757-item "Cadre" album): Google now
+  emits a continuation token even on a short final page (300+300+157), and
+  the past-the-end request answers the null-payload shape — which the loop
+  read as the September transient, burned its single retry on and failed
+  loud, hard-blocking every run on that album. The loop now treats a short
+  page (< 300 items) as terminal — the paginated-API convention — and never
+  requests past it; the single-retry transient recovery stays for the
+  genuine mid-listing case. One new test reproduces the drift with the live
+  page counts.
+
 ## [5.1.8] - 2026-10-01
 
 ### Changed

{pushframe-5.1.8 → pushframe-5.1.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pushframe
-Version: 5.1.8
+Version: 5.1.9
 Summary: Unofficial community CLI for Aura Frames digital photo frames (sync a local folder or a Google Photos album; not affiliated with Aura Frames Inc.)
 Project-URL: Homepage, https://github.com/coredmp95/pushframe
 Project-URL: Repository, https://github.com/coredmp95/pushframe

{pushframe-5.1.8 → pushframe-5.1.9}/docs/ERRORS.md

@@ -58,6 +58,7 @@ twice.
 | `⚠ N of M photos … would be hidden — over the … mass-hide safety threshold` | The plan hides more than 20 % of the frame's photos, so it demands explicit confirmation | Read the counts; `y` only if that is really what you want |
 | `SKIPPED (--scheduled): this plan would hide too many photos …` | Same gate in scheduled mode: the run **skips and logs** instead of proceeding | Run `google-sync` manually, review the plan, apply deliberately |
 | `google-sync failed: reading frame assets failed: …` | The frame listing itself failed | Re-run; if persistent, `pushframe status` / `pushframe inspect` to check the account and frame |
+| `snAcKc carries a null inner payload on HTTP 200 …` | Google answered a page request with an empty envelope. One automatic retry already ran; if it recurs on every run it means the past-the-end request shape Google now answers with null (a listing-size change) — fixed in 5.1.9+; on older versions, update | Update pushframe; if it persists on the latest version, wait a few minutes and re-run |
 
 ## Writes, budget, pacing
 

{pushframe-5.1.8 → pushframe-5.1.9}/pushframe/__init__.py

@@ -1,4 +1,4 @@
 # pushframe — unofficial community CLI for Aura Frames (not affiliated).
 # Version single source for the CLI surface (REL-02); pyproject keeps the
 # packaging metadata version and tests/test_version.py enforces lockstep.
-__version__ = "5.1.8"
+__version__ = "5.1.9"

{pushframe-5.1.8 → pushframe-5.1.9}/pushframe/google/enumerate.py

@@ -189,8 +189,10 @@ def enumerate_album(session, album_id: str, *, page_key: str | None = None,
     """Enumerate EVERY photo of an album (D-06) via the RPC-first protocol:
 
     batch-1 = snAcKc(album_id, None, None, page_key) → 300 items; swap the
-    LAST AH_ cursor per page until no token or a 0-item page. Fail-loud on
-    HTTP != 200 or a malformed envelope (never a silent partial listing).
+    LAST AH_ cursor per page until no token, a 0-item page, or a SHORT page
+    (< PAGE_SIZE — the live-proven terminal signal, see the loop below).
+    Fail-loud on HTTP != 200 or a malformed envelope (never a silent
+    partial listing).
 
     `album_id` is the SHARE token (the /share/<id> path segment — what the
     /albums listing carries and what the live proof used). `page_key` is
@@ -213,7 +215,16 @@ def enumerate_album(session, album_id: str, *, page_key: str | None = None,
         known = {i["id"] for i in listing.items}
         fresh = [i for i in page_items if i["id"] not in known]
         listing.items.extend(fresh)
-        if next_token is None or not page_items:
+        # A SHORT page (< PAGE_SIZE) is the end of the listing — even when
+        # Google still mints a continuation token on it. Live drift,
+        # 2026-10-01 (the 757-item "Cadre" album: 300+300+157, page 3 short
+        # WITH a token): requesting the past-the-end page answers the
+        # null-payload shape, which used to read as the September transient,
+        # burn the single retry and fail loud — blocking the user entirely.
+        # Stopping on the short page is the paginated-API convention and
+        # costs nothing: the items above are already the complete listing.
+        if (next_token is None or not page_items
+                or len(page_items) < PAGE_SIZE):
             listing.exhausted_cleanly = True
             return listing
         token = next_token

{pushframe-5.1.8 → pushframe-5.1.9}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pushframe"
-version = "5.1.8"
+version = "5.1.9"
 description = "Unofficial community CLI for Aura Frames digital photo frames (sync a local folder or a Google Photos album; not affiliated with Aura Frames Inc.)"
 readme = "README.md"
 requires-python = ">=3.14"

{pushframe-5.1.8 → pushframe-5.1.9}/tests/test_cli_google_album.py

@@ -24,6 +24,7 @@ from pushframe.cli import (  # noqa: E402
     resolve_album,
     run_google_album,
 )
+from pushframe.google import enumerate as _enum  # noqa: E402
 from pushframe.google.client import GoogleSession  # noqa: E402
 from pushframe.google.parsers import AlbumSummary  # noqa: E402
 
@@ -156,7 +157,10 @@ def _session() -> GoogleSession:
                          transport=httpx.MockTransport(_router()))
 
 
-def test_google_album_by_name_full_enumeration_and_disk_weight(capsys):
+def test_google_album_by_name_full_enumeration_and_disk_weight(capsys, monkeypatch):
+    # The synthetic pages carry 5 items each — the full-page size, so the
+    # short-page-terminal rule (live drift 2026-10-01) never fires early.
+    monkeypatch.setattr(_enum, "PAGE_SIZE", 5)
     rc = run_google_album("corse", session=_session())
     assert rc == 0
     out = capsys.readouterr().out
@@ -189,9 +193,10 @@ def test_google_album_not_found_prints_albums_and_exits_2(capsys):
     assert "Famille 2024" in out
 
 
-def test_google_album_by_direct_link_resolves_and_enumerates(capsys):
+def test_google_album_by_direct_link_resolves_and_enumerates(capsys, monkeypatch):
     """A pasted share link works: the id is extracted and the page_key from
     the listing (when the link lacks one) comes from the /albums match."""
+    monkeypatch.setattr(_enum, "PAGE_SIZE", 5)   # synthetic full pages = 5 items
     rc = run_google_album(f"https://photos.google.com/share/{ALBUM_ID_1}",
                           session=_session())
     assert rc == 0

{pushframe-5.1.8 → pushframe-5.1.9}/tests/test_google_enumerate.py

@@ -79,7 +79,8 @@ class _GoogleRouter:
     def __init__(self, pages: list[int], *, page_key: str | None = PAGE_KEY,
                  rpc_status: int = 200, rpc_body: str | None = None,
                  albums_page: str | None = None,
-                 null_positions: set[int] | None = None) -> None:
+                 null_positions: set[int] | None = None,
+                 token_on_last_page: bool = False) -> None:
         self.pages = pages
         self.page_key = page_key
         self.rpc_status = rpc_status
@@ -88,6 +89,10 @@ class _GoogleRouter:
         # 1-based POST indices that answer the live-observed null-payload
         # transient instead of a page (a null consumes nothing).
         self.null_positions = null_positions or set()
+        # Live drift (2026-10-01, the 757-item "Cadre" album): Google emits
+        # a continuation token even on a SHORT final page; requesting past
+        # it answers the null-payload shape.
+        self.token_on_last_page = token_on_last_page
         self.albums_gets = 0
         self.post_count = 0
         self.nulls_served = 0
@@ -143,7 +148,11 @@ class _GoogleRouter:
             if page_idx >= len(self.pages):
                 return httpx.Response(400, text="too many pages requested")
             has_next = page_idx + 1 < len(self.pages)
-            next_token = _next_token(page_idx + 2) if has_next else None
+            if has_next or (self.token_on_last_page
+                            and page_idx + 1 == len(self.pages)):
+                next_token = _next_token(page_idx + 2)
+            else:
+                next_token = None
             self._expected_token = next_token
             return httpx.Response(200, text=_rpc_response(
                 _rpc_items(page_idx + 1, self.pages[page_idx]), next_token))
@@ -289,4 +298,19 @@ def test_null_retry_recovers_on_later_page_within_budget():
     assert listing.page_count == 2
     assert listing.exhausted_cleanly is True
     assert router.post_count == 3      # page 1, null, retried page 2
-    assert router.nulls_served == 1
+
+
+def test_short_page_is_terminal_even_when_google_mints_a_token():
+    """The 2026-10-01 live drift (the 757-item 'Cadre' album): Google emits
+    a continuation token even on a SHORT final page (157 < 300), and the
+    call past it answers the null-payload shape — which the old loop read
+    as the September transient, retried once, and failed loud, hard-blocking
+    the user. A short page IS the end of the listing: stop there, exhausted
+    cleanly, and never request the past-the-end page."""
+    router = _GoogleRouter([300, 300, 157], token_on_last_page=True)
+    listing = enumerate_album(_session(router), ALBUM_ID, page_key=PAGE_KEY)
+    assert len(listing.items) == 757            # 300 + 300 + 157, live counts
+    assert listing.exhausted_cleanly is True
+    assert listing.page_count == 3
+    assert router.post_count == 3               # the null page is never requested
+    assert router.nulls_served == 0

{pushframe-5.1.8 → pushframe-5.1.9}/tests/test_version.py

@@ -19,7 +19,7 @@ PYPROJECT = Path(__file__).resolve().parent.parent / "pyproject.toml"
 def test_version_is_first_distribution_release():
     """The current distribution release (moved off 0.1.0 at 5.0.0; page
     metadata fix shipped in 5.0.4)."""
-    assert __version__ == "5.1.8"
+    assert __version__ == "5.1.9"
 
 
 def test_version_matches_pyproject_single_source():

{pushframe-5.1.8 → pushframe-5.1.9}/uv.lock

@@ -432,7 +432,7 @@ wheels = [
 
 [[package]]
 name = "pushframe"
-version = "5.1.8"
+version = "5.1.9"
 source = { editable = "." }
 dependencies = [
     { name = "boto3" },