tasks island: parallel list + delete confirmation + SDK #29 workaround #97

New issue

Closed

opened 2026-04-21 01:26:55 +00:00 by zaelgohary · 0 comments

zaelgohary commented

2026-04-21 01:26:55 +00:00

Member

Three HIGH-severity issues in the tasks island mirroring what was already fixed in Projects:

fetch_tasks in services/task_service.rs sequentially awaits each task_get(&sid) in a loop — N+1 latency on list load. Should batch via futures::future::join_all.
TaskDetailView and the list-view inline row both call delete_task on a single click with no confirmation modal. Use hero_archipelagos_core::ConfirmDialog.
delete_task returns Err on the SDK bool/String deserialization bug tracked as hero_osis #29. Swallow the "invalid type: boolean" error the same way projects does.

Three HIGH-severity issues in the tasks island mirroring what was already fixed in Projects: - `fetch_tasks` in `services/task_service.rs` sequentially awaits each `task_get(&sid)` in a loop — N+1 latency on list load. Should batch via `futures::future::join_all`. - `TaskDetailView` and the list-view inline row both call `delete_task` on a single click with no confirmation modal. Use `hero_archipelagos_core::ConfirmDialog`. - `delete_task` returns `Err` on the SDK bool/String deserialization bug tracked as hero_osis #29. Swallow the `"invalid type: boolean"` error the same way projects does.