Skip to content

fix(server): use fs-extra default import compat#76

Merged
yxsj245 merged 1 commit intoGSManagerXZ:featurefrom
hengwuming:feature/issue-75-mrpack-download-fix
Apr 5, 2026
Merged

fix(server): use fs-extra default import compat#76
yxsj245 merged 1 commit intoGSManagerXZ:featurefrom
hengwuming:feature/issue-75-mrpack-download-fix

Conversation

@hengwuming
Copy link
Copy Markdown
Contributor

Issue: #75

Reason:

  • mrpack deployment could fail under ESM with "fs.readFile is not a function"
  • importing fs-extra via is not reliable in this runtime

Changes:

  • add a shared fs-extra compatibility wrapper
  • switch mrpack and related deployment modules to use the wrapper

Refs: #75

Issue: #75

Reason:
- mrpack deployment could fail under ESM with "fs.readFile is not a function"
- importing fs-extra via  is not reliable in this runtime

Changes:
- add a shared fs-extra compatibility wrapper
- switch mrpack and related deployment modules to use the wrapper

Refs: #75
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2026

⚠ 本 PR 已进入自动化审查流程,正在进行代码审查,请稍后

本次变更文件 (5 个):

  • server/src/modules/game/othergame/factorio-deployer.ts (modified)
  • server/src/modules/game/othergame/minecraft-server-api.ts (modified)
  • server/src/modules/game/othergame/mrpack-server-api.ts (modified)
  • server/src/modules/game/othergame/unified-functions.ts (modified)
  • server/src/utils/fsExtraCompat.ts (added)

正在进行变更类型识别与 TypeScript 语法检查...

@yxsj245 yxsj245 changed the base branch from main to feature April 5, 2026 05:00
@yxsj245 yxsj245 merged commit 08aa5bb into GSManagerXZ:feature Apr 5, 2026
1 check passed
@yxsj245
Copy link
Copy Markdown
Collaborator

yxsj245 commented Apr 5, 2026

✅ 您的 PR 已通过自动化检查,现已合并至 feature测试分支。
维护者将进行最终检查,若无问题会合并到 main 分支并在下个版本发布时包含。感谢贡献!❤

@yxsj245
Copy link
Copy Markdown
Collaborator

yxsj245 commented Apr 5, 2026

您的pr已通过最终审核 感谢贡献 我们将在下周四发布更新

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants