@object-ui/data-objectstack 4.0.10 → 4.0.11

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # @object-ui/data-objectstack
 
+## 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

@@ -962,6 +962,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 +1104,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 = () => {
@@ -1640,7 +1656,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

@@ -919,6 +919,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 +1061,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 = () => {
@@ -1597,7 +1613,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.11",
   "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.11",
+    "@object-ui/types": "4.0.11"
   },
   "devDependencies": {
     "tsup": "^8.5.1",

package/src/index.ts

@@ -275,6 +275,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 +475,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 +500,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);
@@ -1188,11 +1208,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}`;
     }