fix(publish): Run pre and postpack lifecycle scripts on publish (yarn…

…pkg#5712)

**Summary**

npm runs the pre and post pack scripts on publish, but yarn was not. Added these scripts.

fixes yarnpkg#5707

**Test plan**

Added a test that regex's the output to look for scripts being run.
If you look at the test, the regex is kinda weird, but I did it that way to make sure the log statements were in the correct order in the output.

__tests__/commands/publish.js

@@ -100,6 +100,25 @@ test.concurrent('publish should allow `--access` to override publishConfig.acces
   });
 });
 
+test.concurrent('publish should run lifecycle scripts in the correct order', () => {
+  return runPublish([], {newVersion: '1.0.0'}, 'lifecycle-scripts', (config, reporter, stdout) => {
+    expect(stdout).toMatch(
+      new RegExp(
+        [
+          'running the prepublish hook[\\s\\S]*',
+          'running the prepare hook[\\s\\S]*',
+          'running the prepublishOnly hook[\\s\\S]*',
+          'running the prepack hook[\\s\\S]*',
+          'running the postpack hook[\\s\\S]*',
+          'running the publish hook[\\s\\S]*',
+          'running the postpublish hook',
+        ].join(''),
+        'm',
+      ),
+    );
+  });
+});
+
 test.concurrent('can specify a path', () => {
   return runPublish(['mypkg'], {newVersion: '0.0.1'}, 'subdir', config => {
     expect(config.registries.npm.request).toBeCalledWith(

__tests__/fixtures/publish/lifecycle-scripts/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "lifecycle-scripts",
+  "version": "1.0.0",
+  "main": "index.js",
+  "license": "MIT",
+  "scripts": {
+    "prepublish": "echo running the prepublish hook",
+    "prepublishOnly": "echo running the prepublishOnly hook",
+    "publish": "echo running the publish hook",
+    "postpublish": "echo running the postpublish hook",
+    "prepare": "echo running the prepare hook",
+    "prepack": "echo running the prepack hook",
+    "postpack": "echo running the postpack hook"
+  }
+}

src/cli/commands/publish.js

@@ -45,6 +45,7 @@ async function publish(config: Config, pkg: any, flags: Object, dir: string): Pr
   await config.executeLifecycleScript('prepublish');
   await config.executeLifecycleScript('prepare');
   await config.executeLifecycleScript('prepublishOnly');
+  await config.executeLifecycleScript('prepack');
 
   // get tarball stream
   const stat = await fs.lstat(dir);
@@ -62,6 +63,8 @@ async function publish(config: Config, pkg: any, flags: Object, dir: string): Pr
     stream.on('data', data.push.bind(data)).on('end', () => resolve(Buffer.concat(data))).on('error', reject);
   });
 
+  await config.executeLifecycleScript('postpack');
+
   // copy normalized package and remove internal keys as they may be sensitive or yarn specific
   pkg = Object.assign({}, pkg);
   for (const key in pkg) {