Skip to content

fix(openclaw): remove machine-specific default paths - #344

Open
WODE25500 wants to merge 1 commit into
microsoft:mainfrom
WODE25500:fix/openclaw-portability
Open

WODE25500 wants to merge 1 commit into
microsoft:mainfrom
WODE25500:fix/openclaw-portability

Conversation

@WODE25500

@WODE25500 WODE25500 commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This refreshes #252 on current main. The original patch only changed three
scripts; current main still contains machine-specific OpenClaw paths in
the runner, slash helper, cron wrapper, and sample config.

Changes

  • Derive the SkillOpt checkout, plugin, task, and default config paths from the script location.
  • Prefer an installed OpenClaw config while retaining the bundled sample as a fallback.
  • Resolve blank agent/workspace settings through OPENCLAW_HOME, OPENCLAW_STATE_DIR, and OPENCLAW_PROFILE.
  • Match OpenClaw path semantics by preserving symlink paths and honoring HOME before platform defaults.
  • Match OpenClaw named-profile semantics, including the default sentinel and invalid profile names.
  • Make the cron wrapper portable and use the relocated OpenClaw state .env.
  • Keep the OpenClaw directory clearly labeled as a reference, not a ready-to-run integration.
  • Add isolated portability and subprocess regressions.

Verification

  • python -m pytest tests/test_openclaw_portability.py tests/test_plugin_sync.py -q: 23 passed, 1 symlink test skipped on Windows.
  • Ruff passes for the new path resolver and regression file.
  • bash -n plugins/openclaw/run_sleep_cron.sh passes.
  • No ethanclaw paths remain under plugins/openclaw.
  • The current upstream Windows suite has 22 pre-existing failures; the same selected failures reproduce on the unmodified base commit.

@WODE25500
WODE25500 force-pushed the fix/openclaw-portability branch 3 times, most recently from 6306997 to ad850b9 Compare October 30, 2026 10:53
Resolve plugin, repository, task, and config paths from the script location.
Map blank agent/workspace settings through OPENCLAW_HOME, OPENCLAW_STATE_DIR,
and OPENCLAW_PROFILE, and cover the defaults with portability regressions.
@WODE25500
WODE25500 force-pushed the fix/openclaw-portability branch from ad850b9 to cec8f41 Compare October 30, 2026 10:55
lucatreiber2007 added a commit to lucatreiber2007/SkillOpt that referenced this pull request Nov 1, 2026
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
lucatreiber2007 added a commit to lucatreiber2007/SkillOpt that referenced this pull request Nov 1, 2026
Tests schlagen auf macOS fehl (Kimi-Pfad/Slug bzw. bash-3.2-Syntax).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant