@fuzzle/opencode-accountant 0.10.6-next.1 → 0.10.6

package/README.md

@@ -270,15 +270,6 @@ The pipeline automatically manages account declarations:
 
 **No manual account setup required.** Account declarations are created proactively before import attempts.
 
-#### Commodity Declarations
-
-For Swissquote investment imports, the pipeline automatically manages commodity declarations:
-
-- Collects all stock ticker symbols from trades, dividends, and corporate actions
-- Writes sorted `commodity "SYMBOL"` declarations to `ledger/investments/commodities.journal`
-- Merges with existing declarations (idempotent)
-- Ensures `hledger check --strict` validation passes for quoted commodity symbols
-
 #### Unknown Postings
 
 When a transaction doesn't match any `if` pattern in the rules file, hledger assigns it to `income:unknown` or `expenses:unknown` depending on the transaction direction. The pipeline will fail at the validation step, reporting the unknown postings so you can add appropriate rules before retrying.

package/agent/accountant.md

@@ -164,15 +164,6 @@ The import pipeline automatically:
 - Prevents `hledger check --strict` failures due to missing account declarations
 - No manual account setup required
 
-### Automatic Commodity Declarations
-
-For Swissquote investment imports, the pipeline automatically:
-
-- Collects all stock ticker symbols from trades, dividends, and corporate actions
-- Writes sorted `commodity "SYMBOL"` declarations to `ledger/investments/commodities.journal`
-- Merges with existing declarations (idempotent)
-- Prevents `hledger check --strict` failures due to missing commodity declarations
-
 ### Automatic Balance Detection
 
 The reconciliation step attempts to extract closing balance from:

package/dist/index.js

