Comparing 7980a044cc..afa685acd2 - NowChessSystems - Gitea

NowChess/NowChessSystems

Author	SHA1	Message	Date
shahdlala66	afa685acd2	fix: removed unneeded file Build & Test (NowChessSystems) TeamCity build finished Details	2026-04-11 11:56:12 +02:00
shahdlala66	2cec27d471	feat: I/O json export import with 100% coverage Build & Test (NowChessSystems) TeamCity build finished Details - Full FEN parser tests - Json exporter with all move types covered - CastleKingside branch coverage Co-Authored-By: Claude Opus 4.6	2026-04-08 21:01:41 +02:00
TeamCity	9d11d25b99	ci: bump version with Build-35	2026-04-08 07:37:40 +00:00
lq64	7a045d31d7	feat: NCS-31 FastParse FEN (#22 ) Build & Test (NowChessSystems) TeamCity build finished Details Summary - Added fastparse_3:3.0.2 dependency to modules/io - Implemented FenParserFastParse as a second alternative FEN parser using FastParse, with the same public API as FenParser and FenParserCombinators - Parsers are built bottom-up using (using P[Any]) Scala 3 syntax with NoWhitespace.* to prevent implicit whitespace skipping; rank sum validation uses Pass/Fail inside .flatMap - Added FenParserFastParseTest mirroring FenParserCombinatorsTest to prove behavioural equivalence across all three implementations Test plan - All existing tests pass — FenParser, FenParserCombinators, and all other modules untouched - FenParserFastParseTest covers all cases: valid FEN, invalid color, invalid castling, invalid board shapes, en passant, rank overflow, round-trip via FenExporter - All parser logic branches genuinely covered — known scoverage gap documented in docs/unresolved.md (FastParse inline macro generates synthetic proxy methods that scoverage instruments but that never execute at runtime) Co-authored-by: LQ63 <lkhermann@web.de> Reviewed-on: #22 Reviewed-by: Janis <janis-e@gmx.de> Co-authored-by: Leon Hermann <lq@blackhole.local> Co-committed-by: Leon Hermann <lq@blackhole.local>	2026-04-08 09:32:57 +02:00
TeamCity	b518c704fa	ci: bump version with Build-34	2026-04-07 19:42:37 +00:00
Janis	fe8e3c0539	fix: NCS-32 Queenside Castle doesn't care about pieces in the way (#23 ) Build & Test (NowChessSystems) TeamCity build finished Details Reviewed-on: #23 Co-authored-by: Janis <janis.e.20@gmx.de> Co-committed-by: Janis <janis.e.20@gmx.de>	2026-04-07 20:32:48 +02:00
TeamCity	1b16adcc72	ci: bump version with Build-33	2026-04-07 18:02:38 +00:00
lq64	b4bc72f7e4	feat: NCS-30 FEN Parser using ParserCombinators (#21 ) Build & Test (NowChessSystems) TeamCity build finished Details Summary - Added scala-parser-combinators_3:2.4.0 dependency to modules/io - Implemented FenParserCombinators as an alternative FEN parser using RegexParsers, with the same public API as the existing FenParser - Parsers are built bottom-up: piece characters → rank tokens → rank → board, composed with explicit field separators into a full FEN parser - Added FenParserCombinatorsTest mirroring the existing FenParserTest to prove behavioural equivalence Test plan - All existing tests pass — FenParser and all other modules untouched - FenParserCombinatorsTest covers all cases: valid FEN, invalid color, invalid castling, invalid board shapes, en passant, round-trip via FenExporter - 100% line/branch/method coverage on FenParserCombinators Co-authored-by: LQ63 <lkhermann@web.de> Reviewed-on: #21 Reviewed-by: Janis <janis-e@gmx.de> Co-authored-by: Leon Hermann <lq@blackhole.local> Co-committed-by: Leon Hermann <lq@blackhole.local>	2026-04-07 12:28:43 +02:00
Janis	8959c3a849	refactor: NCS-24 update CLAUDE.md for improved structure and clarity (#20 ) Build & Test (NowChessSystems) TeamCity build finished Details Reviewed-on: #20 Reviewed-by: Leon Hermann <lq@blackhole.local> Co-authored-by: Janis <janis-e@gmx.de> Co-committed-by: Janis <janis-e@gmx.de>	2026-04-07 12:25:33 +02:00
TeamCity	47032378e2	ci: bump version with Build-31	2026-04-07 06:25:56 +00:00
lq64	217f14f899	refactor: NCS-19 Currying (#18 ) Build & Test (NowChessSystems) TeamCity build finished Details Summary - Curried candidateMoves, legalMoves, and applyMove in the RuleSet trait to separate (context) as the world being operated on from the computation parameter - Updated DefaultRules overrides and all internal call sites - Updated all external call sites: GameEngine, PgnParser, PgnExporter, ChessBoardView, and all affected tests Test plan - All existing tests pass (./gradlew build) - No behaviour changes — pure style refactoring, existing test suite is the regression guard Co-authored-by: LQ63 <lkhermann@web.de> Reviewed-on: #18 Reviewed-by: Janis <janis-e@gmx.de> Co-authored-by: Leon Hermann <lq@blackhole.local> Co-committed-by: Leon Hermann <lq@blackhole.local>	2026-04-06 21:03:17 +02:00

Diff Content Not Available