Skip to content

fix(planning): widen the interview watcher debounce so accepts seconds apart share a wake #5457

Description

@kyle-sexton

No related issue: split out of #5191 after a repro pass on planning 0.46.0 (origin/main at 4d1d9d8).

Summary

Finding 13 of #5191, reported against planning 0.42.0: Q7, Q8, Q9 accepted at 17:57:12, :23 and :30 arrived as three separate wakes.

Fix

Owner picks the window; then change the constants.

Verification

Repro pass on origin/main, playwright-cli driving Chromium against a copy of plugins/planning/surface/, plus round.sh, watch.sh and curl. Verdict: reproduces.

A debounce exists but only batches events under QUIET_SECONDS = 0.3 s apart; BURST_SECONDS = 2.0 s only caps the total hold (settle(), plugins/planning/surface/server.py:68-69,898-910). Live: two accepts 1.5 s apart, the watcher exited on the first (seq 28) and the second needed a second wake. The reported gaps were 7 and 11 s. Widening the window delays every single accept, so the value is an owner tuning call.

Related

Refs #5191. #4653 tracks the commitment and export gaps behind findings 9, 17 and part of 18.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority: mediumReal value, no hard deadline; normal backlog flow.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions