@objectstack/metadata 17.2.0 → 17.4.0

package/dist/view-container.cjs

@@ -0,0 +1,37 @@
+"use strict";
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+
+// src/view-container.ts
+var view_container_exports = {};
+__export(view_container_exports, {
+  deriveViewContainerObject: () => deriveViewContainerObject
+});
+module.exports = __toCommonJS(view_container_exports);
+function deriveViewContainerObject(container) {
+  if (!container || typeof container !== "object") return void 0;
+  const c = container;
+  const own = typeof c.object === "string" && c.object ? c.object : void 0;
+  const byName = typeof c.name === "string" && c.name ? c.name : void 0;
+  return own ?? c?.list?.data?.object ?? c?.form?.data?.object ?? byName;
+}
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  deriveViewContainerObject
+});
+//# sourceMappingURL=view-container.cjs.map

package/dist/view-container.cjs.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/view-container.ts"],"mappings":";;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AA4EO,SAAS,0BAA0B,WAAwC;AAChF,MAAI,CAAC,aAAa,OAAO,cAAc,SAAU,QAAO;AACxD,QAAM,IAAI;AACV,QAAM,MAAM,OAAO,EAAE,WAAW,YAAY,EAAE,SAAS,EAAE,SAAS;AAClE,QAAM,SAAS,OAAO,EAAE,SAAS,YAAY,EAAE,OAAO,EAAE,OAAO;AAC/D,SAAO,OAAO,GAAG,MAAM,MAAM,UAAU,GAAG,MAAM,MAAM,UAAU;AAClE;","names":[]}

package/dist/view-container.d.cts

@@ -0,0 +1,76 @@
+/**
+ * `@objectstack/metadata/view-container` — the one spelling of "which object
+ * does an aggregated `defineView` container bind to", as a LEAF entry point.
+ *
+ * ## Why this subpath exists
+ *
+ * This is the `/errors` pattern taken a second time, for the same reason
+ * (`src/errors.ts` carries the first telling). `@objectstack/metadata`'s ROOT
+ * entry pulls `MetadataPlugin` → `NodeMetadataManager` → `chokidar`, plus
+ * `glob` and `js-yaml`; a cross-package consumer that wants a six-line pure
+ * function should not have to load any of that.
+ *
+ * The consumer that forced it is `packages/objectql`'s ADR-0076 lean entry.
+ * `@objectstack/objectql/core` re-exports `engine.ts`, whose boot-loop
+ * registrar has to mint the same registry key as this package's registrars
+ * (#14399), so it imports {@link deriveViewContainerObject} rather than
+ * hand-copying the chain a fifth time — and reaching it through the root entry
+ * made that lean closure load the manager and the filesystem machinery for a
+ * function that touches neither. Measured on the built artifacts (#14680):
+ * through the root entry that call site loaded SIX extra modules into
+ * `@objectstack/objectql/core`'s module-init closure —
+ * `packages/metadata/dist/index.js`, `js-yaml`, `glob`, `chokidar` (two
+ * files) and `readdirp`, 499,162 B — for ~22 ms of extra init. Through this
+ * entry it loads one 469-byte module and nothing else.
+ *
+ * ## Why the function LIVES here rather than being re-exported from
+ * `view-container-expansion.ts`
+ *
+ * A re-export shim was written first and measured, because `/errors`' header
+ * states the requirement the shim has to meet: "this entry re-exports one leaf
+ * module and nothing else, so the cross-package edge stays a leaf edge".
+ * `view-container-expansion.ts` is not a leaf — its other export,
+ * `expandRuntimeViewContainer`, needs `@objectstack/spec` and
+ * `@objectstack/spec/shared` — and esbuild tree-shakes the unused FUNCTION but
+ * keeps both `import` statements, since it cannot prove an external package is
+ * side-effect-free. Measured: the shim's own closure was 84
+ * modules / 3,035 KiB (all of `@objectstack/spec` and `zod`, reached through an
+ * import statement for a function that had been shaken out); this module's is
+ * 1 module / 469 B.
+ *
+ * So the derivation lives in this module, which imports nothing at all, and
+ * `view-container-expansion.ts` imports it from here and re-exports it — every
+ * existing importer (`index.ts`'s root export, `plugin.ts`) keeps its spelling.
+ *
+ * ## Scope of the promise
+ *
+ * Only {@link deriveViewContainerObject} is exported here. Its former module
+ * sibling `expandRuntimeViewContainer` is deliberately left off: it is internal
+ * to this package (`metadata-manager.ts` is its only caller, and the root entry
+ * does not export it either), and an exported symbol nobody imports is a promise
+ * made for nothing — Prime Directive #10 pointed at our own API surface, the
+ * same call `src/errors.ts` made about `isSchemaAlreadyExistsError`.
+ *
+ * The symbol stays on the ROOT entry as well: this subpath is an additional
+ * door, not a relocation, and the root export is a published promise with
+ * possible out-of-repo consumers.
+ */
+/**
+ * Which object an aggregated view container binds to.
+ *
+ * The container's OWN top-level `object` field — `ViewSchema.object`,
+ * documented there as "how a stack-level `views: [...]` entry says which object
+ * its views belong to; read by `getViewsByObject()` / `GET /meta/view?object=`"
+ * — is the authorial, explicit signal and is consulted FIRST (#13407). The
+ * three-deep fallback below it is kept unchanged for every container written
+ * before that field was read here: `list.data.object`, then `form.data.object`,
+ * then the row's own `name` — which is the bound object only by convention, and
+ * is why a container that set the top-level field but not `list.data.object`
+ * used to bind under the wrong key or not at all.
+ *
+ * Returns `undefined` when no binding can be derived; every caller treats that
+ * as "no expansion" rather than an error.
+ */
+declare function deriveViewContainerObject(container: unknown): string | undefined;
+
+export { deriveViewContainerObject };

