@trainheroic-unofficial/athlete-mcp 1.7.1 → 1.7.2

package/dist/server.mjs

@@ -1159,8 +1159,10 @@ function toSetResults(results) {
 * `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 logged data.
+*   - `"log"`: the values ARE a performed result, so `param_N_made` is 1 where the slot has reps
+*     entered (`param1`) and the exercise `completed` flag is 1 when any set is performed. A slot
+*     carrying only a weight (`param2`) with no reps is a target, not a performed set, so it stays
+*     un-made — matching the app, which shows no completion checkmark until reps are logged.
 *   - `"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.
@@ -1205,7 +1207,7 @@ function buildExerciseSetPayload(savedWorkoutSetExerciseId, savedWorkoutSetId, w
 		if (target) {
 			p1 = target.param1 !== void 0 ? String(target.param1) : "";
 			p2 = target.param2 !== void 0 ? String(target.param2) : "";
-			made = performed && (p1 !== "" || p2 !== "") ? 1 : 0;
+			made = performed && p1 !== "" ? 1 : 0;
 		} else if (carryOver && coerceInt(existing?.[`param_${i}_made`]) === 1) {
 			p1 = existingSlotData(existing, `param_1_data_${i}`);
 			p2 = existingSlotData(existing, `param_2_data_${i}`);
@@ -1233,16 +1235,19 @@ function exerciseHasLoggedData(ex) {
 	for (let i = 1; i <= 10; i += 1) if (coerceInt(ex[`param_${i}_made`]) === 1) return true;
 	return false;
 }
-/** True when a result carries at least one non-empty param value (so it produces a performed slot). */
-function resultHasData(result) {
-	return result.sets.some((s) => s.param1 !== void 0 && String(s.param1) !== "" || s.param2 !== void 0 && String(s.param2) !== "");
+/** True when a result carries at least one set with reps entered (so it produces a performed slot).
+* A weight-only set (`param2` set, `param1` blank) is a target, not a performed result, and so does
+* not count — mirroring the `made` rule in {@link buildExerciseSetPayload}. */
+function resultHasPerformedSet(result) {
+	return result.sets.some((s) => s.param1 !== void 0 && String(s.param1) !== "");
 }
 /**
 * Whether every exercise in a saved workout set now has logged data — either written with data in
 * this call (`loggedIds`) or already carrying a performed slot. Gates the set-completion PUT: a
 * superset/circuit stays open until the last exercise is logged, so completing it on a partial log
-* does not flip its still-empty siblings to "done". An exercise written with only empty values does
-* not count (it would not be marked performed), so an all-empty log never completes the set.
+* does not flip its still-empty siblings to "done". An exercise written with no reps — empty values,
+* or a weight (`param2`) with no reps (`param1`) — does not count (it would not be marked performed),
+* so a no-reps log never completes the set.
 */
 function isSetFullyLogged(exercises, loggedIds) {
 	return exercises.every((ex) => {
@@ -1449,7 +1454,7 @@ async function writeSetResults(client, target, workouts, savedWorkoutSetId, resu
 	}
 	let setCompleted = false;
 	if (mode === "log") {
-		if (isSetFullyLogged(exercises, new Set(results.filter(resultHasData).map((r) => r.savedWorkoutSetExerciseId)))) {
+		if (isSetFullyLogged(exercises, new Set(results.filter(resultHasPerformedSet).map((r) => r.savedWorkoutSetExerciseId)))) {
 			const setBody = {
 				...buildSetCompletePayload(rawSet, exercises.map((e) => coerceInt(e.id)).filter((n) => n !== null), true),
 				...extra
@@ -2052,7 +2057,7 @@ function registerAthleteTrainingTools(server, ctx) {
 }
 //#endregion
 //#region package.json
-var version = "1.7.1";
+var version = "1.7.2";
 //#endregion
 //#region src/server.ts
 async function main() {

package/package.json

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