Skip to content

Improve asset load error messages#298

Merged
han4wluc merged 1 commit into
mainfrom
fix/asset-load-error-messages
Jun 14, 2026
Merged

Improve asset load error messages#298
han4wluc merged 1 commit into
mainfrom
fix/asset-load-error-messages

Conversation

@han4wluc

Copy link
Copy Markdown
Contributor

Summary

  • Add display-safe asset load errors while preserving technical diagnostics in error.details and error.cause
  • Surface fetch status, missing URL, empty data, audio decode, font, image, and video root causes
  • Document the error contract in the Assets & Loading guide and bump route-graphics to v1.23.3

Validation

  • bunx vitest run spec/RouteGraphics.publicApi.spec.js spec/audio/audioAsset.spec.js spec/util/createAssetBufferManager.spec.js
  • bun run lint
  • bun run build

Notes

  • bun run test / pre-push hook currently fails unrelated parser fixture assertions in spec/parser/parseText.test.yaml and spec/parser/parseTextRevealing.test.yaml due local text metric differences such as expected height 20 vs received 19. The focused asset-loading tests pass.

@han4wluc han4wluc force-pushed the fix/asset-load-error-messages branch from a4b49cf to 9c7d185 Compare June 13, 2026 10:41
@han4wluc han4wluc force-pushed the fix/asset-load-error-messages branch from 9c7d185 to d45deaf Compare June 13, 2026 10:45
@han4wluc han4wluc merged commit 055edc6 into main Jun 14, 2026
2 checks passed
@han4wluc han4wluc deleted the fix/asset-load-error-messages branch June 14, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant