Move makeId to Executor

app/src/main/scala/Executor.scala

@@ -61,7 +61,7 @@ object Executor:
                       val (newState, io) = task.clearAssignedKey match
                         case None =>
                           state - workId -> Logger[IO].warn(
-                            s"Give up move due to invalid move $response of key $key for $task"
+                            s"Give up move due to invalid move $response of $key for $task"
                           )
                         case Some(updated) => state.updated(task.id, updated) -> IO.unit
                       newState -> io *> failure(task, key)
@@ -92,11 +92,13 @@ object Executor:
             )
 
   def fromRequest(req: Lila.Request): IO[Work.Task] =
-    (IO(Work.makeId), IO.realTimeInstant).mapN: (id, now) =>
+    (makeId, IO.realTimeInstant).mapN: (id, now) =>
       Work.Task(
         id = id,
         request = req,
         tries = 0,
         acquired = None,
         createdAt = now
       )
+
+  def makeId: IO[WorkId] = IO(WorkId(scala.util.Random.alphanumeric.take(8).mkString))

app/src/main/scala/Work.scala

@@ -40,5 +40,3 @@ object Work:
       )
     override def toString =
       s"id:$id game:${request.id} variant:${request.variant.key} level:${request.level} tries:$tries created:$createdAt acquired:$acquired move: ${request.moves}"
-
-  def makeId = WorkId(scala.util.Random.alphanumeric.take(8).mkString)