incremnt 0.7.0 → 0.7.1

package/README.md

@@ -67,6 +67,9 @@ incremnt login --session-file ~/Downloads/session.json
 | `increment-score current` | Latest Increment Score summary with components, drivers, trend, and data-quality flags |
 | `increment-score history` | Historical Increment Score snapshots |
 | `increment-score upload --file <file>` | Upload Increment Score snapshots |
+| `coach observations list` | Current persisted coach observations |
+| `coach observations seen --id <id>` | Mark a coach observation as seen |
+| `coach observations dismiss --id <id>` | Dismiss an observation and suppress matching follow-ups |
 | `programs propose --file <file>` | Submit a program proposal |
 | `programs proposals` | List proposals |
 | `programs proposal dismiss --id <id>` | Dismiss a proposal |
@@ -142,7 +145,7 @@ The MCP server uses the same auth session as the CLI — run `incremnt login` fi
 
 The MCP server exposes two tool families:
 
-- Command-shaped tools from the public CLI contract, including sessions, programs, cycles, goals, health, training load, ask history/show, `increment-score-current`, `increment-score-history`, `increment-score-upload`, program proposals, and program shares.
+- Command-shaped tools from the public CLI contract, including sessions, programs, cycles, goals, health, training load, ask history/show, `increment-score-current`, `increment-score-history`, `increment-score-upload`, `coach-observations-current`, program proposals, and program shares.
 - Typed coach read tools for agent-native context retrieval, including `get_increment_score`, `get_recent_sessions`, `get_exercise_history`, `get_next_session`, `get_readiness_snapshot`, `get_body_weight_snapshot`, `get_goal_status`, and `get_records`.
 
 `get_increment_score` returns the same privacy-safe score summary as `increment-score current`: score, snapshot timestamp, formula version, data tier, component scores, positive/negative drivers, day-over-day delta, recent trend, and data-quality flags. It does not expose raw HealthKit values.

package/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: incremnt-cli
-description: Query a user's strength training data (sessions, programs, records, Increment Score, health vitals) from the incremnt iOS app via CLI or MCP.
+description: Query a user's strength training data (sessions, programs, records, Increment Score, coach observations, health vitals) from the incremnt iOS app via CLI or MCP.
 binary: incremnt
 mcp_binary: incremnt-mcp
 contract_command: incremnt contract
@@ -20,7 +20,7 @@ contract_command: incremnt contract
 
 ## Write-flow commands
 
-Five commands mutate state. Two additional lookup commands are listed in `writeSchema` because they support write flows. All require an authenticated session.
+Seven commands mutate state. Two additional lookup commands are listed in `writeSchema` because they support write flows. All require an authenticated session.
 
 Append `--dry-run` to any mutating command to preview the HTTP request as `{ dryRun: true, command, request: { method, url, body } }` without sending it. Prefer this before any irreversible action.
 
