Skip to content

feat: add stderr progress + timing summaries for import and connector sync#297

Merged
hurttlocker merged 1 commit intomainfrom
xmate/progress-292
Mar 1, 2026
Merged

feat: add stderr progress + timing summaries for import and connector sync#297
hurttlocker merged 1 commit intomainfrom
xmate/progress-292

Conversation

@hurttlocker
Copy link
Owner

Implements #292.

What shipped

cortex import

  • Added periodic stderr progress lines during file import:
    • import progress: <current>/<total> files (<filename>)
  • Added periodic stderr extraction progress during --extract:
    • extraction progress: <done>/<total> memories, <facts> facts
  • Added final stderr timing summary:
    • files scanned/imported/skipped
    • total facts extracted
    • total duration

cortex connect sync

  • Added per-provider progress callback plumbing in internal/connect/sync.go.
  • CLI now emits periodic stderr progress lines per provider:
    • sync progress [provider]: <current>/<total> records (imported=..., skipped=...)
  • Added final stderr timing summary for:
    • --all (aggregate counts + duration)
    • --provider <name> (provider counts + duration)

Design goals met

  • ✅ Progress output goes to stderr (stdout remains clean for piping)
  • ✅ No spinner requirement; newline progress is used
  • ✅ Includes counters requested for import + connect sync
  • ✅ Includes final timing summary

Validation

  • go test ./internal/connect ./cmd/cortex -run 'Progress|Import|Connect' -v
  • go test ./...

@hurttlocker hurttlocker merged commit abc4fdc into main Mar 1, 2026
4 checks passed
@hurttlocker hurttlocker deleted the xmate/progress-292 branch March 1, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants