Expose package export types

Author: AbrahamGreenman

.changeset/export-type-conditions.md

@@ -0,0 +1,6 @@
+---
+"@razroo/ray-core": patch
+"@razroo/ray-sdk": patch
+---
+
+Add explicit package export type conditions so TypeScript consumers resolving through `exports` find the published declarations.

packages/core/package.json

@@ -20,6 +20,7 @@
   "types": "./dist/index.d.ts",
   "exports": {
     ".": {
+      "types": "./dist/index.d.ts",
       "development": "./src/index.ts",
       "default": "./dist/index.js"
     }

packages/sdk/package.json

@@ -22,6 +22,7 @@
   "types": "./dist/index.d.ts",
   "exports": {
     ".": {
+      "types": "./dist/index.d.ts",
       "development": "./src/index.ts",
       "default": "./dist/index.js"
     }

scripts/release/pack-check.mjs

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

scripts/release/pack-check.test.ts

@@ -532,6 +532,7 @@ function safePackedManifest(overrides: Record<string, unknown> = {}): Record<str
     types: "./dist/index.d.ts",
     exports: {
       ".": {
+        types: "./dist/index.d.ts",
         development: "./src/index.ts",
         default: "./dist/index.js",
       },
@@ -557,6 +558,7 @@ test("assertPackedPackageManifest rejects broken entry point targets", () => {
         safePackedManifest({
           exports: {
             ".": {
+              types: "./dist/index.d.ts",
               default: "./dist/missing.js",
             },
           },
@@ -579,6 +581,39 @@ test("assertPackedPackageManifest rejects broken entry point targets", () => {
   );
 });
 
+test("assertPackedPackageManifest requires root export types", () => {
+  assert.throws(
+    () =>
+      assertPackedPackageManifest(
+        "@razroo/ray-sdk",
+        safePackedManifest({
+          exports: {
+            ".": {
+              development: "./src/index.ts",
+              default: "./dist/index.js",
+            },
+          },
+        }),
+        safePackedManifestEntries,
+      ),
+    /@razroo\/ray-sdk package\.json exports\["\."\]\.types must match package\.json types/,
+  );
+
+  assert.throws(
+    () =>
+      assertPackedPackageManifest(
+        "@razroo/ray-sdk",
+        safePackedManifest({
+          exports: {
+            ".": "./dist/index.js",
+          },
+        }),
+        safePackedManifestEntries,
+      ),
+    /@razroo\/ray-sdk package\.json exports\["\."\] must be an object/,
+  );
+});
+
 test("assertPackedPackageManifest rejects local-only dependencies and package scripts", () => {
   assert.throws(
     () =>