incremnt 0.8.0 → 0.8.2

package/src/sync-service.js

@@ -4,6 +4,7 @@ import { formatIncrementScorePrelude } from './score-prelude.js';
 import {
   askVerificationMetadata,
   buildAskAnswerRepairContext,
+  degradeAskAnswer,
   safeAskVerificationFallback,
   shouldRepairAskAnswer,
   verifyAskAnswer
@@ -602,6 +603,7 @@ export function buildAskInteractionLogPayload({
     fallback: askResult?.fallback === true ? true : undefined,
     route: routingMetadata?.route ?? evidencePlan?.route,
     effectiveRoute: routingMetadata?.effectiveRoute ?? evidencePlan?.effectiveRoute,
+    responseProfile: routingMetadata?.responseProfile ?? routingMetadata?.intent?.responseProfile,
     requestedAction: routingMetadata?.intent?.requestedAction,
     intentConfidence: typeof routingMetadata?.intent?.confidence === 'number' ? routingMetadata.intent.confidence : undefined,
     structuredConfidence: typeof structured?.confidence === 'string' ? structured.confidence : undefined,
@@ -620,6 +622,8 @@ export function buildAskInteractionLogPayload({
     hasProgramDraft: structured?.programDraft != null ? true : undefined,
     askVerificationStatus: answerVerification.status,
     askVerificationRetryCount: typeof answerVerification.retryCount === 'number' ? answerVerification.retryCount : undefined,
+    askVerificationDegraded: answerVerification.degraded === true ? true : undefined,
+    askVerificationRedactedCount: typeof answerVerification.redactedCount === 'number' ? answerVerification.redactedCount : undefined,
     askVerificationBlockingFailureCount: typeof answerVerification.blockingFailureCount === 'number' ? answerVerification.blockingFailureCount : undefined,
     askVerificationAdvisoryFailureCount: typeof answerVerification.advisoryFailureCount === 'number' ? answerVerification.advisoryFailureCount : undefined,
     askVerificationFailureKeys: logStringArray(answerVerification.failureKeys),
@@ -5165,6 +5169,11 @@ export function createSyncServiceRequestHandler({
         const coachObservationFollowUp = selectAskCoachObservationFollowUp(requestedCoachObservation, coachObservations);
         const missingRequestedCoachObservation = Boolean(requestedCoachObservation && !coachObservationFollowUp);
 
+        const persistedKind = persistedConversation?.kind ?? (conversationId?.startsWith('weekly-checkin:') ? 'weekly-checkin' : 'ask');
+        // The weekly check-in shares this ask path but runs under the terse
+        // WEEKLY_CHECKIN_PROMPT; force the defensive profile so the expansive
+        // evidence merge and score headline do not contradict that prompt.
+        const askResponseProfileOverride = persistedKind === 'weekly-checkin' ? 'defensive' : null;
         const routedContext = coachObservationFollowUp
           ? askObservationFollowUpContext(snapshot, question, coachObservationFollowUp, {
               exclude,
@@ -5177,9 +5186,11 @@ export function createSyncServiceRequestHandler({
                 intent: requestedCoachObservation.intent,
                 today: new Date()
               })
-            : askRoutedContext(snapshot, question, { exclude, coachFacts, coachObservations, history: canonicalHistory });
-        const persistedKind = persistedConversation?.kind ?? (conversationId?.startsWith('weekly-checkin:') ? 'weekly-checkin' : 'ask');
-        const incrementScorePrelude = formatIncrementScorePrelude(scoreSnapshots, { question });
+            : askRoutedContext(snapshot, question, { exclude, coachFacts, coachObservations, history: canonicalHistory, responseProfileOverride: askResponseProfileOverride });
+        const incrementScorePrelude = formatIncrementScorePrelude(scoreSnapshots, {
+          question,
+          responseProfile: routedContext.metadata?.responseProfile ?? routedContext.metadata?.intent?.responseProfile
+        });
 
         const preludes = [incrementScorePrelude].filter(Boolean);
         const ctx = preludes.length > 0
@@ -5296,6 +5307,8 @@ export function createSyncServiceRequestHandler({
           let verificationRetryCount = 0;
           let verificationRepaired = false;
           let verificationFallback = false;
+          let verificationDegraded = false;
+          let verificationRedactedCount = 0;
 
           if (persistedKind === 'ask' && shouldRepairAskAnswer(verification)) {
             verificationRetryCount = 1;
@@ -5322,20 +5335,42 @@ export function createSyncServiceRequestHandler({
           }
 
           if (persistedKind === 'ask' && shouldRepairAskAnswer(verification)) {
-            verificationFallback = true;
-            attempt = {
-              ...attempt,
-              assistantAnswer: safeAskVerificationFallback(),
-              programDraft: undefined,
-              planChangeset: undefined
-            };
+            // Graceful degrade before refusing: strip the specific unsupported
+            // sentences/bullets and ship the rest if it re-verifies clean. A
+            // coaching answer minus one clause beats a blanket refusal.
+            const degraded = degradeAskAnswer(attempt.assistantAnswer, verification);
+            const degradedVerification = degraded.usable
+              ? verifyAskAnswer({
+                  answer: degraded.text,
+                  snapshot,
+                  routingMetadata,
+                  today: new Date(),
+                  exclude: [...exclude]
+                })
+              : null;
+            if (degradedVerification && degradedVerification.blockingFailureCount === 0) {
+              attempt = { ...attempt, assistantAnswer: degraded.text };
+              verification = degradedVerification;
+              verificationDegraded = true;
+              verificationRedactedCount = degraded.redactedCount;
+            } else {
+              verificationFallback = true;
+              attempt = {
+                ...attempt,
+                assistantAnswer: safeAskVerificationFallback(),
+                programDraft: undefined,
+                planChangeset: undefined
+              };
+            }
           }
 
           const answerVerification = persistedKind === 'ask'
             ? askVerificationMetadata(verification, {
                 retryCount: verificationRetryCount,
                 repaired: verificationRepaired,
-                fallback: verificationFallback
+                fallback: verificationFallback,
+                degraded: verificationDegraded,
+                redactedCount: verificationRedactedCount
               })
             : undefined;
           if (answerVerification) {