Skip to content

fix: correct powershell import casing#570

Merged
ximu3 merged 1 commit intoximu3:mainfrom
qianxuu:fix/typecheck-casing
May 3, 2026
Merged

fix: correct powershell import casing#570
ximu3 merged 1 commit intoximu3:mainfrom
qianxuu:fix/typecheck-casing

Conversation

@qianxuu
Copy link
Copy Markdown
Contributor

@qianxuu qianxuu commented Apr 11, 2026

修复 typecheck 和 build 报错

Copilot AI review requested due to automatic review settings April 11, 2026 15:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes TypeScript typecheck/build failures caused by incorrect import/export path casing for the PowerShell utility module (important on case-sensitive filesystems).

Changes:

  • Update src/main/utils/index.ts to export from ./powershell (lowercase) instead of ./Powershell.
  • Update src/main/utils/common.ts to import psManager from ./powershell (lowercase) instead of ./Powershell.

Reviewed changes

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

File Description
src/main/utils/index.ts Fixes barrel export path casing to match powershell.ts.
src/main/utils/common.ts Fixes psManager import path casing to match powershell.ts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fishyy119
Copy link
Copy Markdown
Contributor

该修复曾在#364 引入又被作者在之后回退。我估计是因为作者的本地仓库此处的文件名是大小写混合的,为了检查通过又改了回来。而其他人直接云端拉下来的仓库是纯小写的。

造成这种现象的原因主要是Windows系统对文件名大小写不敏感,这导致云端与直接拉下来的仓库对应文件名都是纯小写的,但是本地文件改名为大小写混合后不会被git检测到更改。这种不一致的大小写会被检查器识别到。

我认为此处也确实值得统一,无论是修改仓库的文件命名还是修改导入处的文件名。

此条留言只是作为一个提醒:这其中有这样一个大小写不敏感的问题。

@ximu3 ximu3 merged commit a3d0e48 into ximu3:main May 3, 2026
3 of 4 checks passed
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.

4 participants