Appease linter

Author: expede

eslint.config.mjs

@@ -29,7 +29,13 @@ export default tseslint.config(
         },
       ],
       "no-empty": ["warn", { allowEmptyCatch: true }],
-      "prefer-const": "warn",
+      // `ignoreReadBeforeAssign` keeps the rule from flagging the legitimate
+      // pattern where a `let` is declared first so it can be captured by a
+      // closure defined immediately afterward, and is then assigned later
+      // (e.g. `let timer; const cleanup = () => clearTimeout(timer); timer = setTimeout(...)`).
+      // Rewriting those to `const` would force restructuring just to silence
+      // the lint without any real readability gain.
+      "prefer-const": ["warn", { ignoreReadBeforeAssign: true }],
     },
   },
 );

src/core/sync-engine.ts

@@ -231,7 +231,7 @@ export class SyncEngine {
 	 * Used by --force to re-sync every file.
 	 */
 	async resetSnapshot(): Promise<void> {
-		let snapshot = await this.snapshotManager.load()
+		const snapshot = await this.snapshotManager.load()
 		if (!snapshot) return
 		this.snapshotManager.clear(snapshot)
 		await this.snapshotManager.save(snapshot)
@@ -243,7 +243,7 @@ export class SyncEngine {
 	 * documents. The root directory document itself is preserved.
 	 */
 	async nuclearReset(): Promise<void> {
-		let snapshot = await this.snapshotManager.load()
+		const snapshot = await this.snapshotManager.load()
 		if (!snapshot) return
 
 		// Clear the root directory document's entries