Skip to content

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

Open
distorx wants to merge 1 commit into
tinyhumansai:mainfrom
distorx:docs/readme-linux-launch-workaround
Open

Document Linux/Wayland AppImage launch-crash workaround in README#2465
distorx wants to merge 1 commit into
tinyhumansai:mainfrom
distorx:docs/readme-linux-launch-workaround

Conversation

@distorx
Copy link
Copy Markdown

@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
    • Added Linux-specific guidance regarding AppImage compatibility with Wayland and certain Arch-based distributions, including information on workarounds and troubleshooting resources.

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

📝 Walkthrough

Walkthrough

The README is updated with a Linux-specific troubleshooting note warning users that the AppImage can crash on launch under Wayland and certain Arch-based distributions. The note references GitHub issue #2463 for the underlying cause and environment-variable workarounds.

Changes

Linux AppImage Compatibility Note

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

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A note for Linux friends so keen,
When Wayland graphics grace the screen,
AppImage crashes on the way?
Fear not! Issue #2463 saves the day! 🌿

🚥 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...").

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

1 participant