aw-spec-cli 0.1.18 → 0.1.19

package/assets/.appweaver/scripts/bash/common.sh

@@ -26,17 +26,44 @@ find_specify_root() {
 
 # Get repository root, prioritizing .appweaver directory
 # This prevents using a parent repository when aw-spec is initialized in a subdirectory
+#
+# A `[` or `]` ANYWHERE IN THE ROOT REFUSES THE RUN, in both shells. Those two characters
+# are a wildcard character class to PowerShell's `-Path` family and to a shell/Python
+# glob, so under such a root a path test answers about a DIFFERENT path — `…/a[b]/x` is
+# looked up as `…/ab/x` — and an enumeration quietly returns nothing. The kit resolves
+# hundreds of paths against this root, and the failure mode is not an error but a wrong
+# answer: a gate that finds no delta reports a clean tree, and a merge that finds no
+# section merges nothing and says it succeeded.
+#
+# The per-call fixes are in (glob.escape on every pattern's directory here; -LiteralPath
+# throughout the PowerShell twin), and they cover every site the kit resolves through a
+# pattern. What they cannot cover is the ~150 plain existence tests, and a half-guarded
+# root is worse than a refused one because it looks like it works. So the root itself is
+# the gate, checked once, naming the path.
+_reject_bracketed_root() {
+    case "$1" in
+        *'['*|*']'*)
+            echo "aw-spec: this repository's path contains '[' or ']' — $1" >&2
+            echo "aw-spec: those characters are a wildcard character class to the path APIs this kit uses (PowerShell -Path, shell and Python globs), so file lookups would silently resolve to a different path or to nothing at all, and a gate that finds nothing reports success. Move or rename the checkout so no directory in its path contains a square bracket, then re-run." >&2
+            return 1 ;;
+    esac
+    return 0
+}
+
 get_repo_root() {
     # First, look for .appweaver directory (aw-spec's own marker)
     local specify_root
     if specify_root=$(find_specify_root); then
+        _reject_bracketed_root "$specify_root" || exit 1
         echo "$specify_root"
         return
     fi
 
     # Final fallback to script location
     local script_dir="$(CDPATH="" cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-    (cd "$script_dir/../../.." && pwd)
+    local fallback_root; fallback_root=$(cd "$script_dir/../../.." && pwd)
+    _reject_bracketed_root "$fallback_root" || exit 1
+    echo "$fallback_root"
 }
 
 # FEATURE RESOLUTION IS SINGLE-CHANNEL: `.appweaver/feature.json`, written by