Skip to content

[codex] add texture storage guardrails - #48

Merged
mikechaves merged 1 commit into
mainfrom
mike/texture-storage-guardrails
May 23, 2026
Merged

[codex] add texture storage guardrails#48
mikechaves merged 1 commit into
mainfrom
mike/texture-storage-guardrails

Conversation

@mikechaves

Copy link
Copy Markdown
Owner

Summary

  • Add shared material storage budget helpers for embedded texture payloads.
  • Show the current draft texture storage footprint in the editor texture panel.
  • Warn or block projected material saves before the local library becomes too large for browser storage.
  • Block JSON imports that would push the local material library over the storage budget.
  • Fix mobile spacing so the collapsed sidebar menu button does not overlap the first editor heading.

Validation

  • npm run quality:full
  • Browser QA at http://127.0.0.1:5173/ on desktop and mobile 390x844
  • Desktop and mobile console checks showed no warnings or errors

Notes

This improves feedback and prevents over-budget local saves/imports. It does not yet compress or downscale texture images automatically.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a material storage budget system to manage local browser storage limits. It includes a new utility for analyzing and summarizing texture storage, UI updates to display storage usage in the TextureControls component, and logic to warn or block users when saving or importing materials that would exceed defined thresholds. Additionally, it adds comprehensive unit and E2E tests to ensure the storage limits are correctly enforced and reported. I have no feedback to provide as there were no review comments.

@mikechaves
mikechaves marked this pull request as ready for review May 23, 2026 00:53
@mikechaves
mikechaves merged commit 091241e into main May 23, 2026
2 checks passed
@mikechaves
mikechaves deleted the mike/texture-storage-guardrails branch May 23, 2026 00:54
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