[Bug Fix / PR] Fix TypeError: Cannot mix BigInt and other types on BigIntStats in fs-local probe

[suyuhao1206]

Branch: https://github.com/suyuhao1206/deepseek-harness/tree/fix/fs-mode-bigint-conversion
Commit: suyuhao1206@9a5c288a00

[PerryLink]

⚠️ 本仓不接受外部 PR——请把 diff 直接贴进这条讨论（并去 #8842 报到）

1. 先说你这条最重要的一件事

你给的是你自己 fork 上的分支与提交：

Branch: …/deepseek-harness/tree/fix/fs-mode-bigint-conversion
Commit: …/commit/9a5c288a00

而这个仓库的协作方式不是 PR：

• Issues 已关闭（只有 Discussions）——你在报告里应该已经注意到；
• CONTRIBUTING.md 明确写着 "we are sorry that we cannot accept external pull requests at the moment"。

⇒ 也就是说：即使你在 fork 上把补丁写好，它也不会通过 PR 被合并。建议把改动直接内联进这条讨论（diff 片段 + 说明），这在这里就是有效的提交形态——维护者看到 diff 就能自己落地。只给分支链接，等于把可用的东西放在了没人会去的地方。

2. ⇒ 请把这两样贴上（决定这条能否被采纳）

1. git diff 原文（改动的文件、函数、前后对照）——它必须能被人在同一页里读完；
2. 一句话根因：fs-local 探测里把 BigInt 版 stat 的字段与 Number 做了算术（比较不抛、算术才抛）⇒ 改成显式转换（Number(...)）或统一用非 bigint 形态。

3. 你要去 #8842 报到（同一天、同一缺陷）

同期另有一条独立报告：#8842「Every path inside app.asar fails: Cannot mix BigInt and other types」——同一个 TypeError，但它来自用户视角（asar 内路径 ⇒ 本地技能全部消失）。

⇒ 我在那边已经建议"与 #8815 合并"；现在反过来请你在这条里引用 #8842，并把你这条定位为"修复侧"：

• #8842 提供触发面与影响（asar 路径 ⇒ 功能整块失效）；
• 你这条提供修法。

两条合起来才是一份完整报告。分开的话，最常见的结局是各修一半。

4. 请补三样

1. 为什么 asar 路径才会触发（普通磁盘路径是否不触发）——这是你这条相对 #8842 独有的技术细节；
2. 改后如何验证（一条能跑的命令或断言）⇒ 让维护者能独立确认，而不是只能读 diff；
3. 版本：请写明你改的是哪个版本（当前最新是 0.2.1-alpha.1）。

一条边界

我确认的是本仓不接受外部 PR 这一协作事实（CONTRIBUTING 原文 + Issues 关闭）。你的 diff 是否正确我没有看（只给了链接，我无法在不下载分支的情况下核对）⇒ 请按第 2 节内联，我下一轮可以对着 HEAD 核一遍。

[suyuhao1206]

感谢指引！本帖作为 #8842 的修复侧（#8842 提供触发面与用户侧影响，本帖 #8815 提供修法与验证），现按要求将完整 git diff 原文及全部细节内联如下：

1. 版本与关联报告

• 修改版本：0.2.1-alpha.1（当前最新 master HEAD 5badb15009；桌面端触发环境为 0.2.0-rc.2）
• 关联报告：配合 [Bug] Every path inside `app.asar` fails: "Cannot mix BigInt and other types" (`fs-local` probe assumes BigInt stats) #8842（[Bug] Every path inside `app.asar` fails: "Cannot mix BigInt and other types" (`fs-local` probe assumes BigInt stats) #8842 报告了 app.asar 路径触发 Cannot mix BigInt 导致本地技能消失与内置文件无法读取；本帖提供 packages/fs/fs-local 的具体修复 diff 与验证）。

2. 一句话根因

packages/fs/fs-local/src/fsio.ts 的 probe()（第 252 行）与 probeNoFollow()（第 268 行）调用 stat/lstat(path, { bigint: true }) 后直接执行了 Number(info.mode & 0o777n)，拿 info.mode 与 BigInt 字面量 0o777n 做按位与算术运算（比较不抛、算术才抛）⇒ 改为先显式转换 Number(info.mode) & 0o777，并在 versionOf()（第 76 行）对缺失的 mtimeNs/ctimeNs 回退到 mtimeMs/ctimeMs。

