Merged in bugfix/EL-364-serialization-issues-with-non-tri (pull request #184)

EL-364: Added fixes for the serialization and archetyupe.

Approved-by: Keith Hudnall

Author: patrickt

deployment-jetty/src/main/java/dev/getelements/elements/deployment/jetty/loader/JakartaRsLoader.java

@@ -13,6 +13,7 @@
 import org.eclipse.jetty.ee10.servlet.ServletContextHandler;
 import org.eclipse.jetty.ee10.servlet.ServletHolder;
 import org.eclipse.jetty.server.Handler.Sequence;
+import org.glassfish.jersey.server.ServerProperties;
 import org.glassfish.jersey.servlet.ServletContainer;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -28,6 +29,7 @@
 import static dev.getelements.elements.sdk.model.Constants.APP_OUTSIDE_URL;
 import static org.glassfish.jersey.CommonProperties.MOXY_JSON_FEATURE_DISABLE;
 import static org.glassfish.jersey.server.ResourceConfig.forApplication;
+import static org.glassfish.jersey.server.ServerProperties.FEATURE_AUTO_DISCOVERY_DISABLE;
 
 /**
  * Searches an {@link Element} for a service of type {@link Application} and loads it into the application container
@@ -109,6 +111,13 @@ private JettyDeploymentRecord deploy(final PendingDeployment pending,
             config.property(MOXY_JSON_FEATURE_DISABLE, true);
         }
 
+        if (!config.hasProperty(FEATURE_AUTO_DISCOVERY_DISABLE)) {
+            // We know this interferes with Kotlin based elements or any Element which tries to include its own
+            // jackson provider. So we disable support for this by default. Even in non-kotlin projects this should
+            // be avoided because we end up polluting the Element's classpath with services it may not want to use.
+            config.property(FEATURE_AUTO_DISCOVERY_DISABLE, true);
+        }
+
         final var container = new ServletContainer(config);
         final var holder = new ServletHolder(container);
 

sdk-element-standard-kt/src/main/resources/archetype-resources/element/pom.xml

@@ -185,6 +185,19 @@
                             <prependGroupId>true</prependGroupId>
                         </configuration>
                     </execution>
+                    <!-- Kotlin Specific. The API must have the Kotlin libraries included -->
+                    <execution>
+                        <id>elm-copy-kotlin-stdlib-api</id>
+                        <phase>prepare-package</phase>
+                        <goals>
+                            <goal>copy-dependencies</goal>
+                        </goals>
+                        <configuration>
+                            <outputDirectory>${elm.element.dir}/api</outputDirectory>
+                            <includeGroupIds>org.jetbrains.kotlin</includeGroupIds>
+                            <prependGroupId>true</prependGroupId>
+                        </configuration>
+                    </execution>
                 </executions>
             </plugin>