fix: fix fastify swagger error when properties attribute is missing

packages/cli/src/schema/generateSchema.ts

@@ -2,6 +2,7 @@ import $RefParser from '@apidevtools/json-schema-ref-parser';
 import * as ts from 'ts-json-schema-generator';
 import { Config } from 'ts-json-schema-generator';
 import { log } from '../log.js';
+import { addProperties } from './helper/add-properties.js';
 import { convertEmptyObject } from './helper/convertEmptyObject.js';
 import { convertNullToNullable } from './helper/convertNullToNullable.js';
 
@@ -30,10 +31,10 @@ export class GenerateGlobalSchema {
       let schema = ts.createGenerator(config).createSchema() as any;
 
       schema = addId(schema.definitions);
-
       schema = convertNullToNullable(schema);
-
       schema = convertEmptyObject(schema);
+      schema = addProperties(schema);
+
       return JSON.stringify(schema, null, 2);
     } catch (e: any) {
       if (e.message.includes('NoRootNamesError')) {

packages/cli/src/schema/helper/add-properties.ts

@@ -0,0 +1,11 @@
+// schemas without properties cause fastify-swagger to fail
+export function addProperties(definitions: Record<string, any>) {
+  Object.keys(definitions).forEach((key: string) => {
+    const definition = definitions[key];
+    if (definition.type === 'object' && !definition.properties) {
+      definition.properties = {};
+    }
+  });
+
+  return definitions;
+}

packages/integration-tests/src/index.ts

@@ -1,11 +1,12 @@
 import { flux } from '@fluxapi/common';
 import { openapi } from '@fluxapi/plugins';
 import { CrudController } from './crud/crud.controller';
+import { InputController } from './input/input.controller';
 import { ResponseController } from './response/response.controller';
 
 const fastify = flux({
   plugins: [openapi()],
-  controllers: [ResponseController, CrudController],
+  controllers: [ResponseController, CrudController, InputController],
 });
 
 fastify.listen(8080, '127.0.0.1', (err, address) => {

packages/integration-tests/src/input/input.controller.ts

@@ -0,0 +1,10 @@
+import { Controller, Get } from '@fluxapi/common';
+import { InputEmptyQuery } from './input.schema';
+
+@Controller('/inputs', { tags: ['responses'] })
+export class InputController {
+  @Get('/empty-query')
+  async emptyQuery(query: InputEmptyQuery): Promise<void> {
+    return;
+  }
+}

packages/integration-tests/src/input/input.schema.ts

@@ -0,0 +1 @@
+export interface InputEmptyQuery {}

packages/integration-tests/src/response/response.controller.ts

@@ -1,5 +1,12 @@
 import { Controller, Delete, Get, Status } from '@fluxapi/common';
-import { AnyResponse, NullResponse, ObjectResponse, AddionalResponse, UndefinedResponse } from './response.schema';
+import {
+  AnyResponse,
+  NullResponse,
+  ObjectResponse,
+  AddionalResponse,
+  UndefinedResponse,
+  Foobar,
+} from './response.schema';
 
 @Controller('/responses', { tags: ['responses'] })
 export class ResponseController {
@@ -65,4 +72,9 @@ export class ResponseController {
       objectNullUndefined: null,
     };
   }
+
+  @Get('/empty-interface-response')
+  async emptyInterfaceResponse(): Promise<Foobar> {
+    return { foo: 'bar' };
+  }
 }

packages/integration-tests/src/response/response.schema.ts

@@ -36,3 +36,5 @@ export interface UndefinedResponse {
   objectNullUndefined?: { id: number; name: string } | null;
   objectNullUndefined2?: { id: number; name: string } | null;
 }
+
+export interface Foobar {}