Skip to content

fix: openclaw runtime error#7709

Merged
louis-jan merged 7 commits into
release/v0.7.8from
fix/openclaw-runtime-error
Mar 11, 2026
Merged

fix: openclaw runtime error#7709
louis-jan merged 7 commits into
release/v0.7.8from
fix/openclaw-runtime-error

Conversation

@louis-jan

Copy link
Copy Markdown
Contributor

Describe Your Changes

  • Attempt to fix missing runtime issue on Mac

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Copilot AI review requested due to automatic review settings March 11, 2026 09:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-jan louis-jan changed the title Fix/openclaw runtime error fix: openclaw runtime error Mar 11, 2026
Copilot AI review requested due to automatic review settings March 11, 2026 09:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src-tauri/src/core/openclaw/sandbox_direct.rs
@github-actions

github-actions Bot commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Barecheck - Code coverage report

Total: 21.79%

Your code coverage diff: 0.00% ▴

✅ All code changes are covered

Copilot AI review requested due to automatic review settings March 11, 2026 09:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (2)

src-tauri/src/core/openclaw/mod.rs:80

  • This directory rename/migration logic is user-data dependent and easy to regress across platforms. Consider adding a small unit test in src-tauri/src/core/openclaw/tests.rs that validates get_openclaw_runtime_dir() chooses the expected directory (and any fallback/migration behavior you implement), so the “missing runtime” fix is covered going forward.
/// OpenClaw runtime directory (where Bun and OpenClaw are installed)
pub fn get_openclaw_runtime_dir() -> Result<std::path::PathBuf, String> {
    let base = get_openclaw_base_dir()?;
    let runtime_dir = base.join("bunx");
    if !runtime_dir.exists() {
        std::fs::create_dir_all(&runtime_dir).map_err(|e| e.to_string())?;
    }
    Ok(runtime_dir)

src-tauri/src/core/openclaw/mod.rs:80

  • get_openclaw_runtime_dir() now points at openclaw/bunx instead of the previous openclaw/runtime. If existing installs/data live under the old directory, they will no longer be discovered and may cause unnecessary reinstalls or “missing runtime” behavior. Consider a migration/fallback (e.g., if bunx is empty but runtime exists, use/move it) and update nearby docs/comments that still refer to “openclaw-runtime”.
/// OpenClaw runtime directory (where Bun and OpenClaw are installed)
pub fn get_openclaw_runtime_dir() -> Result<std::path::PathBuf, String> {
    let base = get_openclaw_base_dir()?;
    let runtime_dir = base.join("bunx");
    if !runtime_dir.exists() {
        std::fs::create_dir_all(&runtime_dir).map_err(|e| e.to_string())?;
    }
    Ok(runtime_dir)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src-tauri/src/core/openclaw/sandbox_direct.rs
Comment thread src-tauri/src/core/openclaw/sandbox_direct.rs
Comment thread src-tauri/src/core/openclaw/commands.rs
@louis-jan louis-jan merged commit 8c81f73 into release/v0.7.8 Mar 11, 2026
13 checks passed
@louis-jan louis-jan deleted the fix/openclaw-runtime-error branch March 11, 2026 10:11
@github-project-automation github-project-automation Bot moved this to QA in Jan Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

3 participants