Fix adding-asset from asset-library

[nighca]

There may be assets with field builder_id in asset-library, which are added by Spritehub rather than Builder.

Now we do control includeID: bool during both export and load processes, allowing us to exclude builder_id when adding asset from asset-library, to avoid issues caused by duplicate ID.

[qiniu-prow]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[qiniu-ci]

This PR has been deployed to the preview environment. You can explore it using the preview URL.

Warning

Please note that deployments in the preview environment are temporary and will be automatically cleaned up after a certain period. Make sure to explore it before it is removed. For any questions, contact the Go+ Builder team.

Copilot reviewed 5 out of 9 changed files in this pull request and generated no comments.

Files not reviewed (4)

• spx-gui/src/models/project/index.ts: Evaluated as low risk
• spx-gui/src/models/sound.ts: Evaluated as low risk
• spx-gui/src/models/sprite.test.ts: Evaluated as low risk
• spx-gui/src/models/sprite.ts: Evaluated as low risk

Comments suppressed due to low confidence (4)

spx-gui/src/models/backdrop.ts:51

• Ensure that the new includeId behavior is covered by tests.

{ includeId = true }: BackdropExportLoadOptions

spx-gui/src/models/common/asset.ts:31

• The getFiles function should be awaited if it is asynchronous. Ensure that getFiles is awaited to avoid potential issues.

const files = getFiles(assetData.files)

spx-gui/src/models/common/asset.ts:57

• The getFiles function should be awaited if it is asynchronous. Ensure that getFiles is awaited to avoid potential issues.

const files = getFiles(assetData.files)

spx-gui/src/models/common/asset.ts:75

• The getFiles function should be awaited if it is asynchronous. Ensure that getFiles is awaited to avoid potential issues.

const files = getFiles(assetData.files)