shipwright-cli 3.0.0 → 3.2.0

package/scripts/postinstall.mjs

@@ -11,8 +11,11 @@ import {
   readFileSync,
   writeFileSync,
   appendFileSync,
+  chmodSync,
+  readdirSync,
 } from "fs";
-import { join } from "path";
+import { join, basename } from "path";
+import { execSync } from "child_process";
 
 const HOME = process.env.HOME || process.env.USERPROFILE;
 const PKG_DIR = join(import.meta.dirname, "..");
@@ -130,6 +133,77 @@ try {
     success("Migrated legacy config (originals preserved)");
   }
 
+  // Set executable bits on all scripts (npm strips them on some platforms)
+  const scriptsDir = join(PKG_DIR, "scripts");
+  if (existsSync(scriptsDir)) {
+    let madeExecutable = 0;
+    for (const file of readdirSync(scriptsDir)) {
+      const fp = join(scriptsDir, file);
+      try {
+        chmodSync(fp, 0o755);
+        madeExecutable++;
+      } catch (_) {
+        // skip non-files
+      }
+    }
+    const libDir = join(scriptsDir, "lib");
+    if (existsSync(libDir)) {
+      for (const file of readdirSync(libDir)) {
+        try {
+          chmodSync(join(libDir, file), 0o755);
+          madeExecutable++;
+        } catch (_) {}
+      }
+    }
+    success(`Set executable bits on ${madeExecutable} scripts`);
+  }
+
+  // Install shell completions for the user's current shell
+  const completionsDir = join(PKG_DIR, "completions");
+  if (existsSync(completionsDir)) {
+    const shell = basename(process.env.SHELL || "/bin/bash");
+    try {
+      if (shell === "bash") {
+        const dest =
+          process.env.BASH_COMPLETION_USER_DIR ||
+          join(
+            process.env.XDG_DATA_HOME || join(HOME, ".local", "share"),
+            "bash-completion",
+            "completions",
+          );
+        ensureDir(dest);
+        cpSync(
+          join(completionsDir, "shipwright.bash"),
+          join(dest, "shipwright"),
+        );
+        cpSync(join(completionsDir, "shipwright.bash"), join(dest, "sw"));
+        success(`Installed bash completions to ${dest}`);
+      } else if (shell === "zsh") {
+        const dest = join(HOME, ".zfunc");
+        ensureDir(dest);
+        cpSync(join(completionsDir, "_shipwright"), join(dest, "_shipwright"));
+        cpSync(join(completionsDir, "_shipwright"), join(dest, "_sw"));
+        success(`Installed zsh completions to ${dest}`);
+      } else if (shell === "fish") {
+        const dest = join(
+          process.env.XDG_CONFIG_HOME || join(HOME, ".config"),
+          "fish",
+          "completions",
+        );
+        ensureDir(dest);
+        cpSync(
+          join(completionsDir, "shipwright.fish"),
+          join(dest, "shipwright.fish"),
+        );
+        cpSync(join(completionsDir, "shipwright.fish"), join(dest, "sw.fish"));
+        success(`Installed fish completions to ${dest}`);
+      }
+    } catch (e) {
+      warn(`Could not auto-install completions: ${e.message}`);
+      info(`Run: shipwright init  (or: bash scripts/install-completions.sh)`);
+    }
+  }
+
   // Print success banner
   console.log();
   console.log(`${GREEN}${BOLD}Shipwright CLI installed!${RESET} Next steps:`);

package/scripts/signals/example-collector.sh

