fix(app): make linux CEF deb package runnable

[VectorJet]

Summary

• update Debian package dependencies for trixie so the generated .deb installs without hand-editing DEBIAN/control
• teach the packaged app to find openhuman-core from Linux install locations like /usr/bin/openhuman-core
• skip tray setup on linux + cef builds to avoid the packaged GTK panic during tray/menu creation
• add grep-friendly core binary resolution logs to make packaged startup failures easier to diagnose

Problem

• the Linux CEF .deb was not runnable after install on Debian 13 / trixie
• the package depended on obsolete Debian package names (libwebkit2gtk-4.0-37, libgdk-pixbuf2.0-0)
• the GUI runtime looked for openhuman-core next to the app binary/resources, but the .deb installs it in /usr/bin
• packaged startup also panicked when tray setup hit GTK too early on Linux CEF

Solution

• update tauri.conf.json Linux deb dependencies to trixie package names
• extend default_core_bin() to probe packaged Linux locations before falling back to sibling/resource lookup
• split setup_tray() by cfg and short-circuit tray creation on linux + cef with an explicit warning log
• keep the existing RunEvent::Ready tray deferral for other targets/runtime combinations

Submission Checklist

• Unit tests — Vitest (app/) and/or cargo test (core) for logic you add or change
• E2E / integration — Where behavior is user-visible or crosses UI → Tauri → sidecar → JSON-RPC; use existing harnesses (app/test/e2e, mock backend, tests/json_rpc_e2e.rs as appropriate)
• N/A — Not practical here: this change fixes Linux packaging/runtime integration and was validated with cargo check --manifest-path src-tauri/Cargo.toml --target aarch64-unknown-linux-gnu plus a local packaged install repro
• Doc comments — Existing comments were kept and expanded only where the runtime behavior was non-obvious
• Inline comments — Existing tray/core resolution comments remain in place; added logging for the non-obvious packaged lookup path

Impact

• impacts packaged Linux desktop builds using the CEF runtime
• improves Debian 13 / trixie compatibility for generated .deb artifacts
• avoids a Linux startup panic by disabling tray creation for the currently broken linux + cef path; tray behavior on other targets is unchanged
• no core JSON-RPC contract or frontend API changes

Related

• Closes:
• Follow-up PR(s)/TODOs:
  • investigate restoring Linux CEF tray support once GTK/tray initialization is safe in packaged runs
  • optionally teach the installer/package to preserve chrome-sandbox permissions automatically

Summary by CodeRabbit

• New Features

  • Added ARM Linux (aarch64) build support with complete setup and runtime instructions.

• Bug Fixes

  • Fixed GTK initialization issue affecting tray setup on Linux.
  • Enhanced core binary discovery with improved diagnostics and logging.

• Documentation

  • Updated build documentation with new package manager instructions and ARM Linux build guide.

• Chores

  • Migrated build tooling from Yarn to pnpm across all workflows and scripts.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The PR migrates the project from Yarn to pnpm across CI/CD workflows, npm scripts, and configuration files. It updates Tauri build settings, enhances core process binary resolution logging, refactors tray initialization to execute after app ready state, adds ARM Linux build documentation, removes redirect_links controller registration, and refactors E2E test infrastructure.

Changes

Cohort / File(s)	Summary
Package Manager Migration (Workflows) .github/workflows/build-windows.yml, build.yml, e2e-agent-review.yml, release.yml, test.yml, typecheck.yml	Consistent migration from Yarn to pnpm: added pnpm setup steps (v10.10.0 with caching), removed Yarn-specific cache configuration, switched installation commands to pnpm install --no-frozen-lockfile, updated workspace invocations from yarn workspace to pnpm --filter. Added NODE_OPTIONS env vars for increased heap size in some builds.
Package Manager Migration (Root & App Config) package.json, app/package.json, .husky/pre-push, app/src-tauri/tauri.conf.json	Switched all npm scripts from yarn to pnpm invocations across workspace commands, test runners, formatters, and linters. Added packageManager: "pnpm@10.10.0" pinning to root package.json. Updated Tauri build commands from npm to pnpm. Added pnpm-workspace.yaml configuration.
Core Rust Process & Initialization app/src-tauri/src/core_process.rs, app/src-tauri/src/lib.rs	Enhanced default_core_bin with structured diagnostics (info/warn logs on binary discovery, precedence, and resolution failures). Refactored tray initialization: moved from .setup() closure to RunEvent::Ready with platform-specific conditionals (no-op on Linux with CEF feature to avoid GTK panics). Added comprehensive test coverage for binary resolution, precedence, and tray behavior.
Build & Dependency Configuration packages/deb/control.in, app/src-tauri/tauri.conf.json, pnpm-workspace.yaml	Added runtime library dependencies to Debian package control (GTK 3, WebKit2GTK, X11, GDK-Pixbuf, GLib). Updated Tauri beforeDevCommand and beforeBuildCommand from npm to pnpm. Created pnpm workspace configuration.
E2E & Test Infrastructure app/test/wdio.conf.ts, tests/webview_apis_bridge.rs, app/test/e2e/helpers/artifacts.ts, app/test/e2e/specs/agent-review.spec.ts, app/scripts/e2e-build.sh, app/scripts/e2e-run-spec.sh	Migrated E2E execution from npx to pnpm exec (wdio, tauri). Refactored WebSocket mock server to spawn on dedicated Tokio runtime thread with port communication via mpsc channel. Updated WDIO config to use absolute projectRoot-based spec paths. Removed eslint-disable suppressions around console statements.
Frontend Components app/src/AppRoutes.tsx, app/src/pages/Conversations.tsx, app/src/components/settings/panels/AIPanel.tsx	Removed redundant /notifications route in AppRoutes. Changed message ID generation from Date.now() + Math.random() to crypto.randomUUID() in Conversations. Updated AIPanel useEffect dependencies from empty array to [loadAIPreview, loadLocalAiStatus] for reactive behavior.
Controller Registry & Scripting src/core/all.rs, src/openhuman/webhooks/bus.rs, scripts/review/README.md, scripts/review/cli.sh	Removed redirect_links controller and schema from central registry and test assertions. Removed unused _skill variable computation in agent-webhook handling. Updated review tool documentation and help text from yarn to pnpm references.
Documentation docs/BUILDING.md	Switched all build/dev commands from yarn to pnpm. Added comprehensive ARM Linux (aarch64) build guide including headless dependency setup, Tauri aarch64 target build, GTK/CEF library path configuration, .deb installation, and GTK initialization fix for tray startup errors.

Sequence Diagram(s)

sequenceDiagram
    participant App as Tauri App
    participant Event as RunEvent Handler
    participant GTK as GTK (Linux)
    participant Tray as Tray Setup

    App->>Event: Application starts
    Event->>Event: Execute .setup() closure<br/>(without tray init)
    Note over App: Platform-specific init...<br/>GTK initialization, etc.
    
    Event->>Event: RunEvent::Ready triggered
    rect rgba(100, 200, 100, 0.5)
        Event->>GTK: Check if Linux + CEF feature?
        alt Linux + CEF
            Event->>Tray: Log warning (no-op)
        else Other platforms
            Event->>Tray: setup_tray(app_handle)
            Tray->>Tray: Create tray menu
            Tray->>Tray: Register event handlers
        end
    end
    Tray-->>App: Tray ready

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

Possibly related PRs

• fix(tests): run webview_apis mock bridge on a dedicated runtime #893: Refactors mock WebSocket server in tests/webview_apis_bridge.rs to spawn on dedicated Tokio runtime thread with port communication, directly related to WebSocket test infrastructure changes.
• feat(linux-arm): add ARM Linux build support and GTK init fix #840: Adds ARM Linux build instructions and implements GTK initialization fix for tray startup, closely aligned with new BUILDING.md documentation and lib.rs tray refactoring.
• chore: migrate from yarn to pnpm #886: Performs repo-wide Yarn to pnpm migration across workflows, package.json, and build scripts, overlapping significantly with package manager migration scope.

Suggested reviewers

• senamakel

---

🐰 Hops with glee through pnpm's speed
No more yarn tangles, we plant new seed
Workflows now swifter, tray blooms at dawn
ARM Linux runs, the old redirect's gone ✨
Tests bundled tight on their own thread spun—
A brighter codebase for everyone! 🌱

🚥 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 'fix(app): make linux CEF deb package runnable' directly addresses the main objective: making the Linux CEF Debian package functional by fixing core binary discovery, GTK tray initialization, and Debian package dependencies.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

⚔️ Resolve merge conflicts

• Resolve merge conflict in branch fix/linux-cef-deb-runtime

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[senamakel]

jfyi some files are outdated since we merged pnpm changes. just pull from main, merge and push into ur PR

[VectorJet]

Got it closing this as it contains unnecessary commits. Creating a new one