incremnt 0.8.3 → 0.8.5

package/src/sync-service.js

@@ -25,9 +25,11 @@ import { sanitizeHistory, detectSystemPromptLeak, stripXMLTagBlocks } from './pr
 import { enrichScoreSnapshots } from './score-context.js';
 import { extractAskProgramDraft } from './program-draft.js';
 import { extractPlanChangeset } from './plan-changeset.js';
+import { extractProgramScheduleAction } from './program-schedule-action.js';
 
 const MAX_BODY_BYTES = 10 * 1024 * 1024; // 10 MB
 const DEFAULT_RATE_LIMIT_WINDOW_MS = 60_000;
+const COACH_OBSERVATION_DAILY_REFRESH_COOLDOWN_MS = 20 * 60 * 60 * 1000;
 const WEEKLY_CHECKIN_COMPLETION_USER_TURNS = 3;
 const WEEKLY_CHECKIN_RECAP_LOCAL_HOUR = 16;
 const DEFAULT_RATE_LIMIT_RULES = {
@@ -70,6 +72,8 @@ const DEFAULT_RATE_LIMIT_RULES = {
   'program-share-list': 60,
   'program-share-public': 120,
   'program-share-revoke': 30,
+  'workout-share-create': 30,
+  'workout-share-public': 120,
   'mobile-sync-bootstrap': 60,
   'mobile-sync-pull': 120,
   'mobile-sync-push': 60,
@@ -282,6 +286,7 @@ export function sanitizeAskStructuredResponseForStorage(structured) {
   const limitations = askStorageStringArray(structured.limitations, { maxItems: ASK_STRUCTURED_MAX_ITEMS, maxLength: 240 });
   const answerVerification = sanitizeAskAnswerVerificationReceipt(structured.answerVerification);
   const programDraft = sanitizeAskProgramDraftForStorage(structured.programDraft);
+  const programScheduleAction = sanitizeAskProgramDraftForStorage(structured.programScheduleAction);
 
   return {
     ...(answer ? { answer } : {}),
@@ -291,7 +296,8 @@ export function sanitizeAskStructuredResponseForStorage(structured) {
     followUpSuggestions,
     limitations,
     ...(answerVerification ? { answerVerification } : {}),
-    programDraft
+    programDraft,
+    programScheduleAction
   };
 }
 
@@ -645,6 +651,7 @@ export function buildAskInteractionLogPayload({
     followUpSuggestionCount: Array.isArray(structured?.followUpSuggestions) ? structured.followUpSuggestions.length : undefined,
     limitationCount: Array.isArray(structured?.limitations) ? structured.limitations.length : undefined,
     hasProgramDraft: structured?.programDraft != null ? true : undefined,
+    hasProgramScheduleAction: structured?.programScheduleAction != null ? true : undefined,
     askVerificationStatus: answerVerification.status,
     askVerificationRetryCount: typeof answerVerification.retryCount === 'number' ? answerVerification.retryCount : undefined,
     askVerificationDegraded: answerVerification.degraded === true ? true : undefined,
@@ -947,6 +954,38 @@ function coachObservationGenerationMetadata(result, sourceTrigger) {
   };
 }
 
+function skippedCoachObservationGenerationMetadata(skippedReason, sourceTrigger) {
+  return {
+    attempted: false,
+    generated: 0,
+    persisted: 0,
+    skippedReason,
+    sourceTrigger: sourceTrigger ?? null
+  };
+}
+
+function recentDailyRefreshObservation(observations, now = new Date()) {
+  const threshold = now.getTime() - COACH_OBSERVATION_DAILY_REFRESH_COOLDOWN_MS;
+  return (Array.isArray(observations) ? observations : []).some((observation) => {
+    if (observation?.sourceTrigger !== 'daily_refresh') return false;
+    const generatedAt = new Date(observation.generatedAt ?? observation.windowEnd ?? 0).getTime();
+    return Number.isFinite(generatedAt) && generatedAt >= threshold;
+  });
+}
+
+async function coachObservationGenerationSkipReason({
+  account,
+  uploadResult,
+  sourceTrigger,
+  listCurrentCoachObservationsForAccount
+}) {
+  if (sourceTrigger !== 'daily_refresh') return null;
+  if (Number(uploadResult?.inserted ?? 0) <= 0) return 'no_score_snapshot_change';
+  if (typeof listCurrentCoachObservationsForAccount !== 'function') return null;
+  const observations = await listCurrentCoachObservationsForAccount(account, { limit: 20 });
+  return recentDailyRefreshObservation(observations) ? 'recent_daily_refresh' : null;
+}
+
 function normalizeAskCoachObservationFollowUp(value) {
   if (!value || typeof value !== 'object') return null;
   const id = String(value.id ?? '').trim();
@@ -1177,6 +1216,20 @@ function routeRequest(url, method) {
     }
   }
 
+  if (pathname === '/cli/workout-share') {
+    return { command: 'workout-share-create', options: {} };
+  }
+
+  {
+    const workoutSharePublicMatch = pathname.match(/^\/workout-share\/([^/]+)$/);
+    if (workoutSharePublicMatch) {
+      return {
+        command: 'workout-share-public',
+        options: { token: decodeURIComponent(workoutSharePublicMatch[1]) }
+      };
+    }
+  }
+
   {
     const programShareRevokeMatch = pathname.match(/^\/cli\/program-share\/([^/]+)\/revoke$/);
     if (programShareRevokeMatch) {
@@ -1211,6 +1264,16 @@ function routeRequest(url, method) {
     };
   }
 
+  if (pathname === '/cli/programs/history') {
+    return {
+      command: 'program-history',
+      options: {
+        'program-id': url.searchParams.get('program-id') ?? undefined,
+        limit: url.searchParams.get('limit') ?? undefined
+      }
+    };
+  }
+
   const programShowMatch = pathname.match(/^\/cli\/programs\/([^/]+)$/);
   if (programShowMatch) {
     return { command: 'program-detail', options: { id: programShowMatch[1] } };
@@ -2316,6 +2379,8 @@ export function createSyncServiceRequestHandler({
   listProgramSharesForAccount = null,
   readPublicProgramShare = null,
   revokeProgramShareForAccount = null,
+  createWorkoutShareForAccount = null,
+  readPublicWorkoutShare = null,
   updateAnalysisConsentForAccount = null,
   updateDisplayNameForAccount = null,
   saveAskConversationForAccount = null,
@@ -3004,6 +3069,45 @@ export function createSyncServiceRequestHandler({
         }
       }
 
+      if (route.command === 'workout-share-public') {
+        if (request.method !== 'GET') {
+          methodNotAllowed(response, 'Use GET for /workout-share/:token.');
+          return;
+        }
+        if (!readPublicWorkoutShare) {
+          methodNotAllowed(response, 'Workout sharing is not enabled for this service mode.');
+          return;
+        }
+        try {
+          const shared = await readPublicWorkoutShare(route.options.token);
+          if (shared.status === 'not_found') {
+            notFound(response, 'Workout share not found.');
+            return;
+          }
+          if (shared.status === 'revoked' || shared.status === 'expired') {
+            json(response, 410, { error: 'Workout share is no longer available.' });
+            return;
+          }
+          json(response, 200, {
+            ok: true,
+            token: route.options.token,
+            version: shared.share.version,
+            workoutName: shared.share.workoutName,
+            workoutPayload: shared.share.workoutPayload,
+            createdAt: shared.share.createdAt,
+            expiresAt: shared.share.expiresAt
+          });
+          return;
+        } catch (error) {
+          if (error?.message === 'Invalid workout share token.') {
+            badRequest(response, error.message);
+            return;
+          }
+          internalError(response, error, onError);
+          return;
+        }
+      }
+
       if (route.command === 'google-mobile') {
         if (request.method !== 'POST') {
           methodNotAllowed(response, 'Use POST for /auth/google/mobile.');
@@ -3322,18 +3426,33 @@ export function createSyncServiceRequestHandler({
             const result = await insertScoreSnapshotsForAccount(account, body.snapshots);
             if (generateCoachObservationsForAccount) {
               const sourceTrigger = coachObservationSourceTriggerForScoreSnapshots(body.snapshots);
+              let skipReason = null;
               try {
-                const generationResult = await generateCoachObservationsForAccount(account, { sourceTrigger });
-                result.coachObservations = coachObservationGenerationMetadata(generationResult, sourceTrigger);
+                skipReason = await coachObservationGenerationSkipReason({
+                  account,
+                  uploadResult: result,
+                  sourceTrigger,
+                  listCurrentCoachObservationsForAccount
+                });
               } catch (error) {
-                console.error('Coach observation generation after score upload failed:', error.message);
-                result.coachObservations = {
-                  attempted: true,
-                  generated: 0,
-                  persisted: 0,
-                  skippedReason: 'generation_failed',
-                  sourceTrigger
-                };
+                console.error('Coach observation generation skip check failed:', error.message);
+              }
+              if (skipReason) {
+                result.coachObservations = skippedCoachObservationGenerationMetadata(skipReason, sourceTrigger);
+              } else {
+                try {
+                  const generationResult = await generateCoachObservationsForAccount(account, { sourceTrigger });
+                  result.coachObservations = coachObservationGenerationMetadata(generationResult, sourceTrigger);
+                } catch (error) {
+                  console.error('Coach observation generation after score upload failed:', error.message);
+                  result.coachObservations = {
+                    attempted: true,
+                    generated: 0,
+                    persisted: 0,
+                    skippedReason: 'generation_failed',
+                    sourceTrigger
+                  };
+                }
               }
             }
             json(response, 200, result);
@@ -3604,6 +3723,53 @@ export function createSyncServiceRequestHandler({
         }
       }
 
+      if (route.command === 'workout-share-create') {
+        if (request.method !== 'POST') {
+          methodNotAllowed(response, 'Use POST for /cli/workout-share.');
+          return;
+        }
+        if (!createWorkoutShareForAccount) {
+          methodNotAllowed(response, 'Workout sharing is not enabled for this service mode.');
+          return;
+        }
+        const account = connectedWriteAuthenticator
+          ? await connectedWriteAuthenticator(requestToken)
+          : null;
+        if (!account) {
+          if (await rejectInsufficientScopeForReadToken(response, request, requestToken, readAuthenticator)) {
+            return;
+          }
+          unauthorized(response, request);
+          return;
+        }
+        try {
+          const body = await readJsonBody(request);
+          const workoutPayload = body.workoutPayload ?? body;
+          const share = await createWorkoutShareForAccount(account, workoutPayload);
+          json(response, 201, {
+            ok: true,
+            shareId: share.id,
+            tokenHint: share.tokenHint,
+            token: share.token,
+            workoutName: share.workoutName,
+            createdAt: share.createdAt,
+            expiresAt: share.expiresAt,
+            revokedAt: share.revokedAt,
+            version: share.version,
+            link: `${resolvedPublicOrigin}/workout-share/${share.token}`,
+            deepLink: `incremnt://workout-share/${share.token}?base=${encodeURIComponent(resolvedPublicOrigin)}`
+          });
+          return;
+        } catch (error) {
+          if (error?.message === 'Workout share payload is malformed.') {
+            badRequest(response, error.message);
+            return;
+          }
+          internalError(response, error, onError);
+          return;
+        }
+      }
+
       if (route.command === 'program-share-list') {
         if (request.method !== 'GET') {
           methodNotAllowed(response, 'Use GET for /cli/programs/:programId/shares.');
@@ -4024,45 +4190,8 @@ export function createSyncServiceRequestHandler({
           const limit = parseLimit(route.options.limit, { defaultValue: 5, max: 20 });
           let refresh = null;
           if (route.options.refresh === 'morning_open') {
-            const writeAccount = writeAuthenticator
-              ? await writeAuthenticator(requestToken)
-              : requestToken === token
-                ? account
-                : null;
-            if (!writeAccount) {
-              if (await rejectInsufficientScopeForReadToken(response, request, requestToken, readAuthenticator)) {
-                return;
-              }
-              unauthorized(response, request);
-              return;
-            }
             const sourceTrigger = 'daily_refresh';
-            if (generateCoachObservationsForAccount) {
-              try {
-                const generationResult = await generateCoachObservationsForAccount(writeAccount, {
-                  sourceTrigger,
-                  refresh: route.options.refresh
-                });
-                refresh = coachObservationGenerationMetadata(generationResult, sourceTrigger);
-              } catch (error) {
-                console.error('Coach observation generation on morning open failed:', error.message);
-                refresh = {
-                  attempted: true,
-                  generated: 0,
-                  persisted: 0,
-                  skippedReason: 'generation_failed',
-                  sourceTrigger
-                };
-              }
-            } else {
-              refresh = {
-                attempted: false,
-                generated: 0,
-                persisted: 0,
-                skippedReason: 'generation_unavailable',
-                sourceTrigger
-              };
-            }
+            refresh = skippedCoachObservationGenerationMetadata('morning_open_read_only', sourceTrigger);
           }
           const observations = await listCurrentCoachObservationsForAccount(account, {
             limit
@@ -5264,7 +5393,12 @@ export function createSyncServiceRequestHandler({
               canonicalizeExerciseName: canonicalExerciseName
             });
             const changesetParsed = extractPlanChangeset(parsed.answerText);
+            const scheduleActionParsed = extractProgramScheduleAction(changesetParsed.answerText, {
+              expectedStartDate: routedContext.metadata?.programScheduleActionStartDate ?? null,
+              requireTrailing: true
+            });
             const requestedPlanAdjustment = requestedCoachObservation?.intent === 'plan_adjustment';
+            const requestedProgramScheduleAction = routedContext.metadata?.intent?.requestedAction === 'schedule_program_action';
             const draft = missingRequestedCoachObservation && requestedCoachObservation?.intent === 'successor_plan'
               ? undefined
               : parsed.programDraft;
@@ -5273,8 +5407,11 @@ export function createSyncServiceRequestHandler({
             const changeset = !requestedPlanAdjustment || missingRequestedCoachObservation
               ? undefined
               : changesetParsed.planChangeset;
-            const answer = stripXMLTagBlocks(changesetParsed.answerText);
-            return { askResult: result, programDraft: draft, planChangeset: changeset, assistantAnswer: answer };
+            const programScheduleAction = requestedProgramScheduleAction
+              ? scheduleActionParsed.programScheduleAction ?? undefined
+              : undefined;
+            const answer = stripXMLTagBlocks(scheduleActionParsed.answerText);
+            return { askResult: result, programDraft: draft, planChangeset: changeset, programScheduleAction, assistantAnswer: answer };
           };
 
           let attempt = await generateAttempt(ctx);
@@ -5384,7 +5521,8 @@ export function createSyncServiceRequestHandler({
                 ...attempt,
                 assistantAnswer: safeAskVerificationFallback(),
                 programDraft: undefined,
-                planChangeset: undefined
+                planChangeset: undefined,
+                programScheduleAction: undefined
               };
             }
           }
@@ -5410,7 +5548,7 @@ export function createSyncServiceRequestHandler({
             });
           }
 
-          const { askResult, assistantAnswer, programDraft, planChangeset } = attempt;
+          const { askResult, assistantAnswer, programDraft, planChangeset, programScheduleAction } = attempt;
           const promptSurface = askResult.promptSurface
             ?? (persistedKind === 'weekly-checkin' ? 'weekly-checkin' : 'ask');
           const promptVersion = askResult.promptVersion
@@ -5424,7 +5562,7 @@ export function createSyncServiceRequestHandler({
             ...(programDraftRetryCount > 0 ? { programDraftRetryCount } : {}),
             ...(planChangesetRetryCount > 0 ? { planChangesetRetryCount } : {})
           };
-          const structured = buildAskStructuredResponse(assistantAnswer, routingMetadata, { programDraft, planChangeset, question });
+          const structured = buildAskStructuredResponse(assistantAnswer, routingMetadata, { programDraft, planChangeset, programScheduleAction, question });
           console.log(`ask-coach-meta ${JSON.stringify(buildAskInteractionLogPayload({
             accountId: account.id,
             status: 'ok',
@@ -5527,7 +5665,8 @@ export function createSyncServiceRequestHandler({
             metadata,
             structured,
             programDraft,
-            planChangeset
+            planChangeset,
+            programScheduleAction
           });
         } catch (err) {
           console.error('AI ask error:', err.message);