shelving 1.142.0 → 1.143.0

package/markup/util/internal.js

@@ -1,2 +1,2 @@
 /** React security symbol — see https://github.com/facebook/react/pull/4832 */
-export const REACT_ELEMENT_TYPE = Symbol.for("react.element");
+export const REACT_ELEMENT_TYPE = Symbol.for("react.transitional.element");

package/package.json

@@ -11,7 +11,7 @@
 		"state-management",
 		"query-builder"
 	],
-	"version": "1.142.0",
+	"version": "1.143.0",
 	"repository": "https://github.com/dhoulb/shelving",
 	"author": "Dave Houlbrooke <dave@shax.com>",
 	"license": "0BSD",
@@ -57,19 +57,19 @@
 		"build:test:unit": "bun test ./dist/**/*.test.js --bail"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "^1.8.3",
-		"@google-cloud/firestore": "^7.9.0",
-		"@types/bun": "^1.1.6",
-		"@types/react": "^18.3.3",
-		"@types/react-dom": "^18.3.0",
-		"firebase": "^10.12.5",
-		"react": "^18.3.1",
-		"react-dom": "^18.3.1",
-		"typescript": "^5.8.2"
+		"@biomejs/biome": "^1.9.4",
+		"@google-cloud/firestore": "^7.11.3",
+		"@types/bun": "^1.2.18",
+		"@types/react": "^19.1.8",
+		"@types/react-dom": "^19.1.6",
+		"firebase": "^11.10.0",
+		"react": "^19.1.0",
+		"react-dom": "^19.1.0",
+		"typescript": "^5.8.3"
 	},
 	"peerDependencies": {
-		"@google-cloud/firestore": ">=4.0.0",
-		"firebase": ">=9.0.0",
-		"react": ">=17.0.0"
+		"@google-cloud/firestore": ">=7.0.0",
+		"firebase": ">=11.0.0",
+		"react": ">=19.0.0"
 	}
 }

package/react/createCacheContext.js

@@ -1,21 +1,22 @@
-import { createContext, createElement, useContext, useRef } from "react";
+import { jsx as _jsx } from "react/jsx-runtime";
+import { createContext, useContext } from "react";
 import { UnexpectedError } from "../error/UnexpectedError.js";
+import { useMap } from "./useMap.js";
 /**
  * Create a cache context that can be provided to React elements and allows them to call `useCache()`
  * - Cache is a `Map` indexed by strings that can be used to store any value.
  */
 export function createCacheContext() {
-    const context = createContext(undefined);
+    const Context = createContext(undefined);
     const useCache = () => {
-        const cache = useContext(context);
+        const cache = useContext(Context);
         if (!cache)
-            throw new UnexpectedError("useCache() must be used inside <Cache>", { caller: useCache });
+            throw new UnexpectedError("useCache() must be used inside <CacheContext>", { caller: useCache });
         return cache;
     };
     const CacheContext = ({ children }) => {
-        // biome-ignore lint/suspicious/noAssignInExpressions: This is the most efficient way to do this.
-        const cache = (useRef().current ||= new Map());
-        return createElement(context.Provider, { children, value: cache });
+        const cache = useMap();
+        return _jsx(Context, { value: cache, children: children });
     };
     return { useCache, CacheContext };
 }

package/react/index.d.ts

@@ -1,6 +1,7 @@
 export * from "./createCacheContext.js";
 export * from "./createDataContext.js";
-export * from "./useInstance.js";
+export * from "./useProps.js";
+export * from "./useMap.js";
 export * from "./useLazy.js";
 export * from "./useReduce.js";
 export * from "./useSequence.js";

package/react/index.js

@@ -1,6 +1,7 @@
 export * from "./createCacheContext.js";
 export * from "./createDataContext.js";
-export * from "./useInstance.js";
+export * from "./useProps.js";
+export * from "./useMap.js";
 export * from "./useLazy.js";
 export * from "./useReduce.js";
 export * from "./useSequence.js";

package/react/useInstance.js

@@ -1,17 +1,16 @@
 import { isArrayEqual } from "../util/equal.js";
