Merge pull request #1257 from c-bata/remove-axios-client

Replace axios with fetch-based API client

Author: c-bata

jupyterlab/package.json

@@ -76,7 +76,6 @@
         "@types/react-dom": "^18.3.1",
         "@types/react-syntax-highlighter": "^15.5.5",
         "@types/signals": "^1.0.2",
-        "axios": "^1.11.0",
         "css-loader": "^6.8.1",
         "elkjs": "^0.9.1",
         "jotai": "^2.12.1",

jupyterlab/yarn.lock

@@ -2751,17 +2751,6 @@ __metadata:
   languageName: node
   linkType: hard
 
-"axios@npm:^1.11.0":
-  version: 1.11.0
-  resolution: "axios@npm:1.11.0"
-  dependencies:
-    follow-redirects: ^1.15.6
-    form-data: ^4.0.4
-    proxy-from-env: ^1.1.0
-  checksum: 0a33dc600b588bfd3111b198d5985527ed89f722817455d7cdb66c1d055e5f8859cc2bebb7320888957fc8458ebe77d5f83af02af9cd260217c91c4e92b6dfb6
-  languageName: node
-  linkType: hard
-
 "babel-plugin-macros@npm:^3.1.0":
   version: 3.1.0
   resolution: "babel-plugin-macros@npm:3.1.0"
@@ -5362,7 +5351,6 @@ __metadata:
     "@types/react-dom": ^18.3.1
     "@types/react-syntax-highlighter": ^15.5.5
     "@types/signals": ^1.0.2
-    axios: ^1.11.0
     css-loader: ^6.8.1
     elkjs: ^0.9.1
     jotai: ^2.12.1

optuna_dashboard/package-lock.json

@@ -15,19 +15,17 @@
   },
   "author": "Masashi Shibata",
   "license": "MIT",
-  "dependencies": {
-  },
   "devDependencies": {
-    "@optuna/types": "../tslib/types/",
-    "@optuna/react": "../tslib/react",
     "@babel/core": "^7.23.9",
     "@babel/preset-env": "^7.23.9",
+    "@optuna/react": "../tslib/react",
+    "@optuna/types": "../tslib/types/",
     "@types/node": ">=22.8.0",
-    "@types/react": "^18.2.55",
+    "@types/papaparse": "^5.3.14",
     "@types/plotly.js": "^2.12.32",
+    "@types/react": "^18.2.55",
     "@types/react-dom": "^18.2.18",
     "@types/react-syntax-highlighter": "^15.5.11",
-    "@types/papaparse": "^5.3.14",
     "@types/three": "^0.160.0",
     "compression-webpack-plugin": "^11.0.0",
     "css-loader": "^6.9.1",
@@ -39,25 +37,24 @@
     "webpack-cli": "^5.1.4"
   },
   "peerDependencies": {
-    "react": ">=16.0.0",
-    "react-dom": ">=16.0.0",
     "@emotion/react": ">=11.0.0",
     "@emotion/styled": ">=11.0.0",
     "@mui/icons-material": ">=6.0.0",
     "@mui/lab": ">=6.0.0-beta.0",
     "@mui/material": ">=6.0.0",
     "@mui/system": ">=6.0.0",
-    "plotly.js-dist-min": ">=2.0.0",
     "@react-three/drei": ">=9.0.0",
     "@react-three/fiber": ">=8.0.0",
-    "@tanstack/react-table": ">=8.0.0",
     "@tanstack/react-query": ">=5.0.0",
+    "@tanstack/react-table": ">=8.0.0",
     "@tanstack/react-virtual": ">=3.0.0",
-    "axios": ">=1.0.0",
     "elkjs": ">=0.8.0",
     "jotai": "^2.12.1",
     "notistack": ">=3.0.0",
     "papaparse": ">=5.0.0",
+    "plotly.js-dist-min": ">=2.0.0",
+    "react": ">=16.0.0",
+    "react-dom": ">=16.0.0",
     "react-markdown": ">=9.0.0",
     "react-router-dom": ">=6.0.0",
     "react-syntax-highlighter": ">=15.0.0",

optuna_dashboard/package.json

@@ -177,6 +177,28 @@ export type ReGeneratePlotlyGraphQueryResponse = {
   re_generated_plotly_graph_func_str: string
 }
 
+export class FetchAPIClientError<T = unknown> extends Error {
+  response?: {
+    status: number
+    data: T
+  }
+
+  constructor(message: string, status?: number, data?: T) {
+    super(message)
+    this.name = "FetchAPIClientError"
+    if (status !== undefined && data !== undefined) {
+      this.response = {
+        status,
+        data,
+      }
+    }
+  }
+}
+
+export const isFetchAPIClientError = <T = unknown>(
+  error: unknown
+): error is FetchAPIClientError<T> => error instanceof FetchAPIClientError
+
 export abstract class APIClient {
   constructor() {}