package/dist/view-container.d.ts

@@ -0,0 +1,76 @@
+/**
+ * `@objectstack/metadata/view-container` — the one spelling of "which object
+ * does an aggregated `defineView` container bind to", as a LEAF entry point.
+ *
+ * ## Why this subpath exists
+ *
+ * This is the `/errors` pattern taken a second time, for the same reason
+ * (`src/errors.ts` carries the first telling). `@objectstack/metadata`'s ROOT
+ * entry pulls `MetadataPlugin` → `NodeMetadataManager` → `chokidar`, plus
+ * `glob` and `js-yaml`; a cross-package consumer that wants a six-line pure
+ * function should not have to load any of that.
+ *
+ * The consumer that forced it is `packages/objectql`'s ADR-0076 lean entry.
+ * `@objectstack/objectql/core` re-exports `engine.ts`, whose boot-loop
+ * registrar has to mint the same registry key as this package's registrars
+ * (#14399), so it imports {@link deriveViewContainerObject} rather than
+ * hand-copying the chain a fifth time — and reaching it through the root entry
+ * made that lean closure load the manager and the filesystem machinery for a
+ * function that touches neither. Measured on the built artifacts (#14680):
+ * through the root entry that call site loaded SIX extra modules into
+ * `@objectstack/objectql/core`'s module-init closure —
+ * `packages/metadata/dist/index.js`, `js-yaml`, `glob`, `chokidar` (two
+ * files) and `readdirp`, 499,162 B — for ~22 ms of extra init. Through this
+ * entry it loads one 469-byte module and nothing else.
+ *
+ * ## Why the function LIVES here rather than being re-exported from
+ * `view-container-expansion.ts`
+ *
+ * A re-export shim was written first and measured, because `/errors`' header
+ * states the requirement the shim has to meet: "this entry re-exports one leaf
+ * module and nothing else, so the cross-package edge stays a leaf edge".
+ * `view-container-expansion.ts` is not a leaf — its other export,
+ * `expandRuntimeViewContainer`, needs `@objectstack/spec` and
+ * `@objectstack/spec/shared` — and esbuild tree-shakes the unused FUNCTION but
+ * keeps both `import` statements, since it cannot prove an external package is
+ * side-effect-free. Measured: the shim's own closure was 84
+ * modules / 3,035 KiB (all of `@objectstack/spec` and `zod`, reached through an
+ * import statement for a function that had been shaken out); this module's is
+ * 1 module / 469 B.
+ *
+ * So the derivation lives in this module, which imports nothing at all, and
+ * `view-container-expansion.ts` imports it from here and re-exports it — every
+ * existing importer (`index.ts`'s root export, `plugin.ts`) keeps its spelling.
+ *
+ * ## Scope of the promise
+ *
+ * Only {@link deriveViewContainerObject} is exported here. Its former module
+ * sibling `expandRuntimeViewContainer` is deliberately left off: it is internal
+ * to this package (`metadata-manager.ts` is its only caller, and the root entry
+ * does not export it either), and an exported symbol nobody imports is a promise
+ * made for nothing — Prime Directive #10 pointed at our own API surface, the
+ * same call `src/errors.ts` made about `isSchemaAlreadyExistsError`.
+ *
+ * The symbol stays on the ROOT entry as well: this subpath is an additional
+ * door, not a relocation, and the root export is a published promise with
+ * possible out-of-repo consumers.
+ */
+/**
+ * Which object an aggregated view container binds to.
+ *
+ * The container's OWN top-level `object` field — `ViewSchema.object`,
+ * documented there as "how a stack-level `views: [...]` entry says which object
+ * its views belong to; read by `getViewsByObject()` / `GET /meta/view?object=`"
+ * — is the authorial, explicit signal and is consulted FIRST (#13407). The
+ * three-deep fallback below it is kept unchanged for every container written
+ * before that field was read here: `list.data.object`, then `form.data.object`,
+ * then the row's own `name` — which is the bound object only by convention, and
+ * is why a container that set the top-level field but not `list.data.object`
+ * used to bind under the wrong key or not at all.
+ *
+ * Returns `undefined` when no binding can be derived; every caller treats that
+ * as "no expansion" rather than an error.
+ */
+declare function deriveViewContainerObject(container: unknown): string | undefined;
+
+export { deriveViewContainerObject };

