@fuzzle/opencode-accountant 0.0.8 → 0.0.9-next.1

package/README.md

@@ -115,7 +115,9 @@ providers:
 
   ubs:
     detect:
-      - header: 'Trade date,Trade time,Booking date,Value date,Currency,Debit,Credit,Individual amount,Balance,Transaction no.,Description1,Description2,Description3,Footnotes'
+      # Note: UBS exports have a trailing semicolon in the header row, which creates
+      # an empty field when parsed. The header must include a trailing comma to match.
+      - header: 'Trade date,Trade time,Booking date,Value date,Currency,Debit,Credit,Individual amount,Balance,Transaction no.,Description1,Description2,Description3,Footnotes,'
         currencyField: Currency
         skipRows: 9
         delimiter: ';'
@@ -147,7 +149,7 @@ providers:
 | Field             | Required | Description                                                |
 | ----------------- | -------- | ---------------------------------------------------------- |
 | `filenamePattern` | No       | Regex pattern to match against filename                    |
-| `header`          | Yes      | Expected CSV header row (exact match)                      |
+| `header`          | Yes      | Expected CSV header row (comma-separated, exact match)\*   |
 | `currencyField`   | Yes      | Column name containing the currency/symbol                 |
 | `skipRows`        | No       | Number of rows to skip before header (default: 0)          |
 | `delimiter`       | No       | CSV delimiter character (default: `,`)                     |
@@ -155,6 +157,8 @@ providers:
 | `metadata`        | No       | Array of metadata extraction rules (see below)             |
 | `currencies`      | Yes      | Map of raw currency values to normalized folder names      |
 
+\* **Note on trailing delimiters:** If the CSV header row ends with a trailing delimiter (e.g., `Field1;Field2;`), this creates an empty field when parsed. The `header` config must include a trailing comma to account for this (e.g., `Field1,Field2,`).
+
 **Metadata Extraction Rules:**
 
 | Field       | Required | Description                                             |

package/dist/index.js

@@ -1341,7 +1341,7 @@ var require_papaparse = __commonJS((exports, module) => {
 });
 
 // src/index.ts
-import { dirname, join as join5 } from "path";
+import { dirname as dirname2, join as join5 } from "path";
 import { fileURLToPath } from "url";
 
 // src/utils/agentLoader.ts
@@ -16848,35 +16848,6 @@ function findCSVFiles(importsDir) {
     return fs4.statSync(fullPath).isFile();
   });
 }
