shelving 1.185.0 → 1.185.2

package/api/provider/MockAPIProvider.d.ts

@@ -1,7 +1,7 @@
 import type { AnyCaller } from "../../util/function.js";
 import type { RequestHandler, RequestOptions } from "../../util/http.js";
 import type { AnyEndpoint, Endpoint } from "../endpoint/Endpoint.js";
-import { ClientAPIProvider } from "./ClientAPIProvider.js";
+import type { APIProvider } from "./APIProvider.js";
 import { ThroughAPIProvider } from "./ThroughAPIProvider.js";
 export type MockAPIFetchCall = {
     readonly request: Request;
@@ -28,7 +28,7 @@ export declare class MockAPIProvider<P = unknown, R = unknown> extends ThroughAP
     readonly fetchCalls: MockAPIFetchCall[];
     readonly responseCalls: MockAPIResponseCall[];
     readonly handler: RequestHandler;
-    constructor(handler?: RequestHandler, source?: ClientAPIProvider<P, R>);
+    constructor(handler?: RequestHandler, source?: APIProvider<P, R>);
     getRequest<PP extends P, RR extends R>(endpoint: Endpoint<PP, RR>, payload: PP, options?: RequestOptions, caller?: AnyCaller): Request;
     parseResponse<PP extends P, RR extends R>(endpoint: Endpoint<PP, RR>, response: Response, caller?: AnyCaller): Promise<RR>;
     fetch(request: Request): Promise<Response>;

package/api/provider/MockAPIProvider.js

@@ -1,8 +1,9 @@
+import { debugRequest } from "../../util/debug.js";
 import { ClientAPIProvider } from "./ClientAPIProvider.js";
 import { ThroughAPIProvider } from "./ThroughAPIProvider.js";
 /** Default handler just echoes back the input request as text. */
-async function _passthroughHandler(request) {
-    return new Response(await request.text());
+async function _mockHandler(request) {
+    return new Response(`Mocked response to ${debugRequest(request)}`, { status: 200, statusText: "OK" });
 }
 /**
  * Provider that logs API calls without sending network requests.
@@ -14,7 +15,7 @@ export class MockAPIProvider extends ThroughAPIProvider {
     fetchCalls = [];
     responseCalls = [];
     handler;
-    constructor(handler = _passthroughHandler, source = new ClientAPIProvider({ url: "https://api.mock.com" })) {
+    constructor(handler = _mockHandler, source = new ClientAPIProvider({ url: "https://api.mock.com" })) {
         super(source);
         this.handler = handler;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "shelving",
-	"version": "1.185.0",
+	"version": "1.185.2",
 	"author": "Dave Houlbrooke <dave@shax.com>",
 	"repository": {
 		"type": "git",

package/schema/ChoiceSchema.d.ts

@@ -20,7 +20,7 @@ export interface ChoiceSchemaOptions<K extends string> extends Omit<SchemaOption
 }
 /** Choose from an allowed set of values. */
 export declare class ChoiceSchema<K extends string> extends Schema<K> implements Iterable<ChoiceOption<K>> {
-    readonly value: K;
+    readonly value: K | undefined;
     readonly options: ChoiceOptions<K>;
     constructor({ one, title, placeholder, options, value, ...rest }: ChoiceSchemaOptions<K>);
     validate(unsafeValue?: unknown): K;

package/schema/ChoiceSchema.js

@@ -1,4 +1,4 @@
-import { isArray, requireFirst } from "../util/array.js";
+import { isArray } from "../util/array.js";
 import { getKeys, getProps, isProp } from "../util/object.js";
 import { Schema } from "./Schema.js";
 /** Get the options for a choice field as an array. */
@@ -19,14 +19,14 @@ export function isOption(options, option) {
 /** Choose from an allowed set of values. */
 export class ChoiceSchema extends Schema {
     options;
-    constructor({ one = "choice", title = "Choice", placeholder = `No ${one}`, options, value = requireFirst(isArray(options) ? options : getKeys(options)), ...rest }) {
+    constructor({ one = "choice", title = "Choice", placeholder = `No ${one}`, options, value, ...rest }) {
         super({ one, title, value, placeholder, ...rest });
         this.options = options;
     }
     validate(unsafeValue = this.value) {
         if (typeof unsafeValue === "string" && isOption(this.options, unsafeValue))
             return unsafeValue;
-        throw "Unknown value";
+        throw unsafeValue ? "Unknown value" : "Required";
     }
     // Implement iterable.
     *[Symbol.iterator]() {