agor-live 0.7.1 → 0.7.2

package/dist/core/permissions/index.cjs

@@ -50,9 +50,10 @@ var PermissionService = class {
    *
    * @param requestId - Unique permission request ID
    * @param taskId - Task waiting for permission (used for timeout/cancel fallback)
+   * @param sessionId - Session ID for tracking pending requests per session
    * @param signal - AbortSignal for cancellation
    */
-  waitForDecision(requestId, taskId, signal) {
+  waitForDecision(requestId, taskId, sessionId, signal) {
     return new Promise((resolve) => {
       signal.addEventListener("abort", () => {
         const pending = this.pendingRequests.get(requestId);
@@ -86,7 +87,7 @@ var PermissionService = class {
           // System-initiated timeout
         });
       }, 6e4);
-      this.pendingRequests.set(requestId, { resolve, timeout });
+      this.pendingRequests.set(requestId, { sessionId, resolve, timeout });
     });
   }
   /**
@@ -105,6 +106,37 @@ var PermissionService = class {
       console.warn(`\u26A0\uFE0F  No pending request found for ${decision.requestId}`);
     }
   }
+  /**
+   * Cancel all pending permission requests for a session
+   * Used when a permission is denied and we need to stop task execution
+   *
+   * @param sessionId - Session to cancel pending requests for
+   */
+  cancelPendingRequests(sessionId) {
+    let cancelledCount = 0;
+    for (const [requestId, pending] of this.pendingRequests.entries()) {
+      if (pending.sessionId === sessionId) {
+        clearTimeout(pending.timeout);
+        pending.resolve({
+          requestId,
+          taskId: "",
+          // Will be ignored since we're cancelling
+          allow: false,
+          reason: "Cancelled due to previous permission denial",
+          remember: false,
+          scope: "once" /* ONCE */,
+          decidedBy: "system"
+        });
+        this.pendingRequests.delete(requestId);
+        cancelledCount++;
+      }
+    }
+    if (cancelledCount > 0) {
+      console.log(
+        `\u{1F6E1}\uFE0F  Cancelled ${cancelledCount} pending permission request(s) for session ${sessionId.substring(0, 8)}`
+      );
+    }
+  }
 };
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {

package/dist/core/permissions/index.d.cts

@@ -69,13 +69,21 @@ declare class PermissionService {
      *
      * @param requestId - Unique permission request ID
      * @param taskId - Task waiting for permission (used for timeout/cancel fallback)
+     * @param sessionId - Session ID for tracking pending requests per session
      * @param signal - AbortSignal for cancellation
      */
-    waitForDecision(requestId: string, taskId: TaskID, signal: AbortSignal): Promise<PermissionDecision>;
+    waitForDecision(requestId: string, taskId: TaskID, sessionId: SessionID, signal: AbortSignal): Promise<PermissionDecision>;
     /**
      * Resolve a pending permission request with a decision from the UI
      */
     resolvePermission(decision: PermissionDecision): void;
+    /**
+     * Cancel all pending permission requests for a session
+     * Used when a permission is denied and we need to stop task execution
+     *
+     * @param sessionId - Session to cancel pending requests for
+     */
+    cancelPendingRequests(sessionId: SessionID): void;
 }
 
 export { type PermissionDecision, type PermissionRequest, PermissionService };

package/dist/core/permissions/index.d.ts

@@ -69,13 +69,21 @@ declare class PermissionService {
      *
      * @param requestId - Unique permission request ID
      * @param taskId - Task waiting for permission (used for timeout/cancel fallback)
+     * @param sessionId - Session ID for tracking pending requests per session
      * @param signal - AbortSignal for cancellation
      */
-    waitForDecision(requestId: string, taskId: TaskID, signal: AbortSignal): Promise<PermissionDecision>;
+    waitForDecision(requestId: string, taskId: TaskID, sessionId: SessionID, signal: AbortSignal): Promise<PermissionDecision>;
     /**
      * Resolve a pending permission request with a decision from the UI
      */
     resolvePermission(decision: PermissionDecision): void;
+    /**
+     * Cancel all pending permission requests for a session
+     * Used when a permission is denied and we need to stop task execution
+     *
+     * @param sessionId - Session to cancel pending requests for
+     */
+    cancelPendingRequests(sessionId: SessionID): void;
 }
 
 export { type PermissionDecision, type PermissionRequest, PermissionService };

package/dist/core/permissions/index.js

@@ -24,9 +24,10 @@ var PermissionService = class {
    *
    * @param requestId - Unique permission request ID
    * @param taskId - Task waiting for permission (used for timeout/cancel fallback)
+   * @param sessionId - Session ID for tracking pending requests per session
    * @param signal - AbortSignal for cancellation
    */
-  waitForDecision(requestId, taskId, signal) {
+  waitForDecision(requestId, taskId, sessionId, signal) {
     return new Promise((resolve) => {
       signal.addEventListener("abort", () => {
         const pending = this.pendingRequests.get(requestId);
@@ -60,7 +61,7 @@ var PermissionService = class {
           // System-initiated timeout
         });
       }, 6e4);
-      this.pendingRequests.set(requestId, { resolve, timeout });
+      this.pendingRequests.set(requestId, { sessionId, resolve, timeout });
     });
   }
   /**
@@ -79,6 +80,37 @@ var PermissionService = class {
       console.warn(`\u26A0\uFE0F  No pending request found for ${decision.requestId}`);
     }
   }
+  /**
+   * Cancel all pending permission requests for a session
+   * Used when a permission is denied and we need to stop task execution
+   *
+   * @param sessionId - Session to cancel pending requests for
+   */
+  cancelPendingRequests(sessionId) {
+    let cancelledCount = 0;
+    for (const [requestId, pending] of this.pendingRequests.entries()) {
+      if (pending.sessionId === sessionId) {
+        clearTimeout(pending.timeout);
+        pending.resolve({
+          requestId,
+          taskId: "",
+          // Will be ignored since we're cancelling
+          allow: false,
+          reason: "Cancelled due to previous permission denial",
+          remember: false,
+          scope: "once" /* ONCE */,
+          decidedBy: "system"
+        });
+        this.pendingRequests.delete(requestId);
+        cancelledCount++;
+      }
+    }
+    if (cancelledCount > 0) {
+      console.log(
+        `\u{1F6E1}\uFE0F  Cancelled ${cancelledCount} pending permission request(s) for session ${sessionId.substring(0, 8)}`
+      );
+    }
+  }
 };
 export {
   PermissionService

package/dist/core/tools/index.cjs

@@ -408,7 +408,8 @@ var SDKMessageProcessor = class {
       enableTokenStreaming: options.enableTokenStreaming ?? true,
       idleTimeoutMs: options.idleTimeoutMs ?? 3e4,
       // 30s default
-      contentBlockStack: []
+      contentBlockStack: [],
+      toolInputChunkCount: 0
     };
   }
   /**
@@ -516,13 +517,11 @@ var SDKMessageProcessor = class {
     const hasText = Array.isArray(content) && content.some((b) => b.type === "text");
     if (hasToolResult) {
       const toolResults = content.filter((b) => b.type === "tool_result");
+      const errorCount = toolResults.filter((tr) => tr.is_error).length;
+      const successCount = toolResults.length - errorCount;
       console.log(
-        `\u{1F527} SDK user message with ${toolResults.length} tool result(s) (uuid: ${uuid?.substring(0, 8)})`
+        `\u{1F527} SDK user message with ${toolResults.length} tool result(s) (\u2705 ${successCount}, \u274C ${errorCount})`
       );
-      toolResults.forEach((tr, i) => {
-        const preview = typeof tr.content === "string" ? tr.content.substring(0, 100) : JSON.stringify(tr.content).substring(0, 100);
-        console.log(`   Result ${i + 1}: ${tr.is_error ? "\u274C ERROR" : "\u2705"} ${preview}`);
-      });
       return [
         {
           type: "complete",
@@ -630,9 +629,9 @@ var SDKMessageProcessor = class {
           agentSessionId: this.state.capturedAgentSessionId
         });
       } else if (delta?.type === "input_json_delta") {
-        const partialJson = delta.partial_json;
-        if (partialJson) {
-          console.debug(`\u{1F527} Tool input chunk: ${partialJson.substring(0, 50)}...`);
+        this.state.toolInputChunkCount++;
+        if (this.state.toolInputChunkCount % 10 === 0) {
+          console.debug(`\u{1F527} Tool input chunk (${this.state.toolInputChunkCount})`);
         }
       }
     }
@@ -1598,6 +1597,7 @@ function createPreToolUseHook(sessionId, taskId, deps) {
       const decision = await deps.permissionService.waitForDecision(
         requestId,
         taskId,
+        sessionId,
         options.signal
       );
       if (deps.messagesService) {
@@ -1618,11 +1618,23 @@ function createPreToolUseHook(sessionId, taskId, deps) {
       await deps.tasksService.patch(taskId, {
         status: decision.allow ? TaskStatus.RUNNING : TaskStatus.FAILED
       });
-      if (deps.sessionsService) {
-        await deps.sessionsService.patch(sessionId, {
-          status: "running"
-        });
-        console.log(`\u2705 Session ${sessionId} restored to running after permission decision`);
+      if (!decision.allow) {
+        console.log(`\u{1F6D1} Permission denied for ${input.tool_name}, stopping task execution...`);
+        deps.permissionService.cancelPendingRequests(sessionId);
+        if (deps.sessionsService) {
+          await deps.sessionsService.patch(sessionId, {
+            status: "idle"
+          });
+          console.log(`\u2705 Session ${sessionId} set to idle after permission denial`);
+        }
+        throw new Error(`Permission denied for tool: ${input.tool_name}`);
+      } else {
+        if (deps.sessionsService) {
+          await deps.sessionsService.patch(sessionId, {
+            status: "running"
+          });
+          console.log(`\u2705 Session ${sessionId} restored to running after permission approval`);
+        }
       }
       if (decision.remember) {
         const freshSession = await deps.sessionsRepo.findById(sessionId);
@@ -1684,21 +1696,18 @@ function createPreToolUseHook(sessionId, taskId, deps) {
       try {
         const errorMessage = error instanceof Error ? error.message : String(error);
         const timestamp = (/* @__PURE__ */ new Date()).toISOString();
