From 259b3d65b8cfa9eeabd46edea835b63adba12839 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 27 Jul 2026 00:10:59 +0000 Subject: [PATCH 1/2] fix: use valid Nix comment syntax in modules --- modules/module/settings/cat-packages.nix | 8 ++++---- modules/module/specs/plugins.nix | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/modules/module/settings/cat-packages.nix b/modules/module/settings/cat-packages.nix index 49a33e6..fdc7145 100644 --- a/modules/module/settings/cat-packages.nix +++ b/modules/module/settings/cat-packages.nix @@ -5,10 +5,10 @@ ... }: let - -- Include packages from a category only if that category is enabled. - -- NOTE: The package list expression is still evaluated (packages in Nix are - -- lazy by default, so derivations are not built), so keep side-effecting - -- expressions out of these lists. + # Include packages from a category only if that category is enabled. + # NOTE: The package list expression is still evaluated (packages in Nix are + # lazy by default, so derivations are not built), so keep side-effecting + # expressions out of these lists. maybe = cat: pkgsList: lib.optionals (config.cats.${cat} or false) pkgsList; rPackages = (pkgs.baseRPackages or [ ]) ++ config.settings.lang_packages.r; diff --git a/modules/module/specs/plugins.nix b/modules/module/specs/plugins.nix index ff072cc..ffd7cff 100644 --- a/modules/module/specs/plugins.nix +++ b/modules/module/specs/plugins.nix @@ -190,9 +190,9 @@ in { ]; }; - -- Lazy-loaded plugins needed when the `r` cat is on. Kept separate from - -- `utils-lazy` so users with `r=true` and `utils=false` still get the - -- R debugger (via vscDebugger) and in-buffer image rendering for plots. + # Lazy-loaded plugins needed when the `r` cat is on. Kept separate from + # `utils-lazy` so users with `r=true` and `utils=false` still get the + # R debugger (via vscDebugger) and in-buffer image rendering for plots. config.specs.r-lazy = lib.mkIf (config.cats.r or false) { lazy = true; data = with pkgs.vimPlugins; [ From f4b960c6040ace861d44509fa7cb0f556bb175e6 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 27 Jul 2026 00:11:43 +0000 Subject: [PATCH 2/2] docs: clarify Nix laziness comment --- modules/module/settings/cat-packages.nix | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/module/settings/cat-packages.nix b/modules/module/settings/cat-packages.nix index fdc7145..e68ff93 100644 --- a/modules/module/settings/cat-packages.nix +++ b/modules/module/settings/cat-packages.nix @@ -6,9 +6,9 @@ }: let # Include packages from a category only if that category is enabled. - # NOTE: The package list expression is still evaluated (packages in Nix are - # lazy by default, so derivations are not built), so keep side-effecting - # expressions out of these lists. + # NOTE: Package list expressions are lazily evaluated, and derivations are + # not built until needed, so keep side-effecting expressions out of these + # lists. maybe = cat: pkgsList: lib.optionals (config.cats.${cat} or false) pkgsList; rPackages = (pkgs.baseRPackages or [ ]) ++ config.settings.lang_packages.r;