Game code is circular dependant.

[Naviary2]

The game code currently is majorly circular-dependent. Issues this creates:

• The game logic is not modular, we can't extract the logic for legal move calculation, check detection, checkmate algorithm, etc. to use on the server-side, without having to pull out nearly every script in the game, because they are all dependent on each other.

Solution

Gradually reduce the dependancies of each game script, one by one, reformatting them to not depend on so many other scripts.
Theoretically, scripts like math.js, gamefileutility.js, and movesscript.js should have zero or very close to zero dependancies. And main.js would be at the top of the tree, depending on everything else, but nothing depending on main.js!

[Heinrich-XIAO]

This sounds like an enhancement for making unit tests.

[Naviary2]

I'm sharing this dependancy graph generated by Idonotus. The red scripts are all circular dependant.

#222 Would work on converting each game script into an ESM, to help us make them more modular, and allow use of the same scripts on both the server and client-side.

[Naviary2]

Once the new gamefile format is finished, and the server is able to borrow the same logic to perform legal move calculations, that opens the door for #25 , as that feature should wait until the new gamefile format, because games will need to be converted into BigInts instead of integers when people move beyond Number.MAX_SAFE_INTEGER. @Idonotus I'm not sure if you're currently working on a new format or not, I know you had a blueprint, but let's just keep this in mind!

[Heinrich-XIAO]

That looks like a mess, but I've seen worse.

[Naviary2]

Updated dependancy graph. There's still a lot of red, but it does look better!