From fd633f674e1f39180d1771da9e5d4ce084f9ca02 Mon Sep 17 00:00:00 2001 From: iris Date: Tue, 8 Sep 2026 13:19:29 +0200 Subject: [PATCH] docs: drop the comma outside the closing quote in coordinator.md argus's review on this PR caught a real regression: the earlier passive-voice rewrite ('answers "did the agent come back up?", the same way...') put a comma immediately outside a closing double-quote, which vale's Microsoft.Quotes rule flags ("punctuation should be inside the quotes") -- a brand new error not present on main. The original phrasing never had a quote directly followed by a comma, so this was a real side effect of the rewrite, not a pre-existing hit. Simplest fix: drop the comma. The sentence reads fine without it and the closing "?" already does the job. Verified via vale: back to 2 errors (matching main's baseline, down from the 3 this rewrite introduced), write-good.Passive count unchanged at 24. --- docs/scheduler/coordinator.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/scheduler/coordinator.md b/docs/scheduler/coordinator.md index 7cdd44d8..5d59515a 100644 --- a/docs/scheduler/coordinator.md +++ b/docs/scheduler/coordinator.md @@ -450,7 +450,7 @@ own. Its `DeployApply` node doesn't build: it merges, opens the two-phase meta deploy, and returns the chain above as a subgraph the scheduler grafts into the live DAG under that node. A `FinalizeDeploy` node gated on the graft's completion then plants the deploy tag — so `Reconcile`'s success -answers "did the agent come back up?", the same way it does for every +answers "did the agent come back up?" the same way it does for every other rebuild, instead of a fused inline start. The grafted nodes land _inside_ `DeployWindow`'s subtree, so they re-enter