From d871843fe77ca31c390f063c74324f14eae5fac4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Pierzcha=C5=82a?= Date: Wed, 1 Jul 2026 07:26:39 +0200 Subject: [PATCH] docs: clarify request-count gate vs. restored cost.runnerRoundTrips in Phase 3 plan The Step (c) request-count bullet claimed both the dev-only CI gate AND the runtime `cost.runnerRoundTrips` surface were removed in #968. #970 restored the public agent-cost field, so the bullet is stale/misleading for the next Apple/agent-cost worker. Split the bullet: the dev-only request-count CI gate (the #966 --debug ndjson counter + smoke-ios assertion) stays removed (zero runner events on main runs); the runtime `cost.runnerRoundTrips` agent-cost field (ResponseCost / buildResponseCost over RUNNER_ROUND_TRIP_PHASES) is a separate pre-existing surface, restored in #970, and remains part of the agent-cost contract. --- plans/phase3-platform-plugin-progress.md | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/plans/phase3-platform-plugin-progress.md b/plans/phase3-platform-plugin-progress.md index ce55f6b18..c84ddd51f 100644 --- a/plans/phase3-platform-plugin-progress.md +++ b/plans/phase3-platform-plugin-progress.md @@ -122,9 +122,14 @@ This step retired the stale standalone Apple plan by landing the low-risk reloca 3. **visionOS groundwork** — the runner profile, SDK/platform metadata, Xcode supported-platform list, build script case, discovery tagging, and Swift interaction guard now recognize visionOS. Live spatial-input QA is still future work. -4. **request-count gate removal** — the runner request-count CI gate and `cost.runnerRoundTrips` runtime surface - were removed because successful `main` runs captured zero runner events, so the signal did not prove runner - behavior. Apple runner regressions are now guarded by the normal unit/build gates plus live smoke replay. +4. **request-count CI gate removal (dev-only)** — the runner request-count **CI gate** (the daemon `--debug` + ndjson counter + `smoke-ios` assertion added in #966) was removed because successful `main` runs captured + zero runner events, so the signal did not prove runner behavior. Apple runner regressions are now guarded by + the normal unit/build gates plus live smoke replay. **Distinct from that gate:** the public runtime + `cost.runnerRoundTrips` agent-cost field (`ResponseCost` in `src/kernel/contracts.ts`; computed by + `buildResponseCost` over `RUNNER_ROUND_TRIP_PHASES` in `src/daemon/request-router.ts`) is a separate, + pre-existing surface — briefly dropped alongside the gate but **restored in #970** — and remains part of the + agent-cost contract. Only the dev-only CI gate stays removed. ## Step (d) — remaining Apple leaf/plugin work ⛔ DO NOT AUTO-MERGE