fix: tiny improvements

Author: qubka

include/plugify/config.hpp

@@ -28,7 +28,7 @@ namespace plugify {
 
 	constexpr Severity DefaultVerbosity
 #ifndef NDEBUG
-	= Severity::Trace
+	= Severity::Debug
 #else
 	= Severity::Error
 #endif

schemas/config.schema.json

@@ -30,7 +30,7 @@
     "logSeverity": {
       "type": "string",
       "description": "Log severity.",
-      "enum": ["none", "fatal", "error", "warning", "info", "debug", "verbose"]
+      "enum": ["none", "fatal", "error", "warning", "info", "debug", "trace"]
     },
     "preferOwnSymbols": {
       "type": "boolean",

src/core/config.cpp

@@ -237,12 +237,14 @@ Result<void> Config::Validate() const{
 
 	for (size_t i = 0; i < pathList.size(); ++i) {
 		for (size_t j = i + 1; j < pathList.size(); ++j) {
-			if (*pathList[i].second == *pathList[j].second) {
+			const auto& [iname, ipath] = pathList[i];
+			const auto& [jname, jpath] = pathList[j];
+			if (*ipath == *jpath) {
 				return MakeError(
 					"{} and {} directories are the same: {}",
-					pathList[i].first,
-					pathList[j].first,
-					plg::as_string(*pathList[i].second)
+					iname,
+					jname,
+					plg::as_string(*ipath)
 				);
 			}
 		}

src/core/extension.cpp

@@ -8,9 +8,9 @@ using namespace plugify;
 // Implementation structure
 struct Extension::Impl {
 	// Core identity
-	UniqueId id{ -1 };
-	ExtensionType type{ ExtensionType::Unknown };
-	ExtensionState state{ ExtensionState::Unknown };
+	UniqueId id{};
+	ExtensionType type{};
+	ExtensionState state{};
 
 	// Runtime state
 	MethodTable methodTable;

src/core/extension_loader.hpp

@@ -101,7 +101,7 @@ namespace plugify {
 
 		// Module Operations
 		Result<void> LoadModule(Extension& module) {
-			auto timer = ScopedTimer([&](std::chrono::milliseconds elapsed) {
+			[[maybe_unused]] auto timer = ScopedTimer([&](std::chrono::milliseconds elapsed) {
 				_stats.totalLoadTime += elapsed;
 				if (elapsed > _stats.slowestModuleLoad) {
 					_stats.slowestModuleLoad = elapsed;
@@ -163,7 +163,7 @@ namespace plugify {
 
 		// Plugin Operations
 		Result<void> LoadPlugin(const Extension& module, Extension& plugin) {
-			auto timer = ScopedTimer([&](std::chrono::milliseconds elapsed) {
+			[[maybe_unused]] auto timer = ScopedTimer([&](std::chrono::milliseconds elapsed) {
 				_stats.totalLoadTime += elapsed;
 				if (elapsed > _stats.slowestPluginLoad) {
 					_stats.slowestPluginLoad = elapsed;