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

Abierto
  • ¿Algo no está bien?
#415 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@cristim ya está trabajando en esto.

Desde el 9/11/2026.

  • #425 de @cristim — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
30/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
rust
Área
desktop

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
Rust
Estrellas
4
Forks
2
Merge medio
1 d 1 h
PR fusionados (30 d)
225

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de cristim/AztecCraft

Todos los issues de cristim/AztecCraft

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.