Remove old service builder machinery (#2161)

* Remove HandlerGenerator and RegistryGenerator * Shrink RequestRejection * Remove inlineable * Remove Router and RequestParts * Remove old service builder tests * Add missing test dependency * Add missing dev dependency * Remove unused test * Move Router types * Switch AllowUnused to AllowUnusedVariables
smithy-lang · Jan 20, 2023 · 4436d9a · 4436d9a
1 parent a559048
commit 4436d9a
Show file tree

Hide file tree

Showing 33 changed files with 478 additions and 2,188 deletions.
diff --git a/...in/software/amazon/smithy/rust/codegen/server/python/smithy/PythonServerCodegenVisitor.kt b/...in/software/amazon/smithy/rust/codegen/server/python/smithy/PythonServerCodegenVisitor.kt
@@ -10,16 +10,19 @@ import software.amazon.smithy.build.PluginContext
 import software.amazon.smithy.codegen.core.CodegenException
 import software.amazon.smithy.model.Model
 import software.amazon.smithy.model.knowledge.NullableIndex
+import software.amazon.smithy.model.shapes.OperationShape
 import software.amazon.smithy.model.shapes.ServiceShape
 import software.amazon.smithy.model.shapes.StringShape
 import software.amazon.smithy.model.shapes.StructureShape
 import software.amazon.smithy.model.shapes.UnionShape
 import software.amazon.smithy.model.traits.EnumTrait
+import software.amazon.smithy.rust.codegen.core.rustlang.RustModule
 import software.amazon.smithy.rust.codegen.core.rustlang.RustWriter
 import software.amazon.smithy.rust.codegen.core.smithy.CodegenTarget
 import software.amazon.smithy.rust.codegen.core.smithy.RustCrate
 import software.amazon.smithy.rust.codegen.core.smithy.SymbolVisitorConfig
 import software.amazon.smithy.rust.codegen.server.python.smithy.generators.PythonServerEnumGenerator
+import software.amazon.smithy.rust.codegen.server.python.smithy.generators.PythonServerOperationHandlerGenerator
 import software.amazon.smithy.rust.codegen.server.python.smithy.generators.PythonServerServiceGenerator
 import software.amazon.smithy.rust.codegen.server.python.smithy.generators.PythonServerStructureGenerator
 import software.amazon.smithy.rust.codegen.server.smithy.ServerCodegenContext
@@ -164,4 +167,11 @@ class PythonServerCodegenVisitor(
         )
             .render()
     }
+
+    override fun operationShape(shape: OperationShape) {
+        super.operationShape(shape)
+        rustCrate.withModule(RustModule.public("python_operation_adaptor")) {
+            PythonServerOperationHandlerGenerator(codegenContext, shape).render(this)
+        }
+    }
 }
