cm-chess 2.4.4 → 2.5.1

package/README.md

@@ -1,6 +1,6 @@
 # cm-chess 
 
-It is like [chess.js](https://github.com/jhlywa/chess.js) (and uses chess.js), 
+It is like [chess.js](https://github.com/jhlywa/chess.js) (and is based on an older version of chess.js), 
 but can handle history variations (tree-structured move history), 
 PGN header values, Nags and Comments.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cm-chess",
-  "version": "2.4.4",
+  "version": "2.5.1",
   "description": "It's like chess.js, but in ES6 and can handle variations (tree-structured move history)",
   "main": "./src/cm-chess/Chess.js",
   "type": "module",
@@ -25,10 +25,10 @@
   "homepage": "https://github.com/shaack/cm-chess#readme",
   "dependencies": {
     "chess.mjs": "^1.3.1",
-    "cm-pgn": "^2.2.2",
+    "cm-pgn": "2.3.2",
     "modlib": "^1.2.12"
   },
   "devDependencies": {
-    "teevi": "^2.2.3"
+    "teevi": "^2.2.4"
   }
 }

package/src/cm-chess/Chess.js

@@ -250,17 +250,14 @@ export class Chess {
     }
 
     /**
-     * This one is not fully implemented in cm-pgn. For now, it just uses pgn() of chess.js.
+     * Render the game as PGN with header, comments and NAGs
+     * @param renderHeader optional, default true
+     * @param renderComments optional, default true
+     * @param renderNags optional, default true
      * @returns {string} the PGN of the game.
      */
-    renderPgn() {
-        // TODO create pgn with variations, annotations, nags (for now just render main variation)
-        const chess = new ChessJs(this.setUpFen())
-        const moves = this.pgn.history.moves
-        for (const move of moves) {
-            chess.move(move)
-        }
-        return chess.pgn()
+    renderPgn(renderHeader = true, renderComments = true, renderNags = true) {
+        return this.pgn.render(renderHeader, renderComments, renderNags);
     }
 
     /**

package/test/TestChess.js

@@ -97,6 +97,7 @@ describe("Chess", function () {
         const chess = new Chess()
         const pgn = `[SetUp "1"]
 [FEN "8/8/b2Bq3/7Q/3kp3/5pP1/8/3K4 w - - 0 1"]
+[Result "1-0"]
 
 1. Qc5+ Kd3 2. Qc2+ Kd4 3. Qd2+ Bd3 4. Qe3+ Kxe3 (4... Kc3 5. Qc1+ Kb3 6. Qa3+ Kc4 7. Qb4+ Kd5 8. Qc5#) 5. Bc5# 
 1-0`
@@ -111,11 +112,12 @@ describe("Chess", function () {
         assert.equal(chess.lastMove().inCheckmate, true)
         assert.equal(chess.inCheckmate(), true)
         assert.equal(chess.inDraw(), false)
-        // TODO result, 1-0 is missing
         assert.equal(chess.renderPgn(), `[SetUp "1"]
 [FEN "8/8/b2Bq3/7Q/3kp3/5pP1/8/3K4 w - - 0 1"]
+[Result "1-0"]
 
-1. Qc5+ Kd3 2. Qc2+ Kd4 3. Qd2+ Bd3 4. Qe3+ Kxe3 5. Bc5#`)
+1. Qc5+ Kd3 2. Qc2+ Kd4 3. Qd2+ Bd3 4. Qe3+ Kxe3 (4... Kc3 5. Qc1+ Kb3 6. Qa3+
+Kc4 7. Qb4+ Kd5 8. Qc5#) 5. Bc5# 1-0`)
     })
 
     it('should add move at the end of the history', () => {