chore: e2e strict-mode

kubb-labs · Sep 1, 2024 · 1686428 · 1686428
1 parent c15a3b6
commit 1686428
Show file tree

Hide file tree

Showing 18 changed files with 406 additions and 758 deletions.
diff --git a/e2e/kubb.config.js b/e2e/kubb.config.js
@@ -43,7 +43,7 @@ const baseConfig = {
     clean: true,
   },
   hooks: {
-    done: ['npm run typecheck', 'biome format --write ./', 'biome lint --apply-unsafe ./src'],
+    done: ['npm run typecheck', 'npm run typecheck:strict', 'biome format --write ./', 'biome lint --apply-unsafe ./src'],
   },
   plugins: [
     pluginOas({

diff --git a/e2e/package.json b/e2e/package.json
@@ -18,7 +18,8 @@
     "generate": "kubb generate",
     "generate:bun": "kubb generate --bun",
     "generate:debug": "node --inspect ../packages/cli/bin/kubb.js",
-    "typecheck": "tsc -p ./tsconfig.json --noEmit --emitDeclarationOnly false"
+    "typecheck": "tsc -p ./tsconfig.strict.json --noEmit --emitDeclarationOnly false",
+    "typecheck:strict": "tsc -p ./tsconfig.strict.json --noEmit --emitDeclarationOnly false"
   },
   "dependencies": {
     "@faker-js/faker": "^8.4.1",

diff --git a/e2e/tsconfig.json b/e2e/tsconfig.json
@@ -6,7 +6,7 @@
     "sourceMap": true,
     "strictNullChecks": true,
     "jsx": "react-jsx",
-    "outDir": "es",
+    "outDir": "dist",
     "experimentalDecorators": true,
     "skipLibCheck": true,
     "baseUrl": ".",

diff --git a/e2e/tsconfig.strict.json b/e2e/tsconfig.strict.json
@@ -0,0 +1,7 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "alwaysStrict": true,
+    "strict": true
+  }
+}
diff --git a/examples/advanced/tsconfig.json b/examples/advanced/tsconfig.json
@@ -16,7 +16,9 @@
     "allowJs": true,
     "allowImportingTsExtensions": true,
     "verbatimModuleSyntax": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*", "./templates/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/client/package.json b/examples/client/package.json
@@ -31,6 +31,7 @@
   "devDependencies": {
     "@kubb/config-ts": "workspace:*",
     "react": "^18.3.1",
+    "@types/react": "^18.3.5",
     "tsup": "^8.2.4",
     "typescript": "^5.5.4"
   },

diff --git a/examples/client/src/gen/clients/axios/petService/uploadFile.ts b/examples/client/src/gen/clients/axios/petService/uploadFile.ts
@@ -20,7 +20,7 @@ export async function uploadFile(
   const formData = new FormData()
   if (data) {
     Object.keys(data).forEach((key) => {
-      const value = data[key]
+      const value = data[key as keyof typeof data]
       if (typeof key === 'string' && (typeof value === 'string' || value instanceof Blob)) {
         formData.append(key, value)
       }

diff --git a/examples/client/tsconfig.json b/examples/client/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*", "./templates/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/msw/tsconfig.json b/examples/msw/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/simple-single/src/gen2/index.ts b/examples/simple-single/src/gen2/index.ts
diff --git a/examples/simple-single/tsconfig.json b/examples/simple-single/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/solid-query/tsconfig.json b/examples/solid-query/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/svelte-query/tsconfig.json b/examples/svelte-query/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/swr/tsconfig.json b/examples/swr/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/typescript/tsconfig.json b/examples/typescript/tsconfig.json
@@ -15,7 +15,9 @@
     "esModuleInterop": true,
     "allowJs": true,
     "allowImportingTsExtensions": true,
-    "noEmit": true
+    "noEmit": true,
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/examples/vue-query/tsconfig.json b/examples/vue-query/tsconfig.json
@@ -16,7 +16,9 @@
     "allowJs": true,
     "allowImportingTsExtensions": true,
     "noEmit": true,
-    "paths": {}
+    "paths": {},
+    "alwaysStrict": true,
+    "strict": true
   },
   "include": ["./src/**/*", "vite.config.ts", "shims-vue.d.ts"],
   "exclude": ["**/node_modules", "**/types/**", "**/mocks/**"]

diff --git a/packages/plugin-client/src/components/Client.tsx b/packages/plugin-client/src/components/Client.tsx
@@ -104,7 +104,7 @@ export function Client({ name, options, typedSchemas, operation }: Props): KubbN
    const formData = new FormData()
    if(data) {
     Object.keys(data).forEach((key) => {
-      const value = data[key];
+      const value = data[key as keyof typeof data];
       if (typeof key === "string" && (typeof value === "string" || value instanceof Blob)) {
         formData.append(key, value);
       }

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml