Add observe-only monitor milestone for issue #4 - #93
Merged
Merged
Conversation
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Contributor
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
hyperpolymath
approved these changes
Sep 28, 2026
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
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.
Scope
Adds a bounded observe-only monitor milestone and safe development entrypoints. The collector uses bounded proc/sys reads, monotonic host sampling, explicit invalid/stale observations, stable device identity selection, single-instance ownership, and fixture-tested per-session PID/start-time, RSS, and transcript-motion primitives. A new foreground JSONL CLI exposes the host observer only; it does not auto-start, enumerate sessions, or enforce policy.
The follow-up adds:
scripts/run.sh/scripts/dev.shentrypoints,just monitor, and Debug/ReleaseSafe CLI build/help/version coverage.just check,just lint,just test, andjust qualityrecipes.just cleanpreserves trackedbuild/sources.No hooks, signals, process writes, LMDB integration, or memory balloon were added. Existing licensing declarations and issue #2 are unchanged.
Verification
just check,just lint, andjust qualitypass locally. Quality reports the existing missing SPDX header insrc/interface/ffi/src/main.zigas an owner-only warning; it was not altered.just doctor,just --list, shell syntax checks,zig fmt --check, and ShellCheck on changed shell scripts pass.just build,just install, andjust e2efail explicitly rather than claim success;just cleanpreservesbuild/.invalid:missingfor the initial baseline and a valididle_tellobservation on the next sample. Lock exclusion and rejection of symlink/permissive lock files and a symlink runtime directory were exercised. This is one host, not topology acceptance.Not a closure claim
Issue #4 is not ready to close. Paired accuracy across supported hosts/storage topologies, managed session discovery/startup identity capture, versioned schema/retention/IPC, actual OFF propagation through an integration, resource-pressure/failure acceptance, and host/VM durability evidence remain open. No balloon or global hook was installed.
scripts/run.shis a developer dispatcher, not a conformant desktop/service launcher: there is no application runtime, so service start and integration modes fail. Full launcher-standard conformance is deferred until a real application exists. Issue #2 remains owner-only/open.Closes no issue; references #4 for incremental review.