@evolu/common 8.4.0 → 8.5.0

package/src/Resource.ts

@@ -392,7 +392,7 @@ export const createSharedResource =
         // only.
         if (sharedResourceRun.getState().type !== "Running") return true;
 
-        if (idleDisposeAfter) {
+        if (idleDisposeAfter !== undefined) {
           idleDisposeFiber = sharedResourceRun.abortable<void, never>(
             async (run) => {
               await run.ok(sleep(idleDisposeAfter));
@@ -414,7 +414,9 @@ export const createSharedResource =
         resource: current as unknown as BorrowedResource<T>,
         created,
         release,
-        [Symbol.dispose]: release,
+        [Symbol.dispose]: () => {
+          release();
+        },
       };
 
       leakDetector.track(
@@ -832,6 +834,7 @@ export function createSharedResourceByKey<
         using leaseDisposer = new DisposableStack();
         const leasesByKey: Array<readonly [Lease<T>, K]> = [];
 
+        // oxlint-disable-next-line unicorn/no-useless-spread -- Snapshot keys because each acquisition awaits and can mutate the registry.
         for (const [key] of [...sharedResourcesByKey]) {
           const lease = await run.ok(sharedResourceByKey.acquireCurrent(key));
           if (lease) leasesByKey.push([leaseDisposer.use(lease), key]);
@@ -1346,7 +1349,9 @@ export const createSharedResourceByKeyWithClaims =
           const release = (): boolean => releaseClaimLease(heldClaimLease);
           const claimLease: ClaimLease = {
             release,
-            [Symbol.dispose]: release,
+            [Symbol.dispose]: () => {
+              release();
+            },
           };
 
           leakDetector.track(
@@ -1367,6 +1372,7 @@ export const createSharedResourceByKeyWithClaims =
               resourceKey,
             ] of firstClaimTransitions) {
               onFirstClaimAdded?.(claim, resource, resourceKey);
+              // oxlint-disable-next-line eslint/no-loop-func -- Captures this iteration's block-scoped transition values for rollback.
               compensations.defer(() => {
                 if (!succeeded) {
                   notifyLastClaimRemoved(claim, resource, resourceKey);

package/src/Result.ts

@@ -189,6 +189,7 @@
  * @module
  */
 import {
+  createMutableArray,
   emptyArray,
   isNonEmptyArray,
   type NonEmptyReadonlyArray,
@@ -442,9 +443,8 @@ export const isErr = <T, E>(result: Result<T, E>): result is Err<E> =>
 export const getOrThrow = <T, E>(result: Result<T, E>): T => {
   if (result.ok) {
     return result.value;
-  } else {
-    throw new Error("getOrThrow", { cause: result.error });
   }
+  throw new Error("getOrThrow", { cause: result.error });
 };
 
 /**
@@ -572,7 +572,7 @@ export function trySync<T, E>(
 export function trySync<T, E>(
   fn: () => T,
   mapError?: (error: unknown) => E,
-): Result<T, E | unknown> {
+): Result<T, unknown> {
   try {
     return ok(fn());
   } catch (error) {
@@ -644,7 +644,7 @@ export function tryAsync<T, E>(
 export async function tryAsync<T, E>(
   promiseThunk: Thunk<Awaitable<T>>,
   mapError?: (error: unknown) => E,
-): Promise<Result<T, E | unknown>> {
+): Promise<Result<T, unknown>> {
   try {
     return ok(await promiseThunk());
   } catch (error) {
@@ -1181,7 +1181,7 @@ export function allResult(
     const length = input.length;
     if (length === 0) return ok(emptyArray);
 
-    const values = new Array<unknown>(length);
+    const values = createMutableArray<unknown>(length);
     for (let i = 0; i < length; i++) {
       const result = fn ? fn(input[i]) : (input[i] as AnyResult);
       if (!result.ok) return result;

package/src/Sqlite.ts

@@ -256,12 +256,13 @@ export const createSqlite =
             console.debug({ query });
 
             const label =
-              query.options?.logQueryExecutionTime &&
-              `SqliteQueryExecutionTime ${query.sql}`;
+              query.options?.logQueryExecutionTime === true
+                ? `SqliteQueryExecutionTime ${query.sql}`
+                : null;
 
-            if (label) console.time(label);
+            if (label !== null) console.time(label);
             const result = driver.exec(query);
-            if (label) console.timeEnd(label);
+            if (label !== null) console.timeEnd(label);
 
             if (query.options?.logExplainQueryPlan) {
               const result = driver.exec({
@@ -343,11 +344,12 @@ const drawSqliteQueryPlan = (rows: Array<SqliteQueryPlanRow>): string =>
       let indent = 0;
 
       do {
+        // oxlint-disable-next-line eslint/no-loop-func -- find invokes the callback synchronously before parentId changes.
         const parent = rows.find((r) => r.id === parentId);
         if (!parent) break;
         parentId = parent.parent;
         indent++;
-        // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition, no-constant-condition
+        // oxlint-disable-next-line eslint/no-constant-condition -- A missing parent terminates the traversal above.
       } while (true);
 
       return `${"  ".repeat(indent)}${row.detail}`;
@@ -395,7 +397,7 @@ export const createPreparedStatementsCache = <P>(
         if (alwaysPrepare !== true && !query.options?.prepare)
           return null as never;
         let statement = statementsBySql.get(query.sql);
-        if (!statement) {
+        if (statement === undefined) {
           statement = factory(query.sql);
           statementsBySql.set(query.sql, statement);
         }
@@ -492,7 +494,7 @@ export const sql = (
 sql.identifier = (identifier: string): SqlIdentifier => ({
   type: "SqlIdentifier",
   // From Kysely
-  sql: `"${identifier.replace(/"/g, '""')}"` as SafeSql,
+  sql: `"${identifier.replaceAll('"', '""')}"` as SafeSql,
 });
 
 /**

package/src/Store.ts

@@ -70,7 +70,9 @@ export const createStore = <T>(initialState: T, eq?: Eq<T>): Store<T> => {
     get: ref.get,
     subscribe: (listener) => {
       listeners.add(listener);
-      return () => listeners.delete(listener);
+      return () => {
+        listeners.delete(listener);
+      };
     },
 
     set: (state) => {

package/src/String.ts

@@ -9,7 +9,7 @@
  * pattern.
  */
 export const escapeRegExp = (value: string): string =>
-  value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  value.replaceAll(/[.*+?^${}()|[\]\\]/gu, "\\$&");
 
 /** Converts an unknown value to a compact, human-readable diagnostic string. */
 export const safelyStringifyUnknownValue = (value: unknown): string => {

package/src/Task.ts

@@ -3,36 +3,34 @@
  *
  * JavaScript-native structured concurrency.
  *
- * Structured concurrency makes ownership of asynchronous work explicit.
- * Operations form a tree where every child belongs to a parent. A parent waits
- * for its children before it completes, and abort follows the tree: aborting a
- * parent requests abort of all its descendants. Races and fail-fast operations
- * also abort their remaining sibling branches.
+ * Structured concurrency organizes running tasks into a tree. Every child
+ * belongs to a parent, a parent waits for its children before it completes, and
+ * abort propagates from parents to descendants. Races and fail-fast control
+ * flow abort siblings that are no longer needed.
  *
  * With plain {@link AbortController} code, these guarantees depend on call-site
- * discipline: someone must remember the `finally` that aborts started work and
- * the await that waits for cleanup. {@link Run} makes both structural:
- * `run(task)` registers every child before it starts, and the parent settles
+ * discipline: someone must remember the `finally` that aborts started tasks and
+ * the await that waits for cleanup. Evolu makes both structural: `run(task)`
+ * registers every child before it starts, and the parent {@link Run} settles
  * only after child cleanup finishes.
  *
- * Evolu models structured concurrency with ordinary JavaScript:
+ * Evolu implements structured concurrency with:
  *
- * - A {@link Task} describes an asynchronous operation and its dependencies.
+ * - A {@link Task} is a function passed to Run that returns an {@link Awaitable}
+ *   {@link Result} and declares its dependencies.
  * - A {@link Run} starts Tasks and owns their lifetimes.
  * - A {@link Fiber} is the Promise-backed handle returned when a Run starts a
  *   Task.
  * - An {@link AbortableFiber} adds explicit abort and async disposal.
  *
- * The runtime core is deliberately small: ordinary functions, a callable Run
- * with closed-over state, Promise-backed Fibers, {@link AbortSignal}
- * propagation, and JavaScript resource management. Together, these primitives
- * provide abort, cleanup, defect handling, dependency injection, monitoring,
- * concurrency, and resource bracketing.
+ * Together, these APIs provide abort, cleanup, defect handling, dependency
+ * injection, monitoring, and resource management.
  *
- * Tasks return domain success or failure as {@link Result}. Abort is control
- * flow represented by {@link AbortError}. If a Task throws or rejects with
- * anything else, that is a defect: the root Run reports it and shuts down its
- * tree so code does not continue in a potentially invalid state.
+ * Tasks return a {@link Result} containing either success or a domain error.
+ * Abort is control flow represented by {@link AbortError}. If a Task throws or
+ * rejects with anything else, that is a defect: the root Run reports it and
+ * shuts down its tree so code does not continue in a potentially invalid
+ * state.
  *
  * ```ts
  * import {
@@ -201,7 +199,7 @@
  *
  * {@link fetch} with a body mode already returns a plain value, so resilience is
  * ordinary Task composition. Combine {@link timeout} and {@link retry} to bound
- * each attempt and retry recoverable domain failures:
+ * each attempt and retry recoverable domain errors:
  *
  * ```ts
  * import {
@@ -424,9 +422,9 @@
  * expect(socketDisposed).toBe(true);
  * ```
  *
- * Use {@link Run.ok} with `await using` when an infallible Task returns a
- * disposable value. Use {@link acquireUseRelease} when acquisition and release
- * are separate operations rather than a disposable value.
+ * Use {@link Run.ok} with `await using` when a Task whose error type is `never`
+ * returns a disposable value. Use {@link acquireUseRelease} when acquisition and
+ * release are separate steps rather than a disposable value.
  *
  * ## Awaitable
  *
@@ -439,7 +437,7 @@
  *
  * A Task is an async ownership boundary, not a general unit of program
  * decomposition. Calling `run(task)` always creates a child Run by design. Use
- * ordinary promises when an async operation does not need its own Run.
+ * a plain async function when it does not need its own Run.
  *
  * A unified sync/async effect API is technically possible. It can detect
  * Promise-like values with {@link isPromiseLike}, dispose synchronous resources
@@ -457,7 +455,7 @@
  * code performs effects with the result. For example, a pure function can
  * accept a {@link RandomNumber} value instead of depending on {@link Random}.
  *
- * Large CPU-bound operations, such as parsing large JSON, sorting millions of
+ * Large CPU-bound computations, such as parsing large JSON, sorting millions of
  * items, or complex cryptography, belong in a worker. Model the asynchronous
  * call to that worker as a Task so Run can provide timeout, abort, cleanup, and
  * monitoring.
@@ -517,10 +515,10 @@
  * ### What should Task code do with defects?
  *
  * Nothing. Once a defect reaches the {@link Run}, it is too late: the root Run
- * panics, running Tasks are aborted, and the Run tree shuts down. If an
- * operation can throw or reject for a recoverable reason, wrap that operation
- * with {@link trySync} or {@link tryAsync} so the failure becomes a typed
- * {@link Result} error. Let unrecoverable failures propagate as defects.
+ * panics, running Tasks are aborted, and the Run tree shuts down. Use
+ * {@link trySync} or {@link tryAsync} to turn recoverable exceptions and Promise
+ * rejections into typed {@link Result} errors. Let unrecoverable failures
+ * propagate as defects.
  *
  * ### Why does a defect panic the whole Run tree?
  *
@@ -619,13 +617,6 @@
  * periodically await {@link yieldNow} for cooperative scheduling, and move
  * CPU-bound work to a worker.
  *
- * ### Should a Task be called directly?
- *
- * Only inside Task internals that explicitly require same-Run execution. A
- * direct call, `task(run)`, uses the current Run instead of creating a child
- * Run, so it bypasses child lifetime tracking, scheduling metadata, and child
- * disposal boundaries. Application code should use `run(task)`.
- *
  * ### Where are fork and join?
  *
  * Calling `run(task)` is fork: it starts a child Task and returns a
@@ -756,16 +747,78 @@ import type {
   Predicate,
 } from "./Types.ts";
 
-// Core
-
 /**
- * An operation run by {@link Run} that returns a {@link Result} synchronously or
- * asynchronously and declares its dependencies through `D`.
- *
- * Its return type is {@link Awaitable}.
+ * A function passed to {@link Run} that returns an {@link Awaitable}
+ * {@link Result} and declares its dependencies through `D`.
  *
  * See the {@link @evolu/common!Task | Task overview}.
  *
+ * ### Example
+ *
+ * A Task that can't fail with a domain error:
+ *
+ * ```ts
+ * import { createRun, ok, type Task } from "@evolu/common";
+ *
+ * const greet: Task<string> = () => ok("Hello!");
+ * const main: Task<string> = async (run) => await run(greet);
+ *
+ * await using run = createRun();
+ * expectOk(await run(main), "Hello!");
+ *
+ * // Without domain errors, `run.ok` returns the Ok value.
+ * expect(await run.ok(main)).toBe("Hello!");
+ * ```
+ *
+ * A Task that can fail with a domain error:
+ *
+ * ```ts
+ * import { createRun, err, type Task, type Typed } from "@evolu/common";
+ *
+ * const findUser =
+ *   (id: string): Task<string, UserNotFoundError> =>
+ *   () =>
+ *     err({ type: "UserNotFound", id });
+ *
+ * interface UserNotFoundError extends Typed<"UserNotFound"> {
+ *   readonly id: string;
+ * }
+ *
+ * await using run = createRun();
+ * expectErr(await run(findUser("user-1")), {
+ *   type: "UserNotFound",
+ *   id: "user-1",
+ * });
+ * ```
+ *
+ * A Task with dependencies:
+ *
+ * ```ts
+ * import { createRun, ok, type Task } from "@evolu/common";
+ *
+ * interface Config {
+ *   readonly greeting: string;
+ * }
+ *
+ * interface ConfigDep {
+ *   readonly config: Config;
+ * }
+ *
+ * const greet: Task<string, never, ConfigDep> = (run) =>
+ *   ok(`${run.deps.config.greeting}!`);
+ *
+ * const config: Config = { greeting: "Hello" };
+ * await using run = createRun({ config });
+ * expectOk(await run(greet), "Hello!");
+ * ```
+ *
+ * Start Tasks with `run(task)`, as shown above.
+ *
+ * A Task can also be called directly as `task(run)`, but this is rarely needed.
+ * The call executes the Task inline in the current Run, as if its body were
+ * part of the parent Task, so it does not create a child Run. This is mainly
+ * useful for Task composition helpers.
+ *
  * @group Core
  */
 export type Task<T, E = never, D = unknown> = (
@@ -937,6 +990,13 @@ export interface Run<D = unknown> {
    * expectOk(userResult, "Ada");
    * expectOk(savedResult, undefined);
    * ```
+   *
+   * Start Tasks with `run(task)`, as shown above.
+   *
+   * A Task can also be called directly as `task(run)`, but this is rarely
+   * needed. The call executes the Task inline in the current Run, as if its
+   * body were part of the parent Task, so it does not create a child Run. This
+   * is mainly useful for Task composition helpers.
    */
   <T, E>(task: Task<T, E, D>): Fiber<T, E, D>;
 
@@ -1196,8 +1256,8 @@ export interface Run<D = unknown> {
    * Creates a {@link DisposableRun} attached to the root {@link Run} with this
    * Run's deps.
    *
-   * Use it when you need a Run that can be reused across multiple operations.
-   * For a single long-lived {@link Task}, use {@link Run.daemon}.
+   * Use it to give multiple related Tasks a shared lifetime. For a single
+   * long-lived {@link Task}, use {@link Run.daemon}.
    *
    * Use deps to replace the created Run's custom deps. Default deps
    * ({@link RunDefaultDeps}) are inherited unless explicitly replaced with
@@ -2520,7 +2580,6 @@ const createRunInternal = <D extends object>(
           result = await scheduler.postTask(
             () => {
               startSignal.throwIfAborted();
-              // eslint-disable-next-line evolu/no-direct-task-call -- The executor invokes the Task with its child Run.
               return task(taskRun);
             },
             {
@@ -2530,7 +2589,6 @@ const createRunInternal = <D extends object>(
           );
         } else {
           startSignal.throwIfAborted();
-          // eslint-disable-next-line evolu/no-direct-task-call -- The executor invokes the Task with its child Run.
           result = await task(taskRun);
         }
 
@@ -2595,7 +2653,7 @@ const createRunInternal = <D extends object>(
     getOrThrow(await run(task, taskDeps))) as Run<D>["orThrow"];
 
   run.ok = (async (task: TaskInternal, taskDeps?: object) =>
-    getOk((await run(task, taskDeps)) as Result<any, never>)) as Run<D>["ok"];
+    getOk((await run(task, taskDeps)) as Result<any>)) as Run<D>["ok"];
   /* eslint-enable @typescript-eslint/no-unsafe-return */
 
   run.abortable = ((task: TaskInternal, deps?: object) =>
@@ -2724,15 +2782,12 @@ const withTaskMeta =
         taskInternal[taskMetaSymbol]?.abortBehavior === undefined,
       "abort behavior helpers cannot wrap the same Task",
     );
-    // eslint-disable-next-line evolu/no-direct-task-call -- Preserve the wrapped Task's child Run.
     const wrapped: TaskInternal<T, E, D> = (run) => task(run);
     const taskMeta = taskInternal[taskMetaSymbol];
     wrapped[taskMetaSymbol] = taskMeta ? { ...taskMeta, ...meta } : meta;
     return wrapped;
   };
 
-// Task helpers
-
 /**
  * A readonly record whose values are {@link Task}s.
  *
@@ -2768,7 +2823,7 @@ export type InferTaskRecordDeps<TTasks extends TaskRecord> = InferTasksDeps<
  * Options shared by {@link Task} collection helpers.
  *
  * `concurrency` controls how many Tasks run at once. It defaults to `1`. For
- * CPU-bound Tasks backed by workers or parallel native operations, a platform
+ * CPU-bound Tasks backed by workers or native parallelism, a platform
  * `availableParallelism()` result is often a good limit. For network or
  * database Tasks, choose a limit based on the transport, server, connection
  * pool, and rate limits.
@@ -2993,7 +3048,7 @@ export function all<
   TTask extends AnyTask,
 >(
   values: TValues,
-  // eslint-disable-next-line @typescript-eslint/unified-signatures -- Separate array and record overloads keep callback parameter inference precise.
+  // Separate array and record overloads keep callback parameter inference precise.
   fn: (value: TValues[keyof TValues], key: keyof TValues) => TTask,
   options: AllOptions,
 ): Task<void, InferTaskErr<TTask>, InferTasksDeps<ReadonlyArray<TTask>>>;
@@ -3103,7 +3158,7 @@ export function all<
   TTask extends AnyTask,
 >(
   values: TValues,
-  // eslint-disable-next-line @typescript-eslint/unified-signatures -- Separate array and record overloads keep callback parameter inference precise.
+  // Separate array and record overloads keep callback parameter inference precise.
   fn: (value: TValues[keyof TValues], key: keyof TValues) => TTask,
   options?: TaskCollectionOptions,
 ): Task<
@@ -3392,7 +3447,7 @@ export function allSettled<
   TTask extends AnyTask,
 >(
   values: TValues,
-  // eslint-disable-next-line @typescript-eslint/unified-signatures -- Separate array and record overloads keep callback parameter inference precise.
+  // Separate array and record overloads keep callback parameter inference precise.
   fn: (value: TValues[keyof TValues], key: keyof TValues) => TTask,
   options?: TaskCollectionOptions,
 ): Task<
@@ -3499,7 +3554,7 @@ const mapInput = (
  * This helper is a callback bridge. If `reject` forwards an Error created in a
  * separate async chain, V8 cannot reconstruct the caller's zero-cost async
  * stack through this bridge. Prefer native promise APIs and `await` when the
- * wrapped operation already has a promise-shaped API.
+ * wrapped API already returns a Promise.
  *
  * One-shot settlement applies only to `resolve` and `reject`. A synchronous
  * throw from the setup function is a defect that panics the Run tree even after
@@ -4484,7 +4539,7 @@ export const each =
           nextIndex += 1;
 
           const result = await run(tasks[index]);
-          // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- stopped can flip across the await via sibling workers
+          // stopped can flip across the await via sibling workers.
           if (stopped) break;
           run.signal.throwIfAborted();
           const decision = onResult(result, index);
@@ -4502,6 +4557,7 @@ export const each =
 
       active -= 1;
       if (active === 0) wake.resolve();
+      // oxlint-disable-next-line typescript/return-await -- StackTrace.test.ts measures this direct await edge in the worker topology.
       return await parked;
     };
 
@@ -4609,8 +4665,6 @@ export const yieldNow: Task<void> = async (run) => {
   return ok();
 };
 
-// Abortability
-
 /**
  * Waits until the current {@link Run} aborts, then rejects with its
  * {@link AbortError}.
@@ -4685,8 +4739,8 @@ export const waitForAbort: Task<never> = async (run) => {
  * execution. The daemon Task continues under root Run ownership until it
  * settles, observes abort, or the root Run is disposed.
  *
- * This is not a replacement for direct {@link AbortSignal} support in operations
- * that can observe abort, such as {@link fetch}, timers that accept a signal, or
+ * This is not a replacement for direct {@link AbortSignal} support in APIs that
+ * can observe abort, such as {@link fetch}, timers that accept a signal, or
  * callback APIs that accept a signal. Use it as an escape hatch for Tasks that
  * ignore abort when an abort request must stop waiting immediately.
  *
@@ -4747,7 +4801,7 @@ export const waitForAbort: Task<never> = async (run) => {
  * expect(finished).toBe(true);
  * ```
  *
- * Promise-producing operations should start inside the Task, not before it.
+ * Promise-returning functions should be called inside the Task, not before it.
  *
  * ```ts
  * import { createRun, ok, type Result, type Task } from "@evolu/common";
@@ -4856,7 +4910,7 @@ export const unabortable = /*#__PURE__*/ withTaskMeta({
  *
  * An abort request before the mask Task starts prevents entering the mask. Once
  * the body starts, plain child Tasks inherit the mask, so acquire and release
- * can run after abort. Put release operations directly in the original mask's
+ * can run after abort. Start release Tasks directly in the original mask's
  * `finally`; do not wrap release in a nested `unabortableMask`, which is a new
  * critical-section entry and may not start after abort.
  *
@@ -4938,7 +4992,7 @@ export const unabortableMask = <T, E, D = unknown>(
       runInternal.abortMask > abortableMask,
       "unabortableMask requires a masked Run; use run(task), not a direct call",
     );
-    const restoreToken = Symbol() as RestoreToken;
+    const restoreToken = Symbol("restore") as RestoreToken;
 
     // The token is local to this Task Run; descendant Runs inherit the token
     // set so helpers can receive restore while the mask Task is alive.
@@ -4973,7 +5027,7 @@ export const unabortableMask = <T, E, D = unknown>(
  * Prefer native `using`, `await using`, or {@link AsyncDisposableStack} for
  * owned values that implement {@link Disposable} or {@link AsyncDisposable}. Use
  * `acquireUseRelease` when acquisition must be balanced with a separate release
- * operation, such as unlocking, returning a pooled value, releasing a lease, or
+ * step, such as unlocking, returning a pooled value, releasing a lease, or
  * logging out of a session.
  *
  * ### Example
@@ -5045,7 +5099,7 @@ export const acquireUseRelease = <
       if (!resourceResult.ok) return resourceResult;
 
       try {
-        // eslint-disable-next-line react-hooks/rules-of-hooks -- `use` is an acquireUseRelease callback, not a React Hook.
+        // oxlint-disable-next-line react/rules-of-hooks -- `use` is an acquireUseRelease callback, not a React Hook.
         return await run(restore(use(resourceResult.value)));
       } finally {
         await run.ok(release(resourceResult.value));
@@ -5053,8 +5107,6 @@ export const acquireUseRelease = <
     },
   );
 
-// Concurrency primitives
-
 /**
  * A one-shot value resolved from outside the waiting {@link Task}.
  *
@@ -5230,7 +5282,6 @@ export const createGate = ({
 
   return {
     // Direct same-Run delegation is intentional so wait observes the current deferred.
-    // eslint-disable-next-line evolu/no-direct-task-call
     wait: (run) => deferred.task(run),
     open: () => {
       if (isOpen) return false;
@@ -5259,8 +5310,8 @@ export const createGate = ({
  *
  * Use {@link Semaphore.withPermit} or {@link Semaphore.withPermits} to acquire
  * permits for one Task and release them when it settles. Use
- * {@link Semaphore.take} when permits must be held across multiple operations;
- * the returned {@link SemaphorePermit} owns release and is disposable.
+ * {@link Semaphore.take} when one permit must cover several child Tasks; the
+ * returned {@link SemaphorePermit} owns release and is disposable.
  *
  * Requests are not capped by the current permit count because
  * {@link Semaphore.resize} can increase it later.
@@ -5362,8 +5413,8 @@ export interface SemaphorePermit extends Disposable {
  * semaphore does not reorder requests to maximize utilization.
  *
  * Use `"fifo"` when fairness and predictable progress matter, such as tenant
- * sync, API quota, or database operations where large requests must not be
- * starved by a stream of smaller requests.
+ * sync, API quota, or database pools where large requests must not be starved
+ * by a stream of smaller requests.
  *
  * Use `"greedy"` when permits represent a shared budget and smaller or
  * latency-sensitive requests should proceed around larger queued requests. For
@@ -5666,8 +5717,8 @@ export const createMutex = (): Mutex => {
  * resources, making idle-key cleanup less predictable and making accidental key
  * retention easier.
  *
- * Use Semaphore directly when callers need to hold permits across multiple
- * operations or resize a permit pool. Use `SemaphoreByKey` when permit
+ * Use Semaphore directly when callers need to hold permits while starting
+ * several child Tasks or resize a permit pool. Use `SemaphoreByKey` when permit
  * ownership should be tied to one Task lifetime and idle keys can be forgotten
  * automatically.
  *
@@ -5847,9 +5898,9 @@ export function createMutexByKey<K, L = K>({
 /**
  * {@link Ref} protected by a {@link Mutex}.
  *
- * `MutexRef` serializes reads, writes, and updates through an internal Mutex,
- * so every operation observes one consistent value transition at a time. When
- * an update fails or is aborted, the previous value is preserved.
+ * `MutexRef` serializes reads, writes, and updates through an internal Mutex.
+ * Reads see a stable value, while writes and updates commit one transition at a
+ * time. When an update fails or is aborted, the previous value is preserved.
  *
  * `MutexRef` is non-reentrant. Updaters and modifiers run while holding the
  * internal Mutex, so calling another method on the same MutexRef from inside
@@ -5859,8 +5910,8 @@ export function createMutexByKey<K, L = K>({
  * read-modify-write. Plain Ref cannot express that — between a sync read and a
  * later write, a concurrent transition can interleave and get lost.
  *
- * `MutexRef` operations are Tasks and incur normal {@link Run} lifecycle
- * overhead. Use {@link Ref} instead for synchronous state transitions,
+ * `MutexRef` reads, writes, and updates are Tasks and incur normal {@link Run}
+ * lifecycle overhead. Use {@link Ref} instead for synchronous state transitions,
  * especially on allocation-sensitive hot paths.
  *
  * @group Concurrency primitives
@@ -5981,7 +6032,7 @@ export const createMutexRef = <T>(initialValue: T): MutexRef<T> => {
 //   filtering, and pluggable log sinks.
 // - Tracing spans with names, timing, parent-child relationships, attributes,
 //   error status, and helpers for annotating the current or child spans.
-// - Metrics for counters, gauges, histograms, and operation durations.
+// - Metrics for counters, gauges, histograms, and Task execution durations.
 // - Resource metadata for service name, service version, deployment
 //   environment, and user-provided attributes.
 // - Exporters for production telemetry backends, including OTLP-compatible
@@ -5994,5 +6045,5 @@ export const createMutexRef = <T>(initialValue: T): MutexRef<T> => {
 // - Run labels and structured annotations for rendering useful snapshot trees
 //   instead of anonymous ids.
 // - Snapshot and trace views should preserve ownership boundaries, so reusable
-//   resources and long-lived operations appear as labeled subtrees instead of
-//   unrelated child operations.
+//   resources and long-lived Runs appear as labeled subtrees instead of
+//   unrelated child Runs.

package/src/Test.ts

@@ -50,5 +50,6 @@ export const testCreateId = (): TestCreateId => {
     randomLib: testCreateRandomLib(),
   });
 
+  // oxlint-disable-next-line typescript/no-unnecessary-type-arguments -- Explicit never resolves createId's conditional brand-validation rest parameter; inference rejects the argument without it.
   return (() => createId<never>({ randomBytes })) as TestCreateId;
 };