@quantiya/codevibe-claude-plugin 2.0.15 → 2.0.17

package/hooks/permission-request.sh

@@ -104,7 +104,10 @@ if [ -n "$TRANSCRIPT_PATH" ] && [ -f "$TRANSCRIPT_PATH" ]; then
       # Take messages after the real user prompt, filter to unsent assistant text
       .[($startIdx + 1):] |
       .[] |
-      select(.type == "assistant") |
+      # Claude Code's synthetic API-error record (isApiErrorMessage == true, model
+      # "<synthetic>") is NOT assistant text: the daemon's transcript backstop relays
+      # it as a NOTIFICATION (src/transcript-api-error.ts). Same exclusion as stop.sh.
+      select(.type == "assistant" and (.isApiErrorMessage != true)) |
       select(.uuid as $u | $sentList | any(. == $u) | not) |
       {
         uuid: .uuid,

package/hooks/stop.sh

@@ -45,6 +45,9 @@ EVENTS_SENT=0
 # → walker submits → Claude responds with "Got it..." while transcript
 # write is still in flight).
 ASSISTANT_TEXT_SENT=0
+# base64 texts of API-error records seen in this turn (space-separated tokens);
+# the last_assistant_message fallback must never re-send one of them.
+API_ERROR_TEXTS_B64=""
 
 # Track sent message UUIDs to avoid duplicates (shared with PermissionRequest hook)
 SENT_UUIDS_FILE="${CODEVIBE_TMPDIR}/codevibe-claude-sent-uuids-${SESSION_ID}.txt"
@@ -100,11 +103,23 @@ read -r -d '' JQ_FILTER <<'JQEOF' || true
   else
     # Parse sent UUIDs
     ($sent | split("\n") | map(select(. != ""))) as $sentList |
-    # Take messages after the real user prompt
+    # Take messages after the real user prompt. A synthetic API-error record
+    # (isApiErrorMessage == true — Claude Code's own yellow "There's an issue
+    # with the selected model…" line, written with model "<synthetic>") is NOT an
+    # assistant response: the daemon's transcript backstop relays it as a
+    # NOTIFICATION (see src/transcript-api-error.ts), so it must never be sent
+    # here as ASSISTANT_RESPONSE (a Stop after an in-turn fallback retry would
+    # otherwise double-post it as a green assistant bubble).
     .[($startIdx + 1):] |
     .[] |
     select(.type == "assistant") |
     . as $msg |
+    if ($msg.isApiErrorMessage == true) then
+      # Excluded record class: report it (base64 text) so the fallback below can
+      # recognise the same text in last_assistant_message and skip it too.
+      ([$msg.message.content[]? | select(.type == "text") | .text // empty] | join("\n\n")) as $errText |
+      "\($msg.uuid)\tapi_error\t\($errText | @base64)"
+    else
     # Emit text content (base64-encoded to preserve newlines)
     (
       ($msg.uuid) as $uuid |
@@ -123,6 +138,7 @@ read -r -d '' JQ_FILTER <<'JQEOF' || true
       $msg.message.content[]? | select(.type == "tool_use") |
       "\($msg.uuid)\ttool_use\t\(. | tojson | @base64)"
     )
+    end
   end
 JQEOF
 JQ_STDERR=$(mktemp)
@@ -152,6 +168,14 @@ if [ -n "$TRANSCRIPT_EVENTS" ]; then
       continue
     fi
 
+    if [ "$MSG_TYPE" = "api_error" ]; then
+      # Claude Code's synthetic API-error record (isApiErrorMessage). The daemon's
+      # transcript backstop relays it as a NOTIFICATION; never an ASSISTANT_RESPONSE.
+      log "INFO" "Skipping API-error record UUID: $MSG_UUID (relayed by the daemon as NOTIFICATION)"
+      [ -n "$MSG_CONTENT" ] && API_ERROR_TEXTS_B64="$API_ERROR_TEXTS_B64 $MSG_CONTENT"
+      continue
+    fi
+
     if [ "$MSG_TYPE" = "text" ] && [ -n "$MSG_CONTENT" ]; then
       # Decode base64-encoded text content
       DECODED_CONTENT=$(echo "$MSG_CONTENT" | base64 -d 2>/dev/null || echo "$MSG_CONTENT")
@@ -281,6 +305,17 @@ fi
 #      is still in flight) silently swallows the final response.
 if [ "$ASSISTANT_TEXT_SENT" -eq 0 ]; then
   LAST_ASSISTANT_MSG=$(echo "$INPUT" | jq -r '.last_assistant_message // empty')
+  # Never re-send an API-error record's text through the fallback either: if
+  # last_assistant_message equals a record the transcript scan excluded above,
+  # the daemon owns it (NOTIFICATION) and the fallback is suppressed.
+  for ERR_B64 in $API_ERROR_TEXTS_B64; do
+    ERR_TEXT=$(echo "$ERR_B64" | base64 -d 2>/dev/null || true)
+    if [ -n "$ERR_TEXT" ] && [ "$LAST_ASSISTANT_MSG" = "$ERR_TEXT" ]; then
+      log "INFO" "Fallback suppressed: last_assistant_message is the API-error record (relayed by the daemon as NOTIFICATION)"
+      LAST_ASSISTANT_MSG=""
+      break
+    fi
+  done
   if [ -n "$LAST_ASSISTANT_MSG" ] && [ "$LAST_ASSISTANT_MSG" != "null" ]; then
     log "INFO" "Assistant text missing from transcript scan (EVENTS_SENT=$EVENTS_SENT), using last_assistant_message fallback"
     FALLBACK_PAYLOAD=$(jq -n \

package/node_modules/@quantiya/codevibe-core/dist/appsync/appsync-client.d.ts

@@ -3,7 +3,7 @@ import { CreateEventInput, CreateSessionInput, UpdateSessionInput, UpdateEventSt
 import type { PostDecisionAction } from '../orchestration-shell/types';
 import type { InvocationUsageSnapshot } from '../reviewer/token-usage';
 import type { CreateTaskGroupInput, CreateTaskGroupResult, SubmitMergeGateVerdictInput, SubmitMergeGateVerdictResult, SubmitTrackVerificationOutcomeInput, SubmitTrackVerificationOutcomeResult, ClaimGroupDecisionInput, ClaimGroupDecisionResult } from '../local-executor/task-group-types';
-import type { ReviewerVerdict, ReviewerRole, AgentKind as ReviewerAgentKind } from '../reviewer/types';
+import type { ReviewScopeContext, ReviewerVerdict, ReviewerRole, AgentKind as ReviewerAgentKind } from '../reviewer/types';
 import type { PlannerDecision } from '../planner/types';
 /**
  * Dispose a watcher/subscription WebSocket without crashing the process
@@ -171,6 +171,8 @@ export interface ReviewerPromptResult {
     role: ReviewerRole;
     /** Decrypted plaintext rendered prompt (UTF-8) → ReviewerSpec.prompt_template. */
     prompt: string;
+    /** Server-authored P24 scope; prior finding text is redacted because it is already in the encrypted prompt. */
+    reviewScope?: ReviewScopeContext | null;
 }
 /** One reviewer seat in a round's `reviewers[]` (decrypted). */
 export interface TaskReviewSummaryReviewer {
@@ -924,6 +926,8 @@ export declare class AppSyncClient {
         roundNumber: number;
         rawOutput: string;
         usage?: InvocationUsageSnapshot;
+        /** P24 explicit comprehensive-baseline reset for a revise round. */
+        reviewScopeReset?: boolean;
     }, sessionKeyBase64: string): Promise<{
         gateId: string;
         workflowState: string;