ci: fix restore-mtime exit code when last file is deleted on macos runners

[desmondcheongzx]

GitHub Actions shell: bash runs with set -eo pipefail. The [ -f "$file" ] && touch_epoch pattern returns exit code 1 when the file doesn't exist on disk (deleted in a later commit), and if this happens on the last iteration of the while loop, pipefail propagates that exit code and kills the step.

Switch to if/then/fi so a non-existent file is a no-op with exit code 0.

Verified with a temporary smoke-test job on both ubuntu-latest (11s) and macos-latest (54s) that confirmed mtimes are correctly restored on both platforms.

Fixes the restore-mtime failures seen on macOS runners in #6264.

[greptile-apps]

Greptile Summary

This PR fixes a critical CI issue where the restore-mtime action was failing on macOS runners when the last file processed in the loop was deleted in a later commit.

The root cause is that GitHub Actions runs bash scripts with set -eo pipefail, which means any command returning a non-zero exit code will terminate the script. The original pattern [ -f "$file" ] && touch_epoch returns exit code 1 when the file doesn't exist, and when this occurs on the last iteration of the while loop, pipefail propagates that exit code and kills the step.

The fix replaces the shorthand && pattern with an explicit if/then/fi block, ensuring that non-existent files are handled as a no-op with exit code 0 instead of causing the script to fail.

• Changed from: [ -f "$file" ] && touch_epoch "$ts" "$file"
• Changed to: if [ -f "$file" ]; then touch_epoch "$ts" "$file"; fi

This is a correct and safe fix that maintains the same logical behavior while preventing spurious CI failures.

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The change is a straightforward bug fix that correctly addresses a shell scripting edge case. The fix transforms a pattern that can fail ([ -f ] &&) into an equivalent pattern that always succeeds (if [ -f ]; then ... fi). This maintains the exact same logical behavior while preventing spurious exit code failures. The change is minimal, well-documented, and directly addresses the reported issue in PR ci: cache workspace crates and share rust caches across all workflows #6264.
• No files require special attention

Important Files Changed

Filename	Overview
.github/actions/restore-mtime/action.yaml	Fixed shell script exit code handling by replacing && pattern with explicit if/then/fi to prevent pipeline failures when files don't exist

_{Last reviewed commit: 7f096bb}

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.44%. Comparing base (8cb880d) to head (bf183b0).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6268      +/-   ##
==========================================
- Coverage   73.44%   73.44%   -0.01%     
==========================================
  Files        1003     1003              
  Lines      133436   133156     -280     
==========================================
- Hits        98005    97794     -211     
+ Misses      35431    35362      -69     

see 23 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.