fix: high resolution time for mtime

metatypedev · Jan 1, 2025 · 1689473 · 1689473
1 parent b9a0679
commit 1689473
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/docs/known-issues.md b/docs/known-issues.md
@@ -1,10 +1,5 @@
 # Known issues
 
-## Cache invalidation of imported scripts
-
-Currently, ghjk is unable to track changes to local typescript files imported by the ghjk.ts file.
-You can force re-serialization of the ghjkfile by deleting the `.ghjk/hash.json` file.
-
 ## GitHub API rate-limit
 
 The GitHub API is rate-limited to 60 calls per hour for unauthenticated requests. 

diff --git a/install/hook.fish b/install/hook.fish
@@ -1,11 +1,13 @@
 function __ghjk_get_mtime_ts 
     switch (uname -s | tr '[:upper:]' '[:lower:]')
         case "linux"
-            stat -c "%Y" $argv
+            stat -c "%.Y" $argv
         case "darwin"
-            stat -f "%Sm" -t "%s" $argv
+            # darwin stat doesn't support ms since epoch so we bring out the big guns
+            deno eval 'console.log((await Deno.stat(Deno.args[0])).mtime.getTime())' $argv
+            # stat -f "%Sm" -t "%s" $argv
         case "*"
-            stat -c "%Y" $argv
+            stat -c "%.Y" $argv
     end
 end
 

diff --git a/install/hook.sh b/install/hook.sh
@@ -7,7 +7,9 @@ __ghjk_get_mtime_ts () {
             stat -c "%Y" "$1"
         ;;
         "darwin")
-            stat -f "%Sm" -t "%s" "$1"
+            # darwin stat doesn't support ms since epoch so we bring out the big guns
+            deno eval 'console.log((await Deno.stat(Deno.args[0])).mtime.getTime())' "$1"
+            # stat -f "%Sm" -t "%s" "$1"
         ;;
         "*")
             stat -c "%Y" "$1"

diff --git a/tests/utils.ts b/tests/utils.ts
@@ -66,8 +66,9 @@ exec ${ghjkExePath.resolve().toString()} "$@"`,
     HOME: tmpDir.toString(),
     GHJK_REPO_ROOT: import.meta.resolve("../"),
     // share the system's deno cache
-    GHJK_DENO_DIR: Deno.env.get("DENO_DIR") ??
-      $.path(Deno.env.get("HOME")!).join(".cache", "deno").toString(),
+    GHJK_DENO_DIR: Deno.env.get("DENO_DIR")
+      ? $.path(Deno.env.get("DENO_DIR")!).resolve().toString()
+      : $.path(Deno.env.get("HOME")!).resolve(".cache", "deno").toString(),
     RUST_LOG: Deno.env.get("RUST_LOG"),
     GHJK_LOG: Deno.env.get("GHJK_LOG"),
     ...testEnvs,