package/dist/view-container.js

@@ -0,0 +1,12 @@
+// src/view-container.ts
+function deriveViewContainerObject(container) {
+  if (!container || typeof container !== "object") return void 0;
+  const c = container;
+  const own = typeof c.object === "string" && c.object ? c.object : void 0;
+  const byName = typeof c.name === "string" && c.name ? c.name : void 0;
+  return own ?? c?.list?.data?.object ?? c?.form?.data?.object ?? byName;
+}
+export {
+  deriveViewContainerObject
+};
+//# sourceMappingURL=view-container.js.map

package/dist/view-container.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/view-container.ts"],"mappings":";AA4EO,SAAS,0BAA0B,WAAwC;AAChF,MAAI,CAAC,aAAa,OAAO,cAAc,SAAU,QAAO;AACxD,QAAM,IAAI;AACV,QAAM,MAAM,OAAO,EAAE,WAAW,YAAY,EAAE,SAAS,EAAE,SAAS;AAClE,QAAM,SAAS,OAAO,EAAE,SAAS,YAAY,EAAE,OAAO,EAAE,OAAO;AAC/D,SAAO,OAAO,GAAG,MAAM,MAAM,UAAU,GAAG,MAAM,MAAM,UAAU;AAClE;","names":[]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@objectstack/metadata",
-  "version": "17.2.0",
+  "version": "17.4.0",
   "license": "Apache-2.0",
   "description": "Metadata loading, saving, and persistence for ObjectStack",
   "type": "module",
@@ -8,24 +8,54 @@
   "types": "dist/index.d.ts",
   "exports": {
     ".": {
-      "types": "./dist/index.d.ts",
-      "import": "./dist/index.js",
-      "require": "./dist/index.cjs"
+      "import": {
+        "types": "./dist/index.d.ts",
+        "default": "./dist/index.js"
+      },
+      "require": {
+        "types": "./dist/index.d.cts",
+        "default": "./dist/index.cjs"
+      }
     },
     "./node": {
-      "types": "./dist/node.d.ts",
-      "import": "./dist/node.js",
-      "require": "./dist/node.cjs"
+      "import": {
+        "types": "./dist/node.d.ts",
+        "default": "./dist/node.js"
+      },
+      "require": {
+        "types": "./dist/node.d.cts",
+        "default": "./dist/node.cjs"
+      }
     },
     "./migrations": {
-      "types": "./dist/migrations/index.d.ts",
-      "import": "./dist/migrations/index.js",
-      "require": "./dist/migrations/index.cjs"
+      "import": {
+        "types": "./dist/migrations/index.d.ts",
+        "default": "./dist/migrations/index.js"
+      },
+      "require": {
+        "types": "./dist/migrations/index.d.cts",
+        "default": "./dist/migrations/index.cjs"
+      }
     },
     "./errors": {
-      "types": "./dist/errors.d.ts",
-      "import": "./dist/errors.js",
-      "require": "./dist/errors.cjs"
+      "import": {
+        "types": "./dist/errors.d.ts",
+        "default": "./dist/errors.js"
+      },
+      "require": {
+        "types": "./dist/errors.d.cts",
+        "default": "./dist/errors.cjs"
+      }
+    },
+    "./view-container": {
+      "import": {
+        "types": "./dist/view-container.d.ts",
+        "default": "./dist/view-container.js"
+      },
+      "require": {
+        "types": "./dist/view-container.d.cts",
+        "default": "./dist/view-container.cjs"
+      }
     }
   },
   "files": [
@@ -45,19 +75,19 @@
     "glob": "^13.0.6",
     "js-yaml": "^5.2.3",
     "zod": "^4.4.3",
-    "@objectstack/core": "17.2.0",
-    "@objectstack/metadata-core": "17.2.0",
-    "@objectstack/metadata-fs": "17.2.0",
-    "@objectstack/platform-objects": "17.2.0",
-    "@objectstack/spec": "17.2.0",
-    "@objectstack/types": "17.2.0"
+    "@objectstack/core": "17.4.0",
+    "@objectstack/metadata-core": "17.4.0",
+    "@objectstack/metadata-fs": "17.4.0",
+    "@objectstack/platform-objects": "17.4.0",
+    "@objectstack/spec": "17.4.0",
+    "@objectstack/types": "17.4.0"
   },
   "devDependencies": {
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^26.2.0",
     "typescript": "^6.0.3",
     "vitest": "^4.1.10",
-    "@objectstack/driver-sqlite-wasm": "17.2.0"
+    "@objectstack/driver-sqlite-wasm": "17.4.0"
   },
   "author": "ObjectStack",
   "repository": {
@@ -74,9 +104,10 @@
     "node": ">=22.0.0"
   },
   "scripts": {
-    "build": "tsup",
+    "build": "tsup && node ../../scripts/check-dts-emitted.mjs",
     "dev": "tsc --watch",
     "clean": "rm -rf dist",
+    "typecheck": "tsc --noEmit",
     "test": "vitest run",
     "test:watch": "vitest",
     "test:coverage": "vitest run --coverage"