Skip to content

feat(nixos): add Orca desktop AppImage for Linux #2243

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
shunkakinoki wants to merge 1 commit into from
Closed

feat(nixos): add Orca desktop AppImage for Linux #2243

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions home-manager/packages/default.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -235,6 +235,7 @@ with pkgs;
networkmanagerapplet
nwg-drawer
obsidian
orca-desktop
pavucontrol
playerctl
polari
Expand Down
36 changes: 31 additions & 5 deletions overlays/default.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,20 +76,20 @@
(_: prev: {
moshi-hook = prev.stdenv.mkDerivation rec {
pname = "moshi-hook";
version = "0.2.69";
version = "0.2.70";
src = prev.fetchurl {
url = "https://cdn.getmoshi.app/hook/v${version}/moshi-hook_${
if prev.stdenv.isDarwin then "Darwin" else "Linux"
}_${if prev.stdenv.hostPlatform.isAarch64 then "arm64" else "x86_64"}.tar.gz";
sha256 =
if prev.stdenv.isLinux && prev.stdenv.hostPlatform.isx86_64 then
"3903e2e5d1dba02f9e1f53df8cea6e2b3260e1581461b9a07ca65f18814b8b08"
"da319ee343777fe1e9dda59cdd3c1f014a580a1d816b8c1b1332d6412224354f"
else if prev.stdenv.isLinux && prev.stdenv.hostPlatform.isAarch64 then
"0a30e081399543551bbd0ba3320f3b28be814a585e5c591e168f8fd6d9565f07"
"df598ee8a1d6f929f44b9a1abd8131360ee67512378eab2fdc70b269558a5c78"
else if prev.stdenv.isDarwin && prev.stdenv.hostPlatform.isAarch64 then
"52258126b675dad210a8f04b83d8e90b359951af37474ff985d2c3f49102d981"
"f7c6b0b9c4c685afbec88aa8456eda897dc8d5880a3512942a7d091604765861"
else
"7cf24d316bafffc59d30e05d6ad6b27d4c03c9953ce901056f57f03c25e4b83b";
"ade7ecfb35073b135e1a946e54204f1959ff9970f831111fe4b913119d9b4bc4";
};
sourceRoot = ".";
dontConfigure = true;
Expand All @@ -100,6 +100,32 @@
'';
};
})
(_: prev: {
orca-desktop = prev.appimageTools.wrapType2 rec {
pname = "orca-desktop";
version = "1.4.168";
src = prev.fetchurl {
url = "https://github.com/stablyai/orca/releases/download/v${version}/orca-linux${
if prev.stdenv.hostPlatform.isAarch64 then "-arm64" else ""
}.AppImage";
sha256 =
if prev.stdenv.hostPlatform.isx86_64 then
"0vdzx9xjcympl35s91n5r9x89s1wdd664hcibxniqf6k5rg3ngqm"
else
"1ikz24ffb9c7fhs164g07vpc07akaz6w7wfsdfqh04cm3gj5glks";
};
extraInstallCommands =
let
contents = prev.appimageTools.extractType2 { inherit pname version src; };
in
''
install -Dm644 ${contents}/orca-ide.desktop $out/share/applications/orca-ide.desktop
substituteInPlace $out/share/applications/orca-ide.desktop \

@indent-zero indent-zero Bot Aug 4, 2026

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.

Silent Exec breakage on upstream change: --replace-warn only warns when Exec=AppRun isn't found. If a future Orca release rewords the Exec line (e.g., ships Exec=./AppRun or Exec=orca-ide), this substitution will no-op, and orca-ide.desktop will be installed with the original Exec=AppRun --no-sandbox %U — which is not on user PATH, so the desktop launcher stops working with no build-time signal. Prefer --replace-fail 'Exec=AppRun' so the build fails loudly and forces the fix.

Suggested change
substituteInPlace $out/share/applications/orca-ide.desktop \
substituteInPlace $out/share/applications/orca-ide.desktop \
--replace-fail 'Exec=AppRun' "Exec=$out/bin/${pname}"

--replace-warn 'Exec=AppRun' "Exec=$out/bin/${pname}"
cp -r ${contents}/usr/share/icons $out/share/icons 2>/dev/null || true

@indent-zero indent-zero Bot Aug 4, 2026

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.

Silent icon loss on upstream restructure: 2>/dev/null || true masks every failure mode of this copy — missing source dir, permission errors, etc. Today the AppImage does ship usr/share/icons/hicolor/*/apps/orca-ide.png (verified via --appimage-extract), so the guard is unnecessary. If upstream later moves icons, the build will still succeed and users get a .desktop with Icon=orca-ide that resolves to nothing. Recommend dropping the guard (or replacing it with an explicit if [ -d ... ] + comment explaining why it's optional).

Suggested change
cp -r ${contents}/usr/share/icons $out/share/icons 2>/dev/null || true
cp -r ${contents}/usr/share/icons $out/share/icons

@cubic-dev-ai cubic-dev-ai Bot Aug 4, 2026

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.

P3: The cp -r ... 2>/dev/null || true swallows every failure mode of this icon copy (missing source dir, permission errors, etc.), so if a future Orca release restructures or removes the icons directory, the build will still succeed silently while the installed .desktop entry's Icon=orca-ide resolves to nothing. Consider dropping the error-suppressing guard, or replace it with an explicit if [ -d ... ] check plus a comment explaining why missing icons are acceptable.

Prompt for AI agents 
Check if this issue is valid — if so, understand the root cause and fix it. At overlays/default.nix, line 125:

<comment>The `cp -r ... 2>/dev/null || true` swallows every failure mode of this icon copy (missing source dir, permission errors, etc.), so if a future Orca release restructures or removes the icons directory, the build will still succeed silently while the installed `.desktop` entry's `Icon=orca-ide` resolves to nothing. Consider dropping the error-suppressing guard, or replace it with an explicit `if [ -d ... ]` check plus a comment explaining why missing icons are acceptable.</comment>

<file context>
@@ -100,6 +100,32 @@
+          install -Dm644 ${contents}/orca-ide.desktop $out/share/applications/orca-ide.desktop
+          substituteInPlace $out/share/applications/orca-ide.desktop \
+            --replace-warn 'Exec=AppRun' "Exec=$out/bin/${pname}"
+          cp -r ${contents}/usr/share/icons $out/share/icons 2>/dev/null || true
+        '';
+    };
</file context>
Suggested change
cp -r ${contents}/usr/share/icons $out/share/icons 2>/dev/null || true
cp -r ${contents}/usr/share/icons $out/share/icons

'';
};
})
(final: prev: {
nightlyPkgs = import inputs.nixpkgs-nightly {
inherit (prev) system config;
Expand Down
Loading