@@ -0,0 +1,36 @@
+#!/usr/bin/env bash
+# ╔═══════════════════════════════════════════════════════════════════════════╗
+# ║  Example External Signal Collector for Shipwright Decision Engine        ║
+# ║  Place custom collectors in scripts/signals/ — they're auto-discovered  ║
+# ╚═══════════════════════════════════════════════════════════════════════════╝
+#
+# Output: one JSON candidate per line (JSONL).
+# Required fields: id, signal, category, title, description, risk_score,
+#                  confidence, dedup_key
+# Optional fields: evidence (object)
+#
+# The decision engine collects output from all scripts/signals/*.sh files,
+# validates each line as JSON, and includes valid candidates in the scoring
+# pipeline.
+#
+# Categories (determines autonomy tier):
+#   auto:    deps_patch, deps_minor, security_patch, test_coverage,
+#            doc_sync, dead_code
+#   propose: refactor_hotspot, architecture_drift, performance_regression,
+#            deps_major, security_critical, recurring_failure, dora_regression
+#   draft:   new_feature, breaking_change, business_logic, api_change,
+#            data_model_change
+#
+# Example: detect a custom condition and emit a candidate
+#
+
+set -euo pipefail
+
+# Example: check if a TODO count exceeds a threshold
+TODO_COUNT=$(grep -r "TODO" --include="*.ts" --include="*.js" --include="*.sh" . 2>/dev/null | wc -l | tr -d ' ' || echo "0")
+
+if [[ "${TODO_COUNT:-0}" -gt 50 ]]; then
+    cat <<EOF
+{"id":"custom-todo-cleanup","signal":"custom","category":"dead_code","title":"Clean up ${TODO_COUNT} TODOs","description":"Codebase has ${TODO_COUNT} TODO comments — consider a cleanup sprint","evidence":{"todo_count":${TODO_COUNT}},"risk_score":15,"confidence":"0.70","dedup_key":"custom:todo:cleanup"}
+EOF
+fi

package/scripts/sw

@@ -5,7 +5,7 @@
 # ╚═══════════════════════════════════════════════════════════════════════════╝
 set -euo pipefail
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 
 # Resolve symlinks (required for npm global install where bin/ symlinks to node_modules/)
 SOURCE="${BASH_SOURCE[0]}"
@@ -193,7 +193,8 @@ route_quality() {
         security-audit|audit) exec "$SCRIPT_DIR/sw-security-audit.sh" "$@" ;;
         testgen)          exec "$SCRIPT_DIR/sw-testgen.sh" "$@" ;;
         hygiene)          exec "$SCRIPT_DIR/sw-hygiene.sh" "$@" ;;
-        help|*) echo "Usage: shipwright quality {code-review|security-audit|testgen|hygiene}"; exit 1 ;;
+        validate|gate)    exec "$SCRIPT_DIR/sw-quality.sh" "$@" ;;
+        help|*) echo "Usage: shipwright quality {code-review|security-audit|testgen|hygiene|validate}"; exit 1 ;;
     esac
 }
 
@@ -530,14 +531,17 @@ main() {
         evidence|ev)
             exec "$SCRIPT_DIR/sw-evidence.sh" "$@"
             ;;
+        decide)
+            exec "$SCRIPT_DIR/sw-decide.sh" "$@"
+            ;;
         otel)
             exec "$SCRIPT_DIR/sw-otel.sh" "$@"
             ;;
         triage)
             exec "$SCRIPT_DIR/sw-triage.sh" "$@"
             ;;
-        quality)
-            exec "$SCRIPT_DIR/sw-quality.sh" "$@"
+        pipeline-composer|composer)
+            exec "$SCRIPT_DIR/sw-pipeline-composer.sh" "$@"
             ;;
         oversight)
             exec "$SCRIPT_DIR/sw-oversight.sh" "$@"

package/scripts/sw-activity.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
 # ─── Cross-platform compatibility ──────────────────────────────────────────
