chore: lint fixes

Author: dervoeti

backend/application/commons/migrations/0004_settings_background_exploits_import_crontab_hours_and_more.py

@@ -38,15 +38,11 @@ class Migration(migrations.Migration):
         migrations.AddField(
             model_name="settings",
             name="exploits_last_sync_github_poc",
-            field=models.DateTimeField(
-                help_text="Last sync date of GitHub PoC data", null=True
-            ),
+            field=models.DateTimeField(help_text="Last sync date of GitHub PoC data", null=True),
         ),
         migrations.AddField(
             model_name="settings",
             name="exploits_last_sync_vulncheck_kev",
-            field=models.DateTimeField(
-                help_text="Last sync date of Vulncheck KEV data", null=True
-            ),
+            field=models.DateTimeField(help_text="Last sync date of Vulncheck KEV data", null=True),
         ),
     ]

backend/application/commons/migrations/0016_merge_20250328_1143.py

@@ -6,9 +6,8 @@
 class Migration(migrations.Migration):
 
     dependencies = [
-        ('commons', '0013_merge_20241028_1602'),
-        ('commons', '0015_settings_exploit_information_max_age_years_and_more'),
+        ("commons", "0013_merge_20241028_1602"),
+        ("commons", "0015_settings_exploit_information_max_age_years_and_more"),
     ]
 
-    operations = [
-    ]
+    operations = []

backend/application/commons/models.py

@@ -166,13 +166,9 @@ class Settings(Model):
     feature_disable_user_login = BooleanField(default=False, help_text="Disable user login")
     feature_general_rules_need_approval = BooleanField(default=False, help_text="General rules need approval")
 
-    exploits_last_sync_vulncheck_kev = DateTimeField(
-        null=True, help_text="Last sync date of Vulncheck KEV data"
-    )
+    exploits_last_sync_vulncheck_kev = DateTimeField(null=True, help_text="Last sync date of Vulncheck KEV data")
 
-    exploits_last_sync_github_poc = DateTimeField(
-        null=True, help_text="Last sync date of GitHub PoC data"
-    )
+    exploits_last_sync_github_poc = DateTimeField(null=True, help_text="Last sync date of GitHub PoC data")
 
     background_exploits_import_crontab_minutes = IntegerField(
         default=0,

backend/application/core/api/filters.py

@@ -188,18 +188,13 @@ def get_for_license_components(
     )
 
     def filter_branch_name_with_product(
-        self, queryset, field_name, value
-    ):  # pylint: disable=unused-argument
+        self, queryset: QuerySet, field_name: Any, value: Any
+    ) -> QuerySet:  # pylint: disable=unused-argument
         # field_name is used as a positional argument
 
         # check if branch or product name matches the search value
         return queryset.filter(
-            Q(name__icontains=value)
-            | Q(
-                product_id__in=Product.objects.filter(name__icontains=value).values(
-                    "id"
-                )
-            )
+            Q(name__icontains=value) | Q(product_id__in=Product.objects.filter(name__icontains=value).values("id"))
         )
 
     class Meta:

backend/application/core/api/serializers_observation.py

@@ -380,7 +380,7 @@ def update(self, instance: Observation, validated_data: dict) -> Observation:
                 status=log_status,
                 comment="Observation changed manually",
                 vex_justification=actual_vex_justification,
-                vex_remediations=actual_vex_remediations,
+                vex_remediations=str(actual_vex_remediations) if actual_vex_remediations is not None else "",
                 assessment_status=Assessment_Status.ASSESSMENT_STATUS_AUTO_APPROVED,
                 risk_acceptance_expiry_date=log_risk_acceptance_expiry_date,
             )
@@ -475,7 +475,9 @@ def create(self, validated_data: dict) -> Observation:
             status=observation.current_status,
             comment="Observation created manually",
             vex_justification=observation.current_vex_justification,
-            vex_remediations=observation.current_vex_remediations,
+            vex_remediations=(
+                str(observation.current_vex_remediations) if observation.current_vex_remediations is not None else ""
+            ),
             assessment_status=Assessment_Status.ASSESSMENT_STATUS_AUTO_APPROVED,
             risk_acceptance_expiry_date=observation.risk_acceptance_expiry_date,
         )

