fix(harness): auto-detect ROOSYNC_SHARED_PATH in listener install (#2431)#2432
Merged
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ipt (#2431) The install script did not configure ROOSYNC_SHARED_PATH, causing the listener to crash immediately with 'SharedPath not set' on every restart. Now auto-detects from common GDrive paths and sets the user-level env var. Warns if no path found, with manual instructions. Root cause: po-2026 listener dead since May 17 — 13 days of missed WAKEs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jsboige
requested review from
MyIA-Web1,
myia-ai-01 and
myia-po-2023
as code owners
7 tasks
clusterManager-Myia
approved these changes
May 30, 2026
Collaborator
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[Hermes] — APPROVED
SHA reviewed: 9472477 (HEAD)
Security scan
✅ CLEAN — no hardcoded credentials.
CI status
✅ All green (build-and-test Node 20+22, check-submodule-pointer, sync).
Review
- Submodule bump (win-cli/server): CI check-submodule-pointer validates the pointer is intentional.
- Auto-detect ROOSYNC_SHARED_PATH: 3 hardcoded candidate paths (French GDrive layout) + fallback warning with manual instructions. Appropriate defensive improvement for the install script — prevents silent listener failure when the env var is missing.
- Pattern consistent with existing pwsh guard (exit on missing PowerShell).
APPROVE — low-risk install script improvement + submodule bump, CI green, security clean.
Drops the accidental win-cli downgrade (1781def6 -> f9652c5a) introduced by auto-commit 894e64f ("Auto-commit uncommitted worker changes"), which reverted the output-truncation fix landed in b2bd371 (#2425). The substantive fix in this PR (install-dashboard-listener-schtask.ps1, commit 9472477) is unaffected. Net submodule diff vs main is now zero. Refs #2431 #2425
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
3 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.
Problem
Issue #2431 — Dashboard listener dead on executant machines (po-2026 confirmed, likely others). The listener was installed but crashed immediately with
SharedPath not set. Set ROOSYNC_SHARED_PATH.because the install script never configured this environment variable.Root Cause
install-dashboard-listener-schtask.ps1creates the scheduled task without ensuringROOSYNC_SHARED_PATHis set. On po-2026, the listener was dead since May 17 (13 days of missed[WAKE-CLAUDE]signals).Fix
Add auto-detection of
ROOSYNC_SHARED_PATHin the install script:Testing
Applied manually on po-2026 — listener Running, heartbeat fresh, correctly spawning agents on WAKE-CLAUDE.
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com