Skip to content

fix(linux): re-enable AppImage sandbox workaround (SP-2052)#892

Merged
isasmendiagus merged 1 commit intomainfrom
fix/SP-2052/appimage-sandbox-permission
Mar 10, 2026
Merged

fix(linux): re-enable AppImage sandbox workaround (SP-2052)#892
isasmendiagus merged 1 commit intomainfrom
fix/SP-2052/appimage-sandbox-permission

Conversation

@isasmendiagus
Copy link
Copy Markdown
Collaborator

@isasmendiagus isasmendiagus commented Feb 2, 2026

Summary

  • Re-enables the afterPack hook in package.json that was accidentally removed during the electron-react-boilerplate upgrade (commit de1e8bb).
  • This activates the existing appimage-fix.js script, which creates a wrapper that launches the app with --no-sandbox on Linux, fixing the AppImage crash on Ubuntu 24.04+.

Reference

Summary by CodeRabbit

  • Bug Fixes
    • Fixed AppImage sandbox permission crash on Linux (Ubuntu 24.04+); packaging now includes a post-packaging step to ensure AppImage launches correctly.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 57e3ece1-2f1f-40f6-89a9-c7afbd282167

📥 Commits

Reviewing files that changed from the base of the PR and between 2c936c5 and c0b55b7.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • package.json
🚧 Files skipped from review as they are similar to previous changes (2)
  • package.json
  • CHANGELOG.md

📝 Walkthrough

Walkthrough

Added a changelog entry for an AppImage sandbox permission crash fix on Linux (Ubuntu 24.04+) and wired an afterPack build hook in package.json pointing to ./appimage-fix.js for post-packaging execution.

Changes

Cohort / File(s) Summary
Changelog Documentation
CHANGELOG.md
Added an Unreleased → Fixed entry describing the AppImage sandbox permission crash fix for Linux (Ubuntu 24.04+).
Build Configuration
package.json
Added build.afterPack: "./appimage-fix.js" to run a post-packaging script to address AppImage sandbox permissions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I hopped through build and changelog lines,
AfterPack set, fixing sandbox signs,
Ubuntu's patch snug in place,
AppImage sleeps with a gentle grace,
📦✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(linux): re-enable AppImage sandbox workaround (SP-2052)' accurately reflects the main change: re-enabling an AppImage sandbox fix for Linux, directly matching the CHANGELOG entry and package.json modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/SP-2052/appimage-sandbox-permission

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.

agustingroh
agustingroh previously approved these changes Mar 10, 2026
Copy link
Copy Markdown
Collaborator

@agustingroh agustingroh left a comment

Choose a reason for hiding this comment

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

LGTM

@agustingroh agustingroh self-requested a review March 10, 2026 12:38
Copy link
Copy Markdown
Collaborator

@agustingroh agustingroh left a comment

Choose a reason for hiding this comment

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

Please solve the conflicts before merge it

@isasmendiagus isasmendiagus force-pushed the fix/SP-2052/appimage-sandbox-permission branch from 2c936c5 to c0b55b7 Compare March 10, 2026 15:58
@isasmendiagus isasmendiagus merged commit 70bc691 into main Mar 10, 2026
9 checks passed
@isasmendiagus isasmendiagus deleted the fix/SP-2052/appimage-sandbox-permission branch March 10, 2026 16:14
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.

2 participants