chore: debug bundle release workflow

Author: bennypowers

.github/workflows/bundle.yml

@@ -0,0 +1,31 @@
+name: Release
+
+on:
+  release:
+    types:
+      - published
+
+jobs:
+  release:
+    # Prevents changesets action from creating a PR on forks
+    if: github.repository == 'patternfly/patternfly-elements'
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
+        with:
+          node-version: 16
+          cache: npm
+
+      - name: Install packages
+        run: npm i --prefer-offline
+
+      - name: Bundle
+        id: bundle
+        if: ${{ steps.changesets.outputs.published }}
+        uses: actions/github-script@v6
+        with:
+          github-token: ${{secrets.GITHUB_TOKEN}}
+          script: |
+            const bundle = require('./scripts/bundle-release.cjs');
+            await bundle({ context, github, glob, workspace: '${{ github.workspace }}' });

.github/workflows/release.yml

@@ -35,18 +35,3 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-
-      - name: Bundle
-        id: bundle
-        if: ${{ steps.changesets.outputs.published }}
-        uses: actions/github-script@v6
-        with:
-          github-token: ${{secrets.GITHUB_TOKEN}}
-          script: |
-            const bundle = require('./scripts/bundle-release.cjs');
-            await bundle({
-              github,
-              glob,
-              workspace: '${{ github.workspace }}',
-              publishedPackages: JSON.parse('${{ steps.changesets.outputs.publishedPackages }}'),
-            });

scripts/bundle-release.cjs

@@ -1,86 +1,68 @@
-const sleep = ms => new Promise(res => setTimeout(res, ms));
-
-/** exponential back-off function to await results while avoiding rate limits */
-async function backoff(fn, depth = 0) {
-  try {
-    return await fn();
-  } catch (e) {
-    if (depth > 7) {
-      throw e;
-    } else {
-      await sleep(2 ** depth * 10);
-      return backoff(fn, depth + 1);
-    }
-  }
-}
-
-module.exports = async function({ github, glob, workspace, publishedPackages }) {
+module.exports = async function({ github, glob, workspace, context }) {
   const tar = require('tar');
   const { readFile, copyFile } = require('fs').promises;
 
   const { singleFileBuild } = await import('../tools/pfe-tools/esbuild.js');
   const { execaCommand } = await import('execa');
 
+  const { payload } = context;
+  const { release } = payload;
+
   // https://github.com/patternfly/patternfly-elements
   const owner = 'patternfly';
   const repo = 'patternfly-elements';
 
+  const tag = release.tag_name;
+
   // repo root
   const cwd = `${workspace}`;
   const outfile = `${cwd}/pfe.min.js`;
 
   await singleFileBuild({ outfile });
 
-  async function getRelease({ owner, repo, tag }) {
-    const release = await github.request(`GET /repos/{owner}/{repo}/releases/tags/{tag}`, { owner, repo, tag });
-    if (!release.id) {
-      throw new Error(`Could not find release for tag: ${tag}`);
-    }
-  }
+  const params = {
+    owner,
+    release_id: release.id,
+    repo
+  };
+
+  await copyFile(`${cwd}/core/pfe-styles/pfe.min.css`, `${cwd}/pfe.min.css`);
 
-  for (const { name: packageName, version } of publishedPackages) {
-    // get the tag for the release for this package
-    const tag = `${packageName}@${version}`;
+  const globber = await glob.create('pfe.min.*');
+  const files = await globber.glob();
 
-    // wait until the release is created
-    const release = await backoff(() => getRelease({ owner, repo, tag }));
+  // eslint-disable-next-line
+  console.log('creating tarball for', files);
 
-    const params = {
-      owner,
-      release_id: release.id,
-      repo
-    };
+  await tar.c({ gzip: true, file: 'pfe.min.tgz' }, files);
 
-    await copyFile(`${cwd}/core/pfe-styles/pfe.min.css`, `${cwd}/pfe.min.css`);
+  // upload the bundle to each release
+  await github.rest.repos.uploadReleaseAsset({
+    ...params,
+    name: 'pfe.min.tgz',
+    data: await readFile(`${cwd}/pfe.min.tgz`),
+  });
 
-    const globber = await glob.create('pfe.min.*');
-    const files = await globber.glob();
+  const { packageName } = tag.match(/^(?<packageName>@[-\w]+[/]{1}[-\w]+)@(.*)$/)?.groups ?? {};
 
+  if (!packageName) {
     // eslint-disable-next-line
-    console.log('creating tarball for', files);
+    console.log(release);
+    throw new Error('No Package found');
+  }
 
-    await tar.c({ gzip: true, file: 'pfe.min.tgz' }, files);
+  // make a tarball for the package
+  // this was already published to npm in the changesets action
+  const { stdout } = await execaCommand(`npm run pack -w ${packageName}`);
+  const match = stdout.match(/^[\w-.]+\.tgz$/g);
 
-    // upload the bundle to each release
+  // upload the package tarball to the release
+  if (match) {
+    const [name] = match;
     await github.rest.repos.uploadReleaseAsset({
       ...params,
-      name: 'pfe.min.tgz',
-      data: await readFile(`${cwd}/pfe.min.tgz`),
+      name,
+      data: await readFile(`${cwd}/${name}`),
     });
-
-    // make a tarball for the package
-    // this was already published to npm in the changesets action
-    const { stdout } = await execaCommand(`npm run pack -w ${packageName}`);
-    const match = stdout.match(/^[\w-.]+\.tgz$/g);
-
-    // upload the package tarball to the release
-    if (match) {
-      const [name] = match;
-      await github.rest.repos.uploadReleaseAsset({
-        ...params,
-        name,
-        data: await readFile(`${cwd}/${name}`),
-      });
-    }
   }
 };