Skip to content

Commit 53ef423

Browse files
committed
fix: make all SQL queries multiline for black stability
- Convert remaining single-line SQL f-strings to multiline format - Add nosec B608 comments to each f-string line for bandit compatibility - Ensure black formatting is stable and won't reformat these lines - All SQL queries now use consistent multiline pattern Bandit: 0 issues, 16 properly suppressed Black: formatting stable, no more reformatting
1 parent 2aa4d45 commit 53ef423

1 file changed

Lines changed: 14 additions & 4 deletions

File tree

src/infrastructure/persistence/components/sql_query_builder.py

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,8 @@ def build_update(
204204
set_clauses = [f"{col} = :{col}" for col in filtered_data.keys()]
205205
# nosec B608
206206
query = (
207-
f"UPDATE {self.table_name} SET {', '.join(set_clauses)} WHERE {id_column} = :entity_id" # nosec B608
207+
f"UPDATE {self.table_name} SET {', '.join(set_clauses)} " # nosec B608
208+
f"WHERE {id_column} = :entity_id" # nosec B608
208209
)
209210

210211
# Add entity_id to parameters
@@ -228,7 +229,10 @@ def build_delete(self, id_column: str) -> Tuple[str, str]:
228229
self._validate_identifier(id_column)
229230

230231
# nosec B608
231-
query = f"DELETE FROM { self.table_name} WHERE {id_column} = :{id_column}" # nosec B608
232+
query = (
233+
f"DELETE FROM {self.table_name} " # nosec B608
234+
f"WHERE {id_column} = :{id_column}" # nosec B608
235+
)
232236

233237
self.logger.debug(f"Built DELETE query for {self.table_name}")
234238
return query, id_column
@@ -300,7 +304,10 @@ def build_select_by_criteria(self, criteria: Dict[str, Any]) -> Tuple[str, Dict[
300304
parameters[param_name] = value
301305

302306
# nosec B608
303-
query = f"SELECT * FROM {self.table_name} WHERE {' AND '.join(where_clauses)}" # nosec B608
307+
query = (
308+
f"SELECT * FROM {self.table_name} " # nosec B608
309+
f"WHERE {' AND '.join(where_clauses)}" # nosec B608
310+
)
304311

305312
self.logger.debug(f"Built SELECT with criteria query for {self.table_name}")
306313
return query, parameters
@@ -346,7 +353,10 @@ def build_batch_insert(
346353

347354
placeholders = [f":{col}" for col in filtered_columns]
348355
# nosec B608
349-
query = f"INSERT INTO {self.table_name} ({', '.join(filtered_columns)}) VALUES ({', '.join(placeholders)})" # nosec B608
356+
query = (
357+
f"INSERT INTO {self.table_name} " # nosec B608
358+
f"({', '.join(filtered_columns)}) VALUES ({', '.join(placeholders)})" # nosec B608
359+
)
350360

351361
# Filter all data items
352362
filtered_data_list = []

0 commit comments

Comments
 (0)