Skip to content

Commit 437b4dd

Browse files
LZY3538claude
andcommitted
chore(session-fork): satisfy no-promise-executor-return in timeout helpers
Wrap the setTimeout calls in the deadline/sleep Promise executors in block bodies so the executor no longer returns the timer handle (oxlint no-promise-executor-return). Co-Authored-By: Claude <[email protected]>
1 parent 613e2f7 commit 437b4dd

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

scripts/proof-session-fork-timeout.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,10 @@ import path from "node:path";
2525
// ── Inject filesystem latency (simulate a slow mount) ────────────────
2626
let openDelayMs = 0;
2727
let statDelayMs = 0;
28-
const sleep = (ms: number): Promise<void> => new Promise((resolve) => setTimeout(resolve, ms));
28+
const sleep = (ms: number): Promise<void> =>
29+
new Promise((resolve) => {
30+
setTimeout(resolve, ms);
31+
});
2932

3033
const realOpen = nodeFs.promises.open.bind(nodeFs.promises);
3134
// The bounded transcript tail read opens the file handle here.

src/auto-reply/reply/session-fork.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -336,7 +336,9 @@ type ParentForkTokenResolution =
336336
| { status: "unresolved" };
337337

338338
function resolveAfter<T>(value: T, ms: number): Promise<T> {
339-
return new Promise<T>((resolve) => setTimeout(() => resolve(value), ms));
339+
return new Promise<T>((resolve) => {
340+
setTimeout(() => resolve(value), ms);
341+
});
340342
}
341343

342344
async function resolveParentForkTokenCount(params: {

0 commit comments

Comments
 (0)