frouriojs · solufa · Nov 14, 2024 · Nov 14, 2024 · Nov 14, 2024 · Nov 14, 2024
diff --git a/Makefile b/Makefile
@@ -0,0 +1,5 @@
+release:
+	@echo ${VERSION} | grep -E "^[0-9]+\.[0-9]+\.[0-9]+(-r[0-9]+)?$$"
+	git fetch origin main
+	git tag v${VERSION} FETCH_HEAD
+	git push origin v${VERSION}
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -30,6 +30,8 @@ export default tseslint.config(
       '@typescript-eslint/ban-types': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-var-requires': 'off',
+      '@typescript-eslint/no-require-imports': 'off',
+      '@typescript-eslint/no-empty-object-type': 'off',
       '@typescript-eslint/no-unused-vars': [
         'warn',
         {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -38,23 +38,23 @@
   "devDependencies": {
     "@aspida/axios": "^1.14.0",
     "@aspida/node-fetch": "^1.14.0",
-    "@fastify/multipart": "^8.3.0",
+    "@fastify/multipart": "^9.0.1",
     "@types/busboy": "^1.5.4",
     "@types/node-fetch": "^2.6.11",
-    "axios": "^1.7.2",
-    "eslint": "^8.57.0",
-    "eslint-config-flat-gitignore": "^0.1.8",
+    "axios": "^1.7.7",
+    "eslint": "^9.14.0",
+    "eslint-config-flat-gitignore": "^0.3.0",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-promise": "^6.5.1",
-    "fastify": "^4.28.1",
+    "eslint-plugin-promise": "^7.1.0",
+    "fastify": "^5.1.0",
     "node-fetch": "^2.6.12",
     "prettier": "^3.3.3",
-    "prettier-plugin-organize-imports": "^4.0.0",
+    "prettier-plugin-organize-imports": "^4.1.0",
     "rimraf": "^6.0.1",
-    "typescript": "^5.5.3",
-    "typescript-eslint": "^7.16.1",
+    "typescript": "^5.6.3",
+    "typescript-eslint": "^8.14.0",
     "vite-tsconfig-paths": "^4.3.2",
-    "vitest": "^2.0.3",
+    "vitest": "^2.1.5",
     "zod": "^3.23.8"
   }
 }
diff --git a/servers/all/$server.ts b/servers/all/$server.ts
@@ -104,6 +104,26 @@ export type ServerMethodHandler<T extends AspidaMethodParams,  U extends Record<
   handler: ServerHandler<T, U> | ServerHandlerPromise<T, U>;
 };
 
+const parseStringArrayTypeQueryParams = (stringArrayTypeParams: [string, boolean][]): preValidationHookHandler => (req, reply, done) => {
+  const query: any = req.query;
+
+  for (const [key, isOptional] of stringArrayTypeParams) {
+    const param = query[`${key}[]`] ?? query[key];
+
+    if (!isOptional && param === undefined) {
+      query[key] = [];
+    } else if (!isOptional || param !== undefined) {
+      const vals = (Array.isArray(param) ? param : [param]);
+
+      query[key] = vals;
+    }
+
+    delete query[`${key}[]`];
+  }
+
+  done();
+};
+
 const parseNumberTypeQueryParams = (numberTypeParams: [string, boolean, boolean][]): preValidationHookHandler => (req, reply, done) => {
   const query: any = req.query;
 
@@ -121,7 +141,7 @@ const parseNumberTypeQueryParams = (numberTypeParams: [string, boolean, boolean]
           return;
         }
 
-        query[key] = vals as any;
+        query[key] = vals;
       }
 
       delete query[`${key}[]`];
@@ -133,7 +153,7 @@ const parseNumberTypeQueryParams = (numberTypeParams: [string, boolean, boolean]
         return;
       }
 
-      query[key] = val as any;
+      query[key] = val;
     }
   }
 
@@ -157,7 +177,7 @@ const parseBooleanTypeQueryParams = (booleanTypeParams: [string, boolean, boolea
           return;
         }
 
-        query[key] = vals as any;
+        query[key] = vals;
       }
 
       delete query[`${key}[]`];
