fix: missing scripts section (#414)

denoland · Nov 22, 2024 · 3cbe7da · 3cbe7da
1 parent c6a773f
commit 3cbe7da
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/lib/package_json.test.ts b/lib/package_json.test.ts
@@ -102,7 +102,7 @@ Deno.test("single entrypoint", () => {
       devDependencies: {
         "@types/node": versions.nodeTypes,
       },
-      scripts: undefined,
+      scripts: {},
       exports: {
         ".": {
           import: {
@@ -137,7 +137,7 @@ Deno.test("single entrypoint", () => {
       devDependencies: {
         "@types/node": versions.nodeTypes,
       },
-      scripts: undefined,
+      scripts: {},
       exports: {
         ".": {
           import: {
@@ -169,7 +169,7 @@ Deno.test("single entrypoint", () => {
       devDependencies: {
         "@types/node": versions.nodeTypes,
       },
-      scripts: undefined,
+      scripts: {},
       _generatedBy: "dnt@dev",
     },
   );
@@ -193,7 +193,7 @@ Deno.test("single entrypoint", () => {
       devDependencies: {
         "@types/node": versions.nodeTypes,
       },
-      scripts: undefined,
+      scripts: {},
       exports: {
         ".": {
           import: "./esm/mod.js",
@@ -226,7 +226,7 @@ Deno.test("single entrypoint", () => {
       devDependencies: {
         "@types/node": versions.nodeTypes,
       },
-      scripts: undefined,
+      scripts: {},
       exports: {
         ".": {
           import: "./esm/mod.js",
@@ -353,7 +353,7 @@ Deno.test("multiple entrypoints", () => {
         },
       },
     },
-    scripts: undefined,
+    scripts: {},
     _generatedBy: "dnt@dev",
   });
 });
@@ -423,7 +423,7 @@ Deno.test("binary entrypoints", () => {
         },
       },
     },
-    scripts: undefined,
+    scripts: {},
     _generatedBy: "dnt@dev",
   });
 });

diff --git a/lib/package_json.ts b/lib/package_json.ts
@@ -94,7 +94,7 @@ export function getPackageJson({
       // override with specified scripts
       ...(packageJsonObj.scripts ?? {}),
     })
-    : packageJsonObj.scripts;
+    : packageJsonObj.scripts ?? {};
   const mainExport = exports.length > 0
     ? {
       module: includeEsModule ? `./esm/${exports[0].path}` : undefined,
@@ -112,6 +112,7 @@ export function getPackageJson({
     ...mainExport,
     ...binaryExport,
     ...packageJsonObj,
+    scripts: {},
     ...deleteEmptyKeys({
       exports: {
         ...(includeEsModule || exports.length > 1

diff --git a/tests/integration.test.ts b/tests/integration.test.ts
@@ -214,6 +214,7 @@ Deno.test("should build with all options off", async () => {
       devDependencies: {
         "@types/node": versions.nodeTypes,
       },
+      scripts: {},
       _generatedBy: "dnt@dev",
     });