* feat(invoices): add Plusgiro input to bank details settings Plusgiro was already persisted, validated by the API schema, rendered on the invoice PDF and toggleable via "Visa plusgiro" — but the settings UI had no field to enter the number, so plusgiro-only users could not fill it in. Add the input next to Bankgiro with Luhn validation and hyphen formatting, include it in the save payload (normalised on save so raw digits still match the dashed schema format), and add sv/en strings. Adds validatePlusgiroNumber/formatPlusgiroNumber helpers + tests. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(invoices): respect non-VAT-registered seller in PDF preview + portal tooltips Two user-reported bugs: - PDF preview (/api/invoices/preview-pdf) ignored company.vat_registered and fell back to the customer-driven 25% rate, so a non-momsregistrerad seller saw VAT in the review step even though the created invoice books none. Mirror the server-side write gate (build-invoice-write.ts): force 0% when vat_registered is false (delivery notes excepted). - InfoTooltip rendered TooltipContent without a Portal, so tooltips were clipped by the scrollable DialogContent (overflow-y-auto) in the send-invoice journal-entry review. Wrap in TooltipPrimitive.Portal. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(transactions): book library mall from its literal lines, not a lossy fallback Booking a bank transaction with a user-created booking-template (mall) via the convertible "QuickReview" fast path reduced the template to a single category + one account_override, silently discarding the chosen debit/credit. A kundinbetalning mall (D 1930 / K 1510) booked as a generic cost (D 6991 / K 1930), or with a VAT line as D 1930 / K 1930 / K 2611 — and the result flipped with the direction inferred from the business/settlement line tags, so visually-identical templates produced different verifikationer. Route every library template through the journal-entry editor (applyTemplate -> /book), which posts the literal lines, regardless of convertibility. Add regression tests locking the contract. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(bookkeeping): make the booking-time duplicate guard bypassable TRANSACTION_BOOK_POSSIBLE_DUPLICATE told users they could "book anyway" but the UI dead-ended on a toast with no way to do so. Add a shared DuplicateBookingDialog that surfaces the already-booked sibling and lets the user review it or book anyway (force bound to the reviewed candidate, which the server re-detects so a stale id cannot wave the guard away). - Wire the dialog into the /transactions categorize flow and the manual booking dialog (JournalEntryForm -> /api/transactions/[id]/book) - Bind the override to expected_duplicate_transaction_id OR expected_duplicate_journal_entry_id so ledger-only vouchers (paid invoice, salary run) can be confirmed too - Extend the guard to the pending-operations commit path and the MCP server - Tests for book/categorize routes, detection, and the commit guard Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(bookkeeping): log duplicate-guard bypass to behandlingshistorik in the agent commit path The web /book and /categorize routes append a durable BankTransactionDuplicateDismissed event when a user books over a detected possible double-booking. The agent commit path (commitCategorizeTransaction, commitMarkInvoicePaid) skipped the guard silently on allow_duplicate=true, leaving no behandlingshistorik — an auditor could not reconstruct why the duplicate was allowed (BFNAR 2013:2 kap 8). When allow_duplicate=true, re-detect the candidate and append the dismissal event (BankTransactionDuplicateDismissed for the bank-line path, InvoiceDuplicatePaymentDismissed for mark-paid). Best-effort — a logging failure never blocks a legitimate booking. Payloads stay PII-safe (ids, amounts, dates only — no customer or merchant name). Also fix the misleading DuplicateBookingDialog JSDoc: the retry binds expected_duplicate_journal_entry_id, not candidate.transaction_id, so the systemdokumentation matches the actual control (BFL 7 kap). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * test(mcp-server): stub booking-duplicate guard in receipt-matcher categorize tests The gnubok_categorize_transaction tool runs the booking-time duplicate guard before staging; its detection queries consumed the queued supabase mock results, so the staging assertions saw a thrown duplicate error instead of a staged op. Mock detectBookingDuplicate to "no duplicate" since these tests don't exercise that path. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * refactor(transactions): use roundOre for duplicate-guard öre rounding Replace naive Math.round(x*100)/100 with roundOre() from @/lib/money in the booking-time duplicate guard (detection lib, commit executor, MCP categorize tool), satisfying the no-new-antipatterns ratchet guard. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(sie-export): paginate journal entries and lines to prevent truncation * fix(bookkeeping): keep the Verifikat/Utkast toggle reachable on an empty list The journal entry list early-returned a pristine empty card whenever the visible list was empty and no filter was active, returning before the Verifikat/Utkast toggle rendered. This stranded users with only drafts (no posted entries) and users who emptied the drafts list, who then had to use the main menu to get back to posted entries. Narrow the early return to a genuinely empty ledger (committed view, no drafts, no filters); make the in-list empty placeholder context-aware (no drafts / no filter matches / no posted entries yet); resolve the draft count before clearing loading on an empty committed list to avoid a toggle flicker. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(enable-banking): persist psu_type and reuse it on reconnect Reconnecting a bank connection re-derived psu_type from the company entity_type every time (aktiebolag -> 'business'), silently overriding the type the user actually authorized with. A connection that only signs as 'personal' — common for AB owners who use a personal Mobile BankID, notably at Handelsbanken — flipped back to 'business' on every consent renewal and failed at the bank's signing step. - Add nullable bank_connections.psu_type column (idempotent migration) - Persist psu_type on connect; on reconnect reuse the stored value (explicit client override still wins) - Let users switch account type (Företag/Privat) from the reconnect button - Tests for persistence, reuse, and override Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(import): set maxDuration=300 on bank-file execute to prevent timeout A full-year bank file (300+ rows) runs a sequential per-row ingest that takes ~85s of server time. The execute route set no maxDuration, so it inherited the platform default and was killed mid-run — the import "spins then aborts" for the user. Match the SIE import route and give it a 5-minute budget. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * fix(transactions): add assistant entry point on transaction rows The agent ("Lena") could only be reached from Dokumentinkorgen, and only once an underlag was matched to a transaction. Transaktioner is the most common starting point for booking, so users could not start a booking with the assistant from there at all. Add a per-row "Fråga [namn]" button on unbooked transaction rows that opens the existing transaction.categorization intent with the row's transaction_id. The intent already reads any linked underlag, so it works whether or not a receipt is attached. No new logic — only the missing entry point. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * feat(invoices): enable Swish payment QR on invoices Flip SHOW_SWISH_ON_INVOICE on so the Swish row and payment QR render on the invoice PDF, and make the "Visa Swish" settings toggle live (it was hardcoded disabled). The preview-pdf route now builds the QR too, so it shows in forhandsvisning. Position the QR in the top-right of the payment box. No Swish API integration -- the QR is generated offline and prefills the customer Swish app; reconciliation stays via bank matching. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(bookkeeping): scope verifikat list to current year, add storno action, clarify correction preview Three UI fixes from user feedback; no engine logic changed. - List defaults to the current räkenskapsår instead of all years. Voucher numbers run per fiscal year (one A42/year), so showing every year at once made them look like duplicates. New resolveCurrentPeriodId helper. - Add 'Återför (storno)' action on the entry detail page and list row, wiring the existing reverseEntry — a pure reversal (BFL 5 kap 5§) with no replacement, distinct from 'Rätta'. - Correction 'Effekt per konto' preview now labels a removed account 'tas bort' (vs a bare dash) and warns when the proposal is unbalanced; dialog explains the rows are the full new verifikat. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * feat(bank_connections): add psu_type column to persist chosen authorization type * feat(errors): add CannotReverseStornoError for handling reversal of storno or correction entries --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
320 lines
10 KiB
TypeScript
320 lines
10 KiB
TypeScript
import type { SupabaseClient } from '@supabase/supabase-js'
|
|
import { fetchAllRows } from '@/lib/supabase/fetch-all'
|
|
import { getBranding } from '@/lib/branding/service'
|
|
import { getOpeningBalances } from './opening-balances'
|
|
import type { SIEExportOptions, JournalEntry, JournalEntryLine, BASAccount } from '@/types'
|
|
|
|
function sanitizeProgramName(str: string): string {
|
|
return str.replace(/"/g, '').replace(/[\r\n]/g, ' ').substring(0, 60)
|
|
}
|
|
|
|
/**
|
|
* Generate SIE4 export file
|
|
*
|
|
* SIE (Standard Import Export) is the Swedish standard format for
|
|
* transferring accounting data between systems.
|
|
*
|
|
* Format: CP437 encoded text file (we'll use UTF-8 as modern systems accept it)
|
|
* Line format: #TAG field1 field2 ...
|
|
*/
|
|
export async function generateSIEExport(
|
|
supabase: SupabaseClient,
|
|
companyId: string,
|
|
options: SIEExportOptions
|
|
): Promise<string> {
|
|
|
|
// Fetch fiscal period
|
|
const { data: period } = await supabase
|
|
.from('fiscal_periods')
|
|
.select('*')
|
|
.eq('id', options.fiscal_period_id)
|
|
.eq('company_id', companyId)
|
|
.single()
|
|
|
|
if (!period) {
|
|
throw new Error('Fiscal period not found')
|
|
}
|
|
|
|
// Fetch previous fiscal year for #RAR -1 (per SIE spec, both years should be present)
|
|
const { data: prevPeriod } = await supabase
|
|
.from('fiscal_periods')
|
|
.select('period_start, period_end')
|
|
.eq('company_id', companyId)
|
|
.lt('period_end', period.period_start)
|
|
.order('period_end', { ascending: false })
|
|
.limit(1)
|
|
.single()
|
|
|
|
// Fetch all accounts
|
|
const accounts = await fetchAllRows(({ from, to }) =>
|
|
supabase
|
|
.from('chart_of_accounts')
|
|
.select('*')
|
|
.eq('company_id', companyId)
|
|
.eq('is_active', true)
|
|
.order('account_number')
|
|
.range(from, to)
|
|
)
|
|
|
|
// Fetch all posted journal entries — paginated to avoid truncation.
|
|
// The previous nested `select('*, lines:journal_entry_lines(*)')` hit
|
|
// PostgREST's response-row ceiling on the embedded resource and silently
|
|
// truncated large periods (~30 vouchers). Fetch entries and lines as two
|
|
// separate paginated queries and stitch them together in memory, mirroring
|
|
// journal-register.ts.
|
|
const entries = await fetchAllRows<JournalEntry>(({ from, to }) => {
|
|
let q = supabase
|
|
.from('journal_entries')
|
|
.select('*')
|
|
.eq('company_id', companyId)
|
|
.eq('fiscal_period_id', options.fiscal_period_id)
|
|
.in('status', ['posted', 'reversed'])
|
|
|
|
if (options.exclude_year_end_closing) {
|
|
q = q.neq('source_type', 'year_end')
|
|
}
|
|
|
|
return q.order('voucher_number').range(from, to)
|
|
})
|
|
|
|
// Fetch all lines for those entries, filtered server-side via an inner join
|
|
// so the same company/period/status (and year-end exclusion) constraints
|
|
// apply, then group by journal_entry_id.
|
|
const allLines = await fetchAllRows<JournalEntryLine & { journal_entry_id: string }>(({ from, to }) => {
|
|
let q = supabase
|
|
.from('journal_entry_lines')
|
|
.select('*, journal_entries!inner(company_id, fiscal_period_id, status, source_type)')
|
|
.eq('journal_entries.company_id', companyId)
|
|
.eq('journal_entries.fiscal_period_id', options.fiscal_period_id)
|
|
.in('journal_entries.status', ['posted', 'reversed'])
|
|
|
|
if (options.exclude_year_end_closing) {
|
|
q = q.neq('journal_entries.source_type', 'year_end')
|
|
}
|
|
|
|
// eslint-disable-next-line @typescript-eslint/no-explicit-any
|
|
return q.range(from, to) as any
|
|
})
|
|
|
|
const linesByEntryId = new Map<string, JournalEntryLine[]>()
|
|
for (const line of allLines) {
|
|
const list = linesByEntryId.get(line.journal_entry_id)
|
|
if (list) {
|
|
list.push(line)
|
|
} else {
|
|
linesByEntryId.set(line.journal_entry_id, [line])
|
|
}
|
|
}
|
|
|
|
for (const entry of entries) {
|
|
entry.lines = linesByEntryId.get(entry.id) || []
|
|
}
|
|
|
|
// Fetch cost centers and projects for dimension records
|
|
const { data: costCenters } = await supabase
|
|
.from('cost_centers')
|
|
.select('*')
|
|
.eq('company_id', companyId)
|
|
.eq('is_active', true)
|
|
.order('code')
|
|
|
|
const { data: projects } = await supabase
|
|
.from('projects')
|
|
.select('*')
|
|
.eq('company_id', companyId)
|
|
.eq('is_active', true)
|
|
.order('code')
|
|
|
|
const lines: string[] = []
|
|
const now = new Date()
|
|
|
|
// === Header ===
|
|
lines.push('#FLAGGA 0')
|
|
lines.push('#FORMAT PC8')
|
|
lines.push('#SIETYP 4')
|
|
const programName = sanitizeProgramName(options.program_name || getBranding().appName)
|
|
lines.push(`#PROGRAM "${programName}" "1.0"`)
|
|
lines.push(`#GEN ${formatSIEDate(now)}`)
|
|
|
|
if (options.org_number) {
|
|
lines.push(`#ORGNR ${options.org_number}`)
|
|
}
|
|
|
|
lines.push(`#FNAMN "${escapeQuotes(options.company_name)}"`)
|
|
|
|
// === Fiscal year ===
|
|
// #RAR 0 = current year, #RAR -1 = previous year (both should be present per spec)
|
|
// Use date strings directly to avoid timezone conversion issues
|
|
lines.push(`#RAR 0 ${dateStringToSIE(period.period_start)} ${dateStringToSIE(period.period_end)}`)
|
|
|
|
if (prevPeriod) {
|
|
lines.push(`#RAR -1 ${dateStringToSIE(prevPeriod.period_start)} ${dateStringToSIE(prevPeriod.period_end)}`)
|
|
}
|
|
|
|
// === Dimension definitions ===
|
|
// SIE standard: dimension 1 = kostnadsställe, dimension 6 = projekt
|
|
const hasCostCenters = costCenters && costCenters.length > 0
|
|
const hasProjects = projects && projects.length > 0
|
|
|
|
if (hasCostCenters) {
|
|
lines.push('#DIM 1 "Kostnadsställe"')
|
|
}
|
|
if (hasProjects) {
|
|
lines.push('#DIM 6 "Projekt"')
|
|
}
|
|
|
|
// === Dimension objects (#OBJEKT) ===
|
|
for (const cc of costCenters || []) {
|
|
lines.push(`#OBJEKT 1 "${escapeQuotes(cc.code)}" "${escapeQuotes(cc.name)}"`)
|
|
}
|
|
for (const proj of projects || []) {
|
|
lines.push(`#OBJEKT 6 "${escapeQuotes(proj.code)}" "${escapeQuotes(proj.name)}"`)
|
|
}
|
|
|
|
// === Chart of accounts ===
|
|
for (const account of (accounts as BASAccount[]) || []) {
|
|
lines.push(`#KONTO ${account.account_number} "${escapeQuotes(account.account_name)}"`)
|
|
|
|
// #SRU records from chart_of_accounts.sru_code
|
|
if (account.sru_code) {
|
|
lines.push(`#SRU ${account.account_number} ${account.sru_code}`)
|
|
}
|
|
}
|
|
|
|
// === Opening balances (IB) ===
|
|
// Routes through getOpeningBalances() so we get the same fallback as trial
|
|
// balance / balance sheet: when opening_balance_entry_id is NULL — which is
|
|
// expected after continuation SIE imports (sie-import.ts skips creating an
|
|
// IB entry once prior posted activity exists) — the compute_prior_opening_
|
|
// balances RPC derives IB from earlier journal lines instead of silently
|
|
// emitting zero #IB records and producing wrong #UB values.
|
|
const openingBalancesByAccount = new Map<string, number>()
|
|
const { balances: obBalances } = await getOpeningBalances(supabase, companyId, {
|
|
period_start: period.period_start,
|
|
opening_balance_entry_id: period.opening_balance_entry_id ?? null,
|
|
})
|
|
|
|
for (const [accountNumber, { debit, credit }] of obBalances) {
|
|
const amount = Math.round(((Number(debit) || 0) - (Number(credit) || 0)) * 100) / 100
|
|
if (amount === 0) continue
|
|
lines.push(`#IB 0 ${accountNumber} ${formatAmount(amount)}`)
|
|
openingBalancesByAccount.set(accountNumber, amount)
|
|
}
|
|
|
|
// === Journal entries (VER + TRANS) ===
|
|
for (const entry of (entries as JournalEntry[]) || []) {
|
|
const entryLines = (entry.lines as JournalEntryLine[]) || []
|
|
const entryDate = dateStringToSIE(entry.entry_date)
|
|
const series = entry.voucher_series || 'A'
|
|
const description = escapeQuotes(entry.description)
|
|
|
|
lines.push(`#VER "${series}" ${entry.voucher_number} ${entryDate} "${description}"`)
|
|
lines.push('{')
|
|
|
|
for (const line of entryLines) {
|
|
const amount =
|
|
line.debit_amount > 0
|
|
? line.debit_amount
|
|
: -line.credit_amount
|
|
|
|
const lineDesc = line.line_description
|
|
? ` "${escapeQuotes(line.line_description)}"`
|
|
: ''
|
|
|
|
// Build dimension object list for #TRANS line
|
|
const dimParts: string[] = []
|
|
if (line.cost_center) {
|
|
dimParts.push(`1 "${escapeQuotes(line.cost_center)}"`)
|
|
}
|
|
if (line.project) {
|
|
dimParts.push(`6 "${escapeQuotes(line.project)}"`)
|
|
}
|
|
const objList = dimParts.length > 0 ? `{${dimParts.join(' ')}}` : '{}'
|
|
|
|
lines.push(`\t#TRANS ${line.account_number} ${objList} ${formatAmount(amount)} ${entryDate}${lineDesc}`)
|
|
}
|
|
|
|
lines.push('}')
|
|
}
|
|
|
|
// === Closing balances (UB for balance sheet, RES for income statement) ===
|
|
// Movement balances from journal entries
|
|
const movementBalances = calculateBalances(entries as JournalEntry[])
|
|
|
|
// Merge all accounts that have either IB or movements
|
|
const allAccountNumbers = new Set([
|
|
...openingBalancesByAccount.keys(),
|
|
...movementBalances.keys(),
|
|
])
|
|
|
|
for (const accountNumber of [...allAccountNumbers].sort()) {
|
|
const accountClass = parseInt(accountNumber[0])
|
|
const ib = openingBalancesByAccount.get(accountNumber) || 0
|
|
const movement = movementBalances.get(accountNumber) || 0
|
|
|
|
if (accountClass <= 2) {
|
|
// Balance sheet: UB = IB + movements during period
|
|
const ub = Math.round((ib + movement) * 100) / 100
|
|
lines.push(`#UB 0 ${accountNumber} ${formatAmount(ub)}`)
|
|
} else {
|
|
// Income statement: RES = movements only (IB should be zero)
|
|
lines.push(`#RES 0 ${accountNumber} ${formatAmount(movement)}`)
|
|
}
|
|
}
|
|
|
|
return lines.join('\r\n') + '\r\n'
|
|
}
|
|
|
|
/**
|
|
* Format a Date object for SIE: YYYYMMDD
|
|
*/
|
|
function formatSIEDate(date: Date): string {
|
|
const y = date.getFullYear()
|
|
const m = String(date.getMonth() + 1).padStart(2, '0')
|
|
const d = String(date.getDate()).padStart(2, '0')
|
|
return `${y}${m}${d}`
|
|
}
|
|
|
|
/**
|
|
* Convert a "YYYY-MM-DD" date string to SIE format "YYYYMMDD"
|
|
* without going through Date object (avoids timezone issues)
|
|
*/
|
|
function dateStringToSIE(dateStr: string): string {
|
|
return dateStr.replace(/-/g, '')
|
|
}
|
|
|
|
/**
|
|
* Format amount for SIE (no thousands separator, . as decimal)
|
|
*/
|
|
function formatAmount(amount: number): string {
|
|
const rounded = Math.round(amount * 100) / 100
|
|
return rounded.toFixed(2)
|
|
}
|
|
|
|
/**
|
|
* Escape double quotes in SIE strings
|
|
*/
|
|
function escapeQuotes(str: string): string {
|
|
return str.replace(/"/g, '\\"')
|
|
}
|
|
|
|
/**
|
|
* Calculate net balances per account from journal entries
|
|
*/
|
|
function calculateBalances(
|
|
entries: JournalEntry[]
|
|
): Map<string, number> {
|
|
const balances = new Map<string, number>()
|
|
|
|
for (const entry of entries || []) {
|
|
const lines = (entry.lines as JournalEntryLine[]) || []
|
|
for (const line of lines) {
|
|
const current = balances.get(line.account_number) || 0
|
|
const netAmount = (Number(line.debit_amount) || 0) - (Number(line.credit_amount) || 0)
|
|
balances.set(line.account_number, Math.round((current + netAmount) * 100) / 100)
|
|
}
|
|
}
|
|
|
|
return balances
|
|
}
|