@@ -332,7 +332,6 @@ cmd_stats() {
     local end_time=""
 
     # Read directly to avoid subshell issues
-    echo "DEBUG: Starting read loop..." >&2
     while IFS= read -r line; do
         [ -z "$line" ] && continue
 
@@ -363,14 +362,9 @@ cmd_stats() {
         end_time="$ts"
     done < <(grep -v '^$' "$EVENTS_FILE" 2>/dev/null)
 
-    echo "DEBUG: Read complete, total=$total_events" >&2
-    echo "DEBUG: agents_seen length: ${#agents_seen}" >&2
     local unique_agents
-    echo "DEBUG: About to compute unique..." >&2
     unique_agents=$(sort -u <<< "$agents_seen" | grep -v '^$' | wc -l | tr -d ' ')
-    echo "DEBUG: unique_agents=$unique_agents" >&2
 
-    echo "DEBUG: About to print results..." >&2
     printf "${BOLD}Total Events:${RESET} %d\n" "$total_events"
     printf "${BOLD}Commits:${RESET} %d\n" "$commits"
     printf "${BOLD}Tests:${RESET} %d\n" "$tests"

package/scripts/sw-adaptive.sh

@@ -1,12 +1,12 @@
 #!/usr/bin/env bash
 # ╔═══════════════════════════════════════════════════════════════════════════╗
 # ║  shipwright adaptive — data-driven pipeline tuning                       ║
-# ║  Replace 83+ hardcoded values with learned defaults from historical runs  ║
+# ║  Replace static defaults with learned values from historical runs         ║
 # ╚═══════════════════════════════════════════════════════════════════════════╝
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
 # ─── Cross-platform compatibility ──────────────────────────────────────────
@@ -712,10 +712,10 @@ cmd_compare() {
         esac
     done
 
-    info "Learned vs Hardcoded Values for ${CYAN}${repo}${RESET}"
+    info "Learned vs Default Values for ${CYAN}${repo}${RESET}"
     echo ""
 
-    printf "%-25s %-15s %-15s %-15s\n" "Metric" "Hardcoded" "Learned" "Difference"
+    printf "%-25s %-15s %-15s %-15s\n" "Metric" "Default" "Learned" "Difference"
     printf "%s\n" "$(printf '%.0s─' {1..70})"
 
     # Timeout
@@ -841,7 +841,7 @@ ${BOLD}USAGE${RESET}
 
 ${BOLD}SUBCOMMANDS${RESET}
   ${CYAN}get${RESET} <metric> [--stage S] [--repo R] [--complexity C] [--default V]
-    Return adaptive value for a metric (replaces hardcoded defaults)
+    Return adaptive value for a metric (replaces static defaults)
     Metrics: timeout, iterations, model, team_size, template, poll_interval,
              retry_limit, quality_threshold, coverage_min
 
@@ -852,7 +852,7 @@ ${BOLD}SUBCOMMANDS${RESET}
     Rebuild models from events.jsonl (run after significant pipeline activity)
 
   ${CYAN}compare${RESET} [--repo REPO]
-    Side-by-side table: learned vs hardcoded values
+    Side-by-side table: learned vs default values
 
   ${CYAN}recommend${RESET} --issue N [--repo REPO]
     Full JSON recommendation for an issue (template, model, team_size, etc.)
@@ -876,7 +876,7 @@ ${BOLD}EXAMPLES${RESET}
   # Get complete recommendation for issue #42
   sw adaptive recommend --issue 42
 
-  # Compare learned vs hardcoded
+  # Compare learned vs defaults
   sw adaptive compare
 
 ${BOLD}STORAGE${RESET}

package/scripts/sw-adversarial.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 

package/scripts/sw-architecture-enforcer.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 

package/scripts/sw-auth.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 

package/scripts/sw-autonomous.sh

@@ -7,7 +7,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 

package/scripts/sw-changelog.sh

@@ -7,7 +7,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 

package/scripts/sw-checkpoint.sh

@@ -8,7 +8,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]:-$0}")" && pwd)"
 
 # ─── Cross-platform compatibility ──────────────────────────────────────────

