andgineer / dinary-server / 24611685464

77%

push

github

fix(import): unlink budget_YYYY.duckdb before re-import

`import_year` is the destructive bootstrap path called only by
`inv rebuild-budget --yes`. The pre-existing budget DB on the server may
have been created under an incompatible schema (e.g. the 4D model), and
yoyo's migration tracker would treat the new 0001 schema as already
applied, so the freshly re-opened DB still has the old shape. The DELETE
sequence inside `import_year` then crashes with:

  CatalogException: Table with name expense_tags does not exist!

Unlink the budget file first so that `get_budget_connection` runs the
current 0001_initial_schema.sql against an empty file. The DELETE wipe
stays as defence in depth; on a fresh DB it is a no-op.

Promote `_budget_path` to public `budget_path` so `import_sheet` can
reach it without tripping ruff's SLF001 rule.

Made-with: Cursor

5 of 5 new or added lines in 2 files covered. (100.0%)

1463 of 2097 relevant lines covered (69.77%)

0.7 hits per line