@techstream/quark-core 1.2.0 → 1.4.0

package/src/email.test.js

@@ -1,133 +1,146 @@
 import assert from "node:assert";
-import { mock, test } from "node:test";
+import { test } from "node:test";
 import { createEmailService } from "./email.js";
 
 test("Email Service", async (t) => {
-	await t.test("createEmailService returns object with sendEmail method", () => {
-		const service = createEmailService();
-		assert.strictEqual(typeof service, "object");
-		assert.strictEqual(typeof service.sendEmail, "function");
-	});
-
-	await t.test("SMTP provider: uses Mailhog defaults when no SMTP_HOST set", async () => {
-		// Clear any explicit SMTP env vars
-		const origHost = process.env.SMTP_HOST;
-		const origPort = process.env.SMTP_PORT;
-		delete process.env.SMTP_HOST;
-		delete process.env.SMTP_PORT;
-
-		try {
-			const service = createEmailService({ provider: "smtp" });
-
-			// Mock nodemailer at the transport level
-			let capturedConfig = null;
-			const mockTransport = {
-				sendMail: async (opts) => {
-					return { messageId: "test-id-123", ...opts };
-				},
-			};
-
-			// We test that sendEmail resolves without error
-			// The transport is lazily created, so sendEmail triggers creation
-			// We can't easily intercept the dynamic import, but we can verify
-			// the service was created correctly
-			assert.strictEqual(typeof service.sendEmail, "function");
-		} finally {
-			if (origHost !== undefined) process.env.SMTP_HOST = origHost;
-			else delete process.env.SMTP_HOST;
-			if (origPort !== undefined) process.env.SMTP_PORT = origPort;
-			else delete process.env.SMTP_PORT;
-		}
-	});
-
-	await t.test("SMTP provider: uses explicit SMTP_HOST/SMTP_PORT when set", () => {
-		const origHost = process.env.SMTP_HOST;
-		const origPort = process.env.SMTP_PORT;
-		process.env.SMTP_HOST = "mail.example.com";
-		process.env.SMTP_PORT = "587";
-
-		try {
-			const service = createEmailService({ provider: "smtp" });
+	await t.test(
+		"createEmailService returns object with sendEmail method",
+		() => {
+			const service = createEmailService();
+			assert.strictEqual(typeof service, "object");
 			assert.strictEqual(typeof service.sendEmail, "function");
-		} finally {
-			if (origHost !== undefined) process.env.SMTP_HOST = origHost;
-			else delete process.env.SMTP_HOST;
-			if (origPort !== undefined) process.env.SMTP_PORT = origPort;
-			else delete process.env.SMTP_PORT;
-		}
-	});
-
-	await t.test("Resend provider: throws if RESEND_API_KEY missing", async () => {
-		const origKey = process.env.RESEND_API_KEY;
-		delete process.env.RESEND_API_KEY;
-
-		try {
-			const service = createEmailService({ provider: "resend" });
-
-			await assert.rejects(
-				() => service.sendEmail("test@example.com", "Subject", "<p>body</p>"),
-				(err) =>
-					err instanceof Error &&
-					/RESEND_API_KEY/.test(err.message),
-			);
-		} finally {
-			if (origKey !== undefined) process.env.RESEND_API_KEY = origKey;
-			else delete process.env.RESEND_API_KEY;
-		}
-	});
-
-	await t.test("Resend provider: calls fetch with correct URL, headers, and body", async () => {
-		const origKey = process.env.RESEND_API_KEY;
-		process.env.RESEND_API_KEY = "re_test_key_123";
-
-		// Mock global fetch
-		const originalFetch = globalThis.fetch;
-		let capturedUrl = null;
-		let capturedOptions = null;
-
-		globalThis.fetch = async (url, opts) => {
-			capturedUrl = url;
-			capturedOptions = opts;
-			return {
-				ok: true,
-				json: async () => ({ id: "resend-msg-id" }),
+		},
+	);
+
+	await t.test(
+		"SMTP provider: uses Mailhog defaults when no SMTP_HOST set",
+		async () => {
+			// Clear any explicit SMTP env vars
+			const origHost = process.env.SMTP_HOST;
+			const origPort = process.env.SMTP_PORT;
+			delete process.env.SMTP_HOST;
+			delete process.env.SMTP_PORT;
+
+			try {
+				const service = createEmailService({ provider: "smtp" });
+
+				// Mock nodemailer at the transport level
+				const _capturedConfig = null;
+				const _mockTransport = {
+					sendMail: async (opts) => {
+						return { messageId: "test-id-123", ...opts };
+					},
+				};
+
+				// We test that sendEmail resolves without error
+				// The transport is lazily created, so sendEmail triggers creation
+				// We can't easily intercept the dynamic import, but we can verify
+				// the service was created correctly
+				assert.strictEqual(typeof service.sendEmail, "function");
+			} finally {
+				if (origHost !== undefined) process.env.SMTP_HOST = origHost;
+				else delete process.env.SMTP_HOST;
+				if (origPort !== undefined) process.env.SMTP_PORT = origPort;
+				else delete process.env.SMTP_PORT;
+			}
+		},
+	);
+
+	await t.test(
+		"SMTP provider: uses explicit SMTP_HOST/SMTP_PORT when set",
+		() => {
+			const origHost = process.env.SMTP_HOST;
+			const origPort = process.env.SMTP_PORT;
+			process.env.SMTP_HOST = "mail.example.com";
+			process.env.SMTP_PORT = "587";
+
+			try {
+				const service = createEmailService({ provider: "smtp" });
+				assert.strictEqual(typeof service.sendEmail, "function");
+			} finally {
+				if (origHost !== undefined) process.env.SMTP_HOST = origHost;
+				else delete process.env.SMTP_HOST;
+				if (origPort !== undefined) process.env.SMTP_PORT = origPort;
+				else delete process.env.SMTP_PORT;
+			}
+		},
+	);
+
+	await t.test(
+		"Resend provider: throws if RESEND_API_KEY missing",
+		async () => {
+			const origKey = process.env.RESEND_API_KEY;
+			delete process.env.RESEND_API_KEY;
+
+			try {
+				const service = createEmailService({ provider: "resend" });
+
+				await assert.rejects(
+					() => service.sendEmail("test@example.com", "Subject", "<p>body</p>"),
+					(err) => err instanceof Error && /RESEND_API_KEY/.test(err.message),
+				);
+			} finally {
+				if (origKey !== undefined) process.env.RESEND_API_KEY = origKey;
+				else delete process.env.RESEND_API_KEY;
+			}
+		},
+	);
+
+	await t.test(
+		"Resend provider: calls fetch with correct URL, headers, and body",
+		async () => {
+			const origKey = process.env.RESEND_API_KEY;
+			process.env.RESEND_API_KEY = "re_test_key_123";
+
+			// Mock global fetch
+			const originalFetch = globalThis.fetch;
+			let capturedUrl = null;
+			let capturedOptions = null;
+
+			globalThis.fetch = async (url, opts) => {
+				capturedUrl = url;
+				capturedOptions = opts;
+				return {
+					ok: true,
+					json: async () => ({ id: "resend-msg-id" }),
+				};
 			};
-		};
-
-		try {
-			const service = createEmailService({
-				provider: "resend",
-				from: "Test <test@example.com>",
-			});
-
-			const result = await service.sendEmail(
-				"recipient@example.com",
-				"Test Subject",
-				"<p>Hello</p>",
-				"Hello",
-			);
-
-			assert.strictEqual(capturedUrl, "https://api.resend.com/emails");
-			assert.strictEqual(capturedOptions.method, "POST");
-
-			const headers = capturedOptions.headers;
-			assert.strictEqual(headers.Authorization, "Bearer re_test_key_123");
-			assert.strictEqual(headers["Content-Type"], "application/json");
 
-			const body = JSON.parse(capturedOptions.body);
-			assert.strictEqual(body.from, "Test <test@example.com>");
-			assert.deepStrictEqual(body.to, ["recipient@example.com"]);
-			assert.strictEqual(body.subject, "Test Subject");
-			assert.strictEqual(body.html, "<p>Hello</p>");
-			assert.strictEqual(body.text, "Hello");
-
-			assert.strictEqual(result.id, "resend-msg-id");
-		} finally {
-			globalThis.fetch = originalFetch;
-			if (origKey !== undefined) process.env.RESEND_API_KEY = origKey;
-			else delete process.env.RESEND_API_KEY;
-		}
-	});
+			try {
+				const service = createEmailService({
+					provider: "resend",
+					from: "Test <test@example.com>",
+				});
+
+				const result = await service.sendEmail(
+					"recipient@example.com",
+					"Test Subject",
+					"<p>Hello</p>",
+					"Hello",
+				);
+
+				assert.strictEqual(capturedUrl, "https://api.resend.com/emails");
+				assert.strictEqual(capturedOptions.method, "POST");
+
+				const headers = capturedOptions.headers;
+				assert.strictEqual(headers.Authorization, "Bearer re_test_key_123");
+				assert.strictEqual(headers["Content-Type"], "application/json");
+
+				const body = JSON.parse(capturedOptions.body);
+				assert.strictEqual(body.from, "Test <test@example.com>");
+				assert.deepStrictEqual(body.to, ["recipient@example.com"]);
+				assert.strictEqual(body.subject, "Test Subject");
+				assert.strictEqual(body.html, "<p>Hello</p>");
+				assert.strictEqual(body.text, "Hello");
+
+				assert.strictEqual(result.id, "resend-msg-id");
+			} finally {
+				globalThis.fetch = originalFetch;
+				if (origKey !== undefined) process.env.RESEND_API_KEY = origKey;
+				else delete process.env.RESEND_API_KEY;
+			}
+		},
+	);
 
 	await t.test("Resend provider: throws on non-ok response", async () => {
 		const origKey = process.env.RESEND_API_KEY;
@@ -196,8 +209,7 @@ test("Email Service", async (t) => {
 			// The service should be created without error (defaults to smtp)
 			assert.strictEqual(typeof service.sendEmail, "function");
 		} finally {
-			if (origProvider !== undefined)
-				process.env.EMAIL_PROVIDER = origProvider;
+			if (origProvider !== undefined) process.env.EMAIL_PROVIDER = origProvider;
 			else delete process.env.EMAIL_PROVIDER;
 		}
 	});
@@ -209,7 +221,7 @@ test("Email Service", async (t) => {
 		const originalFetch = globalThis.fetch;
 		let capturedBody = null;
 
-		globalThis.fetch = async (url, opts) => {
+		globalThis.fetch = async (_url, opts) => {
 			capturedBody = JSON.parse(opts.body);
 			return { ok: true, json: async () => ({ id: "msg-1" }) };
 		};

package/src/index.js

@@ -7,6 +7,10 @@ export * from "./authorization.js";
 export * from "./cache.js";
 // CSRF protection exports
 export * from "./csrf.js";
+// Email service exports
+export * from "./email.js";
+// Email template exports
+export * from "./email-templates.js";
 // Error Reporter exports
 export * from "./error-reporter.js";
 // Error exports
@@ -15,8 +19,6 @@ export * from "./errors.js";
 export * from "./logger.js";
 // Mailhog exports
 export * from "./mailhog.js";
-// Email service exports
-export * from "./email.js";
 // Queue exports
 export * from "./queue/index.js";
 // Rate limiting exports

package/src/mailhog.js

@@ -30,8 +30,8 @@ function getMailhogUiUrl() {
 export const getMailhogSmtpConfig = () => {
 	const url = getMailhogSmtpUrl();
 	const match = url.match(/smtp:\/\/([^:]+):(\d+)/);
-	const host = match ? match[1] : (process.env.MAILHOG_HOST || "localhost");
-	const port = match ? match[2] : (process.env.MAILHOG_SMTP_PORT || "1025");
+	const host = match ? match[1] : process.env.MAILHOG_HOST || "localhost";
+	const port = match ? match[2] : process.env.MAILHOG_SMTP_PORT || "1025";
 
 	return {
 		host,

package/src/rate-limiter.js

@@ -143,7 +143,7 @@ class RedisRateLimiter {
 				remaining: maxRequests - currentCount,
 				resetTime,
 			};
-		} catch (error) {
+		} catch (_error) {
 			// Configurable fail-open / fail-closed behaviour
 			if (this.failOpen) {
 				return {

package/.turbo/turbo-lint.log

@@ -1,7 +0,0 @@
-
-
-> @quark/core@1.0.0 lint /Users/david/Projects/quark/packages/core
-> biome format --write && biome check --write
-
-Formatted 17 files in 40ms. No fixes applied.
-Checked 17 files in 64ms. No fixes applied.