@orchestrator-claude/definitions 3.15.0 → 3.16.0

package/hooks/gate-guardian.sh

@@ -1,7 +1,9 @@
 #!/bin/bash
 # gate-guardian.sh — ADR-013 Phase 5 Hook (JSON Structured Output)
 # Trigger: PreToolUse on mcp__orchestrator-tools__advancePhase
-# Purpose: Evaluate gate before phase advance. FAIL-CLOSED: blocks when uncertain.
+# Purpose: Guard IMPLEMENT phase advance (requires human approval). ALLOWs all other phases.
+# Note: POST /gate/evaluate endpoint removed (TD-127). Phase transitions are now handled
+# atomically by setPendingAction() via the ping-pong-enforcer hook.
 #
 # Output: JSON with permissionDecision (deny/allow) + additionalContext
 # Exit 0 with JSON = structured decision
@@ -22,6 +24,25 @@ TARGET_PHASE=$(orch_json_field "$STDIN_DATA" "tool_input.targetPhase")
 
 orch_log "GATE-GUARDIAN: workflow=$WORKFLOW_ID targetPhase=$TARGET_PHASE"
 
+# ADR-013 Phase 6: Check workflow mode before gate evaluation
+# quick and interactive modes skip gate evaluation entirely
+if [ -n "$WORKFLOW_ID" ]; then
+  MODE=$(orch_get_workflow_mode "$WORKFLOW_ID" 2>/dev/null) || MODE=""
+  orch_log "GATE-GUARDIAN: workflow=$WORKFLOW_ID mode=${MODE:-legacy}"
+
+  if [ "$MODE" = "quick" ]; then
+    orch_log "GATE-GUARDIAN: ALLOW (quick mode skips gate evaluation)"
+    echo "{\"hookSpecificOutput\":{\"hookEventName\":\"PreToolUse\",\"permissionDecision\":\"allow\",\"additionalContext\":\"Gate to '${TARGET_PHASE}' allowed: quick mode skips gate evaluation.\"}}"
+    exit 0
+  fi
+
+  if [ "$MODE" = "interactive" ]; then
+    orch_log "GATE-GUARDIAN: ALLOW (interactive mode skips gate evaluation, artifact gates not required)"
+    echo "{\"hookSpecificOutput\":{\"hookEventName\":\"PreToolUse\",\"permissionDecision\":\"allow\",\"additionalContext\":\"Gate to '${TARGET_PHASE}' allowed: interactive mode skips artifact gate evaluation.\"}}"
+    exit 0
+  fi
+fi
+
 # FAIL-CLOSED: if we can't parse input, DENY
 if [ -z "$WORKFLOW_ID" ] || [ -z "$TARGET_PHASE" ]; then
   orch_log "GATE-GUARDIAN: DENY (could not parse input)"
@@ -56,24 +77,8 @@ if [ "$TARGET_LOWER" = "implement" ]; then
   fi
 fi
 
-# Evaluate gate via API — FAIL-CLOSED on failure
-GATE_RESULT=$(orch_evaluate_gate "$WORKFLOW_ID" "$TARGET_PHASE" 2>/dev/null) || GATE_RESULT=""
-if [ -z "$GATE_RESULT" ]; then
-  orch_log "GATE-GUARDIAN: DENY (gate evaluation failed)"
-  echo "{\"hookSpecificOutput\":{\"hookEventName\":\"PreToolUse\",\"permissionDecision\":\"deny\",\"permissionDecisionReason\":\"Gate Guardian: Gate evaluation failed for phase '${TARGET_PHASE}'.\",\"additionalContext\":\"Required artifacts may be missing. Complete the current phase before advancing.\"}}"
-  exit 0
-fi
-
-PASSED=$(orch_json_field "$GATE_RESULT" "passed")
-
-if [ "$PASSED" = "true" ]; then
-  orch_log "GATE-GUARDIAN: ALLOW (gate passed for $TARGET_PHASE)"
-  echo "{\"hookSpecificOutput\":{\"hookEventName\":\"PreToolUse\",\"permissionDecision\":\"allow\",\"additionalContext\":\"Gate to '${TARGET_PHASE}' passed.\"}}"
-  exit 0
-fi
-
-# Gate failed
-REASONS=$(orch_json_field "$GATE_RESULT" "reasons")
-orch_log "GATE-GUARDIAN: DENY (gate failed: $REASONS)"
-echo "{\"hookSpecificOutput\":{\"hookEventName\":\"PreToolUse\",\"permissionDecision\":\"deny\",\"permissionDecisionReason\":\"Gate Guardian: Gate to '${TARGET_PHASE}' did not pass.\",\"additionalContext\":\"Reasons: ${REASONS}. Complete current phase requirements before advancing.\"}}"
+# Non-IMPLEMENT phases: ALLOW directly.
+# Phase transitions are handled atomically by setPendingAction() via ping-pong-enforcer.
+orch_log "GATE-GUARDIAN: ALLOW (non-IMPLEMENT phase, no gate evaluation required)"
+echo "{\"hookSpecificOutput\":{\"hookEventName\":\"PreToolUse\",\"permissionDecision\":\"allow\",\"additionalContext\":\"Gate to '${TARGET_PHASE}' allowed.\"}}"
 exit 0

