Fix migration to PSH

Author: PawelLipski

.github/workflows/standard-workflow.yml

@@ -135,7 +135,7 @@ jobs:
         working-directory: examples/akka-persistence-app
 
   # <PEKKO-REMOVE-START>
-  patch-and-push-to-ash:
+  patch-and-push-to-psh:
     if: github.event_name != 'pull_request'
     needs: [code-style-check, test-212, test-213-1, test-213-2, test-sbt-plugin, run-examples]
     runs-on: ubuntu-latest
@@ -145,20 +145,15 @@ jobs:
           # Fetch all history so that a push to pekko-serialization-helper succeeds.
           # See https://stackoverflow.com/q/76470864
           fetch-depth: 0
-      - uses: coursier/cache-action@v6
-      - uses: coursier/setup-action@v1
       - run: git status
-      - run: git checkout -b chore/from-ash
+      - run: git checkout -b TEMP/from-ash
       - run: curl -sL https://raw.githubusercontent.com/ap/rename/master/rename | sudo tee /usr/local/bin/rename
       - run: sudo chmod +x /usr/local/bin/rename
       - run: ./scripts/akka-to-pekko
       - run: git add .
       - run: git config user.email "psh@virtuslab.com"
       - run: git config user.name "Pekko Serialization Helper Bot"
       - run: git commit -m 'Migrate `main` branch from from akka-serialization-helper'
-      - run: git clean -dxf
-      - run: ./scripts/format-code
-      - run: git commit -a --amend --no-edit
       - env:
           # 2.1 Generate an SSH key in terminal (Leave the passphrase empty)
           # 2.2 Add public key in the external repository: <external repository>/Settings/Deploy keys/Add deploy key
@@ -167,7 +162,7 @@ jobs:
           #     Paste the private key.
           PSH_SSH_PRIVATE_KEY: ${{ secrets.PSH_SSH_PRIVATE_KEY }}
         run: mkdir -p ~/.ssh/ && echo "$PSH_SSH_PRIVATE_KEY" > ~/.ssh/psh && chmod 400 ~/.ssh/psh
-      - run: git config --add --local core.sshCommand 'ssh -i ~/.ssh/psh'
+      - run: git config core.sshCommand "ssh -i $HOME/.ssh/psh"
       - run: git push --force git@github.com:VirtusLab/pekko-serialization-helper.git HEAD:main
   # <PEKKO-REMOVE-END>
 

.scala-steward.conf

@@ -12,7 +12,6 @@ buildRoots = [
 ]
 
 # <PEKKO-UNCOMMENT> updates.allow  = [ { groupId = "org.apache.pekko" } ]
-
 # <PEKKO-REMOVE-START>
 updates.pin = [
   # 	* org.scala-lang.modules:scala-xml_2.12:2.1.0 (early-semver) is selected over 1.2.0

.scalafix.conf

@@ -2,12 +2,15 @@ rules = [
   OrganizeImports
 ]
 
-OrganizeImports.expandRelative = true
-OrganizeImports.removeUnused = true
-OrganizeImports.groupedImports = Explode
-OrganizeImports.groups = [
-  "java.",
-  "scala.",
-  "*",
-  "org.virtuslab.ash"
-]
+OrganizeImports {
+  expandRelative = true
+  removeUnused = true
+  groupedImports = Explode
+  groups = [
+    "java.",
+    "scala.",
+    "akka.",
+    "*",
+    "org.virtuslab.ash"
+  ]
+}

.scalafmt.conf

@@ -5,7 +5,7 @@ style = defaultWithAlign
 runner.dialect = scala213
 docstrings.style = Asterisk
 indentOperator.preset      = spray
-maxColumn                  = 120
+maxColumn                  = 128
 rewrite.rules              = [RedundantParens, AvoidInfix]
 align.tokens               = [{code = "=>", owner = "Case"}]
 align.openParenDefnSite    = false

README.md

@@ -1,6 +1,6 @@
 # Akka Serialization Helper
 
-[![Maven Central](https://maven-badges.herokuapp.com/maven-central/org.virtuslab.ash/sbt-akka-serialization-helper/badge.svg)](https://mvnrepository.com/artifact/org.virtuslab.ash)
+[![Maven Central](https://maven-badges.herokuapp.com/maven-central/org.virtuslab.ash/annotation_2.13/badge.svg)](https://mvnrepository.com/artifact/org.virtuslab.ash)
 [![GitHub Actions](https://github.com/VirtusLab/akka-serialization-helper/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/VirtusLab/akka-serialization-helper/actions)
 [![License: MIT](https://img.shields.io/github/license/VirtusLab/akka-serialization-helper)](LICENSE)
 

annotation/src/main/scala/org/virtuslab/ash/annotation/SerializabilityTrait.scala

@@ -1,8 +1,8 @@
 package org.virtuslab.ash.annotation
 
 /**
- * This annotation is used as a marker for Codec Registration Checker and Serializability Checker. Annotate Akka
- * serialization marker-trait, and the rest is done by compiler plugins.
+ * This annotation is used as a marker for Codec Registration Checker and Serializability Checker. Annotate Akka serialization
+ * marker-trait, and the rest is done by compiler plugins.
  *
  * {{{
  *   @SerializabilityTrait

annotation/src/main/scala/org/virtuslab/ash/annotation/Serializer.scala

@@ -5,16 +5,16 @@ import scala.annotation.nowarn
 /**
  * This annotation is used by Codec Registration Checker for marking serializers.
  *
- * Compiler plugin takes the body of a marked class, collects all types that are present, filters them with provided
- * regex and extracts type parameters. Then filtered types and extracted type parameters are checked against all direct
- * subtypes of `clazz`.
+ * Compiler plugin takes the body of a marked class, collects all types that are present, filters them with provided regex and
+ * extracts type parameters. Then filtered types and extracted type parameters are checked against all direct subtypes of
+ * `clazz`.
  *
  * Sometimes types appear unexpectedly during type class derivation leading to false negatives, i.e. a codec reported as
- * registered even though it's NOT registered. The role of `typeRegexPattern` is to filter detected types, ensuring they
- * are used in the right context.
+ * registered even though it's NOT registered. The role of `typeRegexPattern` is to filter detected types, ensuring they are
+ * used in the right context.
  *
- * For example, if we want to serialize `trait Command`, then type `Registration[Command]` is relevant, while
- * `Option[Command]` is not.
+ * For example, if we want to serialize `trait Command`, then type `Registration[Command]` is relevant, while `Option[Command]`
+ * is not.
  *
  * If you are using `circe-akka-serializer`, set `typeRegexPattern` to `Register.REGISTRATION_REGEX`
  *

build.sbt

@@ -32,15 +32,10 @@ ThisBuild / organization := "org.virtuslab.ash"
 ThisBuild / organizationName := "VirtusLab"
 ThisBuild / versionScheme := Some("early-semver")
 ThisBuild / homepage := Some(url("https://github.com/VirtusLab/akka-serialization-helper"))
-ThisBuild / licenses := List(
-  "MIT License" -> url("https://github.com/VirtusLab/akka-serialization-helper/blob/main/LICENSE"))
+ThisBuild / licenses := List("MIT License" -> url("https://github.com/VirtusLab/akka-serialization-helper/blob/main/LICENSE"))
 ThisBuild / developers := List(
   Developer("MarconZet", "Marcin Złakowski", "mzlakowski@virtuslab.com", url("https://github.com/MarconZet")),
-  Developer(
-    "LukaszKontowski",
-    "Łukasz Kontowski",
-    "lkontowski@virtuslab.com",
-    url("https://github.com/LukaszKontowski")),
+  Developer("LukaszKontowski", "Łukasz Kontowski", "lkontowski@virtuslab.com", url("https://github.com/LukaszKontowski")),
   Developer("PawelLipski", "Paweł Lipski", "plipski@virtuslab.com", url("https://github.com/PawelLipski")))
 
 sonatypeProfileName := "org.virtuslab"

circe-akka-serializer/src/main/scala/org/virtuslab/ash/circe/CirceAkkaSerializer.scala

@@ -8,6 +8,7 @@ import scala.reflect.ClassTag
 import akka.actor.ExtendedActorSystem
 import akka.event.Logging
 import akka.serialization.SerializerWithStringManifest
+
 import io.circe._
 import io.circe.jawn.JawnParser
 
@@ -98,19 +99,17 @@ abstract class CirceAkkaSerializer[Ser <: AnyRef: ClassTag](system: ExtendedActo
   /**
    * The intended usage of this method is to provide any form of support for generic classes.
    *
-   * Because of type erasure, it's impossible to [[org.virtuslab.ash.circe.Register]] one generic class two times with
-   * different type parameters.
+   * Because of type erasure, it's impossible to [[org.virtuslab.ash.circe.Register]] one generic class two times with different
+   * type parameters.
    *
-   * The trick for combating type erasure is to register generic class only once with type parameter being its upper
-   * bound, and provide custom made [[io.circe.Codec]] that can serialize/deserialize all classes that are used as a
-   * type parameter.
+   * The trick for combating type erasure is to register generic class only once with type parameter being its upper bound, and
+   * provide custom made [[io.circe.Codec]] that can serialize/deserialize all classes that are used as a type parameter.
    *
-   * For example, if the upper bound is `Any`, but you know that only `Int` and `String` are used as a type parameter,
-   * then you can create a custom [[io.circe.Codec]] for `Any` that handles `Int` and `String` and throws `Exception`
-   * otherwise.
+   * For example, if the upper bound is `Any`, but you know that only `Int` and `String` are used as a type parameter, then you
+   * can create a custom [[io.circe.Codec]] for `Any` that handles `Int` and `String` and throws `Exception` otherwise.
    *
-   * To use this method correctly, set the upper bound for the type parameter of generic class to `Ser` and put the
-   * returned Codec as implicit in a place that can be seen by type derivation.
+   * To use this method correctly, set the upper bound for the type parameter of generic class to `Ser` and put the returned
+   * Codec as implicit in a place that can be seen by type derivation.
    *
    * Example of generic class:
    * {{{
@@ -129,12 +128,7 @@ abstract class CirceAkkaSerializer[Ser <: AnyRef: ClassTag](system: ExtendedActo
   private def logDuration(action: String, obj: AnyRef, startTime: Long, bytes: Array[Byte]): Unit = {
     if (isDebugEnabled) {
       val durationMicros = (System.nanoTime - startTime) / 1000
-      log.debug(
-        "{} of [{}] took [{}] microsecond, size [{}] bytes",
-        action,
-        obj.getClass.getName,
-        durationMicros,
-        bytes.length)
+      log.debug("{} of [{}] took [{}] microsecond, size [{}] bytes", action, obj.getClass.getName, durationMicros, bytes.length)
     }
   }
 

circe-akka-serializer/src/main/scala/org/virtuslab/ash/circe/CirceTraitCodec.scala

@@ -17,22 +17,21 @@ trait CirceTraitCodec[Ser <: AnyRef] extends Codec[Ser] {
   /**
    * Sequence that must contain [[org.virtuslab.ash.circe.Registration]] for all direct subclasses of Ser.
    *
-   * Each `Registration` is created using [[org.virtuslab.ash.circe.Register]]s
-   * [[org.virtuslab.ash.circe.Register#apply]] method.
+   * Each `Registration` is created using [[org.virtuslab.ash.circe.Register]]s [[org.virtuslab.ash.circe.Register#apply]]
+   * method.
    *
    * To check if all needed classes are registered, use Codec Registration Checker.
    *
    * @see
-   *   [[org.virtuslab.ash.circe.Register]][[org.virtuslab.ash.circe.Register#apply]] for more information about type
-   *   derivation
+   *   [[org.virtuslab.ash.circe.Register]][[org.virtuslab.ash.circe.Register#apply]] for more information about type derivation
    */
   val codecs: Seq[Registration[_ <: Ser]]
 
   /**
    * A sequence containing information used in type migration.
    *
-   * If you ever change the name of a class that is a direct descendant of `Ser` and is persisted in any way, you must
-   * append new pair to this field.
+   * If you ever change the name of a class that is a direct descendant of `Ser` and is persisted in any way, you must append
+   * new pair to this field.
    *   - The first element of the pair is a String with the value of old FQCN.
    *   - The second element of the pair is a Class that had its name changed
    *