@crawlee/core 4.0.0-beta.99 → 4.0.0-rc.1

package/storages/request_loader.d.ts

@@ -2,6 +2,44 @@ import type { Dictionary } from '@crawlee/types';
 import type { Request } from '../request.js';
 import type { IRequestManager } from './request_manager.js';
 import type { RequestQueueOperationInfo } from './request_queue.js';
+/**
+ * A request source's own availability, in a single answer.
+ *
+ * - `ready` — the next {@link IRequestLoader.fetchNextRequest} is expected to hand something over.
+ * - `waiting` — nothing to fetch right now, but the source is not done: requests are in progress, are being
+ *   added in the background, or are held back until `readyAt`.
+ * - `stalled` — the source holds requests it cannot make progress on. Only a manager that paces its own
+ *   dispatch can reach this; see {@link ThrottlingRequestManager}.
+ * - `finished` — everything has been handled.
+ */
+export type RequestSourceStatus = {
+    status: 'ready';
+} | {
+    status: 'waiting';
+    /**
+     * A `Date.now()` timestamp at which the source expects to become `ready`. Absent when the wait has
+     * no clock (an in-progress request, a background add), leaving a consumer to poll.
+     */
+    readyAt?: number;
+} | {
+    status: 'stalled';
+    reason: string;
+} | {
+    status: 'finished';
+};
+/** Loaders never stall — only a manager that paces its own dispatch can. */
+export type RequestLoaderStatus = Exclude<RequestSourceStatus, {
+    status: 'stalled';
+}>;
+/**
+ * Combines two request sources' statuses, with the precedence `ready` > `stalled` > `waiting` > `finished`.
+ *
+ * Binary rather than variadic on purpose: it is on the task loop's probe path and folding a pair allocates
+ * nothing.
+ *
+ * @internal
+ */
+export declare function joinRequestSourceStatuses(a: RequestSourceStatus, b: RequestSourceStatus): RequestSourceStatus;
 /**
  * An abstract interface defining a read-only stream of requests to crawl.
  *
@@ -21,10 +59,10 @@ import type { RequestQueueOperationInfo } from './request_queue.js';
  * "finished with this request", whether processing succeeded or was abandoned after exhausting retries.
  *
  * Honoring this contract matters for three reasons:
- * - **Restarts and migrations:** loaders that persist their state (see {@link IRequestLoader.persistState})
+ * - **Restarts and migrations:** loaders that persist their state (such as {@link RequestList})
  *   treat in-progress requests as interrupted and re-serve them after a restart. A request that is fetched
  *   but never marked handled will be crawled again.
- * - **Termination detection:** {@link IRequestLoader.isFinished} only resolves to `true` once nothing is
+ * - **Termination detection:** {@link IRequestLoader.checkReadiness} only reports `finished` once nothing is
  *   in progress. Leaving a request unmarked keeps the crawler running indefinitely.
  * - **Bookkeeping:** the handled and pending counts are derived from the set of in-progress requests, so
  *   skipping {@link IRequestLoader.markRequestAsHandled} corrupts {@link IRequestLoader.getHandledCount}
@@ -47,15 +85,13 @@ export interface IRequestLoader {
      */
     getHandledCount(): Promise<number>;
     /**
-     * Returns `true` if all requests were already handled and there are no more left.
-     */
-    isFinished(): Promise<boolean>;
-    /**
-     * Resolves to `true` if the next call to {@link IRequestLoader.fetchNextRequest} function
-     * would return `null`, otherwise it resolves to `false`.
-     * Note that even if the loader is empty, there might be some pending requests currently being processed.
+     * Reports whether the loader has a request to hand over, is waiting on one, or is done — see
+     * {@link RequestSourceStatus}.
+     *
+     * A consumer's task loop is gated on this, so implementations MUST answer `ready` before evaluating
+     * anything else. `finished` may arrive late behind distributed storage, but it is never wrong.
      */
-    isEmpty(): Promise<boolean>;
+    checkReadiness(): Promise<RequestSourceStatus>;
     /**
      * Gets the next {@link Request} to process, or `null` if there are no more pending requests.
      *
@@ -76,16 +112,11 @@ export interface IRequestLoader {
      *
      * Call this once you are done with the request — whether processing succeeded or was abandoned after
      * exhausting retries. Because a loader cannot take a request back, marking it handled is the only way to
-     * signal completion; failing to do so prevents {@link IRequestLoader.isFinished} from ever resolving to
-     * `true` and skews the handled and pending counts. See the request lifecycle contract on {@link IRequestLoader}.
+     * signal completion; failing to do so prevents {@link IRequestLoader.checkReadiness} from ever reporting
+     * `finished` and skews the handled and pending counts. See the request lifecycle contract on
+     * {@link IRequestLoader}.
      */
     markRequestAsHandled(request: Request): Promise<RequestQueueOperationInfo | void | null>;
