@rubytech/create-maxy-code 0.1.254 → 0.1.256

package/payload/platform/plugins/admin/hooks/session-end-retrospective.sh

@@ -1,214 +0,0 @@
-#!/usr/bin/env bash
-# Stop hook — gates session termination until the admin agent has performed
-# a four-pass retrospective (technical learnings, operator-relationship
-# updates, graph-completeness sweep, typed-edge auto-extraction) and called
-# the deterministic `session-retrospective-mark-complete` MCP sentinel
-# tool. Task 282; converged onto the Task 439 shared helper.
-#
-# Trigger detection. End-intent is the operator's most recent real-user
-# message matching one of the literal token strings:
-#     /end | /archive | end session | archive this session
-# "Real user" follows the same shape used by
-# askuserquestion-investigate-gate.sh — `type=user, message.role=user,
-# content` is a string OR a list with at least one non-`tool_result` block.
-# `tool_result`-only user records are tool replies, not turn starts, and
-# never count as the latest user message.
-#
-# The Sidebar Archive button POSTs `/:sessionId/archive` which kills the
-# PTY directly — no Stop fires after that, so clicking Archive is the
-# operator's de-facto skip path. The spec lists opt-out as a deferred
-# follow-up; archive-click is its current shape and acceptable behaviour.
-#
-# Sentinel detection. Completion is recognised by greping the operator's
-# own JSONL (transcript_path on the Stop envelope) for a `tool_use` block
-# whose `name` is exactly `mcp__admin__session-retrospective-mark-complete`
-# — that is the prefixed form Claude Code writes for MCP tools in the
-# JSONL; the unprefixed name only appears in the operator-facing
-# instruction text below. The sentinel-grep doubles as the re-entry guard:
-# every Stop that fires after the agent's sentinel call sees the sentinel
-# and exits 0 with `gate-released`, so the agent's wrap-up turn never
-# re-blocks.
-# [[feedback_no_stdout_parsing_for_control_flow]],
-# [[feedback_doctrine_paragraph_is_not_a_gate]] — the LLM is removed from
-# the per-decision path entirely.
-#
-# Gating (common guards live in `lib/admin-graph-pass-common.sh`; this
-# hook adds the end-intent gating layered on top):
-#   - role-not-admin / is-specialist / empty-stdin / missing-transcript:
-#     handled by the shared helper, emit `trigger-skipped` and exit 0.
-#   - no-intent-match: latest real-user message has no end-intent token,
-#     emit `trigger-skipped` and exit 0.
-#
-# Releases:
-#   - gate-released sessionId=<id> (when sentinel present in transcript)
-#
-# Blocks:
-#   - gate-blocked sessionId=<id> reason=sentinel-absent
-#     (exit 2; stderr carries the four-pass retrospective instruction —
-#     stderr is the Stop-hook contract for handing instructions back to
-#     the agent).
-#
-# Input: Claude Code's Stop hook stdin shape
-#   { "session_id": "<intrinsic>", "transcript_path": "<jsonl path>", ... }
-
-set -uo pipefail
-
-AGP_LOG_TAG="session-retrospective"
-AGP_LIB="$(dirname "$0")/lib/admin-graph-pass-common.sh"
-if [ ! -f "$AGP_LIB" ]; then
-  echo "[${AGP_LOG_TAG}] trigger-skipped sessionId=unknown reason=missing-helper path=${AGP_LIB}" >&2
-  exit 0
-fi
-# shellcheck source=lib/admin-graph-pass-common.sh
-source "$AGP_LIB"
-
-agp_setup_logging
-agp_read_stdin
-agp_parse_stdin
-agp_guard_role_specialist
-agp_guard_transcript require-stdin
-
-# Single Python pass walks the transcript and reports two facts:
-#   1. The latest end-intent token in the most recent real-user message,
-#      or empty if no real user message matches.
-#   2. Whether any `tool_use` block in the transcript names the sentinel
-#      tool `mcp__admin__session-retrospective-mark-complete`.
-# Output one line: <intent-token-or-empty>\t<sentinel-present-yes|no>
-INSPECTION=$(TRANSCRIPT_PATH="$TRANSCRIPT_PATH" python3 - <<'PY'
-import os, json
-
-SENTINEL_TOOL = "mcp__admin__session-retrospective-mark-complete"
-INTENT_TOKENS = ("/end", "/archive", "end session", "archive this session")
-
-def is_real_user(rec):
-    if not isinstance(rec, dict) or rec.get("type") != "user":
-        return False
-    msg = rec.get("message")
-    if not isinstance(msg, dict) or msg.get("role") != "user":
-        return False
-    content = msg.get("content")
-    if isinstance(content, str):
-        return True
-    if isinstance(content, list):
-        for b in content:
-            if isinstance(b, dict) and b.get("type") != "tool_result":
-                return True
-        return False
-    return True
-
-def extract_user_text(rec):
-    msg = rec.get("message", {}) or {}
-    content = msg.get("content")
-    if isinstance(content, str):
-        return content
-    if isinstance(content, list):
-        out = []
-        for b in content:
-            if not isinstance(b, dict):
-                continue
-            if b.get("type") == "text":
-                t = b.get("text")
-                if isinstance(t, str):
-                    out.append(t)
-        return "".join(out)
-    return ""
-
-def assistant_tool_use_names(rec):
-    if not isinstance(rec, dict) or rec.get("type") != "assistant":
-        return []
-    msg = rec.get("message", {}) or {}
-    if msg.get("role") != "assistant":
-        return []
-    content = msg.get("content")
-    if not isinstance(content, list):
-        return []
-    names = []
-    for b in content:
-        if isinstance(b, dict) and b.get("type") == "tool_use":
-            n = b.get("name")
-            if isinstance(n, str):
-                names.append(n)
-    return names
-
-path = os.environ["TRANSCRIPT_PATH"]
-records = []
-try:
-    with open(path, "r", encoding="utf-8") as f:
-        for raw in f:
-            raw = raw.strip()
-            if not raw:
-                continue
-            try:
-                records.append(json.loads(raw))
-            except Exception:
-                continue
-except Exception:
-    print("\tno")
-    raise SystemExit(0)
-
-# Sentinel presence — any assistant tool_use of the sentinel tool anywhere
-# in this session's JSONL.
-sentinel_present = False
-for r in records:
-    if SENTINEL_TOOL in assistant_tool_use_names(r):
-        sentinel_present = True
-        break
-
-# Latest real-user message text.
-latest_user_text = ""
-for r in reversed(records):
-    if is_real_user(r):
-        latest_user_text = extract_user_text(r).strip()
-        break
-
-matched = ""
-if latest_user_text:
-    # Case-insensitive whole-message match against the token set. A token
-    # must appear as the entire user message (after .strip()), or as a
-    # standalone line within it — not embedded inside a paragraph. Keeps
-    # the gate from false-positiving on "I'll end session by 5pm" prose.
-    haystacks = {latest_user_text.lower()}
-    for line in latest_user_text.splitlines():
-        s = line.strip().lower()
-        if s:
-            haystacks.add(s)
-    for tok in INTENT_TOKENS:
-        if tok.lower() in haystacks:
-            matched = tok
-            break
-
-print(f"{matched}\t{'yes' if sentinel_present else 'no'}")
-PY
-)
-INTENT_TOKEN="${INSPECTION%%	*}"
-SENTINEL_PRESENT="${INSPECTION#*	}"
-
-if [ -z "$INTENT_TOKEN" ]; then
-  agp_emit_log "trigger-skipped sessionId=${SESSION_ID} reason=no-intent-match"
-  exit 0
-fi
-
-agp_emit_log "trigger sessionId=${SESSION_ID} token=${INTENT_TOKEN}"
-
-if [ "$SENTINEL_PRESENT" = "yes" ]; then
-  agp_emit_log "gate-released sessionId=${SESSION_ID}"
-  exit 0
-fi
-
-agp_emit_log "gate-blocked sessionId=${SESSION_ID} reason=sentinel-absent"
-
-cat >&2 <<'INSTRUCTION'
-The operator signalled session end. Before this session can close, run the four-pass retrospective inside this same admin session — do not spawn a new session, do not background anything. Use the tools you already have.
-
-Pass 1 — Technical learnings. Walk this session for operator corrections and self-detected mistakes. For each one, delegate one graph write to `database-operator` via the Task tool, persisting a Learning or Correction node per the schema. Count what you write.
-
-Pass 2 — Operator-relationship updates. Walk this session for tonal signals, rejected phrasings, and working-style preferences. For each one worth carrying forward, Write the observation into `SOUL.md` at the account-scoped path. Count what you write.
-
-Pass 3 — Graph completeness sweep. Walk this session for any node, edge, or commitment that was discussed but not written to the graph in-flight. For each missing write, delegate to `database-operator` via the Task tool. Count what you write.
-
-Pass 4 — Typed-edge auto-extraction. Delegate to `database-operator` via the Task tool with a prompt naming the typed-edge pass and the session-start timestamp. The specialist calls `memory-typed-edge-pass` with that timestamp and returns the counters. Capture `nodesProcessed` and the accepted count from its reply.
-
-Then call `session-retrospective-mark-complete` with all five counts (learningsCount, preferencesCount, graphWriteCount, typedEdgesCount, nodesProcessed) and reply with one short paragraph in your house voice summarising what you wrote.
-INSTRUCTION
-
-exit 2

