agent-harness-kit 0.10.2 → 0.11.1

package/src/templates/scripts/subagent-stop.sh.hbs

@@ -14,20 +14,9 @@ set -eo pipefail
 
 INPUT=$(cat)
 SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
-have_jq() {
-  [ "${AHK_DISABLE_JQ:-}" = "1" ] && return 1
-  command -v jq >/dev/null 2>&1
-}
-have_jp() {
-  have_jq && return 0
-  command -v node >/dev/null 2>&1 && [ -f "$SCRIPT_DIR/_lib/json-pick.mjs" ] && return 0
-  return 1
-}
-jp() {
-  if have_jq; then jq -r "$1"
-  else node "$SCRIPT_DIR/_lib/json-pick.mjs" "$1"
-  fi
-}
+_LIB_DIR="$SCRIPT_DIR/_lib"
+. "$_LIB_DIR/jp.sh"
+. "$_LIB_DIR/telemetry.sh"
 
 SUBAGENT="(unknown)"
 if have_jp; then
@@ -35,12 +24,12 @@ if have_jp; then
 fi
 
 # Telemetry first so we record every subagent boundary, even if the
-# structural-test bails below.
-mkdir -p .harness
+# structural-test bails below. telemetry_append handles rotation.
 TS=$(date -u +%Y-%m-%dT%H:%M:%SZ)
 SHA=$(git rev-parse --short HEAD 2>/dev/null || echo 'no-git')
-printf '{"ts":"%s","event":"subagent_stop","subagent":"%s","sha":"%s"}\n' \
-  "$TS" "$SUBAGENT" "$SHA" >> .harness/telemetry.jsonl
+LINE=$(printf '{"ts":"%s","event":"subagent_stop","subagent":"%s","sha":"%s"}' \
+  "$TS" "$SUBAGENT" "$SHA")
+telemetry_append "$LINE"
 
 # Skip if structural test disabled.
 if [ -f harness.config.json ] \

package/src/templates/scripts/telemetry-on-skill.sh

@@ -8,25 +8,20 @@
 # v0.7: migrated from `command -v jq` fail-open gate to the kit's jp() helper
 # so the telemetry record still gets written on jq-less CI / Windows. Without
 # the migration, telemetry quietly went dark anywhere jq wasn't installed.
+# v0.10.3: jp/have_jq/have_jp extracted to _lib/jp.sh; AHK_DISABLE_TELEMETRY
+# opt-out + AHK_TELEMETRY_MAX_LINES rotation added.
 set -e
 
+# Opt-out: respect AHK_DISABLE_TELEMETRY=1 before reading stdin so the user
+# can fully disable observability without removing the hook from settings.
+[ "${AHK_DISABLE_TELEMETRY:-}" = "1" ] && exit 0
+
 INPUT=$(cat)
 
 SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
-have_jq() {
-  [ "${AHK_DISABLE_JQ:-}" = "1" ] && return 1
-  command -v jq >/dev/null 2>&1
-}
-have_jp() {
-  have_jq && return 0
-  command -v node >/dev/null 2>&1 && [ -f "$SCRIPT_DIR/_lib/json-pick.mjs" ] && return 0
-  return 1
-}
-jp() {
-  if have_jq; then jq -r "$1"
-  else node "$SCRIPT_DIR/_lib/json-pick.mjs" "$1"
-  fi
-}
+_LIB_DIR="$SCRIPT_DIR/_lib"
+. "$_LIB_DIR/jp.sh"
+. "$_LIB_DIR/telemetry.sh"
 if ! have_jp; then exit 0; fi
 
 TOOL=$(echo "$INPUT" | jp '.tool_name // empty')
@@ -35,14 +30,13 @@ TOOL=$(echo "$INPUT" | jp '.tool_name // empty')
 SKILL=$(echo "$INPUT" | jp '.tool_input.skill // empty')
 [ -z "$SKILL" ] && exit 0
 
-mkdir -p .harness
 TS=$(date -u +%Y-%m-%dT%H:%M:%SZ)
 SHA=$(git rev-parse --short HEAD 2>/dev/null || echo 'no-git')
 
 # Compose JSONL line by hand — same shape as the previous jq-built record.
-# Quoting via printf '%s' so embedded spaces in skill names don't break the
-# line. Skill names are constrained to `[a-z0-9-]+` upstream so we don't
-# need full JSON escaping here.
-printf '{"ts":"%s","event":"skill_invoked","skill":"%s","sha":"%s"}\n' \
-  "$TS" "$SKILL" "$SHA" >> .harness/telemetry.jsonl
+# Skill names are constrained to `[a-z0-9-]+` upstream so we don't need full
+# JSON escaping here. telemetry_append handles mkdir, append, and rotation.
+LINE=$(printf '{"ts":"%s","event":"skill_invoked","skill":"%s","sha":"%s"}' \
+  "$TS" "$SKILL" "$SHA")
+telemetry_append "$LINE"
 exit 0

package/src/templates/scripts/userprompt-guard.sh.hbs

@@ -14,26 +14,8 @@ set -eo pipefail
 
 INPUT=$(cat)
 SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
-have_jq() {
-  [ "${AHK_DISABLE_JQ:-}" = "1" ] && return 1
-  command -v jq >/dev/null 2>&1
-}
-have_jp() {
-  have_jq && return 0
-  command -v node >/dev/null 2>&1 && [ -f "$SCRIPT_DIR/_lib/json-pick.mjs" ] && return 0
-  return 1
-}
-jp() {
-  if have_jq; then
-    if [ -n "$2" ]; then jq -r "$1" "$2"; else jq -r "$1"; fi
-  else
-    if [ -n "$2" ]; then
-      node "$SCRIPT_DIR/_lib/json-pick.mjs" "$1" "$2"
-    else
-      node "$SCRIPT_DIR/_lib/json-pick.mjs" "$1"
-    fi
-  fi
-}
+_LIB_DIR="$SCRIPT_DIR/_lib"
+. "$_LIB_DIR/jp.sh"
 
 if ! have_jp; then
   exit 0