From f1394d779a58d98a28ba02fe59c9cce57caa0726 Mon Sep 17 00:00:00 2001 From: shinchan-zhai Date: Mon, 23 Mar 2026 10:30:32 +0800 Subject: [PATCH] =?UTF-8?q?docs:=20update=20nofxi-tasks.md=20=E2=80=94=20r?= =?UTF-8?q?ecord=20complete=20panic=20recovery=20coverage?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- nofxi-tasks.md | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/nofxi-tasks.md b/nofxi-tasks.md index cb008bc8..e1bb6476 100644 --- a/nofxi-tasks.md +++ b/nofxi-tasks.md @@ -31,9 +31,20 @@ ## Pending +### 2026-03-23 10:22 — Complete Panic Recovery Coverage +- [DONE] Apply `safe.GoNamed` to all 27 remaining bare goroutines across 21 files: + — 9 exchange order_sync (OKX, Hyperliquid, Aster, Bybit, KuCoin, Gate, Bitget, Lighter, Binance×2) + — Drawdown monitor, Brain news+briefs, Sentinel, Scheduler + — x402 + MCP stream idle watchdogs, Rate limiter cleanup + — 3 telemetry sends, CoinAnk WS handler, API server goroutine + — Manual defer/recover for Telegram bot (sends error to user) and trader data fetch (prevents deadlock) +- [DONE] Zero bare `go func()` remaining outside safe.go itself +- Build verified ✅, pushed to feat/nofxi + +## Pending + ### Security - [PENDING] Investigate GitHub Dependabot's 21 reported vulnerabilities (13 high, 7 moderate, 1 low) -- [PENDING] Apply `safe.Go` to remaining 25+ goroutines (order_sync, telegram bot, agent brain, etc.) ### Code Quality - [PENDING] `context.Background()` used in ~69 exchange/trader calls — should propagate request context for proper cancellation (partially done: kline handlers fixed, trader/exchange calls remain)