Release vintasend-medplum@0.8.2

Author: hugobessa

package.json

@@ -1,6 +1,6 @@
 {
   "name": "vintasend-medplum",
-  "version": "0.8.1",
+  "version": "0.8.2",
   "description": "",
   "main": "dist/index.js",
   "scripts": {
@@ -25,7 +25,7 @@
     "@medplum/fhirtypes": "^5.0.10",
     "pdfmake": "^0.2.23",
     "pug": "^3.0.3",
-    "vintasend": "^0.8.1"
+    "vintasend": "^0.8.2"
   },
   "devDependencies": {
     "@medplum/definitions": "^5.0.10",

src/__tests__/medplum-backend.test.ts

@@ -811,6 +811,47 @@ describe('MedplumNotificationBackend', () => {
         'contextName lookup with caseSensitive=false is not supported by MedplumNotificationBackend. Only case-sensitive exact matching is supported.',
       );
     });
+
+    it('should map supported orderBy fields and directions to FHIR _sort', async () => {
+      const searchResourcesSpy = jest
+        .spyOn(medplumClient, 'searchResources')
+        .mockResolvedValue([] as any);
+
+      const testCases = [
+        { orderBy: { field: 'sendAfter', direction: 'asc' } as const, expectedSort: 'sent' },
+        { orderBy: { field: 'sendAfter', direction: 'desc' } as const, expectedSort: '-sent' },
+        { orderBy: { field: 'sentAt', direction: 'asc' } as const, expectedSort: 'sent' },
+        { orderBy: { field: 'sentAt', direction: 'desc' } as const, expectedSort: '-sent' },
+        { orderBy: { field: 'createdAt', direction: 'asc' } as const, expectedSort: '_lastUpdated' },
+        { orderBy: { field: 'createdAt', direction: 'desc' } as const, expectedSort: '-_lastUpdated' },
+        { orderBy: { field: 'updatedAt', direction: 'asc' } as const, expectedSort: '_lastUpdated' },
+        { orderBy: { field: 'updatedAt', direction: 'desc' } as const, expectedSort: '-_lastUpdated' },
+      ];
+
+      for (const testCase of testCases) {
+        searchResourcesSpy.mockClear();
+        await backend.filterNotifications({}, 0, 10, testCase.orderBy);
+
+        expect(searchResourcesSpy).toHaveBeenCalledWith('Communication', [
+          ['_count', '10'],
+          ['_offset', '0'],
+          ['_sort', testCase.expectedSort],
+          ['_tag', 'notification'],
+        ]);
+      }
+    });
+
+    it('should throw for unsupported orderBy.readAt', async () => {
+      const searchResourcesSpy = jest
+        .spyOn(medplumClient, 'searchResources')
+        .mockResolvedValue([] as any);
+
+      await expect(
+        backend.filterNotifications({}, 0, 10, { field: 'readAt', direction: 'asc' }),
+      ).rejects.toThrow("orderBy field 'readAt' is not supported by MedplumNotificationBackend.");
+
+      expect(searchResourcesSpy).not.toHaveBeenCalled();
+    });
   });
 
   describe('getFilterCapabilities', () => {
@@ -877,5 +918,18 @@ describe('MedplumNotificationBackend', () => {
       expect(capabilities['stringLookups.includes']).toBe(false);
       expect(capabilities['stringLookups.caseInsensitive']).toBe(false);
     });
+
+    it('should mark orderBy.readAt as unsupported', () => {
+      const capabilities = backend.getFilterCapabilities();
+      expect(capabilities['orderBy.readAt']).toBe(false);
+    });
+
+    it('should mark supported orderBy fields as supported', () => {
+      const capabilities = backend.getFilterCapabilities();
+      expect(capabilities['orderBy.sendAfter']).toBe(true);
+      expect(capabilities['orderBy.sentAt']).toBe(true);
+      expect(capabilities['orderBy.createdAt']).toBe(true);
+      expect(capabilities['orderBy.updatedAt']).toBe(true);
+    });
   });
 });

src/medplum-backend.ts

@@ -6,6 +6,7 @@ import type {
   BaseNotificationBackend,
   NotificationFilter,
   NotificationFilterFields,
+  NotificationOrderBy,
 } from 'vintasend/dist/services/notification-backends/base-notification-backend';
 import { isFieldFilter } from 'vintasend/dist/services/notification-backends/base-notification-backend';
 import type {
@@ -112,9 +113,33 @@ export class MedplumNotificationBackend<Config extends BaseNotificationTypeConfi
       'stringLookups.endsWith': false,
       'stringLookups.includes': false,
       'stringLookups.caseInsensitive': false,
+      'orderBy.sendAfter': true,
+      'orderBy.sentAt': true,
+      'orderBy.readAt': false,
+      'orderBy.createdAt': true,
+      'orderBy.updatedAt': true,
     };
   }
 
+  private convertNotificationOrderByToFhirSort(orderBy: NotificationOrderBy): string {
+    const fieldMap: Record<NotificationOrderBy['field'], string> = {
+      sendAfter: 'sent',
+      sentAt: 'sent',
+      createdAt: '_lastUpdated',
+      updatedAt: '_lastUpdated',
+      readAt: '',
+    };
+
+    if (orderBy.field === 'readAt') {
+      throw new Error(
+        "orderBy field 'readAt' is not supported by MedplumNotificationBackend.",
+      );
+    }
+
+    const mappedField = fieldMap[orderBy.field];
+    return orderBy.direction === 'desc' ? `-${mappedField}` : mappedField;
+  }
+
   private resolveStringFieldForFhir(fieldName: string, value: StringFieldFilter): string {
     if (!isStringFilterLookup(value)) {
       return value;
@@ -885,6 +910,7 @@ export class MedplumNotificationBackend<Config extends BaseNotificationTypeConfi
     filter: NotificationFilter<Config>,
     page: number,
     pageSize: number,
+    orderBy?: NotificationOrderBy,
   ): Promise<AnyDatabaseNotification<Config>[]> {
     if ('or' in filter) {
       throw new Error(
@@ -895,6 +921,9 @@ export class MedplumNotificationBackend<Config extends BaseNotificationTypeConfi
     const searchParams = this.buildFhirSearchParams(filter);
     searchParams._count = pageSize.toString();
     searchParams._offset = (page * pageSize).toString();
+    if (orderBy) {
+      searchParams._sort = this.convertNotificationOrderByToFhirSort(orderBy);
+    }
 
     // Convert to string[][] so that _tag values become repeated AND parameters
     // (comma-separated _tag in a single param means OR in FHIR, which is wrong here)