pgn2 0.4.0 → 1.1.0

data/spec/serializer_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+describe PGN::Serializer do
+  describe '#to_s' do
+    it 'serializes a simple game with tags and result' do
+      game = PGN::Game.new(%w[e4 e5], { 'White' => 'A', 'Black' => 'B' }, '1-0')
+      expected = "[White \"A\"]\n[Black \"B\"]\n\n1. e4 e5 1-0\n"
+      expect(PGN::Serializer.new(game).to_s).to eq(expected)
+    end
+
+    it 'synthesizes a Result tag when there are no tags' do
+      game = PGN::Game.new(%w[e4 e5])
+      expected = "[Result \"*\"]\n\n1. e4 e5 *\n"
+      expect(PGN::Serializer.new(game).to_s).to eq(expected)
+    end
+
+    it 'serializes an empty game as just the result' do
+      game = PGN::Game.new([], nil, '*')
+      expected = "[Result \"*\"]\n\n*\n"
+      expect(PGN::Serializer.new(game).to_s).to eq(expected)
+    end
+
+    it 'emits * when there is no result' do
+      game = PGN::Game.new(%w[e4 e5], { 'White' => 'A' })
+      expect(PGN::Serializer.new(game).to_s).to end_with("1. e4 e5 *\n")
+    end
+
+    it 'escapes backslashes and quotes in tag values' do
+      game = PGN::Game.new(%w[e4], { 'White' => 'A "B" \\C' }, '1-0')
+      expect(PGN::Serializer.new(game).to_s).to start_with("[White \"A \\\"B\\\" \\\\C\"]\n")
+    end
+
+    it 'serializes castling as O-O / O-O-O' do
+      game = PGN::Game.new(%w[O-O O-O-O])
+      expect(PGN::Serializer.new(game).to_s).to eq("[Result \"*\"]\n\n1. O-O O-O-O *\n")
+    end
+
+    it 'emits a single annotation after the notation' do
+      moves = [PGN::MoveText.new('e4', ['$4'])]
+      game = PGN::Game.new(moves, { 'White' => 'A' }, '1-0')
+      expect(PGN::Serializer.new(game).to_s).to eq("[White \"A\"]\n\n1. e4 $4 1-0\n")
+    end
+
+    it 'emits symbolic annotations verbatim' do
+      moves = [PGN::MoveText.new('e4', ['??'])]
+      game = PGN::Game.new(moves, { 'White' => 'A' }, '1-0')
+      expect(PGN::Serializer.new(game).to_s).to eq("[White \"A\"]\n\n1. e4 ?? 1-0\n")
+    end
+
+    it 'emits multiple annotations in order' do
+      moves = [PGN::MoveText.new('d4'), PGN::MoveText.new('d5', ['$2', '$11'])]
+      game = PGN::Game.new(moves, { 'White' => 'A' }, '1/2-1/2')
+      expect(PGN::Serializer.new(game).to_s).to eq("[White \"A\"]\n\n1. d4 d5 $2 $11 1/2-1/2\n")
+    end
+
+    it 'wraps move comments in braces with spaces' do
+      moves = [PGN::MoveText.new('e4', nil, 'good move')]
+      game = PGN::Game.new(moves, { 'White' => 'A' }, '1-0')
+      expect(PGN::Serializer.new(game).to_s).to eq("[White \"A\"]\n\n1. e4 { good move } 1-0\n")
+    end
+
+    it 'reproduces the variations.pgn movetext shape, including 2... after variations' do
+      game = PGN.parse(File.read('./spec/pgn_files/variations.pgn')).first
+      expected = "[Black \"Petrov\"]\n[White \"Somebody\"]\n\n" \
+                "1. e4 e5 2. Nf3 { comment } " \
+                "(2. f4 exf4 { final variation }) " \
+                "(2. Nc3 { other } 2... d5 (2... f5) 3. exd5) " \
+                "2... Nf6 *\n"
+      expect(PGN::Serializer.new(game).to_s).to eq(expected)
+    end
+
+    it 'emits a game comment as the first movetext token' do
+      game = PGN::Game.new([], nil, '*', nil, 'game comment')
+      expect(PGN::Serializer.new(game).to_s).to eq("[Result \"*\"]\n\n{ game comment } *\n")
+    end
+
+    it 'numbers the first move 1... when starting from a FEN with black to move' do
+      fen = 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR b KQkq - 0 1'
+      game = PGN::Game.new(%w[e5], { 'FEN' => fen }, '*')
+      expected = "[FEN \"#{fen}\"]\n\n1... e5 *\n"
+      expect(PGN::Serializer.new(game).to_s).to eq(expected)
+    end
+
+    it 'serializes -- moves verbatim and alternates color/fullmove' do
+      game = PGN::Game.new(%w[-- e4])
+      expect(PGN::Serializer.new(game).to_s).to eq("[Result \"*\"]\n\n1. -- e4 *\n")
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -7,7 +7,6 @@ require 'pgn'
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true
   config.filter_run :focus
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pgn2
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Stacey Touset
@@ -9,16 +9,30 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-04-01 00:00:00.000000000 Z
+date: 2026-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: whittle
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.3'
+- !ruby/object:Gem::Dependency
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -26,21 +40,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -54,7 +68,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: racc
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -68,7 +82,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: benchmark-ips
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,6 +95,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: memory_profiler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.60'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.60'
 description: A PGN parser and FEN generator for Ruby
 email:
 - stacey@touset.org
@@ -89,45 +131,83 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ci.yml"
+- ".github/workflows/publish.yml"
+- ".github/workflows/release.yml"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - TODO.md
+- bench/.keep
+- bench/IMPROVEMENTS.md
+- bench/baseline_moves.pre-optimization.txt
+- bench/baseline_moves.pre-quickwins.txt
+- bench/baseline_moves.txt
+- bench/baseline_parse.pre-optimization.txt
+- bench/baseline_parse.pre-quickwins.txt
+- bench/baseline_parse.racc.txt
+- bench/baseline_parse.txt
+- bench/profile_moves.rb
+- bench/profile_parse.rb
+- docs/superpowers/plans/2026-08-12-efficiency-optimizations.md
+- docs/superpowers/plans/2026-08-12-efficiency-tests-and-profiling.md
+- docs/superpowers/plans/2026-08-12-to-pgn-serialization.md
+- docs/superpowers/plans/2026-08-13-whittle-to-racc-migration.md
+- docs/superpowers/specs/2026-08-12-to-pgn-serialization-design.md
+- docs/superpowers/specs/2026-08-13-pgn-performance-quick-wins-design.md
 - examples/immortal_game.pgn
 - lib/pgn.rb
 - lib/pgn/board.rb
 - lib/pgn/fen.rb
 - lib/pgn/game.rb
+- lib/pgn/lexer.rb
 - lib/pgn/move.rb
 - lib/pgn/move_calculator.rb
 - lib/pgn/parser.rb
+- lib/pgn/pgn_parser.rb
+- lib/pgn/pgn_parser.y
 - lib/pgn/position.rb
+- lib/pgn/serializer.rb
 - lib/pgn/version.rb
 - pgn2.gemspec
+- spec/board_spec.rb
 - spec/fen_spec.rb
 - spec/game_spec.rb
+- spec/lexer_spec.rb
+- spec/move_calculator_spec.rb
+- spec/move_spec.rb
+- spec/parser_explicit_spec.rb
 - spec/parser_spec.rb
 - spec/pgn_files/alternate_castling.pgn
 - spec/pgn_files/annotations.pgn
 - spec/pgn_files/comments.pgn
+- spec/pgn_files/doublequotes.pgn
 - spec/pgn_files/empty_variation_move.pgn
 - spec/pgn_files/fen.pgn
 - spec/pgn_files/multiline_comments.pgn
 - spec/pgn_files/nested_comments.pgn
 - spec/pgn_files/no_moves.pgn
+- spec/pgn_files/specialcharacters.pgn
 - spec/pgn_files/test.pgn
 - spec/pgn_files/two_annotations.pgn
 - spec/pgn_files/two_games.pgn
 - spec/pgn_files/variations.pgn
 - spec/position_spec.rb
+- spec/serializer_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/muriloime/pgn
 licenses:
 - MIT
-metadata: {}
+metadata:
+  homepage_uri: https://github.com/muriloime/pgn
+  source_code_uri: https://github.com/muriloime/pgn
+  bug_tracker_uri: https://github.com/muriloime/pgn/issues
+  changelog_uri: https://github.com/muriloime/pgn/blob/main/CHANGELOG.md
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -136,32 +216,40 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: A PGN parser for Ruby
 test_files:
+- spec/board_spec.rb
 - spec/fen_spec.rb
 - spec/game_spec.rb
+- spec/lexer_spec.rb
+- spec/move_calculator_spec.rb
+- spec/move_spec.rb
+- spec/parser_explicit_spec.rb
 - spec/parser_spec.rb
 - spec/pgn_files/alternate_castling.pgn
 - spec/pgn_files/annotations.pgn
 - spec/pgn_files/comments.pgn
+- spec/pgn_files/doublequotes.pgn
 - spec/pgn_files/empty_variation_move.pgn
 - spec/pgn_files/fen.pgn
 - spec/pgn_files/multiline_comments.pgn
 - spec/pgn_files/nested_comments.pgn
 - spec/pgn_files/no_moves.pgn
+- spec/pgn_files/specialcharacters.pgn
 - spec/pgn_files/test.pgn
 - spec/pgn_files/two_annotations.pgn
 - spec/pgn_files/two_games.pgn
 - spec/pgn_files/variations.pgn
 - spec/position_spec.rb
+- spec/serializer_spec.rb
 - spec/spec_helper.rb