Add sources and javadoc jars to publication, Fix staging directory

build.gradle.kts

@@ -66,7 +66,8 @@ jreleaser {
                     snapshotUrl = "https://aws.oss.sonatype.org/content/repositories/snapshots"
                     closeRepository.set(true)
                     releaseRepository.set(true)
-                    stagingRepositories.add("${rootProject.buildDir}/staging")
+                    verifyPom.set(true)
+                    stagingRepository(rootProject.layout.buildDirectory.dir("staging").get().asFile.path)
                 }
             }
         }

buildSrc/src/main/kotlin/smithy-java.module-conventions.gradle.kts

@@ -25,10 +25,9 @@ val licenseSpec = copySpec {
  * Extra Jars
  * ============================
  */
-// Build a javadoc JAR too.
-tasks.register<Jar>("javadocJar") {
-    from(tasks.named("javadoc"))
-    archiveClassifier.set("javadoc")
+java {
+    withJavadocJar()
+    withSourcesJar()
 }
 
 // TODO: Remove this once package is ready for docs

buildSrc/src/main/kotlin/smithy-java.publishing-conventions.gradle.kts

@@ -17,7 +17,7 @@ publishing {
     repositories {
         maven {
             name = "stagingRepository"
-            url = uri("${rootProject.layout.buildDirectory}/staging")
+            url = rootProject.layout.buildDirectory.dir("staging").get().asFile.toURI()
         }
     }
     // Add license spec to all maven publications

framework-errors/build.gradle.kts

@@ -32,10 +32,21 @@ afterEvaluate {
     }
 }
 
+// TODO: This is needed because the type-mapping file is mapped (twice) in the source-set above and source-set below,
+//  but really we shouldn't be doing that
+tasks.withType<Jar> {
+    duplicatesStrategy = DuplicatesStrategy.INCLUDE
+}
+
 tasks.named("compileJava") {
     dependsOn("smithyBuild")
 }
 
+// Needed because sources-jar needs to run after smithy-build is done
+tasks.named("sourcesJar") {
+    mustRunAfter("compileJava")
+}
+
 // Helps Intellij plugin identify models
 sourceSets {
     main {