./scripts/build.js/tsc() function will always fail due to undefined symbol allDiagnostics

[justingrant]

Describe the bug
#4007 added a new ./scripts/build.js file with a tsc() function. This function will always throw because allDiagnostics is always undefined. In fact there's no other place in the amplify-js codebase where this symbol is used.

amplify-js/scripts/build.js

Line 62 in 94b7181

allDiagnostics.forEach(diagnostic => {

To Reproduce
I was able to repro this last night by just running yarn run build. But before filing this issue I wanted to verify on a clean dev environment so I ran yarn run clean and tried a build again and got no errors. So clearly something was corrupted somewhere in my original dev environment. I'm not sure why my previous environment ended up calling tsc() but a clean one didn't.

That said, this tsc() function will always throw if it's run, so either this is unused code that should be removed, or this code will need to be fixed.

[github-actions]

This issue has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.