fplkit 2.3.1__tar.gz → 2.3.2__tar.gz

{fplkit-2.3.1 → fplkit-2.3.2}/CHANGELOG.md

@@ -1,6 +1,14 @@
 # Changelog
 
 All notable changes to this project will be documented in this file.
+## [2.3.1] - 2026-08-26
+
+### Bug Fixes
+
+- keep draft players matched to the main game when FPL renames them (#171)
+- judge a replayed gameweek's blanks by the clubs players had then (#173)
+- keep the club and name a replayed gameweek already recorded (#175)
+
 ## [2.3.0] - 2026-08-26
 
 ### Bug Fixes

{fplkit-2.3.1 → fplkit-2.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: fplkit
-Version: 2.3.1
+Version: 2.3.2
 Summary: CLI tool for Fantasy Premier League analysis - classic and draft formats
 Project-URL: Homepage, https://github.com/rossgroomio/fpl-cli
 Project-URL: Repository, https://github.com/rossgroomio/fpl-cli

{fplkit-2.3.1 → fplkit-2.3.2}/docs/command-reference.md

@@ -794,7 +794,7 @@ it shares the channel: `synthesis_provider_unavailable`.
 | `league_history_backfill_manager_unreachable` | One manager's history could not be fetched; their gameweeks stay unknown and are re-attempted next run |
 | `league_history_backfill_replay_failed` | One gameweek could not be replayed in detail; the others are unaffected |
 | `league_history_backfill_write_failed` | A backfilled gameweek could not be written; the rest of the backfill continues |
-| `league_history_identity_carried` | A replayed gameweek kept the name, club or position it already had recorded for one or more players rather than the ones today's bootstrap gives them, or restored a player reference the replay had lost. Any one of the four on its own raises it |
+| `league_history_identity_carried` | A finished gameweek kept the name, club or position it already had recorded for one or more players rather than the ones today's bootstrap gives them, or restored a player reference this capture had lost. Raised by a re-capture of a finished gameweek as well as by a replay; any one of the four on its own raises it |
 | `synthesis_provider_unavailable` | `--summarise` was asked for but the synthesis provider had no usable key; everything else in the recap, the capture included, ran normally |
 
 None of these change the exit code — `league-recap` exits 0 whenever the recap itself

{fplkit-2.3.1 → fplkit-2.3.2}/fpl_cli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '2.3.1'
-__version_tuple__ = version_tuple = (2, 3, 1)
+__version__ = version = '2.3.2'
+__version_tuple__ = version_tuple = (2, 3, 2)
 
 __commit_id__ = commit_id = None

{fplkit-2.3.1 → fplkit-2.3.2}/fpl_cli/cli/_league_recap_history.py

@@ -944,8 +944,8 @@ def _carry_recorded_identity(
             warnings, HISTORY_WARNING_IDENTITY_CARRIED,
             f"League history: GW{gameweek} kept the name, club, position or player "
             f"reference already recorded for {carried} player(s) rather than the ones "
-            f"they carry today. A replay resolves picks against the current bootstrap, "
-            f"which has moved on.",
+            f"they carry today. Picks are resolved against the current bootstrap, "
+            f"which has moved on since the gameweek was played.",
         )
     return carried
 
@@ -1408,6 +1408,17 @@ async def capture_recap_history(
         captured_at=datetime.now(tz=timezone.utc),
         is_live_gw=is_live_gw,
     )
+    # A gameweek whose fixtures have all finished is done moving: its picks,
+    # captain and transfers are fixed, and this run's rows are resolved
+    # against *today's* bootstrap, which may have moved on since (issue #178).
+    # `finished_gameweeks` -- not `is_live_gw` -- is the gate, because a
+    # gameweek stays "current" (and so keeps landing on this path) for the
+    # whole window between its last fixture and the next deadline, and that
+    # window is exactly when a transfer can restamp a correct recorded row.
+    # `_carry_recorded_identity` already no-ops when nothing is recorded yet,
+    # so a genuine first capture is unaffected.
+    if data["gameweek"] in finished_gameweeks:
+        _carry_recorded_identity(store, data["gameweek"], rows, warnings=warnings)
     if fpl_format == "draft" and rows:
         _fill_draft_cumulative_totals(
             store, rows,

{fplkit-2.3.1 → fplkit-2.3.2}/tests/test_cli_league_recap.py

@@ -2914,6 +2914,73 @@ class TestReplayKeepsRecordedIdentity:
         assert resolved[1].transfers[0].player_out_code == 222
 
 
+# ---------------------------------------------------------------------------
+# issue #178: a current-gameweek recapture must not restamp a recorded row
+# ---------------------------------------------------------------------------
+
+
+class TestLiveCaptureKeepsRecordedIdentity:
+    """`capture_recap_history`'s direct write path -- no `replay_gameweek`
+    involved -- is what the ordinary `league-recap` command runs every time.
+    A gameweek stays "current" (and so keeps landing on this path) for the
+    whole window between its last fixture and the next deadline, and FPL
+    keeps processing transfers throughout that window (issue #178)."""
+
+    def _data(self, squad, *, gameweek: int = 1, **manager_kwargs):
+        return _recap_data(
+            gameweek=gameweek,
+            managers=[_manager(name="Alice", entry_id=1, squad=squad, **manager_kwargs)],
+            cohort=_cohort((1, "Alice", 1, 60, 300)),
+        )
+
+    async def test_a_recapture_of_a_finished_gameweek_keeps_the_recorded_club(self):
+        await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="MCI", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[1],
+        )
+        await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="TOT", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[1],
+        )
+        resolved = _store().resolved_gameweek(1)
+        assert resolved[1].squad[0].team == "MCI"
+
+    async def test_a_recapture_reports_the_carry(self):
+        await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="MCI", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[1],
+        )
+        result = await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="TOT", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[1],
+        )
+        assert any(w["code"] == HISTORY_WARNING_IDENTITY_CARRIED for w in result.warnings)
+
+    async def test_a_gameweek_not_yet_finished_is_not_gated_into_carrying(self):
+        """Absent from `finished_gameweeks`, the gameweek is still being
+        played, so its squad can legitimately change between captures -- the
+        carry must not run."""
+        await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="MCI", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[],
+        )
+        await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="TOT", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[],
+        )
+        resolved = _store().resolved_gameweek(1)
+        assert resolved[1].squad[0].team == "TOT"
+
+    async def test_a_first_capture_is_unaffected(self):
+        result = await capture_recap_history(
+            self._data([_player(name="Mover", code=510_281, team="TOT", is_captain=True)]),
+            season=SEASON, finished_gameweeks=[1],
+        )
+        resolved = _store().resolved_gameweek(1)
+        assert resolved[1].squad[0].team == "TOT"
+        assert not any(w["code"] == HISTORY_WARNING_IDENTITY_CARRIED for w in result.warnings)
+
+
 class TestPairSquads:
     """issue #175 review: which recorded entry each replayed one supersedes."""