jeff flake without py env stuff

[leshy]

flake without python init commands which caused infinite loop with uv venv for some reason

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[greptile-apps]

Greptile Summary

This PR restores MacOS support by re-applying changes from #863 (which was reverted in #907) but removes the automatic Python venv setup/checking logic from the shellHook that was causing infinite loops with uv venv.

Key changes:

• Adds lib (jeff-hykin/quick-nix-toolkits) and xome dependencies for better cross-platform package management
• Removes automatic venv detection/activation/warning logic from shellHook (lines that checked for venv/bin/activate and prompted users)
• Moves venv activation for env/ (not venv/) earlier in shellHook, before motd/pre-commit
• Introduces package aggregator pattern with flags for better organization of LD_LIBRARY_PATH, pkg-config paths, etc
• Adds isolated shell option using xome for stricter development environments
• Fixes NVIDIA libs directory to use static path instead of $$ (prevents creating multiple directories)
• Adds platform-specific package filtering (darwin vs linux) throughout

Confidence Score: 4/5

• This PR is safe to merge with minor risk - it's essentially a revert of a previous revert with a targeted fix
• The changes restore previously reviewed MacOS support (Restore MacOS Support (flake.nix) #863) while removing the problematic venv auto-setup that caused infinite loops. The refactor is well-structured using the aggregator pattern. Minor risk because: (1) Large surface area of changes (200+ lines) increases chance of unexpected behavior, (2) shellHook changes affect every developer's environment initialization, (3) Platform-specific logic could have edge cases. However, the core approach is sound and the infinite loop issue is directly addressed.
• Pay close attention to flake.nix shellHook section - test that entering the nix shell works correctly on both Linux and MacOS, and verify manual venv setup still works as expected

Important Files Changed

Filename	Overview
flake.nix	Major refactor: Removed automatic venv setup/checking logic from shellHook, added MacOS support with lib/xome dependencies, improved package management with aggregator pattern
flake.lock	Added new dependencies: lib (quick-nix-toolkits), xome, home-manager, and related sources for MacOS support

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Shell as nix develop
    participant Hook as shellHook
    participant Env as Environment Setup
    participant Venv as Python venv

    Dev->>Shell: nix develop
    Shell->>Hook: Execute shellHook
    
    alt Linux System
        Hook->>Env: Set CC="cc-no-usr-include"
        Hook->>Env: Create NVIDIA_LIBS_DIR
        Hook->>Env: Symlink NVIDIA libs
    end
    
    Hook->>Env: Export LD_LIBRARY_PATH
    Hook->>Env: Export LIBRARY_PATH
    Hook->>Env: Export GI_TYPELIB_PATH
    Hook->>Env: Export PKG_CONFIG_PATH
    Hook->>Env: Set CFLAGS & LDFLAGS
    Hook->>Env: Alias pytest
    
    Hook->>Venv: Check for env/bin/activate
    alt env/bin/activate exists
        Venv->>Hook: Activate venv
    end
    
    Hook->>Env: Show motd (if exists)
    Hook->>Env: Install pre-commit hooks
    
    Note over Hook,Venv: REMOVED: Automatic venv/ checking<br/>REMOVED: UV venv infinite loop trigger
    
    Hook->>Dev: Shell ready

Loading