Guard packed export entrypoints

Author: AbrahamGreenman

scripts/release/pack-check.mjs

@@ -577,7 +577,7 @@ function assertNoPackedPackageScripts(packageName, manifest) {
   }
 }
 
-function assertRootExportTypes(packageName, manifest) {
+function assertRootExportEntrypoints(packageName, manifest) {
   if (!isRecord(manifest.exports)) {
     throw new Error(`${packageName} package.json exports must be an object`);
   }
@@ -590,6 +590,12 @@ function assertRootExportTypes(packageName, manifest) {
   if (rootExport.types !== manifest.types) {
     throw new Error(`${packageName} package.json exports["."].types must match package.json types`);
   }
+
+  if (rootExport.default !== manifest.main) {
+    throw new Error(
+      `${packageName} package.json exports["."].default must match package.json main`,
+    );
+  }
 }
 
 export function assertPackedPackageManifest(packageName, manifest, entries) {
@@ -617,7 +623,7 @@ export function assertPackedPackageManifest(packageName, manifest, entries) {
   if (manifest.exports === undefined) {
     throw new Error(`${packageName} package.json exports must be present`);
   }
-  assertRootExportTypes(packageName, manifest);
+  assertRootExportEntrypoints(packageName, manifest);
 
   const exportTargets = [];
   collectExportTargets(packageName, manifest.exports, "exports", exportTargets);

scripts/release/pack-check.test.ts

@@ -559,13 +559,14 @@ test("assertPackedPackageManifest rejects broken entry point targets", () => {
           exports: {
             ".": {
               types: "./dist/index.d.ts",
-              default: "./dist/missing.js",
+              development: "./src/missing.ts",
+              default: "./dist/index.js",
             },
           },
         }),
         safePackedManifestEntries,
       ),
-    /@razroo\/ray-sdk package\.json exports\["\."\]\.default points to missing entry package\/dist\/missing\.js/,
+    /@razroo\/ray-sdk package\.json exports\["\."\]\.development points to missing entry package\/src\/missing\.ts/,
   );
 
   assert.throws(
@@ -614,6 +615,26 @@ test("assertPackedPackageManifest requires root export types", () => {
   );
 });
 
+test("assertPackedPackageManifest requires root export default", () => {
+  assert.throws(
+    () =>
+      assertPackedPackageManifest(
+        "@razroo/ray-sdk",
+        safePackedManifest({
+          exports: {
+            ".": {
+              types: "./dist/index.d.ts",
+              development: "./src/index.ts",
+              default: "./dist/other.js",
+            },
+          },
+        }),
+        [...safePackedManifestEntries, "package/dist/other.js"],
+      ),
+    /@razroo\/ray-sdk package\.json exports\["\."\]\.default must match package\.json main/,
+  );
+});
+
 test("assertPackedPackageManifest rejects local-only dependencies and package scripts", () => {
   assert.throws(
     () =>