@ms-cloudpack/bundler-utilities 0.6.6 → 0.7.0

package/lib/constants.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../src/constants.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,eAAO,MAAM,mBAAmB,WAAW,CAAC;AAE5C,eAAO,MAAM,qBAAqB,UAkBjC,CAAC"}
+{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../src/constants.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,eAAO,MAAM,mBAAmB,WAAW,CAAC;AAE5C,eAAO,MAAM,qBAAqB,UAmBjC,CAAC"}

package/lib/constants.js

@@ -24,5 +24,6 @@ export const base64AssetExtensions = [
     '.woff',
     '.woff2',
     '.eot',
+    '.avif',
 ];
 //# sourceMappingURL=constants.js.map

package/lib/constants.js.map

@@ -1 +1 @@
-{"version":3,"file":"constants.js","sourceRoot":"","sources":["../src/constants.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,QAAQ,CAAC;AAE5C,MAAM,CAAC,MAAM,qBAAqB,GAAG;IACnC,MAAM;IACN,MAAM;IACN,MAAM;IACN,OAAO;IACP,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,OAAO;IACP,OAAO;IACP,QAAQ;IACR,MAAM;CACP,CAAC","sourcesContent":["/**\n * Default ES syntax level. 2022 is the minimum for top-level await.\n *\n * ES2023 is probably fine for browsers (definitely for development), and Node 20 also supports it.\n * But for some reason, rspack only lists up to ES2022 in its valid targets (and none of the\n * ES2023 features are obviously relevant for a bundler), so stick with ES2022 for now.\n */\nexport const defaultTargetSyntax = 'es2022';\n\nexport const base64AssetExtensions = [\n  '.aac',\n  '.bmp',\n  '.gif',\n  '.jpeg',\n  '.jpg',\n  '.mp3',\n  '.mp4',\n  '.ogg',\n  '.otf',\n  '.png',\n  '.svg',\n  '.ttf',\n  '.wav',\n  '.webp',\n  '.woff',\n  '.woff2',\n  '.eot',\n];\n"]}
+{"version":3,"file":"constants.js","sourceRoot":"","sources":["../src/constants.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,QAAQ,CAAC;AAE5C,MAAM,CAAC,MAAM,qBAAqB,GAAG;IACnC,MAAM;IACN,MAAM;IACN,MAAM;IACN,OAAO;IACP,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,MAAM;IACN,OAAO;IACP,OAAO;IACP,QAAQ;IACR,MAAM;IACN,OAAO;CACR,CAAC","sourcesContent":["/**\n * Default ES syntax level. 2022 is the minimum for top-level await.\n *\n * ES2023 is probably fine for browsers (definitely for development), and Node 20 also supports it.\n * But for some reason, rspack only lists up to ES2022 in its valid targets (and none of the\n * ES2023 features are obviously relevant for a bundler), so stick with ES2022 for now.\n */\nexport const defaultTargetSyntax = 'es2022';\n\nexport const base64AssetExtensions = [\n  '.aac',\n  '.bmp',\n  '.gif',\n  '.jpeg',\n  '.jpg',\n  '.mp3',\n  '.mp4',\n  '.ogg',\n  '.otf',\n  '.png',\n  '.svg',\n  '.ttf',\n  '.wav',\n  '.webp',\n  '.woff',\n  '.woff2',\n  '.eot',\n  '.avif',\n];\n"]}

package/lib/index.d.ts

@@ -3,4 +3,5 @@ export { getModuleSearchPaths } from './getModuleSearchPaths.js';
 export { getSwcConfig } from './getSwcConfig.js';
 export { shouldExternalizePackage } from './shouldExternalizePackage.js';
 export { getCssRule } from './getCssRule.js';
+export { processAssetInlineExtensions } from './processAssetInlineExtensions.js';
 //# sourceMappingURL=index.d.ts.map

package/lib/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,MAAM,gBAAgB,CAAC;AAC5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,OAAO,EAAE,wBAAwB,EAAE,MAAM,+BAA+B,CAAC;AACzE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,MAAM,gBAAgB,CAAC;AAC5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,OAAO,EAAE,wBAAwB,EAAE,MAAM,+BAA+B,CAAC;AACzE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,4BAA4B,EAAE,MAAM,mCAAmC,CAAC"}

package/lib/index.js

@@ -3,4 +3,5 @@ export { getModuleSearchPaths } from './getModuleSearchPaths.js';
 export { getSwcConfig } from './getSwcConfig.js';
 export { shouldExternalizePackage } from './shouldExternalizePackage.js';
 export { getCssRule } from './getCssRule.js';
+export { processAssetInlineExtensions } from './processAssetInlineExtensions.js';
 //# sourceMappingURL=index.js.map

package/lib/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,MAAM,gBAAgB,CAAC;AAC5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,OAAO,EAAE,wBAAwB,EAAE,MAAM,+BAA+B,CAAC;AACzE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC","sourcesContent":["export { defaultTargetSyntax, base64AssetExtensions } from './constants.js';\nexport { getModuleSearchPaths } from './getModuleSearchPaths.js';\nexport { getSwcConfig } from './getSwcConfig.js';\nexport { shouldExternalizePackage } from './shouldExternalizePackage.js';\nexport { getCssRule } from './getCssRule.js';\n"]}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,MAAM,gBAAgB,CAAC;AAC5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,OAAO,EAAE,wBAAwB,EAAE,MAAM,+BAA+B,CAAC;AACzE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,4BAA4B,EAAE,MAAM,mCAAmC,CAAC","sourcesContent":["export { defaultTargetSyntax, base64AssetExtensions } from './constants.js';\nexport { getModuleSearchPaths } from './getModuleSearchPaths.js';\nexport { getSwcConfig } from './getSwcConfig.js';\nexport { shouldExternalizePackage } from './shouldExternalizePackage.js';\nexport { getCssRule } from './getCssRule.js';\nexport { processAssetInlineExtensions } from './processAssetInlineExtensions.js';\n"]}

package/lib/processAssetInlineExtensions.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Process asset inline extensions based on the input format.
+ *
+ * @param extensions - Can be either:
+ *   - Array: Replace default list with these extensions (without leading dots per documentation)
+ *   - Object: Modify default list (false removes, true adds extensions, without leading dots per documentation)
+ * @returns Processed array of extensions (with leading dots, matching base64AssetExtensions format)
+ */
+export declare function processAssetInlineExtensions(extensions: string[] | Record<string, boolean>): string[];
+//# sourceMappingURL=processAssetInlineExtensions.d.ts.map

package/lib/processAssetInlineExtensions.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"processAssetInlineExtensions.d.ts","sourceRoot":"","sources":["../src/processAssetInlineExtensions.ts"],"names":[],"mappings":"AAWA;;;;;;;GAOG;AACH,wBAAgB,4BAA4B,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,EAAE,CAoBrG"}

package/lib/processAssetInlineExtensions.js

@@ -0,0 +1,36 @@
+import { base64AssetExtensions } from './constants.js';
+/**
+ * Normalize extension by ensuring it has a leading dot (matching base64AssetExtensions format).
+ * Strips the dot if present in input and adds it back for consistency.
+ */
+function normalizeExtension(ext) {
+    const stripped = ext.startsWith('.') ? ext.slice(1) : ext;
+    return `.${stripped}`;
+}
+/**
+ * Process asset inline extensions based on the input format.
+ *
+ * @param extensions - Can be either:
+ *   - Array: Replace default list with these extensions (without leading dots per documentation)
+ *   - Object: Modify default list (false removes, true adds extensions, without leading dots per documentation)
+ * @returns Processed array of extensions (with leading dots, matching base64AssetExtensions format)
+ */
+export function processAssetInlineExtensions(extensions) {
+    // Array format: replace the entire default list
+    if (Array.isArray(extensions)) {
+        return extensions.map(normalizeExtension);
+    }
+    // Object format: modify the default list
+    const result = new Set(base64AssetExtensions);
+    for (const [ext, include] of Object.entries(extensions)) {
+        const normalizedExt = normalizeExtension(ext);
+        if (include) {
+            result.add(normalizedExt);
+        }
+        else {
+            result.delete(normalizedExt);
+        }
+    }
+    return Array.from(result);
+}
+//# sourceMappingURL=processAssetInlineExtensions.js.map

package/lib/processAssetInlineExtensions.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"processAssetInlineExtensions.js","sourceRoot":"","sources":["../src/processAssetInlineExtensions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,qBAAqB,EAAE,MAAM,gBAAgB,CAAC;AAEvD;;;GAGG;AACH,SAAS,kBAAkB,CAAC,GAAW;IACrC,MAAM,QAAQ,GAAG,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC;IAC1D,OAAO,IAAI,QAAQ,EAAE,CAAC;AACxB,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,4BAA4B,CAAC,UAA8C;IACzF,gDAAgD;IAChD,IAAI,KAAK,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,CAAC;QAC9B,OAAO,UAAU,CAAC,GAAG,CAAC,kBAAkB,CAAC,CAAC;IAC5C,CAAC;IAED,yCAAyC;IACzC,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,qBAAqB,CAAC,CAAC;IAE9C,KAAK,MAAM,CAAC,GAAG,EAAE,OAAO,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,CAAC;QACxD,MAAM,aAAa,GAAG,kBAAkB,CAAC,GAAG,CAAC,CAAC;QAE9C,IAAI,OAAO,EAAE,CAAC;YACZ,MAAM,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;QAC5B,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC;QAC/B,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AAC5B,CAAC","sourcesContent":["import { base64AssetExtensions } from './constants.js';\n\n/**\n * Normalize extension by ensuring it has a leading dot (matching base64AssetExtensions format).\n * Strips the dot if present in input and adds it back for consistency.\n */\nfunction normalizeExtension(ext: string): string {\n  const stripped = ext.startsWith('.') ? ext.slice(1) : ext;\n  return `.${stripped}`;\n}\n\n/**\n * Process asset inline extensions based on the input format.\n *\n * @param extensions - Can be either:\n *   - Array: Replace default list with these extensions (without leading dots per documentation)\n *   - Object: Modify default list (false removes, true adds extensions, without leading dots per documentation)\n * @returns Processed array of extensions (with leading dots, matching base64AssetExtensions format)\n */\nexport function processAssetInlineExtensions(extensions: string[] | Record<string, boolean>): string[] {\n  // Array format: replace the entire default list\n  if (Array.isArray(extensions)) {\n    return extensions.map(normalizeExtension);\n  }\n\n  // Object format: modify the default list\n  const result = new Set(base64AssetExtensions);\n\n  for (const [ext, include] of Object.entries(extensions)) {\n    const normalizedExt = normalizeExtension(ext);\n\n    if (include) {\n      result.add(normalizedExt);\n    } else {\n      result.delete(normalizedExt);\n    }\n  }\n\n  return Array.from(result);\n}\n"]}

package/lib/shouldExternalizePackage.d.ts

@@ -1,5 +1,9 @@
 import type { BundleMode, TargetEnvironment } from '@ms-cloudpack/common-types';
 export interface ShouldExternalizePackageOptions {
+    /**
+     * Name of the package being bundled.
+     */
+    packageName: string;
     /**
      * The absolute input path for the entry points to be bundled.
      */

package/lib/shouldExternalizePackage.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"shouldExternalizePackage.d.ts","sourceRoot":"","sources":["../src/shouldExternalizePackage.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAC;AAKhF,MAAM,WAAW,+BAA+B;IAC9C;;OAEG;IACH,SAAS,EAAE,MAAM,CAAC;IAClB;;;OAGG;IACH,EAAE,EAAE,MAAM,CAAC;IACX;;;;OAIG;IACH,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB;;;OAGG;IACH,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;IACnB;;;OAGG;IACH,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;IACpB;;;OAGG;IACH,SAAS,CAAC,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IACxB;;;;;OAKG;IACH,wBAAwB,EAAE,OAAO,CAAC;IAClC;;OAEG;IACH,iBAAiB,EAAE,iBAAiB,CAAC;IACrC;;;;OAIG;IACH,IAAI,EAAE,UAAU,CAAC;CAClB;AAED;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,+BAA+B,GAAG,OAAO,CA+BzF"}
+{"version":3,"file":"shouldExternalizePackage.d.ts","sourceRoot":"","sources":["../src/shouldExternalizePackage.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAC;AAKhF,MAAM,WAAW,+BAA+B;IAC9C;;OAEG;IACH,WAAW,EAAE,MAAM,CAAC;IACpB;;OAEG;IACH,SAAS,EAAE,MAAM,CAAC;IAClB;;;OAGG;IACH,EAAE,EAAE,MAAM,CAAC;IACX;;;;OAIG;IACH,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB;;;OAGG;IACH,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;IACnB;;;OAGG;IACH,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;IACpB;;;OAGG;IACH,SAAS,CAAC,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IACxB;;;;;OAKG;IACH,wBAAwB,EAAE,OAAO,CAAC;IAClC;;OAEG;IACH,iBAAiB,EAAE,iBAAiB,CAAC;IACrC;;;;OAIG;IACH,IAAI,EAAE,UAAU,CAAC;CAClB;AAED;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,+BAA+B,GAAG,OAAO,CAkEzF"}

package/lib/shouldExternalizePackage.js

@@ -22,7 +22,7 @@ import path from 'path';
  * ```
  */
 export function shouldExternalizePackage(params) {
-    const { inputPath, id, parent, inlined, external, polyfills, shouldInlineNodeBuiltins, targetEnvironment, mode } = params;
+    const { packageName, inputPath, id, parent, inlined, external, polyfills, shouldInlineNodeBuiltins, targetEnvironment, mode, } = params;
     if (targetEnvironment === 'node' && isBuiltin(id)) {
         // For Node.js, always externalize built-in modules, even in production mode.
         return true;
@@ -32,19 +32,39 @@ export function shouldExternalizePackage(params) {
         return false;
     }
     const isExternalPackage = isExternal(id, external);
-    return !((targetEnvironment === 'browser' &&
-        // For browser bundles, we should inline node builtins that are handled by the nodePolyfills plugin.
-        // Except if the dependency is listed as external.
+    let idPackageName;
+    // For browser bundles, inline node builtins that are handled by the nodePolyfills plugin,
+    // except if the dependency is explicitly listed as external.
+    if (targetEnvironment === 'browser' &&
         !isExternalPackage &&
         shouldInlineNodeBuiltins &&
-        isInlinedNodeBuiltin(id, polyfills)) ||
-        path.isAbsolute(id) ||
-        isRelative(id) ||
-        // The length check ensures that parseImportString is only called when there are inlined deps
-        (inlined?.length && inlined.includes(parseImportString(id).packageName)) ||
-        // A file outside of the package is doing this import, so we should inline the nested dependencies.
-        // Except if the dependency is listed as external.
-        (isNestedDependency(inputPath, parent) && !isExternalPackage && !isRelayArtifact(parent)));
+        isInlinedNodeBuiltin(id, polyfills)) {
+        return false;
+    }
+    // Early checks that don't require parsing the package name
+    if (path.isAbsolute(id) || isRelative(id)) {
+        return false;
+    }
+    // Check if this is an explicitly inlined dependency
+    // The length check ensures parseImportString is only called when there are inlined deps
+    if (inlined?.length) {
+        idPackageName = parseImportString(id).packageName;
+        if (inlined.includes(idPackageName)) {
+            return false;
+        }
+    }
+    // A file outside of the package is doing this import, so we should inline the nested dependencies,
+    // except if the dependency is explicitly listed as external.
+    if (isNestedDependency(inputPath, parent) && !isExternalPackage && !isRelayArtifact(parent)) {
+        return false;
+    }
+    // Check for self-imports (same package importing itself)
+    idPackageName ??= parseImportString(id).packageName;
+    if (packageName === idPackageName) {
+        return false;
+    }
+    // Default: externalize the package
+    return true;
 }
 function isInlinedNodeBuiltin(id, polyfills) {
     return isBuiltin(id) || !!polyfills?.has(id);
@@ -54,8 +74,12 @@ function isRelative(id) {
 }
 // Check if the import comes from an inlined dependency.
 function isNestedDependency(inputPath, parent) {
-    const relativeParentPath = parent ? path.relative(inputPath, parent) : undefined;
-    return !!(relativeParentPath?.startsWith('..') || relativeParentPath?.includes('node_modules'));
+    if (!parent)
+        return false;
+    const relativeParentPath = path.relative(inputPath, parent);
+    return !!(relativeParentPath.startsWith('..') || // The import comes from an inlined internal package.
+        relativeParentPath.includes('node_modules') // The import comes from an inlined external package.
+    );
 }
 function isExternal(id, external) {
     return !!external?.find((ex) => id.startsWith(ex));

package/lib/shouldExternalizePackage.js.map

@@ -1 +1 @@
-{"version":3,"file":"shouldExternalizePackage.js","sourceRoot":"","sources":["../src/shouldExternalizePackage.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACnC,OAAO,IAAI,MAAM,MAAM,CAAC;AAoDxB;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,wBAAwB,CAAC,MAAuC;IAC9E,MAAM,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,wBAAwB,EAAE,iBAAiB,EAAE,IAAI,EAAE,GAC9G,MAAM,CAAC;IAET,IAAI,iBAAiB,KAAK,MAAM,IAAI,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC;QAClD,6EAA6E;QAC7E,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;QACvB,iFAAiF;QACjF,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,iBAAiB,GAAG,UAAU,CAAC,EAAE,EAAE,QAAQ,CAAC,CAAC;IAEnD,OAAO,CAAC,CACN,CAAC,iBAAiB,KAAK,SAAS;QAC9B,oGAAoG;QACpG,kDAAkD;QAClD,CAAC,iBAAiB;QAClB,wBAAwB;QACxB,oBAAoB,CAAC,EAAE,EAAE,SAAS,CAAC,CAAC;QACtC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;QACnB,UAAU,CAAC,EAAE,CAAC;QACd,6FAA6F;QAC7F,CAAC,OAAO,EAAE,MAAM,IAAI,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAC,WAAW,CAAC,CAAC;QACxE,mGAAmG;QACnG,kDAAkD;QAClD,CAAC,kBAAkB,CAAC,SAAS,EAAE,MAAM,CAAC,IAAI,CAAC,iBAAiB,IAAI,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC,CAC1F,CAAC;AACJ,CAAC;AAED,SAAS,oBAAoB,CAAC,EAAU,EAAE,SAAkC;IAC1E,OAAO,SAAS,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC;AAC/C,CAAC;AAED,SAAS,UAAU,CAAC,EAAU;IAC5B,OAAO,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,GAAG,IAAI,EAAE,KAAK,IAAI,CAAC;AACxG,CAAC;AAED,wDAAwD;AACxD,SAAS,kBAAkB,CAAC,SAAiB,EAAE,MAA0B;IACvE,MAAM,kBAAkB,GAAG,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;IACjF,OAAO,CAAC,CAAC,CAAC,kBAAkB,EAAE,UAAU,CAAC,IAAI,CAAC,IAAI,kBAAkB,EAAE,QAAQ,CAAC,cAAc,CAAC,CAAC,CAAC;AAClG,CAAC;AAED,SAAS,UAAU,CAAC,EAAU,EAAE,QAA8B;IAC5D,OAAO,CAAC,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC,CAAC;AACrD,CAAC;AAED,kFAAkF;AAClF,gEAAgE;AAChE,+DAA+D;AAC/D,SAAS,eAAe,CAAC,MAAe;IACtC,OAAO,CAAC,CAAC,MAAM,EAAE,QAAQ,CAAC,eAAe,CAAC,CAAC;AAC7C,CAAC","sourcesContent":["import type { BundleMode, TargetEnvironment } from '@ms-cloudpack/common-types';\nimport { parseImportString } from '@ms-cloudpack/path-string-parsing';\nimport { isBuiltin } from 'module';\nimport path from 'path';\n\nexport interface ShouldExternalizePackageOptions {\n  /**\n   * The absolute input path for the entry points to be bundled.\n   */\n  inputPath: string;\n  /**\n   * The module ID (import path) being evaluated for externalization.\n   * This can be a package name or a relative/absolute path.\n   */\n  id: string;\n  /**\n   * The absolute file path of the module importing this dependency.\n   * Used to determine if the import is coming from outside the package.\n   * If it is undefined, the import is considered to be from the same package (this is common in Rollup).\n   */\n  parent?: string;\n  /**\n   * List of dependencies which should be explicitly inlined rather than externalized.\n   * These will always be bundled with the package regardless of other settings.\n   */\n  inlined?: string[];\n  /**\n   * List of dependencies which should be explicitly externalized.\n   * These will never be bundled with the package.\n   */\n  external?: string[];\n  /**\n   * Additional polyfill package names that should be inlined instead of externalized\n   * if `shouldInlineNodeBuiltins` is true. This does NOT need to include Node.js built-ins.\n   */\n  polyfills?: Set<string>;\n  /**\n   * Whether Node.js built-in modules should be inlined (with polyfills) rather than externalized.\n   * When true, modules like 'fs' or 'path' will be replaced with browser-compatible versions.\n   *\n   * If `targetEnvironment` is `'node'`, this will always be ignored since built-ins should always be externalized.\n   */\n  shouldInlineNodeBuiltins: boolean;\n  /**\n   * Where the bundled code will run.\n   */\n  targetEnvironment: TargetEnvironment;\n  /**\n   * Bundle mode. In production/development mode, all packages will be inlined, except Node built-ins if\n   * `targetEnvironment` is `'node'`. (You can skip calling this function and just inline everything for\n   * `targetEnvironment: 'browser'`.)\n   */\n  mode: BundleMode;\n}\n\n/**\n * Determines if a package should be externalized from the bundle.\n * Don't externalize inlined packages, node builtins, absolute paths, relative paths,\n * or dependencies of inlined dependencies except if they are explicitly marked as external.\n *\n * Example params (`<root>` is the absolute repo root path):\n * ```ts\n * {\n *   inputPath: \"<root>/app\",\n *   id: \"<root>/app/src/about.tsx\",\n *   parent: \"<root>/app/src/index.tsx\",\n *   inlined: [\"foo\"],\n *   external: [\"bar\", \"baz\"],\n *   polyfills: new Set([\"global\"]),\n *   shouldInlineNodeBuiltins: false,\n *   targetEnvironment: \"browser\",\n *   mode: \"library\"\n * )\n * ```\n */\nexport function shouldExternalizePackage(params: ShouldExternalizePackageOptions): boolean {\n  const { inputPath, id, parent, inlined, external, polyfills, shouldInlineNodeBuiltins, targetEnvironment, mode } =\n    params;\n\n  if (targetEnvironment === 'node' && isBuiltin(id)) {\n    // For Node.js, always externalize built-in modules, even in production mode.\n    return true;\n  }\n\n  if (mode !== 'library') {\n    // For production/development mode, don't externalize anything (except as above).\n    return false;\n  }\n\n  const isExternalPackage = isExternal(id, external);\n\n  return !(\n    (targetEnvironment === 'browser' &&\n      // For browser bundles, we should inline node builtins that are handled by the nodePolyfills plugin.\n      // Except if the dependency is listed as external.\n      !isExternalPackage &&\n      shouldInlineNodeBuiltins &&\n      isInlinedNodeBuiltin(id, polyfills)) ||\n    path.isAbsolute(id) ||\n    isRelative(id) ||\n    // The length check ensures that parseImportString is only called when there are inlined deps\n    (inlined?.length && inlined.includes(parseImportString(id).packageName)) ||\n    // A file outside of the package is doing this import, so we should inline the nested dependencies.\n    // Except if the dependency is listed as external.\n    (isNestedDependency(inputPath, parent) && !isExternalPackage && !isRelayArtifact(parent))\n  );\n}\n\nfunction isInlinedNodeBuiltin(id: string, polyfills: Set<string> | undefined): boolean {\n  return isBuiltin(id) || !!polyfills?.has(id);\n}\n\nfunction isRelative(id: string): boolean {\n  return id.startsWith('/') || id.startsWith('./') || id.startsWith('../') || id === '.' || id === '..';\n}\n\n// Check if the import comes from an inlined dependency.\nfunction isNestedDependency(inputPath: string, parent: string | undefined): boolean {\n  const relativeParentPath = parent ? path.relative(inputPath, parent) : undefined;\n  return !!(relativeParentPath?.startsWith('..') || relativeParentPath?.includes('node_modules'));\n}\n\nfunction isExternal(id: string, external: string[] | undefined): boolean {\n  return !!external?.find((ex) => id.startsWith(ex));\n}\n\n// Check if the importer is an artifact as it might be in a grandparent directory.\n// Rollup parent is the filename ../__generated__/foo.graphql.ts\n// while webpack/rspack parent is the dirname ../__generated__/\nfunction isRelayArtifact(parent?: string): boolean {\n  return !!parent?.includes('__generated__');\n}\n"]}
+{"version":3,"file":"shouldExternalizePackage.js","sourceRoot":"","sources":["../src/shouldExternalizePackage.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACnC,OAAO,IAAI,MAAM,MAAM,CAAC;AAwDxB;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,wBAAwB,CAAC,MAAuC;IAC9E,MAAM,EACJ,WAAW,EACX,SAAS,EACT,EAAE,EACF,MAAM,EACN,OAAO,EACP,QAAQ,EACR,SAAS,EACT,wBAAwB,EACxB,iBAAiB,EACjB,IAAI,GACL,GAAG,MAAM,CAAC;IAEX,IAAI,iBAAiB,KAAK,MAAM,IAAI,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC;QAClD,6EAA6E;QAC7E,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;QACvB,iFAAiF;QACjF,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,iBAAiB,GAAG,UAAU,CAAC,EAAE,EAAE,QAAQ,CAAC,CAAC;IACnD,IAAI,aAAiC,CAAC;IAEtC,0FAA0F;IAC1F,6DAA6D;IAC7D,IACE,iBAAiB,KAAK,SAAS;QAC/B,CAAC,iBAAiB;QAClB,wBAAwB;QACxB,oBAAoB,CAAC,EAAE,EAAE,SAAS,CAAC,EACnC,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;IAED,2DAA2D;IAC3D,IAAI,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC,IAAI,UAAU,CAAC,EAAE,CAAC,EAAE,CAAC;QAC1C,OAAO,KAAK,CAAC;IACf,CAAC;IAED,oDAAoD;IACpD,wFAAwF;IACxF,IAAI,OAAO,EAAE,MAAM,EAAE,CAAC;QACpB,aAAa,GAAG,iBAAiB,CAAC,EAAE,CAAC,CAAC,WAAW,CAAC;QAClD,IAAI,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,CAAC;YACpC,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IAED,mGAAmG;IACnG,6DAA6D;IAC7D,IAAI,kBAAkB,CAAC,SAAS,EAAE,MAAM,CAAC,IAAI,CAAC,iBAAiB,IAAI,CAAC,eAAe,CAAC,MAAM,CAAC,EAAE,CAAC;QAC5F,OAAO,KAAK,CAAC;IACf,CAAC;IAED,yDAAyD;IACzD,aAAa,KAAK,iBAAiB,CAAC,EAAE,CAAC,CAAC,WAAW,CAAC;IACpD,IAAI,WAAW,KAAK,aAAa,EAAE,CAAC;QAClC,OAAO,KAAK,CAAC;IACf,CAAC;IAED,mCAAmC;IACnC,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,oBAAoB,CAAC,EAAU,EAAE,SAAkC;IAC1E,OAAO,SAAS,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC;AAC/C,CAAC;AAED,SAAS,UAAU,CAAC,EAAU;IAC5B,OAAO,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,GAAG,IAAI,EAAE,KAAK,IAAI,CAAC;AACxG,CAAC;AAED,wDAAwD;AACxD,SAAS,kBAAkB,CAAC,SAAiB,EAAE,MAA0B;IACvE,IAAI,CAAC,MAAM;QAAE,OAAO,KAAK,CAAC;IAC1B,MAAM,kBAAkB,GAAG,IAAI,CAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC;IAC5D,OAAO,CAAC,CAAC,CACP,kBAAkB,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,qDAAqD;QAC5F,kBAAkB,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,qDAAqD;KAClG,CAAC;AACJ,CAAC;AAED,SAAS,UAAU,CAAC,EAAU,EAAE,QAA8B;IAC5D,OAAO,CAAC,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC,CAAC;AACrD,CAAC;AAED,kFAAkF;AAClF,gEAAgE;AAChE,+DAA+D;AAC/D,SAAS,eAAe,CAAC,MAAe;IACtC,OAAO,CAAC,CAAC,MAAM,EAAE,QAAQ,CAAC,eAAe,CAAC,CAAC;AAC7C,CAAC","sourcesContent":["import type { BundleMode, TargetEnvironment } from '@ms-cloudpack/common-types';\nimport { parseImportString } from '@ms-cloudpack/path-string-parsing';\nimport { isBuiltin } from 'module';\nimport path from 'path';\n\nexport interface ShouldExternalizePackageOptions {\n  /**\n   * Name of the package being bundled.\n   */\n  packageName: string;\n  /**\n   * The absolute input path for the entry points to be bundled.\n   */\n  inputPath: string;\n  /**\n   * The module ID (import path) being evaluated for externalization.\n   * This can be a package name or a relative/absolute path.\n   */\n  id: string;\n  /**\n   * The absolute file path of the module importing this dependency.\n   * Used to determine if the import is coming from outside the package.\n   * If it is undefined, the import is considered to be from the same package (this is common in Rollup).\n   */\n  parent?: string;\n  /**\n   * List of dependencies which should be explicitly inlined rather than externalized.\n   * These will always be bundled with the package regardless of other settings.\n   */\n  inlined?: string[];\n  /**\n   * List of dependencies which should be explicitly externalized.\n   * These will never be bundled with the package.\n   */\n  external?: string[];\n  /**\n   * Additional polyfill package names that should be inlined instead of externalized\n   * if `shouldInlineNodeBuiltins` is true. This does NOT need to include Node.js built-ins.\n   */\n  polyfills?: Set<string>;\n  /**\n   * Whether Node.js built-in modules should be inlined (with polyfills) rather than externalized.\n   * When true, modules like 'fs' or 'path' will be replaced with browser-compatible versions.\n   *\n   * If `targetEnvironment` is `'node'`, this will always be ignored since built-ins should always be externalized.\n   */\n  shouldInlineNodeBuiltins: boolean;\n  /**\n   * Where the bundled code will run.\n   */\n  targetEnvironment: TargetEnvironment;\n  /**\n   * Bundle mode. In production/development mode, all packages will be inlined, except Node built-ins if\n   * `targetEnvironment` is `'node'`. (You can skip calling this function and just inline everything for\n   * `targetEnvironment: 'browser'`.)\n   */\n  mode: BundleMode;\n}\n\n/**\n * Determines if a package should be externalized from the bundle.\n * Don't externalize inlined packages, node builtins, absolute paths, relative paths,\n * or dependencies of inlined dependencies except if they are explicitly marked as external.\n *\n * Example params (`<root>` is the absolute repo root path):\n * ```ts\n * {\n *   inputPath: \"<root>/app\",\n *   id: \"<root>/app/src/about.tsx\",\n *   parent: \"<root>/app/src/index.tsx\",\n *   inlined: [\"foo\"],\n *   external: [\"bar\", \"baz\"],\n *   polyfills: new Set([\"global\"]),\n *   shouldInlineNodeBuiltins: false,\n *   targetEnvironment: \"browser\",\n *   mode: \"library\"\n * )\n * ```\n */\nexport function shouldExternalizePackage(params: ShouldExternalizePackageOptions): boolean {\n  const {\n    packageName,\n    inputPath,\n    id,\n    parent,\n    inlined,\n    external,\n    polyfills,\n    shouldInlineNodeBuiltins,\n    targetEnvironment,\n    mode,\n  } = params;\n\n  if (targetEnvironment === 'node' && isBuiltin(id)) {\n    // For Node.js, always externalize built-in modules, even in production mode.\n    return true;\n  }\n\n  if (mode !== 'library') {\n    // For production/development mode, don't externalize anything (except as above).\n    return false;\n  }\n\n  const isExternalPackage = isExternal(id, external);\n  let idPackageName: string | undefined;\n\n  // For browser bundles, inline node builtins that are handled by the nodePolyfills plugin,\n  // except if the dependency is explicitly listed as external.\n  if (\n    targetEnvironment === 'browser' &&\n    !isExternalPackage &&\n    shouldInlineNodeBuiltins &&\n    isInlinedNodeBuiltin(id, polyfills)\n  ) {\n    return false;\n  }\n\n  // Early checks that don't require parsing the package name\n  if (path.isAbsolute(id) || isRelative(id)) {\n    return false;\n  }\n\n  // Check if this is an explicitly inlined dependency\n  // The length check ensures parseImportString is only called when there are inlined deps\n  if (inlined?.length) {\n    idPackageName = parseImportString(id).packageName;\n    if (inlined.includes(idPackageName)) {\n      return false;\n    }\n  }\n\n  // A file outside of the package is doing this import, so we should inline the nested dependencies,\n  // except if the dependency is explicitly listed as external.\n  if (isNestedDependency(inputPath, parent) && !isExternalPackage && !isRelayArtifact(parent)) {\n    return false;\n  }\n\n  // Check for self-imports (same package importing itself)\n  idPackageName ??= parseImportString(id).packageName;\n  if (packageName === idPackageName) {\n    return false;\n  }\n\n  // Default: externalize the package\n  return true;\n}\n\nfunction isInlinedNodeBuiltin(id: string, polyfills: Set<string> | undefined): boolean {\n  return isBuiltin(id) || !!polyfills?.has(id);\n}\n\nfunction isRelative(id: string): boolean {\n  return id.startsWith('/') || id.startsWith('./') || id.startsWith('../') || id === '.' || id === '..';\n}\n\n// Check if the import comes from an inlined dependency.\nfunction isNestedDependency(inputPath: string, parent: string | undefined): boolean {\n  if (!parent) return false;\n  const relativeParentPath = path.relative(inputPath, parent);\n  return !!(\n    relativeParentPath.startsWith('..') || // The import comes from an inlined internal package.\n    relativeParentPath.includes('node_modules') // The import comes from an inlined external package.\n  );\n}\n\nfunction isExternal(id: string, external: string[] | undefined): boolean {\n  return !!external?.find((ex) => id.startsWith(ex));\n}\n\n// Check if the importer is an artifact as it might be in a grandparent directory.\n// Rollup parent is the filename ../__generated__/foo.graphql.ts\n// while webpack/rspack parent is the dirname ../__generated__/\nfunction isRelayArtifact(parent?: string): boolean {\n  return !!parent?.includes('__generated__');\n}\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ms-cloudpack/bundler-utilities",
-  "version": "0.6.6",
+  "version": "0.7.0",
   "description": "Utilities used by multiple Cloudpack bundlers implementations",
   "license": "MIT",
   "type": "module",
@@ -15,14 +15,14 @@
   },
   "dependencies": {
     "@ms-cloudpack/json-utilities": "^0.1.11",
-    "@ms-cloudpack/package-utilities": "^13.1.4",
+    "@ms-cloudpack/package-utilities": "^13.2.0",
     "@ms-cloudpack/path-string-parsing": "^1.2.7",
-    "@ms-cloudpack/path-utilities": "^3.1.30",
+    "@ms-cloudpack/path-utilities": "^3.1.32",
     "get-tsconfig": "^4.7.2",
     "tsconfig-to-swcconfig": "^2.7.0"
   },
   "devDependencies": {
-    "@ms-cloudpack/common-types": "^0.31.1",
+    "@ms-cloudpack/common-types": "^0.32.0",
     "@ms-cloudpack/eslint-plugin-internal": "^0.0.1",
     "@ms-cloudpack/scripts": "^0.0.1",
     "@ms-cloudpack/test-utilities": "^0.5.0",