Skip to content

status-core: integrate accepted nodeprobe 863c four-axis state - #70

Merged
Florious95 merged 1 commit into
mainfrom
pr/status-core-nodeprobe-863c
Sep 7, 2026
Merged

status-core: integrate accepted nodeprobe 863c four-axis state#70
Florious95 merged 1 commit into
mainfrom
pr/status-core-nodeprobe-863c

Conversation

@Florious95

@Florious95 Florious95 commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Goal

Independent status-core integration of accepted nodeprobe 863c316ff66add549f8fa6dd215cedeedd037f8b (parent 76fdf428). Does not reuse corral-core #66 / corral-serve #4 (pr/status-core-exact-nodeprobe) and does not mix PR#69 UI/menu/upload.

Provenance

  • Product baseline: 4605951e427f9ba6627375498dcb3c757c05bf36
  • Head: 064bf78b84737e5fca941876f0598d4704d85b2f
  • Accepted source tree: ce667e3c464d0efb2ac9483862ff84a38008e674
  • Observer: tools/nodeprobe/pi/nodeprobe-pi-activity.js SHA-256 ee8b088ef78428e7e74b905d8e770d4271562f2778a083da489612713f43533c
  • Runtime binary coordinate: /Users/alauda/.local/bin/nodeprobe size 770432 SHA-256 8ef9c5fe89ae76f2c8eb6aeb16bf5fb9060a79bcf81aa4d790df3f4614c2fb5f
  • Final install acceptance: final-install-acceptance-luna-20260903.md (PASS)
  • Canonical corpora: only tools/nodeprobe/fixtures/{titles,providers}.tsv (not copied)
  • Default Pi channel dir: $HOME/.local/state/nodeprobe/pi-activity (0700/0600, NODEPROBE_PI_ACTIVITY_DIR override)

Tests

  • python3 tools/gate/nodeprobe-status-core.py passed
  • cargo test --offline in tools/nodeprobe: 41+1+1 passed, including pi_missing_channel_with_process_is_normal_health
  • go test -count=1 ./internal/nodeprobe ./internal/protocol ./internal/api ./cmd/agentmirrord ok
  • node tools/nodeprobe/tests/pi_extension.mjs: lifecycle protocol ok
  • APP :core-protocol:test ConnCodecTest and :app:testDebugUnitTest L2UnknownStatusTest --rerun-tasks BUILD SUCCESSFUL
  • Install-state hash of binary+.js matched the embedded manifest
  • Isolated installed-binary fail-closed: missing→health=normal; working→idle; stale/future/disconnect/PID/instance/json/schema→abnormal
  • nodeprobe web one refresh: schema 1, 123 nodes, 31 sockets, errors 0, wall 2.875s; task port cleaned

Risks

  • Global skill wording still describes missing/unset channel as health=unknown. Accepted 863c unit and install acceptance freeze health=normal for unique Pi + missing/unset. External documentation sync only; not a product blocker. .team/stable-pr/nodeprobe-accuracy/VERDICT.md is superseded local-contract evidence.
  • Production daemon still requires the accepted macOS arm64 installed binary/hash; this PR does not deploy or change the global install.
  • Not merged.

Rollback

Revert this one commit on pr/status-core-nodeprobe-863c.

Synchronize tools/nodeprobe from 863c316 (auto-discovered .js observer,
default pi-activity dir, web /api/status schema 1, unique-Pi missing
channel health=normal). Replace Go title/process classifiers with a
typed nodeprobe adapter, extend protocol/APP data models with four
axes, and leave list/icon/animation/favorite UI unchanged.
@Florious95 Florious95 changed the title pr/status-core-nodeprobe-863c status-core: integrate accepted nodeprobe 863c four-axis state Sep 2, 2026
@Florious95
Florious95 merged commit cf97d54 into main Sep 7, 2026
@Florious95
Florious95 deleted the pr/status-core-nodeprobe-863c branch September 8, 2026 00:40
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.

2 participants