opticedge-cloud-utils 1.1.25 → 1.1.27

package/dist/task.js

@@ -81,9 +81,7 @@ async function createTask(projectId, region, queueId, data, serviceAccount, audi
         return response.name;
     };
     try {
-        const name = await (0, retry_1.retry)(() => doCreate(), retryOpts);
-        console.log(`✅ Created Cloud Task: ${name}`);
-        return name;
+        return await (0, retry_1.retry)(() => doCreate(), retryOpts);
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
     }
     catch (err) {

package/dist/validator.d.ts

@@ -1,3 +1,4 @@
 /** Basic email validation — good enough for most production use cases */
 declare const isValidEmail: (email: string) => boolean;
-export { isValidEmail };
+declare const isValidObjectId: (id: string) => boolean;
+export { isValidEmail, isValidObjectId };

package/dist/validator.js

@@ -1,6 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.isValidEmail = void 0;
+exports.isValidObjectId = exports.isValidEmail = void 0;
+const mongodb_1 = require("mongodb");
 /** Basic email validation — good enough for most production use cases */
 const isValidEmail = (email) => /^[^\s@]+@[^\s@]+\.[^\s@]+$/.test(email.trim());
 exports.isValidEmail = isValidEmail;
+const isValidObjectId = (id) => {
+    return mongodb_1.ObjectId.isValid(id) && new mongodb_1.ObjectId(id).toHexString() === id;
+};
+exports.isValidObjectId = isValidObjectId;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opticedge-cloud-utils",
-  "version": "1.1.25",
+  "version": "1.1.27",
   "description": "Common utilities for cloud functions",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/task.ts

@@ -106,9 +106,7 @@ export async function createTask(
   }
 
   try {
-    const name = await retry(() => doCreate(), retryOpts)
-    console.log(`✅ Created Cloud Task: ${name}`)
-    return name
+    return await retry(() => doCreate(), retryOpts)
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
   } catch (err: any) {
     // If ALREADY_EXISTS happened (and was not considered retryable), you may prefer to

package/src/validator.ts

@@ -1,4 +1,10 @@
+import { ObjectId } from 'mongodb'
+
 /** Basic email validation — good enough for most production use cases */
 const isValidEmail = (email: string) => /^[^\s@]+@[^\s@]+\.[^\s@]+$/.test(email.trim())
 
-export { isValidEmail }
+const isValidObjectId = (id: string): boolean => {
+  return ObjectId.isValid(id) && new ObjectId(id).toHexString() === id
+}
+
+export { isValidEmail, isValidObjectId }

package/tests/validator.test.ts

@@ -0,0 +1,69 @@
+import { describe, it, expect } from '@jest/globals'
+import { ObjectId } from 'mongodb'
+import { isValidEmail, isValidObjectId } from '../src'
+
+describe('isValidEmail', () => {
+  it('returns true for simple valid emails', () => {
+    expect(isValidEmail('test@example.com')).toBe(true)
+    expect(isValidEmail('user.name@domain.co')).toBe(true)
+    expect(isValidEmail('user_name+tag@sub.domain.org')).toBe(true)
+  })
+
+  it('returns false for missing parts', () => {
+    expect(isValidEmail('')).toBe(false)
+    expect(isValidEmail('plainaddress')).toBe(false)
+    expect(isValidEmail('@no-local-part.com')).toBe(false)
+    expect(isValidEmail('username@')).toBe(false)
+    expect(isValidEmail('user@domain')).toBe(false)
+    expect(isValidEmail('username@.com')).toBe(false)
+  })
+
+  it('handles whitespace correctly', () => {
+    expect(isValidEmail('   test@example.com   ')).toBe(true)
+    expect(isValidEmail('\nuser@domain.com\t')).toBe(true)
+  })
+
+  it('rejects clearly invalid characters or formats', () => {
+    expect(isValidEmail('user@@domain.com')).toBe(false)
+    expect(isValidEmail('user@domain,com')).toBe(false)
+    expect(isValidEmail('user@domain com')).toBe(false)
+  })
+
+  it('accepts minimal valid domain structures', () => {
+    expect(isValidEmail('x@y.z')).toBe(true)
+  })
+})
+
+describe('isValidObjectId', () => {
+  it('returns true for valid ObjectId strings', () => {
+    const id = new ObjectId().toHexString()
+    expect(isValidObjectId(id)).toBe(true)
+  })
+
+  it('returns false for invalid length', () => {
+    expect(isValidObjectId('123')).toBe(false)
+    expect(isValidObjectId('12345678901234567890123')).toBe(false)
+    expect(isValidObjectId('1234567890123456789012345')).toBe(false)
+  })
+
+  it('returns false for non-hex characters', () => {
+    expect(isValidObjectId('zzzzzzzzzzzzzzzzzzzzzzzz')).toBe(false)
+    expect(isValidObjectId('12345678901234567890123g')).toBe(false)
+  })
+
+  it('rejects 12-byte strings that ObjectId.isValid would accept', () => {
+    expect(isValidObjectId('123456789012')).toBe(false)
+  })
+
+  it('is case-sensitive to exact hex string match', () => {
+    const id = new ObjectId().toHexString()
+    expect(isValidObjectId(id.toUpperCase())).toBe(false)
+  })
+
+  it('returns false for empty or malformed input', () => {
+    expect(isValidObjectId('')).toBe(false)
+    expect(isValidObjectId('   ')).toBe(false)
+    expect(isValidObjectId('null')).toBe(false)
+    expect(isValidObjectId('undefined')).toBe(false)
+  })
+})

package/tests/validator.ts

@@ -1,37 +0,0 @@
-// isValidEmail.test.ts
-import { describe, it, expect } from '@jest/globals'
-import { isValidEmail } from '../src'
-
-describe('isValidEmail', () => {
-  it('returns true for simple valid emails', () => {
-    expect(isValidEmail('test@example.com')).toBe(true)
-    expect(isValidEmail('user.name@domain.co')).toBe(true)
-    expect(isValidEmail('user_name+tag@sub.domain.org')).toBe(true)
-  })
-
-  it('returns false for missing parts', () => {
-    expect(isValidEmail('')).toBe(false)
-    expect(isValidEmail('plainaddress')).toBe(false)
-    expect(isValidEmail('@no-local-part.com')).toBe(false)
-    expect(isValidEmail('username@')).toBe(false)
-    expect(isValidEmail('user@domain')).toBe(false)
-    expect(isValidEmail('username@.com')).toBe(false)
-  })
-
-  it('handles whitespace correctly', () => {
-    expect(isValidEmail('   test@example.com   ')).toBe(true) // trims input
-    expect(isValidEmail('\nuser@domain.com\t')).toBe(true)
-  })
-
-  it('rejects invalid characters or formats', () => {
-    expect(isValidEmail('user@@domain.com')).toBe(false)
-    expect(isValidEmail('user@domain,com')).toBe(false)
-    expect(isValidEmail('user@domain..com')).toBe(false)
-    expect(isValidEmail('user@.domain.com')).toBe(false)
-    expect(isValidEmail('user@domain com')).toBe(false)
-  })
-
-  it('accepts minimal valid domain structures', () => {
-    expect(isValidEmail('x@y.z')).toBe(true) // still valid
-  })
-})