Release 0.22.2 — agent diagnostics: interceptor diagnose + daemon lock file#128
Merged
Conversation
…k file interceptor diagnose (community contribution — thanks @trillium, #105): a single post-failure snapshot for agent self-orientation. Reports the daemon with the actual binary path it's running, probes every connected browser context in parallel (extension reachability, active tab, interactive element count), and reads monitor session state — all without ever auto-spawning a daemon. --context <id> scopes to one context; --json for structured output. Binary-mismatch detection: the daemon now writes a lock file (/tmp/interceptor.lock, INTERCEPTOR_LOCK_PATH to override) recording its pid/version/execPath. diagnose compares that execPath against every installed NMH manifest and flags the split-brain failure mode where Chrome spawns one daemon binary (the manifest's) while the CLI talks to another (the socket's) — previously a silent 15-second timeout, now a labeled warning with the fix. Detection covers the full installable browser set (Chrome, Brave, Beta, Canary, Dev, Chrome for Testing, Edge, Vivaldi) via one shared manifest table also used by status --verbose. Context-kind-aware probing: ios:/cdp: contexts report as connected devices/ apps instead of a false "extension not responding" from browser-verb probes. Daemon lifecycle hardening: lock-file cleanup rides gracefulShutdown and the exit handler (never a competing signal handler, which would skip CDP/iOS manager teardown and orphan tunnel/runner child processes); SIGHUP now takes the graceful path alongside SIGTERM/SIGINT; the lock is written only after winning the WS-port singleton election so a losing duplicate can never clobber the running daemon's record. diagnose joins help, help --all, and the interceptor manifest capability catalog with full returns semantics. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe version field was incremented from 0.22.1 to 0.22.2 across three metadata files: extension/dist-mv2/manifest.json, extension/manifest.json, and package.json. No other changes were made. ChangesVersion bump
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version bump + release for the diagnose feature set merged in #105 (community contribution by @trillium, hardened with maintainer follow-ups). See the release commit message for the full change summary.
🤖 Generated with Claude Code
Summary by CodeRabbit