Enhancement/custom uni UI lint

.gitignore

@@ -8,6 +8,7 @@ build/
 *.bin
 .flutter-plugins*
 packages/uni_ui/pubspec.lock
+packages/uni_lint/pubspec.lock
 
 # IDE files
 .DS_Store
@@ -19,3 +20,6 @@ packages/uni_ui/pubspec.lock
 # Key properties
 uni/android/key.properties
 **.jks
+
+# Custom Lint log
+custom_lint.log

packages/uni_lint/README.md

@@ -0,0 +1,7 @@
+# uni_lint
+
+This package contains custom lint rules for uni development. 
+
+Try to keep dependencies to the minimum. This package, ideally,
+should not have any dependencies, except for the analyzer package
+and custom_lint_builder.

packages/uni_lint/lib/uni_lint.dart

@@ -0,0 +1,58 @@
+import 'package:analyzer/error/listener.dart';
+import 'package:custom_lint_builder/custom_lint_builder.dart';
+import 'package:analyzer/dart/ast/ast.dart';
+
+PluginBase createPlugin() => UniUILint();
+
+class UniUILint extends PluginBase {
+  @override
+  List<LintRule> getLintRules(CustomLintConfigs configs) => [
+    NoStringLiteralsInWidgetsLint(),
+      ];
+}
+
+class NoStringLiteralsInWidgetsLint extends DartLintRule {
+  NoStringLiteralsInWidgetsLint() : super(code: _code);
+
+  static const _code = LintCode(
+    name: 'string_literals_lint',
+    problemMessage: 'String literals are not allowed inside a widget. Please pass this value as a parameter for the widget.',
+  );
+
+  @override
+  void run(
+    CustomLintResolver resolver,
+    ErrorReporter reporter,
+    CustomLintContext context,
+  ) {
+    context.registry.addStringLiteral((node) {
+      if (isInsideWidgetClass(node) && node.parent is VariableDeclaration) {
+        reporter.atNode(node, code);
+        //if (isStringLiteral(node)) {
+        //  reporter.atNode(node, code);
+        //}
+      }
+    });
+  }
+
+  bool isInsideWidgetClass(StringLiteral node) {
+    var parent = node.thisOrAncestorOfType<ClassDeclaration>();
+    if (parent == null) return false;
+
+    final extendsClause = parent.extendsClause;
+    if (extendsClause != null) {
+      final superclass = extendsClause.superclass;
+      print(superclass.element?.displayName == "StatelessWidget" || superclass.element?.displayName == "StatefulWidget");
+      return superclass.element?.displayName == "StatelessWidget" || superclass.element?.displayName == "StatefulWidget";
+    }
+    return false;
+  }
+
+  bool isStringLiteral(VariableDeclaration node) {
+    final initializer = node.initializer;
+    if (initializer is StringLiteral) {
+      return true;
+    }
+    return false;
+  }
+}

packages/uni_lint/pubspec.yaml

@@ -0,0 +1,18 @@
+name: uni_lint
+description: Custom lint rules for Uni.
+publish_to: none
+version: 1.0.0
+
+environment:
+  sdk: ">=3.4.0 <4.0.0"
+  # flutter: 3.24.3
+
+dependencies:
+  custom_lint_builder: ^0.6.8
+  analyzer: ^6.7.0
+  analyzer_plugin: ^0.11.3
+  # flutter:
+  #   sdk: flutter
+
+dev_dependencies:
+  test: any

packages/uni_ui/analysis_options.yaml

@@ -0,0 +1,8 @@
+analyzer:
+  plugins:
+    - custom_lint
+
+custom_lint:
+  debug: true
+  rules:
+    - string_literals_lint