feat(packages): add Claude Desktop to Linux desktop packages - #2114
Conversation
|
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
You do not have enough credits to review this pull request. Please purchase more credits to continue. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe Linux desktop package list now includes ChangesLinux Desktop Packages
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Code Review
This pull request adds the pkgs.llm-agents.claude-desktop package to the list of default packages in home-manager/packages/default.nix. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| celluloid | ||
| cheese | ||
| chromium | ||
| pkgs.llm-agents.claude-desktop |
There was a problem hiding this comment.
claude-desktop attribute missing at the pinned llm-agents revision.
flake.lock pins numtide/llm-agents.nix to 8d041d85b7f0a93b11c04983c069d96eb59467a1 (committer date 2026-06-29). Listing packages/ in the upstream repo at that ref returns 134 directories and does not include claude-desktop — only auto-claude, claude-agent-acp, claude-code, claude-code-router, claude-plugins, claudebox, oh-my-claudecode.
Upstream history for packages/claude-desktop:
- init at v0.12.55 — 2025-07-30 (
a937853e) - drop package — 2025-11-23 (
574ab6ea) - re-init at v1.17377.2 — 2026-07-02 (
727aca9d) - bumped to v1.22209.0 — 2026-07-17 (
44a2771e)
The pinned commit sits inside the “dropped” window, so pkgs.llm-agents.claude-desktop evaluates to attribute 'claude-desktop' missing. Any evaluation of a Linux desktop configuration (matic, viper, or a linux+desktop home-manager profile) will fail — the PR test plan (“Rebuild NixOS on matic”) cannot succeed without also bumping flake.lock.
Fix: nix flake lock --update-input llm-agents to bring the input past 727aca9d (ideally 44a2771e, matching v1.22209.0 from the PR body) and commit the updated flake.lock.
Mesa DescriptionTL;DRAdds Claude Desktop to the list of packages for Linux desktop environments. What changed?
Description generated by Mesa. Update settings |
Summary
pkgs.llm-agents.claude-desktopto theisLinux && isDesktoppackage listllm-agents.nixflake input (v1.22209.0)Test plan
Summary by cubic
Adds
pkgs.llm-agents.claude-desktopto the Linux desktop package set so Claude Desktop is installed on Linux builds. Uses the existingllm-agents.nixflake input (v1.22209.0).Written for commit 98d3b7a. Summary will update on new commits.