factEnrichmentWorker walks the pending queue twice per tick to write one log line #191

2026-08-06T22:33:01+02:00

claude commented

2026-08-06 22:33:01 +02:00

Closes Vikunja #647.

Acceptance criteria and quality gate are in TASK.md on this branch.
Review the review, not the diff — leave comments and the agent will apply them via task start 647.

Closes Vikunja #647. Acceptance criteria and quality gate are in `TASK.md` on this branch. Review the review, not the diff — leave comments and the agent will apply them via `task start 647`.

claude added 1 commit 2026-08-06 22:33:02 +02:00

Count the enrichment backlog without a second query (V-647) 2c0334c4fe

`tick` read `PendingFactResolutions` at the scan limit, then `status`
read it again with the same limit for one log line. Up to 2000 rows per
tick on a database that serialises reads, to say how long the queue is.

`statusOf` counts over a batch the caller already holds, and the tick
passes it the batch it just read. A resolved fact leaves the queue, so
the loop collects what is still pending rather than reporting the
pre-tick count. `status(ctx)` stays as the querying form, for a caller
outside the tick with no batch in hand.

No behaviour change: the three counts still describe one row set, and
the same facts are attempted per tick.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

kami merged commit 7b507dec94 into master

2026-08-06 22:33:54 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kami/Maven#191