package/hooks/orch-helpers.sh

@@ -100,18 +100,20 @@ orch_get_next_action() {
     -H "X-Project-ID: $PROJECT_ID" 2>/dev/null
 }
 
-# Call evaluateGate for a workflow
-orch_evaluate_gate() {
+# ADR-013 Phase 6: Get the mode of a workflow (quick/standard/full/interactive)
+# Returns the mode string, or empty string for legacy workflows (mode=null)
+# Returns exit code 1 on network/auth error
+orch_get_workflow_mode() {
   local workflow_id="$1"
-  local target_phase="$2"
   local token
   token=$(orch_get_token) || return 1
 
-  curl -sf --max-time 10 -X POST "${API_URL}/api/v1/workflows/${workflow_id}/gate/evaluate" \
-    -H "Content-Type: application/json" \
+  local resp
+  resp=$(curl -sf --max-time 3 "${API_URL}/api/v1/workflows/${workflow_id}/status" \
     -H "Authorization: Bearer $token" \
-    -H "X-Project-ID: $PROJECT_ID" \
-    -d "{\"targetPhase\":\"${target_phase}\"}" 2>/dev/null
+    -H "X-Project-ID: $PROJECT_ID" 2>/dev/null) || return 1
+
+  orch_json_field "$resp" "mode"
 }
 
 # Extract a field from JSON string using node

package/manifest.json

@@ -1,8 +1,8 @@
 {
   "schema": "orchestrator-definitions/v1",
   "namespace": "orchestrator",
-  "version": "3.15.0",
-  "generatedAt": "2026-03-19T13:54:14.745Z",
+  "version": "3.16.0",
+  "generatedAt": "2026-03-19T22:13:52.697Z",
   "definitions": {
     "agents": [
       {
@@ -245,7 +245,7 @@
         "slug": "gate-guardian",
         "file": "hooks/gate-guardian.sh",
         "version": "1.0.0",
-        "checksum": "sha256:7a5add785c53b4040bfde3ba68af8d2f43f65ecc101b600793f6f47a3e411f15",
+        "checksum": "sha256:1631a7be326e1099698f6492caead2f05423582869bba0f69a50754055e1be2d",
         "supersedes": null,
         "metadata": {}
       },
@@ -253,7 +253,7 @@
         "slug": "orch-helpers",
         "file": "hooks/orch-helpers.sh",
         "version": "1.0.0",
-        "checksum": "sha256:0038a622374b5c88b9c674a8fc49744a72752c390db3e45a00948e6676e42959",
+        "checksum": "sha256:906d72e990592bcac675d01c8e8f7ea2df42794b59492992fd8a15975b50608d",
         "supersedes": null,
         "metadata": {}
       },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orchestrator-claude/definitions",
-  "version": "3.15.0",
+  "version": "3.16.0",
   "description": "Built-in definitions (agents, skills, hooks, workflows, KB seeds) for the Orchestrator ecosystem",
   "type": "module",
   "main": "dist/index.js",