chore: minor repo cleanups (doc drift, unused unfree entry, editorconfig, gitignore, magic LAN, overlay re-import) #53

New Issue

2026-06-29T13:02:58+01:00

lyrathorpe commented

2026-06-29 13:02:58 +01:00

Minor cleanups (low priority)

A bundle of small, independent tidy-ups surfaced during a repo audit:

Doc drift: home/README.md lists the Nix LSP as nil, but home/editor.nix enables nil_ls. Update the doc.
Unused unfree entry: flake.nix unfreePackages lists lens-desktop, which nothing references (only lens is used). Remove it.
.editorconfig: the second rule block duplicates the [*] defaults and is a no-op. Remove it (or add real per-type overrides).
.gitignore vs firmware: modules/firmware/* is ignored while those blobs are force-tracked. It works, but hides them from git status and risks accidental git clean. Consider un-ignoring (\!modules/firmware/) or replacing with a comment.
Hardcoded LAN: hosts/RPi5/docker.nix hardcodes 10.187.1.0/24 in an nftables rule. Fine for the homelab, but a magic value worth turning into a documented let binding or option.
Overlay re-import: the claude-code overlay does import nixpkgs-unstable per evaluation (per system, and again in mkHome), pulling a full nixpkgs-unstable each time for one package. Minor eval cost; could be hoisted to a single import.

None of these affects host evaluation; they are hygiene.

## Minor cleanups (low priority) A bundle of small, independent tidy-ups surfaced during a repo audit: - **Doc drift:** `home/README.md` lists the Nix LSP as `nil`, but `home/editor.nix` enables `nil_ls`. Update the doc. - **Unused unfree entry:** `flake.nix` `unfreePackages` lists `lens-desktop`, which nothing references (only `lens` is used). Remove it. - **`.editorconfig`:** the second rule block duplicates the `[*]` defaults and is a no-op. Remove it (or add real per-type overrides). - **`.gitignore` vs firmware:** `modules/firmware/*` is ignored while those blobs are force-tracked. It works, but hides them from `git status` and risks accidental `git clean`. Consider un-ignoring (`\!modules/firmware/`) or replacing with a comment. - **Hardcoded LAN:** `hosts/RPi5/docker.nix` hardcodes `10.187.1.0/24` in an nftables rule. Fine for the homelab, but a magic value worth turning into a documented `let` binding or option. - **Overlay re-import:** the `claude-code` overlay does `import nixpkgs-unstable` per evaluation (per system, and again in `mkHome`), pulling a full nixpkgs-unstable each time for one package. Minor eval cost; could be hoisted to a single import. None of these affects host evaluation; they are hygiene.

lyrathorpe referenced this issue from a commit

2026-06-29 13:29:39 +01:00

chore: minor cleanups (lens unfree name, LSP doc, editorconfig, docker subnet)

lyrathorpe referenced this issue

2026-06-29 13:29:53 +01:00

chore: post-refactor cleanups (#50-#53) #54

lyrathorpe referenced this issue from a commit

2026-06-29 13:34:29 +01:00

chore: minor cleanups (remove unused lens, LSP doc, editorconfig, docker subnet)

lyrathorpe referenced this issue from a commit

2026-06-29 14:09:39 +01:00

chore: post-refactor cleanups (#50-#53) (#54)