@zairakai/js-dev-tools 1.4.0 → 1.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zairakai/js-dev-tools",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "The unified and centralized development toolkit for JavaScript/TypeScript projects - ESLint, Prettier, Stylelint, Knip, Vitest, Git Hooks, and GitLab CI pipeline templates",
   "keywords": [
     "dev-tools",

package/scripts/config.sh

@@ -132,12 +132,25 @@ ensure_bin_optional() {
 }
 export -f ensure_bin_optional
 
-# Check if files matching pattern exist
+# Check if files matching any of the given patterns exist.
+#
+# Variadic: accepts one or more glob patterns, returns success if ANY matches.
+# Callers needing a brace-alternation pattern (e.g. *.{js,ts,vue}) must leave that
+# portion unquoted at the call site so bash's brace expansion splits it into
+# separate arguments BEFORE this function ever sees it — brace expansion only
+# applies to unquoted literal text, never to the contents of a variable, so a
+# pattern like "**/*.{js,ts}" passed as a single quoted string is matched
+# literally (and never matches a real file) instead of being expanded.
 has_files() {
-    local pattern="$1"
-    # SC2086: unquoted intentionally — we need word splitting + glob expansion via nullglob+globstar
-    # shellcheck disable=SC2086
-    ( shopt -s nullglob globstar; set -- $pattern; [ $# -gt 0 ] )
+    local pattern
+    for pattern in "$@"; do
+        # SC2086: unquoted intentionally — we need word splitting + glob expansion via nullglob+globstar
+        # shellcheck disable=SC2086
+        if ( shopt -s nullglob globstar; set -- $pattern; [ $# -gt 0 ] ); then
+            return 0
+        fi
+    done
+    return 1
 }
 export -f has_files
 

package/scripts/eslint-fix.sh

@@ -21,8 +21,18 @@ fi
 
 log_step "Fixing: $LINT_TARGET"
 
-# Skip gracefully if no matching files exist
-if ! has_files "${PROJECT_ROOT}/${LINT_TARGET}**/*.{js,ts,vue,jsx,tsx}"; then
+# Normalize away any trailing slash before appending our own — LINT_TARGET can be
+# "resources/js/", "src/" or "." (via override) depending on project type, and
+# concatenating "." directly with "**/*.ext" produces the invalid glob
+# ".**/*.ext" (no match ever).
+LINT_TARGET_GLOB_BASE="${LINT_TARGET%/}/"
+
+# Skip gracefully if no matching files exist.
+# The brace portion is intentionally left unquoted so bash's brace expansion
+# splits it into separate arguments before has_files() sees it (see has_files
+# doc comment in config.sh — brace expansion never applies to quoted text or
+# to the contents of a variable).
+if ! has_files "${PROJECT_ROOT}/${LINT_TARGET_GLOB_BASE}"**/*.{js,ts,vue,jsx,tsx}; then
     log_info "No JS/TS files found in ${LINT_TARGET} — skipping"
     exit 0
 fi

package/scripts/eslint.sh

@@ -31,8 +31,17 @@ fi
 
 log_step "Checking: $LINT_TARGET"
 
-# Skip gracefully if no matching files exist
-if ! has_files "${PROJECT_ROOT}/${LINT_TARGET}**/*.{js,ts,vue,jsx,tsx}"; then
+# Normalize away any trailing slash before appending our own — LINT_TARGET can be
+# "resources/js/", "src/" or "." depending on project type, and concatenating "."
+# directly with "**/*.ext" produces the invalid glob ".**/*.ext" (no match ever).
+LINT_TARGET_GLOB_BASE="${LINT_TARGET%/}/"
+
+# Skip gracefully if no matching files exist.
+# The brace portion is intentionally left unquoted so bash's brace expansion
+# splits it into separate arguments before has_files() sees it (see has_files
+# doc comment in config.sh — brace expansion never applies to quoted text or
+# to the contents of a variable).
+if ! has_files "${PROJECT_ROOT}/${LINT_TARGET_GLOB_BASE}"**/*.{js,ts,vue,jsx,tsx}; then
     log_info "No JS/TS files found in ${LINT_TARGET} — skipping"
     exit 0
 fi