fix(core): fix the value check for publishConfig.provenance

Signed-off-by: Sora Morimoto <[email protected]>

Author: smorimoto

packages/yarnpkg-core/sources/Manifest.ts

@@ -523,6 +523,9 @@ export class Manifest {
       if (typeof data.publishConfig.registry === `string`)
         this.publishConfig.registry = data.publishConfig.registry;
 
+      if (typeof data.publishConfig.provenance === `boolean`)
+        this.publishConfig.provenance = data.publishConfig.provenance;
+
       if (typeof data.publishConfig.bin === `string`) {
         if (this.name !== null) {
           this.publishConfig.bin = new Map([[this.name.name, normalizeSlashes(data.publishConfig.bin)]]);

packages/yarnpkg-core/tests/Manifest.test.ts

@@ -55,4 +55,21 @@ describe(`Manifest`, () => {
       expect(manifest.exportTo({}).bin).toEqual({bin2: `./bin2.js`});
     });
   });
+
+  describe(`publishConfig`, () => {
+    it(`should parse provenance field when set to true`, () => {
+      const manifest = Manifest.fromText(`{ "publishConfig": { "provenance": true } }`);
+      expect(manifest.publishConfig?.provenance).toBe(true);
+    });
+
+    it(`should parse provenance field when set to false`, () => {
+      const manifest = Manifest.fromText(`{ "publishConfig": { "provenance": false } }`);
+      expect(manifest.publishConfig?.provenance).toBe(false);
+    });
+
+    it(`should ignore non-boolean provenance values`, () => {
+      const manifest = Manifest.fromText(`{ "publishConfig": { "provenance": "true" } }`);
+      expect(manifest.publishConfig?.provenance).toBeUndefined();
+    });
+  });
 });