refactor(ui): optimize config refresh with SWR auto-revalidation

[the-dev-z]

Summary

優化添加 AI 模型/交易所後的刷新體驗，使用 SWR 自動重新驗證。

Changes

Frontend (web/src/components/AITradersPage.tsx):

• 將 allModels 和 allExchanges 從 useState 改為 useSWR
• 保存後自動調用 mutate() 觸發重新請求
• 簡化 useEffect：移除手動加載邏輯，SWR 自動處理

Before

const [allModels, setAllModels] = useState<AIModel[]>([])
// useEffect 手動加載
// 保存後手動 setAllModels(refreshedModels)

After

const { data: allModels, mutate: mutateModels } = useSWR<AIModel[]>(
  user && token ? 'models' : null,
  api.getModelConfigs
)
// 保存後自動 mutateModels()

Benefits

1. 自動刷新：保存後立即更新 UI，無需手動刷新頁面
2. 代碼簡化：減少 useState 和手動請求邏輯 (-11 lines net)
3. 一致性：與 traders 列表使用相同的 SWR 模式
4. 性能優化：SWR 內建快取和去重機制

Technical Details

• handleSaveModelConfig: await api.updateModelConfigs() → mutateModels()
• handleSaveExchangeConfig: await api.updateExchangeConfigsEncrypted() → mutateExchanges()
• handleDeleteModelConfig: setAllModels([...items]) → mutateModels()
• handleDeleteExchangeConfig: setAllExchanges([...items]) → mutateExchanges()
• 保留 supportedModels/supportedExchanges 為 useState（無需自動刷新）

User Experience

• 添加/編輯模型後，列表立即更新顯示新配置
• 添加/編輯交易所後，列表立即更新顯示新配置
• 刪除配置後，列表立即移除
• 無需等待或手動刷新頁面

Testing

• ✅ Frontend lint passed
• ✅ Tested add/edit/delete model config
• ✅ Tested add/edit/delete exchange config
• ✅ Verified SWR auto-revalidation

🤖 Generated with Claude Code

[github-actions]

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟢 Small (95 lines: +42 -53)

🔧 Backend Checks

Go Formatting: ⚠️ Needs formatting

Files needing formatting

decision/prompt_manager_test.go
market/data_test.go
trader/hyperliquid_trader_race_test.go

Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

---

📖 Resources

• Contributing Guidelines
• Migration Guide

Questions? Feel free to ask in the comments! 🙏

---

These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.

[github-actions]

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟢 Small (97 lines: +43 -54)

🔧 Backend Checks

Go Formatting: ⚠️ Needs formatting

Files needing formatting

decision/prompt_manager_test.go
market/data_test.go
trader/hyperliquid_trader_race_test.go

Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

---

📖 Resources

• Contributing Guidelines
• Migration Guide

Questions? Feel free to ask in the comments! 🙏

---

These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.

[hzb1115]

Will this get models from backend service?

[the-dev-z]

Good catch! 🎯

Yes, mutateModels() will fetch from backend service.

However, I found a redundancy issue you helped uncover:

Before (had bug):

const refreshedItems = await config.refreshApi()  // 1st backend call
config.setItems(refreshedItems)                    // 2nd backend call (mutateModels ignores refreshedItems)

After (fixed in d87fe51):

config.setItems()  // Only 1 backend call via mutateModels()

Changes:

• Removed redundant refreshApi parameter
• Now only calls backend once via SWR mutate
• More efficient and follows SWR best practices ✅

Thanks for the review! 🙏

[github-actions]

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟢 Small (107 lines: +46 -61)

🔧 Backend Checks

Go Formatting: ✅ Good
Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

---

📖 Resources

• Contributing Guidelines
• Migration Guide

Questions? Feel free to ask in the comments! 🙏

---

These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.

[hzb1115]

@0xEmberZz Could you pls help review this?