fix: resolve 2 SonarQube issues (2 minor)

[sonarqube-agent]

This PR includes automated code changes to address 2 SonarQube issues: MINOR (2).

View Project in SonarCloud

---

Fixed Issues

python:S7498 - Replace this constructor call with a literal. • MINOR • View issue

Location: src/operations/apply_filter.py:34

What changed

This change replaces the use of the dict() constructor with a dictionary literal when building the payload: it changes payload=dict(message=f"{warn_message}: '{left}'") to payload={'message': f"{warn_message}: '{left}'"}. The static analysis warning was triggered because using dict() for simple dictionary initialization is less idiomatic and slightly less efficient than a literal. By switching to the literal form, the code now follows the recommended, more Pythonic style and eliminates the reported code smell.

--- a/src/operations/apply_filter.py
+++ b/src/operations/apply_filter.py
@@ -34,1 +34,1 @@ def execute(left, right, operator, warn_message=None, **kwargs):
-                  payload=dict(message=f"{warn_message}: '{left}'"))
+                  payload={'message': f"{warn_message}: '{left}'"})

python:S7498 - Replace this constructor call with a literal. • MINOR • View issue

Location: src/operations/http_request/base.py:103

What changed

Replaces the use of the dict() constructor to create an empty dictionary with the literal {} in async_safe_json_request. The static analysis warning was about preferring literal syntax for creating empty collections for readability, consistency, and minor performance benefits. Changing status_code, js = None, dict() to status_code, js = None, {} directly addresses that concern.

--- a/src/operations/http_request/base.py
+++ b/src/operations/http_request/base.py
@@ -103,1 +103,1 @@ async def async_safe_json_request(client, method, url, log_attributes: dict, rai
-        status_code, js = None, dict()
+        status_code, js = None, {}

---

SonarQube Remediation Agent uses AI. Check for mistakes.

[sonarqubecloud]

SonarQube reviewer guide

Summary: Code cleanup refactoring that improves readability and maintainability by extracting magic strings to constants, simplifying list comprehensions, and reformatting dictionary literals for consistency.

Review Focus: Verify that the extracted constants (HTTPS_PREFIX, HTTPS_API_PREFIX) are correctly applied across all URL transformations in the migrate/reset/full_migrate functions. Ensure the dictionary literal reformatting doesn't introduce any subtle syntax issues, particularly in the JSON serialization calls where formatting changes are most extensive.

Start review at: src/main.py. This file contains the majority of changes (70+ lines) and includes the critical constant extraction at the top that affects multiple functions. The refactoring of large dictionary literals in full_migrate is especially important to validate for correctness in JSON output.

💬 Please send your feedback

Quality Gate failed

Failed conditions
15.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud