refactor: code cleanup

Author: vicb

packages/open-next/src/build/edge/createEdgeBundle.ts

@@ -21,16 +21,15 @@ import { openNextResolvePlugin } from "../../plugins/resolve.js";
 import { type BuildOptions, isEdgeRuntime } from "../helper.js";
 import { copyOpenNextConfig, esbuildAsync } from "../helper.js";
 
+type Override = OverrideOptions & {
+  originResolver?: LazyLoadedOverride<OriginResolver> | IncludedOriginResolver;
+};
 interface BuildEdgeBundleOptions {
   middlewareInfo?: MiddlewareInfo;
   entrypoint: string;
   outfile: string;
   options: BuildOptions;
-  overrides?: OverrideOptions & {
-    originResolver?:
-      | LazyLoadedOverride<OriginResolver>
-      | IncludedOriginResolver;
-  };
+  overrides?: Override;
   defaultConverter?: IncludedConverter;
   additionalInject?: string;
   includeCache?: boolean;
@@ -53,6 +52,14 @@ export async function buildEdgeBundle({
   name,
 }: BuildEdgeBundleOptions) {
   const isInCloudfare = await isEdgeRuntime(overrides);
+  const getOverride = function getOverride<T extends keyof Override>(
+    target: T,
+    defaultValue: Override[T],
+  ) {
+    return typeof overrides?.[target] === "string"
+      ? overrides[target]
+      : defaultValue;
+  };
   await esbuildAsync(
     {
       entryPoints: [entrypoint],
@@ -64,38 +71,17 @@ export async function buildEdgeBundle({
       plugins: [
         openNextResolvePlugin({
           overrides: {
-            wrapper:
-              typeof overrides?.wrapper === "string"
-                ? overrides.wrapper
-                : "aws-lambda",
-            converter:
-              typeof overrides?.converter === "string"
-                ? overrides.converter
-                : defaultConverter,
+            wrapper: getOverride("wrapper", "aws-lambda"),
+            converter: getOverride("converter", defaultConverter),
             ...(includeCache
               ? {
-                  tagCache:
-                    typeof overrides?.tagCache === "string"
-                      ? overrides.tagCache
-                      : "dynamodb-lite",
-                  incrementalCache:
-                    typeof overrides?.incrementalCache === "string"
-                      ? overrides.incrementalCache
-                      : "s3-lite",
-                  queue:
-                    typeof overrides?.queue === "string"
-                      ? overrides.queue
-                      : "sqs-lite",
+                  tagCache: getOverride("tagCache", "dynamodb-lite"),
+                  incrementalCache: getOverride("incrementalCache", "s3-lite"),
+                  queue: getOverride("queue", "sqs-lite"),
                 }
               : {}),
-            originResolver:
-              typeof overrides?.originResolver === "string"
-                ? overrides.originResolver
-                : "pattern-env",
-            proxyExternalRequest:
-              typeof overrides?.proxyExternalRequest === "string"
-                ? overrides.proxyExternalRequest
-                : "node",
+            originResolver: getOverride("originResolver", "pattern-env"),
+            proxyExternalRequest: getOverride("proxyExternalRequest", "node"),
           },
           fnName: name,
         }),