@@ -169,7 +189,7 @@ const parseBooleanTypeQueryParams = (booleanTypeParams: [string, boolean, boolea
         return;
       }
 
-      query[key] = val as any;
+      query[key] = val;
     }
   }
 
@@ -334,6 +354,7 @@ export default (fastify: FastifyInstance, options: FrourioOptions = {}) => {
       onRequest: hooks_gx3glp.onRequest,
       preParsing: hooks_gx3glp.preParsing,
       preValidation: [
+        callParserIfExistsQuery(parseStringArrayTypeQueryParams([['strArray', false], ['optionalStrArray', true]])),
         callParserIfExistsQuery(parseNumberTypeQueryParams([['requiredNum', false, false], ['optionalNum', true, false], ['optionalNumArr', true, true], ['emptyNum', true, false], ['requiredNumArr', false, true]])),
         callParserIfExistsQuery(parseBooleanTypeQueryParams([['bool', false, false], ['optionalBool', true, false], ['boolArray', false, true], ['optionalBoolArray', true, true]])),
       ],
@@ -349,6 +370,7 @@ export default (fastify: FastifyInstance, options: FrourioOptions = {}) => {
       onRequest: hooks_gx3glp.onRequest,
       preParsing: hooks_gx3glp.preParsing,
       preValidation: [
+        parseStringArrayTypeQueryParams([['strArray', false], ['optionalStrArray', true]]),
         parseNumberTypeQueryParams([['requiredNum', false, false], ['optionalNum', true, false], ['optionalNumArr', true, true], ['emptyNum', true, false], ['requiredNumArr', false, true]]),
         parseBooleanTypeQueryParams([['bool', false, false], ['optionalBool', true, false], ['boolArray', false, true], ['optionalBoolArray', true, true]]),
         formatMultipartData([['optionalNumArr', true], ['requiredNumArr', false], ['boolArray', false], ['optionalBoolArray', true]], [['requiredNum', false, false], ['optionalNum', true, false], ['optionalNumArr', true, true], ['emptyNum', true, false], ['requiredNumArr', false, true]], [['bool', false, false], ['optionalBool', true, false], ['boolArray', false, true], ['optionalBoolArray', true, true]]),
@@ -368,6 +390,7 @@ export default (fastify: FastifyInstance, options: FrourioOptions = {}) => {
       onRequest: hooks_gx3glp.onRequest,
       preParsing: hooks_gx3glp.preParsing,
       preValidation: [
+        parseStringArrayTypeQueryParams([['strArray', false], ['optionalStrArray', true]]),
         parseNumberTypeQueryParams([['requiredNum', false, false], ['optionalNum', true, false], ['optionalNumArr', true, true], ['emptyNum', true, false], ['requiredNumArr', false, true]]),
         parseBooleanTypeQueryParams([['bool', false, false], ['optionalBool', true, false], ['boolArray', false, true], ['optionalBoolArray', true, true]]),
         ...controller_14i7wcv.put.hooks.preValidation,

diff --git a/servers/all/api/controller.ts b/servers/all/api/controller.ts
@@ -46,6 +46,8 @@ export default defineController(
           emptyNum: z.number().optional(),
           requiredNumArr: z.array(z.number()),
           id: z.string(),
+          strArray: z.array(z.string()),
+          optionalStrArray: z.array(z.string()).optional(),
           disable: z.string(),
           bool: z.boolean(),
           optionalBool: z.boolean().optional(),

diff --git a/servers/all/api/index.ts b/servers/all/api/index.ts
@@ -27,6 +27,8 @@ export type Methods = DefineMethods<{
       emptyNum?: number | undefined;
       requiredNumArr: number[];
       id: string;
+      strArray: string[];
+      optionalStrArray?: string[] | undefined;
       disable: string;
       bool: boolean;
       optionalBool?: boolean | undefined;

diff --git a/servers/all/api/users/_userId@number/controller.ts b/servers/all/api/users/_userId@number/controller.ts
@@ -8,6 +8,7 @@ export default defineController(() => ({
   get: {
     hooks: {
       preHandler: (req, _, done) => {
+        // eslint-disable-next-line @typescript-eslint/no-unused-expressions
         !req.name && done();
       },
     },

diff --git a/servers/all/validators.ts b/servers/all/validators.ts
@@ -8,6 +8,8 @@ export type Query = {
   emptyNum?: number | undefined;
   requiredNumArr: number[];
   id: string;
+  strArray: string[];
+  optionalStrArray?: string[] | undefined;
   disable: string;
   bool: boolean;
   optionalBool?: boolean | undefined;
@@ -55,6 +57,8 @@ export const queryValidator: z.ZodType<Query> = z.object({
   emptyNum: z.number().optional(),
   requiredNumArr: z.array(z.number()),
   id: z.string().regex(/^\d+$/),
+  strArray: z.array(z.string()),
+  optionalStrArray: z.array(z.string()).optional(),
   disable: z.enum(['true', 'false']),
   bool: z.boolean(),
   optionalBool: z.boolean().optional(),

diff --git a/src/buildServerFile.ts b/src/buildServerFile.ts
@@ -15,6 +15,7 @@ const ${isAsync ? 'asyncM' : 'm'}ethodToHandler = (
 
 export default (input: string, project?: string) => {
   const { imports, consts, controllers } = createControllersText(`${input}/api`, project ?? input);
+  const hasStringArrayTypeQuery = controllers.includes('parseStringArrayTypeQueryParams(');
   const hasNumberTypeQuery = controllers.includes('parseNumberTypeQueryParams(');
   const hasBooleanTypeQuery = controllers.includes('parseBooleanTypeQueryParams(');
   const hasOptionalQuery = controllers.includes(' callParserIfExistsQuery(');
@@ -132,8 +133,32 @@ export type ServerMethodHandler<T extends AspidaMethodParams,  U extends Record<
   handler: ServerHandler<T, U> | ServerHandlerPromise<T, U>;
 };
 ${
-  hasNumberTypeQuery
+  hasStringArrayTypeQuery
     ? `
+const parseStringArrayTypeQueryParams = (stringArrayTypeParams: [string, boolean][]): preValidationHookHandler => (req, reply, done) => {
+  const query: any = req.query;
+
+  for (const [key, isOptional] of stringArrayTypeParams) {
+    const param = query[\`\${key}[]\`] ?? query[key];
+
+    if (!isOptional && param === undefined) {
+      query[key] = [];
+    } else if (!isOptional || param !== undefined) {
+      const vals = (Array.isArray(param) ? param : [param]);
+
+      query[key] = vals;
+    }
+
+    delete query[\`\${key}[]\`];
+  }
+
+  done();
+};
+`
+    : ''
+}${
+      hasNumberTypeQuery
+        ? `
 const parseNumberTypeQueryParams = (numberTypeParams: [string, boolean, boolean][]): preValidationHookHandler => (req, reply, done) => {
   const query: any = req.query;
 
@@ -151,7 +176,7 @@ const parseNumberTypeQueryParams = (numberTypeParams: [string, boolean, boolean]
           return;
         }
 
-        query[key] = vals as any;
+        query[key] = vals;
       }
 
       delete query[\`\${key}[]\`];
@@ -163,15 +188,15 @@ const parseNumberTypeQueryParams = (numberTypeParams: [string, boolean, boolean]
         return;
       }
 
-      query[key] = val as any;
+      query[key] = val;
     }
   }
 
   done();
 };
 `
-    : ''
-}${
+        : ''
+    }${
       hasBooleanTypeQuery
         ? `
 const parseBooleanTypeQueryParams = (booleanTypeParams: [string, boolean, boolean][]): preValidationHookHandler => (req, reply, done) => {
@@ -191,7 +216,7 @@ const parseBooleanTypeQueryParams = (booleanTypeParams: [string, boolean, boolea
           return;
         }
 
-        query[key] = vals as any;
+        query[key] = vals;
       }
 
       delete query[\`\${key}[]\`];
@@ -203,7 +228,7 @@ const parseBooleanTypeQueryParams = (booleanTypeParams: [string, boolean, boolea
         return;
       }
 
-      query[key] = val as any;
+      query[key] = val;
     }
   }
 

diff --git a/src/cleanStaleRoutes.ts b/src/cleanStaleRoutes.ts
@@ -14,7 +14,7 @@ export const isStaleRouteDir = (routeDir: string): boolean => {
       if (!isManagedJSTSFile(p.name)) return false;
     }
     return true;
-  } catch (e: unknown) {
+  } catch (_e: unknown) {
     return false;
   }
 };
@@ -29,7 +29,7 @@ export const cleanStaleRouteDir = (routeDir: string) => {
       fs.unlinkSync(path.resolve(routeDir, p.name));
     }
     fs.rmdirSync(routeDir);
-  } catch (e: unknown) {}
+  } catch (_e: unknown) {}
 };
 
 export default (dir: string, event: string, file: string): void => {

diff --git a/src/createControllersText.ts b/src/createControllersText.ts
@@ -458,6 +458,12 @@ export default (appDir: string, project: string) => {
                 ? checker.getTypeOfSymbolAtLocation(m, m.valueDeclaration).getProperties()
                 : [];
               const query = props.find(p => p.name === 'query');
+              const stringArrayTypeQueryParams =
+                query &&
+                getSomeTypeParams('string', query)
+                  ?.filter(params => params !== null)
+                  .filter(params => params.endsWith(', true]'))
+                  .map(params => params.replace(', true]', ']'));
               const numberTypeQueryParams = query && getSomeTypeParams('number', query);
               const booleanTypeQueryParams = query && getSomeTypeParams('boolean', query);
               const reqFormat = props.find(p => p.name === 'reqFormat');
@@ -476,6 +482,15 @@ export default (appDir: string, project: string) => {
                 .map(key => {
                   if (key === 'preValidation') {
                     const texts = [
+                      stringArrayTypeQueryParams?.length
+                        ? query?.declarations?.some(
+                            d => d.getChildAt(1).kind === ts.SyntaxKind.QuestionToken,
+                          )
+                          ? `callParserIfExistsQuery(parseStringArrayTypeQueryParams([${stringArrayTypeQueryParams.join(
+                              ', ',
+                            )}]))`
+                          : `parseStringArrayTypeQueryParams([${stringArrayTypeQueryParams.join(', ')}])`
+                        : '',
                       numberTypeQueryParams?.length
                         ? query?.declarations?.some(
                             d => d.getChildAt(1).kind === ts.SyntaxKind.QuestionToken,

diff --git a/tests/index.spec.ts b/tests/index.spec.ts
@@ -41,6 +41,7 @@ test('GET: 200', () =>
         requiredNum: 1,
         requiredNumArr: [1, 2],
         id: '1',
+        strArray: [],
         disable: 'false',
         bool: true,
         boolArray: [false, true],
@@ -50,6 +51,7 @@ test('GET: 200', () =>
         emptyNum: 0,
         requiredNumArr: [],
         id: '1',
+        strArray: ['aa'],
         disable: 'false',
         bool: false,
         optionalBool: true,
@@ -201,6 +203,7 @@ test('PUT: zod validations', async () => {
           requiredNum: 0,
           requiredNumArr: [],
           id: '1',
+          strArray: [],
           disable: 'true',
           bool: false,
           boolArray: [],
@@ -214,6 +217,7 @@ test('PUT: zod validations', async () => {
           requiredNum: 0,
           requiredNumArr: [],
           id: '1',
+          strArray: [],
           disable: 'true',
           bool: 1 as any,
           boolArray: [],
@@ -229,6 +233,8 @@ test('PUT: zod validations', async () => {
         requiredNum: 0,
         requiredNumArr: [],
         id: '1',
+        strArray: ['aa', 'bb'],
+        optionalStrArray: ['cc'],
         disable: 'true',
         bool: false,
         boolArray: [],
@@ -366,6 +372,8 @@ test('controller dependency injection', async () => {
         requiredNum: 1,
         requiredNumArr: [0],
         disable: 'true',
+        strArray: ['aa'],
+        optionalStrArray: [],
         bool: false,
         boolArray: [],
       },