@jini-ai/core 0.4.3 → 0.4.5

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @jini-ai/core
 
+## 0.4.5 — 2026-10-13
+
+- `FetchQueryCache`: a `replace()` that lands after a read's transport resolves but before publication now wins; arbitration and publication share one continuation.
+
+## 0.4.4 — 2026-10-12
+
+- New `./records` subpath exports `isRecord` and `isNonArrayRecord` without loading the core barrel; the root and `registration-kit` keep re-exporting both.
+
 ## 0.4.2 — pending release
 
 - Add source-backed `./plugin-directory` publisher identity and live metadata lookup. Warn and omit invalid advisory keywords; scope legacy `unknown/<name>` IDs to each plugin family, with optional family selection for ambiguous lookups.

package/dist/fetch/cache.js

@@ -91,20 +91,26 @@ export class FetchQueryCache {
             this.publish(entry, { ...entry.state, isFetching: false });
             return this.runOnline({ run });
         }).then(value => {
-            const data = replacement === entry.replacement ? value : entry.state.data;
             // Undefined is never successful query data, including an initial or background read.
-            if (data === undefined)
+            // Thrown here so the rejection path below handles it (a replacement still supersedes it).
+            if (value === undefined)
                 throw new Error(`${JSON.stringify(entry.key)} data is undefined`);
-            return data;
-        }).then(data => {
+            return value;
+        }).then(value => {
             // Only the current request may publish or clear invalidation. An older transport
             // need not support abort: its callers follow the replacement request instead.
             if (entry.pending !== pending)
                 return (entry.pending ?? entry.state.data);
+            // Arbitration and publication share this continuation: a microtask between them would
+            // let an older result overwrite a replacement accepted after arbitration.
+            if (replacement !== entry.replacement) {
+                this.publish(entry, { ...entry.state, isFetching: false });
+                return entry.state.data;
+            }
             entry.updatedAt = this.clock.nowMs();
             entry.invalidated = false;
-            this.publish(entry, { data, error: null, status: 'success', isFetching: false });
-            return data;
+            this.publish(entry, { data: value, error: null, status: 'success', isFetching: false });
+            return value;
         }, error => {
             if (entry.pending !== pending)
                 return (entry.pending ?? entry.state.data);

package/dist/records.d.ts

@@ -0,0 +1,24 @@
+/** @file Pure object-shape predicates shared by tool input readers and render-time decoders. */
+/** Accept any non-null object, including arrays; never sanitizes or reads properties.
+ * @complexity O(1) time and space.
+ * @example isRecord({ value: [] }); // true
+ */
+export declare function isRecord(required: {
+    value: unknown;
+}, _optional?: Record<string, never>): required is {
+    value: Record<string, unknown>;
+};
+/**
+ * Accepts non-null, non-array objects, including class instances and null-prototype records.
+ * This is a shape check, not plain-object validation or sanitization. Keep `isRecord`'s
+ * broader contract: existing tool readers deliberately continue to accept arrays.
+ * @param required.value - The untrusted value whose object shape is checked.
+ * @complexity O(1) time and space.
+ * @example isNonArrayRecord({ value: [] }); // false
+ */
+export declare function isNonArrayRecord(required: {
+    value: unknown;
+}, _optional?: Record<string, never>): required is {
+    value: Record<string, unknown>;
+};
+//# sourceMappingURL=records.d.ts.map

package/dist/records.js

@@ -0,0 +1,21 @@
+/** @file Pure object-shape predicates shared by tool input readers and render-time decoders. */
+/** Accept any non-null object, including arrays; never sanitizes or reads properties.
+ * @complexity O(1) time and space.
+ * @example isRecord({ value: [] }); // true
+ */
+export function isRecord(required, _optional = {}) {
+    const { value } = required;
+    return typeof value === "object" && value !== null;
+}
+/**
+ * Accepts non-null, non-array objects, including class instances and null-prototype records.
+ * This is a shape check, not plain-object validation or sanitization. Keep `isRecord`'s
+ * broader contract: existing tool readers deliberately continue to accept arrays.
+ * @param required.value - The untrusted value whose object shape is checked.
+ * @complexity O(1) time and space.
+ * @example isNonArrayRecord({ value: [] }); // false
+ */
+export function isNonArrayRecord(required, _optional = {}) {
+    return isRecord(required) && !Array.isArray(required.value);
+}
+//# sourceMappingURL=records.js.map

package/dist/registration-kit.d.ts

@@ -1,3 +1,4 @@
+export { isRecord, isNonArrayRecord } from "./records.js";
 /**
  * @file The cross-domain half of the agent-tool wiring split: everything a domain's own
  * `tool-registrations.ts` needs that is NOT specific to that domain.
@@ -134,22 +135,6 @@ export declare function indexCatalogById<T extends {
 }>(required: {
     catalog: readonly T[];
 }, _optional?: Record<string, never>): ReadonlyMap<string, T>;
-export declare function isRecord(required: {
-    value: unknown;
-}, _optional?: Record<string, never>): required is {
-    value: Record<string, unknown>;
-};
-/**
- * Accepts non-null, non-array objects, including class instances and null-prototype records.
- * This is a shape check, not plain-object validation or sanitization. Keep `isRecord`'s
- * broader contract: existing tool readers deliberately continue to accept arrays.
- * @param required.value - The untrusted value whose object shape is checked.
- */
-export declare function isNonArrayRecord(required: {
-    value: unknown;
-}, _optional?: Record<string, never>): required is {
-    value: Record<string, unknown>;
-};
 /** Narrows `ctx.input` to a record, refusing anything else. The first line of most handlers. */
 export declare function requireInputRecord(required: {
     input: unknown;

package/dist/registration-kit.js

@@ -1,3 +1,5 @@
+import { isRecord } from "./records.js";
+export { isRecord, isNonArrayRecord } from "./records.js";
 /**
  * @file The cross-domain half of the agent-tool wiring split: everything a domain's own
  * `tool-registrations.ts` needs that is NOT specific to that domain.
@@ -100,19 +102,6 @@ export function indexCatalogById(required, _optional = {}) {
 // ---------------------------------------------------------------------------
 // Input readers — the vocabulary every handler uses to read `ctx.input`.
 // ---------------------------------------------------------------------------
-export function isRecord(required, _optional = {}) {
-    const { value } = required;
-    return typeof value === "object" && value !== null;
-}
-/**
- * Accepts non-null, non-array objects, including class instances and null-prototype records.
- * This is a shape check, not plain-object validation or sanitization. Keep `isRecord`'s
- * broader contract: existing tool readers deliberately continue to accept arrays.
- * @param required.value - The untrusted value whose object shape is checked.
- */
-export function isNonArrayRecord(required, _optional = {}) {
-    return isRecord(required) && !Array.isArray(required.value);
-}
 /** Narrows `ctx.input` to a record, refusing anything else. The first line of most handlers. */
 export function requireInputRecord(required, _optional = {}) {
     const { input } = required;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jini-ai/core",
-  "version": "0.4.3",
+  "version": "0.4.5",
   "description": "The Jini kernel: typed DI tokens, pack composition and lifecycle, the tool registry, redaction, and origin/API-token auth primitives. Zero dependencies.",
   "license": "Apache-2.0",
   "repository": {
@@ -19,6 +19,7 @@
       "./model-facing-tool-errors": "universal",
       "./contribution-registry": "universal",
       "./naming": "universal",
+      "./records": "universal",
       "./primitives": "universal",
       "./fetch": "universal",
       "./text": "universal",
@@ -42,7 +43,8 @@
         ]
       },
       "shared": [
-        "src/tool-permissions.ts"
+        "src/tool-permissions.ts",
+        "src/records.ts"
       ],
       "forbidden": {
         "universal": [
@@ -101,6 +103,11 @@
       "import": "./dist/naming.js",
       "default": "./dist/naming.js"
     },
+    "./records": {
+      "types": "./dist/records.d.ts",
+      "import": "./dist/records.js",
+      "default": "./dist/records.js"
+    },
     "./primitives": {
       "types": "./dist/primitives/index.d.ts",
       "import": "./dist/primitives/index.js",