Merge pull request #910 from avisikta17pal/feature/logging-validation-console

fix(logging): redirect request validation warnings & errors to in-app console (#906, #587)

lib/main.dart

@@ -14,7 +14,6 @@ void main() async {
   await Stac.initialize();
 
   //Load all LLMs
-  // await LLMManager.fetchAvailableLLMs();
   await ModelManager.fetchAvailableModels();
 
   var settingsModel = await getSettingsFromSharedPrefs();
@@ -30,9 +29,6 @@ void main() async {
     settingsModel = settingsModel?.copyWithPath(workspaceFolderPath: null);
   }
 
-  // TODO: Load all models at init
-  // await ModelManager.loadAvailableLLMs();
-
   runApp(
     ProviderScope(
       overrides: [

lib/providers/collection_providers.dart

@@ -350,8 +350,20 @@ class CollectionStateNotifier
           executionRequestModel.httpRequestModel!);
     }
 
-    // Terminal: start network log
+    // Terminal
     final terminal = ref.read(terminalStateProvider.notifier);
+
+    var valRes = getValidationResult(substitutedHttpRequestModel);
+    if (valRes != null) {
+      terminal.logSystem(
+        category: 'validation',
+        message: valRes,
+        level: TerminalLevel.error,
+      );
+      ref.read(showTerminalBadgeProvider.notifier).state = true;
+    }
+
+    // Terminal: start network log
     final logId = terminal.startNetwork(
       apiType: executionRequestModel.apiType,
       method: substitutedHttpRequestModel.method,

lib/providers/ui_providers.dart

@@ -13,6 +13,7 @@ final historyCodePaneVisibleStateProvider = StateProvider<bool>((ref) => false);
 final saveDataStateProvider = StateProvider<bool>((ref) => false);
 final clearDataStateProvider = StateProvider<bool>((ref) => false);
 final hasUnsavedChangesProvider = StateProvider<bool>((ref) => false);
+final showTerminalBadgeProvider = StateProvider<bool>((ref) => false);
 
 // final nameTextFieldControllerProvider =
 //     StateProvider.autoDispose<TextEditingController>((ref) {

lib/screens/dashboard.dart

@@ -76,13 +76,21 @@ class Dashboard extends ConsumerWidget {
                       style: Theme.of(context).textTheme.labelSmall,
                     ),
                     kVSpacer10,
-                    IconButton(
-                      isSelected: railIdx == 3,
-                      onPressed: () {
-                        ref.read(navRailIndexStateProvider.notifier).state = 3;
-                      },
-                      icon: const Icon(Icons.terminal_outlined),
-                      selectedIcon: const Icon(Icons.terminal),
+                    Badge(
+                      backgroundColor: Theme.of(context).colorScheme.error,
+                      isLabelVisible:
+                          ref.watch(showTerminalBadgeProvider) && railIdx != 3,
+                      child: IconButton(
+                        isSelected: railIdx == 3,
+                        onPressed: () {
+                          ref.read(navRailIndexStateProvider.notifier).state =
+                              3;
+                          ref.read(showTerminalBadgeProvider.notifier).state =
+                              false;
+                        },
+                        icon: const Icon(Icons.terminal_outlined),
+                        selectedIcon: const Icon(Icons.terminal),
+                      ),
                     ),
                     Text(
                       'Logs',

lib/utils/utils.dart

@@ -8,4 +8,5 @@ export 'http_utils.dart';
 export 'js_utils.dart';
 export 'save_utils.dart';
 export 'ui_utils.dart';
+export 'validation_utils.dart';
 export 'window_utils.dart';

lib/utils/validation_utils.dart

@@ -0,0 +1,18 @@
+import 'package:apidash_core/apidash_core.dart';
+
+String? getValidationResult(HttpRequestModel requestModel) {
+  if (requestModel.url.trim().isEmpty) {
+    return 'Request URL is empty. Please provide a valid URL.';
+  }
+  if (requestModel.method == HTTPVerb.get && requestModel.hasAnyBody) {
+    return 'GET request contains a body. This is not supported.';
+  }
+  if (requestModel.hasJsonData) {
+    try {
+      kJsonDecoder.convert(requestModel.body!);
+    } catch (e) {
+      return 'Invalid JSON in request body: ${e.toString()}';
+    }
+  }
+  return null;
+}

packages/better_networking/lib/models/http_request_model.dart

@@ -47,6 +47,10 @@ class HttpRequestModel with _$HttpRequestModel {
   bool get hasTextContentType => bodyContentType == ContentType.text;
   int get contentLength => utf8.encode(body ?? "").length;
   bool get hasBody => hasJsonData || hasTextData || hasFormData;
+  bool get hasAnyBody =>
+      (hasJsonContentType && contentLength > 0) ||
+      (hasTextContentType && contentLength > 0) ||
+      (hasFormDataContentType && formDataMapList.isNotEmpty);
   bool get hasJsonData =>
       kMethodsWithBody.contains(method) &&
       hasJsonContentType &&