package/scripts/sw-ci.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 
@@ -227,15 +227,20 @@ cmd_analyze() {
     info "Analyzing workflow efficiency"
 
     local job_count step_count matrix_enabled
-    job_count=$(grep -c "^  [a-z_-]*:" "$workflow_file" 2>/dev/null || echo "0")
-    step_count=$(grep -c "      - name:" "$workflow_file" 2>/dev/null || echo "0")
-    matrix_enabled=$(grep -c "matrix:" "$workflow_file" 2>/dev/null || echo "0")
+    job_count=$(grep -c "^  [a-z_-]*:" "$workflow_file" 2>/dev/null || true)
+    job_count="${job_count:-0}"
+    step_count=$(grep -c "      - name:" "$workflow_file" 2>/dev/null || true)
+    step_count="${step_count:-0}"
+    matrix_enabled=$(grep -c "matrix:" "$workflow_file" 2>/dev/null || true)
+    matrix_enabled="${matrix_enabled:-0}"
 
     local has_cache
-    has_cache=$(grep -c "actions/cache" "$workflow_file" 2>/dev/null || echo "0")
+    has_cache=$(grep -c "actions/cache" "$workflow_file" 2>/dev/null || true)
+    has_cache="${has_cache:-0}"
 
     local has_timeout
-    has_timeout=$(grep -c "timeout-minutes:" "$workflow_file" 2>/dev/null || echo "0")
+    has_timeout=$(grep -c "timeout-minutes:" "$workflow_file" 2>/dev/null || true)
+    has_timeout="${has_timeout:-0}"
 
     echo ""
     echo -e "${BOLD}Workflow Analysis: $(basename "$workflow_file")${RESET}"

package/scripts/sw-cleanup.sh

@@ -5,7 +5,7 @@
 # ║  Default: dry-run (shows what would be cleaned).                         ║
 # ║  Use --force to actually kill sessions and remove files.                 ║
 # ╚═══════════════════════════════════════════════════════════════════════════╝
-VERSION="3.0.0"
+VERSION="3.2.0"
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 

package/scripts/sw-code-review.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 
@@ -72,7 +72,8 @@ detect_code_smells() {
 
     # Check 1: Long functions (>60 lines in bash)
     local func_count
-    func_count=$(grep -c "^[a-zA-Z_][a-zA-Z0-9_]*().*{" "$target_file" 2>/dev/null || echo "0")
+    func_count=$(grep -c "^[a-zA-Z_][a-zA-Z0-9_]*().*{" "$target_file" 2>/dev/null || true)
+    func_count="${func_count:-0}"
     if [[ "$func_count" -gt 0 ]]; then
         while IFS= read -r line; do
             if [[ "$line" =~ ^[a-zA-Z_][a-zA-Z0-9_]*\(\).*\{ ]]; then
@@ -106,7 +107,8 @@ detect_code_smells() {
 
     # Check 3: Duplicate code patterns (repeated >3 times)
     local dup_count=0
-    dup_count=$(grep -c '^\s*\(cd\|cd\|mkdir\|rm\|echo\)' "$target_file" 2>/dev/null || echo "0")
+    dup_count=$(grep -c '^\s*\(cd\|cd\|mkdir\|rm\|echo\)' "$target_file" 2>/dev/null || true)
+    dup_count="${dup_count:-0}"
     if [[ $dup_count -gt 3 ]]; then
         issues+=("REPEATED_PATTERNS: Common operations appear $dup_count times (consider helper functions)")
     fi
@@ -139,7 +141,8 @@ check_solid_principles() {
 
     # Single Responsibility: Check if scripts do multiple unrelated things
     local sourced_count
-    sourced_count=$(grep -c '^\s*source\|^\s*\.\s' "$target_file" 2>/dev/null || echo "0")
+    sourced_count=$(grep -c '^\s*source\|^\s*\.\s' "$target_file" 2>/dev/null || true)
+    sourced_count="${sourced_count:-0}"
     if [[ $sourced_count -gt 3 ]]; then
         violations+=("SRP_VIOLATION: Script sources $sourced_count modules (too many responsibilities)")
     fi
@@ -159,7 +162,8 @@ check_solid_principles() {
         if [[ "$line" =~ ^[a-zA-Z_][a-zA-Z0-9_]*\(\) ]]; then
             local func_name="${line%%(*}"
             local body
-            body=$(awk "/^${func_name}\\(\\)/,/^}/" "$target_file" | grep -c '\$[0-9]' || echo "0")
+            body=$(awk "/^${func_name}\\(\\)/,/^}/" "$target_file" | grep -c '\$[0-9]' || true)
+            body="${body:-0}"
             if [[ $body -gt 5 ]]; then
                 violations+=("ISP_VIOLATION: Function $func_name uses >5 parameters")
             fi
@@ -227,7 +231,9 @@ analyze_complexity() {
 
             # Cyclomatic complexity: count decision points (if, elif, case, &&, ||)
             local cc=1
-            cc=$((cc + $(sed -n "${start_line},${end_line}p" "$target_file" | grep -cE '\s(if|elif|case|&&|\|\|)' || echo 0)))
+            local _cc_count
+            _cc_count=$(sed -n "${start_line},${end_line}p" "$target_file" | grep -cE '\s(if|elif|case|&&|\|\|)' || true)
+            cc=$((cc + ${_cc_count:-0}))
 
             # Lines of code
             local loc=$((end_line - start_line))
@@ -266,8 +272,12 @@ check_style_consistency() {
     local has_trap=false
     local has_set_e=false
 
-    [[ $(grep -c 'trap.*ERR' "$target_file" 2>/dev/null || echo 0) -gt 0 ]] && has_trap=true
-    [[ $(grep -c 'set -e' "$target_file" 2>/dev/null || echo 0) -gt 0 ]] && has_set_e=true
+    local _trap_count
+    _trap_count=$(grep -c 'trap.*ERR' "$target_file" 2>/dev/null || true)
+    [[ "${_trap_count:-0}" -gt 0 ]] && has_trap=true
+    local _set_e_count
+    _set_e_count=$(grep -c 'set -e' "$target_file" 2>/dev/null || true)
+    [[ "${_set_e_count:-0}" -gt 0 ]] && has_set_e=true
 
     if [[ "$has_set_e" == "true" ]] && [[ "$has_trap" == "false" ]]; then
         issues+=("STYLE: Missing ERR trap despite 'set -e' (inconsistent error handling)")
@@ -276,8 +286,10 @@ check_style_consistency() {
     # Check for inconsistent quote usage
     local single_quotes
     local double_quotes
-    single_quotes=$(grep -o "'" "$target_file" 2>/dev/null | wc -l || echo 0)
-    double_quotes=$(grep -o '"' "$target_file" 2>/dev/null | wc -l || echo 0)
+    single_quotes=$(grep -o "'" "$target_file" 2>/dev/null | wc -l || true)
+    single_quotes="${single_quotes:-0}"
+    double_quotes=$(grep -o '"' "$target_file" 2>/dev/null | wc -l || true)
+    double_quotes="${double_quotes:-0}"
     if [[ $single_quotes -gt $((double_quotes * 3)) ]] || [[ $double_quotes -gt $((single_quotes * 3)) ]]; then
         issues+=("STYLE: Inconsistent quote style (mix of single and double quotes)")
     fi
@@ -285,8 +297,10 @@ check_style_consistency() {
     # Check for inconsistent spacing/indentation
     local tab_count
     local space_count
-    tab_count=$(grep -c $'^\t' "$target_file" 2>/dev/null || echo 0)
-    space_count=$(grep -c '^  ' "$target_file" 2>/dev/null || echo 0)
+    tab_count=$(grep -c $'^\t' "$target_file" 2>/dev/null || true)
+    tab_count="${tab_count:-0}"
+    space_count=$(grep -c '^  ' "$target_file" 2>/dev/null || true)
+    space_count="${space_count:-0}"
     if [[ $tab_count -gt 0 ]] && [[ $space_count -gt 0 ]]; then
         issues+=("STYLE: Mixed tabs and spaces")
     fi
@@ -333,7 +347,8 @@ auto_fix() {
 
     # Fix 2: Trailing whitespace
     local trailing_ws
-    trailing_ws=$(grep -c '[[:space:]]$' "$target_file" 2>/dev/null || echo "0")
+    trailing_ws=$(grep -c '[[:space:]]$' "$target_file" 2>/dev/null || true)
+    trailing_ws="${trailing_ws:-0}"
     if [[ $trailing_ws -gt 0 ]]; then
         sed -i '' 's/[[:space:]]*$//' "$target_file"
         info "Removed $trailing_ws lines of trailing whitespace"
@@ -349,7 +364,8 @@ auto_fix() {
 
     # Fix 4: Consistent spacing around operators (simple cases)
     local spacing_fixes=0
-    spacing_fixes=$(grep -c '==' "$target_file" 2>/dev/null || echo "0")
+    spacing_fixes=$(grep -c '==' "$target_file" 2>/dev/null || true)
+    spacing_fixes="${spacing_fixes:-0}"
     if [[ $spacing_fixes -gt 0 ]]; then
         info "Flagged $spacing_fixes operator spacing cases (manual review recommended)"
     fi
@@ -507,9 +523,12 @@ scan_codebase() {
         local solids=0
         local arch_issues=0
 
-        smells=$(detect_code_smells "$file" 2>/dev/null | wc -l || echo 0)
-        solids=$(check_solid_principles "$file" 2>/dev/null | wc -l || echo 0)
-        arch_issues=$(check_architecture_boundaries "$file" 2>/dev/null | wc -l || echo 0)
+        smells=$(detect_code_smells "$file" 2>/dev/null | wc -l || true)
+        smells="${smells:-0}"
+        solids=$(check_solid_principles "$file" 2>/dev/null | wc -l || true)
+        solids="${solids:-0}"
+        arch_issues=$(check_architecture_boundaries "$file" 2>/dev/null | wc -l || true)
+        arch_issues="${arch_issues:-0}"
 
         local file_issues=$((smells + solids + arch_issues))
         total_issues=$((total_issues + file_issues))

package/scripts/sw-connect.sh

@@ -8,7 +8,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
 # ─── Cross-platform compatibility ──────────────────────────────────────────

package/scripts/sw-context.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="${SHIPWRIGHT_REPO_DIR:-$(cd "$SCRIPT_DIR/.." && pwd)}"
 

package/scripts/sw-cost.sh

@@ -6,7 +6,7 @@
 set -euo pipefail
 trap 'echo "ERROR: $BASH_SOURCE:$LINENO exited with status $?" >&2' ERR
 
-VERSION="3.0.0"
+VERSION="3.2.0"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 
@@ -142,9 +142,9 @@ cost_record() {
     local cost_usd
     cost_usd=$(cost_calculate "$input_tokens" "$output_tokens" "$model")
 
-    # Try SQLite first
+    # Try SQLite first (arg order must match db_record_cost signature: tokens, tokens, model, cost, stage, issue)
     if type db_record_cost >/dev/null 2>&1; then
-        db_record_cost "$input_tokens" "$output_tokens" "$model" "$stage" "$cost_usd" "$issue" 2>/dev/null || true
+        db_record_cost "$input_tokens" "$output_tokens" "$model" "$cost_usd" "$stage" "$issue" 2>/dev/null || true
     fi
 
     # Always write to JSON (dual-write period)
@@ -154,7 +154,7 @@ cost_record() {
         fi
         local tmp_file
         tmp_file=$(mktemp "${COST_FILE}.tmp.XXXXXX")
-        jq --argjson input "$input_tokens" \
+        if ! jq --argjson input "$input_tokens" \
            --argjson output "$output_tokens" \
            --arg model "$model" \
            --arg stage "$stage" \
@@ -172,7 +172,16 @@ cost_record() {
                ts: $ts,
                ts_epoch: $epoch
            }] | .entries = (.entries | .[-1000:])' \
-           "$COST_FILE" > "$tmp_file" && mv "$tmp_file" "$COST_FILE" || rm -f "$tmp_file"
+           "$COST_FILE" > "$tmp_file" 2>/dev/null; then
+            error "Cost jq transformation failed — entry may be lost"
+            rm -f "$tmp_file"
+            # Continue without updating cost file
+        else
+            mv "$tmp_file" "$COST_FILE" || {
+                error "Failed to update cost file"
+                rm -f "$tmp_file"
+            }
+        fi
     ) 200>"${COST_FILE}.lock"
 
     emit_event "cost.record" \
@@ -264,6 +273,11 @@ cost_remaining_budget() {
     budget_usd=$(jq -r '.daily_budget_usd' "$BUDGET_FILE" 2>/dev/null || echo "0")
 
     if [[ "$budget_enabled" != "true" || "$budget_usd" == "0" ]]; then
+        if [[ -z "${_BUDGET_UNCONFIGURED_WARNED:-}" ]]; then
+            info "Budget not configured — unlimited. Use 'shipwright cost budget set <amount>'"
+            emit_event "cost.budget_unconfigured" "status=unlimited"
+            _BUDGET_UNCONFIGURED_WARNED=1
+        fi
         echo "unlimited"
         return 0
     fi
@@ -785,6 +799,54 @@ cost_dashboard() {
         fi
     fi
 
+    # Context efficiency (from loop.context_efficiency events)
+    local events_file="${HOME}/.shipwright/events.jsonl"
+    if [[ -f "$events_file" ]]; then
+        local ctx_events
+        ctx_events=$(grep '"type":"loop.context_efficiency"' "$events_file" 2>/dev/null | tail -200 || true)
+        local ctx_count
+        ctx_count=$(echo "$ctx_events" | grep -c '"loop.context_efficiency"' 2>/dev/null || echo "0")
+        ctx_count="${ctx_count:-0}"
+
+        if [[ "$ctx_count" -gt 0 ]]; then
+            # Parse metrics using awk (Bash 3.2 safe — no arrays needed)
+            local avg_utilization avg_trim_ratio total_raw total_trimmed trim_count
+            eval "$(echo "$ctx_events" | awk -F'"' '
+                /"loop.context_efficiency"/ {
+                    for (i=1; i<=NF; i++) {
+                        if ($i == "budget_utilization") util = $(i+2)
+                        if ($i == "trim_ratio") ratio = $(i+2)
+                        if ($i == "raw_prompt_chars") raw = $(i+2)
+                        if ($i == "trimmed_prompt_chars") trimmed = $(i+2)
+                    }
+                    total_util += util; total_ratio += ratio
+                    total_raw += raw; total_trimmed += trimmed
+                    if (ratio + 0 > 0) trim_events++
+                    n++
+                }
+                END {
+                    if (n > 0) {
+                        printf "avg_utilization=%.1f\n", total_util / n
+                        printf "avg_trim_ratio=%.1f\n", total_ratio / n
+                    } else {
+                        printf "avg_utilization=0\navg_trim_ratio=0\n"
+                    }
+                    printf "total_raw=%d\ntotal_trimmed=%d\ntrim_count=%d\n", total_raw, total_trimmed, (trim_events+0)
+                }
+            ')"
+
+            local total_discarded=$(( total_raw - total_trimmed ))
+
+            echo -e "${BOLD}  CONTEXT EFFICIENCY${RESET}"
+            echo -e "    Avg budget used     ${avg_utilization}%"
+            echo -e "    Avg trim ratio      ${avg_trim_ratio}%"
+            echo -e "    Chars generated     $(printf "%'d" "$total_raw")"
+            echo -e "    Chars discarded     $(printf "%'d" "$total_discarded")"
+            echo -e "    Trim events         ${trim_count} / ${ctx_count} iterations"
+            echo ""
+        fi
+    fi
+
     echo -e "${PURPLE}${BOLD}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${RESET}"
     echo ""
 }
@@ -894,6 +956,8 @@ show_help() {
 }
 
 # ─── Command Router ─────────────────────────────────────────────────────────
+# Only run CLI when executed directly (not when sourced by sw-pipeline.sh)
+if [[ "${BASH_SOURCE[0]}" == "$0" ]]; then
 
 SUBCOMMAND="${1:-help}"
 shift 2>/dev/null || true
@@ -943,3 +1007,5 @@ case "$SUBCOMMAND" in
         exit 1
         ;;
 esac
+
+fi  # end source guard