Skip to content

fix: unref advisory monitor handles#19

Merged
dorukardahan merged 1 commit intomainfrom
codex/fix-hooks-check-open-handles
Apr 22, 2026
Merged

fix: unref advisory monitor handles#19
dorukardahan merged 1 commit intomainfrom
codex/fix-hooks-check-open-handles

Conversation

@dorukardahan
Copy link
Copy Markdown
Owner

Summary

  • Unrefs ZeroAPI advisory file watchers and debounce timers.
  • Adds a regression test that verifies watcher and timer handles are unrefed.
  • Prepares patch release metadata for v3.7.2.

Why

  • OpenClaw CLI diagnostics such as openclaw hooks check --json load hook-only plugins to inspect them. ZeroAPI v3.7.1 started advisory watchers during plugin registration, which could keep short-lived CLI diagnostics alive after JSON output was produced.

Tests

  • npm test
  • JSON parse check
  • old version grep, with only historical changelog headings remaining

Non-goals

  • No routing policy changes.
  • No OpenClaw runtime patch changes in this PR.

@dorukardahan
Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@dorukardahan dorukardahan merged commit 8f82812 into main Apr 22, 2026
3 checks passed
@dorukardahan dorukardahan deleted the codex/fix-hooks-check-open-handles branch April 22, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant