fix(ecs): a component instance with a string name field is stored under that value

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

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
74/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
cpp, typescript
Área
backend, game-dev

Línea de trabajo

Start with modules/ecs/wasm/Utils.cpp:34-43 and trace how get_js_class_name chooses a component key; check modules/ecs/src/shared/editor-world.ts:293, 300 for the editor tracker’s use of the same behavior. Reproduce with a component instance that has a string name field, then verify instances are stored under their class name and getZipper([Tag]) finds them.

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

Descripción

bug modules:ecs
Which package is this bug report for?

ecs

Issue description

get_js_class_name reads c.name before c.constructor.name. For an instance new Tag("player") whose class has a name field, the component array key is "player", not "Tag", and each distinct value creates its own array, so getZipper([Tag]) never finds it. Common Name / Tag components are silently stored in the wrong array. The editor tracker relies on the same behavior (editor-world.ts:293, 300).

Where:

Evidence:

return get_js_class_type_name(c).value_or(get_js_class_var_name(c).value_or(UNKNOWN_COMPONENT_TYPE));
...
const std::optional<emscripten::val> name = get_js_member(c, "name");

Suggested fix: Use .name only when c is a function (a class) or a plain { name } descriptor; for instances use constructor.name.

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

Medium (should be fixed soon)

I have tested this issue on a development release

be72076

Lenguaje dominante
TypeScript
Estrellas
7
Forks
2
Merge medio
1 d 17 h
PR fusionados (30 d)
17

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 NanoForge-dev/Engine

Todos los issues de NanoForge-dev/Engine

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.