fix(ecs): getZipper([]) and getIndexedZipper([]) loop up to SIZE_MAX
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 2/5
- 預估耗時
- 1-3 小時
- 新手友好度
- 74/100
- Issue 類型
- 缺陷
- 描述清晰度
- 描述清楚
- 活躍度
- 活躍
- 技術堆疊
- cpp, typescript, wasm
研究方向
閱讀 modules/ecs/wasm/Registry.hpp:300-322 和 338-360 中的兩個函式，重點關注空元件清單如何設定 max，以及迴圈如何建立 JavaScript 物件。檢查 modules/ecs/test 中相關的 ECS 測試；issue 指出目前沒有 zipper 測試。完成標準是：兩個傳入空元件清單的 zipper 呼叫都傳回空陣列，std::size_t 的拼寫錯誤獲得修正，且測試涵蓋此行為。
由索引模型根據 Issue 內容生成。
描述
Which package is this bug report for?
ecs
Issue description
With no component in the list, max stays SIZE_MAX, every index passes, and a JS object is pushed for each one: one call hangs the tab or the server and fills memory. std:size_t (lines 309 and 347) is a typo that compiles as a label. The two functions are near copies.
Where:
Evidence:
std::size_t max = SIZE_MAX;
...
std:size_t zipper_idx = 0;
for (std::size_t idx = 0; idx < max; idx++) {
emscripten::val obj = emscripten::val::object();
Suggested fix: Return an empty array when comps is empty, fix the std::size_t typo, and merge the two zippers behind an indexed flag. Add zipper tests (modules/ecs/test has none).
Found during a code audit of main at be72076.
Code sample
No response
Versions
- Engine
mainat 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
- 主要語言
- TypeScript
- 星號
- 7
- 分支
- 2
- 平均合併
- 1 天 17 小時
- 30 天內合併 PR
- 17
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
NanoForge-dev/Engine 的其他 Issue
-
dependencies enhancement good first issue
難度 2/5 1-3 小時 新手友好度 78/100
NanoForge-dev/Engine#513 ·
維護者通常 1 天內回覆
-
bug chore tests
難度 2/5 1-3 小時 新手友好度 77/100
NanoForge-dev/Engine#511 ·
維護者通常 1 天內回覆
-
docs(network): TSDoc says ICE_SERVERS defaults to an empty list, the code uses public STUN servers未關閉documentation good first issue modules:network
難度 1/5 1 小時以內 新手友好度 88/100
NanoForge-dev/Engine#509 ·
維護者通常 1 天內回覆
-
documentation modules:ecs packages:asset packages:core
難度 2/5 1-3 小時 新手友好度 82/100
NanoForge-dev/Engine#508 ·
維護者通常 1 天內回覆
-
bug good first issue modules:music modules:sound
難度 2/5 1-3 小時 新手友好度 78/100
NanoForge-dev/Engine#505 ·
維護者通常 1 天內回覆
查看 NanoForge-dev/Engine 的全部 Issue
相似的 Issue
-
Mend: dependency security vulnerability
難度 1/5 1-3 小時 新手友好度 65/100
fdm-monster/fdm-monster#5563 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
難度 2/5 1-3 小時 新手友好度 79/100
維護者通常 1 天內回覆
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
難度 2/5 1-3 小時 新手友好度 65/100
growilabs/growi#12246 · 3 則留言 ·
維護者通常 1 天內回覆
-
🐛 The Rulekeeper - An index run that gives up on the lock ends Succeeded可能已有人在做 @github-actions 今天認領。 未關閉
難度 2/5 1-3 小時 新手友好度 72/100
SSWConsulting/SSW.Rules#2809 · 1 則留言 ·
維護者通常 1 天內回覆