@objectstack/core 17.0.0-rc.6 → 17.0.0

package/dist/index.js

@@ -1385,6 +1385,7 @@ function createMemoryJob() {
 }
 
 // src/fallbacks/memory-i18n.ts
+import { normalizeSupportedLocales } from "@objectstack/spec/system";
 function deepMerge(target, source) {
   const result = { ...target };
   for (const key of Object.keys(source)) {
@@ -1418,6 +1419,7 @@ function createMemoryI18n() {
   const translations = /* @__PURE__ */ new Map();
   const authored = /* @__PURE__ */ new Map();
   let defaultLocale = "en";
+  let supportedLocales;
   function resolveKey(data, key) {
     const parts = key.split(".");
     let current = data;
@@ -1483,9 +1485,29 @@ function createMemoryI18n() {
         authored.set(locale, { ...data });
       }
     },
+    /**
+     * Report the locales this stack offers.
+     *
+     * [#7679] When the app declared `i18n.supportedLocales`, that declaration
+     * IS the answer — in declared order, and including a declared locale no
+     * bundle was ever loaded for (declared-but-unserved). Reporting the
+     * declaration rather than an intersection is what gives a client the
+     * signal that the locale it is being offered has nothing behind it yet;
+     * quietly dropping it would leave the gap invisible on both sides. It is
+     * also the only answer that does not depend on how much had loaded by the
+     * time this was called.
+     *
+     * With nothing declared, the loaded set — the behaviour every app that
+     * never opted in already has.
+     */
     getLocales() {
+      if (supportedLocales) return [...supportedLocales];
       return [.../* @__PURE__ */ new Set([...translations.keys(), ...authored.keys()])];
     },
+    /** @see II18nService.setSupportedLocales — [#7679] */
+    setSupportedLocales(locales) {
+      supportedLocales = normalizeSupportedLocales(locales);
+    },
     getDefaultLocale() {
       return defaultLocale;
     },
@@ -1495,14 +1517,42 @@ function createMemoryI18n() {
   };
 }
 
+// src/metadata-service-contract.ts
+import { pluralToSingular } from "@objectstack/spec/shared";
+var REGISTER_REFUSAL_CODE = "VALIDATION_ERROR";
+function canonicalMetadataServiceType(type) {
+  return pluralToSingular(type);
+}
+function registerRefusal(message) {
+  const err = new Error(message);
+  err.code = REGISTER_REFUSAL_CODE;
+  err.status = 400;
+  return err;
+}
+function assertMetadataRegisterContract(type, name, data) {
+  if (typeof data !== "object" || data === null || Array.isArray(data)) {
+    const shape = data === null ? "null" : Array.isArray(data) ? "an array" : `a ${typeof data}`;
+    throw registerRefusal(
+      `IMetadataService.register('${type}', '${name}'): data is ${shape}, not a metadata document. register() stores plain-object documents only \u2014 accepting a value the service cannot key was measured as accept-then-drop on document-keyed stores (#7378 row 3: refuse loudly, never coerce into storability). Wrap the value in a document object whose shape the '${type}' type's schema accepts, or store it under a type that declares one.`
+    );
+  }
+  const documentName = data.name;
+  if (documentName !== void 0 && documentName !== name) {
+    throw registerRefusal(
+      `IMetadataService.register('${type}', '${name}'): data.name is '${String(documentName)}', which disagrees with the name argument '${name}'. A disagreement is almost always an authoring bug, and resolving it silently in either direction can file the item under a key the caller never wrote (#7378 row 1: refuse loudly, locate the mismatch). Register under one name: pass the intended key as the argument and make data.name match it, or omit data.name.`
+    );
+  }
+}
+
 // src/fallbacks/memory-metadata.ts
 function createMemoryMetadata() {
   const store = /* @__PURE__ */ new Map();
   function getTypeMap(type) {
-    let map = store.get(type);
+    const canonical = canonicalMetadataServiceType(type);
+    let map = store.get(canonical);
     if (!map) {
       map = /* @__PURE__ */ new Map();
-      store.set(type, map);
+      store.set(canonical, map);
     }
     return map;
   }
@@ -1517,6 +1567,7 @@ function createMemoryMetadata() {
     },
     _serviceName: "metadata",
     async register(type, name, data) {
+      assertMetadataRegisterContract(type, name, data);
       getTypeMap(type).set(name, data);
     },
     // Mirror MetadataManager.registerInMemory (synchronous, no persistence).
@@ -1528,7 +1579,11 @@ function createMemoryMetadata() {
     // so `defineStack({ datasources })` entries silently never reached the
     // registry and were absent from GET /api/v1/datasources and
     // GET /api/v1/meta/datasource (ADR-0015 §18). This store is already
-    // in-memory only, so registerInMemory and register share an implementation.
+    // in-memory only, so registerInMemory and register share a store — but
+    // NOT the [#7378] refusals: the ruling names `register`, and this member
+    // is a boot-time seeding primitive for source-control-owned artefacts
+    // (see assertMetadataRegisterContract's header for the boundary). It does
+    // share the row-2 canonical type fold, via getTypeMap.
     registerInMemory(type, name, data) {
       getTypeMap(type).set(name, data);
     },
@@ -2524,11 +2579,29 @@ var TestRunner = class {
 };
 
 // src/qa/http-adapter.ts
+import { RestApiConfigSchema, CrudEndpointsConfigSchema } from "@objectstack/spec/api";
+var dataPathCache;
+function defaultDataPath() {
+  if (dataPathCache === void 0) {
+    const api = RestApiConfigSchema.parse({});
+    const crud = CrudEndpointsConfigSchema.parse({});
+    dataPathCache = `${api.apiPath ?? `${api.basePath}/${api.version}`}${crud.dataPrefix}`;
+  }
+  return dataPathCache;
+}
 var HttpTestAdapter = class {
   constructor(baseUrl, authToken) {
     this.baseUrl = baseUrl;
     this.authToken = authToken;
   }
+  /** `{baseUrl}{apiBasePath}{dataPrefix}/{object}` — the collection URL. */
+  collectionUrl(objectName) {
+    return `${this.baseUrl}${defaultDataPath()}/${encodeURIComponent(objectName)}`;
+  }
+  /** `{collection}/{id}` — the single-record URL. */
+  recordUrl(objectName, id) {
+    return `${this.collectionUrl(objectName)}/${encodeURIComponent(String(id))}`;
+  }
   async execute(action, _context) {
     const headers = {
       "Content-Type": "application/json"
@@ -2560,7 +2633,7 @@ var HttpTestAdapter = class {
     }
   }
   async createRecord(objectName, data, headers) {
-    const response = await fetch(`${this.baseUrl}/api/data/${objectName}`, {
+    const response = await fetch(this.collectionUrl(objectName), {
       method: "POST",
       headers,
       body: JSON.stringify(data)
@@ -2568,19 +2641,19 @@ var HttpTestAdapter = class {
     return this.handleResponse(response);
   }
   async updateRecord(objectName, data, headers) {
-    const id = data.id;
+    const { id, ...fields } = data;
     if (!id) throw new Error("Update record requires id in payload");
-    const response = await fetch(`${this.baseUrl}/api/data/${objectName}/${id}`, {
-      method: "PUT",
+    const response = await fetch(this.recordUrl(objectName, id), {
+      method: "PATCH",
       headers,
-      body: JSON.stringify(data)
+      body: JSON.stringify(fields)
     });
     return this.handleResponse(response);
   }
   async deleteRecord(objectName, data, headers) {
     const id = data.id;
     if (!id) throw new Error("Delete record requires id in payload");
-    const response = await fetch(`${this.baseUrl}/api/data/${objectName}/${id}`, {
+    const response = await fetch(this.recordUrl(objectName, id), {
       method: "DELETE",
       headers
     });
@@ -2589,14 +2662,14 @@ var HttpTestAdapter = class {
   async readRecord(objectName, data, headers) {
     const id = data.id;
     if (!id) throw new Error("Read record requires id in payload");
-    const response = await fetch(`${this.baseUrl}/api/data/${objectName}/${id}`, {
+    const response = await fetch(this.recordUrl(objectName, id), {
       method: "GET",
       headers
     });
     return this.handleResponse(response);
   }
   async queryRecords(objectName, data, headers) {
-    const response = await fetch(`${this.baseUrl}/api/data/${objectName}/query`, {
+    const response = await fetch(`${this.collectionUrl(objectName)}/query`, {
       method: "POST",
       headers,
       body: JSON.stringify(data)
@@ -4546,6 +4619,18 @@ function shouldDenyAnonymous(input) {
   return true;
 }
 
+// src/security/audience-binding-suggestion-status.ts
+var AUDIENCE_BINDING_SUGGESTION_STATUSES = {
+  pending: true,
+  confirmed: true,
+  dismissed: true
+};
+var AUDIENCE_BINDING_SUGGESTION_STATUS_VALUES = Object.keys(
+  AUDIENCE_BINDING_SUGGESTION_STATUSES
+);
+var isAudienceBindingSuggestionStatus = (value) => Object.prototype.hasOwnProperty.call(AUDIENCE_BINDING_SUGGESTION_STATUSES, value);
+var unknownAudienceBindingSuggestionStatusMessage = (value) => `Unknown status filter '${value}' \u2014 expected one of: ${AUDIENCE_BINDING_SUGGESTION_STATUS_VALUES.join(", ")}`;
+
 // src/security/operation-private-keys.ts
 var OPERATION_PRIVATE_KEY_PREFIX = "__";
 function withoutOperationPrivateKeys(exec) {
@@ -4799,6 +4884,27 @@ async function bulkWrite(rows, opts) {
   return results;
 }
 
+// src/utils/internal-write-response.ts
+function collectInternalWriteResponseFields(schema) {
+  const fields = schema?.fields;
+  if (!fields || typeof fields !== "object") return [];
+  const out = [];
+  for (const [name, def] of Object.entries(fields)) {
+    if (def && def.internal === true) out.push(name);
+  }
+  return out;
+}
+function omitInternalFieldsFromWriteResponse(schema, records) {
+  if (!records) return;
+  const internalFields = collectInternalWriteResponseFields(schema);
+  if (internalFields.length === 0) return;
+  const list = Array.isArray(records) ? records : [records];
+  for (const row of list) {
+    if (!row || typeof row !== "object") continue;
+    for (const field of internalFields) delete row[field];
+  }
+}
+
 // src/utils/migration-journal.ts
 import { createHash as createHash2, randomUUID } from "crypto";
 import {
@@ -5394,6 +5500,15 @@ function filterTokenContextFrom(execCtx, now) {
   };
 }
 
+// src/utils/record-not-found.ts
+function recordNotFoundError(object, id) {
+  const err = new Error(`Record ${id} not found in ${object}`);
+  err.code = "RECORD_NOT_FOUND";
+  err.status = 404;
+  err.object = object;
+  return err;
+}
+
 // src/health-monitor.ts
 var PluginHealthMonitor = class {
   constructor(logger) {
@@ -6376,6 +6491,8 @@ export {
   ANONYMOUS_DENY_MESSAGE,
   ANONYMOUS_DENY_STATUS,
   API_KEY_PREFIX,
+  AUDIENCE_BINDING_SUGGESTION_STATUSES,
+  AUDIENCE_BINDING_SUGGESTION_STATUS_VALUES,
   CORE_FALLBACK_FACTORIES,
   DependencyResolver,
   ENTRY_EXECUTION_CONTEXT_FIELDS,
@@ -6409,11 +6526,14 @@ export {
   assembleExecutionContext,
   assembleExecutionContextOrGuest,
   assertInitServiceRequirements,
+  assertMetadataRegisterContract,
   bucketKeyToCalendarRange,
   buildPermissionsFromGrants,
   bulkWrite,
   calendarPartsInTz,
   calendarPartsInTzOrUtc,
+  canonicalMetadataServiceType,
+  collectInternalWriteResponseFields,
   counterSignPayload,
   createLogger,
   createMemoryCache,
@@ -6438,6 +6558,7 @@ export {
   getMemoryUsage,
   hashApiKey,
   hashMigrationPlan,
+  isAudienceBindingSuggestionStatus,
   isAuthGateAllowlisted,
   isExpired,
   isGrantActive,
@@ -6445,12 +6566,14 @@ export {
   isNode,
   nextUtcCalendarDay,
   normalizeAuthGate,
+  omitInternalFieldsFromWriteResponse,
   parseScopes,
   parseSignature,
   planChunks,
   postureVisibleRows,
   readAuthoredTranslationLayer,
   readRunJournal,
+  recordNotFoundError,
   resolveApiKeyPrincipal,
   resolveAuthzContext,
   resolveFilterToken,
@@ -6464,6 +6587,7 @@ export {
   safeExit,
   shouldDenyAnonymous,
   signPayload,
+  unknownAudienceBindingSuggestionStatusMessage,
   utcInstantMs,
   validateInitServiceContract,
   verifyPayload,