flakes: Don't run shellHook on cached dev shells

This makes the behavior of `use flake` consistent with that of `use
nix`.

Resolve #90.

direnvrc

@@ -135,6 +135,7 @@ use_flake() {
 
     log_status renewed cache
   else
+    sed -i '/eval \"\$shellHook\"/d' "$profile_rc"
     log_status using cached dev shell
   fi
 

tests/test.py

@@ -55,6 +55,7 @@ def test_nix_shell(self) -> None:
         )
         sys.stderr.write(out1.stderr)
         self.assertIn("renewed cache and derivation link", out1.stderr)
+        self.assertIn("Executing shellHook.", out1.stderr)
         self.assertEqual(out1.returncode, 0)
 
         run(["nix-collect-garbage"], check=True)
@@ -67,6 +68,7 @@ def test_nix_shell(self) -> None:
         )
         sys.stderr.write(out2.stderr)
         self.assertIn("using cached derivation", out2.stderr)
+        self.assertNotIn("Executing shellHook.", out2.stderr)
         self.assertEqual(out2.returncode, 0)
 
     @unittest.skipUnless(support_flakes(), "requires flakes")
@@ -86,6 +88,7 @@ def test_nix_flake(self) -> None:
         )
         sys.stderr.write(out1.stderr)
         self.assertIn("renewed cache", out1.stderr)
+        self.assertIn("Executing shellHook.", out1.stderr)
         self.assertEqual(out1.returncode, 0)
 
         run(["nix-collect-garbage"], check=True)
@@ -107,6 +110,7 @@ def test_nix_flake(self) -> None:
         for symlink in inputs:
             self.assertTrue(symlink.is_dir())
         self.assertIn("using cached dev shell", out2.stderr)
+        self.assertNotIn("Executing shellHook.", out2.stderr)
 
         self.assertEqual(out2.returncode, 0)
 

tests/testenv/shell.nix

@@ -1,4 +1,7 @@
 { pkgs ? import <nixpkgs> {} }:
 pkgs.mkShell {
   nativeBuildInputs = [ pkgs.hello ];
+  shellHook = ''
+    echo "Executing shellHook."
+  '';
 }