Prefer Annotating Types Over Asserting

containers/ecr-viewer/src/app/services/ecrMetadataService.ts

@@ -58,7 +58,7 @@ export const evaluateEcrMetadata = (
   fhirBundle: Bundle,
   mappings: PathMappings,
 ): EcrMetadata => {
-  const rrDetails = evaluate(fhirBundle, mappings.rrDetails) as Observation[];
+  const rrDetails: Observation[] = evaluate(fhirBundle, mappings.rrDetails);
 
   let reportableConditionsList: ReportableConditions = {};
 
@@ -97,11 +97,11 @@ export const evaluateEcrMetadata = (
   }
 
   const custodianRef = evaluate(fhirBundle, mappings.eicrCustodianRef)[0] ?? "";
-  const custodian = evaluateReference(
+  const custodian: Organization = evaluateReference(
     fhirBundle,
     mappings,
     custodianRef,
-  ) as Organization;
+  );
 
   const eicrReleaseVersion = (fhirBundle: any, mappings: any) => {
     const releaseVersion = evaluate(fhirBundle, mappings.eicrReleaseVersion)[0];

containers/ecr-viewer/src/app/services/evaluateFhirDataService.ts

@@ -594,8 +594,8 @@ export const evaluateEmergencyContact = (
   fhirBundle: Bundle,
   mappings: PathMappings,
 ) => {
-  const contacts = (evaluate(fhirBundle, mappings.patientEmergencyContact) ??
-    []) as PatientContact[];
+  const contacts: PatientContact[] =
+    evaluate(fhirBundle, mappings.patientEmergencyContact) ?? [];
 
   if (contacts.length === 0) return undefined;
 
@@ -655,19 +655,19 @@ export const evaluateValue = (entry: Element, path: string | Path): string => {
   ) {
     value = originalValue.toString();
   } else if (originalValuePath === "Quantity") {
-    const data = originalValue as Quantity;
+    const data: Quantity = originalValue;
     let unit = data.unit;
     const firstLetterRegex = /^[a-z]/i;
     if (unit?.match(firstLetterRegex)) {
       unit = " " + unit;
     }
     value = `${data.value ?? ""}${unit ?? ""}`;
   } else if (originalValuePath === "CodeableConcept") {
-    const data = originalValue as CodeableConcept;
+    const data: CodeableConcept = originalValue;
     value =
       data.coding?.[0].display || data.text || data.coding?.[0].code || "";
   } else if (originalValuePath === "Coding") {
-    const data = originalValue as Coding;
+    const data: Coding = originalValue;
     value = data?.display || data?.code || "";
   } else if (typeof originalValue === "object") {
     console.log(`Not implemented for ${originalValue.__path__}`);

containers/ecr-viewer/src/app/services/formatService.tsx

@@ -384,9 +384,8 @@ export function formatTablesToJSON(htmlString: string): TableJson[] {
   }
 
   // <table><caption>{name}</caption></table>
-  const tableWithCaptionArray = doc.querySelectorAll(
-    "table:has(caption)",
-  ) as NodeListOf<HTMLTableElement>;
+  const tableWithCaptionArray: NodeListOf<HTMLTableElement> =
+    doc.querySelectorAll("table:has(caption)");
   if (tableWithCaptionArray.length > 0) {
     tableWithCaptionArray.forEach((table) => {
       const resultName = getElementText(table.caption as Element);

containers/ecr-viewer/src/app/services/labsService.tsx

@@ -382,7 +382,7 @@ export const evaluateDiagnosticReportData = (
       columnName: "Test Method",
       infoPath: "observationDeviceReference",
       applyToValue: (ref) => {
-        const device = evaluateReference(fhirBundle, mappings, ref) as Device;
+        const device: Device = evaluateReference(fhirBundle, mappings, ref);
         return safeParse(device.deviceName?.[0]?.name ?? "");
       },
       className: "minw-10 width-20",

containers/ecr-viewer/src/app/view-data/components/PatientBanner.tsx

@@ -25,8 +25,9 @@ const PatientBanner = ({ bundle, mappings }: PatientBannerProps) => {
       </span>
       <span className="patient-banner-dob">
         {bundle && mappings
-          ? formatDate((evaluate(bundle, mappings.patientDOB) as any[])[0]) ||
-            ""
+          ? formatDate(
+              (evaluate(bundle, mappings.patientDOB) as string[])[0],
+            ) || ""
           : ""}
       </span>
     </div>

containers/ecr-viewer/src/app/view-data/components/common.tsx

@@ -85,11 +85,11 @@ export const returnCareTeamTable = (
       (entry.period as any).text = formatStartEndDate(start, end);
     }
 
-    const practitioner = evaluateReference(
+    const practitioner: Practitioner = evaluateReference(
       bundle,
       mappings,
       entry?.member?.reference || "",
-    ) as Practitioner;
+    );
     const practitionerNameObj = practitioner.name?.find(
       (nameObject) => nameObject.family,
     );
@@ -144,11 +144,11 @@ export const returnImmunizations = (
   immunizationsArray.forEach((entry) => {
     entry.occurrenceDateTime = formatDateTime(entry.occurrenceDateTime ?? "");
 
-    const manufacturer = evaluateReference(
+    const manufacturer: Organization = evaluateReference(
       fhirBundle,
       mappings,
       entry.manufacturer?.reference || "",
-    ) as Organization;
+    );
     if (manufacturer) {
       (entry.manufacturer as any).name = manufacturer.name || "";
     }