From 03bd9b6ad5335aff65d9cb6fb1d117b5cb4b7919 Mon Sep 17 00:00:00 2001 From: Arpit Chakladar Date: Sat, 18 Jul 2026 14:55:21 +0530 Subject: refactor(scripts): split monolithic module into per-script submodules Extract shared `mkScript` and `mkScriptModule` helpers into `lib.nix` Split each script into `modules/scripts//` with its own `default.nix`, `script.sh`, and `assertions.nix` Remove `conditions` parameter from `mkScriptModule`; move condition assertions into per-script `assertions.nix` files with inline checks and error messages Simplify `mkScriptModule` to return `{ options, moduleConfig }` instead of a module function, taking `config` as a direct parameter Add `description` field to `mkScript` that inserts the description as a `# ` comment between the shebang and env vars Thread `description` through `mkScriptModule` to each `default.nix` Move top comment lines from each `script.sh` into the `description` field and remove from sources to avoid duplication Remove old flat `.sh` files from `modules/scripts/` root --- modules/scripts/file-preview/assertions.nix | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) create mode 100644 modules/scripts/file-preview/assertions.nix (limited to 'modules/scripts/file-preview/assertions.nix') diff --git a/modules/scripts/file-preview/assertions.nix b/modules/scripts/file-preview/assertions.nix new file mode 100644 index 0000000..41fd4d6 --- /dev/null +++ b/modules/scripts/file-preview/assertions.nix @@ -0,0 +1,19 @@ +{ config, ... }: +[ + { + assertion = !config.scripts.file-preview.enable || config.terminal.kitty.enable; + message = "scripts.file-preview is enabled but requires `terminal.kitty.enable`."; + } + { + assertion = !config.scripts.file-preview.enable || config.media.ffmpeg.enable; + message = "scripts.file-preview is enabled but requires `media.ffmpeg.enable`."; + } + { + assertion = !config.scripts.file-preview.enable || config.file-management.ouch.enable; + message = "scripts.file-preview is enabled but requires `file-management.ouch.enable`."; + } + { + assertion = !config.scripts.file-preview.enable || config.terminal.bat.enable; + message = "scripts.file-preview is enabled but requires `terminal.bat.enable`."; + } +] -- cgit v1.2.3