Address PR review feedback: enhance error handling and fix redundant logic

[Copilot]

Addresses review feedback from PR #264 to improve error handling robustness and eliminate redundant conditional logic.

Changes

• Enhanced connection error handling in save-token route

  • Added ECONNREFUSED to network error code list alongside existing codes (ETIMEDOUT, ENOTFOUND, EHOSTUNREACH, etc.)
  • Ensures consistent "service unavailable" responses for all connection failures

• Simplified auth error detection in Supabase middleware

  • Removed duplicate 'invalid'/'expired' checks in nested token condition
  • Outer-level conditions already capture these cases

• Glob dependency note

  • Review suggested updating to 13.2.0, but npm registry shows 13.0.1 is latest
  • Current version is correct; no changes needed

// Before: Missing ECONNREFUSED
if (isAxiosError(err) && err.code && 
    ['ECONNABORTED', 'ETIMEDOUT', 'ENOTFOUND', 'EHOSTUNREACH', 'ECONNRESET', 'ENETUNREACH'].includes(err.code))

// After: Complete connection error coverage
if (isAxiosError(err) && err.code && 
    ['ECONNABORTED', 'ETIMEDOUT', 'ENOTFOUND', 'EHOSTUNREACH', 'ECONNRESET', 'ENETUNREACH', 'ECONNREFUSED'].includes(err.code))

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR addresses review feedback from PR #264 by refining error handling and removing redundant logic. The changes enhance the robustness of connection error handling in the authentication flow and simplify the token error detection logic in middleware by eliminating duplicate checks.

Changes:

• Added ECONNREFUSED to network error code handling in save-token route for comprehensive connection failure coverage
• Removed redundant 'invalid' and 'expired' checks from nested token condition in Supabase middleware, as these are already caught by outer-level checks

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/app/api/auth/save-token/route.ts	Added ECONNREFUSED to the list of handled network error codes for more complete connection error coverage
src/lib/supabase/middleware.ts	Simplified auth error detection by removing redundant 'invalid' and 'expired' checks from nested token condition

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.