From 8d2c5403e817cb4553061b93767dd251fdbacdf2 Mon Sep 17 00:00:00 2001
From: Spencer Brower <spencer.brower@proton.me>
Date: Fri, 12 Jun 2026 17:25:08 -0400
Subject: [PATCH] fix: Removed duplicate insert calls.

---
 TODOS.md      | 2 --
 cmd_sync.odin | 3 ---
 2 files changed, 5 deletions(-)

diff --git a/TODOS.md b/TODOS.md
index 4fcb184..97ed147 100644
--- a/TODOS.md
+++ b/TODOS.md
@@ -8,8 +8,6 @@ Note: These todos can wait until all the subcommands have been ported.
 
 3. **config.odin:52-54** — `os.user_home_dir` error silently ignored. If it fails, `home` is `""` and all paths become relative (`".envr"` instead of `"~/.envr"`).
 
-30. **cmd_sync.odin:46-50, 64-68** — Double `db_insert` when `BackedUp`: first insert on line 48, then `db_update_required` is also true for `BackedUp` so second insert runs on line 65. Redundant and wasteful.
-
 ## MEDIUM
 
 4. **db.odin:29-35** — `make_temp_path` never calls `strings.builder_destroy`. Leaks builder buffer every call.
diff --git a/cmd_sync.odin b/cmd_sync.odin
index d1acc22..adb6af0 100644
--- a/cmd_sync.odin
+++ b/cmd_sync.odin
@@ -45,9 +45,6 @@ cmd_sync :: proc(cmd: ^Command) {
 			}
 		case .BackedUp in result:
 			status = "Backed Up"
-			if !db_insert(&db, file) {
-				return
-			}
 		case .Restored in result:
 			status = "Restored"
 		case .DirUpdated in result: