incremnt 0.7.2 → 0.8.0

package/src/score-context.js

@@ -29,17 +29,24 @@ export function computeScoreBand(score) {
 // Component-keyed action templates surfaced as `recommendedNextActions` for
 // each top-2 negative driver. Keep these short, imperative, single-line.
 const COMPONENT_ACTIONS = {
-  coverage: 'Add the missing muscle groups to your next session to close coverage gaps.',
-  recovery: 'Prioritise sleep and an easier session to let recovery rebound.',
-  stimulus: 'Push closer to productive weekly volume on your lagging muscle groups.',
-  execution: 'Hit your planned sets and reps with cleaner technique next session.',
-  progression: 'Add a small load or rep increase on your main lifts next session.'
+  coverage: 'Add the missing muscle work before adding more of what is already covered.',
+  recovery: 'Go easier next session unless sleep and soreness feel clearly better.',
+  stimulus: 'Put the short muscle groups earlier next time.',
+  execution: 'Follow the planned work more closely next session.',
+  progression: 'Hold jumps until the work looks repeatable.'
 };
 
-const GENERIC_ACTION = 'Address this driver in your next session to lift the score.';
+const GENERIC_ACTION = 'Pick one thing from this note to fix next session.';
 
 function actionForDriver(driver) {
   if (!driver || typeof driver !== 'object') return GENERIC_ACTION;
+  const message = driverDisplayMessage(driver)?.toLowerCase() ?? '';
+  if (message.includes('glute')) return 'Put glutes earlier next time.';
+  if (/\babs\b/.test(message)) return 'Put abs before accessories next time.';
+  if (message.includes('calf') || message.includes('calves')) return 'Put calf work earlier next time.';
+  if (message.includes('missed') && message.includes('session')) return 'Move or shrink one planned day before adding more work.';
+  if (message.includes('dropped') && message.includes('reps')) return 'Start the first set a touch easier, or rest longer before set two.';
+  if (message.includes('hrv') || message.includes('sleep')) return 'Go easier next session unless sleep and HRV have bounced back.';
   const component = typeof driver.component === 'string' ? driver.component : null;
   if (component && COMPONENT_ACTIONS[component]) {
     return COMPONENT_ACTIONS[component];
@@ -123,7 +130,7 @@ export function computeSummaryText(enriched) {
   const topNeg = Array.isArray(enriched.topNegativeDrivers) ? enriched.topNegativeDrivers[0] : null;
   const topNegMessage = driverDisplayMessage(topNeg);
   if (topNegMessage) {
-    parts.push(`Top drag: ${topNegMessage}.`);
+    parts.push(`Main thing: ${topNegMessage}.`);
   }
 
   const firstAction = Array.isArray(enriched.recommendedNextActions) ? enriched.recommendedNextActions[0] : null;

package/src/score-prelude.js

@@ -0,0 +1,113 @@
+// Voice-safe Increment Score context for Ask Coach.
+//
+// Lives in its own module (no server deps) so both the live ask path
+// (sync-service.js) and the eval harness (summary-evals.js) can build the same
+// context the model actually reads.
+//
+// This block is *context the model reads*, not user-facing prose, but the model
+// parrots whatever numbers it is handed. The coach-observation-voice spec
+// (docs/copy/coach-observation-voice.md) puts raw component sub-scores in Tier 1
+// — "never surface, anywhere" — because they are meaningless to a user. So we
+// hand the model the *signal* (which area is weakest/strongest, which direction
+// the score is moving, what the drivers are) without raw component numbers,
+// decimals, or a bare list of daily scores it can dump back verbatim. The
+// overall Score is a named feature, so the rounded headline value is allowed;
+// nothing below it is.
+
+// Maps Increment Score component keys to training-reality phrases. The bare keys
+// (coverage/stimulus/execution/progression) read as engine internals; "recovery"
+// is already plain language. Used only for the weakest/strongest steer.
+export const SCORE_COMPONENT_PHRASES = {
+  coverage: 'muscle-group coverage',
+  stimulus: 'training stimulus',
+  execution: 'set execution',
+  progression: 'progression',
+  recovery: 'recovery'
+};
+
+export function scoreComponentPhrase(name) {
+  return SCORE_COMPONENT_PHRASES[String(name).toLowerCase()] ?? 'another training area';
+}
+
+// True when the user's question is actually about the Increment Score. Only then
+// should the prelude hand the model the bare numeric headline — otherwise the
+// model parrots "your score is 92/100" into answers about sessions, volume, or
+// plans where it was never asked for.
+export function isScoreQuestion(question) {
+  return /\b(?:increment\s+)?score\b/i.test(String(question ?? ''));
+}
+
+export function formatIncrementScorePrelude(snapshots, { question = '' } = {}) {
+  if (!Array.isArray(snapshots) || snapshots.length === 0) return null;
+  const latest = snapshots[0];
+  if (latest == null || typeof latest.score !== 'number') return null;
+
+  const lines = [
+    '[Increment Score — context only. Speak in training reality (recovery, fatigue, consistency, density). Never recite component values, sub-scores, decimals, or daily score numbers. Do not volunteer the overall score number unless the user asked about the score.]'
+  ];
+  // Hand over the numeric headline only when the question is score-related; the
+  // weakest/strongest area and direction below are always safe to provide.
+  if (isScoreQuestion(question)) {
+    lines.push(`- Current: ${Math.round(latest.score)}/100`);
+  }
+
+  // Component NAMES only — which area is dragging the score and which is
+  // carrying it — as training-reality phrases, never the underlying sub-scores
+  // or even the bare component keys (which read as engine internals).
+  if (latest.components && typeof latest.components === 'object') {
+    const scored = [];
+    for (const [name, value] of Object.entries(latest.components)) {
+      const num = typeof value === 'number' ? value : value?.score;
+      if (typeof num === 'number') scored.push({ name, num });
+    }
+    if (scored.length > 0) {
+      const min = Math.min(...scored.map((c) => c.num));
+      const max = Math.max(...scored.map((c) => c.num));
+      // Treat values within a few points as a tie rather than asserting a single
+      // weakest/strongest off an arbitrary order-dependent pick.
+      const TIE = 3;
+      const phrases = (items) => items.map((c) => scoreComponentPhrase(c.name)).join(' and ');
+      if (max - min <= TIE) {
+        lines.push('- Areas: all roughly even right now');
+      } else {
+        const weak = scored.filter((c) => c.num - min <= TIE);
+        const strong = scored.filter((c) => max - c.num <= TIE);
+        lines.push(`- Weakest: ${phrases(weak)}. Strongest: ${phrases(strong)}.`);
+      }
+    }
+  }
+
+  const driverLabels = (list) => {
+    if (!Array.isArray(list) || list.length === 0) return null;
+    return list
+      .slice(0, 3)
+      .map((d) => d?.label ?? d?.id ?? d?.driver)
+      .filter(Boolean)
+      .join('; ');
+  };
+  const positives = driverLabels(latest.topPositiveDrivers);
+  if (positives) lines.push(`- Helping the score: ${positives}`);
+  const negatives = driverLabels(latest.topNegativeDrivers);
+  if (negatives) lines.push(`- Holding the score back: ${negatives}`);
+
+  // Direction words only — no delta number, no daily-score list.
+  if (snapshots.length > 1) {
+    const prior = snapshots[1];
+    if (typeof prior?.score === 'number') {
+      const delta = latest.score - prior.score;
+      const dir = delta > 0 ? 'up' : delta < 0 ? 'down' : 'flat';
+      lines.push(`- Day-over-day: ${dir}`);
+    }
+    const recent = snapshots
+      .slice(0, 7)
+      .map((s) => (typeof s?.score === 'number' ? s.score : null))
+      .filter((s) => s != null);
+    if (recent.length >= 3) {
+      const span = recent[0] - recent[recent.length - 1];
+      const trend = span > 2 ? 'rising' : span < -2 ? 'falling' : 'steady';
+      lines.push(`- Last ${recent.length} days: ${trend}`);
+    }
+  }
+
+  return lines.join('\n');
+}

package/src/service-url.js

@@ -1,3 +1,5 @@
+export const DEFAULT_SERVICE_BASE_URL = 'https://incremnt-sync.onrender.com';
+
 export function resolveServiceUrl(baseUrl, servicePath) {
   const url = new URL(baseUrl);
   const basePath = url.pathname === '/' ? '' : url.pathname.replace(/\/+$/, '');
@@ -5,3 +7,10 @@ export function resolveServiceUrl(baseUrl, servicePath) {
   url.pathname = `${basePath}${suffix}` || '/';
   return url;
 }
+
+export function resolveConfiguredBaseUrl(options = {}, session = null) {
+  if (options['base-url']) return options['base-url'];
+  if ('INCREMNT_BASE_URL' in process.env) return process.env.INCREMNT_BASE_URL || null;
+  if (session?.transport?.baseUrl) return session.transport.baseUrl;
+  return DEFAULT_SERVICE_BASE_URL;
+}