dotemacs/docs/design/2026-07-10-ledger-config-audit.org, branch mainMy Emacs configuration
https://git.cjennings.net/dotemacs/atom?h=main2026-07-10T06:23:31+00:00fix(flycheck): lint ledger buffers, which nothing ever did2026-07-10T06:23:31+00:00Craig Jenningsc@cjennings.net2026-07-10T06:23:31+00:00urn:sha1:55b85754d78f268907ad5fa5be886973bbef921d
flycheck-ledger registers a ledger checker, but a checker only runs where flycheck-mode is on, and flycheck-mode was hooked to sh-mode and emacs-lisp-mode only. There's no global-flycheck-mode. So an unbalanced transaction in a financial file produced no warning at all, while the ledger module's commentary advertised linting.
The hook goes in flycheck-config, not ledger-config, so that list stays the one answer to "where is flycheck turned on?".
Verified on an unbalanced fixture: flycheck now reports "Transaction does not balance" with the $10.00 remainder, on the right line.
I also accepted the whole-buffer sort the audit flagged. ledger-sort-startkey keys on the ISO date alone, and sort-subr is stable, so a save orders by date and leaves same-day transactions where they were typed. The audit note records both.
test(ledger): audit ledger-config and pin its current behavior2026-07-10T05:44:39+00:00Craig Jenningsc@cjennings.net2026-07-10T05:44:39+00:00urn:sha1:621287447f149eb571e00c067e410704c4ebf968
The linting doesn't run. flycheck-ledger loads and registers the ledger checker, but flycheck-mode is hooked to sh-mode and emacs-lisp-mode only, and there's no global-flycheck-mode. I checked a live ledger-mode buffer: checker registered, flycheck-mode nil. Unbalanced transactions and typo'd account names produce no warning, on a financial file, while the module commentary claims linting.
Three smaller findings. Every save reorders the whole file, because ledger-mode-clean-buffer calls ledger-sort-buffer over point-min to point-max. The demoted error around it lets a save proceed after a partial rewrite, with nothing to roll back. Reconcile has no confirmation and clears whole transactions.
Five characterization tests pin what exists now, including the clean-on-save default the audit questions. I fixed nothing: the guardrail choices are a separate task, and only the flycheck gap is a defect rather than a preference.