Revert "Restore MacOS Support (flake.nix)"

[leshy]

Reverts #863

breaks uv .venv, shell in some sort of a permanent loop

[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

Reverts PR #863 which added MacOS support through complex Nix flake configuration. The MacOS support implementation caused issues with uv .venv setup and created an infinite shell loop, blocking development workflow.

Changes Reverted

• flake.nix: Removes 230+ lines of MacOS-specific workarounds including:

  • Complex aggregation system with lib.aggregator for package flagging
  • cc-no-usr-include wrapper script to patch system include paths
  • Platform-specific conditionals (stdenv.isDarwin/stdenv.isLinux)
  • MacOS-specific LCM patches for fsync/fdatasync issues
  • xome isolated shell environment with home-manager integration
  • Automatic venv detection and bin/_dev_init prompting

• flake.lock: Removes dependencies on lib (quick-nix-toolkits), xome, and home-manager flake inputs

• bin/_dev_init: Deletes 127-line interactive setup script that handled:

  • Git LFS installation and model downloads
  • Virtual environment creation
  • MacOS-specific workarounds (LCM, old MacOS version detection)
  • Interactive prompts for CUDA/CPU/simulation dependencies

• README.md: Removes instructions to manually enable Nix flakes and MacOS ulimit TODO

Impact

This PR restores the simpler, Linux-focused Nix configuration (matching commit 3ec11e0). The codebase returns to a working state for Linux users, but MacOS support is lost. The project now uses uv for Python environment management instead of the flake-managed venv approach that was causing loops.

Confidence Score: 5/5

• This PR is safe to merge - it's a clean revert that fixes blocking development issues
• This is a straightforward git revert of a recent problematic PR. The changes cleanly restore the previous working state (matching commit 3ec11e0) that was known to work on Linux. The revert is justified by the reported issue that PR Restore MacOS Support (flake.nix) #863 broke the uv .venv workflow and caused infinite shell loops, which are critical blockers for development. While this does remove MacOS support, it's the right tradeoff to unblock the team while MacOS issues can be investigated separately.
• No files require special attention - this is a clean revert

Important Files Changed

Filename	Overview
flake.nix	Reverts complex MacOS support changes, simplifies to Linux-only configuration removing 230+ lines of MacOS-specific workarounds and abstraction layers
flake.lock	Removes dependencies on lib, xome, and home-manager flake inputs that were added for MacOS support
bin/_dev_init	Deletes 127-line interactive setup script that handled venv creation, git lfs, and MacOS-specific workarounds
README.md	Removes flakes enablement instructions and MacOS ulimit TODO from installation section

Sequence Diagram

sequenceDiagram
    participant User
    participant NixDevelop as nix develop
    participant FlakeNix as flake.nix
    participant ShellHook as shellHook
    participant DevInit as bin/_dev_init
    participant VirtualEnv as venv/uv

    Note over User,VirtualEnv: Before Revert (PR #863)
    User->>NixDevelop: Run nix develop
    NixDevelop->>FlakeNix: Load complex flake with xome/lib
    FlakeNix->>ShellHook: Execute multi-platform shellHook
    ShellHook->>VirtualEnv: Check for venv, activate if exists
    alt No venv found
        ShellHook->>User: Prompt to run bin/_dev_init
        User->>DevInit: Run bin/_dev_init
        DevInit->>VirtualEnv: Create venv, install deps
        Note over DevInit,VirtualEnv: Issue: Causes infinite loop<br/>with uv .venv
    end
    ShellHook->>ShellHook: Set CC=cc-no-usr-include
    ShellHook->>ShellHook: Complex env setup (PKG_CONFIG_PATH, etc)
    
    Note over User,VirtualEnv: After Revert (Current PR)
    User->>NixDevelop: Run nix develop
    NixDevelop->>FlakeNix: Load simple flake (Linux-only)
    FlakeNix->>ShellHook: Execute simplified shellHook
    ShellHook->>ShellHook: Setup NVIDIA libs only
    ShellHook->>VirtualEnv: Activate env/bin/activate if exists
    Note over ShellHook: No automatic venv creation<br/>No _dev_init prompting<br/>No uv loop issues

Loading