Skip to content

fix: lichess bugs#183

Merged
thomas-mauran merged 3 commits intomainfrom
fix/lichess-bugs
Dec 29, 2025
Merged

fix: lichess bugs#183
thomas-mauran merged 3 commits intomainfrom
fix/lichess-bugs

Conversation

@thomas-mauran
Copy link
Owner

Fix bugs and improve lichess code

Description

The goal here is to enable polling back on both side to make sure when we play on the same account online and in chess-tui at the same time we sync it properly.
We also make sure we fetch the game state when we join an ongoing game by code

Fixes #177 #178

How Has This Been Tested?

Manual testing

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Additional changes

  • refactored some of the code to remove useless fallbacks
  • removed unused methods
  • fetch the last move when joining the game so we dont wait for the first poll

Signed-off-by: Thomas Mauran <thomasmauran@yahoo.com>
Signed-off-by: Thomas Mauran <thomasmauran@yahoo.com>
@thomas-mauran thomas-mauran self-assigned this Dec 19, 2025
@thomas-mauran thomas-mauran added the bug Something isn't working label Dec 19, 2025
Signed-off-by: Thomas Mauran <thomasmauran@yahoo.com>
@thomas-mauran
Copy link
Owner Author

Hello @craftmaster1231 do you want to check this branch before I merge ? I tried it on different setup seems to patch the issue !

@thomas-mauran thomas-mauran merged commit 97ae599 into main Dec 29, 2025
17 checks passed
@thomas-mauran thomas-mauran deleted the fix/lichess-bugs branch December 29, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Polling on only opponent move is problematic

1 participant