[GH-ISSUE #148] fix(application): include failure reason in reference import deduplication key #96

New issue

Closed

opened 2026-10-15 18:21:05 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-15 18:21:05 +00:00

Owner

Originally created by @morluto on GitHub (Jul 14, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/148

Description

ReferenceSourceImport deduplicates parse failures by (path, parser) only, dropping separate failure reasons for the same path+parser pair.

Impact

Diagnostics may hide meaningful distinctions between multiple import failures and reduce forensic clarity.

Expected behavior

Include failure reason/normalized error data in dedupe key or group key.

Files

src/application/ReferenceSourceImport.ts (around lines 207-218)

Originally created by @morluto on GitHub (Jul 14, 2026). Original GitHub issue: https://github.com/morluto/rea/issues/148 ## Description `ReferenceSourceImport` deduplicates parse failures by `(path, parser)` only, dropping separate failure reasons for the same path+parser pair. ## Impact Diagnostics may hide meaningful distinctions between multiple import failures and reduce forensic clarity. ## Expected behavior Include failure reason/normalized error data in dedupe key or group key. ## Files - `src/application/ReferenceSourceImport.ts` (around lines 207-218)

bsakaguchi

2026-10-15 18:21:05 +00:00