@siteping/adapter-drizzle 0.1.0 → 0.2.0

package/README.md

@@ -14,7 +14,7 @@
 npm install @siteping/adapter-drizzle @siteping/server drizzle-orm
 ```
 
-**Peer dependency:** `drizzle-orm` `>=0.45 <1` · Node ≥ 20.
+**Peer dependency:** `drizzle-orm` `>=0.45 <1` · Node ≥ 22.
 
 ## Quick start
 

package/dist/{chunk-6G6UAERF.js → chunk-6DO75JHR.js}

@@ -41,6 +41,13 @@ var StoreLimitError = class extends Error {
     this.name = "StoreLimitError";
   }
 };
+var StoreValueTooLongError = class extends Error {
+  code = "STORE_VALUE_TOO_LONG";
+  constructor(message = "Value too long for the store", options) {
+    super(message, options);
+    this.name = "StoreValueTooLongError";
+  }
+};
 function hasErrorCode(error, code) {
   return hasOwn(error, "code") && error.code === code;
 }
@@ -56,6 +63,14 @@ function isStorePersistence(error) {
   if (error instanceof StorePersistenceError) return true;
   return hasErrorCode(error, "STORE_PERSISTENCE");
 }
+function isStoreLimit(error) {
+  if (error instanceof StoreLimitError) return true;
+  return hasErrorCode(error, "STORE_LIMIT");
+}
+function isStoreValueTooLong(error) {
+  if (error instanceof StoreValueTooLongError) return true;
+  return hasErrorCode(error, "STORE_VALUE_TOO_LONG");
+}
 
 // ../core/src/concurrency.ts
 async function settleWithConcurrencyLimit(items, concurrency, task) {
@@ -92,6 +107,15 @@ function isUnreachableOffset(skip) {
   return !Number.isSafeInteger(skip);
 }
 
+// ../core/src/logger.ts
+function warnSafely(logger, message, context) {
+  try {
+    Promise.resolve(logger.warn(message, context)).catch(() => {
+    });
+  } catch {
+  }
+}
+
 // ../core/src/screenshot-storage.ts
 var SCREENSHOT_DELETE_CONCURRENCY = 32;
 function screenshotMimeType(dataUrl) {
@@ -164,6 +188,26 @@ function buildFeedbackRecord(input, ctx) {
   };
 }
 
+// ../core/src/text.ts
+var UNSTORABLE_CODE_UNITS = /\0|[\uD800-\uDBFF](?![\uDC00-\uDFFF])|(?<![\uD800-\uDBFF])[\uDC00-\uDFFF]/g;
+var REPLACEMENT_CHARACTER = "\uFFFD";
+function toStorableText(text) {
+  return text.replace(UNSTORABLE_CODE_UNITS, REPLACEMENT_CHARACTER);
+}
+function isPlainObject(value) {
+  if (typeof value !== "object" || value === null) return false;
+  const prototype = Object.getPrototypeOf(value);
+  return prototype === Object.prototype || prototype === null;
+}
+function toStorableValue(value) {
+  if (typeof value === "string") return toStorableText(value);
+  if (Array.isArray(value)) return value.map(toStorableValue);
+  if (!isPlainObject(value)) return value;
+  return Object.fromEntries(
+    Object.entries(value).map(([key, item]) => [toStorableText(key), toStorableValue(item)])
+  );
+}
+
 // src/constants/errors.ts
 var DRIZZLE_STORE_MESSAGE_PREFIX = "[siteping] DrizzleStore";
 var DRIZZLE_QUERY_ERROR_MESSAGE_PREFIX = "Failed query:";
@@ -229,28 +273,6 @@ function withDriverErrors(gateway) {
   });
 }
 
-// src/constants/text.ts
-var UNSTORABLE_CODE_UNITS = /\0|[\uD800-\uDBFF](?![\uDC00-\uDFFF])|(?<![\uD800-\uDBFF])[\uDC00-\uDFFF]/g;
-var REPLACEMENT_CHARACTER = "\uFFFD";
-
-// src/shared/text.ts
-function toStorableText(text) {
-  return text.replace(UNSTORABLE_CODE_UNITS, REPLACEMENT_CHARACTER);
-}
-function isPlainObject(value) {
-  if (typeof value !== "object" || value === null) return false;
-  const prototype = Object.getPrototypeOf(value);
-  return prototype === Object.prototype || prototype === null;
-}
-function toStorableValue(value) {
-  if (typeof value === "string") return toStorableText(value);
-  if (Array.isArray(value)) return value.map(toStorableValue);
-  if (!isPlainObject(value)) return value;
-  return Object.fromEntries(
-    Object.entries(value).map(([key, item]) => [toStorableText(key), toStorableValue(item)])
-  );
-}
-
 // src/shared/store.ts
 var silentLogger = {
   warn() {
@@ -303,8 +325,11 @@ var DrizzleSitepingStore = class {
     this.logger = options.logger ?? silentLogger;
     this.now = options.now ?? systemClock;
   }
+  /** Throws `StoreDuplicateError` when a feedback with the same `clientId` exists. */
   async createFeedback(data) {
-    return (await this.createFeedbackIfAbsent(data)).feedback;
+    const { feedback, created } = await this.createFeedbackIfAbsent(data);
+    if (!created) throw new StoreDuplicateError(`A feedback with clientId ${data.clientId} already exists`);
+    return feedback;
   }
   /**
    * Insert the feedback unless a row with the same `clientId` exists. The
@@ -486,7 +511,15 @@ var DrizzleSitepingStore = class {
    * @throws `StoreLimitError` when a `client` comment meets a thread already holding `MAX_COMMENTS_PER_FEEDBACK` of them.
    * @throws `StorePersistenceError` when a database call fails.
    */
-  async addComment(submittedFeedbackId, submitted) {
+  async addComment(feedbackId, data) {
+    return (await this.addCommentIfAbsent(feedbackId, data)).comment;
+  }
+  /**
+   * {@link addComment}, reporting whether this call inserted the comment: the
+   * single statement inserts at most one row per `clientId`, so of racing
+   * posts exactly one reports `created: true`.
+   */
+  async addCommentIfAbsent(submittedFeedbackId, submitted) {
     const feedbackId = toStorableText(submittedFeedbackId);
     const data = toStorableValue(submitted);
     const comment = buildCommentRecord(data, { id: crypto.randomUUID(), feedbackId, now: this.now() });
@@ -507,13 +540,13 @@ var DrizzleSitepingStore = class {
       if (feedbackGone) throw new StoreNotFoundError();
       throw error;
     }
-    if (inserted) return comment;
+    if (inserted) return { comment, created: true };
     const replayed = await persistMutation(
       "addComment",
       identifiers,
       () => this.gateway.findCommentByClientId(data.clientId)
     );
-    if (replayed) return replayed;
+    if (replayed) return { comment: replayed, created: false };
     const projectName = await persistMutation(
       "addComment",
       identifiers,
@@ -580,7 +613,8 @@ var DrizzleSitepingStore = class {
         });
         return url;
       } catch (error) {
-        this.logger.warn(
+        warnSafely(
+          this.logger,
           "[siteping] DrizzleStore: screenshotStorage.upload failed \u2014 feedback saved without screenshot",
           { clientId, feedbackId, error }
         );
@@ -589,7 +623,8 @@ var DrizzleSitepingStore = class {
     }
     if (!this.inlineScreenshotWarned) {
       this.inlineScreenshotWarned = true;
-      this.logger.warn(
+      warnSafely(
+        this.logger,
         "[siteping] DrizzleStore: no screenshotStorage configured \u2014 screenshots are stored inline as base64. Configure a ScreenshotStorage for production.",
         {}
       );
@@ -632,7 +667,8 @@ var DrizzleSitepingStore = class {
     try {
       referenced = await this.findReferencedScreenshotUrls(candidates);
     } catch (lookupError) {
-      this.logger.warn(
+      warnSafely(
+        this.logger,
         `${DRIZZLE_STORE_MESSAGE_PREFIX}: screenshot references could not be checked \u2014 screenshots kept`,
         { ...context, screenshotUrls: candidates, error: lookupError }
       );
@@ -646,11 +682,15 @@ var DrizzleSitepingStore = class {
     );
     results.forEach((result, index) => {
       if (result.status === "rejected") {
-        this.logger.warn(`${DRIZZLE_STORE_MESSAGE_PREFIX}: screenshotStorage.delete failed \u2014 object left in place`, {
-          ...context,
-          screenshotUrl: unreferenced[index],
-          error: result.reason
-        });
+        warnSafely(
+          this.logger,
+          `${DRIZZLE_STORE_MESSAGE_PREFIX}: screenshotStorage.delete failed \u2014 object left in place`,
+          {
+            ...context,
+            screenshotUrl: unreferenced[index],
+            error: result.reason
+          }
+        );
       }
     });
   }
@@ -792,7 +832,12 @@ export {
   StoreDuplicateError,
   StorePersistenceError,
   StoreLimitError,
+  StoreValueTooLongError,
+  isStoreNotFound,
+  isStoreDuplicate,
   isStorePersistence,
+  isStoreLimit,
+  isStoreValueTooLong,
   DRIZZLE_STORE_MESSAGE_PREFIX,
   DrizzleSitepingStore,
   CASE_INSENSITIVE_LIKE_OPERATOR,
@@ -811,4 +856,4 @@ export {
   selectValues,
   monotonicUpdatedAt
 };
-//# sourceMappingURL=chunk-6G6UAERF.js.map
+//# sourceMappingURL=chunk-6DO75JHR.js.map