-    /**
-     * Persists the current state of the loader into the default {@link KeyValueStore}.
-     *
-     * Not all loaders support persistence; implementations that do not should leave this `undefined`.
-     */
-    persistState?(): Promise<void>;
     /**
      * Combines the loader with a request manager to support adding and reclaiming requests.
      *

package/storages/request_loader.js

@@ -1 +1,36 @@
-export {};
+/**
+ * Combines two request sources' statuses, with the precedence `ready` > `stalled` > `waiting` > `finished`.
+ *
+ * Binary rather than variadic on purpose: it is on the task loop's probe path and folding a pair allocates
+ * nothing.
+ *
+ * @internal
+ */
+export function joinRequestSourceStatuses(a, b) {
+    if (a.status === 'ready') {
+        return a;
+    }
+    if (b.status === 'ready') {
+        return b;
+    }
+    // `ready` outranking `stalled` masks a stalled source while the other still has work. That is parity with
+    // the crawler before this was a single answer: the stall check was only reached from `isFinishedFunction`,
+    // which the task loop calls only when nothing is in flight and nothing is ready. "Fixing" the masking
+    // turns a crawl that is progressing elsewhere into a `PersistentRateLimitError`. `stalled` outranking
+    // `waiting` is the same parity - that call site fired regardless of other domains' clocks.
+    if (a.status === 'stalled') {
+        return a;
+    }
+    if (b.status === 'stalled') {
+        return b;
+    }
+    if (a.status === 'waiting') {
+        // The earlier of the two known wake-up times - unknown only if neither source announced one.
+        if (b.status !== 'waiting' || b.readyAt === undefined) {
+            return a;
+        }
+        return a.readyAt !== undefined && a.readyAt <= b.readyAt ? a : b;
+    }
+    // `a` is finished, so `b` decides.
+    return b;
+}

package/storages/request_manager.d.ts

@@ -1,3 +1,4 @@
+import type { LiteralUnion } from 'type-fest';
 import type { Request, Source } from '../request.js';
 import type { IRequestLoader } from './request_loader.js';
 import type { AddRequestsBatchedOptions, AddRequestsBatchedResult, RequestQueueOperationInfo, RequestQueueOperationOptions } from './request_queue.js';
@@ -30,4 +31,89 @@ export interface IRequestManager extends IRequestLoader {
      * this hint may leave it `undefined`.
      */
     setExpectedRequestProcessingTimeSecs?(secs: number): Promise<void>;
+    /**
+     * Records something said about the pace requests should go out at, so that a manager which paces its own
+     * dispatch can hold requests back.
+     *
+     * Required rather than optional, so that a wrapping manager always forwards it and a pacer nested in a
+     * composition still receives it; a manager that does not pace returns `false`.
+     *
+     * @returns `true` if anything in the composition took responsibility for the signal.
+     */
+    recordPacingSignal(signal: PacingSignal): boolean;
+    /**
+     * Extends the lock on a request previously handed out by `fetchNextRequest()` and still being
+     * processed, on storage backends that reserve requests via locking (e.g. via
+     * {@link CrawlingContext.extendTimeout|`context.extendTimeout`}).
+     *
+     * @returns `true` when the lock was prolonged, `false` when this manager does not lock
+     * requests or no longer holds this one; non-locking implementations may leave it `undefined`,
+     * which callers treat as `false`.
+     */
+    extendRequestProcessingTimeSecs?(request: Request, secs: number): Promise<boolean>;
 }
