fix: avoid self referencing import for external build transforms (#1453)

Author: kuhe

.changeset/sweet-horses-impress.md

@@ -0,0 +1,5 @@
+---
+"@smithy/core": patch
+---
+
+avoid self referencing submodule import

packages/core/scripts/lint.js

@@ -110,6 +110,19 @@ const walk = require("../../../scripts/utils/walk");
         );
       }
     }
+
+    const subModuleImports = [
+      ...new Set(
+        (sourceCode.toString().match(/(from |import\()"\@smithy\/core\/(.*?)";/g) || []).map(
+          (_) => _.match(/@smithy\/core\/(.*?)"/)[1]
+        )
+      ),
+    ];
+    const ownModule = item.match(/src\/submodules\/(.*?)\//)?.[1];
+
+    if (subModuleImports.includes(ownModule)) {
+      errors.push(`self-referencing submodule import found in ${item}`);
+    }
   }
 })().then(() => {
   if (errors.length) {

packages/core/src/submodules/protocols/requestBuilder.ts

@@ -1,7 +1,8 @@
-import { resolvedPath } from "@smithy/core/protocols";
 import { HttpRequest } from "@smithy/protocol-http";
 import type { SerdeContext } from "@smithy/types";
 
+import { resolvedPath } from "./resolve-path";
+
 /**
  * @internal
  * used in code-generated serde.