cufetchpm: args: remove --silence-warning and replace with --yes

most short duration feature ever :wilted_rose:

Author: Toni500github

cufetchpm/include/pluginManager.hpp

@@ -39,7 +39,7 @@ namespace fs = std::filesystem;
 inline struct operations_t
 {
     bool                     install_force   = false;
-    bool                     install_no_warn = false;
+    bool                     install_shut_up = false;
     bool                     list_verbose    = false;
     std::vector<std::string> arguments;
 } options;

cufetchpm/src/main.cpp

@@ -109,7 +109,7 @@ bool parse_install_args(int argc, char* argv[])
     const struct option long_opts[] = {
         {"force",           no_argument, nullptr, 'f'},
         {"help",            no_argument, nullptr, 'h'},
-        {"silence-warning", no_argument, nullptr, 'w'},
+        {"yes",             no_argument, nullptr, 'y'},
         {0, 0, 0, 0}
     };
     // clang-format on
@@ -123,7 +123,7 @@ bool parse_install_args(int argc, char* argv[])
             case '?': help_install(EXIT_FAILURE); break;
 
             case 'f': options.install_force = true; break;
-            case 'w': options.install_no_warn = true; break;
+            case 'y': options.install_shut_up = true; break;
         }
     }
 

cufetchpm/src/pluginManager.cpp

@@ -62,7 +62,7 @@ static bool has_deps(const std::vector<std::string>& dependencies)
 
 static bool find_plugin_prefix(const plugin_t& plugin, const plugin_t& pending_plugin)
 {
-    for (const auto& prefix : pending_plugin.prefixes)
+    for (const std::string& prefix : pending_plugin.prefixes)
         if (std::find(plugin.prefixes.begin(), plugin.prefixes.end(), prefix) != plugin.prefixes.end())
             return true;
     return false;
@@ -178,7 +178,7 @@ void PluginManager::build_plugins(const fs::path& working_dir)
         }
     }
 
-    if (!options.install_no_warn)
+    if (!options.install_shut_up)
     {
         warn("{}",
              "You should never blindly trust anything in life that you never saw/know about.\n"
@@ -199,9 +199,9 @@ void PluginManager::build_plugins(const fs::path& working_dir)
 
     if (!manifest.get_dependencies().empty())
     {
-        info("The repository {} requires the following dependencies, check if you have them installed:\n    {}",
+        info("The plugin repository {} requires the following dependencies, check if you have them installed:\n    {}",
              manifest.get_repo_name(), fmt::join(manifest.get_dependencies(), ", "));
-        if (!askUserYorN(true, "Are these dependencies installed?"))
+        if (!options.install_shut_up && !askUserYorN(true, "Are these dependencies installed?"))
             die("Balling out, re-install the repository again after installing all dependencies.");
     }
 
@@ -230,7 +230,7 @@ void PluginManager::build_plugins(const fs::path& working_dir)
             warn("Plugin '{}' has conflicting prefixes with other plugins.", plugin.name);
             warn("Check with 'cufetchpm list' the plugins that have one of the following prefixes: {}",
                  fmt::join(plugin.prefixes, ", "));
-            if (!askUserYorN(false, "Wanna continue?"))
+            if (!options.install_shut_up && !askUserYorN(false, "Wanna continue?"))
             {
                 fs::remove_all(working_dir);
                 die("Balling out");
@@ -283,7 +283,7 @@ void PluginManager::build_plugins(const fs::path& working_dir)
             const fs::path& library_config_path = manifest_config_path / library.path().filename();
             if (fs::exists(library_config_path) && (!options.install_force || !is_update))
             {
-                if (askUserYorN(false, "Plugin '{}' already exists. Replace it?", library_config_path.string()))
+                if (options.install_shut_up || askUserYorN(false, "Plugin '{}' already exists. Replace it?", library_config_path.string()))
                     fs::remove_all(library_config_path);
                 else
                     continue;

cufetchpm/src/stateManager.cpp

@@ -174,12 +174,14 @@ std::vector<manifest_t> StateManager::get_all_repos()
 
 void StateManager::remove_repo(const std::string& repo)
 {
-    toml::table& repo_tbl = *m_state["repositories"].as_table();
-    if (!repo_tbl.contains(repo))
+    toml::table* repo_tbl = m_state["repositories"].as_table();
+    if (!repo_tbl->contains(repo))
         return;
 
-    repo_tbl.erase(repo);
+    repo_tbl->erase(repo);
     std::stringstream ss;
+    ss << "# AUTO-GENERATED FILE. DO NOT EDIT THIS FILE.\n";
+    ss << "# YOU GONNA MESS SHIT UP. unless you know what you doing ofc\n";
     ss << m_state;
 
     if (!writeState(ss.str(), m_path))