Conversation
Production Linux now injects the trusted procfs boot_id into both the journal clock and SystemBootClock, using CLOCK_BOOTTIME so deadlines include suspend. Explicit notify uses a thin session D-Bus adapter for navigation=none and never falls back to beeep after a possible effect. setup.Apply stays macOS-only; this does not start the click-to-focus daemon or claim native Linux E2E. Co-authored-by: Илия <iliyazelenkog@gmail.com>
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks 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 |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## feat/agent-notify-e2e #186 +/- ##
=========================================================
+ Coverage 65.27% 65.30% +0.03%
=========================================================
Files 168 170 +2
Lines 17182 17333 +151
=========================================================
+ Hits 11215 11319 +104
- Misses 4360 4388 +28
- Partials 1607 1626 +19 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Status no longer requires a macOS native DecoderFloor on Linux. A managed, non-recovery install is eligible for session D-Bus delivery. Also give policy-disable recovery a 30s budget so Windows CI does not starve several Commits on a shared 5s fixture. Co-authored-by: Илия <iliyazelenkog@gmail.com>
Linux Apply/Inspect no longer fail as unsupported_platform. Session delivery provisions the journal without a macOS native helper. Local .app routing stays Darwin-only and still requires DecoderFloor 1. Co-authored-by: Илия <iliyazelenkog@gmail.com>
The existing-installer enable path now provisions a journal without a native helper. A real dbus-daemon test covers NameHasOwner readiness and one Notify submit through the production session adapter. Co-authored-by: Илия <iliyazelenkog@gmail.com>
…D-Bus test Linux navigation-none setup now registers MCP, so the Codex e2e no longer expects an unsupported-OS warning. The session-bus fixture counts Notify calls with an atomic to match the D-Bus worker. Co-authored-by: Илия <iliyazelenkog@gmail.com>
cursor
Bot
changed the title
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
2 participants
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.
This is the R3/P7 Linux runtime slice on top of #177. It is independent of the wizard/SDK stack and of the Windows P7 PR.
What landed
/proc/sys/kernel/random/boot_idviajournal.DefaultClock().notifier.SystemBootClocksharesjournal.LinuxBootSample.FreedesktopDelivery: bounded session-bus connect,NameHasOwnerreadiness, oneNotifyfornavigation=none, no beeep after a possible D-Bus effect, no click-to-focus daemon.Statusis eligible without nativeDecoderFloor.setup.Apply/Inspectaccept Linux for navigation none and reject local.approuting.setup-notifications enable --navigation noneworks without native.-race).Honest non-goals
--agent-notifyon Linux is the child PR fix(notify): wire Linux MCP configure onto navigation-none setup #188.