Protocol in update lsp config

CHANGELOG.md

@@ -19,6 +19,7 @@ Thank you!
 * Fixes an issue in which union members targetting Unit would fail to compile when used as traits (see [#1600](https://github.com/disneystreaming/smithy4s/pull/1600)).
 * Make the `transform` method in generated `*Gen` algebras final. This should make it possible to derive e.g. `FunctorK` instances in cats-tagless automatically (see [#1588](https://github.com/disneystreaming/smithy4s/pull/1588)).
 * Fixes commons.toKebabCase() sometimes drops the first letter (see [#1603](https://github.com/disneystreaming/smithy4s/pull/1603)).
+* Adds `com.disneystreaming.smithy4s:smithy4s-protocol dependency` to the generation of `smithy-build.json` in the `smithy4sUpdateLspConfig` tasks of the codegen plugins (see [#1610](https://github.com/disneystreaming/smithy4s/pull/1610)).
 
 # 0.18.24
 

build.sbt

@@ -407,7 +407,9 @@ lazy val codegen = projectMatrix
       "smithyOrg" -> Dependencies.Smithy.org,
       "smithyVersion" -> Dependencies.Smithy.smithyVersion,
       "alloyOrg" -> Dependencies.Alloy.org,
-      "alloyVersion" -> Dependencies.Alloy.alloyVersion
+      "alloyVersion" -> Dependencies.Alloy.alloyVersion,
+      "smithy4sOrg" -> "com.disneystreaming.smithy4s",
+      "protocolArtifactName" -> "smithy4s-protocol",
     ),
     buildInfoPackage := "smithy4s.codegen",
     libraryDependencies ++= Seq(

modules/codegen-plugin/src/sbt-test/codegen-plugin/update-lsp-config/build.sbt

@@ -8,13 +8,13 @@ val root = project
   .aggregate(subproj, subproj2)
   .settings(
     TaskKey[Unit]("checkSmithyBuild") := {
-      val generated = IO.readLines(file(".") / "smithy-build.json")
-      val expected = IO.readLines(file(".") / "expected.json")
+      val generated = IO.readLines(file(".") / "smithy-build.json").mkString("\n")
+      val expected = IO.readLines(file(".") / "expected.json").mkString("\n").replace("${SMITHY4S_VERSION}", smithy4sVersion.value)
       val compare = s"""|generated:
-                        |${generated.mkString("\n")}
+                        |$generated
                         |===================================
                         |expected:
-                        |${expected.mkString("\n")}
+                        |$expected
                         |===================================
                         |""".stripMargin
 

modules/codegen-plugin/src/sbt-test/codegen-plugin/update-lsp-config/expected.json

@@ -6,7 +6,8 @@
     ],
     "maven" : {
         "dependencies" : [
-            "com.disneystreaming.alloy:alloy-core:0.3.13"
+            "com.disneystreaming.alloy:alloy-core:0.3.13",
+            "com.disneystreaming.smithy4s:smithy4s-protocol:${SMITHY4S_VERSION}"
         ],
         "repositories" : [
             {

modules/codegen/src/smithy4s/codegen/SmithyBuildJson.scala

@@ -25,6 +25,10 @@ import smithy4s.codegen.internals.SmithyBuildMavenRepository
 import scala.collection.immutable.ListSet
 
 private[codegen] object SmithyBuildJson {
+
+  val protocolDependency =
+    s"${BuildInfo.smithy4sOrg}:${BuildInfo.protocolArtifactName}:${BuildInfo.version}"
+
   def toJson(
       sources: ListSet[String],
       dependencies: ListSet[String],
@@ -35,7 +39,7 @@ private[codegen] object SmithyBuildJson {
         version = "1.0",
         sources,
         SmithyBuildMaven(
-          dependencies,
+          dependencies + protocolDependency,
           repositories.map(SmithyBuildMavenRepository.apply)
         )
       )

modules/codegen/test/src/smithy4s/codegen/internals/SmithyBuildSpec.scala

@@ -16,14 +16,14 @@
 
 package smithy4s.codegen.internals
 
-import smithy4s.codegen.SmithyBuildJson
+import smithy4s.codegen.{SmithyBuildJson, BuildInfo}
 import software.amazon.smithy.model.shapes.ShapeId
 import software.amazon.smithy.openapi.OpenApiVersion
 
 import scala.collection.immutable.ListSet
 
 final class SmithyBuildSpec extends munit.FunSuite {
-  test("generate json") {
+  test("generate json with SmithyBuild.writeJson") {
     val actual = SmithyBuild.writeJson(
       SmithyBuild.Serializable(
         "1.0",
@@ -201,4 +201,32 @@ final class SmithyBuildSpec extends munit.FunSuite {
     assertEquals(Set.empty[os.FilePath], actual.sources.toSet)
     assertEquals(Set.empty[SmithyBuildPlugin], actual.plugins.toSet)
   }
+
+  test("generate json with SmithyBuildJson.toJson") {
+    val actual = SmithyBuildJson.toJson(
+      ListSet("src/"),
+      ListSet("dep"),
+      ListSet("repo")
+    )
+    assertEquals(
+      actual,
+      s"""|{
+          |    "version" : "1.0",
+          |    "sources" : [
+          |        "src/"
+          |    ],
+          |    "maven" : {
+          |        "dependencies" : [
+          |            "dep",
+          |            "com.disneystreaming.smithy4s:smithy4s-protocol:${BuildInfo.version}"
+          |        ],
+          |        "repositories" : [
+          |            {
+          |                "url" : "repo"
+          |            }
+          |        ]
+          |    }
+          |}""".stripMargin
+    )
+  }
 }

modules/mill-codegen-plugin/test/src/smithy4s/codegen/mill/SmithyLSPConfigSpec.scala

@@ -67,7 +67,8 @@ class SmithyLSPConfigSpec extends munit.FunSuite {
           |  "maven": {
           |    "dependencies": [
           |       "com.disneystreaming.alloy:alloy-core:${smithy4s.codegen.BuildInfo.alloyVersion}",
-          |       "software.amazon.smithy:smithy-aws-iam-traits:${smithy4s.codegen.BuildInfo.smithyVersion}"
+          |       "software.amazon.smithy:smithy-aws-iam-traits:${smithy4s.codegen.BuildInfo.smithyVersion}",
+          |       "com.disneystreaming.smithy4s:smithy4s-protocol:${smithy4s.codegen.BuildInfo.version}"
           |    ],
           |    "repositories": [
           |       { "url": "https://some.corpo.example.com/artifactory" }