@object-ui/data-objectstack 4.0.10 → 4.0.12

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @object-ui/data-objectstack
 
+## 4.0.12
+
+### Patch Changes
+
+- @object-ui/types@4.0.12
+- @object-ui/core@4.0.12
+
+## 4.0.11
+
+### Patch Changes
+
+- @object-ui/types@4.0.11
+- @object-ui/core@4.0.11
+
 ## 4.0.10
 
 ### Patch Changes

package/dist/index.cjs

@@ -881,6 +881,26 @@ function objectFilterEntryToAST(entry) {
   if (!field || !op) return null;
   return [String(field), op, entry.value];
 }
+function translateFilterToAST(filter) {
+  if (filter === void 0 || filter === null) return void 0;
+  if (Array.isArray(filter)) {
+    if (filter.length === 0) return void 0;
+    const first = filter[0];
+    const isObjectForm = filter.length > 0 && typeof first === "object" && first !== null && !Array.isArray(first) && first.field !== void 0;
+    if (isObjectForm) {
+      const tuples = filter.map((entry) => objectFilterEntryToAST(entry)).filter((t) => t !== null);
+      if (tuples.length === 0) return void 0;
+      if (tuples.length === 1) return tuples[0];
+      return ["and", ...tuples];
+    }
+    return filter;
+  }
+  if (typeof filter === "object") {
+    if (Object.keys(filter).length === 0) return void 0;
+    return filter;
+  }
+  return void 0;
+}
 var discoveryCache = /* @__PURE__ */ new Map();
 async function getSharedDiscovery(baseUrl, fetcher) {
   const key = baseUrl || "<default>";
@@ -962,6 +982,11 @@ var ObjectStackAdapter = class {
     // multiple sibling components requesting the same dataset on first paint)
     // into a single network round trip.
     __publicField(this, "inflightFinds", /* @__PURE__ */ new Map());
+    // Resources that have responded 404 at least once (collection not installed
+    // on this backend). Subsequent find() calls short-circuit to an empty result
+    // so optional collections like sys_presence don't hammer the server with
+    // failing requests on every record open / panel render.
+    __publicField(this, "missingResources", /* @__PURE__ */ new Set());
     this.client = new import_client.ObjectStackClient({ ...config, logger: createQuietHttpLogger() });
     this.metadataCache = new MetadataCache(config.cache);
     this.autoReconnect = config.autoReconnect ?? true;
@@ -1099,18 +1124,29 @@ var ObjectStackAdapter = class {
    * Converts OData-style params to ObjectStack query options.
    */
   async find(resource, params) {
+    if (this.missingResources.has(resource)) {
+      return { data: [], total: 0 };
+    }
     const key = `${resource}::${stableStringify(params)}`;
     const existing = this.inflightFinds.get(key);
     if (existing) return existing;
     const promise = (async () => {
       await this.connect();
       if (params?.$expand && params.$expand.length > 0) {
-        const result2 = await this.rawFindWithPopulate(resource, params);
-        return this.normalizeQueryResult(result2, params);
+        const result = await this.rawFindWithPopulate(resource, params);
+        return this.normalizeQueryResult(result, params);
       }
       const queryOptions = this.convertQueryParams(params);
-      const result = await this.client.data.find(resource, queryOptions);
-      return this.normalizeQueryResult(result, params);
+      try {
+        const result = await this.client.data.find(resource, queryOptions);
+        return this.normalizeQueryResult(result, params);
+      } catch (err) {
+        if (is404Error(err)) {
+          this.missingResources.add(resource);
+          return { data: [], total: 0 };
+        }
+        throw err;
+      }
     })();
     this.inflightFinds.set(key, promise);
     const cleanup = () => {
@@ -1371,9 +1407,9 @@ var ObjectStackAdapter = class {
       }
     }
     if (params.$filter !== void 0 && params.$filter !== null) {
-      const isEmpty = Array.isArray(params.$filter) ? params.$filter.length === 0 : typeof params.$filter === "object" && Object.keys(params.$filter).length === 0;
-      if (!isEmpty) {
-        queryParams.set("filter", JSON.stringify(params.$filter));
+      const translated = translateFilterToAST(params.$filter);
+      if (translated !== void 0) {
+        queryParams.set("filter", JSON.stringify(translated));
       }
     }
     const baseUrl = this.baseUrl.replace(/\/$/, "");
@@ -1640,7 +1676,10 @@ var ObjectStackAdapter = class {
     await this.connect();
     let name = String(spec?.name || "").trim();
     if (!name) {
-      const base = String(spec?.label || objectName || "view").toLowerCase().replace(/[^a-z0-9_]+/g, "_").replace(/^_+|_+$/g, "").slice(0, 40) || "view";
+      let base = String(spec?.label || objectName || "view").toLowerCase().replace(/[^a-z0-9_]+/g, "_").replace(/^_+|_+$/g, "").slice(0, 40);
+      if (!base || /^[0-9]/.test(base)) {
+        base = base ? `view_${base}` : "view";
+      }
       const suffix = Date.now().toString(36);
       name = `${base}_${suffix}`;
     }

package/dist/index.d.cts

@@ -734,6 +734,7 @@ declare class ObjectStackAdapter<T = unknown> implements DataSource<T> {
     private token?;
     private fetchImpl;
     private inflightFinds;
+    private missingResources;
     constructor(config: {
         baseUrl: string;
         token?: string;

package/dist/index.d.ts

@@ -734,6 +734,7 @@ declare class ObjectStackAdapter<T = unknown> implements DataSource<T> {
     private token?;
     private fetchImpl;
     private inflightFinds;
+    private missingResources;
     constructor(config: {
         baseUrl: string;
         token?: string;

package/dist/index.js

@@ -838,6 +838,26 @@ function objectFilterEntryToAST(entry) {
   if (!field || !op) return null;
   return [String(field), op, entry.value];
 }
+function translateFilterToAST(filter) {
+  if (filter === void 0 || filter === null) return void 0;
+  if (Array.isArray(filter)) {
+    if (filter.length === 0) return void 0;
+    const first = filter[0];
+    const isObjectForm = filter.length > 0 && typeof first === "object" && first !== null && !Array.isArray(first) && first.field !== void 0;
+    if (isObjectForm) {
+      const tuples = filter.map((entry) => objectFilterEntryToAST(entry)).filter((t) => t !== null);
+      if (tuples.length === 0) return void 0;
+      if (tuples.length === 1) return tuples[0];
+      return ["and", ...tuples];
+    }
+    return filter;
+  }
+  if (typeof filter === "object") {
+    if (Object.keys(filter).length === 0) return void 0;
+    return filter;
+  }
+  return void 0;
+}
 var discoveryCache = /* @__PURE__ */ new Map();
 async function getSharedDiscovery(baseUrl, fetcher) {
   const key = baseUrl || "<default>";
@@ -919,6 +939,11 @@ var ObjectStackAdapter = class {
     // multiple sibling components requesting the same dataset on first paint)
     // into a single network round trip.
     __publicField(this, "inflightFinds", /* @__PURE__ */ new Map());
+    // Resources that have responded 404 at least once (collection not installed
+    // on this backend). Subsequent find() calls short-circuit to an empty result
+    // so optional collections like sys_presence don't hammer the server with
+    // failing requests on every record open / panel render.
+    __publicField(this, "missingResources", /* @__PURE__ */ new Set());
     this.client = new ObjectStackClient({ ...config, logger: createQuietHttpLogger() });
     this.metadataCache = new MetadataCache(config.cache);
     this.autoReconnect = config.autoReconnect ?? true;
@@ -1056,18 +1081,29 @@ var ObjectStackAdapter = class {
    * Converts OData-style params to ObjectStack query options.
    */
   async find(resource, params) {
+    if (this.missingResources.has(resource)) {
+      return { data: [], total: 0 };
+    }
     const key = `${resource}::${stableStringify(params)}`;
     const existing = this.inflightFinds.get(key);
     if (existing) return existing;
     const promise = (async () => {
       await this.connect();
       if (params?.$expand && params.$expand.length > 0) {
-        const result2 = await this.rawFindWithPopulate(resource, params);
-        return this.normalizeQueryResult(result2, params);
+        const result = await this.rawFindWithPopulate(resource, params);
+        return this.normalizeQueryResult(result, params);
       }
       const queryOptions = this.convertQueryParams(params);
-      const result = await this.client.data.find(resource, queryOptions);
-      return this.normalizeQueryResult(result, params);
+      try {
+        const result = await this.client.data.find(resource, queryOptions);
+        return this.normalizeQueryResult(result, params);
+      } catch (err) {
+        if (is404Error(err)) {
+          this.missingResources.add(resource);
+          return { data: [], total: 0 };
+        }
+        throw err;
+      }
     })();
     this.inflightFinds.set(key, promise);
     const cleanup = () => {
@@ -1328,9 +1364,9 @@ var ObjectStackAdapter = class {
       }
     }
     if (params.$filter !== void 0 && params.$filter !== null) {
-      const isEmpty = Array.isArray(params.$filter) ? params.$filter.length === 0 : typeof params.$filter === "object" && Object.keys(params.$filter).length === 0;
-      if (!isEmpty) {
-        queryParams.set("filter", JSON.stringify(params.$filter));
+      const translated = translateFilterToAST(params.$filter);
+      if (translated !== void 0) {
+        queryParams.set("filter", JSON.stringify(translated));
       }
     }
     const baseUrl = this.baseUrl.replace(/\/$/, "");
@@ -1597,7 +1633,10 @@ var ObjectStackAdapter = class {
     await this.connect();
     let name = String(spec?.name || "").trim();
     if (!name) {
-      const base = String(spec?.label || objectName || "view").toLowerCase().replace(/[^a-z0-9_]+/g, "_").replace(/^_+|_+$/g, "").slice(0, 40) || "view";
+      let base = String(spec?.label || objectName || "view").toLowerCase().replace(/[^a-z0-9_]+/g, "_").replace(/^_+|_+$/g, "").slice(0, 40);
+      if (!base || /^[0-9]/.test(base)) {
+        base = base ? `view_${base}` : "view";
+      }
       const suffix = Date.now().toString(36);
       name = `${base}_${suffix}`;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@object-ui/data-objectstack",
-  "version": "4.0.10",
+  "version": "4.0.12",
   "description": "ObjectStack Data Adapter for Object UI",
   "license": "MIT",
   "type": "module",
@@ -23,8 +23,8 @@
   ],
   "dependencies": {
     "@objectstack/client": "^4.0.5",
-    "@object-ui/core": "4.0.10",
-    "@object-ui/types": "4.0.10"
+    "@object-ui/core": "4.0.12",
+    "@object-ui/types": "4.0.12"
   },
   "devDependencies": {
     "tsup": "^8.5.1",

package/src/index.ts

@@ -78,6 +78,54 @@ function objectFilterEntryToAST(entry: any): [string, string, any] | null {
   return [String(field), op, entry.value];
 }
 
+/**
+ * Translate any of the filter shapes accepted by ObjectUI into the AST format
+ * understood by the ObjectStack server's `parseFilterAST()`.
+ *
+ * Accepted inputs:
+ *   - `[{ field, operator, value }, ...]` — ViewFilterRule[] from view configs
+ *   - `[field, op, value]`                — single AST tuple (passed through)
+ *   - `['and'|'or', ...children]`         — logical AST node (passed through)
+ *   - `[[...], [...]]`                    — legacy nested AST (passed through)
+ *   - `{ field: value }` / `{ field: { $op: value } }` — MongoDB-style object
+ *
+ * Returns `undefined` when the input is empty/unrecognized so callers can
+ * skip emitting `?filter=` entirely.
+ */
+function translateFilterToAST(filter: unknown): unknown | undefined {
+  if (filter === undefined || filter === null) return undefined;
+
+  if (Array.isArray(filter)) {
+    if (filter.length === 0) return undefined;
+
+    // Object form: [{ field, operator, value }, ...]
+    const first = filter[0];
+    const isObjectForm = filter.length > 0
+      && typeof first === 'object'
+      && first !== null
+      && !Array.isArray(first)
+      && (first as any).field !== undefined;
+    if (isObjectForm) {
+      const tuples = (filter as any[])
+        .map(entry => objectFilterEntryToAST(entry))
+        .filter((t): t is [string, string, any] => t !== null);
+      if (tuples.length === 0) return undefined;
+      if (tuples.length === 1) return tuples[0];
+      return ['and', ...tuples];
+    }
+
+    // Already AST — pass through.
+    return filter;
+  }
+
+  if (typeof filter === 'object') {
+    if (Object.keys(filter as Record<string, unknown>).length === 0) return undefined;
+    return filter;
+  }
+
+  return undefined;
+}
+
 // Module-level discovery cache. Multiple ObjectStackAdapter instances pointed
 // at the same baseUrl (e.g. ConditionalAuthWrapper's throwaway adapter +
 // AdapterProvider's main adapter) would otherwise each fire `/discovery`. By
@@ -275,6 +323,11 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
   // multiple sibling components requesting the same dataset on first paint)
   // into a single network round trip.
   private inflightFinds = new Map<string, Promise<QueryResult<T>>>();
+  // Resources that have responded 404 at least once (collection not installed
+  // on this backend). Subsequent find() calls short-circuit to an empty result
+  // so optional collections like sys_presence don't hammer the server with
+  // failing requests on every record open / panel render.
+  private missingResources = new Set<string>();
 
   constructor(config: {
     baseUrl: string;
@@ -470,6 +523,12 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
    * Converts OData-style params to ObjectStack query options.
    */
   async find(resource: string, params?: QueryParams): Promise<QueryResult<T>> {
+    // Short-circuit when this resource has previously responded 404 — the
+    // collection isn't installed on this backend. Callers (AppHeader,
+    // RecordDetailView, …) treat empty data as "feature unavailable".
+    if (this.missingResources.has(resource)) {
+      return { data: [], total: 0 } as QueryResult<T>;
+    }
     const key = `${resource}::${stableStringify(params)}`;
     const existing = this.inflightFinds.get(key);
     if (existing) return existing;
@@ -489,8 +548,17 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
       }
 
       const queryOptions = this.convertQueryParams(params);
-      const result: unknown = await this.client.data.find<T>(resource, queryOptions);
-      return this.normalizeQueryResult(result, params);
+      try {
+        const result: unknown = await this.client.data.find<T>(resource, queryOptions);
+        return this.normalizeQueryResult(result, params);
+      } catch (err) {
+        if (is404Error(err)) {
+          // Mark the resource so subsequent calls don't repeat the 404.
+          this.missingResources.add(resource);
+          return { data: [], total: 0 } as QueryResult<T>;
+        }
+        throw err;
+      }
     })();
 
     this.inflightFinds.set(key, promise);
@@ -824,13 +892,15 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
       }
     }
 
-    // Filter — drop empty arrays/objects so we don't send `?filter=%5B%5D`
+    // Filter — translate ViewFilterRule[] (`[{field, operator, value}]`)
+    // and other shapes into AST tuples the server understands. Without this,
+    // server-driven views (e.g. `at_risk_accounts`, `hot_leads`) ship raw
+    // `[{field,operator,value}]` arrays which `parseFilterAST` silently
+    // discards, returning every record instead of the filtered subset.
     if (params.$filter !== undefined && params.$filter !== null) {
-      const isEmpty = Array.isArray(params.$filter)
-        ? params.$filter.length === 0
-        : typeof params.$filter === 'object' && Object.keys(params.$filter).length === 0;
-      if (!isEmpty) {
-        queryParams.set('filter', JSON.stringify(params.$filter));
+      const translated = translateFilterToAST(params.$filter);
+      if (translated !== undefined) {
+        queryParams.set('filter', JSON.stringify(translated));
       }
     }
 
@@ -1188,11 +1258,17 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
     await this.connect();
     let name = String(spec?.name || '').trim();
     if (!name) {
-      const base = String(spec?.label || objectName || 'view')
+      let base = String(spec?.label || objectName || 'view')
         .toLowerCase()
         .replace(/[^a-z0-9_]+/g, '_')
         .replace(/^_+|_+$/g, '')
-        .slice(0, 40) || 'view';
+        .slice(0, 40);
+      // Spec requires snake_case starting with a letter or underscore.
+      // Labels like "表格 1" collapse to "1" after non-ascii stripping, so we
+      // need a fallback / prefix to keep the identifier valid.
+      if (!base || /^[0-9]/.test(base)) {
+        base = base ? `view_${base}` : 'view';
+      }
       const suffix = Date.now().toString(36);
       name = `${base}_${suffix}`;
     }