-        await deps.tasksService.patch(taskId, {
-          status: TaskStatus.FAILED,
-          report: `Error: ${errorMessage}
+        const isPermissionDenial = error instanceof Error && error.message.startsWith("Permission denied for tool:");
+        if (!isPermissionDenial) {
+          await deps.tasksService.patch(taskId, {
+            status: TaskStatus.FAILED,
+            report: `Error: ${errorMessage}
 Timestamp: ${timestamp}`
-        });
+          });
+        }
       } catch (updateError) {
         console.error("Failed to update task status:", updateError);
       }
-      return {
-        hookSpecificOutput: {
-          hookEventName: "PreToolUse",
-          permissionDecision: "deny",
-          permissionDecisionReason: `Permission hook failed: ${error instanceof Error ? error.message : String(error)}`
-        }
-      };
+      throw error;
     } finally {
       if (releaseLock) {
         releaseLock();

package/dist/core/tools/index.js

@@ -350,7 +350,8 @@ var SDKMessageProcessor = class {
       enableTokenStreaming: options.enableTokenStreaming ?? true,
       idleTimeoutMs: options.idleTimeoutMs ?? 3e4,
       // 30s default
-      contentBlockStack: []
+      contentBlockStack: [],
+      toolInputChunkCount: 0
     };
   }
   /**
@@ -458,13 +459,11 @@ var SDKMessageProcessor = class {
     const hasText = Array.isArray(content) && content.some((b) => b.type === "text");
     if (hasToolResult) {
       const toolResults = content.filter((b) => b.type === "tool_result");
+      const errorCount = toolResults.filter((tr) => tr.is_error).length;
+      const successCount = toolResults.length - errorCount;
       console.log(
-        `\u{1F527} SDK user message with ${toolResults.length} tool result(s) (uuid: ${uuid?.substring(0, 8)})`
+        `\u{1F527} SDK user message with ${toolResults.length} tool result(s) (\u2705 ${successCount}, \u274C ${errorCount})`
       );
-      toolResults.forEach((tr, i) => {
-        const preview = typeof tr.content === "string" ? tr.content.substring(0, 100) : JSON.stringify(tr.content).substring(0, 100);
-        console.log(`   Result ${i + 1}: ${tr.is_error ? "\u274C ERROR" : "\u2705"} ${preview}`);
-      });
       return [
         {
           type: "complete",
@@ -572,9 +571,9 @@ var SDKMessageProcessor = class {
           agentSessionId: this.state.capturedAgentSessionId
         });
       } else if (delta?.type === "input_json_delta") {
-        const partialJson = delta.partial_json;
-        if (partialJson) {
-          console.debug(`\u{1F527} Tool input chunk: ${partialJson.substring(0, 50)}...`);
+        this.state.toolInputChunkCount++;
+        if (this.state.toolInputChunkCount % 10 === 0) {
+          console.debug(`\u{1F527} Tool input chunk (${this.state.toolInputChunkCount})`);
         }
       }
     }
@@ -1540,6 +1539,7 @@ function createPreToolUseHook(sessionId, taskId, deps) {
       const decision = await deps.permissionService.waitForDecision(
         requestId,
         taskId,
+        sessionId,
         options.signal
       );
       if (deps.messagesService) {
@@ -1560,11 +1560,23 @@ function createPreToolUseHook(sessionId, taskId, deps) {
       await deps.tasksService.patch(taskId, {
         status: decision.allow ? TaskStatus.RUNNING : TaskStatus.FAILED
       });
-      if (deps.sessionsService) {
-        await deps.sessionsService.patch(sessionId, {
-          status: "running"
-        });
-        console.log(`\u2705 Session ${sessionId} restored to running after permission decision`);
+      if (!decision.allow) {
+        console.log(`\u{1F6D1} Permission denied for ${input.tool_name}, stopping task execution...`);
+        deps.permissionService.cancelPendingRequests(sessionId);
+        if (deps.sessionsService) {
+          await deps.sessionsService.patch(sessionId, {
+            status: "idle"
+          });
+          console.log(`\u2705 Session ${sessionId} set to idle after permission denial`);
+        }
+        throw new Error(`Permission denied for tool: ${input.tool_name}`);
+      } else {
+        if (deps.sessionsService) {
+          await deps.sessionsService.patch(sessionId, {
+            status: "running"
+          });
+          console.log(`\u2705 Session ${sessionId} restored to running after permission approval`);
+        }
       }
       if (decision.remember) {
         const freshSession = await deps.sessionsRepo.findById(sessionId);
@@ -1626,21 +1638,18 @@ function createPreToolUseHook(sessionId, taskId, deps) {
       try {
         const errorMessage = error instanceof Error ? error.message : String(error);
         const timestamp = (/* @__PURE__ */ new Date()).toISOString();
-        await deps.tasksService.patch(taskId, {
-          status: TaskStatus.FAILED,
-          report: `Error: ${errorMessage}
+        const isPermissionDenial = error instanceof Error && error.message.startsWith("Permission denied for tool:");
+        if (!isPermissionDenial) {
+          await deps.tasksService.patch(taskId, {
+            status: TaskStatus.FAILED,
+            report: `Error: ${errorMessage}
 Timestamp: ${timestamp}`
-        });
+          });
+        }
       } catch (updateError) {
         console.error("Failed to update task status:", updateError);
       }
