e-arveldaja-mcp 0.3.2 → 0.4.0

package/dist/account-validation.d.ts

@@ -0,0 +1,6 @@
+import type { Account } from "./types/api.js";
+export interface AccountValidationTarget {
+    id: number;
+    label: string;
+}
+export declare function validateAccounts(accounts: Account[], targets: AccountValidationTarget[]): string[];

package/dist/account-validation.js

@@ -0,0 +1,22 @@
+export function validateAccounts(accounts, targets) {
+    const accountMap = new Map(accounts.map(account => [account.id, account]));
+    const seen = new Set();
+    const errors = [];
+    for (const target of targets) {
+        const key = `${target.id}:${target.label}`;
+        if (seen.has(key))
+            continue;
+        seen.add(key);
+        const account = accountMap.get(target.id);
+        if (!account) {
+            errors.push(`${target.label} ${target.id} not found in chart of accounts. ` +
+                `Activate it in e-arveldaja: Seaded → Kontoplaan → find account ${target.id} and enable it.`);
+            continue;
+        }
+        if (!account.is_valid) {
+            errors.push(`${target.label} ${target.id} (${account.name_est}) is inactive. ` +
+                `Activate it in e-arveldaja: Seaded → Kontoplaan → ${account.name_est} → mark as active.`);
+        }
+    }
+    return errors;
+}

package/dist/api/purchase-invoices.api.d.ts

@@ -1,5 +1,5 @@
 import type { HttpClient } from "../http-client.js";