-import { useInternals } from "./useInternals.js";
+import { useProps } from "./useProps.js";
 /**
  * Use a memoised class instance.
  * - Creates a new instance of `Constructor` using `args`
  * - Returns same instance for as long as `args` is equal to previous `args`.
  */
 export function useInstance(Constructor, ...args) {
-    const internals = useInternals();
+    const internals = useProps();
     // Update `internals` if `args` changes or `instance` is not set.
-    if (!internals.args || !isArrayEqual(args, internals.args)) {
+    if (!internals.args || !internals.instance || !isArrayEqual(args, internals.args)) {
         internals.instance = new Constructor(...args);
         internals.args = args;
     }
-    // biome-ignore lint/style/noNonNullAssertion: We know this is set.
     return internals.instance;
 }

package/react/useLazy.js

@@ -1,13 +1,12 @@
 import { isArrayEqual } from "../util/equal.js";
 import { getLazy } from "../util/lazy.js";
-import { useInternals } from "./useInternals.js";
+import { useProps } from "./useProps.js";
 export function useLazy(value, ...args) {
-    const internals = useInternals();
+    const internals = useProps();
     // Update `internals` if `args` changes.
-    if (internals.args === undefined || !isArrayEqual(args, internals.args)) {
+    if (!internals.args || !isArrayEqual(args, internals.args)) {
         internals.value = getLazy(value, ...args);
         internals.args = args;
     }
-    // biome-ignore lint/style/noNonNullAssertion: We know this is set.
     return internals.value;
 }

package/react/useMap.d.ts

@@ -0,0 +1,2 @@
+/** Create a mutable Map that persist for the lifetime of the component. */
+export declare function useMap<K, V>(): Map<K, V>;

package/react/useMap.js

@@ -0,0 +1,8 @@
+import { useRef } from "react";
+/** Create a mutable Map that persist for the lifetime of the component. */
+export function useMap() {
+    const ref = useRef(undefined);
+    if (!ref.current)
+        ref.current = new Map();
+    return ref.current;
+}

package/react/useProps.d.ts

@@ -0,0 +1,6 @@
+type Mutable<T> = {
+    -readonly [K in keyof T]: T[K];
+};
+/** Create an object that persist for the lifetime of the component. */
+export declare function useProps<T>(): Partial<Mutable<T>>;
+export {};

package/react/useProps.js

@@ -0,0 +1,5 @@
+import { useRef } from "react";
+/** Create an object that persist for the lifetime of the component. */
+export function useProps() {
+    return useRef(undefined);
+}

package/react/useStore.js

@@ -2,17 +2,16 @@ import { useSyncExternalStore } from "react";
 import { NONE } from "../util/constants.js";
 import { BLACKHOLE } from "../util/function.js";
 import { runSequence } from "../util/sequence.js";
-import { useInternals } from "./useInternals.js";
+import { useProps } from "./useProps.js";
 export function useStore(store) {
     // Store memoized versions of `subscribe()` and `getSnapshot()` so `useSyncExternalStore()` doesn't re-subscribe on every render.
-    const internals = useInternals();
+    const internals = useProps();
     // Update `internals` if `store` changes.
-    if (store !== internals.store) {
+    if (store !== internals.store || !internals.subscribe || !internals.getSnapshot) {
         internals.subscribe = onStoreChange => (store ? runSequence(store, onStoreChange, onStoreChange) : BLACKHOLE);
         internals.getSnapshot = () => (!store ? undefined : store.loading ? NONE : store.value);
         internals.store = store;
     }
-    // biome-ignore lint/style/noNonNullAssertion: We know these are set.
     useSyncExternalStore(internals.subscribe, internals.getSnapshot);
     return store;
 }

package/util/source.d.ts

@@ -1,3 +1,4 @@
+import type { AnyCaller } from "../error/BaseError.js";
 import type { Class } from "./class.js";
 /** Something that has a source of a specified type. */
 export interface Sourceable<T> {
@@ -6,4 +7,4 @@ export interface Sourceable<T> {
 /** Recurse through `Sourceable` objects and return the first one that is an instance of `type`, or `undefined` if no source object matches. */
 export declare function getSource<T>(type: Class<T>, value: unknown): T | undefined;
 /** Recurse through `Sourceable` objects and return the first one that is an instance of `type`, or throw `RequiredError` if no source object matches. */
-export declare function requireSource<T>(type: Class<T>, data: unknown): T;
+export declare function requireSource<T>(type: Class<T>, data: unknown, caller?: AnyCaller): T;

package/util/source.js

@@ -10,9 +10,9 @@ export function getSource(type, value) {
     }
 }
 /** Recurse through `Sourceable` objects and return the first one that is an instance of `type`, or throw `RequiredError` if no source object matches. */
-export function requireSource(type, data) {
+export function requireSource(type, data, caller = requireSource) {
     const source = getSource(type, data);
     if (!source)
-        throw new RequiredError(`Source "${type.name}" not found`, { received: data, expected: type, caller: requireSource });
+        throw new RequiredError(`Source "${type.name}" not found`, { received: data, expected: type, caller });
     return source;
 }

package/react/useInternals.d.ts

@@ -1,5 +0,0 @@
-import type { Data } from "../util/data.js";
-/** Store internal implementation details for a hook that persist for the lifetime of the component. */
-export declare const useInternals: <T extends Data>() => T | {
-    [K in keyof T]: undefined;
-};

package/react/useInternals.js

@@ -1,3 +0,0 @@
-import { useRef } from "react";
-/** Store internal implementation details for a hook that persist for the lifetime of the component. */
-export const useInternals = useRef;