ory / lumen / 23654796054

68%

Build Type

push

github

Committed by web-flow

Commit Message

fix: reindex stability — sentinel resume, donor safety, worktree subdirs, progress bar (#67)

* docs: add spec for non-blocking semantic_search with partial results

When reindexing takes longer than 15s, semantic_search returns stale
results with a warning instead of blocking the agent indefinitely.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: address spec review feedback for non-blocking search

- Buffered done channel (cap 1) to prevent goroutine leak on timeout
- Goroutine calls touchChecked on success for correct TTL behavior
- Nil progress func in goroutine (request ctx may be gone)
- Log errors from background EnsureFresh at Warn level
- sync.WaitGroup for graceful shutdown in Close()

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: add implementation plan for non-blocking semantic_search

7-task plan with TDD approach: struct changes, WaitGroup, timeout
goroutine, formatSearchResults, and tests including a test hook
(ensureFreshFunc) to exercise the 15s timeout path.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor(cmd): add StaleWarning field and WaitGroup to indexerCache

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat(cmd): Close() waits for background reindex goroutines

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat(cmd): non-blocking ensureIndexed with 15s timeout and background reindex

EnsureFresh now runs in a goroutine. If it completes within 15s, results
are returned normally. If it exceeds the timeout, stale results are
returned immediately with a StaleWarning while reindexing continues in
the background (up to 10min). The goroutine acquires an exclusive flock
to avoid concurrent writes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* style(cmd): use WaitGroup.Go instead of manual Add/Done pattern

Go 1.25+ provides wg.Go() which simplifies goroutine tra... (continued)

Run Details

54 of 73 new or added lines in 5 files covered. (73.97%)

1 existing line in 1 file now uncovered.

2531 of 3725 relevant lines covered (67.95%)

44.04 hits per line