refactor: 💡 clean code

Author: chative-tu

lib/src/chative_widget.dart

@@ -76,15 +76,35 @@ class ChativeWidgetController {
 /// A StatefulWidget that displays a chat interface using WebView
 class ChativeWidget extends StatefulWidget {
   final ChativeWidgetController? controller;
+
+  /// The channel ID to be used for the chat widget
   final String channelId;
+
+  /// The user data to be passed to the chat widget
   final Map<String, dynamic>? user;
+
+  /// The widget to be displayed as the header of the chat widget
   final Widget? headerWidget;
+
+  /// The decoration to be applied to the chat widget container
   final BoxDecoration? containerDecoration;
+
+  /// The top inset of the chat widget
   final double insetTop;
+
+  /// The bottom inset of the chat widget
   final double insetBottom;
+
+  /// Callback function to be called when the chat widget is closed
   final OnClosed? onClosed;
+
+  /// Callback function to be called when the chat widget is loaded
   final OnLoaded? onLoaded;
+
+  /// Callback function to be called when a new message is received
   final OnNewMessage? onNewMessage;
+
+  /// Callback function to be called when an error occurs
   final OnError? onError;
 
   const ChativeWidget({

lib/src/utils.dart

@@ -2,6 +2,7 @@ import 'dart:convert';
 
 import 'package:chative_sdk/src/constants.dart';
 
+/// Generates the JavaScript script to be injected into the WebView
 String generateScript(Map<String, dynamic> user) {
   String userString = jsonEncode(user);
   return '''
@@ -37,6 +38,7 @@ String generateScript(Map<String, dynamic> user) {
   ''';
 }
 
+/// Generates the JavaScript script to be injected into the WebView to get the config
 String generateScriptGetError(String channelId) {
   return '''
     function getTimeZone() {
@@ -69,6 +71,7 @@ String generateScriptGetError(String channelId) {
   ''';
 }
 
+/// Converts a Dart object into a JavaScript object string
 String widgetApi(String event, dynamic data) {
   String dataString;
   if (data is String) {
@@ -118,3 +121,12 @@ bool isScriptSafe(String script) {
 
   return true;
 }
+
+/// Safely parses a JSON string into a Map
+Map<String, dynamic> parseJson(String message) {
+  try {
+    return jsonDecode(message) as Map<String, dynamic>;
+  } catch (e) {
+    return {};
+  }
+}

lib/src/webview.dart

@@ -1,4 +1,3 @@
-import 'dart:convert';
 import 'dart:io';
 
 import 'package:file_picker/file_picker.dart';
@@ -16,22 +15,33 @@ typedef OnNewMessage = void Function();
 typedef OnError = void Function(String message);
 
 class Webview extends StatefulWidget {
+  /// The unique identifier of the channel
   final String channelId;
+
+  /// The user information to be passed to the widget
   final Map<String, dynamic>? user;
+
+  /// Callback when the widget has loaded
   final OnLoaded? onLoaded;
+
+  /// Callback when the widget has been closed
   final OnClosed? onClosedWidget;
+
+  /// Callback when a new message has been received
   final OnNewMessage? onNewMessage;
+
+  /// Callback when an error has occurred
   final OnError? onError;
 
   const Webview({
-    Key? key,
+    super.key,
     required this.channelId,
     this.user,
     this.onLoaded,
     this.onClosedWidget,
     this.onNewMessage,
     this.onError,
-  }) : super(key: key);
+  });
 
   @override
   State<Webview> createState() => WebviewState();
@@ -72,7 +82,7 @@ class WebviewState extends State<Webview> {
 
   /// Handles JavaScript messages received from the WebView
   void _handleJavaScriptMessage(JavaScriptMessage message) {
-    final parsedData = _parseJson(message.message);
+    final parsedData = parseJson(message.message);
 
     switch (parsedData['event']) {
       case 'closed':
@@ -93,15 +103,6 @@ class WebviewState extends State<Webview> {
     }
   }
 
-  /// Safely parses a JSON string into a Map
-  Map<String, dynamic> _parseJson(String message) {
-    try {
-      return jsonDecode(message) as Map<String, dynamic>;
-    } catch (e) {
-      return {};
-    }
-  }
-
   /// Creates a NavigationDelegate for the WebView
   NavigationDelegate _createNavigationDelegate() {
     return NavigationDelegate(
@@ -203,6 +204,12 @@ class WebviewState extends State<Webview> {
 
   /// Displays an external WebView when the URL is outside the widgetUrl
   void _showExternalWebView(String url) {
+    Uri uri = Uri.parse(url);
+
+    if (uri.scheme != 'https' && uri.scheme != 'http') {
+      return;
+    }
+
     _externalController = WebViewController()
       ..setJavaScriptMode(JavaScriptMode.unrestricted)
       ..loadRequest(Uri.parse(url));