package/payload/premium-plugins/writer-craft/hooks/__tests__/voice-session-end-text.test.sh

@@ -1,284 +0,0 @@
-#!/usr/bin/env bash
-# voice-session-end-text.sh hook test suite.
-#
-# Pattern: a local Python HTTP server stands in for /api/admin/log-ingest
-# and records every POST so the assertions can inspect every log line the
-# hook emitted. The CLI is stubbed with a mock node script so tests run
-# without a live Neo4j connection.
-#
-# Contract under test:
-#   - Hook skips with reason=role-not-admin when MAXY_SESSION_ROLE != admin.
-#   - Hook skips with reason=is-specialist when MAXY_SPECIALIST is set.
-#   - Hook skips with reason=empty-stdin when no stdin is piped.
-#   - Hook skips with reason=missing-transcript when the JSONL path in the
-#     stdin envelope does not exist.
-#   - Hook skips with reason=cli-not-found when the compiled CLI is absent.
-#   - Happy path: hook emits trigger + result=ok when all guards pass and the
-#     stub CLI exits 0.
-#   - Dedupe path: stub CLI returns turns=0 skipped=3 → result=ok with those
-#     counts (verifying the hook forwards the CLI stdout to the log-ingest).
-
-set -u
-
-HOOK="$(cd "$(dirname "$0")/.." && pwd)/voice-session-end-text.sh"
-if [[ ! -f "$HOOK" ]]; then
-  echo "FAIL: hook not found at $HOOK" >&2
-  exit 1
-fi
-
-PASS=0
-FAIL=0
-TMPFILES=()
-LISTENER_PIDS=()
-
-cleanup() {
-  for pid in "${LISTENER_PIDS[@]:-}"; do
-    [[ -n "$pid" ]] && { kill "$pid" 2>/dev/null || true; wait "$pid" 2>/dev/null || true; }
-  done
-  for f in "${TMPFILES[@]:-}"; do
-    [[ -n "$f" ]] && rm -f "$f" 2>/dev/null || true
-  done
-}
-trap cleanup EXIT
-
-# ---- helpers ---------------------------------------------------------------
-
-make_tmp_file() {
-  local tmp
-  tmp=$(mktemp)
-  TMPFILES+=("$tmp")
-  echo "$tmp"
-}
-
-# Start a listener on a random port; capture every POST body.
-# Sets LISTENER_PORT and LISTENER_LOG globals.
-start_listener() {
-  LISTENER_LOG=$(make_tmp_file)
-  LISTENER_PORT=$(python3 -c 'import socket; s=socket.socket(); s.bind(("",0)); print(s.getsockname()[1]); s.close()')
-  python3 - "$LISTENER_PORT" "$LISTENER_LOG" <<'PY' &
-import sys, http.server, json
-
-port = int(sys.argv[1])
-log  = sys.argv[2]
-
-class H(http.server.BaseHTTPRequestHandler):
-    def do_POST(self):
-        n = int(self.headers.get("Content-Length", 0))
-        body = self.rfile.read(n).decode()
-        with open(log, "a") as f:
-            f.write(body + "\n")
-        self.send_response(200)
-        self.end_headers()
-    def log_message(self, *a): pass
-
-http.server.HTTPServer(("127.0.0.1", port), H).serve_forever()
-PY
-  local pid=$!
-  LISTENER_PIDS+=("$pid")
-  sleep 0.3   # let the server start
-}
-
-# Run the hook with given env overrides and optional stdin.
-# Returns: HOOK_EXIT, LISTENER_LINES (array of log-ingest line bodies).
-run_hook() {
-  local env_args=("$@")
-  local stdin_data="${HOOK_STDIN:-}"
-  local hook_exit=0
-  if [[ -n "$stdin_data" ]]; then
-    printf '%s' "$stdin_data" | env \
-      MAXY_UI_INTERNAL_PORT="$LISTENER_PORT" \
-      MAXY_SESSION_ROLE="${MAXY_SESSION_ROLE:-admin}" \
-      MAXY_SPECIALIST="${MAXY_SPECIALIST:-}" \
-      PLATFORM_ROOT="${FAKE_PLATFORM_ROOT:-}" \
-      ACCOUNT_ID="${ACCOUNT_ID:-acct-test}" \
-      USER_ID="${USER_ID:-user-test}" \
-      SESSION_ID="" \
-      NEO4J_URI="bolt://localhost:7688" \
-      NEO4J_USER="neo4j" \
-      NEO4J_PASSWORD="test" \
-      ${env_args[@]+"${env_args[@]}"} \
-      bash "$HOOK" >/dev/null 2>/dev/null || hook_exit=$?
-  else
-    env \
-      MAXY_UI_INTERNAL_PORT="$LISTENER_PORT" \
-      MAXY_SESSION_ROLE="${MAXY_SESSION_ROLE:-admin}" \
-      MAXY_SPECIALIST="${MAXY_SPECIALIST:-}" \
-      PLATFORM_ROOT="${FAKE_PLATFORM_ROOT:-}" \
-      ACCOUNT_ID="${ACCOUNT_ID:-acct-test}" \
-      USER_ID="${USER_ID:-user-test}" \
-      SESSION_ID="" \
-      NEO4J_URI="bolt://localhost:7688" \
-      NEO4J_USER="neo4j" \
-      NEO4J_PASSWORD="test" \
-      ${env_args[@]+"${env_args[@]}"} \
-      bash "$HOOK" </dev/null >/dev/null 2>/dev/null || hook_exit=$?
-  fi
-  HOOK_EXIT=$hook_exit
-  # Read all log-ingest POSTs.
-  LISTENER_LINES=()
-  while IFS= read -r raw; do
-    [[ -z "$raw" ]] && continue
-    LISTENER_LINES+=("$raw")
-  done < "$LISTENER_LOG"
-  # Reset log for next run.
-  > "$LISTENER_LOG"
-}
-
-assert_contains_line() {
-  local test_name="$1"
-  local fragment="$2"
-  for line in "${LISTENER_LINES[@]:-}"; do
-    if [[ "$line" == *"$fragment"* ]]; then
-      echo "  PASS: $test_name"
-      PASS=$((PASS + 1))
-      return 0
-    fi
-  done
-  echo "  FAIL: $test_name (expected fragment: '$fragment' not found in log lines)"
-  printf '       lines:\n'
-  for line in "${LISTENER_LINES[@]:-}"; do printf '         %s\n' "$line"; done
-  FAIL=$((FAIL + 1))
-}
-
-assert_exit() {
-  local test_name="$1"
-  local expected="$2"
-  if [[ "$HOOK_EXIT" -eq "$expected" ]]; then
-    echo "  PASS: $test_name (exit $expected)"
-    PASS=$((PASS + 1))
-  else
-    echo "  FAIL: $test_name (expected exit $expected, got $HOOK_EXIT)"
-    FAIL=$((FAIL + 1))
-  fi
-}
-
-# ---- setup -----------------------------------------------------------------
-
-start_listener
-
-# Create a fake platform root structure so the lib source works.
-FAKE_PLATFORM_ROOT=$(mktemp -d)
-TMPFILES+=("$FAKE_PLATFORM_ROOT")
-mkdir -p "$FAKE_PLATFORM_ROOT/plugins/admin/hooks/lib"
-# Copy the real lib so guards work correctly.
-REAL_LIB="$(cd "$(dirname "$0")/../../../../platform/plugins/admin/hooks/lib" && pwd)/admin-graph-pass-common.sh"
-if [[ ! -f "$REAL_LIB" ]]; then
-  echo "FAIL: admin-graph-pass-common.sh not found at $REAL_LIB" >&2
-  exit 1
-fi
-cp "$REAL_LIB" "$FAKE_PLATFORM_ROOT/plugins/admin/hooks/lib/admin-graph-pass-common.sh"
-
-# Create a minimal fake JSONL transcript.
-FAKE_TRANSCRIPT=$(make_tmp_file)
-printf '{"type":"user","message":{"role":"user","content":"Hello world"}}\n' > "$FAKE_TRANSCRIPT"
-
-# Create a stub CLI that exits 0 with a canned result.
-FAKE_CLI_DIR=$(mktemp -d)
-TMPFILES+=("$FAKE_CLI_DIR")
-STUB_CLI="$FAKE_CLI_DIR/ingest-session-text.js"
-cat > "$STUB_CLI" <<'STUB'
-process.stdout.write("turns=3 skipped=0 sessionId=test-session\n");
-process.exit(0);
-STUB
-
-# Stub CLI for dedupe scenario (turns=0).
-STUB_CLI_DEDUP="$FAKE_CLI_DIR/ingest-dedup.js"
-cat > "$STUB_CLI_DEDUP" <<'STUB'
-process.stdout.write("turns=0 skipped=3 sessionId=test-session\n");
-process.exit(0);
-STUB
-
-# Stub CLI that exits 1 (error).
-STUB_CLI_ERR="$FAKE_CLI_DIR/ingest-error.js"
-cat > "$STUB_CLI_ERR" <<'STUB'
-process.stderr.write("simulated error\n");
-process.exit(1);
-STUB
-
-# Valid stdin envelope for the SessionEnd hook.
-valid_envelope() {
-  local transcript="${1:-$FAKE_TRANSCRIPT}"
-  printf '{"session_id":"test-session","transcript_path":"%s","hook_event_name":"SessionEnd"}' "$transcript"
-}
-
-# Override CLI path by symlinking ingest-session-text.js inside a fake platform layout.
-make_platform_with_cli() {
-  local stub="$1"
-  local proot
-  proot=$(mktemp -d)
-  TMPFILES+=("$proot")
-  cp "$FAKE_PLATFORM_ROOT/plugins/admin/hooks/lib/admin-graph-pass-common.sh" \
-     "$proot/plugins/admin/hooks/lib/" 2>/dev/null || \
-     { mkdir -p "$proot/plugins/admin/hooks/lib"; cp "$REAL_LIB" "$proot/plugins/admin/hooks/lib/admin-graph-pass-common.sh"; }
-  mkdir -p "$proot/../premium-plugins/writer-craft/mcp/dist/cli" 2>/dev/null || \
-    mkdir -p "$(dirname "$proot")/../premium-plugins/writer-craft/mcp/dist/cli" 2>/dev/null || true
-  # Simpler: build the expected path from $PLATFORM_ROOT/../premium-plugins/...
-  # The hook computes: ${PLATFORM_ROOT}/../premium-plugins/writer-craft/mcp/dist/cli/ingest-session-text.js
-  # So parent of $proot + /premium-plugins/writer-craft/mcp/dist/cli/
-  local parent
-  parent=$(dirname "$proot")
-  mkdir -p "$parent/premium-plugins/writer-craft/mcp/dist/cli"
-  cp "$stub" "$parent/premium-plugins/writer-craft/mcp/dist/cli/ingest-session-text.js"
-  echo "$proot"
-}
-
-# ---- tests -----------------------------------------------------------------
-echo "[voice-session-end-text.test] starting"
-
-# T1: non-admin role → skip
-MAXY_SESSION_ROLE="public" HOOK_STDIN=$(valid_envelope) run_hook
-assert_exit "T1: non-admin exits 0" 0
-assert_contains_line "T1: non-admin emits trigger-skipped" "trigger-skipped"
-
-# T2: specialist → skip
-unset MAXY_SESSION_ROLE 2>/dev/null || true
-MAXY_SESSION_ROLE="admin" MAXY_SPECIALIST="database-operator" HOOK_STDIN=$(valid_envelope) run_hook
-assert_exit "T2: specialist exits 0" 0
-assert_contains_line "T2: specialist emits trigger-skipped" "trigger-skipped"
-
-# T3: empty stdin → skip
-MAXY_SPECIALIST="" MAXY_SESSION_ROLE="admin"
-HOOK_STDIN="" run_hook   # no stdin piped
-assert_exit "T3: empty-stdin exits 0" 0
-assert_contains_line "T3: empty-stdin emits trigger-skipped" "trigger-skipped"
-
-# T4: missing transcript → skip
-FAKE_MISSING=$(make_tmp_file)
-rm -f "$FAKE_MISSING"
-HOOK_STDIN=$(valid_envelope "$FAKE_MISSING") run_hook
-assert_exit "T4: missing-transcript exits 0" 0
-assert_contains_line "T4: missing-transcript emits trigger-skipped" "trigger-skipped"
-
-# T5: CLI not found → skip
-FAKE_PLATFORM_ROOT="$FAKE_PLATFORM_ROOT" HOOK_STDIN=$(valid_envelope) run_hook
-assert_exit "T5: cli-not-found exits 0" 0
-assert_contains_line "T5: cli-not-found emits trigger-skipped" "cli-not-found"
-
-# T6: happy path — stub CLI exits 0
-HAPPY_PROOT=$(make_platform_with_cli "$STUB_CLI")
-FAKE_PLATFORM_ROOT="$HAPPY_PROOT" HOOK_STDIN=$(valid_envelope) run_hook
-assert_exit "T6: happy path exits 0" 0
-assert_contains_line "T6: happy path emits trigger" "trigger"
-assert_contains_line "T6: happy path emits result=ok" "result=ok"
-
-# T7: dedupe — stub returns turns=0 skipped=3
-DEDUP_PROOT=$(make_platform_with_cli "$STUB_CLI_DEDUP")
-FAKE_PLATFORM_ROOT="$DEDUP_PROOT" HOOK_STDIN=$(valid_envelope) run_hook
-assert_exit "T7: dedupe exits 0" 0
-assert_contains_line "T7: dedupe emits result=ok" "result=ok"
-
-# T8: CLI error — stub exits 1
-ERR_PROOT=$(make_platform_with_cli "$STUB_CLI_ERR")
-FAKE_PLATFORM_ROOT="$ERR_PROOT" HOOK_STDIN=$(valid_envelope) run_hook
-assert_exit "T8: cli-error exits 0" 0
-assert_contains_line "T8: cli-error emits result=err" "result=err"
-
-# ---- summary ---------------------------------------------------------------
-echo ""
-if [[ "$FAIL" -eq 0 ]]; then
-  echo "[voice-session-end-text.test] all $PASS tests passed"
-  exit 0
-else
-  echo "[voice-session-end-text.test] FAILED — $FAIL/$((PASS + FAIL)) tests failed"
-  exit 1
-fi

