fix(web): remove unsafe any types and improve error handling (#70)

Signed-off-by: wibus-wee <62133302+wibus-wee@users.noreply.github.com>

Author: wibus-wee

web/lib/axios-interceptors.ts

@@ -1,11 +1,15 @@
 "use client"
 
-import axios, { AxiosError } from 'axios'
+import axios, { AxiosError, InternalAxiosRequestConfig } from 'axios'
 import { DefaultService as AnclaxService } from '@/api-anclax'
 import toast from 'react-hot-toast'
 import { authActions, getAuthState } from '@/atoms/auth'
 import { setStoredAuthUser } from '@/lib/storage'
 
+interface RetryableRequestConfig extends InternalAxiosRequestConfig {
+  _retry?: boolean
+}
+
 let initialized = false
 let interceptorId: number | null = null
 let refreshPromise: Promise<string> | null = null
@@ -57,7 +61,7 @@ export function initAxiosAuthInterceptors() {
   interceptorId = axios.interceptors.response.use(
     (response) => response,
     async (error: AxiosError) => {
-      const status = error.response?.status ?? (error as any).status
+      const status = error.response?.status ?? error.status
       const requestUrl =
         error.config?.url || error.request?.responseURL || ''
 
@@ -71,15 +75,19 @@ export function initAxiosAuthInterceptors() {
 
       // Handle 401 with single-flight refresh & replay
       if (status === 401) {
-        const originalRequest: any = error.config || {}
+        if (!error.config) {
+          return Promise.reject(error)
+        }
+
+        const originalRequest = error.config as RetryableRequestConfig
         if (originalRequest._retry) {
           return Promise.reject(error)
         }
         originalRequest._retry = true
 
         try {
           const token = await refreshAccessToken()
-          if (originalRequest && token) {
+          if (token) {
             const headers = originalRequest.headers || {}
             if (typeof headers.set === 'function') {
               headers.set('Authorization', `Bearer ${token}`)

web/lib/network-config.ts

@@ -8,7 +8,7 @@ const DEFAULT_INTERNAL_BASE = 'http://localhost:30080/api/v1'
 const DEFAULT_EXTERNAL_BASE = 'http://localhost:8020/api/v1'
 
 
-const resolveAccessToken = async (_options?: any): Promise<string> => {
+const resolveAccessToken = async (): Promise<string> => {
   const current = getAuthState()
   return current?.user?.accessToken ?? ''
 }
@@ -18,7 +18,7 @@ let lastBase = ''
 
 export const resolveBaseUrl = () => {
   if (typeof window === 'undefined') return DEFAULT_INTERNAL_BASE
-  return (window as any).APP_ENDPOINT || DEFAULT_EXTERNAL_BASE
+  return window.APP_ENDPOINT || DEFAULT_EXTERNAL_BASE
 }
 
 export function initNetworkConfig() {

web/lib/storage.ts

@@ -1,4 +1,3 @@
-import { Credentials } from '@/api-anclax'
 import type { AuthUser } from '@/atoms/auth'
 
 const isBrowser = typeof window !== 'undefined'