Remove useless syncs

2026-04-26 12:46:46 +01:00 · 2026-04-26 12:46:46 +01:00 · fc0ff0df1c
commit fc0ff0df1c
parent 8b7be48522
52 changed files with 0 additions and 103 deletions
--- a/frontend/deterministic-tests/src/tests/18-create-rename-create-same-path.test.ts
+++ b/frontend/deterministic-tests/src/tests/18-create-rename-create-same-path.test.ts
@ -18,7 +18,6 @@ export const createRenameCreateSamePathTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/4-coalesced-remote-update-watermark-loss.test.ts
+++ b/frontend/deterministic-tests/src/tests/4-coalesced-remote-update-watermark-loss.test.ts
@ -16,7 +16,6 @@ export const coalescedRemoteUpdateWatermarkLossTest: TestDefinition = {
        { type: "update", client: 0, path: "doc.md", content: "update 1" },
        { type: "update", client: 0, path: "doc.md", content: "update 2" },
        { type: "update", client: 0, path: "doc.md", content: "final update" },
-        { type: "sync", client: 0 },

        { type: "barrier" },
        {
--- a/frontend/deterministic-tests/src/tests/9-concurrent-rename-same-target.test.ts
+++ b/frontend/deterministic-tests/src/tests/9-concurrent-rename-same-target.test.ts
@ -21,7 +21,6 @@ export const concurrentRenameSameTargetTest: TestDefinition = {
        { type: "rename", client: 1, oldPath: "B.md", newPath: "C.md" },

        { type: "enable-sync", client: 1 },
-        { type: "sync", client: 1 },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/delete-during-pending-create.test.ts
+++ b/frontend/deterministic-tests/src/tests/delete-during-pending-create.test.ts
@ -9,7 +9,6 @@ export const deleteDuringPendingCreateTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "pause-server" },
@ -24,7 +23,6 @@ export const deleteDuringPendingCreateTest: TestDefinition = {
        { type: "delete", client: 0, path: "ephemeral.md" },

        { type: "resume-server" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/delete-recreate-concurrent-update.test.ts
+++ b/frontend/deterministic-tests/src/tests/delete-recreate-concurrent-update.test.ts
@ -10,7 +10,6 @@ export const deleteRecreateConcurrentUpdateTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -31,7 +30,6 @@ export const deleteRecreateConcurrentUpdateTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/delete-recreate-different-content.test.ts
+++ b/frontend/deterministic-tests/src/tests/delete-recreate-different-content.test.ts
@ -15,7 +15,6 @@ export const deleteRecreateDifferentContentTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -39,7 +38,6 @@ export const deleteRecreateDifferentContentTest: TestDefinition = {
        { type: "enable-sync", client: 0 },
        { type: "sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/delete-recreate-same-path.test.ts
+++ b/frontend/deterministic-tests/src/tests/delete-recreate-same-path.test.ts
@ -10,7 +10,6 @@ export const deleteRecreateSamePathTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "version 1" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -23,7 +22,6 @@ export const deleteRecreateSamePathTest: TestDefinition = {
        { type: "delete", client: 0, path: "A.md" },
        { type: "create", client: 0, path: "A.md", content: "version 2" },
        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/delete-rename-conflict.test.ts
+++ b/frontend/deterministic-tests/src/tests/delete-rename-conflict.test.ts
@ -11,7 +11,6 @@ export const deleteRenameConflictTest: TestDefinition = {
        { type: "create", client: 0, path: "B.md", content: "content-b" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -28,7 +27,6 @@ export const deleteRenameConflictTest: TestDefinition = {
        { type: "rename", client: 1, oldPath: "A.md", newPath: "C.md" },

        { type: "enable-sync", client: 1 },
-        { type: "sync", client: 1 },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/double-offline-cycle.test.ts
+++ b/frontend/deterministic-tests/src/tests/double-offline-cycle.test.ts
@ -15,7 +15,6 @@ export const doubleOfflineCycleTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -33,7 +32,6 @@ export const doubleOfflineCycleTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -51,7 +49,6 @@ export const doubleOfflineCycleTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -69,7 +66,6 @@ export const doubleOfflineCycleTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
--- a/frontend/deterministic-tests/src/tests/idempotency-after-server-pause.test.ts
+++ b/frontend/deterministic-tests/src/tests/idempotency-after-server-pause.test.ts
@ -9,7 +9,6 @@ export const idempotencyAfterServerPauseTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -22,7 +21,6 @@ export const idempotencyAfterServerPauseTest: TestDefinition = {

        { type: "resume-server" },

-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/interrupted-delete-retry.test.ts
+++ b/frontend/deterministic-tests/src/tests/interrupted-delete-retry.test.ts
@ -10,7 +10,6 @@ export const interruptedDeleteRetryTest: TestDefinition = {
        { type: "create", client: 0, path: "doc.md", content: "to be deleted" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "delete", client: 0, path: "doc.md" },
@ -18,7 +17,6 @@ export const interruptedDeleteRetryTest: TestDefinition = {
        { type: "pause-server" },

        { type: "resume-server" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/key-migration-event-drop.test.ts
+++ b/frontend/deterministic-tests/src/tests/key-migration-event-drop.test.ts
@ -9,7 +9,6 @@ export const keyMigrationEventDropTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "pause-server" },
@ -28,7 +27,6 @@ export const keyMigrationEventDropTest: TestDefinition = {
        },

        { type: "resume-server" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/mc-cross-create-rename-same-target.test.ts
+++ b/frontend/deterministic-tests/src/tests/mc-cross-create-rename-same-target.test.ts
@ -13,7 +13,6 @@ export const mcCrossCreateRenameSameTargetTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -31,7 +30,6 @@ export const mcCrossCreateRenameSameTargetTest: TestDefinition = {
        { type: "rename", client: 1, oldPath: "Y.md", newPath: "Z.md" },

        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/mc-delete-then-offline-rename.test.ts
+++ b/frontend/deterministic-tests/src/tests/mc-delete-then-offline-rename.test.ts
@ -12,7 +12,6 @@ export const mcDeleteThenOfflineRenameTest: TestDefinition = {
        { type: "create", client: 0, path: "C.md", content: "unrelated" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 1 },
@ -23,7 +22,6 @@ export const mcDeleteThenOfflineRenameTest: TestDefinition = {
        { type: "rename", client: 1, oldPath: "A.md", newPath: "B.md" },

        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/mc-multi-delete-offline-rename.test.ts
+++ b/frontend/deterministic-tests/src/tests/mc-multi-delete-offline-rename.test.ts
@ -14,7 +14,6 @@ export const mcMultiDeleteOfflineRenameTest: TestDefinition = {
        { type: "create", client: 0, path: "file-5.md", content: "content-5" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -31,7 +30,6 @@ export const mcMultiDeleteOfflineRenameTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/mc-three-client-rename-offline-update.test.ts
+++ b/frontend/deterministic-tests/src/tests/mc-three-client-rename-offline-update.test.ts
@ -11,7 +11,6 @@ export const mcThreeClientRenameOfflineUpdateTest: TestDefinition = {
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
        { type: "enable-sync", client: 2 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 2 },
@ -28,7 +27,6 @@ export const mcThreeClientRenameOfflineUpdateTest: TestDefinition = {
        },

        { type: "enable-sync", client: 2 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/migrate-key-preserves-existing.test.ts
+++ b/frontend/deterministic-tests/src/tests/migrate-key-preserves-existing.test.ts
@ -9,7 +9,6 @@ export const migrateKeyPreservesExistingTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "pause-server" },
@ -23,7 +22,6 @@ export const migrateKeyPreservesExistingTest: TestDefinition = {
        },

        { type: "resume-server" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/move-and-concurrent-remote-update.test.ts
+++ b/frontend/deterministic-tests/src/tests/move-and-concurrent-remote-update.test.ts
@ -15,7 +15,6 @@ export const moveAndConcurrentRemoteUpdateTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -30,7 +29,6 @@ export const moveAndConcurrentRemoteUpdateTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/move-preserves-remote-update.test.ts
+++ b/frontend/deterministic-tests/src/tests/move-preserves-remote-update.test.ts
@ -15,7 +15,6 @@ export const movePreservesRemoteUpdateTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -31,7 +30,6 @@ export const movePreservesRemoteUpdateTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/move-remote-update-reverts-rename.test.ts
+++ b/frontend/deterministic-tests/src/tests/move-remote-update-reverts-rename.test.ts
@ -10,7 +10,6 @@ export const moveRemoteUpdateRevertsRenameTest: TestDefinition = {
        { type: "create", client: 0, path: "doc.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -24,7 +23,6 @@ export const moveRemoteUpdateRevertsRenameTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "rename", client: 0, oldPath: "doc.md", newPath: "renamed.md" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/move-then-delete-stale-path.test.ts
+++ b/frontend/deterministic-tests/src/tests/move-then-delete-stale-path.test.ts
@ -15,13 +15,11 @@ export const moveThenDeleteStalePathTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "rename", client: 0, oldPath: "A.md", newPath: "B.md" },
        { type: "delete", client: 0, path: "B.md" },

-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/multi-file-operations.test.ts
+++ b/frontend/deterministic-tests/src/tests/multi-file-operations.test.ts
@ -12,7 +12,6 @@ export const multiFileOperationsTest: TestDefinition = {
        { type: "create", client: 0, path: "C.md", content: "content-c" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 1 },
@ -29,7 +28,6 @@ export const multiFileOperationsTest: TestDefinition = {
        { type: "rename", client: 1, oldPath: "A.md", newPath: "D.md" },

        { type: "enable-sync", client: 1 },
-        { type: "sync", client: 1 },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-concurrent-renames.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-concurrent-renames.test.ts
@ -12,7 +12,6 @@ export const offlineConcurrentRenamesTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "shared-content" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -40,7 +39,6 @@ export const offlineConcurrentRenamesTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-create-same-path-binary-conflict.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-create-same-path-binary-conflict.test.ts
@ -23,7 +23,6 @@ export const offlineCreateSamePathMergeableTest: TestDefinition = {
        { type: "enable-sync", client: 0 },
        { type: "sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-delete-vs-remote-update.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-delete-vs-remote-update.test.ts
@ -14,7 +14,6 @@ export const offlineDeleteVsRemoteUpdateTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -35,7 +34,6 @@ export const offlineDeleteVsRemoteUpdateTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-edit-remote-rename.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-edit-remote-rename.test.ts
@ -10,7 +10,6 @@ export const offlineEditRemoteRenameTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -36,7 +35,6 @@ export const offlineEditRemoteRenameTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-edit-then-move-same-content.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-edit-then-move-same-content.test.ts
@ -20,7 +20,6 @@ export const offlineEditThenMoveSameContentTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -37,7 +36,6 @@ export const offlineEditThenMoveSameContentTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-mixed-operations.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-mixed-operations.test.ts
@ -13,7 +13,6 @@ export const offlineMixedOperationsTest: TestDefinition = {
        { type: "create", client: 0, path: "file3.md", content: "content-3" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -42,7 +41,6 @@ export const offlineMixedOperationsTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-move-then-remote-delete.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-move-then-remote-delete.test.ts
@ -15,7 +15,6 @@ export const offlineMoveThenRemoteDeleteTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -25,7 +24,6 @@ export const offlineMoveThenRemoteDeleteTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-multiple-edits.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-multiple-edits.test.ts
@ -11,7 +11,6 @@ export const offlineMultipleEditsTest: TestDefinition = {
        { type: "create", client: 0, path: "doc.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -29,7 +28,6 @@ export const offlineMultipleEditsTest: TestDefinition = {
        { type: "update", client: 0, path: "doc.md", content: "edit-5-final" },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-rename-and-edit.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-rename-and-edit.test.ts
@ -11,7 +11,6 @@ export const offlineRenameAndEditTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -30,7 +29,6 @@ export const offlineRenameAndEditTest: TestDefinition = {
        },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-rename-remote-create-old-path.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-rename-remote-create-old-path.test.ts
@ -11,7 +11,6 @@ export const offlineRenameRemoteCreateOldPathTest: TestDefinition = {
        { type: "create", client: 0, path: "X.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -37,7 +36,6 @@ export const offlineRenameRemoteCreateOldPathTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/offline-update-both-then-delete-one.test.ts
+++ b/frontend/deterministic-tests/src/tests/offline-update-both-then-delete-one.test.ts
@ -23,7 +23,6 @@ export const offlineUpdateBothThenDeleteOneTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -61,7 +60,6 @@ export const offlineUpdateBothThenDeleteOneTest: TestDefinition = {
        { type: "sync", client: 1 },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/overlapping-edits-same-section.test.ts
+++ b/frontend/deterministic-tests/src/tests/overlapping-edits-same-section.test.ts
@ -15,7 +15,6 @@ export const overlappingEditsSameSectionTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -37,7 +36,6 @@ export const overlappingEditsSameSectionTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rapid-create-update-delete-cycle.test.ts
+++ b/frontend/deterministic-tests/src/tests/rapid-create-update-delete-cycle.test.ts
@ -9,7 +9,6 @@ export const rapidCreateUpdateDeleteCycleTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "pause-server" },
--- a/frontend/deterministic-tests/src/tests/rapid-updates-after-merge.test.ts
+++ b/frontend/deterministic-tests/src/tests/rapid-updates-after-merge.test.ts
@ -12,7 +12,6 @@ export const rapidUpdatesAfterMergeTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -37,7 +36,6 @@ export const rapidUpdatesAfterMergeTest: TestDefinition = {
            path: "doc.md",
            content: "update 3"
        },
-        { type: "sync", client: 0 },

        { type: "barrier" },

--- a/frontend/deterministic-tests/src/tests/rename-chain-then-delete.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-chain-then-delete.test.ts
@ -10,7 +10,6 @@ export const renameChainThenDeleteTest: TestDefinition = {
        { type: "create", client: 0, path: "X.md", content: "chain-content" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -39,7 +38,6 @@ export const renameChainThenDeleteTest: TestDefinition = {
        { type: "sync", client: 0 },

        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rename-chain.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-chain.test.ts
@ -20,7 +20,6 @@ export const renameChainTest: TestDefinition = {
        { type: "rename", client: 0, oldPath: "B.md", newPath: "C.md" },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rename-circular.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-circular.test.ts
@ -28,7 +28,6 @@ export const renameCircularTest: TestDefinition = {
        { type: "rename", client: 0, oldPath: "temp-a.md", newPath: "B.md" },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rename-create-conflict.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-create-conflict.test.ts
@ -22,7 +22,6 @@ export const renameCreateConflictTest: TestDefinition = {
        { type: "sync", client: 1 },
        { type: "create", client: 0, path: "B.md", content: "hi" },
        { type: "enable-sync", client: 0 },
-        { type: "sync", client: 0 },
        { type: "barrier" },
        {
            type: "assert-consistent",
--- a/frontend/deterministic-tests/src/tests/rename-roundtrip.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-roundtrip.test.ts
@ -9,7 +9,6 @@ export const renameRoundtripTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -19,7 +18,6 @@ export const renameRoundtripTest: TestDefinition = {
        },

        { type: "rename", client: 0, oldPath: "A.md", newPath: "B.md" },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -30,7 +28,6 @@ export const renameRoundtripTest: TestDefinition = {
        },

        { type: "rename", client: 0, oldPath: "B.md", newPath: "A.md" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rename-to-pending-path-fallback.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-to-pending-path-fallback.test.ts
@ -14,7 +14,6 @@ export const renameToPendingPathFallbackTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -29,7 +28,6 @@ export const renameToPendingPathFallbackTest: TestDefinition = {
        { type: "rename", client: 0, oldPath: "B.md", newPath: "A.md" },

        { type: "enable-sync", client: 0 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rename-to-recently-deleted-path.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-to-recently-deleted-path.test.ts
@ -10,7 +10,6 @@ export const renameToRecentlyDeletedPathTest: TestDefinition = {
        { type: "create", client: 0, path: "B.md", content: "content-b" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 1 },
@ -26,7 +25,6 @@ export const renameToRecentlyDeletedPathTest: TestDefinition = {
        },

        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/rename-update-conflict.test.ts
+++ b/frontend/deterministic-tests/src/tests/rename-update-conflict.test.ts
@ -9,7 +9,6 @@ export const renameUpdateConflictTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -31,7 +30,6 @@ export const renameUpdateConflictTest: TestDefinition = {
        },

        { type: "enable-sync", client: 1 },
-        { type: "sync", client: 1 },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/sequential-create-duplicate-content.test.ts
+++ b/frontend/deterministic-tests/src/tests/sequential-create-duplicate-content.test.ts
@ -14,7 +14,6 @@ export const sequentialCreateDuplicateContentTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -30,7 +29,6 @@ export const sequentialCreateDuplicateContentTest: TestDefinition = {
            path: "B.md",
            content: "identical content here"
        },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/server-pause-both-clients-create.test.ts
+++ b/frontend/deterministic-tests/src/tests/server-pause-both-clients-create.test.ts
@ -8,7 +8,6 @@ export const serverPauseBothClientsCreateTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -28,7 +27,6 @@ export const serverPauseBothClientsCreateTest: TestDefinition = {

        { type: "resume-server" },

-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/server-pause-both-edit-same-file.test.ts
+++ b/frontend/deterministic-tests/src/tests/server-pause-both-edit-same-file.test.ts
@ -14,7 +14,6 @@ export const serverPauseBothEditSameFileTest: TestDefinition = {
        },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "pause-server" },
@ -35,7 +34,6 @@ export const serverPauseBothEditSameFileTest: TestDefinition = {
        },

        { type: "resume-server" },
-        { type: "sync" },
        { type: "barrier" },

        {
@ -55,7 +53,6 @@ export const serverPauseBothEditSameFileTest: TestDefinition = {
            path: "shared.md",
            content: "post-merge edit from client 0"
        },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/server-pause-rename-edit-resume.test.ts
+++ b/frontend/deterministic-tests/src/tests/server-pause-rename-edit-resume.test.ts
@ -16,7 +16,6 @@ export const serverPauseRenameEditResumeTest: TestDefinition = {
            path: "A.md",
            content: "original content"
        },
-        { type: "sync" },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -37,7 +36,6 @@ export const serverPauseRenameEditResumeTest: TestDefinition = {

        { type: "resume-server" },

-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/simultaneous-create-delete-same-path.test.ts
+++ b/frontend/deterministic-tests/src/tests/simultaneous-create-delete-same-path.test.ts
@ -11,7 +11,6 @@ export const simultaneousCreateDeleteSamePathTest: TestDefinition = {
        { type: "create", client: 0, path: "A.md", content: "original from 0" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 1 },
@ -27,7 +26,6 @@ export const simultaneousCreateDeleteSamePathTest: TestDefinition = {
        },

        { type: "enable-sync", client: 1 },
-        { type: "sync", client: 1 },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/update-during-create-processing.test.ts
+++ b/frontend/deterministic-tests/src/tests/update-during-create-processing.test.ts
@ -8,7 +8,6 @@ export const updateDuringCreateProcessingTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "pause-server" },
@ -28,7 +27,6 @@ export const updateDuringCreateProcessingTest: TestDefinition = {
        },

        { type: "resume-server" },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/watermark-advances-on-skip.test.ts
+++ b/frontend/deterministic-tests/src/tests/watermark-advances-on-skip.test.ts
@ -8,7 +8,6 @@ export const watermarkAdvancesOnSkipTest: TestDefinition = {
    steps: [
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
@ -18,14 +17,12 @@ export const watermarkAdvancesOnSkipTest: TestDefinition = {

        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "disable-sync", client: 0 },
        { type: "disable-sync", client: 1 },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {
--- a/frontend/deterministic-tests/src/tests/watermark-gap-remote-update-not-recorded.test.ts
+++ b/frontend/deterministic-tests/src/tests/watermark-gap-remote-update-not-recorded.test.ts
@ -9,15 +9,12 @@ export const watermarkGapRemoteUpdateNotRecordedTest: TestDefinition = {
        { type: "create", client: 0, path: "doc.md", content: "original" },
        { type: "enable-sync", client: 0 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        { type: "update", client: 0, path: "doc.md", content: "update 1" },
        { type: "sync", client: 0 },
        { type: "update", client: 0, path: "doc.md", content: "update 2" },
-        { type: "sync", client: 0 },

-        { type: "sync", client: 1 },
        { type: "barrier" },
        {
            type: "assert-consistent",
@ -28,7 +25,6 @@ export const watermarkGapRemoteUpdateNotRecordedTest: TestDefinition = {

        { type: "disable-sync", client: 1 },
        { type: "enable-sync", client: 1 },
-        { type: "sync" },
        { type: "barrier" },

        {