-function checkPendingFiles(directory, pendingBasePath) {
-  const pendingDir = path4.join(directory, pendingBasePath);
-  const pendingFiles = [];
-  if (!fs4.existsSync(pendingDir)) {
-    return [];
-  }
-  const providers = fs4.readdirSync(pendingDir);
-  for (const provider of providers) {
-    const providerPath = path4.join(pendingDir, provider);
-    if (!fs4.statSync(providerPath).isDirectory())
-      continue;
-    const currencies = fs4.readdirSync(providerPath);
-    for (const currency of currencies) {
-      const currencyPath = path4.join(providerPath, currency);
-      if (!fs4.statSync(currencyPath).isDirectory())
-        continue;
-      const files = fs4.readdirSync(currencyPath).filter((f) => f.toLowerCase().endsWith(".csv"));
-      for (const file2 of files) {
-        pendingFiles.push({
-          provider,
-          currency,
-          filename: file2,
-          path: path4.join(currencyPath, file2)
-        });
-      }
-    }
-  }
-  return pendingFiles;
-}
 function ensureDirectory(dirPath) {
   if (!fs4.existsSync(dirPath)) {
     fs4.mkdirSync(dirPath, { recursive: true });
@@ -16907,16 +16878,6 @@ async function classifyStatementsCore(directory, agent, configLoader = loadImpor
   const importsDir = path4.join(directory, config2.paths.import);
   const pendingDir = path4.join(directory, config2.paths.pending);
   const unrecognizedDir = path4.join(directory, config2.paths.unrecognized);
-  const pendingFiles = checkPendingFiles(directory, config2.paths.pending);
-  if (pendingFiles.length > 0) {
-    return JSON.stringify({
-      success: false,
-      error: `Found ${pendingFiles.length} pending file(s) that must be processed before classifying new statements.`,
-      pendingFiles,
-      classified: [],
-      unrecognized: []
-    });
-  }
   const csvFiles = findCSVFiles(importsDir);
   if (csvFiles.length === 0) {
     return JSON.stringify({
@@ -16926,32 +16887,65 @@ async function classifyStatementsCore(directory, agent, configLoader = loadImpor
       message: `No CSV files found in ${config2.paths.import}`
     });
   }
-  const classified = [];
-  const unrecognized = [];
+  const plannedMoves = [];
+  const collisions = [];
   for (const filename of csvFiles) {
     const sourcePath = path4.join(importsDir, filename);
     const content = fs4.readFileSync(sourcePath, "utf-8");
     const detection = detectProvider(filename, content, config2);
+    let targetPath;
+    let targetFilename;
     if (detection) {
-      const targetFilename = detection.outputFilename || filename;
+      targetFilename = detection.outputFilename || filename;
       const targetDir = path4.join(pendingDir, detection.provider, detection.currency);
+      targetPath = path4.join(targetDir, targetFilename);
+    } else {
+      targetFilename = filename;
+      targetPath = path4.join(unrecognizedDir, filename);
+    }
+    if (fs4.existsSync(targetPath)) {
+      collisions.push({
+        filename,
+        existingPath: targetPath
+      });
+    }
+    plannedMoves.push({
+      filename,
+      sourcePath,
+      targetPath,
+      targetFilename,
+      detection
+    });
+  }
+  if (collisions.length > 0) {
+    return JSON.stringify({
+      success: false,
+      error: `Cannot classify: ${collisions.length} file(s) would overwrite existing pending files.`,
+      collisions,
+      classified: [],
+      unrecognized: []
+    });
+  }
+  const classified = [];
+  const unrecognized = [];
+  for (const move of plannedMoves) {
+    if (move.detection) {
+      const targetDir = path4.dirname(move.targetPath);
       ensureDirectory(targetDir);
-      const targetPath = path4.join(targetDir, targetFilename);
-      fs4.renameSync(sourcePath, targetPath);
+      fs4.renameSync(move.sourcePath, move.targetPath);
       classified.push({
-        filename: targetFilename,
-        originalFilename: detection.outputFilename ? filename : undefined,
-        provider: detection.provider,
-        currency: detection.currency,
-        targetPath: path4.join(config2.paths.pending, detection.provider, detection.currency, targetFilename)
+        filename: move.targetFilename,
+        originalFilename: move.detection.outputFilename ? move.filename : undefined,
+        provider: move.detection.provider,
+        currency: move.detection.currency,
+        targetPath: path4.join(config2.paths.pending, move.detection.provider, move.detection.currency, move.targetFilename)
       });
     } else {
       ensureDirectory(unrecognizedDir);
-      const targetPath = path4.join(unrecognizedDir, filename);
-      fs4.renameSync(sourcePath, targetPath);
+      fs4.renameSync(move.sourcePath, move.targetPath);
       unrecognized.push({
-        filename,
-        targetPath: path4.join(config2.paths.unrecognized, filename)
+        filename: move.filename,
+        targetPath: path4.join(config2.paths.unrecognized, move.filename)
       });
     }
   }
@@ -16975,7 +16969,7 @@ var classify_statements_default = tool({
   }
 });
 // src/index.ts
-var __dirname2 = dirname(fileURLToPath(import.meta.url));
+var __dirname2 = dirname2(fileURLToPath(import.meta.url));
 var AGENT_FILE = join5(__dirname2, "..", "agent", "accountant.md");
 var AccountantPlugin = async () => {
   const agent = loadAgent(AGENT_FILE);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fuzzle/opencode-accountant",
-  "version": "0.0.8",
+  "version": "0.0.9-next.1",
   "description": "An OpenCode accounting agent, specialized in double-entry-bookkepping with hledger",
   "author": {
     "name": "ali bengali",