@colixsystems/widget-sdk 0.67.0 → 0.67.1

package/dist/hooks.js

@@ -593,6 +593,12 @@ function toDatastoreError(err) {
   });
 }
 
+// sc-1579 — consecutive renders with a different serialized query before
+// `useDatastoreQuery` calls the loop out. High enough that no real interaction
+// reaches it (a settled fetch resets the count), low enough to fire instantly
+// on a genuine loop.
+const VOLATILE_QUERY_RENDER_LIMIT = 10;
+
 /**
  * Stateful datastore query hook. Returns { data, loading, error, refetch }.
  *
@@ -682,6 +688,35 @@ export function useDatastoreQuery(table, query) {
       return null;
     }
   })();
+
+  // sc-1579 — a query argument built from a volatile value (`Date.now()`,
+  // `new Date()`, `Math.random()`) serialises differently every render, so the
+  // effect below re-fetches forever and the widget never settles. A genuine
+  // edit (a typed filter) settles within one render because the re-render the
+  // fetch triggers reuses the same argument; a volatile one never does. That
+  // gap is the signal — no timers, no heuristics. Warned once per instance,
+  // and in production too: the Studio canvas and the widget agent's render
+  // smoke check both run production builds, which is where this must surface.
+  const volatileKeyRef = useRef({ key: queryKey, changes: 0, warned: false });
+  const keyWatch = volatileKeyRef.current;
+  if (keyWatch.key === queryKey) {
+    keyWatch.changes = 0;
+  } else {
+    keyWatch.key = queryKey;
+    keyWatch.changes += 1;
+    if (keyWatch.changes >= VOLATILE_QUERY_RENDER_LIMIT && !keyWatch.warned) {
+      keyWatch.warned = true;
+      // eslint-disable-next-line no-console
+      console.warn(
+        `useDatastoreQuery(${JSON.stringify(table)}): the query argument changed on ` +
+          `${keyWatch.changes} consecutive renders, so this widget is re-fetching in a ` +
+          "loop. Something in the query is regenerated every render — usually a date " +
+          "or random value. Compute it once outside render (or with useMemo) and pass " +
+          "the stable value in.",
+      );
+    }
+  }
+
   useEffect(() => {
     doFetch();
     // eslint-disable-next-line react-hooks/exhaustive-deps

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@colixsystems/widget-sdk",
-  "version": "0.67.0",
+  "version": "0.67.1",
   "description": "Common widget interface for AppStudio. Implements WidgetManifest, WidgetContext, property schema, and helper hooks.",
   "type": "module",
   "main": "./dist/index.js",
@@ -48,7 +48,7 @@
   ],
   "scripts": {
     "build": "node scripts/build.js",
-    "test": "node --test src/__tests__/contract.test.js src/__tests__/hooks-users.test.js src/__tests__/hooks-groups.test.js src/__tests__/hooks-schema.test.js src/__tests__/hooks-assets-by-tag.test.js src/__tests__/hooks-filestore-upload.test.js src/__tests__/hooks-filestore-file.test.js src/__tests__/hooks-mutation.test.js src/__tests__/hooks-record-permissions.test.js src/__tests__/hooks-geolocation.test.js src/__tests__/hooks-subscription.test.js src/__tests__/linter-users-scope.test.js src/__tests__/linter-comments.test.js src/__tests__/linter-image-height.test.js src/__tests__/linter-platform.test.js src/__tests__/linter-react-import.test.js src/__tests__/lucide-icon-names.test.js src/__tests__/lucideIconName.test.js src/__tests__/manifest-actions.test.js src/__tests__/widget-translations.test.js src/__tests__/devserver.test.js src/__tests__/host-externals.test.js src/__tests__/datetimepicker.test.js src/__tests__/property-schema-resolve.test.js src/__tests__/theme-components-parity.test.js"
+    "test": "node --test src/__tests__/contract.test.js src/__tests__/hooks-users.test.js src/__tests__/hooks-groups.test.js src/__tests__/hooks-schema.test.js src/__tests__/hooks-assets-by-tag.test.js src/__tests__/hooks-filestore-upload.test.js src/__tests__/hooks-filestore-file.test.js src/__tests__/hooks-mutation.test.js src/__tests__/hooks-record-permissions.test.js src/__tests__/hooks-geolocation.test.js src/__tests__/hooks-subscription.test.js src/__tests__/hooks-volatile-query-key.test.js src/__tests__/linter-users-scope.test.js src/__tests__/linter-comments.test.js src/__tests__/linter-image-height.test.js src/__tests__/linter-platform.test.js src/__tests__/linter-react-import.test.js src/__tests__/lucide-icon-names.test.js src/__tests__/lucideIconName.test.js src/__tests__/manifest-actions.test.js src/__tests__/widget-translations.test.js src/__tests__/devserver.test.js src/__tests__/host-externals.test.js src/__tests__/datetimepicker.test.js src/__tests__/property-schema-resolve.test.js src/__tests__/theme-components-parity.test.js"
   },
   "engines": {
     "node": ">=18"