diff --git a/.../amazon/smithy/rust/codegen/server/python/smithy/generators/PythonApplicationGenerator.kt b/.../amazon/smithy/rust/codegen/server/python/smithy/generators/PythonApplicationGenerator.kt
@@ -199,7 +199,7 @@ class PythonApplicationGenerator(
                         let ${name}_locals = #{pyo3_asyncio}::TaskLocals::new(event_loop);
                         let handler = self.handlers.get("$name").expect("Python handler for operation `$name` not found").clone();
                         let builder = builder.$name(move |input, state| {
-                            #{pyo3_asyncio}::tokio::scope(${name}_locals.clone(), crate::operation_handler::$name(input, state, handler.clone()))
+                            #{pyo3_asyncio}::tokio::scope(${name}_locals.clone(), crate::python_operation_adaptor::$name(input, state, handler.clone()))
                         });
                         """,
                         *codegenScope,

diff --git a/...thy/rust/codegen/server/python/smithy/generators/PythonServerOperationHandlerGenerator.kt b/...thy/rust/codegen/server/python/smithy/generators/PythonServerOperationHandlerGenerator.kt
@@ -13,8 +13,6 @@ import software.amazon.smithy.rust.codegen.core.rustlang.writable
 import software.amazon.smithy.rust.codegen.core.smithy.CodegenContext
 import software.amazon.smithy.rust.codegen.core.util.toSnakeCase
 import software.amazon.smithy.rust.codegen.server.python.smithy.PythonServerCargoDependency
-import software.amazon.smithy.rust.codegen.server.smithy.generators.ServerOperationHandlerGenerator
-import software.amazon.smithy.rust.codegen.server.smithy.generators.protocol.ServerProtocol
 
 /**
  * The Rust code responsible to run the Python business logic on the Python interpreter
@@ -32,9 +30,8 @@ import software.amazon.smithy.rust.codegen.server.smithy.generators.protocol.Ser
  */
 class PythonServerOperationHandlerGenerator(
     codegenContext: CodegenContext,
-    protocol: ServerProtocol,
-    private val operations: List<OperationShape>,
-) : ServerOperationHandlerGenerator(codegenContext, protocol, operations) {
+    private val operation: OperationShape,
+) {
     private val symbolProvider = codegenContext.symbolProvider
     private val runtimeConfig = codegenContext.runtimeConfig
     private val codegenScope =
@@ -47,42 +44,39 @@ class PythonServerOperationHandlerGenerator(
             "tracing" to PythonServerCargoDependency.Tracing.toType(),
         )
 
-    override fun render(writer: RustWriter) {
-        super.render(writer)
+    fun render(writer: RustWriter) {
         renderPythonOperationHandlerImpl(writer)
     }
 
     private fun renderPythonOperationHandlerImpl(writer: RustWriter) {
-        for (operation in operations) {
-            val operationName = symbolProvider.toSymbol(operation).name
-            val input = "crate::input::${operationName}Input"
-            val output = "crate::output::${operationName}Output"
-            val error = "crate::error::${operationName}Error"
-            val fnName = operationName.toSnakeCase()
+        val operationName = symbolProvider.toSymbol(operation).name
+        val input = "crate::input::${operationName}Input"
+        val output = "crate::output::${operationName}Output"
+        val error = "crate::error::${operationName}Error"
+        val fnName = operationName.toSnakeCase()
 
-            writer.rustTemplate(
-                """
-                /// Python handler for operation `$operationName`.
-                pub(crate) async fn $fnName(
-                    input: $input,
-                    state: #{SmithyServer}::Extension<#{SmithyPython}::context::PyContext>,
-                    handler: #{SmithyPython}::PyHandler,
-                ) -> std::result::Result<$output, $error> {
-                    // Async block used to run the handler and catch any Python error.
-                    let result = if handler.is_coroutine {
-                        #{PyCoroutine:W}
-                    } else {
-                        #{PyFunction:W}
-                    };
-                    #{PyError:W}
-                }
-                """,
-                *codegenScope,
-                "PyCoroutine" to renderPyCoroutine(fnName, output),
-                "PyFunction" to renderPyFunction(fnName, output),
-                "PyError" to renderPyError(),
-            )
-        }
+        writer.rustTemplate(
+            """
+            /// Python handler for operation `$operationName`.
+            pub(crate) async fn $fnName(
+                input: $input,
+                state: #{SmithyServer}::Extension<#{SmithyPython}::context::PyContext>,
+                handler: #{SmithyPython}::PyHandler,
+            ) -> std::result::Result<$output, $error> {
+                // Async block used to run the handler and catch any Python error.
+                let result = if handler.is_coroutine {
+                    #{PyCoroutine:W}
+                } else {
+                    #{PyFunction:W}
+                };
+                #{PyError:W}
+            }
+            """,
+            *codegenScope,
+            "PyCoroutine" to renderPyCoroutine(fnName, output),
+            "PyFunction" to renderPyFunction(fnName, output),
+            "PyError" to renderPyError(),
+        )
     }
 
     private fun renderPyFunction(name: String, output: String): Writable =

diff --git a/...mazon/smithy/rust/codegen/server/python/smithy/generators/PythonServerServiceGenerator.kt b/...mazon/smithy/rust/codegen/server/python/smithy/generators/PythonServerServiceGenerator.kt
@@ -33,10 +33,6 @@ class PythonServerServiceGenerator(
         PythonServerOperationErrorGenerator(context.model, context.symbolProvider, operation).render(writer)
     }
 
-    override fun renderOperationHandler(writer: RustWriter, operations: List<OperationShape>) {
-        PythonServerOperationHandlerGenerator(context, protocol, operations).render(writer)
-    }
-
     override fun renderExtras(operations: List<OperationShape>) {
         rustCrate.withModule(RustModule.public("python_server_application", "Python server and application implementation.")) {
             PythonApplicationGenerator(context, protocol, operations)

diff --git a/...rc/main/kotlin/software/amazon/smithy/rust/codegen/server/smithy/ServerCargoDependency.kt b/...rc/main/kotlin/software/amazon/smithy/rust/codegen/server/smithy/ServerCargoDependency.kt
@@ -7,8 +7,6 @@ package software.amazon.smithy.rust.codegen.server.smithy
 import software.amazon.smithy.rust.codegen.core.rustlang.CargoDependency
 import software.amazon.smithy.rust.codegen.core.rustlang.CratesIo
 import software.amazon.smithy.rust.codegen.core.rustlang.DependencyScope
-import software.amazon.smithy.rust.codegen.core.rustlang.InlineDependency
-import software.amazon.smithy.rust.codegen.core.rustlang.RustModule
 import software.amazon.smithy.rust.codegen.core.smithy.RuntimeConfig
 
 /**
@@ -27,32 +25,8 @@ object ServerCargoDependency {
     val Tower: CargoDependency = CargoDependency("tower", CratesIo("0.4"))
     val TokioDev: CargoDependency = CargoDependency("tokio", CratesIo("1.8.4"), scope = DependencyScope.Dev)
     val Regex: CargoDependency = CargoDependency("regex", CratesIo("1.5.5"))
+    val HyperDev: CargoDependency = CargoDependency("hyper", CratesIo("0.14.12"), DependencyScope.Dev)
 
     fun smithyHttpServer(runtimeConfig: RuntimeConfig) = runtimeConfig.smithyRuntimeCrate("smithy-http-server")
     fun smithyTypes(runtimeConfig: RuntimeConfig) = runtimeConfig.smithyRuntimeCrate("smithy-types")
 }
-
-/**
- * A dependency on a snippet of code
- *
- * ServerInlineDependency should not be instantiated directly, rather, it should be constructed with
- * [software.amazon.smithy.rust.codegen.core.smithy.RuntimeType.forInlineFun]
- *
- * ServerInlineDependencies are created as private modules within the main crate. This is useful for any code that
- * doesn't need to exist in a shared crate, but must still be generated exactly once during codegen.
- *
- * CodegenVisitor de-duplicates inline dependencies by (module, name) during code generation.
- */
-object ServerInlineDependency {
-    fun serverOperationHandler(runtimeConfig: RuntimeConfig): InlineDependency =
-        InlineDependency.forRustFile(
-            RustModule.private("server_operation_handler_trait"),
-            "/inlineable/src/server_operation_handler_trait.rs",
-            ServerCargoDependency.smithyHttpServer(runtimeConfig),
-            CargoDependency.Http,
-            ServerCargoDependency.PinProjectLite,
-            ServerCargoDependency.Tower,
-            ServerCargoDependency.FuturesUtil,
-            ServerCargoDependency.AsyncTrait,
-        )
-}
diff --git a/...er/src/main/kotlin/software/amazon/smithy/rust/codegen/server/smithy/ServerRuntimeType.kt b/...er/src/main/kotlin/software/amazon/smithy/rust/codegen/server/smithy/ServerRuntimeType.kt
@@ -19,9 +19,6 @@ object ServerRuntimeType {
 
     fun router(runtimeConfig: RuntimeConfig) = ServerCargoDependency.smithyHttpServer(runtimeConfig).toType().resolve("routing::Router")
 
-    fun operationHandler(runtimeConfig: RuntimeConfig) =
-        forInlineDependency(ServerInlineDependency.serverOperationHandler(runtimeConfig))
-
     fun runtimeError(runtimeConfig: RuntimeConfig) = ServerCargoDependency.smithyHttpServer(runtimeConfig).toType().resolve("runtime_error::RuntimeError")
 
     fun requestRejection(runtimeConfig: RuntimeConfig) = ServerCargoDependency.smithyHttpServer(runtimeConfig).toType().resolve("rejection::RequestRejection")

diff --git a/...re/amazon/smithy/rust/codegen/server/smithy/generators/ServerOperationHandlerGenerator.kt b/...re/amazon/smithy/rust/codegen/server/smithy/generators/ServerOperationHandlerGenerator.kt