Fix age-old timestamp bug

core/src/main/scala/playground/QueryCompiler.scala

@@ -243,8 +243,11 @@ object QueryCompiler extends SchemaVisitor[PartialCompiler] {
           // the actual serialization format will be used by the client when we eventually use it in the Runner.
           val format = TimestampFormat.DATE_TIME
 
-          Timestamp
-            .parse(s.value, format)
+          TimestampPlatform
+            .fixupTimestamp(
+              Timestamp
+                .parse(s.value, format)
+            )
             .toRightIor(
               CompilationError(
                 InvalidTimestampFormat(format),

core/src/main/scalajs/playground/TimestampPlatform.scala

@@ -0,0 +1,16 @@
+package playground
+
+import smithy4s.Timestamp
+import cats.implicits._
+
+// js
+object TimestampPlatform {
+
+  // Workaround for smithy4s 0.13.x bug
+  def fixupTimestamp(opt: Option[Timestamp]): Option[Timestamp] =
+    opt match {
+      case Some(s) if Either.catchNonFatal(s.toString()).isRight => Some(s)
+      case _                                                     => None
+    }
+
+}

core/src/main/scalajvm/playground/TimestampPlatform.scala

@@ -0,0 +1,6 @@
+package playground
+
+// jvm
+object TimestampPlatform {
+  def fixupTimestamp[T](t: T): T = t
+}

core/src/test/scala/playground/smithyql/CompilationTests.scala

@@ -210,14 +210,11 @@ object CompilationTests extends SimpleIOSuite with Checkers {
 
     // todo: timestamp parsing always passes on JS but does something weird.
     // this is fixed in smithy4s 0.14.0
-    if (Platform.isJVM)
-      assert(
-        result == Ior.leftNec(
-          CompilationErrorDetails.InvalidTimestampFormat(TimestampFormat.DATE_TIME)
-        )
+    assert(
+      result == Ior.leftNec(
+        CompilationErrorDetails.InvalidTimestampFormat(TimestampFormat.DATE_TIME)
       )
-    else
-      assert(result.toEither.isRight)
+    )
   }
 
   pureTest("uuid - OK") {