@inlang/core - npm Package Compare versions

Comparing version 0.5.2 to 0.5.3

dist/test/mockEnvironment.js

@@ -42,7 +42,7 @@ import { initialize$import } from "../config/index.js";
         throw new Error(dedent `
-			The directory specified in \`copyDirectory.path\` does not exist.
+The directory specified in \`copyDirectory.path\` does not exist.
 
-			Solution: Make sure that the \`copyDirectory.path\` is relative to the current working directory.
+Solution: Make sure that the \`copyDirectory.path\` is relative to the current working directory.
 
-			Context: The path is relative to the current working directory, not the file that calls \`mockEnvironment\`.
+Context: The path is relative to the current working directory, not the file that calls \`mockEnvironment\`.
 		`);
@@ -49,0 +49,0 @@ }

dist/utilities/pluginBuildConfig.d.ts

@@ -17,3 +17,3 @@ import type { BuildOptions } from "esbuild";
  */
-export declare function pluginBuildConfig(options: Omit<BuildOptions, (typeof propertiesDefinedByInlang)[number]>): BuildOptions;
+export declare function pluginBuildConfig(options: Omit<BuildOptions, (typeof propertiesDefinedByInlang)[number]>): Promise<BuildOptions>;
 export {};

dist/utilities/pluginBuildConfig.js

@@ -1,3 +0,4 @@
-import { NodeModulesPolyfillPlugin } from "@esbuild-plugins/node-modules-polyfill";
 import dedent from "dedent";
+//! DON'T TOP-LEVEL IMPORT ESBUILD PLUGINS. USE DYNAMIC IMPORTS.
+//! See https://github.com/inlang/inlang/issues/486
 /**
@@ -18,3 +19,3 @@ * These properties are defined by inlang and should not be overwritten by the user.
  */
-export function pluginBuildConfig(options) {
+export async function pluginBuildConfig(options) {
     // type casting. This is safe because we are only adding properties to the options object.
@@ -54,2 +55,7 @@ // furthermore, javascript uses references for objects. thus, no performance penalty.
     // ------------ PLUGINS -------------------
+    //! It is important to dynamically import esbuild plugins here.
+    //! Otherwise, the imported plugins are included in
+    //! bundles that have @inlang/core as a dependency.
+    //! See https://github.com/inlang/inlang/issues/486
+    const { NodeModulesPolyfillPlugin } = await import("@esbuild-plugins/node-modules-polyfill");
     if (ops.plugins === undefined) {
@@ -59,5 +65,5 @@ ops.plugins = [];
     ops.plugins.push(
-    // @ts-ignore - for some reason we get a type error here
+    // @ts-expect-error
     NodeModulesPolyfillPlugin());
     return ops;
 }

package.json

 {
 	"name": "@inlang/core",
 	"type": "module",
-	"version": "0.5.2",
+	"version": "0.5.3",
 	"publishConfig": {
@@ -6,0 +6,0 @@ "access": "public"

No alert changes

Improved metrics

Total package byte prevSize

81625

increased by0.41%

Lines of code

2259

increased by0.27%

No dependency changes