improved ts config, fixed types, insttead of PR 404

.vscode/settings.json

@@ -1,3 +1,4 @@
 {
-  "typescript.tsdk": "node_modules/typescript/lib"
+  "typescript.tsdk": "node_modules/typescript/lib",
+  "cSpell.words": ["fets"]
 }

e2e/shared-scripts/src/types.ts

@@ -3,8 +3,8 @@ import { OutputValue, Stack } from '@pulumi/pulumi/automation';
 
 export type DeploymentConfiguration<TProgramOutput = any> = {
   name: string;
-  prerequisites?: (stack: Stack) => Promise<void>;
-  config?: (stack: Stack) => Promise<void>;
+  prerequisites?: ((stack: Stack) => Promise<void>) | undefined;
+  config?: ((stack: Stack) => Promise<void>) | undefined;
   program: () => Promise<{
     [K in keyof TProgramOutput]: Output<TProgramOutput[K]> | TProgramOutput[K];
   }>;

e2e/shared-server/src/index.ts

@@ -1,7 +1,7 @@
 import { createRouter, Response, useErrorHandling } from 'fets';
 import { z } from 'zod';
 
-export function createTestServerAdapter<TServerContext = {}>(base?: string) {
+export function createTestServerAdapter<TServerContext = {}>(base?: string | undefined) {
   return createRouter<TServerContext, {}>({
     base,
     plugins: [useErrorHandling()],

examples/nextjs-example/src/pages/api/[...slug].ts

@@ -38,6 +38,7 @@ export default createRouter({
   openAPI: {
     endpoint: '/api/openapi.json',
   },
+  plugins: [],
 })
   .route({
     method: 'GET',

examples/todolist/src/router.ts

@@ -25,8 +25,9 @@ export const router = createRouter({
       schemas: {
         Todo: TodoSchema,
       },
-    } as const,
+    },
   },
+  plugins: [],
 })
   .route({
     description: 'Get all todos',

examples/zod-example/package.json

@@ -11,7 +11,7 @@
     "fets": "0.4.9",
     "ts-node": "10.9.1",
     "ts-node-dev": "2.0.0",
-    "typescript": "^5.0.0",
+    "typescript": "5.1.6",
     "zod": "3.21.4"
   }
 }

examples/zod-example/src/index.ts

@@ -96,6 +96,12 @@ export const router = createRouter()
     },
   });
 
+// TODO: Type 'IncomingMessage' is not assignable to type 'NodeRequest' with 'exactOptionalPropertyTypes: true'. Consider adding 'undefined' to the types of the target's properties.
+// Types of property 'url' are incompatible.
+// Type 'string | undefined' is not assignable to type 'string'.
+// Type 'undefined' is not assignable to type 'string'.
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
+// @ts-ignore
 createServer(router).listen(3000, () => {
   console.log('SwaggerUI is served at http://localhost:3000/docs');
 });

examples/zod-example/tsconfig.json

@@ -1,4 +1,5 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
     "target": "esnext",
     "moduleResolution": "node",

package.json

@@ -57,7 +57,7 @@
     "lint-staged": "13.2.3",
     "patch-package": "8.0.0",
     "prettier": "3.0.0",
-    "prettier-plugin-tailwindcss": "^0.4.0",
+    "prettier-plugin-tailwindcss": "0.4.1",
     "ts-jest": "29.1.1",
     "typescript": "5.1.6"
   },

packages/fets/src/createRouter.ts

