fix: correct return value for serialHooks helper

[erickzhao]

This PR rewrites serialHooks to actually work as intended (by passing the resulting arg as a single mega-hook into Packager's options).

Usage would look something like this:

const opts = {
  afterCopy: serialHooks([
    async () => {}, // hook 1
    async () => {}, // hook 2
    async () => {}, // hook 3
  ])
}

[malept]

I don't suppose there's a reasonable way to add a regression test for this?

[erickzhao]

@malept I think I just wrote the serialHooks test wrong the first time while I was refactoring. The original problem was that I wrote serialHooks as an alternative to runHooks where it should actually be passed into the latter function. I think the test as-is should be good in catching regressions?

packager/test/hooks.spec.ts

Lines 45 to 53 in b25e5bc

	const serializedHooks = serialHooks(testHooks);
	await runHooks(serializedHooks, {
	buildPath: '',
	electronVersion: '',
	platform: 'darwin',
	arch: 'arm64',
	});
	expect(output).toBe('0 1 2 3 4 5 6 7 8 9 10');
	});

[malept]

Ah OK, that makes sense.

[electron-npm-package-publisher]

🎉 This PR is included in version 19.0.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