@mrclrchtr/supi-code-intelligence 4.9.0 → 5.0.0

package/src/analysis/readiness.ts

@@ -1,4 +1,8 @@
-import { getDefaultWorkspaceRuntime } from "@mrclrchtr/supi-code-runtime/api";
+import {
+  type CodeRequestControl,
+  getDefaultWorkspaceRuntime,
+  throwIfCodeRequestInterrupted,
+} from "@mrclrchtr/supi-code-runtime/api";
 import {
   getWorkspaceLspRuntime,
   type SemanticReadinessResult,
@@ -14,7 +18,10 @@ export async function ensureSemanticReadiness(
   cwd: string,
   scope: SemanticStartupScope,
   timeoutMs: number = DEFAULT_SEMANTIC_STARTUP_TIMEOUT_MS,
+  control?: CodeRequestControl,
 ): Promise<SemanticReadinessResult> {
+  // A cancelled caller stops before any readiness work starts.
+  throwIfCodeRequestInterrupted(control);
   const workspace = getDefaultWorkspaceRuntime().getWorkspace(cwd);
   if (workspace.semantic.provider === null) {
     return {
@@ -24,7 +31,7 @@ export async function ensureSemanticReadiness(
   }
 
   if (workspace.semantic.state.kind === "ready") {
-    return resolveReadySemanticState(cwd, scope, timeoutMs);
+    return resolveReadySemanticState(cwd, scope, timeoutMs, control);
   }
 
   if (workspace.semantic.state.kind !== "pending") {
@@ -34,13 +41,15 @@ export async function ensureSemanticReadiness(
     };
   }
 
-  const deadline = Date.now() + timeoutMs;
+  // The request deadline bounds the internal readiness budget when it is
+  // earlier than the relative startup timeout.
+  const deadline = Math.min(Date.now() + timeoutMs, control?.deadline ?? Infinity);
 
   const remainingAfterService = deadline - Date.now();
   if (remainingAfterService <= 0) {
     return { kind: "timeout" };
   }
-  const lspState = await resolveSemanticServiceState(cwd, remainingAfterService);
+  const lspState = await resolveSemanticServiceState(cwd, remainingAfterService, control);
   if (lspState.kind === "pending") {
     return { kind: "timeout" };
   }
@@ -58,8 +67,13 @@ export async function ensureSemanticReadiness(
   if (remainingAfterLsp <= 0) {
     return { kind: "timeout" };
   }
-  return scope.kind === "workspace"
-    ? lspState.runtime.waitUntilReadyForWorkspace({ timeoutMs: remainingAfterLsp })
+  if (scope.kind === "workspace") {
+    return control
+      ? lspState.runtime.waitUntilReadyForWorkspace({ timeoutMs: remainingAfterLsp }, control)
+      : lspState.runtime.waitUntilReadyForWorkspace({ timeoutMs: remainingAfterLsp });
+  }
+  return control
+    ? lspState.runtime.waitUntilReadyForFile(scope.file, { timeoutMs: remainingAfterLsp }, control)
     : lspState.runtime.waitUntilReadyForFile(scope.file, { timeoutMs: remainingAfterLsp });
 }
 
@@ -67,11 +81,17 @@ function resolveReadySemanticState(
   cwd: string,
   scope: SemanticStartupScope,
   timeoutMs: number,
+  control?: CodeRequestControl,
 ): Promise<SemanticReadinessResult> | SemanticReadinessResult {
   const lspState = getWorkspaceLspRuntime(cwd);
   if (lspState.kind === "ready") {
-    return scope.kind === "workspace"
-      ? lspState.runtime.waitUntilReadyForWorkspace({ timeoutMs })
+    if (scope.kind === "workspace") {
+      return control
+        ? lspState.runtime.waitUntilReadyForWorkspace({ timeoutMs }, control)
+        : lspState.runtime.waitUntilReadyForWorkspace({ timeoutMs });
+    }
+    return control
+      ? lspState.runtime.waitUntilReadyForFile(scope.file, { timeoutMs }, control)
       : lspState.runtime.waitUntilReadyForFile(scope.file, { timeoutMs });
   }
   return {
@@ -86,8 +106,9 @@ function resolveReadySemanticState(
 async function resolveSemanticServiceState(
   cwd: string,
   timeoutMs: number,
+  control?: CodeRequestControl,
 ): Promise<WorkspaceLspRuntimeState> {
   const initialState = getWorkspaceLspRuntime(cwd);
   if (initialState.kind !== "pending") return initialState;
-  return waitForWorkspaceLspRuntime(cwd, timeoutMs);
+  return waitForWorkspaceLspRuntime(cwd, timeoutMs, control);
 }

package/src/analysis/refactor/apply.ts

@@ -16,7 +16,8 @@ import { readFileSync, renameSync, rmSync, statSync, writeFileSync } from "node:
 import { basename, dirname, join } from "node:path";
 import { withFileMutationQueue } from "@earendil-works/pi-coding-agent";
 import type { FileEdit, WorkspaceEdit } from "@mrclrchtr/supi-code-runtime/api";
-import { compareCodePositions } from "./position.ts";
+import { revalidateMutationAuthority } from "./mutation-authority.ts";
+import { compareCodePositions, createLogicalLineIndex } from "./position.ts";
 import { validateEditAgainstFiles } from "./safety.ts";
 
 export type ApplyResult =
@@ -24,8 +25,12 @@ export type ApplyResult =
   | { kind: "error"; reason: string };
 
 export interface ApplyOptions {
+  /** Canonical roots authorized by the semantic provider that made the plan. */
+  authorizedMutationRoots: readonly string[];
   /** Expected SHA-256 fingerprints per file, validated after queue acquisition. */
   expectedFingerprints?: ReadonlyMap<string, string>;
+  /** Current LSP document version, or null when the document is not open. */
+  getOpenDocumentVersion?: (file: string) => number | null;
 }
 
 /**
@@ -49,16 +54,24 @@ export interface ApplyOptions {
  */
 export async function applyWorkspaceEdit(
   edit: WorkspaceEdit,
-  options: ApplyOptions = {},
+  options: ApplyOptions,
 ): Promise<ApplyResult> {
   const grouped = groupEditsByFile(edit.edits);
   const files = [...grouped.keys()].sort();
   return withAllMutationQueues(files, async () => {
+    const authority = revalidateMutationAuthority(files, options.authorizedMutationRoots);
+    if (authority.kind === "unavailable") {
+      return { kind: "error", reason: authority.reason };
+    }
+
     if (options.expectedFingerprints) {
       const freshness = validateFingerprints(files, options.expectedFingerprints);
       if (!freshness.ok) return { kind: "error", reason: freshness.reason };
     }
 
+    const documentState = validateDocumentPreconditions(edit, options.getOpenDocumentVersion);
+    if (!documentState.ok) return { kind: "error", reason: documentState.reason };
+
     const validation = validateEditAgainstFiles(edit);
     if (!validation.safe) {
       return { kind: "error", reason: validation.reason };
@@ -76,6 +89,37 @@ export async function applyWorkspaceEdit(
   });
 }
 
+function validateDocumentPreconditions(
+  edit: WorkspaceEdit,
+  getOpenDocumentVersion: ApplyOptions["getOpenDocumentVersion"],
+): { ok: true } | { ok: false; reason: string } {
+  const preconditions = edit.documentPreconditions ?? [];
+  if (preconditions.length === 0) return { ok: true };
+  if (!getOpenDocumentVersion) {
+    return {
+      ok: false,
+      reason: "Stored document preconditions cannot be revalidated.",
+    };
+  }
+
+  for (const precondition of preconditions) {
+    const currentVersion = getOpenDocumentVersion(precondition.file);
+    if (precondition.kind === "open-document-version" && currentVersion !== precondition.version) {
+      return {
+        ok: false,
+        reason: `File ${precondition.file} open document version has changed since the plan was generated.`,
+      };
+    }
+    if (precondition.kind === "disk-content" && currentVersion !== null) {
+      return {
+        ok: false,
+        reason: `File ${precondition.file} is now open and no longer uses disk content as its master.`,
+      };
+    }
+  }
+  return { ok: true };
+}
+
 function validateFingerprints(
   files: string[],
   expected: ReadonlyMap<string, string>,
@@ -158,18 +202,21 @@ function buildTransformedContents(
 }
 
 function applyEditsToContent(content: string, edits: FileEdit[]): string {
-  const lines = content.split("\n");
-  const sortedEdits = [...edits].sort(
-    (left, right) =>
-      // Descending by start position so later edits don't shift earlier ones.
-      compareCodePositions(right.range.start, left.range.start) ||
-      compareCodePositions(right.range.end, left.range.end),
-  );
+  const lineIndex = createLogicalLineIndex(content);
+  const sortedEdits = edits
+    .map((edit, protocolIndex) => ({ edit, protocolIndex }))
+    .sort(
+      (left, right) =>
+        // Descending positions prevent shifts. Reverse protocol order at one
+        // position so sequential inserts appear in their supplied order.
+        compareCodePositions(right.edit.range.start, left.edit.range.start) ||
+        right.protocolIndex - left.protocolIndex,
+    );
 
   let updated = content;
-  for (const fileEdit of sortedEdits) {
-    const startOffset = toOffset(lines, fileEdit.range.start.line, fileEdit.range.start.character);
-    const endOffset = toOffset(lines, fileEdit.range.end.line, fileEdit.range.end.character);
+  for (const { edit: fileEdit } of sortedEdits) {
+    const startOffset = lineIndex.offsetAt(fileEdit.range.start);
+    const endOffset = lineIndex.offsetAt(fileEdit.range.end);
     updated = updated.slice(0, startOffset) + fileEdit.newText + updated.slice(endOffset);
   }
 
@@ -266,14 +313,6 @@ function rollbackCommittedFiles(
   }
 }
 
-function toOffset(lines: string[], line: number, character: number): number {
-  let offset = 0;
-  for (let index = 0; index < line && index < lines.length; index++) {
-    offset += lines[index].length + 1;
-  }
-  return offset + character;
-}
-
 function toErrorMessage(error: unknown): string {
   return error instanceof Error ? error.message : String(error);
 }

package/src/analysis/refactor/mutation-authority.ts

@@ -0,0 +1,143 @@
+import { lstatSync, realpathSync, statSync } from "node:fs";
+import * as path from "node:path";
+
+export type MutationAuthorityResult =
+  | { kind: "authorized"; canonicalRoots: string[] }
+  | { kind: "unavailable"; reason: string };
+
+/**
+ * Establish canonical provider roots and verify that all mutation files are
+ * regular files inside at least one root.
+ */
+export function establishMutationAuthority(
+  files: readonly string[],
+  providerRoots: readonly string[],
+): MutationAuthorityResult {
+  const roots = canonicalizeProviderRoots(providerRoots);
+  if (roots.kind === "unavailable") return roots;
+
+  const fileValidation = validateFiles(files, roots.canonicalRoots);
+  return fileValidation.kind === "unavailable" ? fileValidation : roots;
+}
+
+/** Revalidate stored canonical roots and mutation files immediately before mutation. */
+export function revalidateMutationAuthority(
+  files: readonly string[],
+  canonicalRoots: readonly string[],
+): MutationAuthorityResult {
+  if (canonicalRoots.length === 0) {
+    return { kind: "unavailable", reason: "Mutation plan has no authorized provider root." };
+  }
+
+  for (const root of canonicalRoots) {
+    if (!path.isAbsolute(root) || path.normalize(root) !== root) {
+      return { kind: "unavailable", reason: `Stored mutation root "${root}" is not canonical.` };
+    }
+    try {
+      if (!lstatSync(root).isDirectory() || realpathSync(root) !== root) {
+        return { kind: "unavailable", reason: `Stored mutation root "${root}" has changed.` };
+      }
+    } catch {
+      return { kind: "unavailable", reason: `Stored mutation root "${root}" is unavailable.` };
+    }
+  }
+
+  return validateFiles(files, canonicalRoots);
+}
+
+function canonicalizeProviderRoots(providerRoots: readonly string[]): MutationAuthorityResult {
+  if (providerRoots.length === 0) {
+    return { kind: "unavailable", reason: "Semantic route did not authorize a mutation root." };
+  }
+
+  const canonicalRoots = new Set<string>();
+  for (const root of providerRoots) {
+    if (!path.isAbsolute(root)) {
+      return { kind: "unavailable", reason: `Mutation root "${root}" is not absolute.` };
+    }
+    try {
+      const canonicalRoot = realpathSync(root);
+      if (!statSync(canonicalRoot).isDirectory()) {
+        return {
+          kind: "unavailable",
+          reason: `Mutation root "${root}" is not a directory.`,
+        };
+      }
+      canonicalRoots.add(canonicalRoot);
+    } catch {
+      return {
+        kind: "unavailable",
+        reason: `Mutation root "${root}" cannot be resolved to a canonical directory.`,
+      };
+    }
+  }
+
+  return { kind: "authorized", canonicalRoots: [...canonicalRoots].sort() };
+}
+
+function validateFiles(
+  files: readonly string[],
+  canonicalRoots: readonly string[],
+): MutationAuthorityResult {
+  for (const file of new Set(files)) {
+    const validation = validateFile(file, canonicalRoots);
+    if (validation) return { kind: "unavailable", reason: validation };
+  }
+  return { kind: "authorized", canonicalRoots: [...canonicalRoots] };
+}
+
+function validateFile(file: string, canonicalRoots: readonly string[]): string | null {
+  if (!path.isAbsolute(file) || path.normalize(file) !== file) {
+    return `Mutation file "${file}" is not a normalized absolute path.`;
+  }
+
+  let canonicalParent: string;
+  try {
+    canonicalParent = realpathSync(path.dirname(file));
+  } catch {
+    return `Mutation file "${file}" does not have a resolvable canonical parent.`;
+  }
+
+  try {
+    if (!lstatSync(file).isFile()) {
+      return `Mutation file "${file}" is not a regular file.`;
+    }
+  } catch {
+    return `Mutation file "${file}" is not a readable regular file.`;
+  }
+
+  let canonicalFile: string;
+  try {
+    canonicalFile = realpathSync(file);
+  } catch {
+    return `Mutation file "${file}" cannot be resolved to a canonical file.`;
+  }
+
+  const expectedCanonicalFile = path.join(canonicalParent, path.basename(file));
+  if (path.resolve(canonicalFile) !== path.resolve(expectedCanonicalFile)) {
+    return `Mutation file "${file}" resolves through an unsupported file link.`;
+  }
+  if (!canonicalRoots.some((root) => isWithinOrEqual(root, canonicalFile))) {
+    return `Mutation file "${file}" is outside the authorized provider roots.`;
+  }
+  return null;
+}
+
+function isWithinOrEqual(root: string, candidate: string): boolean {
+  return isMutationPathWithinRoot(root, candidate, path);
+}
+
+/** Check canonical path containment with the selected platform path rules. */
+export function isMutationPathWithinRoot(
+  root: string,
+  candidate: string,
+  pathApi: Pick<typeof path, "isAbsolute" | "relative" | "sep">,
+): boolean {
+  const difference = pathApi.relative(root, candidate);
+  return (
+    difference === "" ||
+    (difference !== ".." &&
+      !difference.startsWith(`..${pathApi.sep}`) &&
+      !pathApi.isAbsolute(difference))
+  );
+}

package/src/analysis/refactor/position.ts

@@ -1,5 +1,52 @@
 import type { CodePosition } from "@mrclrchtr/supi-code-runtime/api";
 
+interface IndexedLine {
+  readonly startOffset: number;
+  readonly length: number;
+}
+
+/** Logical source lines with offsets into the unchanged source string. */
+export interface LogicalLineIndex {
+  readonly lineCount: number;
+  /** Get one logical line length in UTF-16 code units. */
+  lineLength(line: number): number | undefined;
+  /** Convert a validated zero-based UTF-16 position to a source-string offset. */
+  offsetAt(position: CodePosition): number;
+}
+
+/**
+ * Index LF, CRLF, and bare-CR lines without normalizing source bytes.
+ *
+ * JavaScript string offsets and lengths use UTF-16 code units, which are the
+ * only LSP coordinate encoding that the refactor path supports.
+ */
+export function createLogicalLineIndex(source: string): LogicalLineIndex {
+  const lines: IndexedLine[] = [];
+  let lineStart = 0;
+
+  for (let offset = 0; offset < source.length; offset++) {
+    const codeUnit = source.charCodeAt(offset);
+    if (codeUnit !== 0x0a && codeUnit !== 0x0d) continue;
+
+    lines.push({ startOffset: lineStart, length: offset - lineStart });
+    if (codeUnit === 0x0d && source.charCodeAt(offset + 1) === 0x0a) offset++;
+    lineStart = offset + 1;
+  }
+  lines.push({ startOffset: lineStart, length: source.length - lineStart });
+
+  return {
+    lineCount: lines.length,
+    lineLength: (line) => lines[line]?.length,
+    offsetAt: (position) => {
+      const line = lines[position.line];
+      if (!line || position.character < 0 || position.character > line.length) {
+        throw new RangeError("Position is outside the logical source lines");
+      }
+      return line.startOffset + position.character;
+    },
+  };
+}
+
 /** Compare zero-based source positions lexicographically by line, then character. */
 export function compareCodePositions(left: CodePosition, right: CodePosition): number {
   return left.line - right.line || left.character - right.character;

package/src/analysis/refactor/safety.ts

@@ -7,7 +7,7 @@
 
 import { readFileSync } from "node:fs";
 import type { FileEdit, WorkspaceEdit } from "@mrclrchtr/supi-code-runtime/api";
-import { compareCodePositions } from "./position.ts";
+import { compareCodePositions, createLogicalLineIndex, type LogicalLineIndex } from "./position.ts";
 
 export type ValidationResult = { safe: true } | { safe: false; reason: string };
 
@@ -76,12 +76,12 @@ export function validateEditAgainstFiles(edit: WorkspaceEdit): ValidationResult
       };
     }
 
-    const lines = content.split("\n");
+    const lineIndex = createLogicalLineIndex(content);
     for (const fileEdit of fileEdits) {
-      const lineValidation = validateLineBounds(file, fileEdit, lines);
+      const lineValidation = validateLineBounds(file, fileEdit, lineIndex);
       if (!lineValidation.safe) return lineValidation;
 
-      const characterValidation = validateCharacterBounds(file, fileEdit, lines);
+      const characterValidation = validateCharacterBounds(file, fileEdit, lineIndex);
       if (!characterValidation.safe) return characterValidation;
     }
   }
@@ -89,19 +89,27 @@ export function validateEditAgainstFiles(edit: WorkspaceEdit): ValidationResult
   return { safe: true };
 }
 
-function validateLineBounds(file: string, edit: FileEdit, lines: string[]): ValidationResult {
-  const maxLine = lines.length - 1;
+function validateLineBounds(
+  file: string,
+  edit: FileEdit,
+  lineIndex: LogicalLineIndex,
+): ValidationResult {
+  const maxLine = lineIndex.lineCount - 1;
   if (edit.range.start.line > maxLine || edit.range.end.line > maxLine) {
     return {
       safe: false,
-      reason: `Edit in file "${file}" references line ${Math.max(edit.range.start.line, edit.range.end.line)}, but the file has only ${lines.length} line${lines.length === 1 ? "" : "s"}`,
+      reason: `Edit in file "${file}" references line ${Math.max(edit.range.start.line, edit.range.end.line)}, but the file has only ${lineIndex.lineCount} line${lineIndex.lineCount === 1 ? "" : "s"}`,
     };
   }
   return { safe: true };
 }
 
-function validateCharacterBounds(file: string, edit: FileEdit, lines: string[]): ValidationResult {
-  const startLineLength = lines[edit.range.start.line]?.length ?? 0;
+function validateCharacterBounds(
+  file: string,
+  edit: FileEdit,
+  lineIndex: LogicalLineIndex,
+): ValidationResult {
+  const startLineLength = lineIndex.lineLength(edit.range.start.line) ?? 0;
   if (edit.range.start.character > startLineLength) {
     return {
       safe: false,
@@ -109,7 +117,7 @@ function validateCharacterBounds(file: string, edit: FileEdit, lines: string[]):
     };
   }
 
-  const endLineLength = lines[edit.range.end.line]?.length ?? 0;
+  const endLineLength = lineIndex.lineLength(edit.range.end.line) ?? 0;
   if (edit.range.end.character > endLineLength) {
     return {
       safe: false,

package/src/analysis/relations/callees.ts

@@ -2,6 +2,7 @@
  * Structural callee lookup — finds direct calls in the enclosing scope at a target position.
  */
 
+import type { CodeRequestControl } from "@mrclrchtr/supi-code-runtime/api";
 import type { CalleeEntry, CalleeScope, RelationsServiceDeps } from "./types.ts";
 
 export interface CalleesResult {
@@ -26,12 +27,19 @@ export async function collectCallees(
   deps: RelationsServiceDeps,
   maxResults?: number,
   depth: "direct" | "deep" = "direct",
+  control?: CodeRequestControl,
 ): Promise<CalleesResult> {
   if (!deps.provider?.calleesAt) {
     return unavailableCallees(targetName);
   }
 
-  const result = await deps.provider.calleesAt(targetFile, targetLine, targetCharacter, depth);
+  const depthOrOptions = control ? { depth, control } : depth;
+  const result = await deps.provider.calleesAt(
+    targetFile,
+    targetLine,
+    targetCharacter,
+    depthOrOptions,
+  );
   if (result.kind !== "success" || !result.data) {
     return unavailableCallees(targetName);
   }

package/src/analysis/relations/types.ts

@@ -1,4 +1,4 @@
-import type { SemanticProvider } from "@mrclrchtr/supi-code-runtime/api";
+import type { CodeRequestControl, SemanticProvider } from "@mrclrchtr/supi-code-runtime/api";
 
 /**
  * Shared types for the relations analysis modules.
@@ -58,7 +58,10 @@ export interface RelationsServiceDeps {
       file: string,
       line: number,
       character: number,
-      depth?: "direct" | "deep",
+      depthOrOptions?:
+        | "direct"
+        | "deep"
+        | { depth?: "direct" | "deep"; control?: CodeRequestControl },
     ) => Promise<{
       kind: string;
       data?: {

package/src/analysis/search/ast-scan-timing.ts

@@ -1,9 +1,10 @@
+import type { CodeRequestControl } from "@mrclrchtr/supi-code-runtime/api";
 import { startDebugTimer } from "@mrclrchtr/supi-core/debug";
+import { truncateIdentity } from "@mrclrchtr/supi-lsp/debug-telemetry";
 import type { StructuralSearchOperation } from "@mrclrchtr/supi-tree-sitter/api";
 import type { StructuredFileAnalysis, StructuredPatternParams } from "./pattern-analysis.ts";
 
 interface ScanContext {
-  readonly cwd: string;
   readonly params: StructuredPatternParams;
 }
 
@@ -26,8 +27,11 @@ export interface AstScanTimer {
   record(input: AstScanTimingInput): void;
 }
 
-/** Start one AST scan timer without adding paths to debug-event data. */
-export function startAstScanTimer(): AstScanTimer {
+/** Start one AST scan timer, recording the workspace root as event-level cwd. */
+export function startAstScanTimer(
+  control: CodeRequestControl | undefined,
+  cwd: string,
+): AstScanTimer {
   const timer = startDebugTimer();
   return {
     enumerationCompleted() {
@@ -36,11 +40,12 @@ export function startAstScanTimer(): AstScanTimer {
     record(input) {
       timer.finish(
         () => ({
+          operationId: control?.operationId,
           source: "code-intelligence",
           level: "debug",
           category: "ast-scan.timing",
           message: `AST ${input.context.params.kind} scan analyzed ${input.analysis.analyzedFileCount} files`,
-          cwd: input.context.cwd,
+          cwd: truncateIdentity(cwd),
           data: {
             kind: input.context.params.kind,
             operation: input.operation,

package/src/analysis/search/ast-scan.ts

@@ -292,7 +292,7 @@ class AstFileEnumerator {
   #checkControl(): boolean {
     this.options.signal?.throwIfAborted();
     if (this.#safetyLimited) return false;
-    if (this.#now() <= this.options.deadline) return true;
+    if (this.#now() < this.options.deadline) return true;
     this.#markTimeout();
     return false;
   }

package/src/analysis/search/deadline.ts

@@ -24,7 +24,7 @@ export async function settleByDeadline<T>(
 ): Promise<DeadlineOutcome<T>> {
   control.signal?.throwIfAborted();
   const remainingMs = control.deadline - control.now();
-  if (remainingMs < 0) return { kind: "timeout" };
+  if (remainingMs <= 0) return { kind: "timeout" };
 
   return new Promise<DeadlineOutcome<T>>((resolve, reject) => {
     let settled = false;
@@ -70,7 +70,7 @@ export async function settleByDeadline<T>(
       .then(
         (value) => {
           complete(
-            control.now() > control.deadline ? { kind: "timeout" } : { kind: "completed", value },
+            control.now() >= control.deadline ? { kind: "timeout" } : { kind: "completed", value },
           );
         },
         (error: unknown) => fail(error),