From 09196a44fec4b36337310d0a872b51fa65a0babd Mon Sep 17 00:00:00 2001 From: iris Date: Tue, 8 Sep 2026 12:48:05 +0200 Subject: [PATCH] docs: name hive-forge-notify instead of a dangling pronoun argus's review on this PR flagged a real clarity regression: "Forge notifications are polled by their own process, not this loop" became "Their own process polls forge notifications, not this loop" -- the original's "their" clearly referred back to "Forge notifications" within the same clause; the rewrite put "Their" first with nothing plural preceding it in the paragraph, so the pronoun has no referent when the reader hits it. This was exactly a case where the passive was doing real work (subject-first noun introduction), not just avoiding an active verb. Uses argus's own suggested fix: name the process directly instead of leaning on a pronoun at all, and drop the now-redundant second mention of the same name a few words later. --- docs/turn-loop/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/turn-loop/README.md b/docs/turn-loop/README.md index 0cdf347b..c37fca1e 100644 --- a/docs/turn-loop/README.md +++ b/docs/turn-loop/README.md @@ -96,8 +96,8 @@ opens turn-stats sqlite, prepares the on-boot files (see [claude-invocation](claude-invocation.md#on-boot-files)), installs claude plugins, spawns `web_ui::serve` + `vacuum::run`, and either drops into `serve_loop` directly (`Online`) or parks on -the login flow first (`NeedsLogin`). Their own process polls forge -notifications, not this loop — see `hive-forge-notify` in +the login flow first (`NeedsLogin`). `hive-forge-notify` polls forge +notifications on its own, not this loop — see [`forge.md`](../integrations/forge.md). Boot also opens the todos store and the socket in-container producers