gammonview 2.0.1__tar.gz → 2.0.2__tar.gz

{gammonview-2.0.1 → gammonview-2.0.2}/CHANGELOG.md

@@ -13,7 +13,18 @@ when the repository was opened; they are kept because they record why things
 are the way they are — particularly the breaking changes and the measurements
 behind several design decisions. Dates are the tag dates.
 
-## 2.0.1 — Unreleased
+## 2.0.2 — 2026-10-12
+
+**A set-up start is written once** *(fix)*
+
+2.0.1's fix for v1 set-up starts went too far. A game whose first ply sets up
+the board *with* dice keeps that ply as a ply, and the writers then also took
+the game's `initial_board` as a second start, so a rewritten file read back
+with an extra set-up ply in front and every later ply one place out. Both
+writers now take `initial_board` as the start only when the game has no
+leading set-up ply of either kind, which is the v1 case 2.0.1 was for.
+
+## 2.0.1 — 2026-10-10
 
 **Plies are counted the way v2 counts them** *(fix)*
 

{gammonview-2.0.1 → gammonview-2.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: gammonview
-Version: 2.0.1
+Version: 2.0.2
 Summary: Backgammon match analysis — the pure-Python OGXM/GVA codec (`gvformat`) plus bgsage-powered match and position analysis (`gvanalysis`).
 Project-URL: Homepage, https://github.com/ngvlamis/gammonview-analysis
 Project-URL: Repository, https://github.com/ngvlamis/gammonview-analysis

{gammonview-2.0.1 → gammonview-2.0.2}/gvformat/ogxm2_writer.py

@@ -466,10 +466,15 @@ def _bare_initial_board(g: dict, opening_abs: list, start: int):
     board in the GAME header), so a v1 file rewritten as v2 would otherwise
     lose its start -- and its plies, replayed from the opening, would be
     flagged illegal. A leading set-up ply, where there is one, wins: a v2 read
-    states the board both ways."""
+    states the board both ways. So does one *with* dice, which `_game_start`
+    keeps as a ply: the document already states its start, and taking
+    ``initial_board`` too would write that start twice."""
     ib = g.get("initial_board")
     if start or not isinstance(ib, list) or len(ib) != 26:
         return None
+    plies = g.get("plies") or []
+    if plies and plies[0].get("action_id") == R.ACTION_SET_POSITION:
+        return None
     board_abs = [int(v) for v in ib]
     return None if board_abs == opening_abs else board_abs
 

{gammonview-2.0.1 → gammonview-2.0.2}/gvformat-js/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gammonview/gvformat",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "type": "module",
   "exports": {
     ".": "./src/index.js",

{gammonview-2.0.1 → gammonview-2.0.2}/gvformat-js/src/ogxm2_writer.js

@@ -592,10 +592,14 @@ function _gameStart(g, openingAbs) {
  *  board in the GAME header), so a v1 file rewritten as v2 would otherwise lose
  *  its start -- and its plies, replayed from the opening, would be flagged
  *  illegal. A leading set-up ply, where there is one, wins: a v2 read states
- *  the board both ways. */
+ *  the board both ways. So does one *with* dice, which `_gameStart` keeps as a
+ *  ply: the document already states its start, and taking `initial_board` too
+ *  would write that start twice. */
 function _bareInitialBoard(g, openingAbs, start) {
   const ib = g.initial_board;
   if (start || !Array.isArray(ib) || ib.length !== 26) return null;
+  const plies = g.plies || [];
+  if (plies.length && plies[0].action_id === ACTION_SET_POSITION) return null;
   const boardAbs = ib.map(_int);
   return boardAbs.every((v, i) => v === openingAbs[i]) ? null : boardAbs;
 }

{gammonview-2.0.1 → gammonview-2.0.2}/gvformat-js/test/test-ogxm2-writer.js

@@ -110,5 +110,18 @@ for (const name of bgfs) {
     "a bare initial_board (a v1 read's set-up start) is written as the game's start");
 }
 
+// ...but not when the game already opens on a set-up ply that carries dice.
+// That ply is kept as a ply, so it states the start, and taking
+// `initial_board` as a second start shifted every ply after it by one.
+{
+  const setup = readGvab(toU8(readFileSync(join(here, 'fixtures', 'ogxm2', 'set-up.ogxm'))));
+  const rolled = structuredClone(setup);
+  Object.assign(rolled.games[1].plies[0], { d1: 3, d2: 1 });
+  const back = readGvab(write_gvab(rolled));
+  assert(back.games[1].plies.length === rolled.games[1].plies.length
+    && back.games[1].plies[0].d1 === 3,
+    'a set-up ply with dice beside initial_board is written once, not twice');
+}
+
 console.log(`\n${passed} passed, ${failed} failed`);
 process.exit(failed ? 1 : 0);