Skip to content

Document Linux/Wayland AppImage launch-crash workaround in README#2465

Merged
senamakel merged 2 commits into
tinyhumansai:mainfrom
distorx:docs/readme-linux-launch-workaround
May 23, 2026
Merged

Document Linux/Wayland AppImage launch-crash workaround in README#2465
senamakel merged 2 commits into
tinyhumansai:mainfrom
distorx:docs/readme-linux-launch-workaround

Conversation

@distorx
Copy link
Copy Markdown
Contributor

@distorx distorx commented May 21, 2026

Summary

Problem

Solution

Submission Checklist

If a section does not apply to this change, mark the item as N/A with a one-line reason. Do not delete items.

  • Tests added or updated — N/A: documentation-only change, no code paths.
  • Diff coverage ≥ 80% — N/A: no code lines changed; diff-cover does not score Markdown.
  • Coverage matrix updated — N/A: no feature added/removed/renamed.
  • All affected feature IDs from the matrix listed in ## Related — N/A: no feature rows affected.
  • No new external network dependencies introduced — N/A: documentation-only change.
  • Manual smoke checklist updated — N/A: does not touch release-cut surfaces.
  • Linked issue closed via Closes #NNN — N/A: this PR documents a workaround; it does not fix CEF prewarm webview triggers a fatal X_ConfigureWindow BadWindow crash on Wayland #2463, so no closing keyword is used.

Impact

  • Documentation only. No runtime/platform/performance/security/migration impact.

Related


AI Authored PR Metadata (required for Codex/Linear PRs)

Linear Issue

  • Key: N/A — no Linear issue.
  • URL: N/A

Commit & Branch

  • Branch: docs/readme-linux-launch-workaround
  • Commit SHA: 54bda893aaa669720e32b736d4a7189cc91caafc

Validation Run

  • pnpm --filter openhuman-app format:check — N/A: documentation-only Markdown change; no JS/TS touched.
  • pnpm typecheck — N/A: no code changed.
  • Focused tests: N/A — no code changed.
  • Rust fmt/check (if changed): N/A — no Rust changed.
  • Tauri fmt/check (if changed): N/A — no Rust changed.

Validation Blocked

  • command: N/A
  • error: N/A
  • impact: N/A

Behavior Changes

  • Intended behavior change: none — documentation only.
  • User-visible effect: a Linux caveat now appears in the README install section.

Parity Contract

  • Legacy behavior preserved: N/A — documentation only.
  • Guard/fallback/dispatch parity checks: N/A.

Duplicate / Superseded PR Handling

  • Duplicate PR(s): none.
  • Canonical PR: this one.
  • Resolution: N/A.

Summary by CodeRabbit

  • Documentation
    • Updated documentation with Linux-specific guidance: AppImage may crash on launch under Wayland and on some Arch-based distributions (e.g., "Interpreter not found" errors). Added environment-variable workarounds and links to troubleshooting resources to help users diagnose and mitigate these launch issues.

Review Change Stack

Add a note to the install section linking tinyhumansai#2463, which covers the
CEF-prewarm X_ConfigureWindow crash on Wayland and the sharun
"Interpreter not found!" loader failure on Arch-based distros, both
with env-var workarounds.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@distorx distorx requested a review from a team May 21, 2026 19:57
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7e3328e6-1985-458c-9b26-5f3a4f2fdc91

📥 Commits

Reviewing files that changed from the base of the PR and between 54bda89 and 8793f2b.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README adds a Linux troubleshooting note that the AppImage may crash on launch under Wayland (and on some Arch-based distros with sharun: Interpreter not found!), linking to GitHub issue #2463 for environment-variable workarounds.

Changes

Linux AppImage Compatibility Note

Layer / File(s) Summary
Wayland and Arch-based distro compatibility note
README.md
Adds a troubleshooting line to README noting AppImage launch crashes on Wayland and the sharun: Interpreter not found! error on Arch-based systems, referencing issue #2463 for details and workarounds.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • graycyrus

Poem

🐰 A note for Linux friends so keen,
When Wayland graphics grace the screen,
AppImage may stumble, crash, or sigh—
See issue #2463 and give workarounds a try! 🌿

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: documenting a workaround for Linux/Wayland AppImage launch crashes in the README.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 71: Update the README blockquote for consistency by capitalizing the
first word after the colon: change "Linux: the AppImage can crash on launch..."
to "Linux: The AppImage can crash on launch..."; locate the string in the README
line that begins "**Linux:** the AppImage..." and make the one-word
capitalization change to match the existing blockquote pattern (e.g., like
"**Early Beta**: Under active...").
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1d2cff01-139c-479b-b7d4-0c4c5909902d

📥 Commits

Reviewing files that changed from the base of the PR and between c9ab4b9 and 54bda89.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
irm https://raw.githubusercontent.com/tinyhumansai/openhuman/main/scripts/install.ps1 | iex
```

> **Linux:** the AppImage can crash on launch under Wayland (and on Arch-based distros with `sharun: Interpreter not found!`) — see [#2463](https://github.com/tinyhumansai/openhuman/issues/2463) for the cause and env-var workarounds.
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.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Capitalize "The" for consistency with the established blockquote pattern.

Line 57 capitalizes the first word after the colon (**Early Beta**: Under active...). For consistency, "the AppImage" should be "The AppImage".

📝 Proposed fix
-> **Linux:** the AppImage can crash on launch under Wayland (and on Arch-based distros with `sharun: Interpreter not found!`) — see [`#2463`](https://github.com/tinyhumansai/openhuman/issues/2463) for the cause and env-var workarounds.
+> **Linux:** The AppImage can crash on launch under Wayland (and on Arch-based distros with `sharun: Interpreter not found!`) — see [`#2463`](https://github.com/tinyhumansai/openhuman/issues/2463) for the cause and env-var workarounds.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
> **Linux:** the AppImage can crash on launch under Wayland (and on Arch-based distros with `sharun: Interpreter not found!`) — see [#2463](https://github.com/tinyhumansai/openhuman/issues/2463) for the cause and env-var workarounds.
> **Linux:** The AppImage can crash on launch under Wayland (and on Arch-based distros with `sharun: Interpreter not found!`) — see [`#2463`](https://github.com/tinyhumansai/openhuman/issues/2463) for the cause and env-var workarounds.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 71, Update the README blockquote for consistency by
capitalizing the first word after the colon: change "Linux: the AppImage can
crash on launch..." to "Linux: The AppImage can crash on launch..."; locate the
string in the README line that begins "**Linux:** the AppImage..." and make the
one-word capitalization change to match the existing blockquote pattern (e.g.,
like "**Early Beta**: Under active...").

@senamakel senamakel merged commit ebdd03e into tinyhumansai:main May 23, 2026
17 of 23 checks passed
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.

CEF prewarm webview triggers a fatal X_ConfigureWindow BadWindow crash on Wayland

2 participants