3. 为什么只有 app.asar 路径才会触发（普通磁盘路径不触发）？

• 普通磁盘路径（不触发）：走 Node.js 原生 libuv 的 fs.stat(path, { bigint: true })，返回 BigIntStats 实例（info.constructor.name === 'BigIntStats'），其 info.mode 是 bigint（如 33188n），mtimeNs/ctimeNs 也是 bigint，因此 info.mode & 0o777n 都是 BigInt 运算，不会抛错。
• app.asar 内路径（必现触发）：Electron 的虚拟文件系统层（lib/asar/fs-wrapper）拦截了 fs.stat / fs.lstat，在归档内路径上忽略了 { bigint: true } 参数，直接返回普通的 fs.Stats 实例（info.constructor.name === 'Stats'）：
  • typeof info.mode === 'number'（如 33152）
  • typeof info.size === 'number'
  • info.mtimeNs === undefined、info.ctimeNs === undefined
    当 probe() 拿到这个普通 number 的 info.mode 去和 0o777n 做 & 运算时，立即抛出 TypeError: Cannot mix BigInt and other types, use explicit conversions。
• 附加技术细节：不能在 probeStats 里把 stats.mode 强行改写为 BigInt(stats.mode)，因为 asar 返回的对象原型是普通 Stats.prototype，其 info.isFile() 内部执行的是 (this.mode & S_IFMT) === property（S_IFMT 是 number）。把 mode 改成 BigInt 反而会让 info.isFile() 抛同样的错。所以在 probe() / probeNoFollow() 掩码处改为 Number(info.mode) & 0o777 是唯一同时兼容 Stats 与 BigIntStats 的修法。

4. git diff 原文（基于 0.2.1-alpha.1 / HEAD 5badb15009）

diff --git a/packages/fs/fs-local/src/fsio.ts b/packages/fs/fs-local/src/fsio.ts
index a9d97fd6ac..9aba4255ae 100644
--- a/packages/fs/fs-local/src/fsio.ts
+++ b/packages/fs/fs-local/src/fsio.ts
@@ -72,8 +72,10 @@ async function readFileAbortable(absolutePath: string, verb: 'read' | 'edit', si
 }
 
 /** Opaque version token from high-resolution identity and freshness metadata. */
