Andrea Passaglia

https://github.com/laserdisc-io/tamer/blob/8c8c40ff1f17b3735d44349797257600bdf31f15/example/src/main/scala/tamer/rest/RESTCustomAuth.scala#L21 This can result in code like this for custom auth: ``` val authentication: Authentication[Any] = new Authentication[Any] { override def addAuthentication(request: SttpRequest, supplementalSecret: Option[String]): SttpRequest = // ... }...

https://github.com/laserdisc-io/tamer/blob/a1a33f5aac8bba4a76e4294fd63b967182e5b586/core/src/main/scala/tamer/Tamer.scala#L128 A nice enhancement would be to peek into the kafka producer config, get the transaction timeout and issue a human readable warning in case the transaction is still open,...

https://github.com/laserdisc-io/tamer/blob/0a539b788976e932874dc8ce130ed1930f74d975/core/src/main/scala/tamer/Setup.scala#L18 https://github.com/laserdisc-io/tamer/blob/0a539b788976e932874dc8ce130ed1930f74d975/core/src/main/scala/tamer/Setup.scala#L16 Here `TamerError` leaks to the user, a quick and dirty solution would be to widen to a `RIO`

After #540 there are no more checks of the state topic in case of crashes. On one hand the cases are greatly reduced by the introduction of transactionality, on the...

The field below, while logically belongs to core, it's never used by it and only used by the modules. This is just code smell and not a real problem but...