liuyu/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-06-06 05:51:15 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(file-transfer): centralize dir-list page token parsing

Browse Source
This commit is contained in:
Peter Steinberger
2026-05-29 07:11:21 -04:00
parent 92c1547a89
commit 1951413a0b
2 changed files with 15 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
extensions/file-transfer/src/node-host/dir-list.test.ts
View File

@@ -144,6 +144,19 @@ describe("handleDirList — happy path", () => {
expect(r.entries.map((e) => e.name)).toEqual(["f-0.txt"]);
expect(r.nextPageToken).toBe("1");
});
it("accepts plus-signed page tokens", async () => {
for (let i = 0; i < 3; i++) {
await fs.writeFile(path.join(tmpRoot, `f-${i}.txt`), "x");
}
const r = await handleDirList({ path: tmpRoot, maxEntries: 1, pageToken: "+01" });
if (!r.ok) {
throw new Error("expected ok");
}
expect(r.entries.map((e) => e.name)).toEqual(["f-1.txt"]);
expect(r.nextPageToken).toBe("2");
});
});
describe("handleDirList — limits", () => {

8
extensions/file-transfer/src/node-host/dir-list.ts
View File

@@ -1,4 +1,5 @@
import path from "node:path";
import { parseStrictNonNegativeInteger } from "openclaw/plugin-sdk/number-runtime";
import { root } from "openclaw/plugin-sdk/security-runtime";
import { mimeFromExtension } from "../shared/mime.js";
import {
@@ -63,12 +64,7 @@ function parsePageOffset(input: unknown): number {
if (typeof input !== "string") {
return 0;
}
const trimmed = input.trim();
if (!/^\d+$/.test(trimmed)) {
return 0;
}
const offset = Number(trimmed);
return Number.isSafeInteger(offset) ? offset : 0;
return parseStrictNonNegativeInteger(input) ?? 0;
}
function classifyFsError(err: unknown): DirListErrCode {