@storm-software/workspace-tools 1.49.30 → 1.50.0

package/CHANGELOG.md

@@ -1,3 +1,27 @@
+## 1.50.0 (2024-01-26)
+
+
+### 🚀 Features
+
+- **config-tools:** Added the `externalPackagePatterns` option to Storm config ([78f5c9d4](https://github.com/storm-software/storm-ops/commit/78f5c9d4))
+
+
+### ❤️  Thank You
+
+- Patrick Sullivan
+
+## 1.49.31 (2024-01-26)
+
+
+### 🩹 Fixes
+
+- **workspace-tools:** Use the default `outputPath` for tsup build ([1078da89](https://github.com/storm-software/storm-ops/commit/1078da89))
+
+
+### ❤️  Thank You
+
+- Patrick Sullivan
+
 ## 1.49.30 (2024-01-26)
 
 

package/index.js

@@ -129881,6 +129881,9 @@ var StormConfigSchema = objectType({
   ci: booleanType().default(true).describe("An indicator specifying if the current environment is a CI environment"),
   workspaceRoot: stringType().trim().optional().describe("The root directory of the workspace"),
   packageDirectory: stringType().trim().optional().describe("The root directory of the package"),
+  externalPackagePatterns: arrayType(stringType()).default([]).describe(
+    "The build will use these package patterns to determine if they should be external to the bundle"
+  ),
   buildDirectory: stringType().trim().default("dist").describe("The build directory for the workspace"),
   runtimeDirectory: stringType().trim().default("node_modules/.storm").describe("The runtime directory of Storm"),
   runtimeVersion: stringType().trim().regex(
@@ -139333,17 +139336,18 @@ ${Object.keys(options).map(
       return ret;
     }, options.external);
   }
-  if (options.bundle === false) {
-    for (const thirdPartyDependency of getExternalDependencies(
-      context.projectName,
-      context.projectGraph
+  for (const thirdPartyDependency of getExternalDependencies(
+    context.projectName,
+    context.projectGraph
+  )) {
+    const packageConfig = thirdPartyDependency.node.data;
+    if (packageConfig?.packageName && !config?.externalPackagePatterns?.some(
+      (pattern) => pattern.includes(packageConfig.packageName)
+    ) && !externalDependencies?.some(
+      (externalDependency) => externalDependency.name.includes(packageConfig.packageName)
     )) {
-      const packageConfig = thirdPartyDependency.node.data;
-      if (packageConfig?.packageName) {
-        options.external.push(packageConfig.packageName);
-        if (!packageJson?.devDependencies?.[packageConfig.packageName]) {
-          externalDependencies.push(thirdPartyDependency);
-        }
+      if (!packageJson?.devDependencies?.[packageConfig.packageName] || !packageConfig.hash) {
+        externalDependencies.push(thirdPartyDependency);
       }
     }
   }
@@ -139413,6 +139417,9 @@ ${externalDependencies.map((dep) => {
           packageJson.dependencies ??= {};
           packageJson.dependencies[packageName] = projectGraph.nodes[externalDependency.node.name] ? "latest" : version;
         }
+        if (!options.external.includes(packageName)) {
+          options.external.push(packageName);
+        }
       }
     }
     for (const packageName of internalDependencies) {
@@ -139538,14 +139545,7 @@ ${(0, import_node_fs5.readFileSync)(file, "utf-8")}`,
   }
   await Promise.all(
     entryPoints.map((entryPoint) => {
-      let outputPath = removeExtension(entryPoint).replace(sourceRoot, "");
-      if (outputPath.startsWith(".")) {
-        outputPath = outputPath.substring(1);
-      }
-      if (outputPath.startsWith("/")) {
-        outputPath = outputPath.substring(1);
-      }
-      writeInfo(config, `*** Build output path: ${outputPath} ***`);
+      writeInfo(config, `*** Build output path: ${options.outputPath} ***`);
       return runTsupBuild(
         {
           main: entryPoint,
@@ -139554,11 +139554,7 @@ ${(0, import_node_fs5.readFileSync)(file, "utf-8")}`,
           sourceRoot
         },
         config,
-        {
-          ...options,
-          outputPath,
-          getConfig: options.getConfig
-        }
+        options
       );
     })
   );