nix: Skip pre-commit install if hooks already exist

[leshy]

Avoids running pre-commit install on every shell entry when hooks are already set up.

[greptile-apps]

Greptile Overview

Greptile Summary

Added a check to skip pre-commit install --install-hooks on shell entry if hooks are already installed, preventing redundant installation.

• Only verifies .git/hooks/pre-commit exists, but the configuration installs two hook types: pre-commit and commit-msg
• If commit-msg hook is missing while pre-commit exists, installation will be incorrectly skipped

Confidence Score: 2/5

• This PR has a logic bug that could prevent proper hook installation
• The check only verifies one of two required hook files, which means the commit-msg hook may not be installed in some scenarios
• flake.nix:227 needs to check for both hook files

Important Files Changed

Filename	Overview
flake.nix	Added check to skip pre-commit install, but only verifies one of two required hook files

Sequence Diagram

sequenceDiagram
    participant Shell as Shell Entry
    participant Script as shellHook Script
    participant Git as Git Repository
    participant PreCommit as pre-commit

    Shell->>Script: Enter dev shell
    Script->>Git: Check for .pre-commit-config.yaml
    
    alt Config exists
        Script->>Git: Check for .git/hooks/pre-commit
        
        alt Hook exists (NEW CHECK)
            Script->>Script: Skip installation
            Note over Script,PreCommit: Optimization: avoid redundant install
        else Hook missing
            Script->>PreCommit: pre-commit install --install-hooks
            PreCommit->>Git: Install pre-commit hook
            PreCommit->>Git: Install commit-msg hook
            Note over PreCommit,Git: Issue: commit-msg not verified!
        end
    else Config missing
        Script->>Script: Skip installation
    end
    
    Script->>Shell: Continue shell initialization

Loading

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[jeff-hykin]

LGTM