diff --git a/src/web/routes/edit-history-routes.ts b/src/web/routes/edit-history-routes.ts index 227a16a1..7a156fca 100644 --- a/src/web/routes/edit-history-routes.ts +++ b/src/web/routes/edit-history-routes.ts @@ -7,8 +7,8 @@ * POST /api/sessions/:id/edits/older read the next older block, then list * * Read-only, and reads only on these requests (the panel drives it; nothing syncs in the - * background). Remote, Docker and non-claude sessions answer `state: 'unsupported'` - * without being read. Ownership goes through `findSessionOrFail` with `req`, like every + * background). A remote session is read over ssh and never woken; Docker and non-claude + * sessions answer `state: 'unsupported'` without being read. Ownership goes through `findSessionOrFail` with `req`, like every * session-scoped route. No route here serves a file: an image entry is shown through the * existing file routes (workspace `file-raw`, or the attachment routes outside it). */ @@ -86,7 +86,7 @@ function listOf( return list; } -/** The answer for a session that is never read (remote, Docker, non-claude): no store involved. */ +/** The answer for a session that is never read (Docker, non-claude): no store involved. */ function unsupportedList(reason: string): EditHistoryList { return { state: 'unsupported', reason, backfillTruncated: false, agents: {}, entries: [] }; } diff --git a/test/edit-history-probe-stall.test.ts b/test/edit-history-probe-stall.test.ts index 5c0dd54b..c4332d24 100644 --- a/test/edit-history-probe-stall.test.ts +++ b/test/edit-history-probe-stall.test.ts @@ -79,7 +79,7 @@ describe('edit history image probe on an unresponsive mount', () => { const sync = new EditHistorySync({ transcriptPath: async () => '/p/conv1.jsonl', - readSources: async (_path, requests, budget): Promise => { + readSources: async (_session, _path, requests, budget): Promise => { const req = requests.find((r) => r.relPath === 'main'); const start = Math.min(req?.offset ?? 0, main.length); const len = Math.min(budget, main.length - start); @@ -89,7 +89,7 @@ describe('edit history image probe on an unresponsive mount', () => { metas: {}, }; }, - probe: (paths) => probeLocalPaths(paths), + probe: (_session, paths) => probeLocalPaths(paths), loadStore: (id) => EditHistoryStore.load(id, { dir: stores, debounceMs: 0 }), deleteStore: (id) => EditHistoryStore.delete(id, { dir: stores }), sweepStores: async () => 0, diff --git a/test/edit-history-sync.test.ts b/test/edit-history-sync.test.ts index d1755e1a..915eca82 100644 --- a/test/edit-history-sync.test.ts +++ b/test/edit-history-sync.test.ts @@ -249,11 +249,11 @@ describe('EditHistorySync', () => { }); it('a removal during a read stops it: no further parsing, no probe of printed paths', async () => { - const probe = vi.fn(async (paths: readonly string[]) => paths.map(() => null)); + const probe = vi.fn(async (_session: unknown, paths: readonly string[]) => paths.map(() => null)); let removal: Promise | undefined; const ctx = setup({ probe, - readSources: async (path, req, budget, opts) => { + readSources: async (_session, path, req, budget, opts) => { const res = await ctx.host.read(path, req, budget, opts); removal = ctx.sync.remove('s1'); // the session is deleted while this read is out return res;