@carto/api-client 0.5.0-alpha.9 → 0.5.0

package/src/widget-sources/widget-tileset-source.ts

@@ -55,18 +55,20 @@ export type WidgetTilesetSourceResult = {widgetSource: WidgetTilesetSource};
  * const { widgetSource } = await data;
  * ```
  */
-export class WidgetTilesetSource extends WidgetSource<WidgetTilesetSourceProps> {
+export class WidgetTilesetSource<
+  Props extends WidgetTilesetSourceProps = WidgetTilesetSourceProps,
+> extends WidgetSource<Props> {
   protected _localImpl: WidgetTilesetSourceImpl | null = null;
 
   protected _workerImpl: Worker | null = null;
   protected _workerEnabled: boolean;
   protected _workerNextRequestId = 1;
 
-  constructor(props: WidgetTilesetSourceProps) {
+  constructor(props: Props) {
     super(props);
 
     this._workerEnabled =
-      (props.widgetSourceWorker ?? true) &&
+      (props.widgetWorker ?? true) &&
       TSUP_FORMAT !== 'cjs' &&
       typeof Worker !== 'undefined';
 
@@ -127,15 +129,6 @@ export class WidgetTilesetSource extends WidgetSource<WidgetTilesetSourceProps>
     const worker = this._getWorker();
     const requestId = this._workerNextRequestId++;
 
-    // TODO: ViewState may contain non-serializable data, which we do not need.
-    // Remove this sanitization after sc-469614 is fixed.
-    const options = params[0] as any;
-    if (options?.spatialIndexReferenceViewState) {
-      const {zoom, latitude, longitude} =
-        options.spatialIndexReferenceViewState;
-      options.spatialIndexReferenceViewState = {zoom, latitude, longitude};
-    }
-
     let resolve: ((value: T) => void) | null = null;
     let reject: ((reason: any) => void) | null = null;
 
@@ -145,10 +138,9 @@ export class WidgetTilesetSource extends WidgetSource<WidgetTilesetSourceProps>
     function onMessage(e: MessageEvent) {
       const response = e.data as WorkerResponse;
       if (response.requestId !== requestId) return;
+      if (signal?.aborted) return; // handled by 'abort' listener
 
-      if (signal?.aborted) {
-        reject!(new Error(signal.reason));
-      } else if (response.ok) {
+      if (response.ok) {
         resolve!(response.result as T);
       } else {
         reject!(new Error(response.error));
@@ -199,11 +191,22 @@ export class WidgetTilesetSource extends WidgetSource<WidgetTilesetSourceProps>
 
     const worker = this._getWorker();
 
-    tiles = (tiles as Tile[]).map(({id, bbox, data}) => ({
-      id,
-      bbox,
-      data,
-    }));
+    // We cannot pass an instance of Tile2DHeader to a Web Worker, and must
+    // extract properties required for widget calculations. Note that the
+    // `tile: Tile = {...}` assignment is structured so TS will warn if any
+    // types required by the internal 'Tile' type are missing.
+    tiles = (tiles as Tile[]).map(
+      ({id, index, bbox, isVisible, data}: Tile) => {
+        const tile: Tile = {
+          id,
+          index,
+          isVisible,
+          data,
+          bbox,
+        };
+        return tile;
+      }
+    );
 
     worker.postMessage({
       method: Method.LOAD_TILES,
@@ -220,7 +223,7 @@ export class WidgetTilesetSource extends WidgetSource<WidgetTilesetSourceProps>
     const worker = this._getWorker();
 
     worker.postMessage({
-      type: Method.SET_TILE_FEATURE_EXTRACT_OPTIONS,
+      method: Method.SET_TILE_FEATURE_EXTRACT_OPTIONS,
       params: [options],
     });
   }

package/src/workers/widget-tileset-worker.ts

@@ -18,7 +18,7 @@ addEventListener('message', (e) => {
   if (method === Method.INIT) {
     source = new WidgetTilesetSourceImpl({
       ...(params[0] as WidgetTilesetSourceProps),
-      widgetSourceWorker: false,
+      widgetWorker: false,
     });
     return;
   }