archetech / archon / 22961272368

94%

push

github

fix: Check importBatch error flag before imported/processed flag (#193)

* fix: Check importBatch error flag before imported/processed flag

Fixes #116 — when an item has both `error` and `imported` set, the
previous order skipped it as "already done" before the error check
could fire. Swapping the checks ensures errored items are handled
correctly while preserving the `retry` path used by retryFailedImports.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: Only skip importBatch when processEvents fully completed

Items where processEvents returned {busy: true} or {pending: >0} were
being permanently skipped because any truthy ProcessEventsResult object
satisfied the old check. Now only skip when not busy and pending === 0.
Also adds !retry to the condition so retries can reprocess these items.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

1746 of 1924 branches covered (90.75%)

Branch coverage included in aggregate %.

3736 of 3893 relevant lines covered (95.97%)

1176.72 hits per line