refactor: update getPaginationKeywordsRegExp to accept keywords directly from config input

Author: fml09

packages/openapi-ts/src/ir/__tests__/pagination.test.ts

@@ -46,7 +46,6 @@ describe('paginationKeywordsRegExp', () => {
     'is $value pagination param? $output',
     async ({ result, value }) => {
       const paginationRegExp = getPaginationKeywordsRegExp();
-      paginationRegExp.lastIndex = 0;
       expect(paginationRegExp.test(value)).toEqual(result);
     },
   );
@@ -64,16 +63,11 @@ describe('paginationKeywordsRegExp', () => {
   it.each(customScenarios)(
     'with custom config, $value should match? $result',
     async ({ result, value }) => {
-      const config: Config = {
-        input: {
-          pagination: {
-            keywords: ['customPagination', 'pageSize', 'perPage'],
-          },
-        },
-      } as Config;
+      const pagination: Config['input']['pagination'] = {
+        keywords: ['customPagination', 'pageSize', 'perPage'],
+      };
 
-      const paginationRegExp = getPaginationKeywordsRegExp(config);
-      paginationRegExp.lastIndex = 0;
+      const paginationRegExp = getPaginationKeywordsRegExp(pagination);
       expect(paginationRegExp.test(value)).toEqual(result);
     },
   );

packages/openapi-ts/src/ir/pagination.ts

@@ -10,9 +10,9 @@ export const DEFAULT_PAGINATION_KEYWORDS = [
   'start',
 ];
 
-export function getPaginationKeywordsRegExp(config?: Config): RegExp {
-  const configKeywords = config?.input?.pagination?.keywords;
-  const keywords = configKeywords || DEFAULT_PAGINATION_KEYWORDS;
+export function getPaginationKeywordsRegExp({
+  keywords = DEFAULT_PAGINATION_KEYWORDS,
+}: Config['input']['pagination'] = {}): RegExp {
   const pattern = `^(${keywords.join('|')})$`;
   return new RegExp(pattern);
 }

packages/openapi-ts/src/openApi/2.0.x/parser/pagination.ts

@@ -29,8 +29,9 @@ export const paginationField = ({
         in: undefined;
       };
 }): boolean | string => {
-  const paginationRegExp = getPaginationKeywordsRegExp(context.config);
-  paginationRegExp.lastIndex = 0;
+  const paginationRegExp = getPaginationKeywordsRegExp(
+    context.config.input.pagination,
+  );
   if (paginationRegExp.test(name)) {
     return true;
   }
@@ -84,8 +85,9 @@ export const paginationField = ({
   }
 
   for (const name in schema.properties) {
-    const paginationRegExp = getPaginationKeywordsRegExp(context.config);
-    paginationRegExp.lastIndex = 0;
+    const paginationRegExp = getPaginationKeywordsRegExp(
+      context.config.input.pagination,
+    );
 
     if (paginationRegExp.test(name)) {
       const property = schema.properties[name]!;

packages/openapi-ts/src/openApi/3.0.x/parser/pagination.ts

@@ -28,8 +28,9 @@ export const paginationField = ({
   name: string;
   schema: SchemaObject | ReferenceObject;
 }): boolean | string => {
-  const paginationRegExp = getPaginationKeywordsRegExp(context.config);
-  paginationRegExp.lastIndex = 0;
+  const paginationRegExp = getPaginationKeywordsRegExp(
+    context.config.input.pagination,
+  );
   if (paginationRegExp.test(name)) {
     return true;
   }
@@ -73,8 +74,9 @@ export const paginationField = ({
   }
 
   for (const name in schema.properties) {
-    const paginationRegExp = getPaginationKeywordsRegExp(context.config);
-    paginationRegExp.lastIndex = 0;
+    const paginationRegExp = getPaginationKeywordsRegExp(
+      context.config.input.pagination,
+    );
 
     if (paginationRegExp.test(name)) {
       const property = schema.properties[name]!;

packages/openapi-ts/src/openApi/3.1.x/parser/pagination.ts

@@ -24,8 +24,9 @@ export const paginationField = ({
   name: string;
   schema: SchemaObject;
 }): boolean | string => {
-  const paginationRegExp = getPaginationKeywordsRegExp(context.config);
-  paginationRegExp.lastIndex = 0;
+  const paginationRegExp = getPaginationKeywordsRegExp(
+    context.config.input.pagination,
+  );
   if (paginationRegExp.test(name)) {
     return true;
   }
@@ -69,8 +70,9 @@ export const paginationField = ({
   }
 
   for (const name in schema.properties) {
-    const paginationRegExp = getPaginationKeywordsRegExp(context.config);
-    paginationRegExp.lastIndex = 0;
+    const paginationRegExp = getPaginationKeywordsRegExp(
+      context.config.input.pagination,
+    );
 
     if (paginationRegExp.test(name)) {
       const property = schema.properties[name]!;

packages/openapi-ts/src/plugins/@tanstack/query-core/plugin-legacy.ts

@@ -900,8 +900,9 @@ export const handlerLegacy: Plugin.LegacyHandler<
         let paginationField!: Model | OperationParameter;
 
         const paginationParameter = operation.parameters.find((parameter) => {
-          const paginationRegExp = getPaginationKeywordsRegExp(config);
-          paginationRegExp.lastIndex = 0;
+          const paginationRegExp = getPaginationKeywordsRegExp(
+            config.input.pagination,
+          );
           if (paginationRegExp.test(parameter.name)) {
             paginationField = parameter;
             return true;
@@ -917,8 +918,9 @@ export const handlerLegacy: Plugin.LegacyHandler<
               (model) => model.meta?.$ref === ref,
             );
             return refModel?.properties.find((property) => {
-              const paginationRegExp = getPaginationKeywordsRegExp(config);
-              paginationRegExp.lastIndex = 0;
+              const paginationRegExp = getPaginationKeywordsRegExp(
+                config.input.pagination,
+              );
               if (paginationRegExp.test(property.name)) {
                 paginationField = property;
                 return true;
@@ -927,8 +929,9 @@ export const handlerLegacy: Plugin.LegacyHandler<
           }
 
           return parameter.properties.find((property) => {
-            const paginationRegExp = getPaginationKeywordsRegExp(config);
-            paginationRegExp.lastIndex = 0;
+            const paginationRegExp = getPaginationKeywordsRegExp(
+              config.input.pagination,
+            );
             if (paginationRegExp.test(property.name)) {
               paginationField = property;
               return true;