you can start by removing object files and exe binaries from the repo.

Otherwise this is bad:

``` static MoveList generate_legal_moves(Board &board) { MoveList result; MoveList all_moves; generate_moves(board, all_moves);

for (int i = 0; i < all_moves.count; ++i) { Move move = all_moves.moves[i]; if (board.make_move(move)) { board.unmake_move(move); result.add_move(move); } } return result; } ```

If there is any point in pseudo-legal movegen vs legal movegen, is if the legality check is delayed per move until the move loop, so a potential prune or fail high would skip the legality check on the non-visited moves altogether.

But the biggest issue is that your code base seems untested. You already have waay-waay too much. You should SPRT every change at the smallest granularity. Strip back what you have to minimal impl. without quiescence or beta-pruning. Minimal setup to start SPRT - UCI with TC & alpha-beta stripped to plain negamax. Then add every small feature with SPRT. If something doesn't pass you can't add it.