-import type { PurchaseInvoice, ApiResponse, ApiFile } from "../types/api.js";
+import type { PurchaseInvoice, CreatePurchaseInvoiceData, ApiResponse, ApiFile } from "../types/api.js";
 import { BaseResource } from "./base-resource.js";
 export declare class PurchaseInvoicesApi extends BaseResource<PurchaseInvoice> {
     constructor(client: HttpClient);
@@ -13,7 +13,7 @@ export declare class PurchaseInvoicesApi extends BaseResource<PurchaseInvoice> {
      * because input VAT is not deductible. gross_price is still set to actual payable amount.
      * Items get project_no_vat_gross_price set for VAT tracking.
      */
-    createAndSetTotals(data: Partial<PurchaseInvoice>, vatPrice?: number, grossPrice?: number, isVatRegistered?: boolean): Promise<PurchaseInvoice>;
+    createAndSetTotals(data: CreatePurchaseInvoiceData, vatPrice?: number, grossPrice?: number, isVatRegistered?: boolean): Promise<PurchaseInvoice>;
     /**
      * Confirm a purchase invoice. Automatically fixes vat_price/gross_price if missing or inconsistent.
      * For non-VAT companies: only fixes gross_price, leaves vat_price at 0.

package/dist/api/purchase-invoices.api.js

@@ -66,7 +66,7 @@ export class PurchaseInvoicesApi extends BaseResource {
         const gross = grossPrice !== undefined
             ? grossPrice
             : roundMoney(itemNet + itemVat);
-        if (vat > 0 || gross > 0) {
+        if (vat !== undefined || gross !== undefined) {
             await this.update(id, { vat_price: vat, gross_price: gross, items: invoice.items });
             this.invalidateCache();
         }
@@ -85,8 +85,10 @@ export class PurchaseInvoicesApi extends BaseResource {
             const vat = isVatRegistered ? itemVat : 0;
             const gross = roundMoney(net + itemVat);
             const currentGross = invoice.gross_price;
-            const shouldRepair = !currentGross || Math.abs(currentGross - gross) > 0.02;
-            if (shouldRepair) {
+            const currentVat = invoice.vat_price;
+            const grossNeedsRepair = !currentGross || Math.abs(currentGross - gross) > 0.02;
+            const vatNeedsRepair = isVatRegistered && (currentVat === undefined || currentVat === null || Math.abs(currentVat - vat) > 0.02);
+            if (grossNeedsRepair || vatNeedsRepair) {
                 await this.update(id, { vat_price: vat, gross_price: gross, items });
             }
         }

package/dist/api/transactions.api.js

@@ -26,10 +26,10 @@ export class TransactionsApi extends BaseResource {
                         const inv = await this.client.get(`/sale_invoices/${dist.related_id}`);
                         clientsId = inv?.clients_id;
                     }
-                    if (clientsId)
+                    if (clientsId !== undefined)
                         break;
                 }
-                if (clientsId) {
+                if (clientsId !== undefined) {
                     await this.update(id, { clients_id: clientsId });
                 }
             }

package/dist/cache.js

@@ -20,13 +20,15 @@ export class Cache {
         return entry.data;
     }
     set(key, data, ttlSeconds) {
+        if (ttlSeconds === 0)
+            return; // Zero TTL = don't cache
         // Evict oldest entries if at capacity
         if (this.store.size >= this.maxEntries && !this.store.has(key)) {
             const firstKey = this.store.keys().next().value;
             if (firstKey)
                 this.store.delete(firstKey);
         }
-        const ttlMs = ttlSeconds ? ttlSeconds * 1000 : this.defaultTtlMs;
+        const ttlMs = ttlSeconds !== undefined ? ttlSeconds * 1000 : this.defaultTtlMs;
         this.store.set(key, { data, expiresAt: Date.now() + ttlMs });
     }
     invalidate(pattern) {

package/dist/config.js

@@ -1,23 +1,11 @@
 import dotenv from "dotenv";
 import { resolve } from "path";
 import { readFileSync, existsSync, statSync, readdirSync, realpathSync } from "fs";
+import { getProjectRoot } from "./paths.js";
 const SERVERS = {
     live: "https://rmp-api.rik.ee/v1",
     demo: "https://demo-rmp-api.rik.ee/v1",
 };
-/** Find project root by walking up from import.meta.dirname to package.json. */
-function getProjectRoot() {
-    let dir = import.meta.dirname;
-    for (let i = 0; i < 5; i++) {
-        if (existsSync(resolve(dir, "package.json")))
-            return dir;
-        const parent = resolve(dir, "..");
-        if (parent === dir)
-            break;
-        dir = parent;
-    }
-    return process.cwd();
-}
 const PROJECT_ROOT = getProjectRoot();
 dotenv.config({ path: resolve(PROJECT_ROOT, ".env") });
 function getBaseUrl() {

package/dist/csv.d.ts

@@ -0,0 +1 @@
+export declare function parseCSVLine(line: string): string[];

package/dist/csv.js

@@ -0,0 +1,34 @@
+export function parseCSVLine(line) {
+    const fields = [];
+    let current = "";
+    let inQuotes = false;
+    for (let i = 0; i < line.length; i++) {
+        const ch = line[i];
+        if (inQuotes) {
+            if (ch === '"') {
+                if (i + 1 < line.length && line[i + 1] === '"') {
+                    current += '"';
+                    i++;
+                }
+                else {
+                    inQuotes = false;
+                }
+            }
+            else {
+                current += ch;
+            }
+        }
+        else if (ch === '"') {
+            inQuotes = true;
+        }
+        else if (ch === ",") {
+            fields.push(current);
+            current = "";
+        }
+        else {
+            current += ch;
+        }
+    }
+    fields.push(current);
+    return fields;
+}

package/dist/file-validation.js

@@ -2,6 +2,7 @@ import { stat, realpath } from "fs/promises";
 import { resolve, extname, isAbsolute } from "path";
 import { existsSync, realpathSync } from "fs";
 import { homedir } from "os";
+import { getProjectRoot } from "./paths.js";
 /**
  * Allowed root directories for file reads. Configurable via EARVELDAJA_ALLOWED_PATHS
  * (colon-separated list). Defaults to $HOME and /tmp.
@@ -21,23 +22,6 @@ function getAllowedRoots() {
         }
     });
 }
-/**
- * Get the project root (directory containing package.json).
- * Falls back to process.cwd().
- */
-function getProjectRoot() {
-    // When running from dist/ or src/, go up until we find package.json
-    let dir = import.meta.dirname;
-    for (let i = 0; i < 5; i++) {
-        if (existsSync(resolve(dir, "package.json")))
-            return dir;
-        const parent = resolve(dir, "..");
-        if (parent === dir)
-            break;
-        dir = parent;
-    }
-    return process.cwd();
-}
 /**
  * Resolve a file path. For relative paths, tries:
  * 1. Parent of project root (where accounting documents typically live)

package/dist/http-client.d.ts

@@ -13,6 +13,10 @@ export declare class HttpClient {
     private readonly minIntervalMs;
     constructor(config: Config, cacheNamespace?: string);
     private waitForRateLimitTurn;
+    private static sleep;
+    private static shouldRetryStatus;
+    private static isRetryableError;
+    private static formatNetworkError;
     request<T = unknown>(path: string, options?: RequestOptions): Promise<T>;
     private static getPublicIp;
     get<T = unknown>(path: string, params?: Record<string, string | number | boolean | undefined>): Promise<T>;

package/dist/http-client.js

@@ -1,4 +1,6 @@
 import { createAuthHeaders } from "./auth.js";
+const MAX_RETRIES = 3;
+const INITIAL_RETRY_DELAY_MS = 1_000;
 export class HttpClient {
     config;
     cacheNamespace;
@@ -22,8 +24,22 @@ export class HttpClient {
         this.lastRequest = waitTurn;
         await waitTurn;
     }
+    static async sleep(ms) {
+        await new Promise(resolve => setTimeout(resolve, ms));
+    }
+    static shouldRetryStatus(status) {
+        return status === 429 || status >= 500;
+    }
+    static isRetryableError(error) {
+        return error instanceof Error && (error.name === "AbortError" ||
+            error.name === "TypeError" ||
+            /fetch failed|network/i.test(error.message));
+    }
+    static formatNetworkError(method, path, error) {
+        const suffix = error instanceof Error && error.message ? `: ${error.message}` : "";
+        return new Error(`API request failed: ${method} ${path} → network error${suffix}`);
+    }
     async request(path, options = {}) {
-        await this.waitForRateLimitTurn();
         const { method = "GET", body, params } = options;
         // Build full URL: baseUrl already includes /v1
         const fullUrl = `${this.config.baseUrl}${path}`;
@@ -45,56 +61,74 @@ export class HttpClient {
         if (body !== undefined) {
             headers["Content-Type"] = "application/json";
         }
-        const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(), 60_000);
-        try {
-            const response = await fetch(url.toString(), {
-                method,
-                headers,
-                body: body !== undefined ? JSON.stringify(body) : undefined,
-                signal: controller.signal,
-            });
-            if (!response.ok) {
-                // Parse structured error if available, but don't expose raw upstream details
-                let errorMessage = `API request failed: ${method} ${path} → ${response.status}`;
+        for (let attempt = 0; attempt <= MAX_RETRIES; attempt++) {
+            await this.waitForRateLimitTurn();
+            const controller = new AbortController();
+            const timeoutId = setTimeout(() => controller.abort(), 60_000);
+            try {
+                let response;
                 try {
-                    const body = await response.json();
-                    if (body.messages && Array.isArray(body.messages)) {
-                        errorMessage += `: ${body.messages.join("; ")}`;
+                    response = await fetch(url.toString(), {
+                        method,
+                        headers,
+                        body: body !== undefined ? JSON.stringify(body) : undefined,
+                        signal: controller.signal,
+                    });
+                }
+                catch (error) {
+                    if (HttpClient.isRetryableError(error) && attempt < MAX_RETRIES) {
+                        await HttpClient.sleep(INITIAL_RETRY_DELAY_MS * (2 ** attempt));
+                        continue;
                     }
+                    throw HttpClient.formatNetworkError(method, path, error);
                 }
-                catch {
-                    // Non-JSON error body — don't expose raw text
+                if (!response.ok) {
+                    if (HttpClient.shouldRetryStatus(response.status) && attempt < MAX_RETRIES) {
+                        await HttpClient.sleep(INITIAL_RETRY_DELAY_MS * (2 ** attempt));
+                        continue;
+                    }
+                    // Parse structured error if available, but don't expose raw upstream details
+                    let errorMessage = `API request failed: ${method} ${path} → ${response.status}`;
+                    try {
+                        const body = await response.json();
+                        if (body.messages && Array.isArray(body.messages)) {
+                            errorMessage += `: ${body.messages.join("; ")}`;
+                        }
+                    }
+                    catch {
+                        // Non-JSON error body — don't expose raw text
+                    }
+                    if (response.status === 401) {
+                        const publicIp = await HttpClient.getPublicIp();
+                        errorMessage += `\n\nTroubleshooting 401 Unauthorized:\n` +
+                            `  1. Is the API key downloaded and configured? Check apikey*.txt or environment variables.\n` +
+                            `  2. Is this machine's IP address allowed in e-arveldaja API settings?\n` +
+                            `     Your public IP: ${publicIp}\n` +
+                            `  3. Add the IP above to: e-arveldaja → Seaded → API võtmed → Lubatud IP-aadressid\n` +
+                            `     Multiple IP addresses can be added, separated by ;`;
+                    }
+                    throw new Error(errorMessage);
                 }
-                if (response.status === 401) {
-                    const publicIp = await HttpClient.getPublicIp();
-                    errorMessage += `\n\nTroubleshooting 401 Unauthorized:\n` +
-                        `  1. Is the API key downloaded and configured? Check apikey*.txt or environment variables.\n` +
-                        `  2. Is this machine's IP address allowed in e-arveldaja API settings?\n` +
-                        `     Your public IP: ${publicIp}\n` +
-                        `  3. Add the IP above to: e-arveldaja → Seaded → API võtmed → Lubatud IP-aadressid\n` +
-                        `     Multiple IP addresses can be added, separated by ;`;
+                if (response.status === 204) {
+                    return undefined;
                 }
-                throw new Error(errorMessage);
-            }
-            if (response.status === 204) {
-                return undefined;
+                const contentType = response.headers.get("content-type") ?? "";
+                if (contentType.includes("application/json")) {
+                    return response.json();
+                }
+                // Binary response (e.g. PDF document download) — return as ApiFile-compatible object
+                const arrayBuf = await response.arrayBuffer();
+                const base64 = Buffer.from(arrayBuf).toString("base64");
+                const disposition = response.headers.get("content-disposition") ?? "";
+                const nameMatch = disposition.match(/filename="?([^";\n]+)"?/);
+                const name = nameMatch?.[1] ?? "document";
+                return { name, contents: base64 };
             }
-            const contentType = response.headers.get("content-type") ?? "";
-            if (contentType.includes("application/json")) {
-                return response.json();
+            finally {
+                clearTimeout(timeoutId);
             }
-            // Binary response (e.g. PDF document download) — return as ApiFile-compatible object
-            const arrayBuf = await response.arrayBuffer();
-            const base64 = Buffer.from(arrayBuf).toString("base64");
-            const disposition = response.headers.get("content-disposition") ?? "";
-            const nameMatch = disposition.match(/filename="?([^";\n]+)"?/);
-            const name = nameMatch?.[1] ?? "document";
-            return { name, contents: base64 };
-        }
-        finally {
-            clearTimeout(timeoutId);
         }
+        throw new Error(`API request failed: ${method} ${path} → retries exhausted`);
     }
     static async getPublicIp() {
         try {

package/dist/index.js

@@ -93,7 +93,7 @@ async function main() {
     const api = createScopedApiContext(connectionState, connectionContexts, invocationStorage);
     const server = new McpServer({
         name: "e-arveldaja",
-        version: "0.3.2",
+        version: "0.4.0",
         description: "EXPERIMENTAL, UNOFFICIAL MCP server for the Estonian e-arveldaja (e-Financials) API. " +
             "NOT affiliated with or endorsed by RIK. Use entirely at your own risk — " +
             "this software interacts with live financial data and can create, modify, and delete accounting records. " +
@@ -167,9 +167,10 @@ Reporting:
         }
         const target = allConfigs[index];
         const previousIndex = connectionState.activeIndex;
-        clearAllCaches(previousIndex);
-        connectionState.activeIndex = index;
         connectionState.generation += 1;
+        connectionState.activeIndex = index;
+        clearAllCaches(previousIndex);
+        clearAllCaches(index);
         const snapshot = captureSnapshot(connectionState);
         return {
             content: [{
@@ -201,6 +202,7 @@ Reporting:
                 });
             }
             catch (error) {
+                process.stderr.write(`Tool handler error: ${error instanceof Error ? error.stack ?? error.message : String(error)}\n`);
                 return toolError(error);
             }
         });

package/dist/money.js

@@ -1,2 +1,2 @@
 /** Round to 2 decimal places (cents). Use for all monetary arithmetic. */
-export const roundMoney = (v) => Math.round(v * 100) / 100;
+export const roundMoney = (v) => Math.sign(v) * Math.round((Math.abs(v) + Number.EPSILON) * 100) / 100;

package/dist/paths.d.ts

@@ -0,0 +1,2 @@
+/** Find project root by walking up from import.meta.dirname to package.json. */
+export declare function getProjectRoot(): string;

package/dist/paths.js

@@ -0,0 +1,15 @@
+import { existsSync } from "fs";
+import { resolve } from "path";
+/** Find project root by walking up from import.meta.dirname to package.json. */
+export function getProjectRoot() {
+    let dir = import.meta.dirname;
+    for (let i = 0; i < 5; i++) {
+        if (existsSync(resolve(dir, "package.json")))
+            return dir;
+        const parent = resolve(dir, "..");
+        if (parent === dir)
+            break;
+        dir = parent;
+    }
+    return process.cwd();
+}

package/dist/tools/bank-reconciliation.js

@@ -189,9 +189,10 @@ export function registerBankReconciliationTools(server, api) {
             }
             // Only auto-confirm if exactly one high-confidence match
             if (candidates.length === 1) {
-                consumedInvoiceKeys.add(`${candidates[0].type.replace("_invoice", "")}:${candidates[0].id}`);
                 const match = candidates[0];
+                const invoiceKey = `${match.type.replace("_invoice", "")}:${match.id}`;
                 if (dryRun) {
+                    consumedInvoiceKeys.add(invoiceKey);
                     confirmed.push({
                         transaction_id: tx.id,
                         amount: tx.amount,
@@ -207,6 +208,7 @@ export function registerBankReconciliationTools(server, api) {
                                 related_id: match.id,
                                 amount: tx.amount,
                             }]);
+                        consumedInvoiceKeys.add(invoiceKey);
                         confirmed.push({
                             transaction_id: tx.id,
                             amount: tx.amount,
@@ -215,7 +217,7 @@ export function registerBankReconciliationTools(server, api) {
                         });
                     }
                     catch (err) {
-                        skipped.push({ transaction_id: tx.id, reason: err.message });
+                        skipped.push({ transaction_id: tx.id, reason: err instanceof Error ? err.message : String(err) });
                     }
                 }
             }

package/dist/tools/crud-tools.d.ts

@@ -20,5 +20,8 @@ export interface ApiContext {
 export declare function isCompanyVatRegistered(api: ApiContext): Promise<boolean>;
 export declare const MAX_JSON_INPUT_SIZE: number;
 export declare function safeJsonParse(input: string, label: string): unknown;
+export declare function parseJsonObject(input: string, label: string): Record<string, unknown>;
+export declare function parseJsonObjectArray(input: string, label: string): Record<string, unknown>[];
+export declare function requireFields(items: Record<string, unknown>[], label: string, fields: string[]): void;
 export declare function parsePurchaseInvoiceItems(input: string): PurchaseInvoiceItem[];
 export declare function registerCrudTools(server: McpServer, api: ApiContext): void;

package/dist/tools/crud-tools.js

@@ -21,14 +21,14 @@ export function safeJsonParse(input, label) {
 function isRecord(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-function parseJsonObject(input, label) {
+export function parseJsonObject(input, label) {
     const parsed = safeJsonParse(input, label);
     if (!isRecord(parsed)) {
         throw new Error(`"${label}" must be a JSON object`);
     }
     return parsed;
 }
-function parseJsonObjectArray(input, label) {
+export function parseJsonObjectArray(input, label) {
     const parsed = safeJsonParse(input, label);
     if (!Array.isArray(parsed)) {
         throw new Error(`"${label}" must be a JSON array`);
@@ -40,7 +40,7 @@ function parseJsonObjectArray(input, label) {
     });
     return parsed;
 }
-function requireFields(items, label, fields) {
+export function requireFields(items, label, fields) {
     items.forEach((item, index) => {
         for (const field of fields) {
             if (!(field in item) || item[field] === null || item[field] === undefined || item[field] === "") {
@@ -79,6 +79,8 @@ const pageParam = z.object({
     modified_since: z.string().optional().describe("Return only objects modified since this timestamp (ISO 8601)"),
 });
 const idParam = z.object({ id: z.number().describe("Object ID") });
+const isoDateRegex = /^\d{4}-\d{2}-\d{2}$/;
+const isoDateString = (description) => z.string().regex(isoDateRegex, "Expected YYYY-MM-DD").describe(description);
 export function registerCrudTools(server, api) {
     // =====================
     // CLIENTS
@@ -192,7 +194,7 @@ export function registerCrudTools(server, api) {
     });
     server.tool("create_journal", "Create a journal entry with postings", {
         title: z.string().optional().describe("Journal entry title"),
-        effective_date: z.string().describe("Entry date (YYYY-MM-DD)"),
+        effective_date: isoDateString("Entry date (YYYY-MM-DD)"),
         clients_id: z.number().optional().describe("Related client ID"),
         document_number: z.string().optional().describe("Document number"),
         cl_currencies_id: z.string().optional().describe("Currency (default EUR)"),
@@ -240,7 +242,7 @@ export function registerCrudTools(server, api) {
         type: z.string().describe("Transaction type: D (incoming) or C (outgoing)"),
         amount: z.number().describe("Transaction amount"),
         cl_currencies_id: z.string().optional().describe("Currency (default EUR)"),
-        date: z.string().describe("Transaction date (YYYY-MM-DD)"),
+        date: isoDateString("Transaction date (YYYY-MM-DD)"),
         description: z.string().optional().describe("Description"),
         clients_id: z.number().optional().describe("Related client ID"),
         bank_account_name: z.string().optional().describe("Remitter/beneficiary name"),
@@ -283,8 +285,8 @@ export function registerCrudTools(server, api) {
         clients_id: z.number().describe("Buyer client ID"),
         cl_templates_id: z.number().describe("Invoice template ID"),
         number_suffix: z.string().optional().describe("Invoice number suffix (omit or empty string for auto-assign from invoice series)"),
-        create_date: z.string().describe("Invoice date (YYYY-MM-DD)"),
-        journal_date: z.string().describe("Turnover date (YYYY-MM-DD)"),
+        create_date: isoDateString("Invoice date (YYYY-MM-DD)"),
+        journal_date: isoDateString("Turnover date (YYYY-MM-DD)"),
         term_days: z.number().describe("Payment term in days"),
         cl_currencies_id: z.string().optional().describe("Currency (default EUR)"),
         cl_countries_id: z.string().optional().describe("Country (default EST)"),
@@ -353,8 +355,8 @@ export function registerCrudTools(server, api) {
         clients_id: z.number().describe("Supplier client ID"),
         client_name: z.string().describe("Supplier name"),
         number: z.string().describe("Invoice number"),
-        create_date: z.string().describe("Invoice date (YYYY-MM-DD)"),
-        journal_date: z.string().describe("Turnover date (YYYY-MM-DD)"),
+        create_date: isoDateString("Invoice date (YYYY-MM-DD)"),
+        journal_date: isoDateString("Turnover date (YYYY-MM-DD)"),
         term_days: z.number().describe("Payment term in days"),
         vat_price: z.number().optional().describe("Total VAT amount from original invoice (EXACT, for payment matching)"),
         gross_price: z.number().optional().describe("Total gross amount from original invoice (EXACT, for payment matching)"),
@@ -369,7 +371,7 @@ export function registerCrudTools(server, api) {
         const purchaseArticles = await getPurchaseArticlesWithVat(api);
         const rawItems = parsePurchaseInvoiceItems(params.items);
         const items = rawItems.map(item => applyPurchaseVatDefaults(purchaseArticles, item, isVatReg));
-        const result = await api.purchaseInvoices.createAndSetTotals({
+        const invoiceData = {
             clients_id: params.clients_id,
             client_name: params.client_name,
             number: params.number,
@@ -382,7 +384,8 @@ export function registerCrudTools(server, api) {
             bank_account_no: params.bank_account_no,
             notes: params.notes,
             items,
-        }, params.vat_price, params.gross_price, isVatReg);
+        };
+        const result = await api.purchaseInvoices.createAndSetTotals(invoiceData, params.vat_price, params.gross_price, isVatReg);
         return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
     });
     server.tool("update_purchase_invoice", "Update a purchase invoice", {

package/dist/tools/estonian-tax.js

@@ -3,22 +3,7 @@ import { isCompanyVatRegistered } from "./crud-tools.js";
 import { computeAllBalances } from "./financial-statements.js";
 import { roundMoney } from "../money.js";
 import { create } from "../annotations.js";
-async function validateAccounts(api, ...accountIds) {
-    const accounts = await api.readonly.getAccounts();
-    const accountMap = new Map(accounts.map(account => [account.id, account]));
-    const errors = [];
-    for (const id of new Set(accountIds)) {
-        const account = accountMap.get(id);
-        if (!account) {
-            errors.push(`Account ${id} not found in chart of accounts`);
-            continue;
-        }
-        if (!account.is_valid) {
-            errors.push(`Account ${id} (${account.name_est}) is inactive. Activate in e-arveldaja: Seaded → Kontoplaan → ${account.name_est} → mark as active.`);
-        }
-    }
-    return errors;
-}
+import { validateAccounts } from "../account-validation.js";
 async function computeRetainedEarningsBalance(api, accountId, asOfDate) {
     const allJournals = await api.journals.listAllWithPostings();
     let debit = 0;
@@ -63,7 +48,13 @@ export function registerEstonianTaxTools(server, api) {
         const taxAccount = tax_payable_account ?? 2540;
         const shareCapitalAccount = share_capital_account ?? 3000;
         // Validate all accounts exist in chart of accounts
-        const accountErrors = await validateAccounts(api, retainedAccount, payableAccount, taxAccount, shareCapitalAccount);
+        const accounts = await api.readonly.getAccounts();
+        const accountErrors = validateAccounts(accounts, [
+            { id: retainedAccount, label: "Retained earnings account" },
+            { id: payableAccount, label: "Dividend payable account" },
+            { id: taxAccount, label: "Tax payable account" },
+            { id: shareCapitalAccount, label: "Share capital account" },
+        ]);
         if (accountErrors.length > 0) {
             return {
                 content: [{
@@ -189,7 +180,12 @@ export function registerEstonianTaxTools(server, api) {
         const vat = vat_amount ?? roundMoney(net_amount * vat_rate);
         if (vat > 0 && vatRegistered)
             accountsToCheck.push(vatAcc);
-        const accountErrors = await validateAccounts(api, ...accountsToCheck);
+        const accounts = await api.readonly.getAccounts();
+        const accountErrors = validateAccounts(accounts, [
+            { id: expense_account, label: "Expense account" },
+            ...(vat > 0 && vatRegistered ? [{ id: vatAcc, label: "VAT account" }] : []),
+            { id: payAcc, label: "Payable account" },
+        ]);
         if (accountErrors.length > 0) {
             return {
                 content: [{

package/dist/tools/financial-statements.js

@@ -75,6 +75,7 @@ function getMonthLastDay(month) {
     const [year, monthNumber] = month.split("-").map(Number);
     return new Date(Date.UTC(year, monthNumber, 0)).getUTCDate();
 }
+const monthRegex = /^\d{4}-\d{2}$/;
 export function registerFinancialStatementTools(server, api) {
     server.tool("compute_trial_balance", "Compute trial balance (käibeandmik/proovibilanss) from journal postings. " +
         "Shows debit/credit totals and balance for each account.", {
@@ -186,24 +187,23 @@ export function registerFinancialStatementTools(server, api) {
     });
     server.tool("month_end_close_checklist", "Generate month-end close checklist: unconfirmed journals/invoices, " +
         "unreconciled bank transactions, overdue receivables/payables.", {
-        month: z.string().describe("Month to check (YYYY-MM, e.g. 2026-02)"),
+        month: z.string().regex(monthRegex, "Expected YYYY-MM").describe("Month to check (YYYY-MM, e.g. 2026-02)"),
     }, { ...readOnly, title: "Month-End Close Checklist" }, async ({ month }) => {
         const dateFrom = `${month}-01`;
         const lastDay = getMonthLastDay(month);
         const dateTo = `${month}-${String(lastDay).padStart(2, "0")}`;
-        // Unconfirmed journals
-        const allJournals = await api.journals.listAll();
+        const [allJournals, allTx, allSales, allPurchases] = await Promise.all([
+            api.journals.listAll(),
+            api.transactions.listAll(),
+            api.saleInvoices.listAll(),
+            api.purchaseInvoices.listAll(),
+        ]);
         const unconfirmedJournals = allJournals.filter(j => !j.is_deleted && !j.registered &&
             j.effective_date >= dateFrom && j.effective_date <= dateTo);
-        // Unconfirmed transactions
-        const allTx = await api.transactions.listAll();
         const unconfirmedTx = allTx.filter(tx => !tx.is_deleted && tx.status !== "CONFIRMED" &&
             tx.date >= dateFrom && tx.date <= dateTo);
-        // Unconfirmed invoices
-        const allSales = await api.saleInvoices.listAll();
         const unconfirmedSales = allSales.filter((inv) => inv.status === "PROJECT" &&
             inv.journal_date >= dateFrom && inv.journal_date <= dateTo);
-        const allPurchases = await api.purchaseInvoices.listAll();
         const unconfirmedPurchases = allPurchases.filter((inv) => inv.status === "PROJECT" &&
             inv.journal_date >= dateFrom && inv.journal_date <= dateTo);
         // Overdue receivables (compare to month-end date for reproducibility)

package/dist/tools/lightyear-investments.js

@@ -4,6 +4,8 @@ import { validateFilePath } from "../file-validation.js";
 import { roundMoney } from "../money.js";
 import { readOnly, batch } from "../annotations.js";
 import { reportProgress } from "../progress.js";
+import { parseCSVLine } from "../csv.js";
+import { validateAccounts } from "../account-validation.js";
 const MAX_CSV_SIZE = 10 * 1024 * 1024; // 10 MB
 // BRICEKSP is Lightyear's money market cash fund - not a real investment
 const CASH_FUND_TICKER = "BRICEKSP";
@@ -22,42 +24,6 @@ function parseLightyearDate(d) {
     }
     return d;
 }
-function parseCSVLine(line) {
-    const fields = [];
-    let current = "";
-    let inQuotes = false;
-    for (let i = 0; i < line.length; i++) {
-        const ch = line[i];
-        if (inQuotes) {
-            if (ch === '"') {
-                if (i + 1 < line.length && line[i + 1] === '"') {
-                    current += '"';
-                    i++;
-                }
-                else {
-                    inQuotes = false;
-                }
-            }
-            else {
-                current += ch;
-            }
-        }
-        else {
-            if (ch === '"') {
-                inQuotes = true;
-            }
-            else if (ch === ",") {
-                fields.push(current);
-                current = "";
-            }
-            else {
-                current += ch;
-            }
-        }
-    }
-    fields.push(current);
-    return fields;
-}
 function validateHeaders(actual, expected, label) {
     if (actual.length < expected.length) {
         throw new Error(`${label}: expected ${expected.length} columns, got ${actual.length}. ` +
@@ -431,25 +397,13 @@ export function registerLightyearTools(server, api) {
         const skipSet = new Set((skip_tickers ?? CASH_FUND_TICKER).split(",").map(t => t.trim()));
         // Validate accounts exist and are active
         const accounts = await api.readonly.getAccounts();
-        const accountMap = new Map(accounts.map(a => [a.id, a]));
-        const errors = [];
-        function checkAccount(id, label) {
-            const acc = accountMap.get(id);
-            if (!acc) {
-                errors.push(`${label} ${id} not found in chart of accounts. Activate it in e-arveldaja: Seaded → Kontoplaan → find account ${id} and enable it.`);
-            }
-            else if (!acc.is_valid) {
-                errors.push(`${label} ${id} (${acc.name_est}) is inactive. Activate it in e-arveldaja: Seaded → Kontoplaan → ${acc.name_est} → mark as active.`);
-            }
-        }
-        checkAccount(investment_account, "Investment account");
-        checkAccount(broker_account, "Broker account");
-        if (fee_account)
-            checkAccount(fee_account, "Fee account");
-        if (gain_loss_account)
-            checkAccount(gain_loss_account, "Gain/loss account");
-        if (loss_account)
-            checkAccount(loss_account, "Loss account");
+        const errors = validateAccounts(accounts, [
+            { id: investment_account, label: "Investment account" },
+            { id: broker_account, label: "Broker account" },
+            ...(fee_account ? [{ id: fee_account, label: "Fee account" }] : []),
+            ...(gain_loss_account ? [{ id: gain_loss_account, label: "Gain/loss account" }] : []),
+            ...(loss_account ? [{ id: loss_account, label: "Loss account" }] : []),
+        ]);
         if (errors.length > 0) {
             return {
                 content: [{
@@ -674,22 +628,12 @@ export function registerLightyearTools(server, api) {
         const fee_account = fee_account_param ?? 8610;
         // Validate accounts exist and are active
         const accounts = await api.readonly.getAccounts();
-        const accountMap = new Map(accounts.map(a => [a.id, a]));
-        const errors = [];
-        function checkAccount(id, label) {
-            const acc = accountMap.get(id);
-            if (!acc) {
-                errors.push(`${label} ${id} not found in chart of accounts. Activate it in e-arveldaja: Seaded → Kontoplaan → find account ${id} and enable it.`);
-            }
-            else if (!acc.is_valid) {
-                errors.push(`${label} ${id} (${acc.name_est}) is inactive. Activate it in e-arveldaja: Seaded → Kontoplaan → ${acc.name_est} → mark as active.`);
-            }
-        }
-        checkAccount(broker_account, "Broker account");
-        checkAccount(income_account, "Income account");
-        if (tax_account)
-            checkAccount(tax_account, "Tax account");
-        checkAccount(fee_account, "Fee account");
+        const errors = validateAccounts(accounts, [
+            { id: broker_account, label: "Broker account" },
+            { id: income_account, label: "Income account" },
+            ...(tax_account ? [{ id: tax_account, label: "Tax account" }] : []),
+            { id: fee_account, label: "Fee account" },
+        ]);
         if (errors.length > 0) {
             return {
                 content: [{

package/dist/tools/pdf-workflow.js

@@ -371,6 +371,7 @@ export function registerPdfWorkflowTools(server, api) {
         notes: z.string().optional().describe("Notes (e.g. PDF filename)"),
         ref_number: z.string().optional().describe("Reference number"),
         bank_account_no: z.string().optional().describe("Supplier bank account"),
+        currency: z.string().optional().describe("Currency code (default EUR)"),
     }, { ...create, title: "Create Purchase Invoice from PDF" }, async (params) => {
         const supplier = await api.clients.get(params.supplier_client_id);
         const isVatReg = await isCompanyVatRegistered(api);
@@ -384,7 +385,7 @@ export function registerPdfWorkflowTools(server, api) {
             create_date: params.invoice_date,
             journal_date: params.journal_date,
             term_days: params.term_days,
-            cl_currencies_id: "EUR",
+            cl_currencies_id: params.currency ?? "EUR",
             liability_accounts_id: params.liability_accounts_id ?? 2310,
             bank_ref_number: params.ref_number,
             bank_account_no: params.bank_account_no,

package/dist/tools/purchase-vat-defaults.d.ts

@@ -6,6 +6,7 @@ type PurchaseArticleWithVat = PurchaseArticle & {
     vat_rate_dropdown?: string | null;
     vat_rate?: number | null;
 };
+export declare function normalizeVatRate(value: unknown): string | undefined;
 export declare function getPurchaseArticlesWithVat(api: ApiContext): Promise<PurchaseArticleWithVat[]>;
 export declare function applyPurchaseVatDefaults(purchaseArticles: PurchaseArticleWithVat[], item: PurchaseInvoiceItem, isVatRegistered: boolean): PurchaseInvoiceItem;
 export {};

package/dist/tools/purchase-vat-defaults.js

@@ -16,7 +16,7 @@ function warnFallbackOnce(key, message) {
 function toNumber(value) {
     return typeof value === "number" && Number.isFinite(value) ? value : undefined;
 }
-function normalizeVatRate(value) {
+export function normalizeVatRate(value) {
     if (typeof value === "number" && Number.isFinite(value)) {
         return String(value);
     }

package/dist/tools/wise-import.js

@@ -3,6 +3,7 @@ import { readFile } from "fs/promises";
 import { validateFilePath } from "../file-validation.js";
 import { batch } from "../annotations.js";
 import { reportProgress } from "../progress.js";
+import { parseCSVLine } from "../csv.js";
 const EXPECTED_HEADERS = [
     "ID", "Status", "Direction", "Created on", "Finished on",
     "Source fee amount", "Source fee currency", "Target fee amount", "Target fee currency",
@@ -10,32 +11,6 @@ const EXPECTED_HEADERS = [
     "Target name", "Target amount (after fees)", "Target currency",
     "Exchange rate", "Reference", "Batch", "Created by", "Category", "Note",
 ];
-function parseCSVLine(line) {
-    const fields = [];
-    let current = "";
-    let inQuotes = false;
-    for (let i = 0; i < line.length; i++) {
-        const ch = line[i];
-        if (ch === '"') {
-            if (inQuotes && line[i + 1] === '"') {
-                current += '"';
-                i++;
-            }
-            else {
-                inQuotes = !inQuotes;
-            }
-        }
-        else if (ch === "," && !inQuotes) {
-            fields.push(current);
-            current = "";
-        }
-        else {
-            current += ch;
-        }
-    }
-    fields.push(current);
-    return fields;
-}
 function parseWiseCSV(csv) {
     const lines = csv.trim().split("\n").filter(l => l.trim());
     if (lines.length < 2)

package/dist/types/api.d.ts

@@ -346,6 +346,9 @@ export interface PurchaseInvoice {
     settlements?: number[];
     transactions?: number[];
 }
+export interface CreatePurchaseInvoiceData extends Pick<PurchaseInvoice, "clients_id" | "client_name" | "number" | "create_date" | "journal_date" | "term_days" | "cl_currencies_id" | "liability_accounts_id" | "bank_ref_number" | "bank_account_no" | "notes"> {
+    items: PurchaseInvoiceItem[];
+}
 export interface InvoiceSeries {
     id?: number;
     is_active: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "e-arveldaja-mcp",
-  "version": "0.3.2",
+  "version": "0.4.0",
   "description": "MCP server for Estonian e-arveldaja (e-Financials) API",
   "type": "module",
   "main": "dist/index.js",
@@ -34,7 +34,10 @@
     "build": "tsc",
     "start": "node dist/index.js",
     "dev": "tsx src/index.ts",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "npm run build",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:integration": "vitest run --config vitest.integration.config.ts"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.12.1",
@@ -47,6 +50,7 @@
     "@types/node": "^22.13.14",
     "@types/pdf-parse": "^1.1.4",
     "tsx": "^4.19.3",
-    "typescript": "^5.8.2"
+    "typescript": "^5.8.2",
+    "vitest": "^4.1.0"
   }
 }