Clearer warning comment about structuredClone

Author: jrentlez

libs/execution/src/lib/types/io-types/table.ts

@@ -33,12 +33,21 @@ export class TableColumn<
   clone(): TableColumn<T> {
     const cloned: (T | InternalErrorValueRepresentation)[] = [];
     for (const value of this.values) {
+      // WARNING: structuredClone has unintuitive behavior for Error subclasses.
+      //
+      // structuredClone replaces our custom Error's `name` field with
+      // `"Error"`, which leads to future type checks failing, even though they
+      // should not.
+      //
+      // TODO: Decide whether to implement a custom clone for
+      // InternalValidValueRepresentation. This may require disabling nested
+      // arrays.
+      //
+      // More information:
+      // https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Structured_clone_algorithm#supported_types
       if (ERROR_TYPEGUARD(value)) {
         cloned.push(value.clone());
       } else {
-        // WARNING: structuredClone does not support all types!
-        // See
-        // https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Structured_clone_algorithm#supported_types
         cloned.push(structuredClone(value));
       }
     }