-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Docs Bug]: Correct hook timeout semantics and document a safe cron host adapter #104083
Copy link
Copy link
Closed
Closed
Copy link
Labels
P3Low-priority cleanup, docs, polish, ergonomics, or speculative work.Low-priority cleanup, docs, polish, ergonomics, or speculative work.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.docsImprovements or additions to documentationImprovements or additions to documentationissue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PRmaturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.
Description
Metadata
Metadata
Assignees
Labels
P3Low-priority cleanup, docs, polish, ergonomics, or speculative work.Low-priority cleanup, docs, polish, ergonomics, or speculative work.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.docsImprovements or additions to documentationImprovements or additions to documentationissue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PRmaturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.
Type
Fields
Priority
None yet
Documentation issue
The plugin hook documentation currently describes
timeoutMsas aborting a hook handler. The implementation only stops awaiting the handler; the underlying promise and side effects continue.That distinction is important for host adapters: once the hook timeout expires, Gateway suspension accounting can stop waiting even though an external scheduler write is still in progress.
The cron hook documentation also recommends an initial full-list reconciliation but does not provide a complete safe adapter pattern for coalescing overlapping changes, waiting for durable external acceptance, retrying, or cleanup.
Location
docs/plugins/hooks.mdExpected documentation
timeoutMsbounds how long OpenClaw awaits a hook; it does not cancel the handler.cron_changedevents;gateway_stop.Additional context
This should land after the typed cron reconciliation lifecycle contract so the example documents a real public API rather than a proposed one.