Skip to content

[codex] Reduce idle activity polling work#3960

Merged
louis030195 merged 1 commit into
mainfrom
codex/reduce-idle-activity-polling
Jun 10, 2026
Merged

[codex] Reduce idle activity polling work#3960
louis030195 merged 1 commit into
mainfrom
codex/reduce-idle-activity-polling

Conversation

@louis030195

Copy link
Copy Markdown
Collaborator

Summary

  • remove the unused last_idle_ms bookkeeping from the event-driven capture loop
  • stop calling ActivityFeed::idle_ms() during each timer-only poll when the result is never read
  • refresh a stale comment about repeated idle capture attempts

Why

The event-driven capture loop wakes periodically even when idle so it can service timer-driven captures and visual checks. The old poll_activity path still performed an activity-feed wall-clock read on every wake, but the stored value was dead state after typing/scroll pause triggers moved to the UI recorder. Dropping that read shaves a small amount of steady idle work without changing capture timing or event-triggered behavior.

Validation

  • cargo fmt --package screenpipe-engine -- --check
  • cargo test -p screenpipe-engine event_driven_capture --lib
  • cargo check -p screenpipe-engine

@louis030195 louis030195 force-pushed the codex/reduce-idle-activity-polling branch from fa3314c to cc40fbd Compare June 10, 2026 02:31
@louis030195 louis030195 marked this pull request as ready for review June 10, 2026 03:28
@louis030195 louis030195 merged commit 4e50ef4 into main Jun 10, 2026
23 checks passed
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.

1 participant