Asset compilation failures should break the build

[camertron]

What are you trying to accomplish?

Problems reported during asset compilation do not currently break the build, which lets TypeScript compiler errors into main.

Integration

No changes required in production.

Risk Assessment

• Low risk the change is small, highly observable, and easily rolled back.
• Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
• High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

What approach did you choose and why?

This PR contains a fix for a compiler error introduced in #2414. The build did not fail because of the way our script/build-assets script is written, so that's been fixed too.

Accessibility

• No new axe scan violation - This change does not introduce any new axe scan violations.

[changeset-bot]

🦋 Changeset detected

Latest commit: 9ee7f64

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@primer/view-components	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR