@trainheroic-unofficial/athlete-mcp 1.1.1 → 1.3.0

package/dist/server.mjs

@@ -173,6 +173,11 @@ const logSetArgsSchema = z.object({
 	})).min(1)
 });
 logSetArgsSchema.extend({ athleteId: idArgSchema });
+logSetArgsSchema.extend({ athleteId: idArgSchema });
+z.object({
+	savedWorkoutSetExerciseId: idArgSchema,
+	exerciseId: idArgSchema
+});
 /**
 * Args for logging a whole session by exercise (rather than by saved-workout-set id). Each
 * exercise carries its entered sets and an optional 1-based `order`. The athlete path creates
@@ -891,29 +896,37 @@ function summarizeAthleteWorkouts(list) {
 }
 const MAX_PARAM_SLOTS = 10;
 /**
-* Build the body for `PUT /1.0/athlete/savedworkoutsetexercise/{id}`. The body uses
-* snake_case keys matching the live API response shape. Each set slot (1-10) gets a
-* `param_N_made` flag (1 if data is present, 0 otherwise) and `param_1_data_N` /
-* `param_2_data_N` string values.
+* Build the body for `PUT /1.0/{role}/savedworkoutsetexercise/{id}`. The body uses snake_case
+* keys matching the live API response shape. Each set slot (1-10) carries `param_1_data_N` /
+* `param_2_data_N` string values plus a `param_N_made` flag.
+*
+* `mode` selects which write this is — the same endpoint serves both:
+*   - `"log"`: the values ARE a performed result, so `param_N_made` is 1 where the slot has data
+*     and the exercise `completed` flag is 1 when any set has data.
+*   - `"prescribe"`: the values are prescribed targets, written with every `param_N_made` and
+*     `completed` left at 0 so the set is not marked done. This matches what the app sends when a
+*     coach edits an athlete's prescribed reps/weight.
 *
 * Only `savedWorkoutSetExerciseId`, `savedWorkoutSetId`, and `workoutSetExerciseId` are
 * required from the live exercise record; everything else is derived from `results`.
 *
-* Exported for unit testing — callers should use `logAthleteSet` instead.
+* Exported for unit testing — callers should use `logAthleteSet` / `prescribeForAthlete` instead.
 */
-function buildExerciseLogPayload(savedWorkoutSetExerciseId, savedWorkoutSetId, workoutSetExerciseId, results) {
+function buildExerciseSetPayload(savedWorkoutSetExerciseId, savedWorkoutSetId, workoutSetExerciseId, results, mode) {
 	if (results.length > MAX_PARAM_SLOTS) throw new Error(`At most ${MAX_PARAM_SLOTS} sets are supported per exercise; got ${results.length}.`);
+	const performed = mode === "log";
+	const hasData = results.some((s) => s.param1 !== void 0 || s.param2 !== void 0);
 	const body = {
 		id: savedWorkoutSetExerciseId,
 		saved_workout_set_id: savedWorkoutSetId,
 		workout_set_exercise_id: workoutSetExerciseId,
-		completed: results.some((s) => s.param1 !== void 0 || s.param2 !== void 0) ? 1 : 0
+		completed: performed && hasData ? 1 : 0
 	};
 	for (let i = 1; i <= MAX_PARAM_SLOTS; i += 1) {
 		const slot = results[i - 1];
 		const p1 = slot?.param1 !== void 0 ? String(slot.param1) : "";
 		const p2 = slot?.param2 !== void 0 ? String(slot.param2) : "";
-		body[`param_${i}_made`] = p1 !== "" || p2 !== "" ? 1 : 0;
+		body[`param_${i}_made`] = performed && (p1 !== "" || p2 !== "") ? 1 : 0;
 		body[`param_1_data_${i}`] = p1;
 		body[`param_2_data_${i}`] = p2;
 	}
@@ -1014,7 +1027,11 @@ function buildSetCompletePayload(rawSet, exerciseIds, complete) {
 * by the respective PUT bodies.
 */
 async function logAthleteSet(client, args) {
-	return writeSetResults(client, { role: "athlete" }, await fetchAthleteWorkouts(client, args.date, args.date), args.savedWorkoutSetId, args.results);
+	const r = await writeSetResults(client, { role: "athlete" }, await fetchAthleteWorkouts(client, args.date, args.date), args.savedWorkoutSetId, args.results, "log");
+	return {
+		savedWorkoutSetId: r.savedWorkoutSetId,
+		exercisesLogged: r.exercisesWritten
+	};
 }
 /**
 * Coach "Log for Athlete": record set results for a roster athlete on their behalf, via the
@@ -1028,24 +1045,32 @@ async function logAthleteSet(client, args) {
 */
 async function logForAthlete(client, args) {
 	const workouts = await fetchCoachAthleteWorkouts(client, args.athleteId, args.date, args.date);
-	return writeSetResults(client, {
+	const r = await writeSetResults(client, {
 		role: "coach",
 		athleteId: args.athleteId
-	}, workouts, args.savedWorkoutSetId, args.results);
+	}, workouts, args.savedWorkoutSetId, args.results, "log");
+	return {
+		savedWorkoutSetId: r.savedWorkoutSetId,
+		exercisesLogged: r.exercisesWritten
+	};
 }
 /**
-* Shared two-step set-log write behind {@link logAthleteSet} and {@link logForAthlete}.
-* `target` selects the surface: `athlete` writes `/1.0/athlete/...`; `coach` writes
-* `/1.0/coach/...{athleteId}` and stamps `athleteId` into each body. Step 1 PUTs each
-* exercise's entered data to its own endpoint (the only path that actually stores reps and
-* weight). Step 2 marks the set completed; that body needs the app's camelCase in-memory
-* shape and the full list of savedWorkoutSetExercise IDs in the set (not only the logged ones).
+* Shared set-write behind {@link logAthleteSet}, {@link logForAthlete}, and
+* {@link prescribeForAthlete}. `target` selects the surface: `athlete` writes `/1.0/athlete/...`;
+* `coach` writes `/1.0/coach/...{athleteId}` and stamps `athleteId` into each body.
+*
+* Step 1 PUTs each exercise's per-set values to its own endpoint (the only path that actually
+* stores reps and weight). `mode` decides what those values mean: `"log"` records them as a
+* performed result and runs Step 2, which marks the set completed (that body needs the app's
+* camelCase in-memory shape and the full list of savedWorkoutSetExercise IDs in the set, not only
+* the written ones); `"prescribe"` writes them as a prescription and skips Step 2, leaving the set
+* open.
 */
-async function writeSetResults(client, target, workouts, savedWorkoutSetId, results) {
+async function writeSetResults(client, target, workouts, savedWorkoutSetId, results, mode) {
 	const { exercises, rawSet } = findSavedWorkoutSet(workouts, savedWorkoutSetId);
 	const suffix = target.role === "coach" ? `/${target.athleteId}` : "";
 	const extra = target.role === "coach" ? { athleteId: target.athleteId } : {};
-	let exercisesLogged = 0;
+	let exercisesWritten = 0;
 	for (const result of results) {
 		const ex = exercises.find((e) => coerceInt(e.id) === result.savedWorkoutSetExerciseId);
 		if (!ex) {
@@ -1059,25 +1084,27 @@ async function writeSetResults(client, target, workouts, savedWorkoutSetId, resu
 		const workoutSetExerciseId = coerceInt(ex.workout_set_exercise_id);
 		if (!workoutSetExerciseId) throw new Error(`Could not resolve workout_set_exercise_id for exercise ${result.savedWorkoutSetExerciseId}.`);
 		const body = {
-			...buildExerciseLogPayload(result.savedWorkoutSetExerciseId, savedWorkoutSetId, workoutSetExerciseId, result.sets),
+			...buildExerciseSetPayload(result.savedWorkoutSetExerciseId, savedWorkoutSetId, workoutSetExerciseId, result.sets, mode),
 			...extra
 		};
 		const res = await client.request("PUT", `/1.0/${target.role}/savedworkoutsetexercise/${result.savedWorkoutSetExerciseId}${suffix}`, { body });
 		if (!res.ok) {
-			const readOnly = target.role === "coach" && (res.status === 401 || res.status === 403) ? ` Athlete ${target.athleteId} appears to be read-only for results — TrainHeroic's seeded demo/sample athletes return ${res.status} here; results only persist for real (invited) athletes.` : "";
-			throw new Error(`Failed to log exercise ${result.savedWorkoutSetExerciseId} (HTTP ${res.status}).${readOnly}`);
+			const readOnly = target.role === "coach" && (res.status === 401 || res.status === 403) ? ` Athlete ${target.athleteId} appears to be read-only for changes — TrainHeroic's seeded demo/sample athletes return ${res.status} here; writes only persist for real (invited) athletes.` : "";
+			throw new Error(`Failed to write exercise ${result.savedWorkoutSetExerciseId} (HTTP ${res.status}).${readOnly}`);
 		}
-		exercisesLogged += 1;
+		exercisesWritten += 1;
+	}
+	if (mode === "log") {
+		const setBody = {
+			...buildSetCompletePayload(rawSet, exercises.map((e) => coerceInt(e.id)).filter((n) => n !== null), true),
+			...extra
+		};
+		const setRes = await client.request("PUT", `/1.0/${target.role}/savedworkoutset/${savedWorkoutSetId}${suffix}`, { body: setBody });
+		if (!setRes.ok) throw new Error(`Failed to mark workout set ${savedWorkoutSetId} completed (HTTP ${setRes.status}).`);
 	}
-	const setBody = {
-		...buildSetCompletePayload(rawSet, exercises.map((e) => coerceInt(e.id)).filter((n) => n !== null), true),
-		...extra
-	};
-	const setRes = await client.request("PUT", `/1.0/${target.role}/savedworkoutset/${savedWorkoutSetId}${suffix}`, { body: setBody });
-	if (!setRes.ok) throw new Error(`Failed to mark workout set ${savedWorkoutSetId} completed (HTTP ${setRes.status}).`);
 	return {
 		savedWorkoutSetId,
-		exercisesLogged
+		exercisesWritten
 	};
 }
 /**
@@ -1234,6 +1261,7 @@ function presentExerciseHistory(detail) {
 			description: p.description ?? null,
 			reps: p.reps ?? null,
 			weight: p.weight ?? null,
+			units: p.units ?? null,
 			date: p.dateCompleted ?? null
 		})),
 		sessions: (detail.history ?? []).map((h) => ({
@@ -1268,6 +1296,7 @@ function historyInRange(presented, since, until) {
 		sessions
 	};
 }
+z.number().int().positive().max(36).optional();
 //#endregion
 //#region ../core/src/tools/athlete-training.ts
 /** Identity, profile, prefs, working maxes, leaderboard. */
@@ -1521,7 +1550,7 @@ function registerAthleteTrainingTools(server, ctx) {
 }
 //#endregion
 //#region package.json
-var version = "1.1.1";
+var version = "1.3.0";
 //#endregion
 //#region src/server.ts
 async function main() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trainheroic-unofficial/athlete-mcp",
-  "version": "1.1.1",
+  "version": "1.3.0",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -21,8 +21,8 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
     "zod": "^4.4.3",
-    "@trainheroic-unofficial/core": "1.1.1",
-    "@trainheroic-unofficial/js": "1.1.1"
+    "@trainheroic-unofficial/core": "1.3.0",
+    "@trainheroic-unofficial/js": "1.3.0"
   },
   "devDependencies": {
     "@types/node": "^26.0.0",