Merge pull request #72 from oleksandr-danylchenko/#71-fix-losing-styleClass

#71 Fixed losing the `styleClass` on `W3CTextAnnotation` parsing

Author: rsimon

packages/text-annotator/src/model/w3c/W3CTextFormatAdapter.ts

@@ -38,13 +38,18 @@ const parseW3CTextTargets = (annotation: W3CTextAnnotation) => {
   } = annotation;
 
   const w3cTargets = Array.isArray(target) ? target : [target];
+  if (w3cTargets.length === 0) {
+    return { error: Error(`No targets found for annotation: ${annotation.id}`) };
+  }
 
   const parsed: TextAnnotationTarget = {
     creator: parseW3CUser(creator),
     created: created ? new Date(created) : undefined,
     updated: modified ? new Date(modified) : undefined,
     annotation: annotationId,
-    selector: []
+    selector: [],
+    // @ts-expect-error: `styleClass` is not part of the core `TextAnnotationTarget` type
+    styleClass: 'styleClass' in w3cTargets[0] ? w3cTargets[0].styleClass : undefined
   };
 
   for (const w3cTarget of w3cTargets) {
@@ -64,12 +69,14 @@ const parseW3CTextTargets = (annotation: W3CTextAnnotation) => {
     }, {});
 
     if (isTextSelector(selector)) {
-      parsed.selector.push({
-        ...selector,
-        id: w3cTarget.id,
-        // @ts-ignore
-        scope: w3cTarget.scope
-      });
+      parsed.selector.push(
+        {
+          ...selector,
+          id: w3cTarget.id,
+          // @ts-expect-error: `scope` is not part of the core `TextSelector` type
+          scope: w3cTarget.scope
+        }
+      );
     } else {
       const missingTypes = [
         !selector.start ? 'TextPositionSelector' : undefined,
@@ -127,7 +134,7 @@ export const serializeW3CTextAnnotation = (
     ...targetRest
   } = target;
 
-  const w3cTargets = selector.map((s) => {
+  const w3cTargets = selector.map((s): W3CTextAnnotationTarget => {
     const { id, quote, start, end, range } = s;
 
     const { prefix, suffix } = getQuoteContext(range, container);
@@ -146,10 +153,11 @@ export const serializeW3CTextAnnotation = (
     return {
       ...targetRest,
       id,
+      // @ts-expect-error: `scope` is not part of the core `TextAnnotationTarget` type
       scope: 'scope' in s ? s.scope : undefined,
       source,
       selector: w3cSelectors
-    } as W3CTextAnnotationTarget;
+    };
   });