refactor: split out a requests library

.changeset/slow-ties-admire.md

@@ -0,0 +1,5 @@
+---
+'@siafoundation/request': minor
+---
+
+Introduced a new request library for shared request and network code.

libs/react-core/package.json

@@ -8,7 +8,8 @@
     "swr": "^2.1.1",
     "axios": "^0.27.2",
     "use-local-storage-state": "^18.3.3",
-    "@siafoundation/next": "^0.1.3"
+    "@siafoundation/next": "^0.1.3",
+    "@siafoundation/request": "0.0.0"
   },
   "dependencies": {
     "detect-gpu": "^5.0.34"

libs/react-core/src/request.ts

@@ -2,11 +2,7 @@ import { AxiosRequestConfig, AxiosResponse, AxiosResponseHeaders } from 'axios'
 import { MutatorCallback, MutatorOptions } from 'swr'
 import { SWROptions } from './types'
 import { AppSettings } from './useAppSettings'
-
-export type RequestParams = Record<
-  string,
-  string | string[] | number | boolean
-> | void
+import { RequestParams, parameterizeRoute } from '@siafoundation/request'
 
 export type RequestConfig<Payload, Result> = {
   swr?: SWROptions<Result>
@@ -238,28 +234,6 @@ export function buildAxiosConfig<Params extends RequestParams, Payload, Result>(
   } as AxiosRequestConfig<Payload>
 }
 
-function parameterizeRoute(
-  route: string | null,
-  params: RequestParams
-): string | null {
-  if (route && params) {
-    const paramKeys = Object.keys(params)
-    for (const key of paramKeys) {
-      const value = String(params[key])
-      if (route.includes(`:${key}`)) {
-        route = route.replace(`:${key}`, value)
-      } else {
-        if (!route.includes('?')) {
-          route += `?${key}=${encodeURIComponent(value)}`
-        } else {
-          route += `&${key}=${encodeURIComponent(value)}`
-        }
-      }
-    }
-  }
-  return route
-}
-
 export function buildRouteWithParams<
   Params extends RequestParams,
   Payload,

libs/request/.babelrc

@@ -0,0 +1,12 @@
+{
+  "presets": [
+    [
+      "@nx/react/babel",
+      {
+        "runtime": "automatic",
+        "useBuiltIns": "usage"
+      }
+    ]
+  ],
+  "plugins": []
+}

libs/request/.eslintrc.json

@@ -0,0 +1,21 @@
+{
+  "extends": ["plugin:@nx/react", "../../.eslintrc.json"],
+  "ignorePatterns": ["!**/*"],
+  "rules": {
+    "@nx/dependency-checks": [
+      "error",
+      {
+        "ignoredFiles": ["libs/request/rollup.config.js"]
+      }
+    ]
+  },
+  "overrides": [
+    {
+      "files": ["*.json"],
+      "parser": "jsonc-eslint-parser",
+      "rules": {
+        "@nx/dependency-checks": "error"
+      }
+    }
+  ]
+}

libs/request/README.md

@@ -0,0 +1,3 @@
+# request
+
+Core library for building request APIs.

libs/request/jest.config.ts

@@ -0,0 +1,17 @@
+/* eslint-disable */
+export default {
+  displayName: 'request',
+  preset: '../../jest.preset.js',
+  transform: {
+    '^(?!.*\\.(js|jsx|ts|tsx|css|json)$)': '@nx/react/plugins/jest',
+    '^.+\\.[tj]sx?$': [
+      'babel-jest',
+      {
+        presets: ['@nx/next/babel'],
+        plugins: ['@babel/plugin-transform-private-methods'],
+      },
+    ],
+  },
+  moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx'],
+  coverageDirectory: '../../coverage/libs/request',
+}

libs/request/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "@siafoundation/request",
+  "description": "Core library for building request APIs.",
+  "version": "0.0.0",
+  "license": "MIT",
+  "dependencies": {
+    "axios": "^0.27.2"
+  },
+  "types": "./src/index.d.ts"
+}

libs/request/project.json

@@ -0,0 +1,42 @@
+{
+  "name": "request",
+  "$schema": "../../node_modules/nx/schemas/project-schema.json",
+  "sourceRoot": "libs/request/src",
+  "projectType": "library",
+  "tags": [],
+  "targets": {
+    "build": {
+      "executor": "@nx/rollup:rollup",
+      "outputs": ["{options.outputPath}"],
+      "options": {
+        "outputPath": "dist/libs/request",
+        "tsConfig": "libs/request/tsconfig.lib.json",
+        "project": "libs/request/package.json",
+        "entryFile": "libs/request/src/index.ts",
+        "external": ["react/jsx-runtime"],
+        "compiler": "tsc",
+        "outputFileName": "index.js",
+        "rollupConfig": "libs/request/rollup.config.js",
+        "assets": [
+          {
+            "glob": "libs/request/*.md",
+            "input": ".",
+            "output": "."
+          }
+        ]
+      },
+      "configurations": {}
+    },
+    "lint": {
+      "executor": "@nx/eslint:lint",
+      "outputs": ["{options.outputFile}"]
+    },
+    "test": {
+      "executor": "@nx/jest:jest",
+      "outputs": ["{workspaceRoot}/coverage/libs/request"],
+      "options": {
+        "jestConfig": "libs/request/jest.config.ts"
+      }
+    }
+  }
+}