backend/application/core/api/views.py

@@ -500,7 +500,7 @@ def get_queryset(self) -> QuerySet[Observation]:
             .select_related("parser")
         )
 
-    def filter_queryset(self, queryset):
+    def filter_queryset(self, queryset: QuerySet) -> QuerySet:
         queryset = super().filter_queryset(queryset)
 
         order_by = list(queryset.query.order_by)
@@ -698,23 +698,6 @@ def bulk_approval(self, request: Request) -> Response:
         )
         return Response(status=HTTP_204_NO_CONTENT)
 
-    @extend_schema(
-        methods=["DELETE"],
-        request=ObservationLogBulkApprovalSerializer,
-        responses={HTTP_204_NO_CONTENT: None},
-    )
-    @action(detail=False, methods=["delete"])
-    def bulk_delete(self, request):
-        request_serializer = ObservationLogBulkDeleteSerializer(data=request.data)
-        if not request_serializer.is_valid():
-            raise ValidationError(request_serializer.errors)
-
-        Observation_Log.objects.filter(
-            id__in=request_serializer.validated_data.get("observation_logs"),
-            user=get_current_user(),
-        ).delete()
-        return Response(status=HTTP_204_NO_CONTENT)
-
     @extend_schema(
         methods=["GET"],
         request=None,

backend/application/core/migrations/0042_observation_core_observ_stackab_f65ce5_idx.py

@@ -12,8 +12,6 @@ class Migration(migrations.Migration):
     operations = [
         migrations.AddIndex(
             model_name="observation",
-            index=models.Index(
-                fields=["stackable_score"], name="core_observ_stackab_f65ce5_idx"
-            ),
+            index=models.Index(fields=["stackable_score"], name="core_observ_stackab_f65ce5_idx"),
         ),
     ]

backend/application/core/migrations/0044_observation_patch_available_and_more.py

@@ -16,8 +16,6 @@ class Migration(migrations.Migration):
         ),
         migrations.AddIndex(
             model_name="observation",
-            index=models.Index(
-                fields=["patch_available"], name="core_observ_patch_a_6e4989_idx"
-            ),
+            index=models.Index(fields=["patch_available"], name="core_observ_patch_a_6e4989_idx"),
         ),
     ]

backend/application/core/migrations/0048_exploit_observation_core_observ_in_vuln_bb4e6d_idx_and_more.py

@@ -43,15 +43,11 @@ class Migration(migrations.Migration):
         ),
         migrations.AddIndex(
             model_name="observation",
-            index=models.Index(
-                fields=["in_vulncheck_kev"], name="core_observ_in_vuln_bb4e6d_idx"
-            ),
+            index=models.Index(fields=["in_vulncheck_kev"], name="core_observ_in_vuln_bb4e6d_idx"),
         ),
         migrations.AddIndex(
             model_name="observation",
-            index=models.Index(
-                fields=["exploit_available"], name="core_observ_exploit_bc386d_idx"
-            ),
+            index=models.Index(fields=["exploit_available"], name="core_observ_exploit_bc386d_idx"),
         ),
         migrations.AddIndex(
             model_name="exploit",
@@ -62,8 +58,6 @@ class Migration(migrations.Migration):
         ),
         migrations.AddIndex(
             model_name="exploit",
-            index=models.Index(
-                fields=["-created"], name="core_exploi_created_0f7300_idx"
-            ),
+            index=models.Index(fields=["-created"], name="core_exploi_created_0f7300_idx"),
         ),
     ]

backend/application/core/migrations/0049_observation_exploits_alter_exploit_vulnerability_id.py

@@ -13,9 +13,7 @@ class Migration(migrations.Migration):
         migrations.AddField(
             model_name="observation",
             name="exploits",
-            field=models.ManyToManyField(
-                blank=True, related_name="observations", to="core.exploit"
-            ),
+            field=models.ManyToManyField(blank=True, related_name="observations", to="core.exploit"),
         ),
         migrations.AlterField(
             model_name="exploit",