| #469 |
Performance: O(N log N) sorted() for cache eviction in activity_worker.py
engine/services/activity_worker.py:120-125 uses sorted() for LRU cache eviction.
sorted() is O(N lo...
|
closed |
low |
2025-12-17 01:11 |
- |
|
| #451 |
P4: Dynamic Table Name Discovery in Janitor
Issue #16 from review: Hardcoded pattern breaks on schema changes. Location: engine/services/janitor...
|
closed |
low |
2025-12-16 00:57 |
- |
|
| #450 |
P4: Add Encryption Migration Script
Issue #15 from review: Old data remains unencrypted. Location: engine/security/jsonb_encryption.py. ...
|
closed |
low |
2025-12-16 00:57 |
- |
|
| #449 |
P4: Set Real __file__ Path for Artifacts
Issue #13 from review: Libraries fail with virtual path. Location: engine/tools/python_task.py:237. ...
|
closed |
low |
2025-12-16 00:57 |
- |
|
| #435 |
App: Container Task Runner (container_runner)
## Overview
Execute arbitrary Docker containers as workflow tasks with input/output handling. Enable...
|
closed |
low |
2025-12-14 20:17 |
- |
|
| #433 |
App: Invoice Generator (invoice_generator)
## Overview
Generate professional PDF invoices from structured data with email delivery and S3 archi...
|
closed |
low |
2025-12-14 20:17 |
- |
|
| #412 |
CONSOLIDATED: Low Priority Fixes and Minor Improvements
## Consolidated Low Priority Issues
### Code Quality
1. ~~**Logger f-string** (ex #371, #396)~~: FI...
|
closed |
low |
2025-12-12 14:48 |
- |
|
| #409 |
Automated partition management (Janitor service)
Review Rec 3: Integrate Janitor service to automatically manage and create future database partition...
|
closed |
low |
2025-12-12 06:06 |
- |
|
| #408 |
ParallelGroup DSL context manager for safer parallel workflows
Review Rec 4: The fork-only parallel model is safe but developer-hostile. Introduce a ParallelGroup ...
|
closed |
low |
2025-12-12 06:06 |
- |
|
| #406 |
absurd_client.py inconsistent use of _table() method vs f-strings
engine/absurd_client.py:1023,1107 uses f-strings for table names despite having safe _table() method...
|
closed |
low |
2025-12-12 05:49 |
- |
|
| #405 |
orchestrator.py uses f-strings for SQL table names instead of sql.Identifier()
engine/orchestrator.py:277,372,553,762,794,988 uses f-strings for absurd table names. While queue_na...
|
closed |
low |
2025-12-12 05:49 |
- |
|
| #404 |
assert statements in db.py can be bypassed with Python -O flag
engine/db.py:445-448 uses assert for validation. When Python runs with -O flag, asserts are stripped...
|
closed |
low |
2025-12-12 05:49 |
- |
|
| #401 |
LOW: API pagination offset unbounded for DoS
API endpoints (logs.py:1220, schedules.py:685) don't bound offset. Large offset=999999999 forces Pos...
|
closed |
low |
2025-12-11 22:44 |
- |
|
| #400 |
LOW: Bare except clause in sandbox catches SystemExit
sandbox.py:306 - Uses bare 'except:' catching all exceptions including SystemExit and KeyboardInterr...
|
closed |
low |
2025-12-11 22:44 |
- |
|
| #396 |
LOW: Logger missing f-string prefix in orchestrator failure path
orchestrator.py:574 - logger.exception('Task {task_name} failed') missing f-string prefix. Task name...
|
closed |
low |
2025-12-11 22:21 |
- |
|
| #382 |
CLEANUP: Remove deprecated SchedulerService (superseded by durable_cron)
SchedulerService and scheduler_worker.py are dead code. durable_cron (Issue #19) supersedes them com...
|
closed |
low |
2025-12-11 22:12 |
- |
|
| #372 |
LOW: Magic numbers should be configurable
## Issue
Hardcoded values throughout codebase should be in configuration:
- activity_threshold_sec...
|
closed |
low |
2025-12-11 21:40 |
- |
|
| #371 |
LOW: Logger message missing f-string prefix
## Issue
In engine/orchestrator.py:577, logger message uses wrong format string:
```python
logger....
|
closed |
low |
2025-12-11 21:40 |
- |
|
| #340 |
Flaky tests: test_concurrent_logging and test_async_deferral_multiple_jobs
These tests fail intermittently under parallel load (pytest -n auto) but pass in isolation. Root cau...
|
closed |
low |
2025-12-10 05:17 |
- |
|
| #334 |
Sherlock Phase 4: Dynamic Verifier Injection
PHASE 4: Dynamic Injection (Orchestration)
CONTEXT:
Sherlock suggests 'Topology-Driven Verifier Pla...
|
closed |
low |
2025-12-09 17:52 |
- |
|