CCM-11537: Remove stray filter types

Author: ClareJonesBJSS

lambdas/backend-api/src/__tests__/templates/app/template-client.test.ts

@@ -10,16 +10,14 @@ import type {
 import { TemplateRepository } from '@backend-api/templates/infra';
 import { TemplateClient } from '@backend-api/templates/app/template-client';
 import { LetterUploadRepository } from '@backend-api/templates/infra/letter-upload-repository';
-import {
-  DatabaseTemplate,
-  TemplateFilter,
-} from 'nhs-notify-web-template-management-utils';
+import { DatabaseTemplate } from 'nhs-notify-web-template-management-utils';
 import { ProofingQueue } from '@backend-api/templates/infra/proofing-queue';
 import { createMockLogger } from 'nhs-notify-web-template-management-test-helper-utils/mock-logger';
 import { isoDateRegExp } from 'nhs-notify-web-template-management-test-helper-utils';
 import { ClientConfigRepository } from '@backend-api/templates/infra/client-config-repository';
 import { isRightToLeft } from 'nhs-notify-web-template-management-utils/enum';
 import { TemplateQuery } from '@backend-api/templates/infra/template-repository/query';
+import { TemplateFilter } from 'nhs-notify-backend-client/src/types/filters';
 
 jest.mock('node:crypto');
 jest.mock('nhs-notify-web-template-management-utils/enum');

lambdas/backend-api/src/templates/app/template-client.ts

@@ -11,8 +11,7 @@ import {
   $CreateUpdateNonLetter,
   ClientConfiguration,
   $TemplateDto,
-  $ListTemplateFilters,
-  ListTemplateFilters,
+  $TemplateFilter,
 } from 'nhs-notify-backend-client';
 import { TemplateRepository } from '@backend-api/templates/infra';
 import { LETTER_MULTIPART } from 'nhs-notify-backend-client/src/schemas/constants';
@@ -28,6 +27,7 @@ import { z } from 'zod/v4';
 import { LetterUploadRepository } from '../infra/letter-upload-repository';
 import { ProofingQueue } from '../infra/proofing-queue';
 import { ClientConfigRepository } from '../infra/client-config-repository';
+import { TemplateFilter } from 'nhs-notify-backend-client/src/types/filters';
 
 export class TemplateClient {
   private $LetterForProofing = z.intersection(
@@ -373,10 +373,10 @@ export class TemplateClient {
     user: User,
     filters?: unknown
   ): Promise<Result<TemplateDto[]>> {
-    let parsedFilters: ListTemplateFilters = {};
+    let parsedFilters: TemplateFilter = {};
 
     if (filters) {
-      const validation = await validate($ListTemplateFilters, filters);
+      const validation = await validate($TemplateFilter, filters);
 
       if (validation.error) {
         return validation;

lambdas/backend-client/src/schemas/template.ts

@@ -31,6 +31,7 @@ import {
   TEMPLATE_TYPE_LIST,
   VIRUS_SCAN_STATUS_LIST,
 } from './union-lists';
+import { TemplateFilter } from '../types/filters';
 
 export const $LetterType = schemaFor<LetterType>()(z.enum(LETTER_TYPE_LIST));
 
@@ -163,18 +164,13 @@ export const $TemplateDto = schemaFor<TemplateDto>()(
   ])
 );
 
-export type ListTemplateFilters = {
-  templateStatus?: TemplateStatus;
-  templateType?: TemplateType;
-  language?: Language;
-  letterType?: LetterType;
-};
-
-export const $ListTemplateFilters = schemaFor<ListTemplateFilters>()(
-  z.object({
-    templateStatus: $TemplateStatusActive.optional(),
-    templateType: $TemplateType.optional(),
-    language: $Language.optional(),
-    letterType: $LetterType.optional(),
-  })
+export const $TemplateFilter = schemaFor<TemplateFilter>()(
+  z
+    .object({
+      templateStatus: $TemplateStatusActive,
+      templateType: $TemplateType,
+      language: $Language,
+      letterType: $LetterType,
+    })
+    .partial()
 );

utils/utils/src/types.ts

@@ -123,13 +123,6 @@ export type DatabaseTemplate = {
   supplierReferences?: Record<string, string>;
 } & DbOnlyTemplateProperties;
 
-export type TemplateFilter = Partial<
-  Pick<
-    DatabaseTemplate,
-    'templateStatus' | 'templateType' | 'language' | 'letterType'
-  >
->;
-
 type DbOnlyTemplateProperties = {
   owner: string;
   version: number;