* feat(import): say in the SIE wizard that the chart and fiscal year come along
The preview scored the file's accounts against the BAS reference and said
"matchas mot din kontoplan", so a consultant with a 41-account seeded company
read "150 mappade" as "the file's chart replaces mine". A fiscal-year overlap
with a non-empty period was only refused after the mapping step.
- Parse route adds preview.chart (accounts new to THIS company vs already
present, with a sample) via planChartChanges, and preview.fiscalYear from
precheckFiscalPeriod: the containment/overlap verdict extracted out of
ensureFiscalPeriod, which now consumes it, so preview and import cannot drift.
- Preview card renamed to Kontoplan with the counts and the fiscal-year
verdict (match / create / conflict with the import's own refusal text).
- Review step lists the chart among "Vad händer när du importerar?".
- executeSIEImport reports accountsCreated from the account sync; the result
grid gets a Konton skapade card.
No import logic changed; no migration.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014MgxEaU52nJgDQA41svdtC
* fix(import): preview refuses what the import refuses, and the chart card survives "Skapa saknade konton"
Skeptic pass on #2307 refuted the first cut twice:
- "Skapas vid import" was shown for a #RAR the import then refuses under
BFL 3 kap. (19 months, non-month-end finish, mid-month start after an
earlier year). The shape rules move into precheckFiscalPeriod as a fourth
verdict 'invalid' with the same refusal text; ensureFiscalPeriod stays a
consumer of one verdict, same query order.
- The Kontoplan card counted unmapped sources under "Läggs till" and kept
listing them after the create button, while the result said 0 created.
planChartChanges (now client-safe in lib/import/chart-plan.ts) counts
mapped targets only; the create button moves those accounts from
"Ej mappade" to "Finns redan" in place.
- The review line claimed existing accounts keep their name unless you opt
in; the switch defaults to on. Reworded to match.
- Sample names follow the file for identity mappings, as the sync does.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014MgxEaU52nJgDQA41svdtC
---------
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
* refactor(arcim-migration): remove the dead gateway SIE export path
fetchSIEExport and SIEExportFile have had zero callers since the direct
provider clients replaced the Arcim Sync gateway (#181, #718). The path
returned SIE as a pre-decoded string, and the gateway's decode of CP437
bytes as windows-1252 is what wrote the 2026-03-17 mojibake into posted
entries. Deleting it makes the string-typed SIE fetch impossible to
re-wire; a comment marks the grave. The consent lifecycle and entity
accessors stay untouched.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* fix(import): warn when SIE text carries CP437-as-CP1252 mojibake
The 2026-03-17 migration wrote mojibake ("L"neutbetalning"-style C1
specials) into posted entries because the retired gateway handed the
/import-sie handler an already-decoded string: byte-level encoding
detection never saw it, and nothing downstream checked. The live bug is
gone; this is the tripwire so the signature can never land silently
again.
- lib/import/sie-artifact-scan.ts: pure scanner over parsed SIE account
names and voucher/line descriptions, reusing hasCp1252Artifact from
charset-repair; flags at >= 2 hits so a lone legitimate curly quote or
apostrophe cannot false-positive a whole file.
- arcim-migration /import-sie: warn-never-block; the Swedish warning
rides on result.warnings, which the workspace UI already renders, plus
a server-side log.warn.
- wizard parse route: same scan, surfaced through the existing
parse-issue warnings card in the preview, pointing at the first
affected line.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* test(bookkeeping): pin the reported gateway mojibake strings
Adds the four strings reported from the affected company's journal as
reverse_cp437 cases (all reverse losslessly) plus a false-positive
guard: space-padded typography must never route into the CP437
reversal.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---------
Co-authored-by: Jakob Wennberg <311770904+jakobwennberg-oss@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Mattsson