| #606 |
Bare except Exception: clauses in engine module
167 bare except clauses. Most intentional: config fallback (return defaults), cleanup/shutdown code....
|
open |
medium |
2025-12-17 22:27 |
- |
|
| #605 |
Unnecessary list() conversions
Multiple files use list(dict.keys()) where iteration would suffice. Check: workflow_logging_injector...
|
open |
medium |
2025-12-17 22:27 |
- |
|
| #604 |
Inefficient len() checks: multiple files
Use 'not x' instead of 'len(x)==0', use 'x' instead of 'len(x)>0'. Files: inline_executor.py, operat...
|
open |
medium |
2025-12-17 22:27 |
- |
|
| #596 |
f-string in logging: engine/security/auth_wrapper.py:69
Use lazy formatting instead of f-string
|
open |
high |
2025-12-17 22:27 |
- |
|
| #595 |
f-string in logging: engine/validation.py:264
Use lazy formatting instead of f-string
|
open |
high |
2025-12-17 22:27 |
- |
|
| #594 |
f-string in logging: engine/utils module
event_logger.py:81, chunking.py:112, tracing.py:61,71 - use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #593 |
f-string in logging: engine/durable_context.py
Lines 2032, 2057, 2154 - use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #592 |
f-string in logging: engine/code_versioning.py (15 occurrences)
Lines 102, 110, 115, 121, 144, 158, 227, 256, 289, 294, 401, 428, 528 - use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #591 |
f-string in logging: engine/db.py
Lines 150, 191, 220, 391, 537 - use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #590 |
f-string in logging: engine/sidecar_telemetry.py
Lines 340, 344, 499 - use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #589 |
f-string in logging: engine/policies/tenant_rate_limiter.py
Lines 179, 258, 282, 284, 306 - use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #588 |
f-string in logging: engine/replay module (30+ occurrences)
replay_context.py, replay_service.py, loader.py - many f-string logging calls
|
open |
high |
2025-12-17 22:27 |
- |
|
| #587 |
f-string in logging: engine/interpreters/operators.py (3 occurrences)
Lines 409, 538, 1217: Use lazy formatting
|
open |
high |
2025-12-17 22:27 |
- |
|
| #586 |
f-string in logging: engine/contracts/registry.py:235
Use lazy formatting instead of f-string
|
open |
high |
2025-12-17 22:27 |
- |
|
| #585 |
[LOW] Dict comprehensions could replace explicit loops
Several explicit for loops build dicts that could be more concisely written as dict comprehensions. ...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #584 |
[LOW] Import ordering inconsistent across modules
Import ordering varies between modules (stdlib, third-party, local). Fix: Standardize to PEP 8 impor...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #583 |
[LOW] Redundant isinstance checks after type narrowing
Some code paths have redundant isinstance checks that are always true due to earlier type narrowing....
|
open |
low |
2025-12-17 16:33 |
- |
|
| #582 |
[LOW] HTTP status codes as magic numbers instead of constants
HTTP status codes like 200, 400, 404, 500 are used as literals throughout the code. Fix: Use http.HT...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #581 |
[LOW] Empty except blocks that should log or re-raise
Some except blocks silently pass or return generic errors without logging. This makes debugging diff...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #580 |
[LOW] String literals repeated without constants
String literals like 'application/json', 'Bearer', error codes are repeated multiple times. Fix: Def...
|
open |
low |
2025-12-17 16:33 |
- |
|