@@ -37,13 +37,13 @@ const HTTP_METHODS: HTTPMethod[] = [
 const EMPTY_OBJECT = {};
 const EMPTY_MATCH = { pathname: { groups: {} } } as URLPatternResult;
 
-export function createRouterBase(
+export function createRouterBase<TServerContext>(
   {
     fetchAPI: givenFetchAPI,
     base: basePath = '/',
     plugins = [],
     swaggerUI,
-  }: RouterOptions<any, any> = {},
+  }: RouterOptions<TServerContext, any>,
   openAPIDocument: OpenAPIDocument,
 ): RouterBaseObject<any, any, any> {
   const fetchAPI = {
@@ -242,19 +242,20 @@ export function createRouterBase(
 
 export function createRouter<
   TServerContext,
-  TComponents extends RouterComponentsBase = {},
+  TComponents extends RouterComponentsBase,
   TRouterSDK extends RouterSDK<string, TypedRequest, TypedResponse> = {
     [TKey: string]: never;
   },
 >(
-  options: RouterOptions<TServerContext, TComponents> = {},
+  options?: RouterOptions<TServerContext, TComponents> | undefined,
 ): Router<TServerContext, TComponents, TRouterSDK> {
   const {
     openAPI: { endpoint: oasEndpoint = '/openapi.json', ...openAPIDocument } = {},
     swaggerUI: { endpoint: swaggerUIEndpoint = '/docs', ...swaggerUIOpts } = {},
     plugins: userPlugins = [],
     base = '/',
-  } = options;
+  } = options || {};
+
   openAPIDocument.openapi = openAPIDocument.openapi || '3.0.1';
   const oasInfo = (openAPIDocument.info ||= {} as OpenAPIInfo);
   oasInfo.title ||= 'feTS API';

packages/fets/src/plugins/ajv.ts

@@ -4,8 +4,8 @@ import addFormats from 'ajv-formats';
 import jsonSerializerFactory from '@ardatan/fast-json-stringify';
 import { URL } from '@whatwg-node/fetch';
 import { Response } from '../Response.js';
-import { StatusCode } from '../typed-fetch.js';
-import {
+import type { StatusCode } from '../typed-fetch.js';
+import type {
   JSONSerializer,
   PromiseOrValue,
   RouterComponentsBase,
@@ -18,9 +18,9 @@ import { getHeadersObj } from './utils.js';
 type ValidateRequestFn = (request: RouterRequest) => PromiseOrValue<ErrorObject[]>;
 
 export function useAjv({
-  components = {},
+  components = { schemas: {} },
 }: {
-  components?: RouterComponentsBase;
+  components?: RouterComponentsBase | undefined;
 } = {}): RouterPlugin<any> {
   const ajv = new Ajv({
     strict: false,
@@ -73,10 +73,16 @@ export function useAjv({
     onRoute({ path, schemas, handlers }) {
       const validationMiddlewares = new Map<string, ValidateRequestFn>();
       if (schemas?.request?.headers && !isZodSchema(schemas.request.headers)) {
+        const { headers } = schemas.request;
+
+        // TODO: Property '$async' is missing in type '{ components: RouterComponentsBase; type?: JSONSchema7TypeName | JSONSchema7TypeName[] | undefined; pattern?: string | undefined; ... 46 more ...; [$JSONSchema7]?: unique symbol; }' but required in type 'AsyncSchema'
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
         const validateFn = ajv.compile({
-          ...schemas.request.headers,
+          ...headers,
           components,
         });
+
         validationMiddlewares.set('headers', request => {
           const headersObj = getHeadersObj(request.headers);
           const isValid = validateFn(headersObj);
@@ -87,10 +93,14 @@ export function useAjv({
         });
       }
       if (schemas?.request?.params && !isZodSchema(schemas.request.params)) {
+        // TODO: Property '$async' is missing in type '{ components: RouterComponentsBase; type?: JSONSchema7TypeName | JSONSchema7TypeName[] | undefined; pattern?: string | undefined; ... 46 more ...; [$JSONSchema7]?: unique symbol; }' but required in type 'AsyncSchema'
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
         const validateFn = ajv.compile({
           ...schemas.request.params,
           components,
         });
+
         validationMiddlewares.set('params', request => {
           const isValid = validateFn(request.params);
           if (!isValid) {
@@ -100,10 +110,14 @@ export function useAjv({
         });
       }
       if (schemas?.request?.query && !isZodSchema(schemas.request.query)) {
+        // TODO: Property '$async' is missing in type '{ components: RouterComponentsBase; type?: JSONSchema7TypeName | JSONSchema7TypeName[] | undefined; pattern?: string | undefined; ... 46 more ...; [$JSONSchema7]?: unique symbol; }' but required in type 'AsyncSchema'
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
         const validateFn = ajv.compile({
           ...schemas.request.query,
           components,
         });
+
         validationMiddlewares.set('query', request => {
           const isValid = validateFn(request.query);
           if (!isValid) {
@@ -113,10 +127,14 @@ export function useAjv({
         });
       }
       if (schemas?.request?.json && !isZodSchema(schemas.request.json)) {
+        // TODO: Property '$async' is missing in type '{ components: RouterComponentsBase; type?: JSONSchema7TypeName | JSONSchema7TypeName[] | undefined; pattern?: string | undefined; ... 46 more ...; [$JSONSchema7]?: unique symbol; }' but required in type 'AsyncSchema'
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
         const validateFn = ajv.compile({
           ...schemas.request.json,
           components,
         });
+
         validationMiddlewares.set('json', async request => {
           const contentType = request.headers.get('content-type');
           if (contentType?.includes('json')) {
@@ -134,10 +152,14 @@ export function useAjv({
         });
       }
       if (schemas?.request?.formData && !isZodSchema(schemas.request.formData)) {
+        // TODO: Property '$async' is missing in type '{ components: RouterComponentsBase; type?: JSONSchema7TypeName | JSONSchema7TypeName[] | undefined; pattern?: string | undefined; ... 46 more ...; [$JSONSchema7]?: unique symbol; }' but required in type 'AsyncSchema'
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
         const validateFn = ajv.compile({
           ...schemas.request.formData,
           components,
         });
+
         validationMiddlewares.set('formData', async request => {
           const contentType = request.headers.get('content-type');
           if (