@trainheroic-unofficial/athlete-mcp 0.6.3 → 0.6.5

package/dist/server.mjs

@@ -167,6 +167,7 @@ const logSetArgsSchema = z.object({
 		})).min(1)
 	})).min(1)
 });
+logSetArgsSchema.extend({ athleteId: idArgSchema });
 z.looseObject({
 	title: z.string().min(1),
 	param_1_type: z.number().optional(),
@@ -896,6 +897,7 @@ function buildExerciseLogPayload(savedWorkoutSetExerciseId, savedWorkoutSetId, w
 * record so callers can build the set-completion PUT body via `buildSetCompletePayload`.
 */
 function findSavedWorkoutSet(workouts, savedWorkoutSetId) {
+	const available = [];
 	for (const pw of workouts) {
 		const rec = pw;
 		const ssw = isRecord(rec.summarizedSavedWorkout) ? rec.summarizedSavedWorkout : {};
@@ -904,18 +906,29 @@ function findSavedWorkoutSet(workouts, savedWorkoutSetId) {
 		const sets = Array.isArray(sw.workoutSets) ? sw.workoutSets : [];
 		for (const s of sets) {
 			if (!isRecord(s)) continue;
+			const exercises = Array.isArray(s.workoutSetExercises) ? s.workoutSetExercises.filter(isRecord) : [];
 			if (coerceInt(s.id) === savedWorkoutSetId) {
 				const savedWorkoutId = coerceInt(sw.id);
 				if (!savedWorkoutId) continue;
 				return {
 					savedWorkoutId,
-					exercises: Array.isArray(s.workoutSetExercises) ? s.workoutSetExercises.filter(isRecord) : [],
+					exercises,
 					rawSet: s
 				};
 			}
+			const setId = coerceInt(s.id);
+			if (setId !== null) {
+				const exLabels = exercises.map((ex) => {
+					const exId = coerceInt(ex.id);
+					const title = typeof ex.exercise_title === "string" && ex.exercise_title || typeof ex.title === "string" && ex.title || "exercise";
+					return exId === null ? null : `${exId} (${title})`;
+				}).filter((x) => x !== null);
+				available.push(`set ${setId} → exercise ids: ${exLabels.join(", ") || "none"}`);
+			}
 		}
 	}
-	throw new Error(`Saved workout set ${savedWorkoutSetId} not found on this date.`);
+	const hint = available.length > 0 ? ` Saved workout sets present on this date: ${available.join("; ")}. Pass a "set N" id to --set (the savedWorkoutSetId), and the matching "exercise id" as savedWorkoutSetExerciseId in the results.` : " No saved workout is scheduled on this date — log against a date that has a workout (find it via athlete-workouts / coach athlete-workouts).";
+	throw new Error(`Saved workout set ${savedWorkoutSetId} not found on this date.${hint}`);
 }
 /**
 * Build the body for `PUT /1.0/athlete/savedworkoutset/{id}` that marks the set completed.
@@ -972,23 +985,52 @@ function buildSetCompletePayload(rawSet, exerciseIds, complete) {
 * by the respective PUT bodies.
 */
 async function logAthleteSet(client, args) {
-	const { exercises, rawSet } = findSavedWorkoutSet(await fetchAthleteWorkouts(client, args.date, args.date), args.savedWorkoutSetId);
+	return writeSetResults(client, { role: "athlete" }, await fetchAthleteWorkouts(client, args.date, args.date), args.savedWorkoutSetId, args.results);
+}
+/**
+* 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).
+*/
+async function writeSetResults(client, target, workouts, savedWorkoutSetId, results) {
+	const { exercises, rawSet } = findSavedWorkoutSet(workouts, savedWorkoutSetId);
+	const suffix = target.role === "coach" ? `/${target.athleteId}` : "";
+	const extra = target.role === "coach" ? { athleteId: target.athleteId } : {};
 	let exercisesLogged = 0;
-	for (const result of args.results) {
+	for (const result of results) {
 		const ex = exercises.find((e) => coerceInt(e.id) === result.savedWorkoutSetExerciseId);
-		if (!ex) throw new Error(`savedWorkoutSetExerciseId ${result.savedWorkoutSetExerciseId} not found in saved workout set ${args.savedWorkoutSetId}.`);
+		if (!ex) {
+			const valid = exercises.map((e) => {
+				const id = coerceInt(e.id);
+				const title = typeof e.exercise_title === "string" && e.exercise_title || typeof e.title === "string" && e.title || "exercise";
+				return id === null ? null : `${id} (${title})`;
+			}).filter((x) => x !== null);
+			throw new Error(`savedWorkoutSetExerciseId ${result.savedWorkoutSetExerciseId} not found in saved workout set ${savedWorkoutSetId}. Exercises in this set: ${valid.join(", ") || "none"}.`);
+		}
 		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, args.savedWorkoutSetId, workoutSetExerciseId, result.sets);
-		const res = await client.request("PUT", `/1.0/athlete/savedworkoutsetexercise/${result.savedWorkoutSetExerciseId}`, { body });
-		if (!res.ok) throw new Error(`Failed to log exercise ${result.savedWorkoutSetExerciseId} (HTTP ${res.status}).`);
+		const body = {
+			...buildExerciseLogPayload(result.savedWorkoutSetExerciseId, savedWorkoutSetId, workoutSetExerciseId, result.sets),
+			...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}`);
+		}
 		exercisesLogged += 1;
 	}
-	const setBody = buildSetCompletePayload(rawSet, exercises.map((e) => coerceInt(e.id)).filter((n) => n !== null), true);
-	const setRes = await client.request("PUT", `/1.0/athlete/savedworkoutset/${args.savedWorkoutSetId}`, { body: setBody });
-	if (!setRes.ok) throw new Error(`Failed to mark workout set ${args.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: args.savedWorkoutSetId,
+		savedWorkoutSetId,
 		exercisesLogged
 	};
 }
@@ -1293,7 +1335,7 @@ function registerAthleteTrainingTools(server, ctx) {
 }
 //#endregion
 //#region package.json
-var version = "0.6.3";
+var version = "0.6.5";
 //#endregion
 //#region src/server.ts
 async function main() {

package/package.json

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