@carto/api-client 0.5.0 → 0.5.1-alpha.1

package/package.json

@@ -8,7 +8,7 @@
   "homepage": "https://github.com/CartoDB/carto-api-client#readme",
   "author": "Don McCurdy <donmccurdy@carto.com>",
   "packageManager": "yarn@4.3.1",
-  "version": "0.5.0",
+  "version": "0.5.1-alpha.1",
   "license": "MIT",
   "publishConfig": {
     "access": "public"
@@ -117,15 +117,16 @@
     "rimraf": "^6.0.1",
     "semver": "^7.7.1",
     "thenby": "^1.3.4",
-    "tinybench": "^3.1.1",
+    "tinybench": "^4.0.1",
     "tsup": "^8.3.6",
     "typescript": "~5.8.2",
     "typescript-eslint": "^8.26.1",
     "vite": "^6.2.2",
-    "vitest": "3.0.9"
+    "vitest": "3.1.1"
   },
   "resolutions": {
     "@carto/api-client": "portal:./",
     "rollup": "^4.20.0"
-  }
+  },
+  "stableVersion": "0.5.0"
 }

package/src/filters/tileFeaturesGeometries.ts

@@ -23,7 +23,9 @@ import {
   BinaryAttribute,
   BinaryFeature,
   BinaryGeometryType,
+  BinaryLineFeature,
   BinaryPointFeature,
+  BinaryPolygonFeature,
   TypedArrayConstructor,
 } from '@loaders.gl/schema';
 
@@ -89,8 +91,6 @@ export function tileFeaturesGeometries({
         ? transformToTileCoords(clippedGeometryToIntersect.geometry, bbox)
         : clippedGeometryToIntersect.geometry;
 
-    createIndicesForPoints(tile.data.points!);
-
     calculateFeatures({
       map,
       tileIsFullyVisible,
@@ -214,7 +214,7 @@ function addIntersectedFeaturesInTile({
   uniqueIdProperty?: string;
   options?: TileFeatureExtractOptions;
 }) {
-  const indices = getIndices(data);
+  const indices = getIndices(data, type);
   const storeGeometry = options?.storeGeometry || false;
 
   for (let i = 0; i < indices.length - 1; i++) {
@@ -235,21 +235,24 @@ function addIntersectedFeaturesInTile({
   }
 }
 
-function getIndices(data: BinaryFeature) {
+// Despite TypeScript, 'data.type' is OPTIONAL. So 'type' must be passed in
+// separately. Observed missing .type for Redshift tilesets, 2025-04-09.
+function getIndices(data: BinaryFeature, type: BinaryGeometryType) {
   let indices: BinaryAttribute;
-  switch (data.type) {
-    case 'Point':
-      // @ts-expect-error Missing or changed types?
-      indices = data.pointIndices;
+  switch (type) {
+    case 'Polygon':
+      indices = (data as BinaryPolygonFeature).primitivePolygonIndices;
       break;
     case 'LineString':
-      indices = data.pathIndices;
+      indices = (data as BinaryLineFeature).pathIndices;
       break;
-    case 'Polygon':
-      indices = data.primitivePolygonIndices;
+    case 'Point':
+      indices = createIndicesForPoints(data as BinaryPointFeature);
       break;
     default:
-      throw new Error(`Unexpected type, "${(data as BinaryFeature).type}"`);
+      throw new Error(
+        `Unsupported geometry type: ${type as unknown as string}`
+      );
   }
   return indices.value;
 }
@@ -408,7 +411,7 @@ function addAllFeaturesInTile({
   uniqueIdProperty?: string;
   options?: TileFeatureExtractOptions;
 }) {
-  const indices = getIndices(data);
+  const indices = getIndices(data, type);
   const storeGeometry = options?.storeGeometry || false;
   for (let i = 0; i < indices.length - 1; i++) {
     const startIndex = indices[i];
@@ -427,7 +430,11 @@ function addAllFeaturesInTile({
   }
 }
 
-function createIndicesForPoints(data: BinaryPointFeature) {
+/**
+ * BinaryPointFeature does not include indices, so we generate in-memory
+ * indices to allow processing points similarly to other topologies.
+ */
+function createIndicesForPoints(data: BinaryPointFeature): BinaryAttribute {
   const featureIds = data.featureIds.value;
   const lastFeatureId = featureIds[featureIds.length - 1];
   const PointIndicesArray = featureIds.constructor as TypedArrayConstructor;
@@ -438,7 +445,5 @@ function createIndicesForPoints(data: BinaryPointFeature) {
   };
   pointIndices.value.set(featureIds);
   pointIndices.value.set([lastFeatureId + 1], featureIds.length);
-
-  // @ts-expect-error Missing or changed types?
-  data.pointIndices = pointIndices;
+  return pointIndices;
 }

package/src/widget-sources/types.ts

@@ -18,7 +18,7 @@ export interface ViewState {
 }
 
 /** Common options for {@link WidgetRemoteSource} requests. */
-interface BaseRequestOptions {
+export interface BaseRequestOptions {
   signal?: AbortSignal;
   spatialFilter?: SpatialFilter;
   spatialFiltersMode?: SpatialFilterPolyfillMode;

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

@@ -387,15 +387,15 @@ function assertColumn(
   features: FeatureData[],
   ...columnArgs: string[] | string[][]
 ) {
-  // TODO(cleanup): Can drop support for multiple column shapes here?
-
   // Due to the multiple column shape, we normalise it as an array with normalizeColumns
   const columns = Array.from(new Set(columnArgs.map(normalizeColumns).flat()));
 
   const featureKeys = Object.keys(features[0]);
 
+  // For backward compatibility, '' should pass column validation. For example,
+  // operation='count',operationColumn='' is accepted.
   const invalidColumns = columns.filter(
-    (column) => !featureKeys.includes(column)
+    (column) => column && !featureKeys.includes(column)
   );
 
   if (invalidColumns.length) {