fix(davinci-app): handle ReadOnlyCollector in label component to fix CI crashes

label.ts was typed to accept only RichTextCollector but main.ts was
passing ReadOnlyCollector, causing a runtime crash when accessing
richContent (undefined) that silently killed the collector render loop
and caused e2e timeouts.

Author: ryanbas21

e2e/davinci-app/components/label.ts

@@ -4,11 +4,21 @@
  * This software may be modified and distributed under the terms
  * of the MIT license. See the LICENSE file for details.
  */
-import type { RichTextCollector } from '@forgerock/davinci-client/types';
+import type { ReadOnlyCollector, RichTextCollector } from '@forgerock/davinci-client/types';
 
-export default function (formEl: HTMLFormElement, collector: RichTextCollector) {
+export default function (
+  formEl: HTMLFormElement,
+  collector: ReadOnlyCollector | RichTextCollector,
+) {
   const p = document.createElement('p');
   p.style.whiteSpace = 'pre-line';
+
+  if (collector.type !== 'RichTextCollector') {
+    p.innerText = collector.output.content;
+    formEl?.appendChild(p);
+    return;
+  }
+
   const { richContent } = collector.output;
 
   if (richContent.replacements.length === 0) {

e2e/davinci-app/main.ts

@@ -221,7 +221,7 @@ const urlParams = new URLSearchParams(window.location.search);
           davinciClient.update(collector), // Returns an update function for this collector
           submitForm,
         );
-      } else if (collector.type === 'ReadOnlyCollector') {
+      } else if (collector.type === 'ReadOnlyCollector' || collector.type === 'RichTextCollector') {
         labelComponent(
           formEl, // You can ignore this; it's just for rendering
           collector, // This is the plain object of the collector