Gavin
Gavin
Well I think it's probably more common for people to want auto claiming of draws on threefold repetition or 50-move rule anyway, so still makes sense for that to be...
Added a couple optimisations to this PR, but let me know if you would rather they be a separate PR instead.
Fixed a bug with the new `_getRepetitionCount` implementation: it wasn't adding positions to the record when loading from PGN (sorry ignore the commit name when I said FEN, I meant...
Using `put`/`remove` already break a lot of functionality. Because they are not in the `_history` but still affect `_board`: `pgn`, `history` and `undo` methods all potentially produce non-sense output when...
> As for the API changes to isDraw and isGameOver, would it be better to modify the Chess constructor to take an optional strictFideAdherence flag (or something similar)? I don't...
Latest commits show how `_history` could be updated to include `put`/`remove`s, including updates to `pgn` and `loadPgn` methods to they can output and read from pgns with `put`/`remove`s commented like...
Unfortunately, to actually address this bug would require either a PR with [`chess.js`](https://github.com/jhlywa/chess.js/) or some rather large changes to the way history is handled in this project. Currently the `.history`...
Actually, I suppose there is a third approach: a hacky edgecase check bugfix. When initialising a board, we could check whether the first move is a check/checkmate by creating a...
This is quite annoying. Indeed, the obvious solution is simply to have the alias take precedence over the command when set, ie. it should be displayed as `${alias)} ${command}` rather...