[break]Resolve incorrect data structure in dump#542
Merged
Conversation
* refactor: use lodash/get and lodash/set directly instead of reimplementation - Replace hand-written get/set in dump/utils.ts with lodash/get and lodash/set - Import lodash sub-paths directly at call sites, bypassing scene-bundle's lodash stub - Fix removeComponent callers to pass Component objects instead of uuid strings * fix: pass Component object instead of uuid string to removeComponent removeComponent(component: Component) expects a Component object, but three callers were passing uuid strings. Fix by passing the Component object directly where available, and querying it first where only a uuid is available. * refactor: move scene i18n to main process with dynamic registration - Move i18n files from scene-process/i18n/ to scene/i18n/ - Load scene i18n via registerLanguagePatch in main process init - Remove synchronous require-based loading from worker process - Fix: scene i18n was registered in worker process but translateDumpI18n calls main process via RPC, so translations were never reachable * fix: default project type should be 3D, not 2D
* Add path property to dump data for inspector setProperty - Make IProperty.path required, filled during encode instead of by consumers - Add path to INode (node tree path) and IScene (hardcoded '/') - Fill property-level path in encodeNode (top-level + __comps__) and encodeScene (top-level + _globals) for decodePatch compatibility - Bump cocos-cli-types to 0.0.1-alpha.24.4 * Update dts snapshot to reflect optimized prefab interface Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Add unit tests for dump path property filling --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
qiuguohua
changed the title
knoxHuang
approved these changes
May 14, 2026
star-e
approved these changes
May 14, 2026
bofeng-song
reviewed
May 14, 2026
| // 节点相关的接口 | ||
| Node: NodeProxy, | ||
| // 组件相关的接口 | ||
| Component: ComponentProxy, |
Contributor
There was a problem hiding this comment.
为什么只有node，component有别名？
Contributor
Author
There was a problem hiding this comment.
其他接口涉及的比较少。后面优化在统一改一版
bofeng-song
reviewed
May 14, 2026
| Animation, | ||
| AudioSource, | ||
| Label, | ||
| MeshRenderer, |
bofeng-song
approved these changes
May 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.