package/payload/premium-plugins/writer-craft/hooks/voice-session-end-text.sh

@@ -1,75 +0,0 @@
-#!/usr/bin/env bash
-# SessionEnd hook — auto-ingests operator PTY turns as :Message {format:'text'}
-# corpus nodes in the writer-craft voice corpus.
-#
-# Fired by Claude Code after each PTY session terminates. Receives the full
-# session envelope from stdin:
-#   { "session_id": "...", "transcript_path": "...", "hook_event_name": "SessionEnd", ... }
-#
-# Guards (shared logic via admin-graph-pass-common.sh):
-#   - role != admin     → exit 0 (trigger-skipped reason=role-not-admin)
-#   - MAXY_SPECIALIST set → exit 0 (trigger-skipped reason=is-specialist)
-#   - empty stdin       → exit 0 (trigger-skipped reason=empty-stdin)
-#   - missing transcript → exit 0 (trigger-skipped reason=missing-transcript)
-#
-# Additional guard:
-#   - CLI not found     → exit 0 (trigger-skipped reason=cli-not-found)
-#     Graceful: accounts without writer-craft installed skip silently.
-#
-# Environment (inherited from PTY spawn — set by session manager systemd unit):
-#   PLATFORM_ROOT    — used to locate the admin hooks lib and the CLI
-#   ACCOUNT_ID       — operator's account identifier
-#   USER_ID          — operator's userId
-#   SESSION_ID       — parsed from stdin envelope by agp_parse_stdin
-#   NEO4J_URI        — bolt URI (propagated from systemd unit env)
-#   NEO4J_USER       — neo4j username
-#   NEO4J_PASSWORD   — neo4j password
-#
-# Observability:
-#   [voice-session-end-hook] trigger-skipped sessionId=<id> reason=<reason>
-#   [voice-session-end-hook] trigger sessionId=<id>
-#   [voice-session-end-hook] result=ok sessionId=<id> turns=<n> skipped=<m>
-#   [voice-session-end-hook] result=err sessionId=<id> reason=<detail>
-
-set -uo pipefail
-
-AGP_LOG_TAG="voice-session-end-hook"
-
-# Locate the shared plumbing library via PLATFORM_ROOT (set at PTY spawn).
-# This is more robust than a relative path from the hook's own directory.
-AGP_LIB="${PLATFORM_ROOT:-}/plugins/admin/hooks/lib/admin-graph-pass-common.sh"
-if [ ! -f "$AGP_LIB" ]; then
-  echo "[${AGP_LOG_TAG}] trigger-skipped sessionId=unknown reason=missing-helper path=${AGP_LIB}" >&2
-  exit 0
-fi
-# shellcheck source=../../../platform/plugins/admin/hooks/lib/admin-graph-pass-common.sh
-source "$AGP_LIB"
-
-agp_setup_logging
-agp_read_stdin
-agp_parse_stdin
-agp_guard_role_specialist
-agp_guard_transcript require-stdin
-
-# Locate the ingest CLI. Built alongside the MCP server — lives in dist/cli/.
-CLI_PATH="${PLATFORM_ROOT:-}/../premium-plugins/writer-craft/mcp/dist/cli/ingest-session-text.js"
-if [ ! -f "$CLI_PATH" ]; then
-  agp_emit_log "trigger-skipped sessionId=${SESSION_ID_REPORTED} reason=cli-not-found path=${CLI_PATH}"
-  exit 0
-fi
-
-agp_emit_log "trigger sessionId=${SESSION_ID_REPORTED}"
-
-# Invoke the CLI. All required context is in the inherited env:
-#   ACCOUNT_ID, USER_ID, NEO4J_URI, NEO4J_USER, NEO4J_PASSWORD
-# SESSION_ID is passed explicitly since agp_parse_stdin sets it from stdin.
-result=$(SESSION_ID="${SESSION_ID}" node "$CLI_PATH" "$TRANSCRIPT_PATH" 2>&1)
-exit_code=$?
-
-if [ "$exit_code" -eq 0 ]; then
-  agp_emit_log "result=ok sessionId=${SESSION_ID_REPORTED} ${result}"
-else
-  agp_emit_log "result=err sessionId=${SESSION_ID_REPORTED} reason=${result}"
-fi
-
-exit 0

