@fgv/ts-web-extras 5.1.0-0 → 5.1.0-2

package/lib/test/unit/httpTreeAccessors.test.js

@@ -0,0 +1,1231 @@
+"use strict";
+/*
+ * Copyright (c) 2026 Erik Fortune
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in all
+ * copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+require("@fgv/ts-utils-jest");
+const ts_utils_1 = require("@fgv/ts-utils");
+const file_tree_1 = require("../../packlets/file-tree");
+function makeMockResponse(options) {
+    const { ok, status = ok ? 200 : 400, jsonValue, textValue, throwOnJson, throwOnText } = options;
+    return {
+        ok,
+        status,
+        statusText: ok ? 'OK' : `Error ${status}`,
+        json: throwOnJson
+            ? () => Promise.reject(new Error('JSON parse error'))
+            : () => Promise.resolve(jsonValue),
+        text: throwOnText
+            ? () => Promise.reject(new Error('text error'))
+            : () => Promise.resolve(textValue !== null && textValue !== void 0 ? textValue : `HTTP ${status}`)
+    };
+}
+/**
+ * Creates a mock fetch function that returns responses for each call in order.
+ * Each entry maps to one fetch() invocation in call order.
+ */
+function makeMockFetch(responses) {
+    const calls = [];
+    let callIndex = 0;
+    const fetchImpl = (url, init) => {
+        calls.push({ url: url.toString(), init });
+        const response = responses[callIndex++];
+        if (response === undefined) {
+            return Promise.reject(new Error(`Unexpected fetch call #${callIndex} to ${url.toString()}`));
+        }
+        return Promise.resolve(makeMockResponse(response));
+    };
+    return { fetchImpl: fetchImpl, calls };
+}
+/**
+ * Creates a mock fetch function that throws a network error on every call.
+ */
+function makeThrowingFetch(message) {
+    return (_url, _init) => {
+        return Promise.reject(new Error(message));
+    };
+}
+// ---- Shared test data builders ----
+/** Minimal tree-children response for a single file at root. */
+function rootWithOneFile(fileName = 'data.json') {
+    return {
+        path: '/',
+        children: [{ path: `/${fileName}`, name: fileName, type: 'file' }]
+    };
+}
+/** File response for a JSON file. */
+function fileResponse(path, contents, contentType) {
+    const response = { path, contents };
+    if (contentType !== undefined) {
+        response.contentType = contentType;
+    }
+    return response;
+}
+/** Root children response containing a subdirectory and a file. */
+function rootWithDirAndFile() {
+    return {
+        path: '/',
+        children: [
+            { path: '/subdir', name: 'subdir', type: 'directory' },
+            { path: '/root.json', name: 'root.json', type: 'file' }
+        ]
+    };
+}
+/** Children response for /subdir containing one file. */
+function subdirWithOneFile() {
+    return {
+        path: '/subdir',
+        children: [{ path: '/subdir/nested.json', name: 'nested.json', type: 'file' }]
+    };
+}
+// ---- Tests ----
+describe('HttpTreeAccessors', () => {
+    describe('fromHttp()', () => {
+        test('succeeds with an empty directory (no children)', async () => {
+            const { fetchImpl } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toSucceedAndSatisfy((accessors) => {
+                expect(accessors).toBeInstanceOf(file_tree_1.HttpTreeAccessors);
+                expect(accessors.isDirty()).toBe(false);
+            });
+        });
+        test('succeeds and loads a single file at root', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{"items":{}}') }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            });
+            expect(result).toSucceedAndSatisfy((accessors) => {
+                expect(accessors.getFileContents('/data.json')).toSucceedWith('{"items":{}}');
+            });
+        });
+        test('succeeds with multiple files at root', async () => {
+            const { fetchImpl } = makeMockFetch([
+                {
+                    ok: true,
+                    jsonValue: {
+                        path: '/',
+                        children: [
+                            { path: '/alpha.json', name: 'alpha.json', type: 'file' },
+                            { path: '/beta.json', name: 'beta.json', type: 'file' }
+                        ]
+                    }
+                },
+                { ok: true, jsonValue: fileResponse('/alpha.json', '"alpha"') },
+                { ok: true, jsonValue: fileResponse('/beta.json', '"beta"') }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            });
+            expect(result).toSucceedAndSatisfy((accessors) => {
+                expect(accessors.getFileContents('/alpha.json')).toSucceedWith('"alpha"');
+                expect(accessors.getFileContents('/beta.json')).toSucceedWith('"beta"');
+            });
+        });
+        test('succeeds and recursively loads files in subdirectories', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithDirAndFile() },
+                { ok: true, jsonValue: subdirWithOneFile() },
+                { ok: true, jsonValue: fileResponse('/subdir/nested.json', '"nested"') },
+                { ok: true, jsonValue: fileResponse('/root.json', '"root"') }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            });
+            expect(result).toSucceedAndSatisfy((accessors) => {
+                expect(accessors.getFileContents('/subdir/nested.json')).toSucceedWith('"nested"');
+                expect(accessors.getFileContents('/root.json')).toSucceedWith('"root"');
+            });
+        });
+        test('strips trailing slash from baseUrl', async () => {
+            const { fetchImpl, calls } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000/',
+                fetchImpl
+            });
+            // The URL should start with the base URL without a trailing slash, then /tree/children
+            expect(calls[0].url).toMatch(/^http:\/\/localhost:3000\/tree\/children/);
+            // There should be no double slash in the path portion (after the protocol)
+            const urlPath = calls[0].url.replace('http://', '');
+            expect(urlPath).not.toContain('//');
+        });
+        test('includes namespace in query parameters when provided', async () => {
+            const { fetchImpl, calls } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                namespace: 'my-namespace',
+                fetchImpl
+            });
+            expect(calls[0].url).toContain('namespace=my-namespace');
+        });
+        test('omits namespace from query parameters when not provided', async () => {
+            const { fetchImpl, calls } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(calls[0].url).not.toContain('namespace');
+        });
+        test('applies prefix parameter to loaded file paths', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{"items":{}}') }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                prefix: '/app',
+                fetchImpl,
+                mutable: true
+            });
+            expect(result).toSucceedAndSatisfy((accessors) => {
+                expect(accessors.getFileContents('/app/data.json')).toSucceedWith('{"items":{}}');
+            });
+        });
+        test('fails when the initial children fetch fails with a network error', async () => {
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl: makeThrowingFetch('Network unreachable')
+            });
+            expect(result).toFailWith(/network unreachable/i);
+        });
+        test('fails when the children response is not ok', async () => {
+            const { fetchImpl } = makeMockFetch([{ ok: false, status: 404, textValue: 'Not Found' }]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/not found/i);
+        });
+        test('fails when the children response contains invalid JSON', async () => {
+            const { fetchImpl } = makeMockFetch([{ ok: true, throwOnJson: true }]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/invalid json/i);
+        });
+        test('fails when a file fetch fails with a network error', async () => {
+            let callCount = 0;
+            const fetchImpl = (_url, _init) => {
+                callCount++;
+                if (callCount === 1) {
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                }
+                return Promise.reject(new Error('File fetch failed'));
+            };
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/file fetch failed/i);
+        });
+        test('fails when a file response is not ok', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: false, status: 403, textValue: 'Forbidden' }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/forbidden/i);
+        });
+        test('fails when a file response contains invalid JSON', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, throwOnJson: true }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/invalid json/i);
+        });
+        test('fails when a nested directory fetch fails', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithDirAndFile() },
+                { ok: false, status: 500, textValue: 'Internal Server Error' }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/internal server error/i);
+        });
+    });
+    describe('isDirty() and getDirtyPaths()', () => {
+        test('starts not dirty after loading', async () => {
+            const { fetchImpl } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({ baseUrl: 'http://localhost:3000', fetchImpl })).orThrow();
+            expect(accessors.isDirty()).toBe(false);
+            expect(accessors.getDirtyPaths()).toEqual([]);
+        });
+        test('becomes dirty after saveFileContents', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '{"modified":true}').orThrow();
+            expect(accessors.isDirty()).toBe(true);
+            expect(accessors.getDirtyPaths()).toContain('/data.json');
+        });
+        test('tracks multiple dirty files', async () => {
+            const { fetchImpl } = makeMockFetch([
+                {
+                    ok: true,
+                    jsonValue: {
+                        path: '/',
+                        children: [
+                            { path: '/a.json', name: 'a.json', type: 'file' },
+                            { path: '/b.json', name: 'b.json', type: 'file' }
+                        ]
+                    }
+                },
+                { ok: true, jsonValue: fileResponse('/a.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/b.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/a.json', '"a"').orThrow();
+            accessors.saveFileContents('/b.json', '"b"').orThrow();
+            expect(accessors.getDirtyPaths()).toHaveLength(2);
+            expect(accessors.getDirtyPaths()).toContain('/a.json');
+            expect(accessors.getDirtyPaths()).toContain('/b.json');
+        });
+        test('tracks deleted files as pending dirty paths', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.deleteFile('/data.json').orThrow();
+            expect(accessors.isDirty()).toBe(true);
+            expect(accessors.getDirtyPaths()).toContain('/data.json');
+        });
+    });
+    describe('saveFileContents()', () => {
+        test('fails when the file is not found (immutable tree has no matching path)', async () => {
+            const { fetchImpl } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({ baseUrl: 'http://localhost:3000', fetchImpl })).orThrow();
+            // With mutable: false (default), saveFileContents should fail
+            const result = accessors.saveFileContents('/missing.json', '{}');
+            expect(result).toFail();
+        });
+        test('marks the file as dirty without autoSync', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            const result = accessors.saveFileContents('/data.json', '{"updated":true}');
+            expect(result).toSucceedWith('{"updated":true}');
+            expect(accessors.isDirty()).toBe(true);
+            // No additional fetch calls should have occurred (no autoSync)
+        });
+        test('triggers fire-and-forget autoSync when autoSync is enabled', async () => {
+            const syncResponses = [
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                // PUT /file response for sync
+                { ok: true, jsonValue: { path: '/data.json', contents: '{"auto":true}' } },
+                // POST /sync response
+                { ok: true, jsonValue: { synced: 1 } }
+            ];
+            const { fetchImpl, calls } = makeMockFetch(syncResponses);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true,
+                autoSync: true
+            })).orThrow();
+            const result = accessors.saveFileContents('/data.json', '{"auto":true}');
+            expect(result).toSucceedWith('{"auto":true}');
+            // autoSync fires-and-forgets; wait for microtasks to drain
+            await new Promise((resolve) => setTimeout(resolve, 0));
+            // Verify that PUT + POST /sync were called
+            const methodCalls = calls.slice(2).map((c) => { var _a; return (_a = c.init) === null || _a === void 0 ? void 0 : _a.method; });
+            expect(methodCalls).toContain('PUT');
+            expect(methodCalls).toContain('POST');
+        });
+        test('logs when autoSync returns a failure Result', async () => {
+            const logger = {
+                detail: jest.fn(),
+                info: jest.fn(),
+                warn: jest.fn(),
+                error: jest.fn()
+            };
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true,
+                autoSync: true,
+                logger
+            })).orThrow();
+            accessors.syncToDisk =
+                async () => (0, ts_utils_1.fail)('simulated sync failure');
+            accessors.saveFileContents('/data.json', '{"auto":true}').orThrow();
+            await new Promise((resolve) => setTimeout(resolve, 0));
+            expect(logger.error).toHaveBeenCalledWith(expect.stringContaining('simulated sync failure'));
+        });
+        test('logs when autoSync throws unexpectedly', async () => {
+            const logger = {
+                detail: jest.fn(),
+                info: jest.fn(),
+                warn: jest.fn(),
+                error: jest.fn()
+            };
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true,
+                autoSync: true,
+                logger
+            })).orThrow();
+            accessors.syncToDisk = async () => {
+                throw new Error('simulated throw');
+            };
+            accessors.saveFileContents('/data.json', '{"auto":true}').orThrow();
+            await new Promise((resolve) => setTimeout(resolve, 0));
+            expect(logger.error).toHaveBeenCalledWith(expect.stringContaining('simulated throw'));
+        });
+    });
+    describe('fileIsMutable()', () => {
+        test('returns persistent detail when file exists and mutable is true', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            const result = accessors.fileIsMutable('/data.json');
+            expect(result.isSuccess()).toBe(true);
+            if (result.isSuccess()) {
+                expect(result.value).toBe(true);
+                expect(result.detail).toBe('persistent');
+            }
+        });
+        test('returns not-mutable detail when mutable is false', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: false
+            })).orThrow();
+            const result = accessors.fileIsMutable('/data.json');
+            expect(result.isFailure()).toBe(true);
+            if (result.isFailure()) {
+                expect(result.detail).toBe('not-mutable');
+            }
+        });
+        test('returns persistent detail for any path when mutable is true (no path existence check)', async () => {
+            // InMemoryTreeAccessors checks the mutable config, not path existence.
+            // HttpTreeAccessors layers "persistent" on top of a successful mutable check.
+            const { fetchImpl } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            // Any path succeeds as "persistent" when mutable: true
+            const result = accessors.fileIsMutable('/nonexistent.json');
+            expect(result.isSuccess()).toBe(true);
+            if (result.isSuccess()) {
+                expect(result.value).toBe(true);
+                expect(result.detail).toBe('persistent');
+            }
+        });
+    });
+    describe('syncToDisk()', () => {
+        test('succeeds immediately with no dirty files', async () => {
+            const { fetchImpl } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({ baseUrl: 'http://localhost:3000', fetchImpl })).orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toSucceed();
+        });
+        test('PUTs each dirty file then POSTs /sync', async () => {
+            var _a, _b;
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{"new":1}') },
+                { ok: true, jsonValue: { synced: 1 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '{"new":1}').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toSucceed();
+            const syncCalls = calls.slice(2);
+            expect(syncCalls[0].url).toContain('/file');
+            expect((_a = syncCalls[0].init) === null || _a === void 0 ? void 0 : _a.method).toBe('PUT');
+            expect(syncCalls[1].url).toContain('/sync');
+            expect((_b = syncCalls[1].init) === null || _b === void 0 ? void 0 : _b.method).toBe('POST');
+        });
+        test('clears dirty state after successful sync', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/data.json', '"updated"') },
+                { ok: true, jsonValue: { synced: 1 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            expect(accessors.isDirty()).toBe(true);
+            await accessors.syncToDisk();
+            expect(accessors.isDirty()).toBe(false);
+            expect(accessors.getDirtyPaths()).toEqual([]);
+        });
+        test('includes namespace in PUT body and POST /sync body when provided', async () => {
+            var _a, _b;
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/data.json', '"v2"') },
+                { ok: true, jsonValue: { synced: 1 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                namespace: 'myns',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"v2"').orThrow();
+            await accessors.syncToDisk();
+            const putBody = JSON.parse((_a = calls[2].init) === null || _a === void 0 ? void 0 : _a.body);
+            expect(putBody.namespace).toBe('myns');
+            const syncBody = JSON.parse((_b = calls[3].init) === null || _b === void 0 ? void 0 : _b.body);
+            expect(syncBody.namespace).toBe('myns');
+        });
+        test('omits namespace from PUT body and POST /sync body when not provided', async () => {
+            var _a, _b;
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/data.json', '"v2"') },
+                { ok: true, jsonValue: { synced: 1 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"v2"').orThrow();
+            await accessors.syncToDisk();
+            const putBody = JSON.parse((_a = calls[2].init) === null || _a === void 0 ? void 0 : _a.body);
+            expect(putBody.namespace).toBeUndefined();
+            const syncBody = JSON.parse((_b = calls[3].init) === null || _b === void 0 ? void 0 : _b.body);
+            expect(syncBody.namespace).toBeUndefined();
+        });
+        test('fails when PUT for a dirty file returns a non-ok response', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: false, status: 500, textValue: 'Server Error' }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toFailWith(/sync.*data\.json.*server error/i);
+        });
+        test('fails when PUT for a dirty file encounters a network error', async () => {
+            jest.useFakeTimers();
+            try {
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    return Promise.reject(new Error('PUT network error'));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"updated"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                await jest.advanceTimersByTimeAsync(1500);
+                const result = await syncPromise;
+                expect(result).toFailWith(/put network error/i);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+        test('fails when POST /sync returns a non-ok response', async () => {
+            jest.useFakeTimers();
+            try {
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    if (callCount === 3) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '"v2"') }));
+                    }
+                    // All /sync attempts: persistent 503
+                    return Promise.resolve(makeMockResponse({ ok: false, status: 503, textValue: 'Service Unavailable' }));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"v2"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                // Advance past both backoff delays: 500ms + 1000ms
+                await jest.advanceTimersByTimeAsync(1500);
+                const result = await syncPromise;
+                expect(result).toFailWith(/service unavailable/i);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+        test('fails when POST /sync encounters a network error', async () => {
+            jest.useFakeTimers();
+            try {
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    if (callCount === 3) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '"v2"') }));
+                    }
+                    return Promise.reject(new Error('POST network error'));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"v2"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                await jest.advanceTimersByTimeAsync(1500);
+                const result = await syncPromise;
+                expect(result).toFailWith(/post network error/i);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+        test('syncs multiple dirty files in order', async () => {
+            const { fetchImpl, calls } = makeMockFetch([
+                {
+                    ok: true,
+                    jsonValue: {
+                        path: '/',
+                        children: [
+                            { path: '/a.json', name: 'a.json', type: 'file' },
+                            { path: '/b.json', name: 'b.json', type: 'file' }
+                        ]
+                    }
+                },
+                { ok: true, jsonValue: fileResponse('/a.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/b.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/a.json', '"a-updated"') },
+                { ok: true, jsonValue: fileResponse('/b.json', '"b-updated"') },
+                { ok: true, jsonValue: { synced: 2 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/a.json', '"a-updated"').orThrow();
+            accessors.saveFileContents('/b.json', '"b-updated"').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toSucceed();
+            // Verify 2 PUTs + 1 POST
+            const syncCalls = calls.slice(3);
+            const methods = syncCalls.map((c) => { var _a; return (_a = c.init) === null || _a === void 0 ? void 0 : _a.method; });
+            expect(methods.filter((m) => m === 'PUT')).toHaveLength(2);
+            expect(methods.filter((m) => m === 'POST')).toHaveLength(1);
+        });
+        test('DELETEs pending deletions before POST /sync', async () => {
+            var _a, _b;
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: { deleted: true } },
+                { ok: true, jsonValue: { synced: 0 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.deleteFile('/data.json').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toSucceed();
+            const syncCalls = calls.slice(2);
+            expect((_a = syncCalls[0].init) === null || _a === void 0 ? void 0 : _a.method).toBe('DELETE');
+            expect(syncCalls[0].url).toContain('/file?');
+            expect((_b = syncCalls[1].init) === null || _b === void 0 ? void 0 : _b.method).toBe('POST');
+            expect(accessors.isDirty()).toBe(false);
+        });
+        test('fails when DELETE for pending deletion returns non-ok response', async () => {
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: false, status: 500, textValue: 'Delete failed' }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.deleteFile('/data.json').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toFailWith(/delete.*data\.json.*delete failed/i);
+        });
+    });
+    describe('syncToDisk() - getFileContents failure', () => {
+        test('fails when getFileContents returns failure for a dirty file during sync', async () => {
+            // This covers lines 112-113: the contentsResult.isFailure() branch in syncToDisk.
+            // We load a file, mark it dirty, then sabotage getFileContents to fail.
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '{"new":1}').orThrow();
+            // Sabotage the base getFileContents to simulate a failure
+            const originalGet = accessors.getFileContents.bind(accessors);
+            accessors.getFileContents = (path) => {
+                if (path === '/data.json') {
+                    // eslint-disable-next-line import/no-internal-modules
+                    const { fail: failResult } = require('@fgv/ts-utils');
+                    return failResult('simulated get failure');
+                }
+                return originalGet(path);
+            };
+            const result = await accessors.syncToDisk();
+            expect(result).toFailWith(/simulated get failure/i);
+        });
+    });
+    describe('_request() invalid JSON during sync', () => {
+        test('fails when PUT /file returns invalid JSON', async () => {
+            // Covers lines 225-226: _request() returns fail('invalid JSON response')
+            // when response.json() throws during a sync PUT.
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                // PUT response with ok=true but throwOnJson simulates invalid JSON body
+                { ok: true, throwOnJson: true }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toFailWith(/invalid json/i);
+        });
+    });
+    describe('fromHttp() with inferContentType', () => {
+        test('calls inferContentType when provided and uses the result', async () => {
+            // Covers line 270: params.inferContentType?.(...).orDefault()
+            const { fetchImpl } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}', 'application/json') }
+            ]);
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                inferContentType: (path, provided) => {
+                    if (provided === 'application/json') {
+                        const { succeed: s } = require('@fgv/ts-utils');
+                        return s('json');
+                    }
+                    const { succeed: s } = require('@fgv/ts-utils');
+                    return s(undefined);
+                }
+            });
+            expect(result).toSucceed();
+        });
+    });
+    describe('_request() error handling', () => {
+        test('returns failure with error message for network error (Error instance throw) during sync', async () => {
+            // Covers line 214 true branch: response.err instanceof Error -> uses .message
+            let callCount = 0;
+            const fetchImpl = (_url, _init) => {
+                callCount++;
+                if (callCount === 1) {
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                }
+                if (callCount === 2) {
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                }
+                // PUT request throws an Error instance
+                return Promise.reject(new Error('real Error instance'));
+            };
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toFailWith(/real error instance/i);
+        });
+        test('returns failure with error message for non-Error throw during sync PUT', async () => {
+            // Covers line 214 false branch: thrown value is not an Error instance -> uses String(response.err)
+            // Must be triggered via syncToDisk() which uses the instance _request() method.
+            let callCount = 0;
+            const fetchImpl = (_url, _init) => {
+                callCount++;
+                if (callCount === 1) {
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                }
+                if (callCount === 2) {
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                }
+                // Non-Error throw (e.g., a plain string)
+                return Promise.reject('non-error string rejection');
+            };
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toFailWith(/non-error string rejection/i);
+        });
+        test('returns failure with HTTP status fallback when response.text() throws during sync', async () => {
+            // Covers the text() catch branch in _request(): uses `HTTP ${status}` fallback
+            // 502 is transient so retries will be attempted; use fake timers to avoid real delays
+            jest.useFakeTimers();
+            try {
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    return Promise.resolve(makeMockResponse({ ok: false, status: 502, throwOnText: true }));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"updated"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                await jest.advanceTimersByTimeAsync(1500);
+                const result = await syncPromise;
+                expect(result).toFailWith(/http 502/i);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+    });
+    describe('_requestWithParams() error handling', () => {
+        test('returns failure with error message for non-Error network throw during init', async () => {
+            // _requestWithParams is used for GET requests during fromHttp(); test non-Error throw branch
+            const fetchImpl = (_url, _init) => {
+                return Promise.reject(42);
+            };
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/42/);
+        });
+        test('returns failure with HTTP status fallback when response.text() throws during init', async () => {
+            const fetchImpl = (_url, _init) => {
+                return Promise.resolve(makeMockResponse({ ok: false, status: 504, throwOnText: true }));
+            };
+            const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl
+            });
+            expect(result).toFailWith(/http 504/i);
+        });
+        test('uses globalThis.fetch when no fetchImpl provided', async () => {
+            // Covers line 48: the `fetchImpl ?? globalThis.fetch` right-side branch.
+            // We temporarily replace globalThis.fetch with a mock that returns an empty tree.
+            const originalFetch = globalThis.fetch;
+            let fetchCallCount = 0;
+            globalThis.fetch = (_url, _init) => {
+                fetchCallCount++;
+                return Promise.resolve(makeMockResponse({ ok: true, jsonValue: { path: '/', children: [] } }));
+            };
+            try {
+                const result = await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000'
+                    // No fetchImpl - should fall back to globalThis.fetch
+                });
+                expect(result).toSucceed();
+                expect(fetchCallCount).toBeGreaterThan(0);
+            }
+            finally {
+                globalThis.fetch = originalFetch;
+            }
+        });
+    });
+    describe('userId header in _request()', () => {
+        test('includes X-User-Id header in PUT /file and POST /sync requests when userId is set', async () => {
+            var _a;
+            // Covers line 259: the this._userId branch in _request()
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: fileResponse('/data.json', '"updated"') },
+                { ok: true, jsonValue: { synced: 1 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true,
+                userId: 'test-user-123'
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toSucceed();
+            // The PUT and POST requests should include the X-User-Id header
+            const syncCalls = calls.slice(2);
+            for (const call of syncCalls) {
+                const headers = (_a = call.init) === null || _a === void 0 ? void 0 : _a.headers;
+                expect(headers === null || headers === void 0 ? void 0 : headers['X-User-Id']).toBe('test-user-123');
+            }
+        });
+    });
+    describe('deleteFile()', () => {
+        test('fails when the underlying InMemoryTreeAccessors deleteFile fails (file not found)', async () => {
+            // Covers lines 200-201: result.isFailure() branch in deleteFile
+            const { fetchImpl } = makeMockFetch([{ ok: true, jsonValue: { path: '/', children: [] } }]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({ baseUrl: 'http://localhost:3000', fetchImpl, mutable: true })).orThrow();
+            // Delete a file that does not exist — super.deleteFile should fail
+            const result = accessors.deleteFile('/nonexistent.json');
+            expect(result).toFail();
+        });
+        test('includes namespace in DELETE query params when namespace is configured', async () => {
+            // Covers lines 131-132: the namespace branch in syncToDisk's pending-deletion loop
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                { ok: true, jsonValue: { deleted: true } },
+                { ok: true, jsonValue: { synced: 0 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                namespace: 'my-namespace',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.deleteFile('/data.json').orThrow();
+            const result = await accessors.syncToDisk();
+            expect(result).toSucceed();
+            // The DELETE request URL should include namespace as a query parameter
+            const deleteCalls = calls.filter((c) => { var _a; return ((_a = c.init) === null || _a === void 0 ? void 0 : _a.method) === 'DELETE'; });
+            expect(deleteCalls).toHaveLength(1);
+            expect(deleteCalls[0].url).toContain('namespace=my-namespace');
+        });
+        test('triggers fire-and-forget autoSync after successful delete when autoSync is enabled', async () => {
+            // Covers lines 209-212: the autoSync branch in deleteFile
+            const syncResponses = [
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                // DELETE /file response for pending deletion
+                { ok: true, jsonValue: { deleted: true } },
+                // POST /sync response
+                { ok: true, jsonValue: { synced: 0 } }
+            ];
+            const { fetchImpl, calls } = makeMockFetch(syncResponses);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true,
+                autoSync: true
+            })).orThrow();
+            const result = accessors.deleteFile('/data.json');
+            expect(result).toSucceedWith(true);
+            // autoSync fires-and-forgets; wait for microtasks to drain
+            await new Promise((resolve) => setTimeout(resolve, 0));
+            // Verify that DELETE + POST /sync were called
+            const methodCalls = calls.slice(2).map((c) => { var _a; return (_a = c.init) === null || _a === void 0 ? void 0 : _a.method; });
+            expect(methodCalls).toContain('DELETE');
+            expect(methodCalls).toContain('POST');
+        });
+    });
+    describe('_requestWithRetry()', () => {
+        test('succeeds on second attempt after a transient 503 error', async () => {
+            jest.useFakeTimers();
+            try {
+                // Responses: init (GET tree/children), then PUT fails with 503, then PUT succeeds, then POST /sync succeeds
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    // First PUT attempt: transient 503
+                    if (callCount === 3) {
+                        return Promise.resolve(makeMockResponse({ ok: false, status: 503, textValue: '503 Service Unavailable' }));
+                    }
+                    // Second PUT attempt: success
+                    if (callCount === 4) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '"updated"') }));
+                    }
+                    // POST /sync
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: { synced: 1 } }));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"updated"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                // Advance past the 500ms backoff for the first retry
+                await jest.advanceTimersByTimeAsync(500);
+                const result = await syncPromise;
+                expect(result).toSucceed();
+                // Three PUT-related calls: init (2) + first PUT attempt (1) + retry PUT (1) + POST sync (1)
+                expect(callCount).toBe(5);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+        test('does not retry on a non-transient error (e.g. 404)', async () => {
+            jest.useFakeTimers();
+            try {
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    // PUT attempt: non-transient 404
+                    return Promise.resolve(makeMockResponse({ ok: false, status: 404, textValue: 'Not Found' }));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"updated"').orThrow();
+                const result = await accessors.syncToDisk();
+                // No timers should need advancing - non-transient errors return immediately
+                expect(result).toFailWith(/not found/i);
+                // Only 3 calls: 2 init + 1 PUT (no retries)
+                expect(callCount).toBe(3);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+        test('returns the last error after exhausting all retries on persistent transient errors', async () => {
+            jest.useFakeTimers();
+            try {
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    // All PUT attempts: transient 502 every time
+                    return Promise.resolve(makeMockResponse({ ok: false, status: 502, textValue: '502 Bad Gateway' }));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"updated"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                // Advance past both backoff delays: 500ms (attempt 1) + 1000ms (attempt 2)
+                await jest.advanceTimersByTimeAsync(1500);
+                const result = await syncPromise;
+                expect(result).toFailWith(/502 bad gateway/i);
+                // 2 init + 3 PUT attempts (maxAttempts = 3)
+                expect(callCount).toBe(5);
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+        test('logs retry message with method name from init when retrying', async () => {
+            jest.useFakeTimers();
+            try {
+                const logger = {
+                    detail: jest.fn(),
+                    info: jest.fn(),
+                    warn: jest.fn(),
+                    error: jest.fn()
+                };
+                let callCount = 0;
+                const fetchImpl = (_url, _init) => {
+                    callCount++;
+                    if (callCount === 1) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: rootWithOneFile('data.json') }));
+                    }
+                    if (callCount === 2) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '{}') }));
+                    }
+                    // First PUT attempt: transient 503
+                    if (callCount === 3) {
+                        return Promise.resolve(makeMockResponse({ ok: false, status: 503, textValue: '503 Service Unavailable' }));
+                    }
+                    // Second PUT attempt: success
+                    if (callCount === 4) {
+                        return Promise.resolve(makeMockResponse({ ok: true, jsonValue: fileResponse('/data.json', '"updated"') }));
+                    }
+                    // POST /sync
+                    return Promise.resolve(makeMockResponse({ ok: true, jsonValue: { synced: 1 } }));
+                };
+                const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                    baseUrl: 'http://localhost:3000',
+                    fetchImpl,
+                    mutable: true,
+                    logger
+                })).orThrow();
+                accessors.saveFileContents('/data.json', '"updated"').orThrow();
+                const syncPromise = accessors.syncToDisk();
+                await jest.advanceTimersByTimeAsync(500);
+                await syncPromise;
+                // The retry log message should include the method name (PUT) from init
+                expect(logger.detail).toHaveBeenCalledWith(expect.stringContaining('PUT'));
+            }
+            finally {
+                jest.useRealTimers();
+            }
+        });
+    });
+    describe('syncToDisk() concurrency guard', () => {
+        test('concurrent syncToDisk calls share the same promise and only make one set of network calls', async () => {
+            const { fetchImpl, calls } = makeMockFetch([
+                { ok: true, jsonValue: rootWithOneFile('data.json') },
+                { ok: true, jsonValue: fileResponse('/data.json', '{}') },
+                // PUT /file for the single sync
+                { ok: true, jsonValue: fileResponse('/data.json', '"updated"') },
+                // POST /sync
+                { ok: true, jsonValue: { synced: 1 } }
+            ]);
+            const accessors = (await file_tree_1.HttpTreeAccessors.fromHttp({
+                baseUrl: 'http://localhost:3000',
+                fetchImpl,
+                mutable: true
+            })).orThrow();
+            accessors.saveFileContents('/data.json', '"updated"').orThrow();
+            // Start two concurrent syncToDisk calls without awaiting the first
+            const promise1 = accessors.syncToDisk();
+            const promise2 = accessors.syncToDisk();
+            const [result1, result2] = await Promise.all([promise1, promise2]);
+            expect(result1).toSucceed();
+            expect(result2).toSucceed();
+            // Both promises should have resolved to the same underlying promise result.
+            // Only one PUT and one POST /sync should have been issued (not doubled).
+            const syncCalls = calls.slice(2);
+            const putCalls = syncCalls.filter((c) => { var _a; return ((_a = c.init) === null || _a === void 0 ? void 0 : _a.method) === 'PUT'; });
+            const postCalls = syncCalls.filter((c) => { var _a; return ((_a = c.init) === null || _a === void 0 ? void 0 : _a.method) === 'POST'; });
+            expect(putCalls).toHaveLength(1);
+            expect(postCalls).toHaveLength(1);
+        });
+    });
+});
+//# sourceMappingURL=httpTreeAccessors.test.js.map