libs/request/rollup.config.js

@@ -0,0 +1,18 @@
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const preserveDirectives = require('rollup-plugin-preserve-directives')
+
+// https://github.com/rollup/rollup/issues/4699#issuecomment-1465302665
+function getRollupOptions(options) {
+  return {
+    ...options,
+    output: {
+      ...options.output,
+      preserveModules: true,
+      format: 'esm',
+      sourcemap: true,
+    },
+    plugins: options.plugins.concat(preserveDirectives.default()),
+  }
+}
+
+module.exports = getRollupOptions

libs/request/src/index.ts

@@ -0,0 +1,67 @@
+import { Axios } from 'axios'
+
+export type RequestParams = Record<
+  string,
+  string | string[] | number | boolean
+> | void
+
+export function parameterizeRoute(
+  route: string | null,
+  params: RequestParams
+): string | null {
+  if (route && params) {
+    const paramKeys = Object.keys(params)
+    for (const key of paramKeys) {
+      const value = String(params[key])
+      if (route.includes(`:${key}`)) {
+        route = route.replace(`:${key}`, value)
+      } else {
+        if (!route.includes('?')) {
+          route += `?${key}=${encodeURIComponent(value)}`
+        } else {
+          route += `&${key}=${encodeURIComponent(value)}`
+        }
+      }
+    }
+  }
+  return route
+}
+
+export function buildRequestHandler<
+  Params = void,
+  Payload = void,
+  Response = void
+>(
+  axios: Axios,
+  method: 'get' | 'post' | 'patch' | 'put' | 'delete',
+  route: string
+) {
+  type ParamsArg = Params extends void
+    ? Record<string, never>
+    : { params: Params }
+  type PayloadArg = Payload extends void
+    ? Record<string, never>
+    : { payload: Payload }
+
+  return (
+    ...args: Params extends void
+      ? Payload extends void
+        ? []
+        : [PayloadArg]
+      : [
+          ParamsArg &
+            (Payload extends void
+              ? Record<string, never>
+              : { payload?: Payload })
+        ]
+  ) => {
+    const arg = args[0] || {}
+    const paramRoute =
+      'params' in arg
+        ? // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+          parameterizeRoute(route, arg.params as RequestParams)!
+        : route
+    const payload = 'payload' in arg ? arg.payload : undefined
+    return axios[method]<Response>(paramRoute, payload)
+  }
+}

libs/request/tsconfig.json

@@ -0,0 +1,25 @@
+{
+  "extends": "../../tsconfig.base.json",
+  "compilerOptions": {
+    "jsx": "react-jsx",
+    "allowJs": true,
+    "esModuleInterop": true,
+    "allowSyntheticDefaultImports": true,
+    "forceConsistentCasingInFileNames": true,
+    "strict": true,
+    "noImplicitOverride": true,
+    "noPropertyAccessFromIndexSignature": true,
+    "noImplicitReturns": true,
+    "noFallthroughCasesInSwitch": true
+  },
+  "files": [],
+  "include": [],
+  "references": [
+    {
+      "path": "./tsconfig.lib.json"
+    },
+    {
+      "path": "./tsconfig.spec.json"
+    }
+  ]
+}

libs/request/tsconfig.lib.json

@@ -0,0 +1,22 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "outDir": "../../dist/out-tsc",
+    "types": [
+      "node",
+      "@nx/react/typings/cssmodule.d.ts",
+      "@nx/react/typings/image.d.ts"
+    ]
+  },
+  "exclude": [
+    "**/*.spec.ts",
+    "**/*.test.ts",
+    "**/*.spec.tsx",
+    "**/*.test.tsx",
+    "**/*.spec.js",
+    "**/*.test.js",
+    "**/*.spec.jsx",
+    "**/*.test.jsx"
+  ],
+  "include": ["src/**/*.js", "src/**/*.jsx", "src/**/*.ts", "src/**/*.tsx"]
+}

libs/request/tsconfig.spec.json

@@ -0,0 +1,19 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "outDir": "../../dist/out-tsc",
+    "module": "commonjs",
+    "types": ["jest", "node"]
+  },
+  "include": [
+    "src/**/*.test.ts",
+    "src/**/*.spec.ts",
+    "src/**/*.test.tsx",
+    "src/**/*.spec.tsx",
+    "src/**/*.test.js",
+    "src/**/*.spec.js",
+    "src/**/*.test.jsx",
+    "src/**/*.spec.jsx",
+    "src/**/*.d.ts"
+  ]
+}

tsconfig.base.json

@@ -15,6 +15,7 @@
     "skipDefaultLibCheck": true,
     "baseUrl": ".",
     "paths": {
+      "@siafoundation/request": ["libs/request/src/index.ts"],
       "@siafoundation/data-sources": ["libs/data-sources/src/index.ts"],
       "@siafoundation/design-system": ["libs/design-system/src/index.ts"],
       "@siafoundation/fonts": ["libs/fonts/src/index.ts"],