@@ -33,6 +33,8 @@ Append `--dry-run` to any mutating command to preview the HTTP request as `{ dry
 | `programs share list --program-id <id>` | Read-only lookup | Use before revoking a share. `share-id` is not the public token. |
 | `programs share revoke --share-id <id>` | Irreversible | `share-id` is from `programs share list`, not the public token. |
 | `increment-score upload --file <f>` | Overwrites by timestamp | File shape: `{ snapshots: [...] }`. Same `snapshot_at` replaces prior data. |
+| `coach observations seen --id <id>` | Reversible by later lifecycle updates | IDs come from `coach observations list`. CLI/API-only; not exposed as an MCP write. |
+| `coach observations dismiss --id <id>` | Suppresses matching follow-ups for the server window | IDs come from `coach observations list`. CLI/API-only; not exposed as an MCP write. |
 
 Confirm with the user before any mutating command unless they explicitly authorised the specific action.
 
@@ -42,6 +44,7 @@ Confirm with the user before any mutating command unless they explicitly authori
 - `records` is the cheapest way to discover the canonical exercise names a user has actually trained. Use it before composing a `programs propose` payload.
 - `exercises history --name "Bench Press"` uses canonical synonym matching — it finds `Barbell Bench Press` without pulling in incline/dumbbell variants.
 - `increment-score current` returns a privacy-safe summary only (score, components, drivers, trend, data-quality flags) — no raw HealthKit values.
+- `coach observations list` returns persisted coach insights generated from training patterns. MCP exposes this read surface as `coach-observations-current`; seen/dismiss writes stay CLI/API-only.
 - `health summary --days <n>` and `training load` give physiological context (resting HR, HRV, ATL/CTL/TSB).
 
 ## Input validation (already enforced)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "incremnt",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "Command-line tool for querying your incremnt strength training data",
   "license": "MIT",
   "type": "module",

package/src/contract.js

@@ -1,4 +1,4 @@
-export const contractVersion = 14;
+export const contractVersion = 15;
 
 export const capabilities = {
   readOnly: false,
@@ -203,6 +203,16 @@ export const commandSchema = [
       { name: 'to', type: 'string', required: false, description: 'Latest snapshot_at (ISO 8601)' },
       { name: 'limit', type: 'number', required: false, description: 'Max snapshots to return (default 200, max 1000)' }
     ]
+  },
+  {
+    command: 'coach observations list',
+    id: 'coach-observations-current',
+    description: 'List current persisted coach observations',
+    supportsFields: true,
+    agentNotes: 'Read-only persisted insight surface. Returns generated/seen observations with evidence and lifecycle status. Use IDs from this command for CLI seen/dismiss actions.',
+    options: [
+      { name: 'limit', type: 'number', required: false, description: 'Max observations to return (default 5, max 20)' }
+    ]
   }
 ];
 
@@ -280,6 +290,31 @@ export const writeCommandSchema = [
     options: [
       { name: 'file', type: 'string', format: 'path', required: true, description: 'Path to JSON file with { snapshots: [...] }' }
     ]
+  },
+  {
+    command: 'coach observations seen',
+    id: 'coach-observations-seen',
+    description: 'Mark a coach observation as seen',
+    usage: 'coach observations seen --id <observation-id>',
+    dryRun: true,
+    mcp: false,
+    agentNotes: 'Mutation. IDs come from coach observations list. This is intentionally CLI/API-only for now. Pass --dry-run to preview the request without sending.',
+    options: [
+      { name: 'id', type: 'string', format: 'id', required: true, description: 'Observation ID' },
+      { name: 'seenAt', type: 'string', required: false, description: 'Optional ISO timestamp to record as seenAt' }
+    ]
+  },
+  {
+    command: 'coach observations dismiss',
+    id: 'coach-observations-dismiss',
+    description: 'Dismiss a coach observation and suppress matching follow-ups',
+    usage: 'coach observations dismiss --id <observation-id>',
+    dryRun: true,
+    mcp: false,
+    agentNotes: 'Mutation. IDs come from coach observations list. Creates the server-side suppression window for the same observation pattern. This is intentionally CLI/API-only for now. Pass --dry-run to preview the request without sending.',
+    options: [
+      { name: 'id', type: 'string', format: 'id', required: true, description: 'Observation ID' }
+    ]
   }
 ];
 

package/src/format.js

@@ -744,6 +744,36 @@ function formatIncrementScoreUpload(payload) {
   return ` Uploaded ${chalk.bold(String(inserted))} Increment Score snapshot${inserted === 1 ? '' : 's'}.`;
 }
 
+function formatCoachObservationsCurrent(payload) {
+  const observations = payload?.observations;
+  if (!Array.isArray(observations) || observations.length === 0) {
+    return 'No current coach observations found.';
+  }
+
+  const lines = [header('COACH OBSERVATIONS'), ''];
+  for (const observation of observations) {
+    const status = observation.status ? `${dimDot()}${chalk.dim(observation.status)}` : '';
+    const confidenceValue = Number(observation.confidence);
+    const confidence = Number.isFinite(confidenceValue)
+      ? `${dimDot()}${chalk.dim(`confidence ${confidenceValue.toFixed(2)}`)}`
+      : '';
+    lines.push(` ${chalk.bold(observation.title ?? observation.kind ?? 'Observation')}${status}${confidence}`);
+    if (observation.summary) lines.push(`   ${observation.summary}`);
+    if (observation.actionText) lines.push(`   ${chalk.dim('Action')} ${observation.actionText}`);
+    if (observation.id) lines.push(`   ${chalk.dim(observation.id)}`);
+    lines.push('');
+  }
+
+  while (lines.at(-1) === '') lines.pop();
+  return lines.join('\n');
+}
+
+function formatCoachObservationMutation(payload) {
+  const observation = payload?.observation;
+  if (!observation) return 'Coach observation not found.';
+  return ` Coach observation ${chalk.bold(observation.id)} is ${chalk.bold(observation.status)}.`;
+}
+
 // --- Main export ---
 
 export function formatHelp(opts = {}) {
@@ -815,7 +845,10 @@ export function formatPretty(command, payload) {
     'proposal-dismiss': formatProposalDismissed,
     'increment-score-current': formatIncrementScoreCurrent,
     'increment-score-history': formatIncrementScoreHistory,
-    'increment-score-upload': formatIncrementScoreUpload
+    'increment-score-upload': formatIncrementScoreUpload,
+    'coach-observations-current': formatCoachObservationsCurrent,
+    'coach-observations-seen': formatCoachObservationMutation,
+    'coach-observations-dismiss': formatCoachObservationMutation
   }[command];
 
   return formatter ? formatter(payload) : null;

package/src/mcp.js

@@ -68,7 +68,7 @@ export function registerMcpTools(server, {
   readSessionStateFn = readSessionState,
   createTransportFn = createTransport
 } = {}) {
-  for (const cmd of [...commandSchema, ...writeCommandSchema]) {
+  for (const cmd of [...commandSchema, ...writeCommandSchema].filter((entry) => entry.mcp !== false)) {
     const description = cmd.agentNotes
       ? `${cmd.description}\n\nNotes for agents:\n${cmd.agentNotes}`
       : cmd.description;

package/src/queries.js

@@ -3610,7 +3610,42 @@ function askToolMetadata(tools = [], provenance = []) {
   };
 }
 
-export function askRoutedContext(snapshot, question, { exclude = new Set(), coachFacts = null } = {}) {
+function appendCoachObservationsContextBeforeExcludeNote(lines, observations, exclude = new Set()) {
+  if (exclude.has('coach_observations')) return [];
+  const usable = (Array.isArray(observations) ? observations : [])
+    .filter((observation) => observation?.id && observation?.summary)
+    .slice(0, 3);
+  if (usable.length === 0) return [];
+
+  const note = buildExcludeNote(exclude);
+  const noteAtEnd = note && lines.at(-1) === note;
+  if (noteAtEnd) {
+    lines.pop();
+    if (lines.at(-1) === '') lines.pop();
+  }
+  const section = [
+    '',
+    'Coach observations (derived from training data, not user-stated facts):'
+  ];
+  for (const observation of usable) {
+    const parts = [
+      `[${observation.kind ?? 'observation'}] ${observation.summary}`,
+      observation.actionText ? `Action: ${observation.actionText}` : null,
+      observation.sourceComponent ? `component=${observation.sourceComponent}` : null,
+      `confidence=${Number(observation.confidence ?? 0).toFixed(2)}`,
+      `observation-id=${observation.id}`
+    ].filter(Boolean);
+    section.push(`  ${parts.join(' ')}`);
+  }
+  lines.push(...section);
+  if (noteAtEnd) {
+    lines.push('');
+    lines.push(note);
+  }
+  return usable.map((observation) => observation.id);
+}
+
+export function askRoutedContext(snapshot, question, { exclude = new Set(), coachFacts = null, coachObservations = null } = {}) {
   const { route, namedExercises } = routeAskQuestion(snapshot, question);
   let effectiveRoute = route;
   let fallbackRoute = null;
@@ -3657,6 +3692,7 @@ export function askRoutedContext(snapshot, question, { exclude = new Set(), coac
   const factLines = built.context.split('\n');
   const includedFacts = rankedCoachFactsForAsk(snapshot, question, effectiveRoute, { facts: coachFacts });
   const includedCoachFactIds = appendCoachFactsContextBeforeExcludeNote(factLines, includedFacts, exclude);
+  const includedCoachObservationIds = appendCoachObservationsContextBeforeExcludeNote(factLines, coachObservations, exclude);
   const includedCoachFactKinds = uniqueArray(includedFacts.map((fact) => fact.kind));
   const includedCoachFactSources = uniqueArray(includedFacts.map((fact) => {
     const sourceSessionId = String(fact.sourceSessionId ?? '');
@@ -3666,7 +3702,11 @@ export function askRoutedContext(snapshot, question, { exclude = new Set(), coac
   }).filter(Boolean));
   built = {
     context: factLines.join('\n'),
-    sections: includedFacts.length > 0 ? [...built.sections, 'coach_facts'] : built.sections
+    sections: [
+      ...built.sections,
+      ...(includedFacts.length > 0 ? ['coach_facts'] : []),
+      ...(includedCoachObservationIds.length > 0 ? ['coach_observations'] : [])
+    ]
   };
 
   return {
@@ -3683,6 +3723,8 @@ export function askRoutedContext(snapshot, question, { exclude = new Set(), coac
       coachFactIds: includedCoachFactIds,
       coachFactKinds: includedCoachFactKinds,
       coachFactSources: includedCoachFactSources,
+      includedCoachObservationIds,
+      coachObservationIds: includedCoachObservationIds,
       contextCharCount: built.context.length,
       ...toolMetadata
     }

package/src/remote.js

@@ -42,7 +42,8 @@ const remoteCommandHandlers = {
   'ask-show': executeRemoteRead,
   'program-share-fetch': executeRemoteRead,
   'increment-score-current': executeRemoteRead,
-  'increment-score-history': executeRemoteRead
+  'increment-score-history': executeRemoteRead,
+  'coach-observations-current': executeRemoteRead
 };
 
 async function executeRemoteRead(options, sessionState, normalizedCommand) {
@@ -175,6 +176,11 @@ function endpointForCommand(baseUrl, normalizedCommand, options) {
       if (options.limit) url.searchParams.set('limit', options.limit);
       return url;
     }
+    case 'coach-observations-current': {
+      const url = resolveServiceUrl(baseUrl, '/cli/coach-observations/current');
+      if (options.limit) url.searchParams.set('limit', options.limit);
+      return url;
+    }
     default:
       return resolveServiceUrl(baseUrl, '/');
   }
@@ -321,6 +327,32 @@ export async function buildWriteRequest(commandId, options, sessionState) {
         body
       };
     }
+    case 'coach-observations-seen': {
+      if (!options.id) {
+        const error = new Error('--id is required for coach observations seen.');
+        error.code = 'MISSING_OPTION';
+        throw error;
+      }
+      const body = {};
+      if (options.seenAt) body.seenAt = options.seenAt;
+      return {
+        method: 'POST',
+        url: resolveServiceUrl(baseUrl, `/cli/coach-observations/${encodeURIComponent(options.id)}/seen`).toString(),
+        body
+      };
+    }
+    case 'coach-observations-dismiss': {
+      if (!options.id) {
+        const error = new Error('--id is required for coach observations dismiss.');
+        error.code = 'MISSING_OPTION';
+        throw error;
+      }
+      return {
+        method: 'POST',
+        url: resolveServiceUrl(baseUrl, `/cli/coach-observations/${encodeURIComponent(options.id)}/dismiss`).toString(),
+        body: {}
+      };
+    }
     default: {
       const error = new Error(`Command ${commandId} does not support --dry-run.`);
       error.code = 'UNSUPPORTED_DRY_RUN';
@@ -553,6 +585,76 @@ const remoteWriteCommandHandlers = {
       throw error;
     }
     return response.json();
+  },
+
+  'coach-observations-seen': async (options, sessionState) => {
+    const baseUrl = sessionState.session?.transport?.baseUrl;
+    if (!baseUrl) throw notImplementedError();
+    if (!options.id) {
+      const error = new Error('--id is required for coach observations seen.');
+      error.code = 'MISSING_OPTION';
+      throw error;
+    }
+
+    const body = {};
+    if (options.seenAt) body.seenAt = options.seenAt;
+    const endpoint = resolveServiceUrl(baseUrl, `/cli/coach-observations/${encodeURIComponent(options.id)}/seen`);
+    const response = await fetch(endpoint, {
+      method: 'POST',
+      headers: {
+        'Content-Type': 'application/json',
+        Authorization: `Bearer ${sessionState.session?.auth?.accessToken ?? ''}`
+      },
+      body: JSON.stringify(body)
+    });
+
+    if (response.status === 401 || response.status === 403) throw authenticationFailedError();
+    if (response.status === 404) {
+      const error = new Error(`Coach observation not found: ${options.id}`);
+      error.code = 'REMOTE_NOT_FOUND';
+      throw error;
+    }
+    if (!response.ok) {
+      const payload = await response.json().catch(() => null);
+      const error = new Error(payload?.error ?? `Unexpected error (HTTP ${response.status}).`);
+      error.code = 'REMOTE_HTTP_ERROR';
+      throw error;
+    }
+    return response.json();
+  },
+
+  'coach-observations-dismiss': async (options, sessionState) => {
+    const baseUrl = sessionState.session?.transport?.baseUrl;
+    if (!baseUrl) throw notImplementedError();
+    if (!options.id) {
+      const error = new Error('--id is required for coach observations dismiss.');
+      error.code = 'MISSING_OPTION';
+      throw error;
+    }
+
+    const endpoint = resolveServiceUrl(baseUrl, `/cli/coach-observations/${encodeURIComponent(options.id)}/dismiss`);
+    const response = await fetch(endpoint, {
+      method: 'POST',
+      headers: {
+        'Content-Type': 'application/json',
+        Authorization: `Bearer ${sessionState.session?.auth?.accessToken ?? ''}`
+      },
+      body: JSON.stringify({})
+    });
+
+    if (response.status === 401 || response.status === 403) throw authenticationFailedError();
+    if (response.status === 404) {
+      const error = new Error(`Coach observation not found: ${options.id}`);
+      error.code = 'REMOTE_NOT_FOUND';
+      throw error;
+    }
+    if (!response.ok) {
+      const payload = await response.json().catch(() => null);
+      const error = new Error(payload?.error ?? `Unexpected error (HTTP ${response.status}).`);
+      error.code = 'REMOTE_HTTP_ERROR';
+      throw error;
+    }
+    return response.json();
   }
 };
 

package/src/sync-service.js

@@ -22,6 +22,9 @@ const DEFAULT_RATE_LIMIT_RULES = {
   'weekly-checkin-ack': 30,
   'weekly-checkin-start': 10,
   'weekly-digest-current': 30,
+  'coach-observations-current': 30,
+  'coach-observations-seen': 30,
+  'coach-observations-dismiss': 30,
   'dev-login': 10,
   'device-start': 20,
   'device-poll': 300,
@@ -682,6 +685,11 @@ function createRateLimiter({
   };
 }
 
+function parseLimit(value, { defaultValue, max }) {
+  const parsed = value ? Number.parseInt(value, 10) : defaultValue;
+  return Number.isFinite(parsed) && parsed > 0 ? Math.min(parsed, max) : defaultValue;
+}
+
 function routeRequest(url, method) {
   const pathname = url.pathname;
 
@@ -1055,6 +1063,22 @@ function routeRequest(url, method) {
     return { command: 'weekly-digest-current', options: {} };
   }
 
+  if (pathname === '/cli/coach-observations/current') {
+    return { command: 'coach-observations-current', options: { limit: url.searchParams.get('limit') ?? undefined } };
+  }
+
+  {
+    const coachObservationActionMatch = pathname.match(/^\/cli\/coach-observations\/([^/]+)\/(seen|dismiss)$/);
+    if (coachObservationActionMatch) {
+      return {
+        command: `coach-observations-${coachObservationActionMatch[2]}`,
+        options: {
+          id: decodeURIComponent(coachObservationActionMatch[1])
+        }
+      };
+    }
+  }
+
   if (pathname === '/cli/health/ai') {
     return {
       command: 'health-ai',
@@ -1512,6 +1536,27 @@ async function readJsonBody(request) {
   }
 }
 
+async function readOptionalJsonBody(request) {
+  const chunks = [];
+  let totalSize = 0;
+  for await (const chunk of request) {
+    totalSize += chunk.length;
+    if (totalSize > MAX_BODY_BYTES) {
+      throw new Error('Request body too large.');
+    }
+    chunks.push(chunk);
+  }
+
+  const raw = Buffer.concat(chunks).toString('utf8');
+  if (!raw.trim()) return {};
+
+  try {
+    return JSON.parse(raw);
+  } catch {
+    throw new Error('Invalid JSON in request body.');
+  }
+}
+
 async function readUrlEncodedBody(request) {
   const chunks = [];
   let totalSize = 0;
@@ -1992,6 +2037,9 @@ export function createSyncServiceRequestHandler({
   insertScoreSnapshotsForAccount = null,
   listScoreSnapshotsForAccount = null,
   getCurrentWeeklyScoreDigestForAccount = null,
+  listCurrentCoachObservationsForAccount = null,
+  markCoachObservationSeenForAccount = null,
+  dismissCoachObservationForAccount = null,
   // Social
   social = null,
   onError = null
@@ -3496,6 +3544,92 @@ export function createSyncServiceRequestHandler({
         return;
       }
 
+      if (route.command === 'coach-observations-current') {
+        if (request.method !== 'GET') {
+          methodNotAllowed(response, 'Use GET for /cli/coach-observations/current.');
+          return;
+        }
+        if (!listCurrentCoachObservationsForAccount) {
+          json(response, 503, { error: 'Coach observations not available' });
+          return;
+        }
+        try {
+          const limit = parseLimit(route.options.limit, { defaultValue: 5, max: 20 });
+          const observations = await listCurrentCoachObservationsForAccount(account, {
+            limit
+          });
+          json(response, 200, { observations: observations ?? [] });
+        } catch (err) {
+          console.error('Coach observations read error:', err.message);
+          json(response, 500, { error: 'Failed to read coach observations' });
+        }
+        return;
+      }
+
+      if (route.command === 'coach-observations-seen') {
+        if (request.method !== 'POST') {
+          methodNotAllowed(response, 'Use POST for /cli/coach-observations/:id/seen.');
+          return;
+        }
+        if (!markCoachObservationSeenForAccount) {
+          json(response, 503, { error: 'Coach observations not available' });
+          return;
+        }
+        let body = {};
+        try {
+          body = await readOptionalJsonBody(request);
+        } catch {
+          badRequest(response, 'Invalid request body.');
+          return;
+        }
+        const seenAt = body?.seenAt ? new Date(body.seenAt) : null;
+        if (seenAt && Number.isNaN(seenAt.getTime())) {
+          badRequest(response, 'Invalid seenAt.');
+          return;
+        }
+        try {
+          const observation = await markCoachObservationSeenForAccount(account, route.options.id, { seenAt });
+          if (!observation) {
+            notFound(response, 'Observation not found');
+            return;
+          }
+          json(response, 200, { observation });
+        } catch (err) {
+          console.error('Coach observation seen error:', err.message);
+          json(response, 500, { error: 'Failed to mark observation seen' });
+        }
+        return;
+      }
+
+      if (route.command === 'coach-observations-dismiss') {
+        if (request.method !== 'POST') {
+          methodNotAllowed(response, 'Use POST for /cli/coach-observations/:id/dismiss.');
+          return;
+        }
+        if (!dismissCoachObservationForAccount) {
+          json(response, 503, { error: 'Coach observations not available' });
+          return;
+        }
+        try {
+          await readOptionalJsonBody(request);
+        } catch {
+          badRequest(response, 'Invalid request body.');
+          return;
+        }
+        try {
+          const observation = await dismissCoachObservationForAccount(account, route.options.id, {});
+          if (!observation) {
+            notFound(response, 'Observation not found');
+            return;
+          }
+          json(response, 200, { observation });
+        } catch (err) {
+          console.error('Coach observation dismiss error:', err.message);
+          json(response, 500, { error: 'Failed to dismiss observation' });
+        }
+        return;
+      }
+
       let snapshot;
       try {
         snapshot = loadSnapshotForAccount
@@ -4166,9 +4300,17 @@ export function createSyncServiceRequestHandler({
             history: enrichedSnapshots
           };
         }
+        let coachObservations = [];
+        if (listCurrentCoachObservationsForAccount) {
+          try {
+            coachObservations = await listCurrentCoachObservationsForAccount(account, { limit: 3 }) ?? [];
+          } catch (observationErr) {
+            console.error('Coach observations read error (ask):', observationErr.message);
+          }
+        }
 
         const routedContext = queries.askRoutedContext
-          ? queries.askRoutedContext(snapshot, question, { exclude, coachFacts })
+          ? queries.askRoutedContext(snapshot, question, { exclude, coachFacts, coachObservations })
           : { context: queries.askContext(snapshot, { exclude }), metadata: { route: 'legacy' } };
         const persistedKind = persistedConversation?.kind ?? (conversationId?.startsWith('weekly-checkin:') ? 'weekly-checkin' : 'ask');
         const incrementScorePrelude = formatIncrementScorePrelude(scoreSnapshots);
@@ -4198,7 +4340,8 @@ export function createSyncServiceRequestHandler({
               historyTurnCount: canonicalHistory.filter((m) => m.role === 'user').length,
               coachFactsIncluded: (routedContext.metadata?.includedCoachFactIds ?? []).length > 0,
               coachFactIds: routedContext.metadata?.includedCoachFactIds ?? [],
-              coachFactKinds: routedContext.metadata?.coachFactKinds ?? []
+              coachFactKinds: routedContext.metadata?.coachFactKinds ?? [],
+              coachObservationIds: routedContext.metadata?.includedCoachObservationIds ?? []
             }
           });
 
@@ -4255,6 +4398,15 @@ export function createSyncServiceRequestHandler({
               });
             });
           }
+          if (markCoachObservationSeenForAccount) {
+            for (const observationId of routedContext.metadata?.includedCoachObservationIds ?? []) {
+              try {
+                await markCoachObservationSeenForAccount(account, observationId, {});
+              } catch (seenErr) {
+                console.error('Failed to mark coach observation seen:', seenErr.message);
+              }
+            }
+          }
           const updatedUserTurns = updatedMessages.filter((m) => m.role === 'user').length;
           if (
             persistedKind === 'weekly-checkin' &&