Repository navigation
fix(cli): format page bundler failures - #1406
Conversation
…ts file, line and code frame; a page refuses --verdict and extra arguments; the guide is unchanged; ping checks a page whose import does not resolve Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Thanks, @oxura! Moving the page build under the CLI's error handling was the right fix. Before merging I pushed one commit on top (ab6b689):
Checked with the real CLI: a missing import, a failing |
|
Thanks for the follow-up and merge. Preserving each bundler diagnostic’s code frame/location and rejecting arguments a page would ignore is the right behavior. |
Fixes #1399.
Move the HTML dispatch under the existing CLI error boundary and await it there. Preserve AggregateError child diagnostics instead of replacing the actual missing-page/output-path reason with
AggregateError: Bundle failed. CLI option ordering and successful bundling are unchanged. Document nonzero page-build failure behavior in the guide; permanent token caps are unchanged.Verification on Linux x86_64 / Bun 1.3.13:
Error: Bundle failedplus their concreteBuildMessage; no unhandled-error banner. The existing output file remains byte-identical..bendentry retain their existing errors.bun gates/repo.ts:PASS: 50 / 50.No browser behavior change is claimed. The separately reported empty-chunk/import-validation issue #1400 is not addressed. No mini-cluster gates were run; this is a local actual-CLI before/after smoke, not a mocked Bun.build test.