fix: use overlays.shared-nixpkgs (overlays.default removed upstream) #2

Open
eumel wants to merge 0 commits from eumel/nixos-config:fix/llm-agents-overlay-rename into main
First-time contributor

Upstream numtide/llm-agents.nix removed overlays.default and only keeps overlays.shared-nixpkgs.

Root cause: The CI workflow runs nix flake update before the check step, which pulls the latest llm-agents.nix revision. That revision no longer exports overlays.default, causing:

error: attribute 'default' missing
at modules/nixpkgsConfig.nix:70:5

Fix: Use overlays.shared-nixpkgs instead. It does the same thing (provides pkgs.llm-agents.*) but builds packages against the consumer's nixpkgs instead of using pre-built flake packages.

Affected runs: #3125–#3134 (all failing since the upstream change)

Upstream [numtide/llm-agents.nix](https://github.com/numtide/llm-agents.nix) removed `overlays.default` and only keeps `overlays.shared-nixpkgs`. **Root cause:** The CI workflow runs `nix flake update` before the check step, which pulls the latest llm-agents.nix revision. That revision no longer exports `overlays.default`, causing: ``` error: attribute 'default' missing at modules/nixpkgsConfig.nix:70:5 ``` **Fix:** Use `overlays.shared-nixpkgs` instead. It does the same thing (provides `pkgs.llm-agents.*`) but builds packages against the consumer's nixpkgs instead of using pre-built flake packages. **Affected runs:** #3125–#3134 (all failing since the upstream change)
Upstream llm-agents.nix removed overlays.default in favor of
overlays.shared-nixpkgs. The default overlay was using pre-built
packages from the flake, while shared-nixpkgs builds them against
the consumer's nixpkgs — which is the recommended approach anyway.

This fixes all failing CI runs (#3125–#3134) that fail at the
'nix flake archive/check' step with:
  error: attribute 'default' missing
  at modules/nixpkgsConfig.nix:70:5
update flake.lock: llm-agents -> 5c73869 (overlays.shared-nixpkgs)
All checks were successful
Build all NixOS Configurations / nix build (push) Successful in 57m17s
795a05fc21
All checks were successful
Build all NixOS Configurations / nix build (push) Successful in 57m17s
This branch is already included in the target branch. There is nothing to merge.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u fix/llm-agents-overlay-rename:eumel-fix/llm-agents-overlay-rename
git switch eumel-fix/llm-agents-overlay-rename

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff eumel-fix/llm-agents-overlay-rename
git switch eumel-fix/llm-agents-overlay-rename
git rebase main
git switch main
git merge --ff-only eumel-fix/llm-agents-overlay-rename
git switch eumel-fix/llm-agents-overlay-rename
git rebase main
git switch main
git merge --no-ff eumel-fix/llm-agents-overlay-rename
git switch main
git merge --squash eumel-fix/llm-agents-overlay-rename
git switch main
git merge --ff-only eumel-fix/llm-agents-overlay-rename
git switch main
git merge eumel-fix/llm-agents-overlay-rename
git push origin main
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
palo/nixos-config!2
No description provided.