Money: successful import source cleanup shows a false error #1438

Closed
opened 2026-10-08 03:54:58 +00:00 by kayg · 2 comments
Owner

Steps:

  1. Open Money → Import a Budget.
  2. Select a YNAB JSON export and continue to Review.

Expected: after the source copy moves to Trash, Review shows no cleanup error.
Actual: Review shows “The temporary copy could not be removed” even though the real Files API returned 200 for Trash and 404 for the follow-up stat of the source path. The import can still be published.

Evidence: artifacts/money-bash3/ynab-split-transfer-review-light-1440.png (synthetic export, real production build/server). The local trace showed POST /api/v1/files/trash 200, then GET /api/v1/files/stat 404.

Severity: cosmetic.

Steps: 1. Open Money → Import a Budget. 2. Select a YNAB JSON export and continue to Review. Expected: after the source copy moves to Trash, Review shows no cleanup error. Actual: Review shows “The temporary copy could not be removed” even though the real Files API returned 200 for Trash and 404 for the follow-up stat of the source path. The import can still be published. Evidence: `artifacts/money-bash3/ynab-split-transfer-review-light-1440.png` (synthetic export, real production build/server). The local trace showed `POST /api/v1/files/trash 200`, then `GET /api/v1/files/stat 404`. Severity: cosmetic.
Author
Owner

Fixed on job/bash3-money in commit 89c06896b: Money treats a successful one-item Files Trash response as the returned Trash identity. The regression test covers a Trash name equal to the source path. Focused import-session.svelte.test.ts passed (37 tests). The real production YNAB import probe passed with Files Trash returning HTTP 200 and showed no cleanup error.

Fixed on `job/bash3-money` in commit `89c06896b`: Money treats a successful one-item Files Trash response as the returned Trash identity. The regression test covers a Trash name equal to the source path. Focused `import-session.svelte.test.ts` passed (37 tests). The real production YNAB import probe passed with Files Trash returning HTTP 200 and showed no cleanup error.
Author
Owner

Deployed to calternal.cloud in merge round 35a (build cc1af3c12, live 2026-10-09 00:12 IST). Closing.

Deployed to calternal.cloud in merge round 35a (build cc1af3c12, live 2026-10-09 00:12 IST). Closing.
kayg closed this issue 2026-10-08 18:42:06 +00:00
Commenting is not possible because the repository is archived.
No labels
No milestone
No project
No assignees
1 participant
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal-v0#1438
No description provided.