package/payload/premium-plugins/writer-craft/mcp/src/cli/ingest-session-text.ts

@@ -1,65 +0,0 @@
-#!/usr/bin/env node
-/**
- * CLI wrapper for voice-ingest-session-text.
- *
- * Invoked by the SessionEnd hook after each admin PTY session:
- *   node dist/cli/ingest-session-text.js <transcript_path>
- *
- * All other parameters are read from environment variables inherited from
- * the PTY session (which inherits from the session manager's systemd unit):
- *
- *   ACCOUNT_ID    — operator's account identifier
- *   USER_ID       — operator's userId (= adminUserId)
- *   SESSION_ID    — session identifier (parsed by hook from stdin)
- *   NEO4J_URI     — neo4j bolt URI
- *   NEO4J_USER    — neo4j username
- *   NEO4J_PASSWORD — neo4j password
- *
- * Exit codes:
- *   0 — success (includes no-new-turns case)
- *   1 — error (missing required env, JSONL not found, DB error)
- *
- * stdout: one-line result summary ("turns=N skipped=M") for the hook to
- * capture and forward to the log-ingest endpoint.
- * stderr: detailed diagnostics (tool log lines).
- */
-import { voiceIngestSessionText } from "../tools/voice-ingest-session-text.js";
-import { closeDriver } from "../lib/neo4j.js";
-
-const transcriptPath = process.argv[2];
-if (!transcriptPath) {
-  process.stderr.write("[ingest-session-text] error: transcript path required as argv[2]\n");
-  process.exit(1);
-}
-
-const accountId = process.env.ACCOUNT_ID ?? "";
-const userId = process.env.USER_ID ?? "";
-const sessionId = process.env.SESSION_ID ?? undefined;
-
-if (!accountId || !userId) {
-  process.stderr.write(
-    `[ingest-session-text] error: ACCOUNT_ID and USER_ID must be set in env (accountId=${accountId || "empty"} userId=${userId || "empty"})\n`,
-  );
-  process.exit(1);
-}
-
-try {
-  const result = await voiceIngestSessionText({
-    sessionJsonlPath: transcriptPath,
-    accountId,
-    userId,
-    sessionId,
-  });
-  // Emit one-line summary to stdout for the hook to capture.
-  process.stdout.write(
-    `turns=${result.ingestedCount} skipped=${result.skippedCount} sessionId=${result.sessionId ?? "unknown"}\n`,
-  );
-  process.exit(0);
-} catch (err) {
-  process.stderr.write(
-    `[ingest-session-text] error: ${err instanceof Error ? err.message : String(err)}\n`,
-  );
-  process.exit(1);
-} finally {
-  await closeDriver();
-}