@progressive-development/pd-provider-firebase-functions 0.3.2 → 0.3.4

package/dist/auth-trash.d.ts

@@ -13,7 +13,7 @@ export interface AuthTrashOptions {
  * Move document to trash AND disable the associated Auth user.
  *
  * Order of operations:
- * 1. Disable Auth user (if this fails, nothing else happens)
+ * 1. Disable Auth user (if user doesn't exist, continue anyway)
  * 2. Move document to trash
  *
  * @param collection - The collection path
@@ -30,7 +30,7 @@ export declare function moveToTrashWithAuth<T = unknown>(collection: string, doc
  * Order of operations:
  * 1. Get trash item to find Auth UID
  * 2. Restore document from trash
- * 3. Re-enable Auth user
+ * 3. Re-enable Auth user (if user doesn't exist, log warning)
  *
  * @param trashItemId - The trash document ID
  * @param options - Auth and database options
@@ -42,7 +42,7 @@ export declare function restoreFromTrashWithAuth(trashItemId: string, options?:
  * Order of operations:
  * 1. Get trash item to find Auth UID
  * 2. Delete from trash
- * 3. Delete Auth user
+ * 3. Delete Auth user (ignore if user doesn't exist)
  *
  * @param trashItemId - The trash document ID
  * @param options - Auth and database options

package/dist/auth-trash.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"auth-trash.d.ts","sourceRoot":"","sources":["../src/auth-trash.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EAAgE,KAAK,SAAS,EAAE,MAAM,SAAS,CAAC;AAQvG,MAAM,WAAW,gBAAgB;IAC/B;;;;OAIG;IACH,aAAa,CAAC,EAAE,YAAY,GAAG,MAAM,CAAC;IAEtC,8BAA8B;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAkCD;;;;;;;;;;;;;GAaG;AACH,wBAAsB,mBAAmB,CAAC,CAAC,GAAG,OAAO,EACnD,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,MAAM,EAClB,SAAS,EAAE,MAAM,EACjB,UAAU,EAAE,MAAM,EAClB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CA0BvB;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,wBAAwB,CAC5C,WAAW,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,IAAI,CAAC,CAsBf;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,uBAAuB,CAC3C,WAAW,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,IAAI,CAAC,CAqBf"}
+{"version":3,"file":"auth-trash.d.ts","sourceRoot":"","sources":["../src/auth-trash.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EAAgE,KAAK,SAAS,EAAE,MAAM,SAAS,CAAC;AAQvG,MAAM,WAAW,gBAAgB;IAC/B;;;;OAIG;IACH,aAAa,CAAC,EAAE,YAAY,GAAG,MAAM,CAAC;IAEtC,8BAA8B;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAkCD;;;;;;;;;;;;;GAaG;AACH,wBAAsB,mBAAmB,CAAC,CAAC,GAAG,OAAO,EACnD,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,MAAM,EAClB,SAAS,EAAE,MAAM,EACjB,UAAU,EAAE,MAAM,EAClB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAmCvB;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,wBAAwB,CAC5C,WAAW,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,IAAI,CAAC,CAiCf;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,uBAAuB,CAC3C,WAAW,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,IAAI,CAAC,CA6Bf"}

package/dist/auth-trash.js

@@ -18,8 +18,17 @@ function getAuthUidFromTrashItem(trashItem, options) {
 async function moveToTrashWithAuth(collection, documentId, deletedBy, objectType, options) {
   const authUid = options?.authUidSource === "documentId" || !options?.authUidSource ? documentId : documentId;
   logger.info("Disabling Auth user before trash", { authUid, collection, documentId });
-  await getAuth().updateUser(authUid, { disabled: true });
-  logger.info("Auth user disabled", { authUid });
+  try {
+    await getAuth().updateUser(authUid, { disabled: true });
+    logger.info("Auth user disabled", { authUid });
+  } catch (error) {
+    const authError = error;
+    if (authError.code === "auth/user-not-found") {
+      logger.info("Auth user not found, continuing with trash operation", { authUid });
+    } else {
+      throw error;
+    }
+  }
   const trashItem = await moveToTrash(
     collection,
     documentId,
@@ -45,8 +54,20 @@ async function restoreFromTrashWithAuth(trashItemId, options) {
     originalCollection: trashItem.originalCollection
   });
   await restoreFromTrash(trashItemId, options?.databaseId);
-  await getAuth().updateUser(authUid, { disabled: false });
-  logger.info("Document restored and Auth user re-enabled", { trashItemId, authUid });
+  try {
+    await getAuth().updateUser(authUid, { disabled: false });
+    logger.info("Document restored and Auth user re-enabled", { trashItemId, authUid });
+  } catch (error) {
+    const authError = error;
+    if (authError.code === "auth/user-not-found") {
+      logger.warn("Document restored but Auth user not found - manual Auth user creation required", {
+        trashItemId,
+        authUid
+      });
+    } else {
+      throw error;
+    }
+  }
 }
 async function permanentDeleteWithAuth(trashItemId, options) {
   const trashItem = await getTrashItem(trashItemId, options?.databaseId);
@@ -59,8 +80,17 @@ async function permanentDeleteWithAuth(trashItemId, options) {
     authUid
   });
   await permanentDelete(trashItemId, options?.databaseId);
-  await getAuth().deleteUser(authUid);
-  logger.info("Trash item and Auth user permanently deleted", { trashItemId, authUid });
+  try {
+    await getAuth().deleteUser(authUid);
+    logger.info("Trash item and Auth user permanently deleted", { trashItemId, authUid });
+  } catch (error) {
+    const authError = error;
+    if (authError.code === "auth/user-not-found") {
+      logger.info("Auth user not found, trash item deleted", { trashItemId, authUid });
+    } else {
+      throw error;
+    }
+  }
 }
 
 export { moveToTrashWithAuth, permanentDeleteWithAuth, restoreFromTrashWithAuth };

package/dist/errors.d.ts

@@ -15,4 +15,7 @@ export declare class ForbiddenError extends AppError {
 export declare class NotFoundError extends AppError {
     constructor(message: string);
 }
+export declare class ConflictError extends AppError {
+    constructor(message: string, code?: string);
+}
 //# sourceMappingURL=errors.d.ts.map

package/dist/errors.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"errors.d.ts","sourceRoot":"","sources":["../src/errors.ts"],"names":[],"mappings":"AAEA,qBAAa,QAAS,SAAQ,KAAK;aAGf,IAAI,EAAE,MAAM;aACZ,UAAU,EAAE,MAAM;gBAFlC,OAAO,EAAE,MAAM,EACC,IAAI,EAAE,MAAM,EACZ,UAAU,EAAE,MAAM;CAKrC;AAED,qBAAa,eAAgB,SAAQ,QAAQ;gBAC/B,OAAO,EAAE,MAAM;CAG5B;AAED,qBAAa,iBAAkB,SAAQ,QAAQ;gBACjC,OAAO,SAA4B;CAGhD;AAED,qBAAa,cAAe,SAAQ,QAAQ;gBAC9B,OAAO,SAAkB;CAGtC;AAED,qBAAa,aAAc,SAAQ,QAAQ;gBAC7B,OAAO,EAAE,MAAM;CAG5B"}
+{"version":3,"file":"errors.d.ts","sourceRoot":"","sources":["../src/errors.ts"],"names":[],"mappings":"AAEA,qBAAa,QAAS,SAAQ,KAAK;aAGf,IAAI,EAAE,MAAM;aACZ,UAAU,EAAE,MAAM;gBAFlC,OAAO,EAAE,MAAM,EACC,IAAI,EAAE,MAAM,EACZ,UAAU,EAAE,MAAM;CAKrC;AAED,qBAAa,eAAgB,SAAQ,QAAQ;gBAC/B,OAAO,EAAE,MAAM;CAG5B;AAED,qBAAa,iBAAkB,SAAQ,QAAQ;gBACjC,OAAO,SAA4B;CAGhD;AAED,qBAAa,cAAe,SAAQ,QAAQ;gBAC9B,OAAO,SAAkB;CAGtC;AAED,qBAAa,aAAc,SAAQ,QAAQ;gBAC7B,OAAO,EAAE,MAAM;CAG5B;AAED,qBAAa,aAAc,SAAQ,QAAQ;gBAC7B,OAAO,EAAE,MAAM,EAAE,IAAI,SAAa;CAG/C"}

package/dist/errors.js

@@ -26,5 +26,10 @@ class NotFoundError extends AppError {
     super(message, "NOT_FOUND", 404);
   }
 }
+class ConflictError extends AppError {
+  constructor(message, code = "CONFLICT") {
+    super(message, code, 409);
+  }
+}
 
-export { AppError, ForbiddenError, NotFoundError, UnauthorizedError, ValidationError };
+export { AppError, ConflictError, ForbiddenError, NotFoundError, UnauthorizedError, ValidationError };

package/dist/index.d.ts

@@ -1,6 +1,6 @@
 export type { ApiResponse, AuthContext } from './types';
 export { StatusCode } from './types';
-export { AppError, ValidationError, UnauthorizedError, ForbiddenError, NotFoundError } from './errors';
+export { AppError, ValidationError, UnauthorizedError, ForbiddenError, NotFoundError, ConflictError } from './errors';
 export { requireAuth, requireClaims, requireOwnership } from './auth';
 export { success, created, handleError } from './response';
 export { getDb } from './firestore';

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,YAAY,EAAE,WAAW,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AACxD,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AAGrC,OAAO,EAAE,QAAQ,EAAE,eAAe,EAAE,iBAAiB,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAGvG,OAAO,EAAE,WAAW,EAAE,aAAa,EAAE,gBAAgB,EAAE,MAAM,QAAQ,CAAC;AAGtE,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,MAAM,YAAY,CAAC;AAG3D,OAAO,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAGpC,OAAO,EAAE,cAAc,EAAE,MAAM,WAAW,CAAC;AAG3C,YAAY,EAAE,SAAS,EAAE,MAAM,SAAS,CAAC;AACzC,OAAO,EAAE,WAAW,EAAE,gBAAgB,EAAE,eAAe,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAGlG,YAAY,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AACrD,OAAO,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,uBAAuB,EAAE,MAAM,cAAc,CAAC;AAGtG,OAAO,EAAE,MAAM,EAAE,MAAM,UAAU,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,YAAY,EAAE,WAAW,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AACxD,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AAGrC,OAAO,EAAE,QAAQ,EAAE,eAAe,EAAE,iBAAiB,EAAE,cAAc,EAAE,aAAa,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAGtH,OAAO,EAAE,WAAW,EAAE,aAAa,EAAE,gBAAgB,EAAE,MAAM,QAAQ,CAAC;AAGtE,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,MAAM,YAAY,CAAC;AAG3D,OAAO,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAGpC,OAAO,EAAE,cAAc,EAAE,MAAM,WAAW,CAAC;AAG3C,YAAY,EAAE,SAAS,EAAE,MAAM,SAAS,CAAC;AACzC,OAAO,EAAE,WAAW,EAAE,gBAAgB,EAAE,eAAe,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAGlG,YAAY,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AACrD,OAAO,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,uBAAuB,EAAE,MAAM,cAAc,CAAC;AAGtG,OAAO,EAAE,MAAM,EAAE,MAAM,UAAU,CAAC"}

package/dist/index.js

@@ -1,5 +1,5 @@
 export { StatusCode } from './types.js';
-export { AppError, ForbiddenError, NotFoundError, UnauthorizedError, ValidationError } from './errors.js';
+export { AppError, ConflictError, ForbiddenError, NotFoundError, UnauthorizedError, ValidationError } from './errors.js';
 export { requireAuth, requireClaims, requireOwnership } from './auth.js';
 export { created, handleError, success } from './response.js';
 export { getDb } from './firestore.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@progressive-development/pd-provider-firebase-functions",
-  "version": "0.3.2",
+  "version": "0.3.4",
   "description": "Firebase Functions v2 utilities for pd-spa-helper backend",
   "author": "PD Progressive Development",
   "license": "SEE LICENSE IN LICENSE",