@mulmoclaude/accounting-plugin 0.1.0 → 0.2.0

package/dist/server.js

@@ -1,4 +1,4 @@
-import { ACCOUNTING_ACTIONS, ACCOUNTING_BOOKS_CHANNEL, BOOK_EVENT_KINDS, FISCAL_YEAR_ENDS, SUPPORTED_COUNTRY_CODES, TIME_SERIES_GRANULARITIES, TIME_SERIES_METRICS, bookChannel, errorMessage, fiscalYearEndMonth, isFiscalYearEnd, isSupportedCountryCode } from "./shared.js";
+import { ACCOUNTING_ACTIONS, ACCOUNTING_API, ACCOUNTING_BOOKS_CHANNEL, ACCOUNTING_DIRS, BOOK_EVENT_KINDS, FISCAL_YEAR_ENDS, SUPPORTED_COUNTRY_CODES, TIME_SERIES_GRANULARITIES, TIME_SERIES_METRICS, bookChannel, errorMessage, fiscalYearEndMonth, isFiscalYearEnd, isSupportedCountryCode } from "./shared.js";
 import { Router } from "express";
 import { randomUUID } from "node:crypto";
 import { promises } from "node:fs";
@@ -30,26 +30,42 @@ var log = {
 	info: (namespace, msg, data) => (deps?.logger ?? consoleLogger).info(namespace, msg, data),
 	debug: (namespace, msg, data) => (deps?.logger ?? consoleLogger).debug(namespace, msg, data)
 };
-/** Workspace-relative directories this plugin owns. Mirrors the host
-*  META's `workspaceDirs` (kept in sync by value; the host META stays
-*  the codegen-discoverable source for the aggregator merge). */
-var ACCOUNTING_DIRS = {
-	accounting: "data/accounting",
-	accountingBooks: "data/accounting/books"
-};
 //#endregion
 //#region src/server/atomic.ts
 /** True for a `not found` filesystem error. */
 function isEnoent(err) {
 	return typeof err === "object" && err !== null && "code" in err && err.code === "ENOENT";
 }
-/** Atomic write: tmp alongside destination, then rename. */
+var IS_WINDOWS = process.platform === "win32";
+var RENAME_RETRY_DELAYS_MS = [
+	30,
+	100,
+	300
+];
+function hasErrnoCode(err) {
+	return typeof err === "object" && err !== null && "code" in err && typeof err.code === "string";
+}
+function isTransientRenameError(err) {
+	if (!IS_WINDOWS || !hasErrnoCode(err)) return false;
+	return err.code === "EPERM" || err.code === "EBUSY" || err.code === "EACCES";
+}
+async function renameWithWindowsRetry(fromPath, toPath) {
+	for (const delayMs of RENAME_RETRY_DELAYS_MS) try {
+		await promises.rename(fromPath, toPath);
+		return;
+	} catch (err) {
+		if (!isTransientRenameError(err)) throw err;
+		await new Promise((resolve) => setTimeout(resolve, delayMs));
+	}
+	await promises.rename(fromPath, toPath);
+}
+/** Atomic write: unique tmp alongside destination, then rename. */
 async function writeFileAtomic(filePath, content, opts = {}) {
-	const tmp = opts.uniqueTmp ? `${filePath}.${randomUUID()}.tmp` : `${filePath}.tmp`;
+	const tmp = opts.uniqueTmp ?? true ? `${filePath}.${randomUUID()}.tmp` : `${filePath}.tmp`;
 	await promises.mkdir(path.dirname(filePath), { recursive: true });
 	try {
 		await promises.writeFile(tmp, content, { encoding: "utf-8" });
-		await promises.rename(tmp, filePath);
+		await renameWithWindowsRetry(tmp, filePath);
 	} catch (err) {
 		await promises.unlink(tmp).catch(() => {});
 		throw err;
@@ -1827,7 +1843,7 @@ async function rebuildSnapshots(input, workspaceRoot) {
 //#endregion
 //#region src/server/http.ts
 function asyncHandler(namespace, fallbackMessage, handler) {
-	return async (req, res) => {
+	return async (req, res, next) => {
 		try {
 			await handler(req, res);
 		} catch (err) {
@@ -1837,7 +1853,11 @@ function asyncHandler(namespace, fallbackMessage, handler) {
 				route: expressReq.path,
 				error: errorMessage(err)
 			});
-			if (!expressRes.headersSent) expressRes.status(500).json({ error: fallbackMessage });
+			if (expressRes.headersSent) {
+				next(err);
+				return;
+			}
+			expressRes.status(500).json({ error: fallbackMessage });
 		}
 	};
 }
@@ -2037,7 +2057,7 @@ async function dispatch(body) {
 *  with `app.use(...)`. */
 function createAccountingRouter() {
 	const router = Router();
-	router.post("/api/accounting", asyncHandler("accounting", "accounting dispatch failed", async (req, res) => {
+	router.post(ACCOUNTING_API.dispatch.path, asyncHandler("accounting", "accounting dispatch failed", async (req, res) => {
 		const { body } = req;
 		if (!body || typeof body !== "object" || typeof body.action !== "string") {
 			log.warn("accounting", "POST dispatch: invalid body");