chore: refactor custom field filter schemas for improved clarity and maintainability

Author: trknhr

src/backlog/customFields.ts

@@ -33,17 +33,20 @@ export type CustomFieldFilterInput =
  */
 export function customFieldsToPayload(
   customFields: CustomFieldInput[] | undefined
-): Record<string, string | number | string[] | number[]| undefined> {
+): Record<string, string | number | string[] | number[] | undefined> {
   if (customFields == null) {
     return {};
   }
-  const result: Record<string, string | number | string[] | number[] | undefined> = {};
+  const result: Record<
+    string,
+    string | number | string[] | number[] | undefined
+  > = {};
 
   for (const field of customFields) {
-    if(field.value !== undefined){
+    if (field.value !== undefined) {
       result[`customField_${field.id}`] = field.value;
-    } 
-    if(field.otherValue !== undefined) {
+    }
+    if (field.otherValue !== undefined) {
       result[`customField_${field.id}_otherValue`] = field.otherValue;
     }
   }
@@ -101,11 +104,12 @@ export function customFieldFiltersToPayload(
       }
       default: {
         const exhaustiveCheck: never = field;
-        throw new Error(`Unsupported custom field filter type: ${exhaustiveCheck}`);
+        throw new Error(
+          `Unsupported custom field filter type: ${exhaustiveCheck}`
+        );
       }
     }
   }
 
   return result;
 }
-

src/tools/addIssue.ts

@@ -79,9 +79,13 @@ const addIssueSchema = buildToolSchema((t) => ({
               'The ID of the custom field (e.g., 12345)'
             )
           ),
-        value: z.union([z.number(), z.array(z.number())]).optional()
-        .describe("The ID(s) of the custom field item. For single-select fields, provide a number. For multi-select fields, provide an array of numbers representing the selected item IDs."),
-         otherValue: z
+        value: z
+          .union([z.number(), z.array(z.number())])
+          .optional()
+          .describe(
+            'The ID(s) of the custom field item. For single-select fields, provide a number. For multi-select fields, provide an array of numbers representing the selected item IDs.'
+          ),
+        otherValue: z
           .string()
           .optional()
           .describe(

src/tools/shared/customFieldFiltersSchema.ts

@@ -1,9 +1,7 @@
 import { z, ZodIssueCode } from 'zod';
 import { TranslationHelper } from '../../createTranslationHelper.js';
 
-export const buildCustomFieldFilterSchema = (
-  t: TranslationHelper['t']
-) => {
+export const buildCustomFieldFilterSchema = (t: TranslationHelper['t']) => {
   const schema = z.discriminatedUnion('type', [
     z
       .object({
@@ -66,19 +64,13 @@ export const buildCustomFieldFilterSchema = (
           .string()
           .optional()
           .describe(
-            t(
-              'TOOL_CUSTOM_FIELD_FILTER_DATE_MIN',
-              'Start date (yyyy-MM-dd)'
-            )
+            t('TOOL_CUSTOM_FIELD_FILTER_DATE_MIN', 'Start date (yyyy-MM-dd)')
           ),
         max: z
           .string()
           .optional()
           .describe(
-            t(
-              'TOOL_CUSTOM_FIELD_FILTER_DATE_MAX',
-              'End date (yyyy-MM-dd)'
-            )
+            t('TOOL_CUSTOM_FIELD_FILTER_DATE_MAX', 'End date (yyyy-MM-dd)')
           ),
       })
       .describe(t('TOOL_CUSTOM_FIELD_FILTER_DATE', 'Date custom field filter')),
@@ -103,7 +95,11 @@ export const buildCustomFieldFilterSchema = (
   ]);
 
   return schema.superRefine((data, ctx) => {
-    if (data.type === 'numeric' && data.min === undefined && data.max === undefined) {
+    if (
+      data.type === 'numeric' &&
+      data.min === undefined &&
+      data.max === undefined
+    ) {
       ctx.addIssue({
         code: ZodIssueCode.custom,
         message: t(
@@ -127,4 +123,6 @@ export const buildCustomFieldFilterSchema = (
   });
 };
 
-export type CustomFieldFilterSchema = ReturnType<typeof buildCustomFieldFilterSchema>;
+export type CustomFieldFilterSchema = ReturnType<
+  typeof buildCustomFieldFilterSchema
+>;

src/tools/updateIssue.ts

@@ -112,9 +112,13 @@ const updateIssueSchema = buildToolSchema((t) => ({
               'The ID of the custom field (e.g., 12345)'
             )
           ),
-        value: z.union([z.number(), z.array(z.number())]).optional()
-        .describe("The ID(s) of the custom field item. For single-select fields, provide a number. For multi-select fields, provide an array of numbers representing the selected item IDs."),
-         otherValue: z
+        value: z
+          .union([z.number(), z.array(z.number())])
+          .optional()
+          .describe(
+            'The ID(s) of the custom field item. For single-select fields, provide a number. For multi-select fields, provide an array of numbers representing the selected item IDs.'
+          ),
+        otherValue: z
           .string()
           .optional()
           .describe(