| #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 |
- |
|
| #579 |
[LOW] Missing docstrings on some internal helper functions
Some internal helper functions lack docstrings, making code harder to understand. Fix: Add docstring...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #578 |
[LOW] Model naming inconsistencies between API and engine
Some model/schema names differ between API layer and engine layer (e.g., WorkflowDefinition vs workf...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #577 |
[LOW] Inconsistent async/sync function declarations in blueprints
Some blueprint endpoints use async def while others use def despite all being async handlers. Quart ...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #576 |
[LOW] List literals that should be tuples for immutability
Several list literals are used for constant data that never changes (e.g., HTTP methods, field names...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #575 |
[LOW] Unused parameters in some API endpoint handlers
Some endpoint handlers accept parameters (like request objects) that are never used. Fix: Remove unu...
|
open |
low |
2025-12-17 16:33 |
- |
|
| #574 |
[LOW] Magic numbers in pagination defaults scattered across files
Magic numbers like 20, 50, 100, 200 for pagination defaults are scattered across multiple files. Fix...
|
open |
low |
2025-12-17 16:33 |
- |
|