+/**
+ * How much of the URL space a {@link PacingSignal} covers.
+ *
+ * Open on purpose: `'hostname'` and `'registrableDomain'` are what Crawlee's own reporters send and what
+ * {@link ThrottlingRequestManager} understands, but any string is accepted, so a pacer keyed on something
+ * else can be reported to in its own vocabulary.
+ */
+export type PacingScope = LiteralUnion<'hostname' | 'registrableDomain', string>;
+/**
+ * Something said about the pace requests should go out at, reported to a request manager through
+ * {@link IRequestManager.recordPacingSignal}.
+ *
+ * One shape rather than a method per channel: a pacing manager switches on `reason`, a wrapping one forwards the
+ * value without knowing what is in it, and a new kind of signal costs the interface nothing. The `url` travels
+ * inside the value because the crawl-wide variant has none. Nothing here names the mechanism a signal came
+ * from - status codes, headers and robots.txt are the crawler's business - and every delay is in milliseconds.
+ *
+ * ## Scope
+ *
+ * A manager may apply a signal to a **wider** scope than it was given - a floor that holds for one host still
+ * holds when a whole site is paced by it - but never to a narrower one, which would leave some of the URLs the
+ * signal covers running unpaced. A manager that can only do the latter, or that does not recognise the scope at
+ * all, MUST throw rather than quietly under-apply it.
+ */
+export type PacingSignal = {
+    /**
+     * The source turned a request away because we were going too fast — an HTTP 429 or 503, an exhausted
+     * quota. Reactive and transient: a pacer typically backs off while refusals continue, and lets that
+     * decay once they stop.
+     */
+    reason: 'rateLimited';
+    /** The URL that was turned away. */
+    url: string;
+    /** How long the source asked us to wait before trying again, if it said. */
+    waitMs?: number;
+    /**
+     * How far this refusal reaches, if the reporter can tell — a 429 rarely says. Left out, it asks the
+     * manager to apply the signal however it happens to group requests.
+     */
+    scope?: PacingScope;
+} | {
+    /**
+     * The source declared a standing floor on how often it may be requested — a robots.txt `Crawl-delay`,
+     * a documented quota. A property of the source rather than of the run, so a pacer keeps it for the
+     * whole crawl.
+     */
+    reason: 'minInterval';
+    /** A URL of the source that declared the interval. */
+    url: string;
+    /** The declared minimum interval between two requests to the source. */
+    intervalMs: number;
+    /** Required, since whoever declares an interval knows what it applies to. */
+    scope: PacingScope;
+} | {
+    /**
+     * A standing floor under the pace of **every** domain the manager dispatches to, declared by whoever
+     * owns the crawl rather than by a source — a crawler's `sameDomainDelaySecs`. A manager that paces only
+     * some of its domains MUST throw rather than under-apply it.
+     */
+    reason: 'minIntervalEverywhere';
+    /** The declared minimum interval between two requests to any one source. */
+    intervalMs: number;
+    /** At what granularity the floor applies. */
+    scope: PacingScope;
+};

package/storages/request_manager_tandem.d.ts

@@ -1,7 +1,7 @@
 import type { Dictionary } from '@crawlee/types';
 import type { Request, Source } from '../request.js';
