chore(dev): check modified files only for style (#24106)

* chore(dev): check modified files only for style

* fix git command

Author: pront

scripts/check-style.sh

@@ -6,12 +6,33 @@ set -euo pipefail
 # SUMMARY
 #
 #   Checks that all text files have correct line endings and no trailing spaces.
+#
+# USAGE
+#
+#   ./scripts/check-style.sh [--fix] [--all]
+#
+#   --fix: Fix issues instead of just reporting them
+#   --all: Check all files (default: only check modified files)
 
-if [ "${1:-}" == "--fix" ]; then
-  MODE="fix"
-else
-  MODE="check"
-fi
+MODE="check"
+CHECK_ALL=false
+
+# Parse arguments
+for arg in "$@"; do
+  case "$arg" in
+    --fix)
+      MODE="fix"
+      ;;
+    --all)
+      CHECK_ALL=true
+      ;;
+    *)
+      echo "Unknown option: $arg"
+      echo "Usage: $0 [--fix] [--all]"
+      exit 1
+      ;;
+  esac
+done
 
 ised() {
   local PAT="$1"
@@ -23,8 +44,22 @@ ised() {
   rm "$FILE.bak"
 }
 
+# Determine which files to check
+if [ "$CHECK_ALL" = true ]; then
+  # Check all files tracked by git
+  FILES=$(git ls-files)
+else
+  # Check only files changed in current branch compared to origin/master
+  FILES=$(git diff --name-only "origin/master"...HEAD)
+
+  # If no changed files, fall back to checking all files
+  if [ -z "$FILES" ]; then
+    FILES=$(git ls-files)
+  fi
+fi
+
 EXIT_CODE=0
-for FILE in $(git ls-files); do
+for FILE in $FILES; do
   # Ignore binary files and generated files.
   case "$FILE" in
     *png) continue;;

vdev/src/commands/check/fmt.rs

@@ -9,7 +9,10 @@ pub struct Cli {}
 
 impl Cli {
     pub fn exec(self) -> Result<()> {
-        app::exec::<&str>("scripts/check-style.sh", [], true)?;
+        info!("Checking style (trailing spaces, line endings)...");
+        app::exec("scripts/check-style.sh", ["--all"], true)?;
+
+        info!("Checking Rust formatting...");
         app::exec("cargo", ["fmt", "--", "--check"], true)
     }
 }

vdev/src/commands/fmt.rs

@@ -9,7 +9,10 @@ pub struct Cli {}
 
 impl Cli {
     pub fn exec(self) -> Result<()> {
+        info!("Checking style (trailing spaces, line endings)...");
         app::exec("scripts/check-style.sh", ["--fix"], true)?;
+
+        info!("Formatting Rust code...");
         app::exec("cargo", ["fmt", "--all"], true)
     }
 }