Double check before delete

plugin/src/file-operations/file-operations.ts

@@ -7,6 +7,8 @@ export interface FileOperations {
 
 	getFileSize: (path: RelativePath) => Promise<number>;
 
+	exists: (path: RelativePath) => Promise<boolean>;
+
 	getModificationTime: (path: RelativePath) => Promise<Date>;
 
 	// Create and write the file if it doesn't exist. Otherwise, it has the same behavior as write.

plugin/src/file-operations/obsidian-file-operations.ts

@@ -39,6 +39,11 @@ export class ObsidianFileOperations implements FileOperations {
 		return new Date((await this.statFile(path)).mtime);
 	}
 
+	public async exists(path: RelativePath): Promise<boolean> {
+		Logger.getInstance().debug(`Checking existance of ${path}`);
+		return this.vault.adapter.exists(normalizePath(path));
+	}
+
 	public async create(
 		path: RelativePath,
 		newContent: Uint8Array
@@ -108,7 +113,7 @@ export class ObsidianFileOperations implements FileOperations {
 	public async remove(path: RelativePath): Promise<void> {
 		Logger.getInstance().debug(`Removing file: ${path}`);
 		if (await this.vault.adapter.exists(normalizePath(path))) {
-			return this.vault.adapter.remove(normalizePath(path));
+			await this.vault.adapter.trashSystem(normalizePath(path));
 		}
 	}
 

plugin/src/sync-operations/syncer.ts

@@ -176,6 +176,13 @@ export class Syncer {
 						`Document ${relativePath} has been deleted locally, scheduling sync to delete it`
 					);
 
+					if (await this.operations.exists(relativePath)) {
+						Logger.getInstance().debug(
+							`Document ${relativePath} actually exists locally, skipping`
+						);
+						return Promise.resolve();
+					}
+
 					return this.internalSyncLocallyDeletedFile(relativePath);
 				})
 			);