-import type { IRequestLoader } from './request_loader.js';
-import type { IRequestManager, RequestsLike } from './request_manager.js';
+import type { IRequestLoader, RequestSourceStatus } from './request_loader.js';
+import type { IRequestManager, PacingSignal, RequestsLike } from './request_manager.js';
 import type { AddRequestsBatchedOptions, AddRequestsBatchedResult, RequestQueueOperationInfo, RequestQueueOperationOptions } from './request_queue.js';
 /**
  * A request manager that combines a {@link IRequestLoader} (such as a `RequestList`) with a writable
@@ -9,16 +9,7 @@ import type { AddRequestsBatchedOptions, AddRequestsBatchedResult, RequestQueueO
  * It first reads requests from the loader and then, when needed, transfers them in batches to the manager.
  */
 export declare class RequestManagerTandem implements IRequestManager {
-    private log;
-    private requestLoader;
-    private requestManagerPromise?;
-    private resolvedRequestManager?;
-    private requestManagerFactory;
-    /**
-     * The latest expected request-processing time hinted via {@link setExpectedRequestProcessingTimeSecs}.
-     * Remembered so it can be applied to the writable manager once it is lazily resolved.
-     */
-    private expectedRequestProcessingSecs?;
+    #private;
     /**
      * @param requestLoader The read-only loader to read requests from first.
      * @param requestManager The writable manager to transfer requests into and enqueue new ones. May be passed as a
@@ -27,33 +18,15 @@ export declare class RequestManagerTandem implements IRequestManager {
      */
     constructor(requestLoader: IRequestLoader, requestManager: IRequestManager | (() => IRequestManager | Promise<IRequestManager>));
     /**
-     * Resolves the writable request manager, opening it lazily (via the factory) on first use and memoizing the result.
-     * @private
-     */
-    private getRequestManager;
-    /**
-     * Transfers a single request from the read-only loader to the writable manager.
-     * If the transfer fails, the request is dropped (and logged) rather than reclaimed.
-     *
-     * @returns `true` if a request was successfully transferred (or there was nothing to transfer), and `false` if a
-     *  transfer was attempted but failed - in which case the caller should not fetch from the manager this round.
-     * @private
-     */
-    private transferNextRequestToQueue;
-    /**
-     * Fetches the next request from the request manager. If the manager is empty and the loader
-     * is not finished, it will transfer a request from the loader to the manager first.
+     * Fetches the next request, transferring one from the loader first if the loader still has work.
      * @inheritdoc
      */
     fetchNextRequest<T extends Dictionary = Dictionary>(): Promise<Request<T> | null>;
     /**
+     * The loader and the manager read as one source.
      * @inheritdoc
      */
-    isFinished(): Promise<boolean>;
-    /**
-     * @inheritdoc
-     */
-    isEmpty(): Promise<boolean>;
+    checkReadiness(): Promise<RequestSourceStatus>;
     /**
      * @inheritdoc
      */
@@ -86,11 +59,6 @@ export declare class RequestManagerTandem implements IRequestManager {
      * @inheritdoc
      */
     addRequestsBatched(requests: RequestsLike, options?: AddRequestsBatchedOptions): Promise<AddRequestsBatchedResult>;
-    /**
-     * Persists the state of the underlying read-only loader, if it supports persistence.
-     * @inheritdoc
-     */
-    persistState(): Promise<void>;
     /**
      * Purges the writable request manager so the tandem can be reused (e.g. across repeated `crawler.run()` calls).
      * The read-only loader is immutable and cannot be purged, so only the manager side is reset.
@@ -103,4 +71,12 @@ export declare class RequestManagerTandem implements IRequestManager {
      * @inheritdoc
      */
     setExpectedRequestProcessingTimeSecs(secs: number): Promise<void>;
+    /**
+     * Forwards a pacing signal to the writable manager - the loader side is read-only and dispatches nothing of
+     * its own. Only a resolved manager is signalled; the tandem will not open a queue to answer a question about
+     * pacing.
+     * @inheritdoc
+     */
+    recordPacingSignal(signal: PacingSignal): boolean;
+    extendRequestProcessingTimeSecs(request: Request, secs: number): Promise<boolean>;
 }

package/storages/request_manager_tandem.js

@@ -1,20 +1,21 @@
 import { serviceLocator } from '../service_locator.js';
+import { joinRequestSourceStatuses } from './request_loader.js';
 /**
  * A request manager that combines a {@link IRequestLoader} (such as a `RequestList`) with a writable
  * {@link IRequestManager} (such as a `RequestQueue`).
  * It first reads requests from the loader and then, when needed, transfers them in batches to the manager.
  */
 export class RequestManagerTandem {
-    log;
-    requestLoader;
-    requestManagerPromise;
-    resolvedRequestManager;
-    requestManagerFactory;
+    #log;
+    #requestLoader;
+    #requestManagerPromise;
+    #resolvedRequestManager;
+    #requestManagerFactory;
     /**
      * The latest expected request-processing time hinted via {@link setExpectedRequestProcessingTimeSecs}.
      * Remembered so it can be applied to the writable manager once it is lazily resolved.
      */
-    expectedRequestProcessingSecs;
+    #expectedRequestProcessingSecs;
     /**
      * @param requestLoader The read-only loader to read requests from first.
      * @param requestManager The writable manager to transfer requests into and enqueue new ones. May be passed as a
@@ -22,24 +23,30 @@ export class RequestManagerTandem {
      *  (e.g. a lazily-opened default {@link RequestQueue}).
      */
     constructor(requestLoader, requestManager) {
-        this.log = serviceLocator.getLogger().child({ prefix: 'RequestManagerTandem' });
-        this.requestLoader = requestLoader;
-        this.requestManagerFactory = typeof requestManager === 'function' ? requestManager : () => requestManager;
+        this.#log = serviceLocator.getLogger().child({ prefix: 'RequestManagerTandem' });
+        this.#requestLoader = requestLoader;
+        if (typeof requestManager === 'function') {
+            this.#requestManagerFactory = requestManager;
+        }
+        else {
+            // Nothing to open, so mark it resolved up front - synchronous pacing signals can then reach it.
+            this.#resolvedRequestManager = requestManager;
+            this.#requestManagerFactory = () => requestManager;
+        }
     }
     /**
      * Resolves the writable request manager, opening it lazily (via the factory) on first use and memoizing the result.
-     * @private
      */
-    async getRequestManager() {
-        if (this.resolvedRequestManager === undefined) {
-            this.requestManagerPromise ??= Promise.resolve(this.requestManagerFactory());
-            this.resolvedRequestManager = await this.requestManagerPromise;
+    async #getRequestManager() {
+        if (this.#resolvedRequestManager === undefined) {
+            this.#requestManagerPromise ??= Promise.resolve(this.#requestManagerFactory());
+            this.#resolvedRequestManager = await this.#requestManagerPromise;
             // Apply any hint received before the manager was resolved.
-            if (this.expectedRequestProcessingSecs !== undefined) {
-                await this.resolvedRequestManager.setExpectedRequestProcessingTimeSecs?.(this.expectedRequestProcessingSecs);
+            if (this.#expectedRequestProcessingSecs !== undefined) {
+                await this.#resolvedRequestManager.setExpectedRequestProcessingTimeSecs?.(this.#expectedRequestProcessingSecs);
             }
         }
-        return this.resolvedRequestManager;
+        return this.#resolvedRequestManager;
     }
     /**
      * Transfers a single request from the read-only loader to the writable manager.
@@ -47,80 +54,71 @@ export class RequestManagerTandem {
      *
      * @returns `true` if a request was successfully transferred (or there was nothing to transfer), and `false` if a
      *  transfer was attempted but failed - in which case the caller should not fetch from the manager this round.
-     * @private
      */
-    async transferNextRequestToQueue() {
-        const request = await this.requestLoader.fetchNextRequest();
+    async #transferNextRequestToQueue() {
+        const request = await this.#requestLoader.fetchNextRequest();
         if (request === null) {
             return true;
         }
-        const requestManager = await this.getRequestManager();
+        const requestManager = await this.#getRequestManager();
         try {
             await requestManager.addRequest(request, { forefront: true });
             return true;
         }
         catch (error) {
-            this.log.exception(error, 'Adding request from the RequestLoader to the RequestManager failed, the request has been dropped.', { url: request.url, uniqueKey: request.uniqueKey });
+            this.#log.exception(error, 'Adding request from the RequestLoader to the RequestManager failed, the request has been dropped.', { url: request.url, uniqueKey: request.uniqueKey });
             return false;
         }
         finally {
             // Mark it as handled so that the request doesn't get stuck in the `inProgress` state in the loader.
-            await this.requestLoader.markRequestAsHandled(request);
+            await this.#requestLoader.markRequestAsHandled(request);
         }
     }
     /**
-     * Fetches the next request from the request manager. If the manager is empty and the loader
-     * is not finished, it will transfer a request from the loader to the manager first.
+     * Fetches the next request, transferring one from the loader first if the loader still has work.
      * @inheritdoc
      */
     async fetchNextRequest() {
-        // First, try to transfer a request from the requestList
-        const [listEmpty, listFinished] = await Promise.all([
-            this.requestLoader.isEmpty(),
-            this.requestLoader.isFinished(),
-        ]);
-        if (!listEmpty && !listFinished) {
+        // Only the loader's own state decides this: a manager waiting out a backoff must not freeze the
+        // loader's unrelated requests for the length of it.
+        if ((await this.#requestLoader.checkReadiness()).status === 'ready') {
             // If the transfer failed, the request was dropped; don't fetch from the manager this round (matching
             // crawlee-python behaviour). The next `fetchNextRequest()` call will pick up where we left off.
-            if (!(await this.transferNextRequestToQueue())) {
+            if (!(await this.#transferNextRequestToQueue())) {
                 return null;
             }
         }
         // Try to fetch from manager after the transfer
-        return (await this.getRequestManager()).fetchNextRequest();
+        return (await this.#getRequestManager()).fetchNextRequest();
     }
     /**
+     * The loader and the manager read as one source.
      * @inheritdoc
      */
-    async isFinished() {
-        const requestManager = await this.getRequestManager();
-        const storagesFinished = await Promise.all([this.requestLoader.isFinished(), requestManager.isFinished()]);
-        return storagesFinished.every(Boolean);
-    }
-    /**
-     * @inheritdoc
-     */
-    async isEmpty() {
-        const requestManager = await this.getRequestManager();
-        const storagesEmpty = await Promise.all([this.requestLoader.isEmpty(), requestManager.isEmpty()]);
-        return storagesEmpty.every(Boolean);
+    async checkReadiness() {
+        const requestManager = await this.#getRequestManager();
+        const [loaderStatus, managerStatus] = await Promise.all([
+            this.#requestLoader.checkReadiness(),
+            requestManager.checkReadiness(),
+        ]);
+        return joinRequestSourceStatuses(loaderStatus, managerStatus);
     }
     /**
      * @inheritdoc
      */
     async getHandledCount() {
         // Since one of the stores needs to have priority when both are present, we query the request manager - the request loader will first be dumped into the manager and then left empty.
-        return (await this.getRequestManager()).getHandledCount();
+        return (await this.#getRequestManager()).getHandledCount();
     }
     /**
      * @inheritdoc
      */
     async getTotalCount() {
-        const requestManager = await this.getRequestManager();
+        const requestManager = await this.#getRequestManager();
         const [managerTotal, loaderTotal] = await Promise.all([
             requestManager.getTotalCount(),
             // count only pending to avoid double counting, requests marked as "handled" have been moved to requestManager
-            this.requestLoader.getPendingCount(),
+            this.#requestLoader.getPendingCount(),
         ]);
         return managerTotal + loaderTotal;
     }
@@ -128,10 +126,10 @@ export class RequestManagerTandem {
      * @inheritdoc
      */
     async getPendingCount() {
-        const requestManager = await this.getRequestManager();
+        const requestManager = await this.#getRequestManager();
         const [managerPending, loaderPending] = await Promise.all([
             requestManager.getPendingCount(),
-            this.requestLoader.getPendingCount(),
+            this.#requestLoader.getPendingCount(),
         ]);
         return managerPending + loaderPending;
     }
@@ -150,32 +148,25 @@ export class RequestManagerTandem {
      * @inheritdoc
      */
     async markRequestAsHandled(request) {
-        return (await this.getRequestManager()).markRequestAsHandled(request);
+        return (await this.#getRequestManager()).markRequestAsHandled(request);
     }
     /**
      * @inheritdoc
      */
     async reclaimRequest(request, options) {
-        return (await this.getRequestManager()).reclaimRequest(request, options);
+        return (await this.#getRequestManager()).reclaimRequest(request, options);
     }
     /**
      * @inheritdoc
      */
     async addRequest(requestLike, options) {
-        return (await this.getRequestManager()).addRequest(requestLike, options);
+        return (await this.#getRequestManager()).addRequest(requestLike, options);
     }
     /**
      * @inheritdoc
      */
     async addRequestsBatched(requests, options) {
-        return (await this.getRequestManager()).addRequestsBatched(requests, options);
-    }
-    /**
-     * Persists the state of the underlying read-only loader, if it supports persistence.
-     * @inheritdoc
-     */
-    async persistState() {
-        await this.requestLoader.persistState?.();
+        return (await this.#getRequestManager()).addRequestsBatched(requests, options);
     }
     /**
      * Purges the writable request manager so the tandem can be reused (e.g. across repeated `crawler.run()` calls).
@@ -183,7 +174,7 @@ export class RequestManagerTandem {
      * @inheritdoc
      */
     async purge() {
-        await (await this.getRequestManager()).purge?.();
+        await (await this.#getRequestManager()).purge?.();
     }
     /**
      * Forwards the hint to the writable request manager — that is where requests are fetched from and
@@ -191,7 +182,19 @@ export class RequestManagerTandem {
      * @inheritdoc
      */
     async setExpectedRequestProcessingTimeSecs(secs) {
-        this.expectedRequestProcessingSecs = secs;
-        await this.resolvedRequestManager?.setExpectedRequestProcessingTimeSecs?.(secs);
+        this.#expectedRequestProcessingSecs = secs;
+        await this.#resolvedRequestManager?.setExpectedRequestProcessingTimeSecs?.(secs);
+    }
+    /**
+     * Forwards a pacing signal to the writable manager - the loader side is read-only and dispatches nothing of
+     * its own. Only a resolved manager is signalled; the tandem will not open a queue to answer a question about
+     * pacing.
+     * @inheritdoc
+     */
+    recordPacingSignal(signal) {
+        return this.#resolvedRequestManager?.recordPacingSignal(signal) ?? false;
+    }
+    async extendRequestProcessingTimeSecs(request, secs) {
+        return (await this.#getRequestManager()).extendRequestProcessingTimeSecs?.(request, secs) ?? false;
     }
 }