@jonit-dev/night-watch-cli 1.7.88 → 1.7.89

package/dist/scripts/night-watch-audit-cron.sh

@@ -165,7 +165,7 @@ for AUDIT_ATTEMPT in $(seq 1 "${AUDIT_MAX_RETRIES}"); do
   log "AUDIT: Attempt ${AUDIT_ATTEMPT}/${AUDIT_MAX_RETRIES} starting provider=${PROVIDER_CMD} timeout=${MAX_RUNTIME}s"
 
   # Build provider command array using generic helper
-  mapfile -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${AUDIT_WORKTREE_DIR}" "${AUDIT_PROMPT}")
+  mapfile -d '' -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${AUDIT_WORKTREE_DIR}" "${AUDIT_PROMPT}")
 
   # Execute — always cd into worktree so provider tools resolve project files correctly
   if (cd "${AUDIT_WORKTREE_DIR}" && timeout "${MAX_RUNTIME}" "${PROVIDER_CMD_PARTS[@]}" 2>&1 | tee -a "${LOG_FILE}"); then

package/dist/scripts/night-watch-cron.sh

@@ -542,7 +542,7 @@ while [ "${ATTEMPT}" -lt "${MAX_RETRIES}" ]; do
   LOG_LINE_BEFORE=$(wc -l < "${LOG_FILE}" 2>/dev/null || echo 0)
 
   # Build provider command array using generic helper
-  mapfile -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${WORKTREE_DIR}" "${PROMPT}")
+  mapfile -d '' -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${WORKTREE_DIR}" "${PROMPT}")
 
   # Execute — always cd into worktree so provider tools resolve project files correctly
   if (cd "${WORKTREE_DIR}" && timeout "${SESSION_MAX_RUNTIME}" "${PROVIDER_CMD_PARTS[@]}" 2>&1 | tee -a "${LOG_FILE}"); then

package/dist/scripts/night-watch-helpers.sh

@@ -90,8 +90,9 @@ validate_provider() {
 # ── Generic Provider Command Builder ──────────────────────────────────────────
 
 # Build a provider command from NW_PROVIDER_* environment variables.
-# Outputs one argument per line so callers can capture it as an array via mapfile.
-# This enables generic provider invocation without hardcoded case statements.
+# Outputs one NUL-delimited argument so callers can capture it as an array via
+# `mapfile -d ''`. This preserves multiline prompt arguments verbatim instead of
+# splitting them into multiple argv entries.
 #
 # Required env vars:
 #   NW_PROVIDER_CMD - CLI binary (e.g., "claude", "codex")
@@ -105,44 +106,44 @@ validate_provider() {
 #   NW_PROVIDER_MODEL         - Model value (e.g., "claude-opus-4-6")
 #
 # Usage:
-#   mapfile -t CMD_PARTS < <(build_provider_cmd "/path/to/worktree" "Your prompt here")
+#   mapfile -d '' -t CMD_PARTS < <(build_provider_cmd "/path/to/worktree" "Your prompt here")
 #   "${CMD_PARTS[@]}"
 #
-# Returns: One argument per line (safe for mapfile array capture)
+# Returns: One NUL-delimited argument per entry (safe for mapfile -d '' array capture)
 build_provider_cmd() {
   local workdir="${1:?workdir required}"
   local prompt="${2:?prompt required}"
 
   # Binary
-  printf '%s\n' "${NW_PROVIDER_CMD}"
+  printf '%s\0' "${NW_PROVIDER_CMD}"
 
   # Optional subcommand (e.g., "exec" for codex)
   if [ -n "${NW_PROVIDER_SUBCOMMAND:-}" ]; then
-    printf '%s\n' "${NW_PROVIDER_SUBCOMMAND}"
+    printf '%s\0' "${NW_PROVIDER_SUBCOMMAND}"
   fi
 
   # Working directory flag (if provider supports it)
   if [ -n "${NW_PROVIDER_WORKDIR_FLAG:-}" ]; then
-    printf '%s\n' "${NW_PROVIDER_WORKDIR_FLAG}"
-    printf '%s\n' "${workdir}"
+    printf '%s\0' "${NW_PROVIDER_WORKDIR_FLAG}"
+    printf '%s\0' "${workdir}"
   fi
 
   # Auto-approve flag
   if [ -n "${NW_PROVIDER_APPROVE_FLAG:-}" ]; then
-    printf '%s\n' "${NW_PROVIDER_APPROVE_FLAG}"
+    printf '%s\0' "${NW_PROVIDER_APPROVE_FLAG}"
   fi
 
   # Model selection (only if both flag and value are set)
   if [ -n "${NW_PROVIDER_MODEL_FLAG:-}" ] && [ -n "${NW_PROVIDER_MODEL:-}" ]; then
-    printf '%s\n' "${NW_PROVIDER_MODEL_FLAG}"
-    printf '%s\n' "${NW_PROVIDER_MODEL}"
+    printf '%s\0' "${NW_PROVIDER_MODEL_FLAG}"
+    printf '%s\0' "${NW_PROVIDER_MODEL}"
   fi
 
   # Prompt - either with flag or positional
   if [ -n "${NW_PROVIDER_PROMPT_FLAG:-}" ]; then
-    printf '%s\n' "${NW_PROVIDER_PROMPT_FLAG}"
+    printf '%s\0' "${NW_PROVIDER_PROMPT_FLAG}"
   fi
-  printf '%s\n' "${prompt}"
+  printf '%s\0' "${prompt}"
 }
 
 # Check if the provider uses a working directory flag (vs requiring cd).

package/dist/scripts/night-watch-pr-reviewer-cron.sh

@@ -1076,7 +1076,7 @@ for ATTEMPT in $(seq 1 "${TOTAL_ATTEMPTS}"); do
   REVIEWER_PROMPT="${REVIEWER_PROMPT_BASE}${TARGET_SCOPE_PROMPT}${PRD_CONTEXT_PROMPT}"
 
   # Build provider command array using generic helper
-  mapfile -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${REVIEW_WORKTREE_DIR}" "${REVIEWER_PROMPT}")
+  mapfile -d '' -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${REVIEW_WORKTREE_DIR}" "${REVIEWER_PROMPT}")
 
   # Execute — always cd into worktree so provider tools resolve project files correctly
   if (cd "${REVIEW_WORKTREE_DIR}" && timeout "${ATTEMPT_TIMEOUT}" "${PROVIDER_CMD_PARTS[@]}" 2>&1 | tee -a "${LOG_FILE}"); then

package/dist/scripts/night-watch-qa-cron.sh

@@ -593,7 +593,7 @@ Action: generating QA tests and evidence."
   PROVIDER_OK=0
 
   # Build provider command array using generic helper
-  mapfile -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${QA_WORKTREE_DIR}" "${QA_PROMPT}")
+  mapfile -d '' -t PROVIDER_CMD_PARTS < <(build_provider_cmd "${QA_WORKTREE_DIR}" "${QA_PROMPT}")
 
   # Execute — always cd into worktree so provider tools resolve project files correctly
   if (cd "${QA_WORKTREE_DIR}" && timeout "${MAX_RUNTIME}" "${PROVIDER_CMD_PARTS[@]}" 2>&1 | tee -a "${LOG_FILE}"); then

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jonit-dev/night-watch-cli",
-  "version": "1.7.88",
+  "version": "1.7.89",
   "description": "Autonomous PRD execution using AI Provider CLIs + cron",
   "type": "module",
   "bin": {