-      return {
-        hookSpecificOutput: {
-          hookEventName: "PreToolUse",
-          permissionDecision: "deny",
-          permissionDecisionReason: `Permission hook failed: ${error instanceof Error ? error.message : String(error)}`
-        }
-      };
+      throw error;
     } finally {
       if (releaseLock) {
         releaseLock();

package/dist/core/utils/errors.cjs

@@ -20,6 +20,11 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 // src/utils/errors.ts
 var errors_exports = {};
 __export(errors_exports, {
+  AgorError: () => AgorError,
+  AlreadyExistsError: () => AlreadyExistsError,
+  NotFoundError: () => NotFoundError,
+  UnauthorizedError: () => UnauthorizedError,
+  ValidationError: () => ValidationError,
   createErrorResponse: () => createErrorResponse,
   formatUserError: () => formatUserError,
   getErrorDetails: () => getErrorDetails,
@@ -29,6 +34,41 @@ __export(errors_exports, {
   logError: () => logError
 });
 module.exports = __toCommonJS(errors_exports);
+var AgorError = class extends Error {
+  constructor(message, code) {
+    super(message);
+    this.code = code;
+    this.name = this.constructor.name;
+    if (Error.captureStackTrace) {
+      Error.captureStackTrace(this, this.constructor);
+    }
+  }
+};
+var NotFoundError = class extends AgorError {
+  constructor(resourceType, id) {
+    super(`${resourceType} not found: ${id}`, "NOT_FOUND");
+    this.resourceType = resourceType;
+    this.id = id;
+  }
+  resourceType;
+  id;
+};
+var AlreadyExistsError = class extends AgorError {
+  constructor(resourceType, identifier) {
+    super(`${resourceType} already exists: ${identifier}`, "ALREADY_EXISTS");
+  }
+};
+var ValidationError = class extends AgorError {
+  constructor(message, field) {
+    super(message, "VALIDATION_ERROR");
+    this.field = field;
+  }
+};
+var UnauthorizedError = class extends AgorError {
+  constructor(message = "Not authorized") {
+    super(message, "UNAUTHORIZED");
+  }
+};
 function getErrorMessage(error) {
   if (error instanceof Error) {
     return error.message;
@@ -90,6 +130,11 @@ function logError(error, context, metadata = {}) {
 }
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
+  AgorError,
+  AlreadyExistsError,
+  NotFoundError,
+  UnauthorizedError,
+  ValidationError,
   createErrorResponse,
   formatUserError,
   getErrorDetails,

package/dist/core/utils/errors.d.cts

@@ -4,6 +4,43 @@
  * Provides standardized error formatting, logging, and categorization.
  * Used by daemon services and API handlers for uniform error responses.
  */
+/**
+ * Custom error classes for type-safe error handling
+ */
+/**
+ * Base class for all Agor errors
+ */
+declare class AgorError extends Error {
+    readonly code?: string | undefined;
+    constructor(message: string, code?: string | undefined);
+}
+/**
+ * Thrown when a resource is not found in the database
+ */
+declare class NotFoundError extends AgorError {
+    constructor(resourceType: string, id: string);
+    readonly resourceType: string;
+    readonly id: string;
+}
+/**
+ * Thrown when a resource already exists
+ */
+declare class AlreadyExistsError extends AgorError {
+    constructor(resourceType: string, identifier: string);
+}
+/**
+ * Thrown when validation fails
+ */
+declare class ValidationError extends AgorError {
+    readonly field?: string | undefined;
+    constructor(message: string, field?: string | undefined);
+}
+/**
+ * Thrown when an operation is not authorized
+ */
+declare class UnauthorizedError extends AgorError {
+    constructor(message?: string);
+}
 /**
  * Extract error message from any value
  * Handles Error objects, strings, and unknown types
@@ -48,4 +85,4 @@ declare function createErrorResponse(error: unknown, defaultMessage?: string): {
  */
 declare function logError(error: unknown, context: string, metadata?: Record<string, unknown>): void;
 
-export { createErrorResponse, formatUserError, getErrorDetails, getErrorMessage, handlePromiseError, isBusinessLogicError, logError };
+export { AgorError, AlreadyExistsError, NotFoundError, UnauthorizedError, ValidationError, createErrorResponse, formatUserError, getErrorDetails, getErrorMessage, handlePromiseError, isBusinessLogicError, logError };

package/dist/core/utils/errors.d.ts

@@ -4,6 +4,43 @@
  * Provides standardized error formatting, logging, and categorization.
  * Used by daemon services and API handlers for uniform error responses.
  */
+/**
+ * Custom error classes for type-safe error handling
+ */
+/**
+ * Base class for all Agor errors
+ */
+declare class AgorError extends Error {
+    readonly code?: string | undefined;
+    constructor(message: string, code?: string | undefined);
+}
+/**
+ * Thrown when a resource is not found in the database
+ */
+declare class NotFoundError extends AgorError {
+    constructor(resourceType: string, id: string);
+    readonly resourceType: string;
+    readonly id: string;
+}
+/**
+ * Thrown when a resource already exists
+ */
+declare class AlreadyExistsError extends AgorError {
+    constructor(resourceType: string, identifier: string);
+}
+/**
+ * Thrown when validation fails
+ */
+declare class ValidationError extends AgorError {
+    readonly field?: string | undefined;
+    constructor(message: string, field?: string | undefined);
+}
+/**
+ * Thrown when an operation is not authorized
+ */
+declare class UnauthorizedError extends AgorError {
+    constructor(message?: string);
+}
 /**
  * Extract error message from any value
  * Handles Error objects, strings, and unknown types
@@ -48,4 +85,4 @@ declare function createErrorResponse(error: unknown, defaultMessage?: string): {
  */
 declare function logError(error: unknown, context: string, metadata?: Record<string, unknown>): void;
 
-export { createErrorResponse, formatUserError, getErrorDetails, getErrorMessage, handlePromiseError, isBusinessLogicError, logError };
+export { AgorError, AlreadyExistsError, NotFoundError, UnauthorizedError, ValidationError, createErrorResponse, formatUserError, getErrorDetails, getErrorMessage, handlePromiseError, isBusinessLogicError, logError };

package/dist/core/utils/errors.js

@@ -1,4 +1,39 @@
 // src/utils/errors.ts
+var AgorError = class extends Error {
+  constructor(message, code) {
+    super(message);
+    this.code = code;
+    this.name = this.constructor.name;
+    if (Error.captureStackTrace) {
+      Error.captureStackTrace(this, this.constructor);
+    }
+  }
+};
+var NotFoundError = class extends AgorError {
+  constructor(resourceType, id) {
+    super(`${resourceType} not found: ${id}`, "NOT_FOUND");
+    this.resourceType = resourceType;
+    this.id = id;
+  }
+  resourceType;
+  id;
+};
+var AlreadyExistsError = class extends AgorError {
+  constructor(resourceType, identifier) {
+    super(`${resourceType} already exists: ${identifier}`, "ALREADY_EXISTS");
+  }
+};
+var ValidationError = class extends AgorError {
+  constructor(message, field) {
+    super(message, "VALIDATION_ERROR");
+    this.field = field;
+  }
+};
+var UnauthorizedError = class extends AgorError {
+  constructor(message = "Not authorized") {
+    super(message, "UNAUTHORIZED");
+  }
+};
 function getErrorMessage(error) {
   if (error instanceof Error) {
     return error.message;
@@ -59,6 +94,11 @@ function logError(error, context, metadata = {}) {
   }
 }
 export {
+  AgorError,
+  AlreadyExistsError,
+  NotFoundError,
+  UnauthorizedError,
+  ValidationError,
   createErrorResponse,
   formatUserError,
   getErrorDetails,

package/dist/daemon/adapters/drizzle.d.ts

@@ -40,6 +40,10 @@ interface DrizzleAdapterOptions {
      * Allow multi-record operations (patch/remove without ID)
      */
     multi?: boolean | string[];
+    /**
+     * Resource type name for error messages (e.g., 'Worktree', 'Session')
+     */
+    resourceType?: string;
 }
 /**
  * Repository interface that the adapter expects
@@ -63,6 +67,7 @@ declare class DrizzleService<T = any, D = Partial<T>, P extends Params = Params>
     id: string;
     paginate?: PaginationOptions;
     multi: boolean | string[];
+    resourceType: string;
     emit?: (event: string, ...args: any[]) => boolean;
     constructor(repository: Repository<T>, options?: DrizzleAdapterOptions);
     /**

package/dist/daemon/adapters/drizzle.js

@@ -1,14 +1,17 @@
 // src/adapters/drizzle.ts
+import { NotFoundError } from "@agor/core/utils/errors";
 var DrizzleService = class {
   constructor(repository, options = {}) {
     this.repository = repository;
     this.id = options.id ?? "id";
     this.paginate = options.paginate;
     this.multi = options.multi ?? false;
+    this.resourceType = options.resourceType ?? "Record";
   }
   id;
   paginate;
   multi;
+  resourceType;
   // Event emitter for FeathersJS (will be injected by framework)
   // biome-ignore lint/suspicious/noExplicitAny: FeathersJS event system
   emit;
@@ -121,7 +124,7 @@ var DrizzleService = class {
   async get(id, _params) {
     const result = await this.repository.findById(String(id));
     if (!result) {
-      throw new Error(`No record found for id '${id}'`);
+      throw new NotFoundError(this.resourceType, String(id));
     }
     return result;
   }
@@ -146,10 +149,7 @@ var DrizzleService = class {
    * Update a record (complete replacement)
    */
   async update(id, data, params) {
-    const existing = await this.get(id, params);
-    if (!existing) {
-      throw new Error(`No record found for id '${id}'`);
-    }
+    await this.get(id, params);
     const result = await this.repository.update(String(id), data);
     this.emit?.("updated", result, params);
     this.emit?.("patched", result, params);
@@ -179,10 +179,7 @@ var DrizzleService = class {
       }
       return results;
     }
-    const existing = await this.get(id, params);
-    if (!existing) {
-      throw new Error(`No record found for id '${id}'`);
-    }
+    await this.get(id, params);
     const result = await this.repository.update(String(id), data);
     this.emit?.("patched", result, params);
     return result;
@@ -205,9 +202,6 @@ var DrizzleService = class {
       return records;
     }
     const existing = await this.get(id, params);
-    if (!existing) {
-      throw new Error(`No record found for id '${id}'`);
-    }
     await this.repository.delete(String(id));
     this.emit?.("removed", existing, params);
     return existing;