@@ -4533,7 +4533,7 @@ var init_accountSuggester = __esm(() => {
 
 // src/index.ts
 init_agentLoader();
-import { dirname as dirname7, join as join15 } from "path";
+import { dirname as dirname6, join as join15 } from "path";
 import { fileURLToPath as fileURLToPath3 } from "url";
 
 // node_modules/zod/v4/classic/external.js
@@ -17051,38 +17051,6 @@ function ensureYearJournalExists(directory, year) {
   }
   return yearJournalPath;
 }
-function ensureCommodityDeclarations(commodityJournalPath, symbols, logger) {
-  const existing = new Set;
-  if (fs4.existsSync(commodityJournalPath)) {
-    const content2 = fs4.readFileSync(commodityJournalPath, "utf-8");
-    for (const line of content2.split(`
-`)) {
-      const match = line.trim().match(/^commodity\s+"(.+)"/);
-      if (match) {
-        existing.add(match[1]);
-      }
-    }
-  }
-  const missing = [];
-  for (const symbol2 of symbols) {
-    const upper = symbol2.toUpperCase();
-    if (!existing.has(upper)) {
-      missing.push(upper);
-      existing.add(upper);
-    }
-  }
-  if (missing.length === 0) {
-    return { added: [], updated: false };
-  }
-  const sorted = Array.from(existing).sort((a, b) => a.localeCompare(b));
-  const content = sorted.map((s) => `commodity "${s}"`).join(`
-`) + `
-`;
-  ensureDirectory(path3.dirname(commodityJournalPath));
-  fs4.writeFileSync(commodityJournalPath, content);
-  logger?.info(`Commodity declarations: added ${missing.length} (${missing.join(", ")})`);
-  return { added: missing.sort(), updated: true };
-}
 
 // src/utils/dateUtils.ts
 function formatDateISO(date5) {
@@ -25048,9 +25016,6 @@ function formatDate(dateStr) {
 function escapeDescription(desc) {
   return desc.replace(/[;|]/g, "-").trim();
 }
-function formatCommodity(symbol2) {
-  return `"${symbol2.toUpperCase()}"`;
-}
 function formatQuantity(qty) {
   return qty.toFixed(6).replace(/\.?0+$/, "");
 }
@@ -25068,12 +25033,11 @@ function generateBuyEntry(trade, logger) {
   const fees = trade.costs;
   const cashOut = totalCost + fees;
   logger?.debug(`Generating Buy entry: ${qty} ${trade.symbol} @ ${price} ${trade.currency}`);
-  const commodity = formatCommodity(trade.symbol);
   let entry = `${date5} ${description}
 `;
   entry += `    ; swissquote:order:${trade.orderNum} isin:${trade.isin}
 `;
-  entry += `    assets:investments:stocks:${trade.symbol}    ${qty} ${commodity} @ ${price} ${trade.currency}
+  entry += `    assets:investments:stocks:${trade.symbol}    ${qty} @ ${price} ${trade.currency}
 `;
   if (fees > 0) {
     entry += `    expenses:fees:trading:swissquote    ${formatAmount2(fees, trade.currency)}
@@ -25093,7 +25057,6 @@ function generateSellEntry(trade, consumed, logger) {
   const cashIn = saleProceeds - fees;
   const gain = calculateCapitalGain(consumed, salePrice, trade.quantity);
   logger?.debug(`Generating Sell entry: ${qty} ${trade.symbol} @ ${salePrice} ${trade.currency}, gain: ${gain.toFixed(2)}`);
-  const commodity = formatCommodity(trade.symbol);
   let entry = `${date5} ${description}
 `;
   entry += `    ; swissquote:order:${trade.orderNum} isin:${trade.isin}
@@ -25104,7 +25067,7 @@ function generateSellEntry(trade, consumed, logger) {
   for (const c of consumed) {
     const lotQty = formatQuantity(c.quantity);
     const lotPrice = formatPrice(c.lot.costBasis);
-    entry += `    assets:investments:stocks:${trade.symbol}    -${lotQty} ${commodity} @ ${lotPrice} ${trade.currency}
+    entry += `    assets:investments:stocks:${trade.symbol}    -${lotQty} @ ${lotPrice} ${trade.currency}
 `;
   }
   entry += `    assets:broker:swissquote:${trade.currency.toLowerCase()}    ${formatAmount2(cashIn, trade.currency)}
@@ -25152,14 +25115,13 @@ function generateSplitEntry(action, oldQuantity, newQuantity, logger) {
 `;
   entry += `    ; Ratio: ${ratio.toFixed(4)} (${oldQuantity} -> ${newQuantity})
 `;
-  const commodity = formatCommodity(action.symbol);
-  entry += `    assets:investments:stocks:${action.symbol}    -${formatQuantity(oldQuantity)} ${commodity}
+  entry += `    assets:investments:stocks:${action.symbol}    -${formatQuantity(oldQuantity)}
 `;
-  entry += `    equity:conversion    ${formatQuantity(oldQuantity)} ${commodity}
+  entry += `    equity:conversion    ${formatQuantity(oldQuantity)}
 `;
-  entry += `    equity:conversion    -${formatQuantity(newQuantity)} ${commodity}
+  entry += `    equity:conversion    -${formatQuantity(newQuantity)}
 `;
-  entry += `    assets:investments:stocks:${action.symbol}    ${formatQuantity(newQuantity)} ${commodity}
+  entry += `    assets:investments:stocks:${action.symbol}    ${formatQuantity(newQuantity)}
 `;
   return entry;
 }
@@ -25176,11 +25138,10 @@ function generateWorthlessEntry(action, removedLots, logger) {
 `;
   entry += `    ; Total loss: ${totalCost.toFixed(2)} ${currency}
 `;
-  const commodity = formatCommodity(action.symbol);
   for (const lot of removedLots) {
     const qty = formatQuantity(lot.quantity);
     const price = formatPrice(lot.costBasis);
-    entry += `    assets:investments:stocks:${action.symbol}    -${qty} ${commodity} @ ${price} ${currency}
+    entry += `    assets:investments:stocks:${action.symbol}    -${qty} @ ${price} ${currency}
 `;
   }
   entry += `    expenses:losses:capital    ${formatAmount2(totalCost, currency)}
@@ -25205,18 +25166,16 @@ function generateMultiWayMergerEntry(group, crossCurrencyOutgoingSymbols, logger
   }
   for (const out of group.outgoing) {
     const qty = formatQuantity(Math.abs(out.quantity));
-    const commodity = formatCommodity(out.symbol);
-    entry += `    assets:investments:stocks:${out.symbol}    -${qty} ${commodity}
+    entry += `    assets:investments:stocks:${out.symbol}    -${qty}
 `;
-    entry += `    equity:conversion    ${qty} ${commodity}
+    entry += `    equity:conversion    ${qty}
 `;
   }
   for (const inc of group.incoming) {
     const qty = formatQuantity(Math.abs(inc.quantity));
-    const commodity = formatCommodity(inc.symbol);
-    entry += `    equity:conversion    -${qty} ${commodity}
+    entry += `    equity:conversion    -${qty}
 `;
-    entry += `    assets:investments:stocks:${inc.symbol}    ${qty} ${commodity}
+    entry += `    assets:investments:stocks:${inc.symbol}    ${qty}
 `;
   }
   return entry;
@@ -25226,12 +25185,11 @@ function generateRightsDistributionEntry(action, logger) {
   const qty = formatQuantity(Math.abs(action.quantity));
   const description = escapeDescription(`Rights Distribution: ${action.symbol} - ${action.name}`);
   logger?.debug(`Generating Rights Distribution entry: ${qty} ${action.symbol}`);
-  const commodity = formatCommodity(action.symbol);
   let entry = `${date5} ${description}
 `;
   entry += `    ; swissquote:order:${action.orderNum} isin:${action.isin}
 `;
-  entry += `    assets:investments:stocks:${action.symbol}    ${qty} ${commodity} @ 0.00 CAD
+  entry += `    assets:investments:stocks:${action.symbol}    ${qty} @ 0.00 CAD
 `;
   entry += `    income:capital-gains:rights-distribution    0.00 CAD
 `;
@@ -25799,23 +25757,6 @@ async function preprocessSwissquote(csvPath, projectDir, currency, year, lotInve
       logger?.logStep("write-journal", "success", `Created ${path13.basename(journalFile)} with ${journalEntries.length} entries`);
     }
   }
-  const stockSymbols = new Set;
-  for (const trade of trades) {
-    stockSymbols.add(trade.symbol);
-  }
-  for (const dividend of dividends) {
-    stockSymbols.add(dividend.symbol);
-  }
-  for (const action of corporateActions) {
-    stockSymbols.add(action.symbol);
-    if (action.newSymbol) {
-      stockSymbols.add(action.newSymbol);
-    }
-  }
-  if (stockSymbols.size > 0) {
-    const commodityJournalPath = path13.join(projectDir, "ledger", "investments", "commodities.journal");
-    ensureCommodityDeclarations(commodityJournalPath, stockSymbols, logger);
-  }
   logger?.logResult({
     totalRows: stats.totalRows,
     simpleTransactions: stats.simpleTransactions,
@@ -26744,7 +26685,7 @@ to produce equity conversion entries for Bitcoin purchases.
   }
 });
 // src/index.ts
-var __dirname2 = dirname7(fileURLToPath3(import.meta.url));
+var __dirname2 = dirname6(fileURLToPath3(import.meta.url));
 var AGENT_FILE = join15(__dirname2, "..", "agent", "accountant.md");
 var AccountantPlugin = async () => {
   const agent = loadAgent(AGENT_FILE);

package/docs/tools/import-pipeline.md

@@ -220,8 +220,7 @@ See [classify-statements](classify-statements.md) for details.
    - **Rights Distributions**: adds shares at zero cost basis
    - **Cross-currency mergers**: saves outgoing side to pending state, loads when incoming side is processed
 8. Generates per-year investment journal entries (`ledger/investments/{year}-{currency}.journal`) based on the CSV filename's date range
-9. Collects all stock ticker symbols and writes commodity declarations to `ledger/investments/commodities.journal` (merged with existing, sorted, idempotent)
-10. Outputs filtered CSV (simple transactions only) for hledger rules import
+9. Outputs filtered CSV (simple transactions only) for hledger rules import
 
 **Symbol Map**: Maps Swissquote's internal symbol names (e.g., `GOLD ROYALTY RG`, `VIZSLA WT 12.25`) to canonical ticker symbols (e.g., `GROY`, `VROY-WT`). Applied before any processing, so journal entries, lot inventory filenames, and account names all use the mapped symbols. See [Symbol Map Configuration](../configuration/symbol-map.md).
 

package/package.json

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