fix(js): handle different exports field shapes

nrwl · Oct 14, 2022 · 944bbc3 · 944bbc3
1 parent 840f462
commit 944bbc3
Show file tree

Hide file tree

Showing 3 changed files with 123 additions and 10 deletions.
diff --git a/packages/js/src/utils/package-json/update-package-json.spec.ts b/packages/js/src/utils/package-json/update-package-json.spec.ts
@@ -166,4 +166,96 @@ describe('getUpdatedPackageJsonContent', () => {
       version: '0.0.1',
     });
   });
+
+  it('should support different exports field shape', () => {
+    // exports: string
+    expect(
+      getUpdatedPackageJsonContent(
+        {
+          name: 'test',
+          version: '0.0.1',
+          exports: './custom.js',
+        },
+        {
+          main: 'proj/src/index.ts',
+          outputPath: 'dist/proj',
+          projectRoot: 'proj',
+          format: ['esm', 'cjs'],
+          outputFileExtensionForCjs: '.cjs',
+          generateExportsField: true,
+        }
+      )
+    ).toEqual({
+      name: 'test',
+      main: './src/index.cjs',
+      module: './src/index.js',
+      types: './src/index.d.ts',
+      version: '0.0.1',
+      exports: './custom.js',
+    });
+
+    // exports: { '.': string }
+    expect(
+      getUpdatedPackageJsonContent(
+        {
+          name: 'test',
+          version: '0.0.1',
+          exports: {
+            '.': './custom.js',
+          },
+        },
+        {
+          main: 'proj/src/index.ts',
+          outputPath: 'dist/proj',
+          projectRoot: 'proj',
+          format: ['esm', 'cjs'],
+          outputFileExtensionForCjs: '.cjs',
+          generateExportsField: true,
+        }
+      )
+    ).toEqual({
+      name: 'test',
+      main: './src/index.cjs',
+      module: './src/index.js',
+      types: './src/index.d.ts',
+      version: '0.0.1',
+      exports: {
+        '.': './custom.js',
+      },
+    });
+
+    // exports: { './custom': string }
+    expect(
+      getUpdatedPackageJsonContent(
+        {
+          name: 'test',
+          version: '0.0.1',
+          exports: {
+            './custom': './custom.js',
+          },
+        },
+        {
+          main: 'proj/src/index.ts',
+          outputPath: 'dist/proj',
+          projectRoot: 'proj',
+          format: ['esm', 'cjs'],
+          outputFileExtensionForCjs: '.cjs',
+          generateExportsField: true,
+        }
+      )
+    ).toEqual({
+      name: 'test',
+      main: './src/index.cjs',
+      module: './src/index.js',
+      types: './src/index.d.ts',
+      version: '0.0.1',
+      exports: {
+        '.': {
+          import: './src/index.js',
+          require: './src/index.cjs',
+        },
+        './custom': './custom.js',
+      },
+    });
+  });
 });
diff --git a/packages/js/src/utils/package-json/update-package-json.ts b/packages/js/src/utils/package-json/update-package-json.ts
@@ -88,10 +88,17 @@ export function getUpdatedPackageJsonContent(
     options.projectRoot
   );
   const typingsFile = `${relativeMainFileDir}${mainFile}.d.ts`;
-  const exports = {
-    '.': {},
-    ...packageJson.exports,
-  };
+
+  let exports;
+  const { exports: originalExports } = packageJson;
+  exports = originalExports;
+  if (typeof originalExports === 'string') {
+  } else {
+    exports = {
+      '.': {},
+      ...originalExports,
+    };
+  }
 
   const mainJsFile =
     options.outputFileName ?? `${relativeMainFileDir}${mainFile}.js`;
@@ -105,7 +112,13 @@ export function getUpdatedPackageJsonContent(
       packageJson.main ??= mainJsFile;
     }
 
-    exports['.']['import'] = mainJsFile;
+    if (typeof exports !== 'string') {
+      if (typeof exports['.'] !== 'string') {
+        exports['.']['import'] ??= mainJsFile;
+      } else if (!hasCjsFormat) {
+        exports['.'] ??= mainJsFile;
+      }
+    }
   }
 
   // CJS output may have .cjs or .js file extensions.
@@ -117,7 +130,13 @@ export function getUpdatedPackageJsonContent(
       options.outputFileExtensionForCjs ?? '.js'
     }`;
     packageJson.main ??= cjsMain;
-    exports['.']['require'] = cjsMain;
+    if (typeof exports !== 'string') {
+      if (typeof exports['.'] !== 'string') {
+        exports['.']['require'] ??= cjsMain;
+      } else if (!hasEsmFormat) {
+        exports['.'] ??= cjsMain;
+      }
+    }
   }
 
   if (options.generateExportsField) {

diff --git a/packages/nx/src/utils/package-json.ts b/packages/nx/src/utils/package-json.ts
@@ -37,10 +37,12 @@ export interface PackageJson {
   main?: string;
   types?: string;
   module?: string;
-  exports?: Record<
-    string,
-    { types?: string; require?: string; import?: string }
-  >;
+  exports?:
+    | string
+    | Record<
+        string,
+        string | { types?: string; require?: string; import?: string }
+      >;
   dependencies?: Record<string, string>;
   devDependencies?: Record<string, string>;
   peerDependencies?: Record<string, string>;