Skip to content

Dispose Error3d fallback mesh resources#918

Open
cuongwf1711 wants to merge 1 commit into
tscircuit:mainfrom
cuongwf1711:fix-93-dispose-error3d-fallback
Open

Dispose Error3d fallback mesh resources#918
cuongwf1711 wants to merge 1 commit into
tscircuit:mainfrom
cuongwf1711:fix-93-dispose-error3d-fallback

Conversation

@cuongwf1711
Copy link
Copy Markdown

@cuongwf1711 cuongwf1711 commented May 22, 2026

/claim #93

Summary

  • add a small disposeMeshResources helper for mesh geometry and material cleanup
  • dispose the Error3d fallback box mesh resources when it unmounts
  • add unit coverage for single-material and material-array meshes

Verification

All verification was run inside Docker on a copied workspace:

  • bun test tests/dispose-mesh-resources.test.ts
  • bunx tsc --noEmit
  • bun run format:check
  • bun run build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
3d-viewer Ready Ready Preview, Comment May 22, 2026 1:22am

Request Review

@cuongwf1711
Copy link
Copy Markdown
Author

Quick follow-up for maintainers: this PR is still mergeable and all checks/previews are green. Please let me know if you want any changes for the bounty claim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant