Skip to content

fix: rate-limit UX, auth retry, verbose persistence#407

Merged
koala73 merged 1 commit intomainfrom
fix/rate-limit-ux-auth-verbose
Feb 26, 2026
Merged

fix: rate-limit UX, auth retry, verbose persistence#407
koala73 merged 1 commit intomainfrom
fix/rate-limit-ux-auth-verbose

Conversation

@koala73
Copy link
Owner

@koala73 koala73 commented Feb 26, 2026

Summary

  • Show rate-limited message instead of generic "Failed to load" on Markets, ETF, Commodities, and Sector panels when Yahoo Finance returns 429
  • fetchYahooQuotesBatch returns rateLimited flag with early-exit after 3 consecutive misses
  • ETF panel skips retry loop when rate-limited, shows specific i18n message
  • Fallback Finnhub symbols through Yahoo when API key is missing
  • 401-retry in runtime fetch patch for stale sidecar token after restart
  • diagFetch auth helper for settings window diagnostic endpoints
  • Verbose toggle writes to writable dataDir instead of read-only app bundle

Test plan

  • Launch app without Finnhub key — markets should load via Yahoo fallback
  • When Yahoo rate-limits (429), panels show "rate limited — retrying shortly" not "Failed to load"
  • ETF panel shows rate-limit message and does not retry 3x when rate-limited
  • Toggle verbose logging in settings — persists across app restart
  • Settings diagnostics (traffic log, verbose toggle) work with auth token
  • Sidecar token refresh works after sidecar restart (no persistent 401s)

…sistence

- Show rate-limited message instead of generic "Failed to load" on Markets,
  ETF, Commodities, and Sector panels when Yahoo returns 429
- fetchYahooQuotesBatch returns rateLimited flag; early-exit after 3 misses
- ETF panel skips retry loop when rate-limited, shows specific i18n message
- Fallback Finnhub symbols through Yahoo when API key missing
- 401-retry in runtime fetch patch for stale sidecar token after restart
- diagFetch auth helper for settings window diagnostic endpoints
- Verbose toggle writes to writable dataDir instead of read-only app bundle
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@vercel
Copy link

vercel bot commented Feb 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
worldmonitor Building Building Preview, Comment Feb 26, 2026 10:39am
worldmonitor-finance Building Building Preview, Comment Feb 26, 2026 10:39am
worldmonitor-happy Building Building Preview, Comment Feb 26, 2026 10:39am
worldmonitor-startup Building Building Preview, Comment Feb 26, 2026 10:39am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant