refactor: add typescript support

Author: boschni

.babelrc

@@ -8,6 +8,7 @@
         "exclude": ["@babel/plugin-transform-regenerator"]
       }
     ],
+    "@babel/preset-typescript",
     "@babel/react"
   ],
   "plugins": ["babel-plugin-transform-async-to-promises"],

.eslintrc

@@ -1,10 +1,25 @@
 {
-  "parser": "babel-eslint",
-  "extends": ["react-app", "prettier"],
+  "parser": "@typescript-eslint/parser",
+  "plugins": ["@typescript-eslint"],
+  "extends": [
+    "plugin:@typescript-eslint/eslint-recommended",
+    "plugin:@typescript-eslint/recommended",
+    "react-app",
+    "prettier"
+  ],
   "env": {
     "es6": true
   },
   "parserOptions": {
     "sourceType": "module"
+  },
+  "rules": {
+    "prefer-const": "off",
+    "@typescript-eslint/ban-ts-comment": "off",
+    "@typescript-eslint/explicit-module-boundary-types": "off",
+    "@typescript-eslint/no-empty-interface": "off",
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/no-non-null-assertion": "off",
+    "@typescript-eslint/ban-types": "off"
   }
 }

package.json

@@ -19,6 +19,9 @@
     "test:dev": "jest --watch",
     "test:ci": "jest && yarn dtslint",
     "test:coverage": "yarn test:ci; open coverage/lcov-report/index.html",
+    "test:types": "tsc",
+    "test:eslint": "eslint --ext .ts,.tsx ./src",
+    "gen:types": "tsc --project ./tsconfig.types.json",
     "build": "NODE_ENV=production rollup -c",
     "now-build": "yarn && cd www && yarn && yarn build",
     "start": "rollup -c -w",
@@ -56,10 +59,14 @@
     "@babel/core": "^7.10.2",
     "@babel/preset-env": "^7.10.2",
     "@babel/preset-react": "^7.10.1",
+    "@babel/preset-typescript": "^7.10.4",
     "@rollup/plugin-replace": "^2.3.3",
     "@svgr/rollup": "^5.4.0",
-    "@testing-library/react": "^10.2.1",
+    "@testing-library/react": "^10.4.7",
+    "@types/jest": "^26.0.4",
     "@types/react": "^16.9.41",
+    "@typescript-eslint/eslint-plugin": "^3.6.1",
+    "@typescript-eslint/parser": "^3.6.1",
     "babel-eslint": "^10.1.0",
     "babel-jest": "^26.0.1",
     "babel-plugin-transform-async-to-promises": "^0.8.15",

rollup.config.js

@@ -13,7 +13,11 @@ const globals = {
   react: 'React',
 }
 
-const inputSrc = 'src/react/index.js'
+const inputSrc = 'src/index.ts'
+
+const extensions = ['.js', '.jsx', '.es6', '.es', '.mjs', '.ts', '.tsx']
+const babelConfig = { extensions }
+const resolveConfig = { extensions }
 
 export default [
   {
@@ -24,7 +28,12 @@ export default [
       sourcemap: true,
     },
     external,
-    plugins: [resolve(), babel(), commonJS(), externalDeps()],
+    plugins: [
+      resolve(resolveConfig),
+      babel(babelConfig),
+      commonJS(),
+      externalDeps(),
+    ],
   },
   {
     input: inputSrc,
@@ -34,7 +43,13 @@ export default [
       sourcemap: true,
     },
     external,
-    plugins: [resolve(), babel(), commonJS(), externalDeps(), terser()],
+    plugins: [
+      resolve(resolveConfig),
+      babel(babelConfig),
+      commonJS(),
+      externalDeps(),
+      terser(),
+    ],
   },
   {
     input: inputSrc,
@@ -46,7 +61,12 @@ export default [
       globals,
     },
     external,
-    plugins: [resolve(), babel(), commonJS(), externalDeps()],
+    plugins: [
+      resolve(resolveConfig),
+      babel(babelConfig),
+      commonJS(),
+      externalDeps(),
+    ],
   },
   {
     input: inputSrc,
@@ -60,8 +80,8 @@ export default [
     external,
     plugins: [
       replace({ 'process.env.NODE_ENV': `"production"`, delimiters: ['', ''] }),
-      resolve(),
-      babel(),
+      resolve(resolveConfig),
+      babel(babelConfig),
       commonJS(),
       externalDeps(),
       terser(),

src/core/config.js renamed to src/core/config.ts

@@ -1,6 +1,11 @@
 import { noop, stableStringify, identity, deepEqual } from './utils'
+import { ReactQueryConfig, ArrayQueryKey, QueryKey } from './types'
 
-export const DEFAULT_CONFIG = {
+export interface ReactQueryConfigRef {
+  current: ReactQueryConfig
+}
+
+export const DEFAULT_CONFIG: ReactQueryConfig = {
   shared: {
     suspense: false,
   },
@@ -10,7 +15,7 @@ export const DEFAULT_CONFIG = {
     initialStale: undefined,
     enabled: true,
     retry: 3,
-    retryDelay: attemptIndex => Math.min(1000 * 2 ** attemptIndex, 30000),
+    retryDelay: attemptIndex => Math.min(1000 * 2 ** attemptIndex!, 30000),
     staleTime: 0,
     cacheTime: 5 * 60 * 1000,
     refetchOnWindowFocus: true,
@@ -33,29 +38,27 @@ export const DEFAULT_CONFIG = {
   },
 }
 
-export const defaultConfigRef = {
+export const defaultConfigRef: ReactQueryConfigRef = {
   current: DEFAULT_CONFIG,
 }
 
-export function defaultQueryKeySerializerFn(queryKey) {
-  if (!queryKey) {
-    return []
-  }
+export function defaultQueryKeySerializerFn(
+  queryKey: QueryKey
+): [string, ArrayQueryKey] {
+  let arrayQueryKey: ArrayQueryKey = Array.isArray(queryKey)
+    ? queryKey
+    : [queryKey]
 
-  if (!Array.isArray(queryKey)) {
-    queryKey = [queryKey]
-  }
-
-  if (queryKey.some(d => typeof d === 'function')) {
+  if (
+    arrayQueryKey[0] === '' ||
+    typeof arrayQueryKey[0] !== 'string' ||
+    arrayQueryKey.some(d => typeof d === 'function')
+  ) {
     throw new Error('A valid query key is required!')
   }
 
-  const queryHash = stableStringify(queryKey)
-  queryKey = JSON.parse(queryHash)
-
-  if (!queryHash) {
-    return []
-  }
+  const queryHash = stableStringify(arrayQueryKey)
+  arrayQueryKey = JSON.parse(queryHash)
 
-  return [queryHash, queryKey]
+  return [queryHash, arrayQueryKey]
 }