@nklisch/pi-agile-workflow 0.15.3

package/scripts/tests/bump-version.test.sh

@@ -0,0 +1,552 @@
+#!/usr/bin/env bash
+# bump-version.test.sh — unit tests for channel-metadata lockstep and the
+# version projection blocks in scripts/bump-version.sh (repo-root script).
+#
+# The load-bearing mechanisms under test are the two projection blocks:
+#
+# agile-workflow / work-view block:
+#   * the Rust stamp `work-view/crates/cli/.work-view-version` (NO trailing newline)
+#   * the bash fallback's `WORK_VIEW_VERSION="x.y.z"` literal in scripts/work-view.sh
+#
+# agentic-research / research-view block:
+#   * the Rust stamp `research-view/crates/cli/.research-view-version` (NO trailing newline)
+#   * the bash fallback's `RESEARCH_VIEW_VERSION="x.y.z"` literal in scripts/research-view.sh
+#
+# Both blocks use a Fail-Fast `grep` postcondition that exits 1 if the anchored
+# sed pattern no longer matches the literal.
+#
+# We run the REAL bump-version.sh (not a reimplementation) inside a throwaway
+# `git init` scratch repo whose tree mirrors the paths the script touches. To
+# stop the script's trailing `git commit` / `git push` from escaping, a fake
+# `git` shim is placed first on PATH: it forwards read/stage subcommands
+# (init/config/add/diff/status/rev-parse) to the real git inside the scratch
+# repo but turns `commit` and `push` into no-ops. This still exercises the real
+# projection AND the real `git add` staging.
+#
+# Runnable locally and from CI. Exits non-zero if any assertion fails.
+
+set -uo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+# scripts/tests/ -> scripts/ -> agile-workflow/ -> plugins/ -> repo root
+REPO_ROOT="$(cd "${SCRIPT_DIR}/../../../.." && pwd)"
+BUMP="${REPO_ROOT}/scripts/bump-version.sh"
+REAL_GIT="$(command -v git)"
+
+# ---------------------------------------------------------------------------
+# Counters
+# ---------------------------------------------------------------------------
+PASS=0
+FAIL=0
+ERRORS=()
+
+assert_eq() {
+  local label="$1" expected="$2" actual="$3"
+  if [ "$expected" = "$actual" ]; then
+    echo "  PASS: $label"
+    ((PASS++))
+  else
+    echo "  FAIL: $label"
+    echo "        expected: $(printf '%q' "$expected")"
+    echo "        actual:   $(printf '%q' "$actual")"
+    ((FAIL++))
+    ERRORS+=("$label")
+  fi
+}
+
+assert_true() {
+  local label="$1"
+  if eval "$2"; then
+    echo "  PASS: $label"
+    ((PASS++))
+  else
+    echo "  FAIL: $label"
+    echo "        condition false: $2"
+    ((FAIL++))
+    ERRORS+=("$label")
+  fi
+}
+
+assert_false() {
+  local label="$1"
+  if ! eval "$2"; then
+    echo "  PASS: $label"
+    ((PASS++))
+  else
+    echo "  FAIL: $label"
+    echo "        expected false but was true: $2"
+    ((FAIL++))
+    ERRORS+=("$label")
+  fi
+}
+
+# ---------------------------------------------------------------------------
+# Preflight
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Preflight: bump-version.sh present and jq available ==="
+assert_true "bump-version.sh exists" "[ -f '$BUMP' ]"
+if ! command -v jq >/dev/null 2>&1; then
+  echo "ERROR: jq is required to run bump-version.sh and these tests" >&2
+  exit 1
+fi
+if [ ! -f "$BUMP" ]; then
+  echo "ERROR: cannot locate bump-version.sh at $BUMP" >&2
+  exit 1
+fi
+
+# ---------------------------------------------------------------------------
+# make_fake_git_dir — produce a dir holding a `git` shim first on PATH.
+# Forwards safe subcommands to the real git; no-ops commit/push so the script
+# under test cannot reach a remote or write a published commit.
+# ---------------------------------------------------------------------------
+make_fake_git_dir() {
+  local dir
+  dir="$(mktemp -d)"
+  cat > "${dir}/git" <<EOF
+#!/usr/bin/env bash
+sub="\${1:-}"
+case "\$sub" in
+  commit|push)
+    # Swallow — these would escape the scratch repo / publish a bump commit.
+    exit 0
+    ;;
+  *)
+    exec "${REAL_GIT}" "\$@"
+    ;;
+esac
+EOF
+  chmod +x "${dir}/git"
+  printf '%s' "$dir"
+}
+
+# ---------------------------------------------------------------------------
+# new_scratch_repo — build a throwaway repo mirroring the paths bump-version.sh
+# touches, with a clean working tree (the script refuses a dirty plugin dir).
+#   $1 = plugin.json version to use for agile-workflow (and agentic-research)
+#   $2 = optional extra plugin name to seed (for the non-agile-workflow guard)
+# Echoes the repo path.
+# ---------------------------------------------------------------------------
+new_scratch_repo() {
+  local aw_version="$1" extra_plugin="${2:-}"
+  local repo
+  repo="$(mktemp -d)"
+
+  # agile-workflow plugin manifests + work-view artifacts.
+  mkdir -p \
+    "${repo}/plugins/agile-workflow/.claude-plugin" \
+    "${repo}/plugins/agile-workflow/.codex-plugin" \
+    "${repo}/plugins/agile-workflow/work-view/crates/cli" \
+    "${repo}/plugins/agile-workflow/scripts"
+
+  printf '{\n  "name": "agile-workflow",\n  "version": "%s"\n}\n' "$aw_version" \
+    > "${repo}/plugins/agile-workflow/.claude-plugin/plugin.json"
+  printf '{\n  "name": "agile-workflow",\n  "version": "%s"\n}\n' "$aw_version" \
+    > "${repo}/plugins/agile-workflow/.codex-plugin/plugin.json"
+  printf '{\n  "name": "@nklisch/pi-agile-workflow",\n  "version": "%s",\n  "keywords": ["pi-package"],\n  "pi": { "skills": ["./skills"] }\n}\n' "$aw_version" \
+    > "${repo}/plugins/agile-workflow/package.json"
+
+  # Rust stamp — start with a placeholder, no trailing newline (mirrors prod).
+  printf '%s' "$aw_version" \
+    > "${repo}/plugins/agile-workflow/work-view/crates/cli/.work-view-version"
+
+  # Bash fallback with the anchored WORK_VIEW_VERSION literal.
+  cat > "${repo}/plugins/agile-workflow/scripts/work-view.sh" <<EOF
+#!/usr/bin/env bash
+WORK_VIEW_VERSION="${aw_version}"
+echo "work-view \${WORK_VIEW_VERSION}"
+EOF
+
+  # agentic-research plugin manifests + research-view artifacts.
+  mkdir -p \
+    "${repo}/plugins/agentic-research/.claude-plugin" \
+    "${repo}/plugins/agentic-research/.codex-plugin" \
+    "${repo}/plugins/agentic-research/research-view/crates/cli" \
+    "${repo}/plugins/agentic-research/scripts"
+
+  printf '{\n  "name": "agentic-research",\n  "version": "%s"\n}\n' "$aw_version" \
+    > "${repo}/plugins/agentic-research/.claude-plugin/plugin.json"
+  printf '{\n  "name": "agentic-research",\n  "version": "%s"\n}\n' "$aw_version" \
+    > "${repo}/plugins/agentic-research/.codex-plugin/plugin.json"
+  printf '{\n  "name": "@nklisch/pi-agentic-research",\n  "version": "%s",\n  "keywords": ["pi-package"],\n  "pi": { "skills": ["./skills"] }\n}\n' "$aw_version" \
+    > "${repo}/plugins/agentic-research/package.json"
+
+  # Rust stamp — no trailing newline.
+  printf '%s' "$aw_version" \
+    > "${repo}/plugins/agentic-research/research-view/crates/cli/.research-view-version"
+
+  # Bash fallback with the anchored RESEARCH_VIEW_VERSION literal.
+  cat > "${repo}/plugins/agentic-research/scripts/research-view.sh" <<EOF
+#!/usr/bin/env bash
+RESEARCH_VIEW_VERSION="${aw_version}"
+echo "research-view \${RESEARCH_VIEW_VERSION}"
+EOF
+
+  if [ -n "$extra_plugin" ]; then
+    mkdir -p "${repo}/plugins/${extra_plugin}/.claude-plugin" \
+             "${repo}/plugins/${extra_plugin}/.codex-plugin"
+    printf '{\n  "name": "%s",\n  "version": "%s"\n}\n' "$extra_plugin" "$aw_version" \
+      > "${repo}/plugins/${extra_plugin}/.claude-plugin/plugin.json"
+    printf '{\n  "name": "%s",\n  "version": "%s"\n}\n' "$extra_plugin" "$aw_version" \
+      > "${repo}/plugins/${extra_plugin}/.codex-plugin/plugin.json"
+    printf '{\n  "name": "@nklisch/pi-%s",\n  "version": "%s",\n  "keywords": ["pi-package"],\n  "pi": { "skills": ["./skills"] }\n}\n' "$extra_plugin" "$aw_version" \
+      > "${repo}/plugins/${extra_plugin}/package.json"
+  fi
+
+  # Initialize a clean git repo (real git; commit happens via real git here,
+  # only the script-under-test's commit/push are shimmed away later).
+  (
+    cd "$repo"
+    "$REAL_GIT" init -q
+    "$REAL_GIT" config user.email test@example.com
+    "$REAL_GIT" config user.name "Test"
+    "$REAL_GIT" config commit.gpgsign false
+    "$REAL_GIT" add -A
+    "$REAL_GIT" commit -q -m "seed" >/dev/null 2>&1
+  )
+  printf '%s' "$repo"
+}
+
+# run_bump <repo> <plugin> <bump> — run the real script with the fake git shim
+# first on PATH. Captures stdout/stderr/rc.
+run_bump() {
+  local repo="$1" plugin="$2" bump="$3"
+  local fakedir
+  fakedir="$(make_fake_git_dir)"
+  BUMP_ERR="${repo}/.bump.err"
+  BUMP_OUT=$(
+    cd "$repo" &&
+    PATH="${fakedir}:${PATH}" bash "$BUMP" "$plugin" "$bump" 2>"$BUMP_ERR"
+  )
+  BUMP_RC=$?
+  rm -rf "$fakedir"
+}
+
+# staged? <repo> <relpath> — true if the path is staged in the repo index.
+is_staged() {
+  local repo="$1" rel="$2"
+  ( cd "$repo" && "$REAL_GIT" diff --cached --name-only -- "$rel" | grep -qx "$rel" )
+}
+
+VER_REL="plugins/agile-workflow/work-view/crates/cli/.work-view-version"
+WV_REL="plugins/agile-workflow/scripts/work-view.sh"
+
+RV_VER_REL="plugins/agentic-research/research-view/crates/cli/.research-view-version"
+RV_SH_REL="plugins/agentic-research/scripts/research-view.sh"
+
+# ---------------------------------------------------------------------------
+# Test group 1: patch bump projects new semver into BOTH implementations
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 1: agile-workflow patch bump projects lockstep ==="
+
+REPO1="$(new_scratch_repo "1.2.3")"
+run_bump "$REPO1" agile-workflow patch
+
+assert_eq "patch bump exits 0" "0" "$BUMP_RC"
+
+# .work-view-version equals the new semver with NO trailing newline.
+VER_CONTENT="$(cat "${REPO1}/${VER_REL}")"
+assert_eq ".work-view-version content == new semver" "1.2.4" "$VER_CONTENT"
+VER_BYTES="$(wc -c < "${REPO1}/${VER_REL}" | tr -d ' ')"
+assert_eq ".work-view-version is exactly 5 bytes (no trailing newline)" "5" "$VER_BYTES"
+LAST_BYTE="$(tail -c1 "${REPO1}/${VER_REL}" | od -An -tx1 | tr -d ' \n')"
+assert_false ".work-view-version last byte is not a newline (0a)" "[ '$LAST_BYTE' = '0a' ]"
+
+# work-view.sh WORK_VIEW_VERSION literal equals the new semver.
+assert_true "work-view.sh WORK_VIEW_VERSION literal == new" \
+  "grep -q '^WORK_VIEW_VERSION=\"1.2.4\"\$' '${REPO1}/${WV_REL}'"
+assert_false "old WORK_VIEW_VERSION literal is gone" \
+  "grep -q '^WORK_VIEW_VERSION=\"1.2.3\"\$' '${REPO1}/${WV_REL}'"
+
+# plugin.json manifests and package metadata advanced too (sanity that this was
+# a real bump).
+assert_eq "claude plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO1}/plugins/agile-workflow/.claude-plugin/plugin.json")"
+assert_eq "codex plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO1}/plugins/agile-workflow/.codex-plugin/plugin.json")"
+assert_eq "package.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO1}/plugins/agile-workflow/package.json")"
+
+# Both work-view files are git-staged.
+assert_true "claude plugin.json is staged" \
+  "is_staged '$REPO1' 'plugins/agile-workflow/.claude-plugin/plugin.json'"
+assert_true "codex plugin.json is staged" \
+  "is_staged '$REPO1' 'plugins/agile-workflow/.codex-plugin/plugin.json'"
+assert_true ".work-view-version is staged" "is_staged '$REPO1' '$VER_REL'"
+assert_true "work-view.sh is staged" "is_staged '$REPO1' '$WV_REL'"
+assert_true "agile-workflow package.json is staged" \
+  "is_staged '$REPO1' 'plugins/agile-workflow/package.json'"
+
+# agile-workflow bump must NOT touch research-view artifacts.
+assert_eq "aw-bump: .research-view-version unchanged" "1.2.3" \
+  "$(cat "${REPO1}/${RV_VER_REL}")"
+assert_true "aw-bump: research-view.sh RESEARCH_VIEW_VERSION unchanged" \
+  "grep -q '^RESEARCH_VIEW_VERSION=\"1.2.3\"\$' '${REPO1}/${RV_SH_REL}'"
+assert_false "aw-bump: .research-view-version NOT staged" \
+  "is_staged '$REPO1' '$RV_VER_REL'"
+assert_false "aw-bump: research-view.sh NOT staged" \
+  "is_staged '$REPO1' '$RV_SH_REL'"
+
+rm -rf "$REPO1"
+
+# ---------------------------------------------------------------------------
+# Test group 2: bumping a NON-agile-workflow plugin leaves work-view untouched
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 2: non-agile-workflow plugin does not touch work-view ==="
+
+REPO2="$(new_scratch_repo "1.2.3" "ux-ui-design")"
+VER_BEFORE="$(cat "${REPO2}/${VER_REL}")"
+WV_BEFORE="$(cat "${REPO2}/${WV_REL}")"
+run_bump "$REPO2" ux-ui-design patch
+
+assert_eq "non-aw bump exits 0" "0" "$BUMP_RC"
+assert_eq "non-aw plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO2}/plugins/ux-ui-design/.claude-plugin/plugin.json")"
+assert_eq "non-aw package.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO2}/plugins/ux-ui-design/package.json")"
+assert_eq ".work-view-version unchanged after non-aw bump" "$VER_BEFORE" \
+  "$(cat "${REPO2}/${VER_REL}")"
+assert_eq "work-view.sh unchanged after non-aw bump" "$WV_BEFORE" \
+  "$(cat "${REPO2}/${WV_REL}")"
+assert_false ".work-view-version NOT staged after non-aw bump" "is_staged '$REPO2' '$VER_REL'"
+assert_false "work-view.sh NOT staged after non-aw bump" "is_staged '$REPO2' '$WV_REL'"
+assert_true "non-aw claude plugin.json is staged" \
+  "is_staged '$REPO2' 'plugins/ux-ui-design/.claude-plugin/plugin.json'"
+assert_true "non-aw codex plugin.json is staged" \
+  "is_staged '$REPO2' 'plugins/ux-ui-design/.codex-plugin/plugin.json'"
+assert_true "non-aw package.json is staged" \
+  "is_staged '$REPO2' 'plugins/ux-ui-design/package.json'"
+
+rm -rf "$REPO2"
+
+# ---------------------------------------------------------------------------
+# Test group 3: Fail-Fast postcondition fires on a renamed/indented literal
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 3: postcondition exits 1 when literal anchor no longer matches ==="
+
+REPO3="$(new_scratch_repo "1.2.3")"
+# Break the anchored literal: indent it so `^WORK_VIEW_VERSION="..."` (and the
+# anchored sed substitution) no longer matches. sed exits 0 (no match), so the
+# projection silently no-ops and the grep postcondition must catch it.
+cat > "${REPO3}/${WV_REL}" <<'EOF'
+#!/usr/bin/env bash
+  WORK_VIEW_VERSION="1.2.3"
+echo "work-view ${WORK_VIEW_VERSION}"
+EOF
+( cd "$REPO3" && "$REAL_GIT" add -A && "$REAL_GIT" commit -q -m "indent literal" >/dev/null 2>&1 )
+
+run_bump "$REPO3" agile-workflow patch
+
+assert_eq "indented-literal bump exits 1 (postcondition fires)" "1" "$BUMP_RC"
+POST_ERR="$(cat "$BUMP_ERR" 2>/dev/null || true)"
+assert_true "postcondition reports failed projection" \
+  "printf '%s' \"\$POST_ERR\" | grep -q 'failed to project version into'"
+# The indented literal was not rewritten (anchored pattern never matched).
+assert_false "indented literal NOT rewritten to new semver" \
+  "grep -q 'WORK_VIEW_VERSION=\"1.2.4\"' '${REPO3}/${WV_REL}'"
+
+rm -rf "$REPO3"
+
+# ---------------------------------------------------------------------------
+# Test group 4: minor/major bumps project correctly too
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 4: minor and major bumps project lockstep ==="
+
+REPO4="$(new_scratch_repo "1.2.3")"
+run_bump "$REPO4" agile-workflow minor
+assert_eq "minor bump exits 0" "0" "$BUMP_RC"
+assert_eq "minor .work-view-version == 1.3.0" "1.3.0" "$(cat "${REPO4}/${VER_REL}")"
+assert_true "minor work-view.sh literal == 1.3.0" \
+  "grep -q '^WORK_VIEW_VERSION=\"1.3.0\"\$' '${REPO4}/${WV_REL}'"
+rm -rf "$REPO4"
+
+REPO5="$(new_scratch_repo "1.2.3")"
+run_bump "$REPO5" agile-workflow major
+assert_eq "major bump exits 0" "0" "$BUMP_RC"
+assert_eq "major .work-view-version == 2.0.0" "2.0.0" "$(cat "${REPO5}/${VER_REL}")"
+assert_true "major work-view.sh literal == 2.0.0" \
+  "grep -q '^WORK_VIEW_VERSION=\"2.0.0\"\$' '${REPO5}/${WV_REL}'"
+rm -rf "$REPO5"
+
+# ---------------------------------------------------------------------------
+# Test group 5: package.json mismatch fails before projection
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 5: package.json mismatch fails before projection ==="
+
+REPO6="$(new_scratch_repo "1.2.3")"
+jq '.version = "9.9.9"' "${REPO6}/plugins/agile-workflow/package.json" \
+  > "${REPO6}/plugins/agile-workflow/package.json.tmp"
+mv "${REPO6}/plugins/agile-workflow/package.json.tmp" \
+  "${REPO6}/plugins/agile-workflow/package.json"
+( cd "$REPO6" && "$REAL_GIT" add -A && "$REAL_GIT" commit -q -m "mismatch package version" >/dev/null 2>&1 )
+
+run_bump "$REPO6" agile-workflow patch
+
+assert_eq "package-mismatch bump exits 1" "1" "$BUMP_RC"
+MISMATCH_ERR="$(cat "$BUMP_ERR" 2>/dev/null || true)"
+assert_true "package mismatch reports channel metadata mismatch" \
+  "printf '%s' \"\$MISMATCH_ERR\" | grep -q 'version mismatch between channel metadata'"
+assert_eq "claude manifest remains old after package mismatch" "1.2.3" \
+  "$(jq -r '.version' "${REPO6}/plugins/agile-workflow/.claude-plugin/plugin.json")"
+assert_eq "package manifest remains mismatched after failed bump" "9.9.9" \
+  "$(jq -r '.version' "${REPO6}/plugins/agile-workflow/package.json")"
+assert_eq ".work-view-version unchanged after package mismatch" "1.2.3" \
+  "$(cat "${REPO6}/${VER_REL}")"
+assert_false "claude plugin.json NOT staged after package mismatch" \
+  "is_staged '$REPO6' 'plugins/agile-workflow/.claude-plugin/plugin.json'"
+assert_false "codex plugin.json NOT staged after package mismatch" \
+  "is_staged '$REPO6' 'plugins/agile-workflow/.codex-plugin/plugin.json'"
+assert_false "package.json NOT staged after package mismatch" \
+  "is_staged '$REPO6' 'plugins/agile-workflow/package.json'"
+
+rm -rf "$REPO6"
+
+# ---------------------------------------------------------------------------
+# Test group 6: plugin without package.json still bumps Claude/Codex metadata
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 6: missing package.json remains supported ==="
+
+REPO7="$(new_scratch_repo "1.2.3" "workflow")"
+rm "${REPO7}/plugins/workflow/package.json"
+( cd "$REPO7" && "$REAL_GIT" add -A && "$REAL_GIT" commit -q -m "remove package metadata" >/dev/null 2>&1 )
+
+run_bump "$REPO7" workflow patch
+
+assert_eq "no-package plugin bump exits 0" "0" "$BUMP_RC"
+assert_eq "no-package claude plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO7}/plugins/workflow/.claude-plugin/plugin.json")"
+assert_eq "no-package codex plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO7}/plugins/workflow/.codex-plugin/plugin.json")"
+assert_false "no-package plugin does not create package.json" \
+  "[ -f '${REPO7}/plugins/workflow/package.json' ]"
+
+rm -rf "$REPO7"
+
+# ---------------------------------------------------------------------------
+# Test group 7: agentic-research patch bump projects new semver into BOTH
+# research-view implementations
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 7: agentic-research patch bump projects lockstep ==="
+
+REPO8="$(new_scratch_repo "1.2.3")"
+run_bump "$REPO8" agentic-research patch
+
+assert_eq "ar patch bump exits 0" "0" "$BUMP_RC"
+
+# .research-view-version equals the new semver with NO trailing newline.
+RV_VER_CONTENT="$(cat "${REPO8}/${RV_VER_REL}")"
+assert_eq "ar: .research-view-version content == new semver" "1.2.4" "$RV_VER_CONTENT"
+RV_VER_BYTES="$(wc -c < "${REPO8}/${RV_VER_REL}" | tr -d ' ')"
+assert_eq "ar: .research-view-version is exactly 5 bytes (no trailing newline)" "5" "$RV_VER_BYTES"
+LAST_BYTE_RV="$(tail -c1 "${REPO8}/${RV_VER_REL}" | od -An -tx1 | tr -d ' \n')"
+assert_false "ar: .research-view-version last byte is not a newline (0a)" "[ '$LAST_BYTE_RV' = '0a' ]"
+
+# research-view.sh RESEARCH_VIEW_VERSION literal equals the new semver.
+assert_true "ar: research-view.sh RESEARCH_VIEW_VERSION literal == new" \
+  "grep -q '^RESEARCH_VIEW_VERSION=\"1.2.4\"\$' '${REPO8}/${RV_SH_REL}'"
+assert_false "ar: old RESEARCH_VIEW_VERSION literal is gone" \
+  "grep -q '^RESEARCH_VIEW_VERSION=\"1.2.3\"\$' '${REPO8}/${RV_SH_REL}'"
+
+# plugin.json manifests and package metadata advanced.
+assert_eq "ar: claude plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO8}/plugins/agentic-research/.claude-plugin/plugin.json")"
+assert_eq "ar: codex plugin.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO8}/plugins/agentic-research/.codex-plugin/plugin.json")"
+assert_eq "ar: package.json bumped" "1.2.4" \
+  "$(jq -r '.version' "${REPO8}/plugins/agentic-research/package.json")"
+
+# Both research-view files are git-staged.
+assert_true "ar: claude plugin.json is staged" \
+  "is_staged '$REPO8' 'plugins/agentic-research/.claude-plugin/plugin.json'"
+assert_true "ar: codex plugin.json is staged" \
+  "is_staged '$REPO8' 'plugins/agentic-research/.codex-plugin/plugin.json'"
+assert_true "ar: .research-view-version is staged" "is_staged '$REPO8' '$RV_VER_REL'"
+assert_true "ar: research-view.sh is staged" "is_staged '$REPO8' '$RV_SH_REL'"
+assert_true "ar: agentic-research package.json is staged" \
+  "is_staged '$REPO8' 'plugins/agentic-research/package.json'"
+
+# agentic-research bump must NOT touch work-view artifacts.
+assert_eq "ar-bump: .work-view-version unchanged" "1.2.3" \
+  "$(cat "${REPO8}/${VER_REL}")"
+assert_true "ar-bump: work-view.sh WORK_VIEW_VERSION unchanged" \
+  "grep -q '^WORK_VIEW_VERSION=\"1.2.3\"\$' '${REPO8}/${WV_REL}'"
+assert_false "ar-bump: .work-view-version NOT staged" \
+  "is_staged '$REPO8' '$VER_REL'"
+assert_false "ar-bump: work-view.sh NOT staged" \
+  "is_staged '$REPO8' '$WV_REL'"
+
+rm -rf "$REPO8"
+
+# ---------------------------------------------------------------------------
+# Test group 8: agentic-research Fail-Fast postcondition fires on indented literal
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 8: ar postcondition exits 1 when literal anchor no longer matches ==="
+
+REPO9="$(new_scratch_repo "1.2.3")"
+# Break the anchored literal: indent it so `^RESEARCH_VIEW_VERSION="..."` no longer
+# matches. sed exits 0 (no match), so the projection silently no-ops and the grep
+# postcondition must catch it.
+cat > "${REPO9}/${RV_SH_REL}" <<'EOF'
+#!/usr/bin/env bash
+  RESEARCH_VIEW_VERSION="1.2.3"
+echo "research-view ${RESEARCH_VIEW_VERSION}"
+EOF
+( cd "$REPO9" && "$REAL_GIT" add -A && "$REAL_GIT" commit -q -m "indent rv literal" >/dev/null 2>&1 )
+
+run_bump "$REPO9" agentic-research patch
+
+assert_eq "ar: indented-literal bump exits 1 (postcondition fires)" "1" "$BUMP_RC"
+AR_POST_ERR="$(cat "$BUMP_ERR" 2>/dev/null || true)"
+assert_true "ar: postcondition reports failed projection" \
+  "printf '%s' \"\$AR_POST_ERR\" | grep -q 'failed to project version into'"
+# The indented literal was not rewritten.
+assert_false "ar: indented literal NOT rewritten to new semver" \
+  "grep -q 'RESEARCH_VIEW_VERSION=\"1.2.4\"' '${REPO9}/${RV_SH_REL}'"
+
+rm -rf "$REPO9"
+
+# ---------------------------------------------------------------------------
+# Test group 9: agentic-research minor/major bumps project correctly
+# ---------------------------------------------------------------------------
+echo ""
+echo "=== Test group 9: ar minor and major bumps project lockstep ==="
+
+REPO10="$(new_scratch_repo "1.2.3")"
+run_bump "$REPO10" agentic-research minor
+assert_eq "ar: minor bump exits 0" "0" "$BUMP_RC"
+assert_eq "ar: minor .research-view-version == 1.3.0" "1.3.0" "$(cat "${REPO10}/${RV_VER_REL}")"
+assert_true "ar: minor research-view.sh literal == 1.3.0" \
+  "grep -q '^RESEARCH_VIEW_VERSION=\"1.3.0\"\$' '${REPO10}/${RV_SH_REL}'"
+rm -rf "$REPO10"
+
+REPO11="$(new_scratch_repo "1.2.3")"
+run_bump "$REPO11" agentic-research major
+assert_eq "ar: major bump exits 0" "0" "$BUMP_RC"
+assert_eq "ar: major .research-view-version == 2.0.0" "2.0.0" "$(cat "${REPO11}/${RV_VER_REL}")"
+assert_true "ar: major research-view.sh literal == 2.0.0" \
+  "grep -q '^RESEARCH_VIEW_VERSION=\"2.0.0\"\$' '${REPO11}/${RV_SH_REL}'"
+rm -rf "$REPO11"
+
+# ---------------------------------------------------------------------------
+# Summary
+# ---------------------------------------------------------------------------
+echo ""
+echo "============================================================"
+echo "Results: ${PASS} passed, ${FAIL} failed"
+if [ ${#ERRORS[@]} -gt 0 ]; then
+  echo "Failed tests:"
+  for e in "${ERRORS[@]}"; do
+    echo "  - $e"
+  done
+fi
+echo "============================================================"
+
+[ "$FAIL" -eq 0 ] || exit 1

package/scripts/tests/channel-parity.test.sh

@@ -0,0 +1,117 @@
+#!/usr/bin/env bash
+# channel-parity.test.sh — guard agile-workflow's Claude/Codex/Pi behavior parity.
+
+set -uo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+REPO_ROOT="$(cd "${SCRIPT_DIR}/../../../.." && pwd)"
+PLUGIN_ROOT="${REPO_ROOT}/plugins/agile-workflow"
+ROOT_PACKAGE="${REPO_ROOT}/package.json"
+
+PASS=0
+FAIL=0
+ERRORS=()
+
+assert_true() {
+  local label="$1"
+  if eval "$2"; then
+    echo "  PASS: $label"
+    ((PASS++))
+  else
+    echo "  FAIL: $label"
+    echo "        condition false: $2"
+    ((FAIL++))
+    ERRORS+=("$label")
+  fi
+}
+
+assert_eq() {
+  local label="$1" expected="$2" actual="$3"
+  if [ "$expected" = "$actual" ]; then
+    echo "  PASS: $label"
+    ((PASS++))
+  else
+    echo "  FAIL: $label"
+    echo "        expected: $(printf '%q' "$expected")"
+    echo "        actual:   $(printf '%q' "$actual")"
+    ((FAIL++))
+    ERRORS+=("$label")
+  fi
+}
+
+json_value() {
+  jq -r "$1" "$2"
+}
+
+echo ""
+echo "=== Agile-workflow channel parity ==="
+assert_true "jq is available" "command -v jq >/dev/null 2>&1"
+
+HOOKS_JSON="${PLUGIN_ROOT}/hooks/hooks.json"
+PROMPT_CONTEXT="${PLUGIN_ROOT}/hooks/scripts/prompt-context.py"
+MAINTAINER="${PLUGIN_ROOT}/hooks/scripts/substrate-maintainer.py"
+PI_EXTENSION="${PLUGIN_ROOT}/extensions/agile-workflow.ts"
+PI_TEST="${PLUGIN_ROOT}/extensions/agile-workflow.test.ts"
+
+assert_true "shared hooks manifest exists" "[ -f '${HOOKS_JSON}' ]"
+assert_true "shared prompt-context hook exists" "[ -f '${PROMPT_CONTEXT}' ]"
+assert_true "shared substrate-maintainer hook exists" "[ -f '${MAINTAINER}' ]"
+
+assert_eq "Codex manifest points at shared hooks" "./hooks/hooks.json" \
+  "$(json_value '.hooks // empty' "${PLUGIN_ROOT}/.codex-plugin/plugin.json")"
+assert_true "Claude plugin relies on hooks auto-discovery" \
+  "[ -f '${PLUGIN_ROOT}/.claude-plugin/plugin.json' ] && [ -f '${HOOKS_JSON}' ]"
+assert_eq "Pi package loads agile extension directory" "[\"./extensions\"]" \
+  "$(jq -c '.pi.extensions' "${PLUGIN_ROOT}/package.json")"
+assert_true "root Pi package includes agile-workflow extension" \
+  "jq -e '.pi.extensions | index(\"./plugins/agile-workflow/extensions/agile-workflow.ts\")' '${ROOT_PACKAGE}' >/dev/null"
+
+assert_true "hooks manifest has SessionStart" "jq -e '.hooks.SessionStart' '${HOOKS_JSON}' >/dev/null"
+assert_true "hooks manifest has UserPromptSubmit" "jq -e '.hooks.UserPromptSubmit' '${HOOKS_JSON}' >/dev/null"
+assert_true "hooks manifest has PostCompact" "jq -e '.hooks.PostCompact' '${HOOKS_JSON}' >/dev/null"
+assert_true "hooks manifest has PostToolUse" "jq -e '.hooks.PostToolUse' '${HOOKS_JSON}' >/dev/null"
+
+assert_true "Pi extension invokes prompt-context shared script" \
+  "grep -q 'prompt-context.py' '${PI_EXTENSION}'"
+assert_true "Pi extension invokes substrate-maintainer shared script" \
+  "grep -q 'substrate-maintainer.py' '${PI_EXTENSION}'"
+assert_true "Pi extension maps before_agent_start" \
+  "grep -q 'before_agent_start' '${PI_EXTENSION}'"
+assert_true "Pi extension maps session_start" \
+  "grep -q 'session_start' '${PI_EXTENSION}'"
+assert_true "Pi extension maps session_compact" \
+  "grep -q 'session_compact' '${PI_EXTENSION}'"
+assert_true "Pi extension maps tool_result" \
+  "grep -q 'tool_result' '${PI_EXTENSION}'"
+assert_true "Pi extension uses synthetic PiBeforeAgentStart rules path" \
+  "grep -q 'PiBeforeAgentStart' '${PI_EXTENSION}' && grep -q 'PiBeforeAgentStart' '${PROMPT_CONTEXT}'"
+assert_true "prompt-context supports forced rules for Pi rebuilt prompts" \
+  "grep -q 'force_rules_context' '${PI_EXTENSION}' && grep -q 'force: bool = False' '${PROMPT_CONTEXT}'"
+assert_true "Pi extension injects principles as visible hook-context message" \
+  "grep -q 'agile-workflow-principles' '${PI_EXTENSION}'"
+
+assert_true "Pi tests cover before_agent_start parity" \
+  "grep -q 'before_agent_start' '${PI_TEST}' && grep -q 'shared .agents/rules content' '${PI_TEST}'"
+assert_true "Pi tests cover substrate maintenance parity" \
+  "grep -q 'runs substrate maintenance after mutating tools' '${PI_TEST}'"
+
+assert_true "AGENTS states channel parity posture" \
+  "grep -q 'Channel parity posture' '${REPO_ROOT}/AGENTS.md'"
+
+assert_true "SPEC documents Pi hook parity adapter" \
+  "grep -q 'Pi hook parity adapter' '${PLUGIN_ROOT}/docs/SPEC.md'"
+assert_true "ARCHITECTURE documents Pi hook parity adapter" \
+  "grep -q 'Pi hook parity adapter' '${PLUGIN_ROOT}/docs/ARCHITECTURE.md'"
+
+echo ""
+echo "============================================================"
+echo "Results: ${PASS} passed, ${FAIL} failed"
+if [ ${#ERRORS[@]} -gt 0 ]; then
+  echo "Failed tests:"
+  for e in "${ERRORS[@]}"; do
+    echo "  - $e"
+  done
+fi
+echo "============================================================"
+
+[ "$FAIL" -eq 0 ] || exit 1