[state] Ignore shell mismatch for run (#1774)

## Summary

This is a bit hacky, but was smallest, safest I could come up with. This
fixes the following case:

* User uses fish as shell
* User adds `devbox run` in their init hook
* User gets warning that environment is stale. This happens because
shell doesn't match, but we don't really care that shell doesn't match
for `run`. We only care for `shell` because we use different shellrc for
fish vs POSIX.

## How was it tested?

Added `devbox run echo foo` to init hook. Then ran

```
SHELL=fish devbox run echo bar
```

Author: mikeland73

internal/devbox/devbox.go

@@ -233,6 +233,7 @@ func (d *Devbox) RunScript(ctx context.Context, cmdName string, cmdArgs []string
 		return err
 	}
 
+	lock.SetIgnoreShellMismatch(true)
 	env, err := d.ensureStateIsUpToDateAndComputeEnv(ctx)
 	if err != nil {
 		return err

internal/lock/statehash.go

@@ -13,6 +13,8 @@ import (
 	"go.jetpack.io/devbox/internal/cuecfg"
 )
 
+var ignoreShellMismatch = false
+
 // stateHashFile is a non-shared lock file that helps track the state of the
 // local devbox environment. It contains hashes that may not be the same across
 // machines (e.g. manifest hash).
@@ -46,6 +48,12 @@ func UpdateAndSaveStateHashFile(args UpdateStateHashFileArgs) error {
 	return cuecfg.WriteFile(stateHashFilePath(args.ProjectDir), newLock)
 }
 
+// SetIgnoreShellMismatch is used to disable the shell comparison when checking
+// if the state is up to date. This is useful when we don't load shellrc (e.g. running)
+func SetIgnoreShellMismatch(ignore bool) {
+	ignoreShellMismatch = ignore
+}
+
 func isStateUpToDate(args UpdateStateHashFileArgs) (bool, error) {
 	filesystemStateHash, err := readStateHashFile(args.ProjectDir)
 	if err != nil {
@@ -56,6 +64,10 @@ func isStateUpToDate(args UpdateStateHashFileArgs) (bool, error) {
 		return false, err
 	}
 
+	if ignoreShellMismatch {
+		filesystemStateHash.IsFish = newStateHash.IsFish
+	}
+
 	return *filesystemStateHash == *newStateHash, nil
 }