fix: flush pending responses before runtime maintenance

[hheei]

Summary

Flush ready deferred responses before runtime maintenance and again afterward. This prevents a terminal foreground Bash task from waiting behind a synchronous runtime drain before its NDJSON response can be emitted.

The second flush preserves delivery for responses that become ready during maintenance. Runtime maintenance and task execution behavior are otherwise unchanged.

Fixes #172.

Regression coverage

• Adds a pending-response ordering regression: a ready deferred response must be polled and written before configure-warning maintenance runs.
• Updates the periodic-drain polling expectation for the new before/after flush behavior.

Validation

• cargo +nightly-2026-04-29 test -p agent-file-tools pending_response_tests -- --nocapture
• cargo +nightly-2026-04-29 test -p agent-file-tools decide_bash_step -- --nocapture
• cargo +nightly-2026-04-29 build -p agent-file-tools --release --bin aft
• Direct @cortexkit/aft-bridge smoke against the built binary: configure, Bash launch, terminal status, and output retrieval succeeded.

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

---

Summary by cubic

Flush ready pending responses before and after runtime maintenance so NDJSON from foreground Bash tasks is not blocked by the drain. No change to maintenance or task behavior.

• Bug Fixes
  • Flush pending before drain_runtime_events, then flush again after.
  • Removes latency for terminal Bash NDJSON and preserves delivery for responses that become ready during maintenance.
  • Adds an ordering regression test and updates polling expectations. Fixes Deferred bash responses can be starved by runtime maintenance after task completion #172.

^{Written for commit f957d38. Summary will update on new commits.}

Greptile Summary

Flushes ready deferred responses before runtime maintenance and polls again afterward, with regression coverage for response ordering and the updated polling cadence.

Confidence Score: 5/5

The PR appears safe to merge, with the new ordering preserving both prompt delivery of already-ready responses and delivery of responses made ready during maintenance.

The runtime registry retains one stable application context, and current pending-response polling remains compatible with the intentional before-and-after maintenance cadence; no concrete changed-code failure remains.

Important Files Changed

Filename	Overview
crates/aft/src/main.rs	Reorders pending-response flushing around runtime maintenance and adds focused tests for pre-maintenance delivery and twice-per-tick polling.

_{Reviews (1): Last reviewed commit: "fix: flush pending responses before runt..." | Re-trigger Greptile}

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}