fix(viewport): report a native resource limit during view build as resource_limit

Aperta
  • Qualcosa non va?
#415 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@cristim ci sta già lavorando.

Dal 9/11/2026.

  • #425 di @cristim — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
30/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
rust
Ambito
desktop

Direzione di ricerca

Start with view_error_category in src/app/panels.rs and the each_view_error_has_its_viewport_error_category test. Check .project-docs/mcp.md for the MCP-visible viewport_error category. Done means the native resource-limit case is categorized as resource_limit, other native errors remain scene_geometry, and reverting the change makes the test fail; linked PR #425 indicates work is already underway.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

area/viewport effort/xs impact/few priority/p3 severity/low triaged type/bug urgency/eventually

Problem

Raised by a review of #399. view_error_category (src/app/panels.rs) maps ViewError::Native(NativeError::ResourceLimit) to ViewportError::SceneGeometry, so a view build that fails because an allocation or admission limit was hit is reported as scene_geometry instead of resource_limit. ViewError::ResourceLimit itself already maps to resource_limit. #399 kept the old behaviour on purpose (the catch-all _ => SceneGeometry it replaced did the same) and pins it in each_view_error_has_its_viewport_error_category, so changing it is a separate, small behaviour change.

Change

  • Map ViewError::Native(NativeError::ResourceLimit) to ViewportError::ResourceLimit; keep scene_geometry for the other Native errors.
  • Update the pinned test case to expect resource_limit, and check the MCP-visible viewport_error category in .project-docs/mcp.md still matches.
  • Probe: reverting the new arm must fail the test.

Related: #399, #384.

Lingua principale
Rust
Stelle
4
Fork
2
Merge medio
1g 1h
PR unite (30g)
225

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di cristim/AztecCraft

Tutte le issue di cristim/AztecCraft

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.