From b3dd5f285365f59153a8f4549efa0607ccddf19d Mon Sep 17 00:00:00 2001 From: Ben Sima Date: Sat, 15 Nov 2025 10:06:18 -0500 Subject: Fix NixOS integration: separate package building from OS builds Problem: Calling bild.run inside NixOS configs triggered IFD during OS evaluation. ANSI escape codes from bild broke JSON parsing in Nix sandbox, causing build failures. Root cause: bild.run uses IFD (Import From Derivation) which runs bild --plan during Nix evaluation. When this happened inside NixOS service definitions, it ran recursively and bild output ANSI codes that corrupted the JSON analysis output. Solution: Two-phase architecture + NO_COLOR support 1. Biz/Packages.nix: Pre-builds all packages outside NixOS context 2. Biz.nix: Accepts packages as function argument (default: Packages.nix) 3. Omni/Bild.nix: Sets NO_COLOR=1 in analysis derivation 4. Omni/Log/Terminal.hs: Respects NO_COLOR env var 5. Omni/Log/Terminal.hs: Skip getTerminalSize when NO_COLOR set to avoid escape code output 6. Omni/Log/Concurrent.hs: Skip line initialization without ANSI support Now NixOS builds succeed: - Package IFD happens once at top level - No recursive builds during service evaluation - Clean JSON output from bild --plan in Nix sandbox - NixOS configs reference pre-analyzed packages Changes: - Add Biz/Packages.nix - standalone package builder - Update Biz.nix to accept packages argument - Update Biz/Dragons/Analysis.nix to use Packages.nix - Remove Biz/Targets.nix (replaced by Packages.nix) - Add NO_COLOR support throughout logging stack - Fix ANSI.getTerminalSize outputting escape codes when NO_COLOR set Amp-Thread-ID: https://ampcode.com/threads/T-bc0f6fc7-46bf-4aa2-892e-dd62e7251d4b Co-authored-by: Amp --- Biz/Targets.nix | 19 ------------------- 1 file changed, 19 deletions(-) delete mode 100644 Biz/Targets.nix (limited to 'Biz/Targets.nix') diff --git a/Biz/Targets.nix b/Biz/Targets.nix deleted file mode 100644 index 77e462c..0000000 --- a/Biz/Targets.nix +++ /dev/null @@ -1,19 +0,0 @@ -# Pre-declared build targets for the Biz namespace. -# -# This file exposes all buildable Biz targets as an attribute set, allowing -# NixOS configs to reference them directly without triggering recursive builds. -# -# To add a new target: -# 1. Add the attribute here pointing to bild.run ./path/to/target -# 2. Reference it in Biz.nix or other configs as targets. -{bild}: { - # Web services - storybook = bild.run ./Storybook.py; - podcastitlater-web = bild.run ./PodcastItLater/Web.py; - podcastitlater-worker = bild.run ./PodcastItLater/Worker.py; - - # CLI tools and analysis - dragons-analysis = bild.run ./Dragons/Analysis.hs; - - # Add new Biz targets here as they are created -} -- cgit v1.2.3