Skip to content

Gate PGN logging and add test coverage#14

Draft
RainRat wants to merge 1 commit intomasterfrom
codex/remove-stdout-output-from-read_game_pgn
Draft

Gate PGN logging and add test coverage#14
RainRat wants to merge 1 commit intomasterfrom
codex/remove-stdout-output-from-read_game_pgn

Conversation

@RainRat
Copy link
Owner

@RainRat RainRat commented Sep 20, 2025

Summary

  • gate read_game_pgn SAN logging behind an opt-in flag and send diagnostic output to std::cerr
  • export a setter so callers can enable the diagnostic logging when needed
  • add a mocha test that asserts readGamePGN does not write to stdout for a sample PGN

Testing

  • npm test (fails: Cannot find module './ffish.js')

https://chatgpt.com/codex/tasks/task_e_68ce42df0a3083308d1375a54db23579

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant