Tournament clock

core/src/main/scala/Clock.scala

@@ -1,6 +1,5 @@
 package chess
 
-import cats.syntax.option.none
 import chess.Clock.Config
 
 import java.text.DecimalFormat
@@ -207,16 +206,6 @@ object Clock:
       if limitSeconds == 0 then increment.atLeast(Centis(300))
       else limit
 
-  // [TimeControl "600+2"] -> 10+2
-  def readPgnConfig(str: String): Option[Config] =
-    str.split('+') match
-      case Array(initStr, incStr) =>
-        for
-          init <- initStr.toIntOption
-          inc  <- incStr.toIntOption
-        yield Config(init, inc)
-      case _ => none
-
   def apply(limit: LimitSeconds, increment: IncrementSeconds): Clock = apply(Config(limit, increment))
 
   def apply(config: Config): Clock =

core/src/main/scala/TournamentClock.scala

@@ -0,0 +1,29 @@
+package chess
+
+import cats.syntax.all.*
+
+import Clock.{ LimitSeconds, IncrementSeconds }
+
+case class TournamentClock(limitSeconds: LimitSeconds, incrementSeconds: IncrementSeconds):
+
+  def toClockConfig: Option[Clock.Config] =
+    Clock.Config(limitSeconds, incrementSeconds).some
+
+object TournamentClock:
+
+  object parse:
+
+    private val cleanRegex = "(move|minutes|minute|min|m|seconds|second|sec|s|'|\"|/)".r
+
+    private def make(a: Int, b: Int) =
+      val limit = LimitSeconds(if a > 180 then a else a * 60)
+      TournamentClock(limit, IncrementSeconds(b))
+
+    def apply(str: String): Option[TournamentClock] =
+      cleanRegex
+        .replaceAllIn(str.toLowerCase, "")
+        .replace(" ", "")
+        .split('+') match
+        case Array(a)    => a.toIntOption.map(make(_, 0))
+        case Array(a, b) => (a.toIntOption, b.toIntOption).mapN(make)
+        case _           => none

core/src/main/scala/format/pgn/Reader.scala

@@ -45,4 +45,4 @@ object Reader:
 
   private def makeGame(tags: Tags) =
     val g = Game(variantOption = tags.variant, fen = tags.fen)
-    g.copy(startedAtPly = g.ply, clock = tags.clockConfig.map(Clock.apply))
+    g.copy(startedAtPly = g.ply, clock = tags.clockConfig.flatMap(_.toClockConfig).map(Clock.apply))

core/src/main/scala/format/pgn/Tag.scala

@@ -30,10 +30,10 @@ case class Tags(value: List[Tag]) extends AnyVal:
     val name = which(Tag)
     value.find(_.name == name).map(_.value)
 
-  def clockConfig: Option[Clock.Config] =
+  def clockConfig: Option[TournamentClock] =
     value
       .collectFirst { case Tag(Tag.TimeControl, str) => str }
-      .flatMap(Clock.readPgnConfig)
+      .flatMap(TournamentClock.parse.apply)
 
   def variant: Option[chess.variant.Variant] =
     apply(_.Variant)

test-kit/src/test/scala/TournamentClockTest.scala

@@ -0,0 +1,47 @@
+package chess
+
+import Clock.*
+
+class TournamentClockTest extends ChessTest:
+
+  import TournamentClock.parse
+
+  def someClock(minutes: Int, seconds: Int) = Some:
+    TournamentClock(LimitSeconds(minutes * 60), IncrementSeconds(seconds))
+
+  test("parse"):
+    assertEquals(parse(""), None)
+    assertEquals(parse("nope"), None)
+
+    assertEquals(parse("5+5"), someClock(5, 5))
+    assertEquals(parse("5+0"), someClock(5, 0))
+    assertEquals(parse("30+40"), someClock(30, 40))
+
+    assertEquals(parse("15m + 10s"), someClock(15, 10))
+    assertEquals(parse("15 m + 10 s"), someClock(15, 10))
+    assertEquals(parse("15min + 10sec"), someClock(15, 10))
+    assertEquals(parse("15m + 10 sec"), someClock(15, 10))
+    assertEquals(parse("15 min + 10 sec"), someClock(15, 10))
+    assertEquals(parse("15 min + 10 s"), someClock(15, 10))
+    assertEquals(parse("15 minutes + 10 seconds"), someClock(15, 10))
+    assertEquals(parse("  15    MiNUTes+10SECOnds  "), someClock(15, 10))
+
+    assertEquals(parse("15 min + 10 sec / move"), someClock(15, 10))
+    assertEquals(parse("15 min + 10 s / move"), someClock(15, 10))
+    assertEquals(parse("15 min + 10 seconds / move"), someClock(15, 10))
+    assertEquals(parse("15 minutes + 10 seconds / move"), someClock(15, 10))
+
+    assertEquals(parse("90 min + 30 sec / move"), someClock(90, 30))
+
+    assertEquals(parse("120' + 12\""), someClock(120, 12))
+    assertEquals(parse("120' + 12\"/move"), someClock(120, 12))
+    assertEquals(parse("120' + 12\" / move"), someClock(120, 12))
+    assertEquals(parse("7200+12"), someClock(120, 12))
+
+    assertEquals(parse("3600"), someClock(60, 0))
+    assertEquals(parse("60"), someClock(60, 0))
+    assertEquals(parse("180"), someClock(180, 0))
+    assertEquals(parse("240"), someClock(4, 0))
+
+    // we're not there yet
+    // assertEquals(parse("90 min / 40 moves + 30 min + 30 sec / move"), ???)