@object-ui/core 17.6.0 → 18.0.0

package/dist/adapters/ApiDataSource.js

@@ -5,8 +5,11 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * A DataSource adapter for the `provider: 'api'` ViewData mode.
- * Makes raw HTTP requests using the HttpRequest configs from ViewData.
+ * A DataSource adapter over raw HTTP, built in code from an
+ * `ApiDataSourceConfig` (e.g. the console's approvals inbox). It is not a
+ * `ViewData` mode: the view-level `provider: 'api'` is retired in
+ * `@objectstack/spec` (objectstack#23082), and `resolveDataSource` builds no
+ * `ApiDataSource` (objectui#12325).
  */
 import { emulateBatchTransaction } from './batchTransaction.js';
 // ---------------------------------------------------------------------------
@@ -91,8 +94,9 @@ function mergeHeaders(base, override) {
 /**
  * ApiDataSource — a DataSource adapter for raw HTTP APIs.
  *
- * Used when `ViewData.provider === 'api'`. The read and write HttpRequest
- * configs define the endpoints; all CRUD methods map onto HTTP verbs.
+ * Constructed in code — never resolved from a `ViewData` block (see the file
+ * header). The read and write HttpRequest configs define the endpoints; all
+ * CRUD methods map onto HTTP verbs.
  *
  * Read operations use the `read` config, write operations use the `write` config.
  * Both fall back to each other when one is not provided.
@@ -144,7 +148,7 @@ export class ApiDataSource {
     async request(base, options = {}) {
         if (!base) {
             throw new Error('ApiDataSource: No HTTP configuration provided for this operation. ' +
-                'Ensure the ViewData has read/write configs.');
+                'Ensure the ApiDataSourceConfig has read/write configs.');
         }
         const method = options.method ?? base.method ?? 'GET';
         // Merge query params: base.params + extra queryParams
@@ -213,8 +217,16 @@ export class ApiDataSource {
             });
             return raw ?? null;
         }
-        catch {
-            return null;
+        catch (err) {
+            // `null` is "not found", and only a 404 says that. Every other failure
+            // (a refusal, a 5xx, a transport error) rejects, as `ObjectStackAdapter`
+            // does, so a record page can tell "no access" and "could not load" from
+            // "not found" (objectui#11902, objectui#12032). The status is read from
+            // the message `request` above composes ("ApiDataSource: HTTP 404 …").
+            if (err instanceof Error && /^ApiDataSource: HTTP 404\b/.test(err.message)) {
+                return null;
+            }
+            throw err;
         }
     }
     async create(_resource, data) {

package/dist/adapters/ValueDataSource.d.ts

@@ -63,6 +63,11 @@ export declare class ValueDataSource<T = any> implements DataSource<T> {
     onMutation(callback: (event: DataSourceMutationEvent<T>) => void): () => void;
     /** Get the current number of items */
     get count(): number;
-    /** Get a snapshot of all items (cloned) */
+    /**
+     * Get a snapshot of all items (cloned).
+     *
+     * Same clone as the constructor and for the same reason — see the note
+     * there: `structuredClone`, never a JSON round-trip (objectui#9175).
+     */
     getAll(): T[];
 }