📜 Description
// ❌ PROBLEMATIC CODE
const octokit = useMemo(() => {
if (!username) return null;
if(token){
return new Octokit({ auth: token });
}
return new Octokit(); // Unauthenticated requests have strict rate limits
}, [username, token]);
Issue: When no token is provided, Octokit makes unauthenticated requests (60 requests/hour). Should either require token or document this limitation.
Impact: Users hit rate limits quickly, poor user experience, app becomes unusable after few queries.
Recommended Fix: Add token requirement or implement client-side rate limiting with warning.
What browsers are you seeing the problem on?
No response
📃 Relevant Screenshots (Links)
No response
📜 Description
// ❌ PROBLEMATIC CODE
const octokit = useMemo(() => {
if (!username) return null;
if(token){
return new Octokit({ auth: token });
}
return new Octokit(); // Unauthenticated requests have strict rate limits
}, [username, token]);
Issue: When no token is provided, Octokit makes unauthenticated requests (60 requests/hour). Should either require token or document this limitation.
Impact: Users hit rate limits quickly, poor user experience, app becomes unusable after few queries.
Recommended Fix: Add token requirement or implement client-side rate limiting with warning.
What browsers are you seeing the problem on?
No response
📃 Relevant Screenshots (Links)
No response