fix bug where for local models, delete only the model passed in, not all local models (menloresearch#2207)

ethanova · Ethan Garber · qnixsynapse · web-flow · commit 62b74b7af9f9 · 2025-06-14T11:59:32.000+05:30
Co-authored-by: Ethan Garber &lt;ethancgarber@gmail.com&gt;
Co-authored-by: Akarshan Biswas &lt;akarshan@menlo.ai&gt;
diff --git a/engine/services/model_service.cc b/engine/services/model_service.cc
@@ -500,13 +500,10 @@ cpp::result<void, std::string> ModelService::DeleteModel(
       std::filesystem::remove(yaml_fp);
       CTL_INF("Removed: " << yaml_fp.string());
     } else {
-      // Remove yaml files
-      for (const auto& entry :
-           std::filesystem::directory_iterator(yaml_fp.parent_path())) {
-        if (entry.is_regular_file() && (entry.path().extension() == ".yml")) {
-          std::filesystem::remove(entry);
-          CTL_INF("Removed: " << entry.path().string());
-        }
+      // Is a local model - Remove only this model's yaml file
+      if (std::filesystem::exists(yaml_fp)) {
+        std::filesystem::remove(yaml_fp);
+        CTL_INF("Removed: " << yaml_fp.string());
       }
     }
 
