fix(build): switch to esbuild and tsc instead of pika

[kfcampbell]

Replicating octokit/request.js#584 to another repo.

---

Pika has been deprecated for a while now, and the project has now been archived, and because of that the released npm package is missing most of the files generated by the build step.

Uses esbuild to transpile the TS source code into an ESM source, NodeJs bundle, and a browser bundle

Uses the TypeScript compiler to generate the types.

Behavior

Before the change?

• Uses pika for the build system
• The released npm package is missing most of the files generated by the build step. dist-node, dist-types, dist-web... even though they are generated correctly.

After the change?

• Uses esbuild to generate the JS bundles
• Uses the TypeScript compiler to generate the type definitions
• The correct file matching patterns are used so we publish all the necessary files again.

---

Additional info

Pull request checklist

• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)
• Added the appropriate label for the given change

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

• Yes (Please add the Type: Breaking change label)
• No

If Yes, what's the impact:

• N/A

Pull request type

Please add the corresponding label for change this PR introduces:

• Bugfix: Type: Bug
• Dependencies/code cleanup: Type: Maintenance

---

[nickfloyd]

❤️

[wolfy1339]

Make sure to explicitly mark every type import and export.

[wolfy1339]

I don't think it's the best idea to move @octokit/types to devDependencies.

There was a reasoning of putting it in regular dependencies

If it's not in regular dependencies, then people using this package may not have types in their IDE

[kfcampbell]

Explicit imports/exports and the previously discussed --verbatimModuleSyntax flag have been added. I'm not quite sure what's going on with the Node 14 errors, though I can reproduce them locally.

[wolfy1339]

Octokit is only being used as a type here

[wolfy1339]

I think I see the problem, the problem is the lockfile here. It's using version 3 which isn't backwards compatible with older versions of npm

Can you regenerate it with Node v14?

[kfcampbell]

Thanks for the tip! Done.

[kfcampbell]

@wolfy1339 I've updated this PR with the latest version of octokit/tsconfig. Please take another look when you get a moment!

[github-actions]

🎉 This PR is included in version 7.1.3 🎉

The release is available on:

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

Your semantic-release bot 📦🚀