-function versionOf(info: BigIntStats): FsVersion {
-  return FsVersion(`${info.dev}:${info.ino}:${info.size}:${info.mtimeNs}:${info.ctimeNs}`)
+function versionOf(info: Stats | BigIntStats): FsVersion {
+  const mtime = 'mtimeNs' in info ? info.mtimeNs : info.mtimeMs
+  const ctime = 'ctimeNs' in info ? info.ctimeNs : info.ctimeMs
+  return FsVersion(`${info.dev}:${info.ino}:${info.size}:${mtime}:${ctime}`)
 }
 
 /**
@@ -95,6 +97,10 @@ export interface FsIoInternals {
   linkFile?: (existingPath: string, newPath: string) => Promise<void>
   /** Override target inspection after guarded publication fails. */
   inspectPublicationTarget?: (path: string) => Promise<BigIntStats>
+  /** Override stat for unit coverage. */
+  stat?: (path: string) => Promise<Stats | BigIntStats>
+  /** Override lstat for unit coverage. */
+  lstat?: (path: string) => Promise<Stats | BigIntStats>
   /** Override staging-directory removal for commit-point failure coverage. */
   removeStagingDir?: (stagingDir: string) => Promise<void>
   /** Test hook after the temp file is written/synced but before final chmod+publication. */
@@ -242,14 +248,16 @@ async function probeStats<T extends Stats | BigIntStats>(
 /**
  * Probe a path for its version, mode, type, and size. Null if absent.
  * @param absolutePath - the path to stat (typically a target key; symlinks are followed).
+ * @param internals - test hook to override operations like stat.
  * @returns the metadata, or null when the path — or a parent segment — does not exist.
  */
-export async function probe(absolutePath: string): Promise<PathInfo | null> {
-  const info = await probeStats(absolutePath, path => stat(path, { bigint: true }))
+export async function probe(absolutePath: string, internals: FsIoInternals = {}): Promise<PathInfo | null> {
+  const statFn = internals.stat ?? (path => stat(path, { bigint: true }))
+  const info = await probeStats(absolutePath, statFn)
   if (!info) return null
   return {
     version: versionOf(info),
-    mode: Number(info.mode & 0o777n),
+    mode: Number(info.mode) & 0o777,
     type: pathType(info),
     size: Number(info.size),
   }
@@ -258,14 +266,16 @@ export async function probe(absolutePath: string): Promise<PathInfo | null> {
 /**
  * Probe a path without following the final symlink component.
  * @param absolutePath - the path entry to inspect with `lstat` semantics.
+ * @param internals - test hook to override operations like lstat.
  * @returns path-entry metadata, or null when the entry is absent.
  */
-export async function probeNoFollow(absolutePath: string): Promise<PathLinkInfo | null> {
-  const info = await probeStats(absolutePath, path => lstat(path, { bigint: true }))
+export async function probeNoFollow(absolutePath: string, internals: FsIoInternals = {}): Promise<PathLinkInfo | null> {
+  const lstatFn = internals.lstat ?? (path => lstat(path, { bigint: true }))
+  const info = await probeStats(absolutePath, lstatFn)
   if (!info) return null
   return {
     version: versionOf(info),
-    mode: Number(info.mode & 0o777n),
+    mode: Number(info.mode) & 0o777,
     type: pathLinkType(info),
     size: Number(info.size),
   }
diff --git a/packages/fs/fs-local/tests/fsio.spec.ts b/packages/fs/fs-local/tests/fsio.spec.ts
index 40ed836309..dc24b61f3b 100644
--- a/packages/fs/fs-local/tests/fsio.spec.ts
+++ b/packages/fs/fs-local/tests/fsio.spec.ts
@@ -6,7 +6,7 @@
  */
 
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { chmod, mkdtemp, readFile, rename, rm, stat, symlink, unlink, writeFile, mkdir, readdir, realpath } from 'node:fs/promises'
+import { chmod, lstat, mkdtemp, readFile, rename, rm, stat, symlink, unlink, writeFile, mkdir, readdir, realpath } from 'node:fs/promises'
@@ -191,10 +191,21 @@ describe('probe', () => {
     await writeFile(join(dir, 'afile'), 'i am a file')
     expect(await probe(join(dir, 'afile', 'child.txt'))).toBeNull()
   })
+
+  it('handles a stats object where mode is a number instead of a bigint (e.g. Electron asar)', async () => {
+    const file = join(dir, 'number-mode.txt')
+    await writeFile(file, 'hello')
+    const plainStats = await stat(file)
+    const info = await probe(file, { stat: async () => plainStats })
+    expect(info?.type).toBe('file')
+    expect(info?.mode).toBe(plainStats.mode & 0o777)
+    expect(typeof info?.version).toBe('string')
+    expect(info?.version).not.toContain('undefined')
+  })
 })
 
 describe('probeNoFollow', () => {
@@ -207,6 +218,17 @@ describe('probeNoFollow', () => {
     expect(linkInfo?.size).toBeGreaterThan(0)
   })
 
+  it('handles a stats object where mode is a number instead of a bigint in probeNoFollow', async () => {
+    const file = join(dir, 'number-mode-nofollow.txt')
+    await writeFile(file, 'hello')
+    const plainStats = await lstat(file)
+    const info = await probeNoFollow(file, { lstat: async () => plainStats })
+    expect(info?.type).toBe('file')
+    expect(info?.mode).toBe(plainStats.mode & 0o777)
+    expect(typeof info?.version).toBe('string')
+    expect(info?.version).not.toContain('undefined')
+  })

5. 改后如何验证（可独立运行的命令与断言）

1. 运行新增的单元测试断言（将非 bigint 的 Stats 注入 probe 与 probeNoFollow，验证不抛错且 mode 与 version 正常）：

pnpm exec vitest run packages/fs/fs-local/tests/fsio.spec.ts -t "number instead of a bigint"

2. 用 Electron 二进制对 app.asar 路径与普通磁盘路径做实机对比验证：

ELECTRON_RUN_AS_NODE=1 "D:\app install\DeepSeek Harness\DeepSeek Harness.exe" -e "const fs = require('fs'); for (const p of ['D:\\app install\\DeepSeek Harness\\resources\\app.asar\\dsh\\package.json', 'package.json']) { const info = fs.statSync(p, { bigint: true }); console.log({ constructor: info.constructor.name, isFile: info.isFile(), mode: Number(info.mode) & 0o777 }); }"

验证输出：

{ constructor: 'Stats', isFile: true, mode: 384 }
{ constructor: 'BigIntStats', isFile: true, mode: 438 }