@etweisberg/garmin-connect-mcp 0.1.13 → 0.1.15

package/README.md

@@ -59,51 +59,56 @@ Session cookies expire after a few hours. Re-run the login flow when they do.
 ## Available Tools
 
 ### Session & Auth
-| Tool | Description |
-|------|-------------|
-| `garmin-login` | Returns login instructions for the Playwright MCP browser |
-| `check-session` | Validates the saved session is still active |
-| `run-tests` | Returns a test plan to verify all tools work |
+
+| Tool            | Description                                               |
+| --------------- | --------------------------------------------------------- |
+| `garmin-login`  | Returns login instructions for the Playwright MCP browser |
+| `check-session` | Validates the saved session is still active               |
+| `run-tests`     | Returns a test plan to verify all tools work              |
 
 ### Activities
-| Tool | Description |
-|------|-------------|
-| `list-activities` | List activities with pagination |
-| `get-activity` | Full activity summary (distance, duration, HR, calories) |
-| `get-activity-details` | Time-series metrics (HR, cadence, elevation over time) |
-| `get-activity-splits` | Lap/split data |
-| `get-activity-hr-zones` | Heart rate time-in-zone breakdown |
-| `get-activity-polyline` | Full-resolution GPS track |
-| `get-activity-weather` | Weather conditions during activity |
-| `download-fit` | Download original FIT file |
+
+| Tool                    | Description                                              |
+| ----------------------- | -------------------------------------------------------- |
+| `list-activities`       | List activities with pagination                          |
+| `get-activity`          | Full activity summary (distance, duration, HR, calories) |
+| `get-activity-details`  | Time-series metrics (HR, cadence, elevation over time)   |
+| `get-activity-splits`   | Lap/split data                                           |
+| `get-activity-hr-zones` | Heart rate time-in-zone breakdown                        |
+| `get-activity-polyline` | Full-resolution GPS track                                |
+| `get-activity-weather`  | Weather conditions during activity                       |
+| `download-fit`          | Download original FIT file                               |
 
 ### Daily Health
-| Tool | Description |
-|------|-------------|
-| `get-daily-summary` | Steps, calories, distance, intensity minutes |
-| `get-daily-heart-rate` | Heart rate data throughout the day |
-| `get-daily-stress` | Stress levels throughout the day |
-| `get-daily-summary-chart` | Combined wellness chart data |
-| `get-daily-intensity-minutes` | Intensity minutes for a date |
-| `get-daily-movement` | Movement/activity data |
-| `get-daily-respiration` | Respiration rate data |
+
+| Tool                          | Description                                  |
+| ----------------------------- | -------------------------------------------- |
+| `get-daily-summary`           | Steps, calories, distance, intensity minutes |
+| `get-daily-heart-rate`        | Heart rate data throughout the day           |
+| `get-daily-stress`            | Stress levels throughout the day             |
+| `get-daily-summary-chart`     | Combined wellness chart data                 |
+| `get-daily-intensity-minutes` | Intensity minutes for a date                 |
+| `get-daily-movement`          | Movement/activity data                       |
+| `get-daily-respiration`       | Respiration rate data                        |
 
 ### Sleep / Body Battery / HRV
-| Tool | Description |
-|------|-------------|
-| `get-sleep` | Sleep score, duration, stages, SpO2 |
+
+| Tool               | Description                         |
+| ------------------ | ----------------------------------- |
+| `get-sleep`        | Sleep score, duration, stages, SpO2 |
 | `get-body-battery` | Body battery charged/drained values |
-| `get-hrv` | Heart rate variability data |
+| `get-hrv`          | Heart rate variability data         |
 
 ### Weight / Records / Fitness
-| Tool | Description |
-|------|-------------|
-| `get-weight` | Weight measurements over a date range |
-| `get-personal-records` | All personal records with history |
-| `get-fitness-stats` | Aggregated activity stats by type |
-| `get-vo2max` | Latest VO2 Max estimate |
-| `get-hr-zones-config` | Heart rate zone boundaries |
-| `get-user-profile` | User profile and settings |
+
+| Tool                   | Description                           |
+| ---------------------- | ------------------------------------- |
+| `get-weight`           | Weight measurements over a date range |
+| `get-personal-records` | All personal records with history     |
+| `get-fitness-stats`    | Aggregated activity stats by type     |
+| `get-vo2max`           | Latest VO2 Max estimate               |
+| `get-hr-zones-config`  | Heart rate zone boundaries            |
+| `get-user-profile`     | User profile and settings             |
 
 ## Architecture
 
@@ -141,13 +146,13 @@ npm run build
 
 ### Scripts
 
-| Command | Description |
-|---------|-------------|
-| `npm run build` | Compile TypeScript |
-| `npm run lint` | Run ESLint |
-| `npm run format` | Format with Prettier |
-| `npm run typecheck` | Type check without emitting |
-| `npm test` | Run integration tests (requires valid session) |
+| Command             | Description                                    |
+| ------------------- | ---------------------------------------------- |
+| `npm run build`     | Compile TypeScript                             |
+| `npm run lint`      | Run ESLint                                     |
+| `npm run format`    | Format with Prettier                           |
+| `npm run typecheck` | Type check without emitting                    |
+| `npm test`          | Run integration tests (requires valid session) |
 
 ### Local Integration Testing
 
@@ -159,7 +164,7 @@ npm test
 
 ## Contributing
 
-1. Fork the repo and create a feature branch
+1. Create a feature branch off `main`
 2. Make your changes
 3. Run checks:
    ```bash
@@ -175,20 +180,14 @@ CI runs lint, format check, typecheck, and build on every PR. Integration tests
 
 ### Releasing
 
-Releases are automated via GitHub Actions:
-
-```bash
-# Bump version
-npm version patch  # or minor, major
+Releases are fully automated. Every merge to `main` triggers the release workflow which:
 
-# Push the tag
-git push --follow-tags
+1. Runs CI (lint, format, typecheck, build)
+2. Bumps the patch version
+3. Publishes to npm with provenance
+4. Creates a GitHub Release
 
-# GitHub Actions will:
-# 1. Build the package
-# 2. Publish to npm with provenance
-# 3. Create a GitHub Release
-```
+No manual version bumping or tagging needed — just merge your PR.
 
 ## License
 

package/dist/test.js

@@ -1,207 +1,356 @@
 /**
  * Integration test suite for garmin-connect-mcp.
+ * Tests the actual MCP tool handlers (not just raw API endpoints)
+ * to catch bugs in zip extraction, file I/O, date defaults, etc.
+ *
  * Requires a valid session at ~/.garmin-connect-mcp/session.json.
  * Run: npm test
  */
-import { GarminClient } from "./garmin-client.js";
+import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import { registerTools } from "./tools.js";
+import { existsSync, rmSync } from "node:fs";
+import { getSharedClient } from "./garmin-client.js";
+const TEST_FIT_DIR = "/tmp/garmin-mcp-test-fit";
+async function callTool(server, name, args = {}) {
+    // Access internal tool handler via the server's registered tools
+    const result = (await server._registeredTools[name].handler({ ...args }, { signal: new AbortController().signal }));
+    return result;
+}
+function getToolText(result) {
+    return result.content[0]?.text ?? "";
+}
+function getToolJson(result) {
+    return JSON.parse(getToolText(result));
+}
 const tests = [
-    // ── Session / Profile ──────────────────────────────────────────────
+    // ── Session ────────────────────────────────────────────────────────
     {
         name: "check-session",
-        run: ({ client }) => client.get("userprofile-service/userprofile/user-settings/"),
+        run: async (server) => {
+            const result = await callTool(server, "check-session");
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (data.status !== "ok")
+                throw new Error("status not ok");
+        },
     },
     {
         name: "get-user-profile",
-        run: ({ client }) => client.get("userprofile-service/userprofile/user-settings/"),
+        run: async (server) => {
+            const result = await callTool(server, "get-user-profile");
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!data.id)
+                throw new Error("no user id");
+        },
     },
     // ── Activities ─────────────────────────────────────────────────────
     {
         name: "list-activities",
-        run: ({ client }) => client.get("activitylist-service/activities/search/activities", {
-            limit: 2,
-            start: 0,
-        }),
+        run: async (server) => {
+            const result = await callTool(server, "list-activities", {
+                limit: 2,
+                start: 0,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!Array.isArray(data) || data.length === 0)
+                throw new Error("no activities");
+        },
     },
     {
         name: "get-activity",
-        run: ({ client, activityId }) => client.get(`activity-service/activity/${activityId}`),
+        run: async (server) => {
+            const result = await callTool(server, "get-activity", {
+                activityId: activityId,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!data.summaryDTO)
+                throw new Error("no summaryDTO");
+        },
     },
     {
         name: "get-activity-details",
-        run: ({ client, activityId }) => client.get(`activity-service/activity/${activityId}/details`, {
-            maxChartSize: 100,
-            maxPolylineSize: 0,
-            maxHeatMapSize: 100,
-        }),
+        run: async (server) => {
+            const result = await callTool(server, "get-activity-details", {
+                activityId: activityId,
+                maxChartSize: 10000,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!data.metricDescriptors)
+                throw new Error("no metricDescriptors");
+        },
     },
     {
         name: "get-activity-splits",
-        run: ({ client, activityId }) => client.get(`activity-service/activity/${activityId}/splits`),
+        run: async (server) => {
+            const result = await callTool(server, "get-activity-splits", {
+                activityId: activityId,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!data.lapDTOs)
+                throw new Error("no lapDTOs");
+        },
     },
     {
         name: "get-activity-hr-zones",
-        run: ({ client, activityId }) => client.get(`activity-service/activity/${activityId}/hrTimeInZones`),
+        run: async (server) => {
+            const result = await callTool(server, "get-activity-hr-zones", {
+                activityId: activityId,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!Array.isArray(data) || data.length !== 5)
+                throw new Error(`expected 5 zones, got ${data.length}`);
+        },
     },
     {
         name: "get-activity-polyline",
-        run: ({ client, activityId }) => client.get(`activity-service/activity/${activityId}/polyline/full-resolution/`),
+        run: async (server) => {
+            const result = await callTool(server, "get-activity-polyline", {
+                activityId: activityId,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-activity-weather",
-        run: ({ client, activityId }) => client.get(`activity-service/activity/${activityId}/weather`),
+        run: async (server) => {
+            const result = await callTool(server, "get-activity-weather", {
+                activityId: activityId,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
-        name: "download-fit",
-        run: ({ client, activityId }) => client.getBytes(`download-service/files/activity/${activityId}`),
+        name: "download-fit (zip extraction + file write)",
+        run: async (server) => {
+            // Clean up from previous runs
+            if (existsSync(TEST_FIT_DIR))
+                rmSync(TEST_FIT_DIR, { recursive: true });
+            const result = await callTool(server, "download-fit", {
+                activityId: activityId,
+                outputDir: TEST_FIT_DIR,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const text = getToolText(result);
+            if (!text.includes("Downloaded FIT file"))
+                throw new Error(`unexpected response: ${text}`);
+            // Verify file actually exists on disk
+            const expectedPath = `${TEST_FIT_DIR}/${activityId}.fit`;
+            if (!existsSync(expectedPath))
+                throw new Error(`FIT file not found at ${expectedPath}`);
+        },
     },
-    // ── Daily Health ───────────────────────────────────────────────────
+    // ── Daily Health (test date defaults) ──────────────────────────────
     {
-        name: "get-daily-summary",
-        run: ({ client, displayName, today }) => client.get(`usersummary-service/usersummary/daily/${displayName}`, {
-            calendarDate: today,
-        }),
+        name: "get-daily-summary (default date)",
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-summary");
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!data.calendarDate)
+                throw new Error("no calendarDate");
+        },
     },
     {
         name: "get-daily-heart-rate",
-        run: ({ client, today }) => client.get("wellness-service/wellness/dailyHeartRate", { date: today }),
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-heart-rate");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-daily-stress",
-        run: ({ client, today }) => client.get(`wellness-service/wellness/dailyStress/${today}`),
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-stress");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-daily-summary-chart",
-        run: ({ client, today }) => client.get("wellness-service/wellness/dailySummaryChart/", {
-            date: today,
-        }),
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-summary-chart");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-daily-intensity-minutes",
-        run: ({ client, today }) => client.get(`wellness-service/wellness/daily/im/${today}`),
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-intensity-minutes");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-daily-movement",
-        run: ({ client, today }) => client.get("wellness-service/wellness/dailyMovement", {
-            calendarDate: today,
-        }),
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-movement");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-daily-respiration",
-        run: ({ client, today }) => client.get(`wellness-service/wellness/daily/respiration/${today}`),
+        run: async (server) => {
+            const result = await callTool(server, "get-daily-respiration");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
-    // ── Sleep, Body Battery, HRV ───────────────────────────────────────
+    // ── Sleep / Body Battery / HRV ─────────────────────────────────────
     {
         name: "get-sleep",
-        run: ({ client, today }) => client.get("sleep-service/sleep/dailySleepData", {
-            date: today,
-            nonSleepBufferMinutes: 60,
-        }),
+        run: async (server) => {
+            const result = await callTool(server, "get-sleep");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-body-battery",
-        run: ({ client }) => client.get("wellness-service/wellness/bodyBattery/messagingToday"),
+        run: async (server) => {
+            const result = await callTool(server, "get-body-battery");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-hrv",
-        run: ({ client, today }) => client.get(`hrv-service/hrv/${today}`),
+        run: async (server) => {
+            const result = await callTool(server, "get-hrv");
+            if (result.isError)
+                throw new Error(getToolText(result));
+            // noData is acceptable
+        },
     },
-    // ── Weight ─────────────────────────────────────────────────────────
+    // ── Weight / Records / Fitness ─────────────────────────────────────
     {
         name: "get-weight",
-        run: ({ client, today, thirtyDaysAgo }) => client.get(`weight-service/weight/range/${thirtyDaysAgo}/${today}`, {
-            includeAll: "true",
-        }),
+        run: async (server) => {
+            const thirtyDaysAgo = new Date(Date.now() - 30 * 86400000)
+                .toISOString()
+                .slice(0, 10);
+            const today = new Date().toISOString().slice(0, 10);
+            const result = await callTool(server, "get-weight", {
+                startDate: thirtyDaysAgo,
+                endDate: today,
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
-    // ── Personal Records ───────────────────────────────────────────────
     {
-        name: "get-personal-records",
-        run: ({ client, displayName }) => client.get(`personalrecord-service/personalrecord/prs/${displayName}`, {
-            includeHistory: "true",
-        }),
+        name: "get-personal-records (displayName resolution)",
+        run: async (server) => {
+            const result = await callTool(server, "get-personal-records");
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!Array.isArray(data))
+                throw new Error("expected array");
+        },
     },
-    // ── Fitness Stats / Reports ────────────────────────────────────────
     {
         name: "get-fitness-stats",
-        run: ({ client, today, thirtyDaysAgo }) => client.get("fitnessstats-service/activity", {
-            aggregation: "daily",
-            startDate: thirtyDaysAgo,
-            endDate: today,
-            groupByActivityType: "true",
-            standardizedUnits: "true",
-            groupByParentActivityType: "false",
-            userFirstDay: "sunday",
-            metric: "duration",
-        }),
+        run: async (server) => {
+            const thirtyDaysAgo = new Date(Date.now() - 30 * 86400000)
+                .toISOString()
+                .slice(0, 10);
+            const today = new Date().toISOString().slice(0, 10);
+            const result = await callTool(server, "get-fitness-stats", {
+                startDate: thirtyDaysAgo,
+                endDate: today,
+                aggregation: "daily",
+                metric: "duration",
+            });
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-vo2max",
-        run: ({ client, today }) => client.get(`metrics-service/metrics/maxmet/latest/${today}`),
+        run: async (server) => {
+            const result = await callTool(server, "get-vo2max");
+            if (result.isError)
+                throw new Error(getToolText(result));
+        },
     },
     {
         name: "get-hr-zones-config",
-        run: ({ client }) => client.get("biometric-service/heartRateZones/"),
+        run: async (server) => {
+            const result = await callTool(server, "get-hr-zones-config");
+            if (result.isError)
+                throw new Error(getToolText(result));
+            const data = getToolJson(result);
+            if (!Array.isArray(data) || data.length === 0)
+                throw new Error("no zones returned");
+        },
     },
 ];
+// Resolved during bootstrap
+let activityId = "";
 async function main() {
-    console.log("garmin-connect-mcp integration tests\n");
-    const client = new GarminClient();
-    const today = new Date().toISOString().slice(0, 10);
-    const thirtyDaysAgo = new Date(Date.now() - 30 * 86400000)
-        .toISOString()
-        .slice(0, 10);
-    // Bootstrap: resolve displayName
-    console.log("Bootstrapping...");
-    const settings = (await client.get("userprofile-service/userprofile/settings"));
-    const displayName = settings.displayName;
-    if (!displayName) {
-        console.error("FATAL: Could not resolve displayName from settings");
-        console.error("Response:", JSON.stringify(settings).slice(0, 500));
-        await client.close();
-        process.exit(1);
-    }
-    console.log(`  displayName: ${displayName}`);
+    console.log("garmin-connect-mcp integration tests (tool-level)\n");
+    // Set up a real MCP server with all tools registered
+    const server = new McpServer({
+        name: "garmin-connect-mcp-test",
+        version: "0.0.0",
+    });
+    registerTools(server);
     // Bootstrap: get a recent activityId
-    const activities = (await client.get("activitylist-service/activities/search/activities", { limit: 1, start: 0 }));
-    const activityId = String(activities?.[0]?.activityId ?? "");
+    console.log("Bootstrapping...");
+    const listResult = await callTool(server, "list-activities", {
+        limit: 1,
+        start: 0,
+    });
+    const activities = getToolJson(listResult);
+    activityId = String(activities[0]?.activityId ?? "");
     if (!activityId) {
         console.error("FATAL: No activities found");
-        await client.close();
         process.exit(1);
     }
-    console.log(`  activityId: ${activityId}`);
-    console.log(`  date range: ${thirtyDaysAgo} → ${today}\n`);
-    const ctx = {
-        client,
-        displayName,
-        activityId,
-        today,
-        thirtyDaysAgo,
-    };
+    console.log(`  activityId: ${activityId}\n`);
     let passed = 0;
     let failed = 0;
     for (const test of tests) {
         const start = Date.now();
         try {
-            const result = await test.run(ctx);
-            if (result === undefined) {
-                throw new Error("undefined response");
-            }
+            await test.run(server);
             const ms = Date.now() - start;
-            const noData = result && typeof result === "object" && "noData" in result;
-            console.log(`  PASS  ${test.name} (${ms}ms)${noData ? " [no data for date]" : ""}`);
+            console.log(`  PASS  ${test.name} (${ms}ms)`);
             passed++;
         }
         catch (e) {
             const ms = Date.now() - start;
             const msg = e instanceof Error ? e.message : String(e);
-            // Truncate long error messages
             const short = msg.length > 120 ? msg.slice(0, 120) + "..." : msg;
             console.log(`  FAIL  ${test.name} (${ms}ms) — ${short}`);
             failed++;
         }
     }
+    // Cleanup
+    if (existsSync(TEST_FIT_DIR))
+        rmSync(TEST_FIT_DIR, { recursive: true });
     console.log(`\nResults: ${passed} passed, ${failed} failed, ${passed + failed} total`);
-    await client.close();
+    await getSharedClient().close();
     process.exit(failed > 0 ? 1 : 0);
 }
 main().catch((err) => {

package/dist/tools.js

@@ -396,43 +396,59 @@ Present results as a markdown table: | Tool | Status | Notes |
 Count total passed vs failed at the end.`);
     });
 }
+import { inflateRawSync } from "node:zlib";
 /**
- * Minimal zip extraction — finds and extracts the first .fit file from a zip buffer.
- * Avoids needing a zip library dependency.
+ * Minimal zip extraction — finds the first .fit file using the central
+ * directory (which always has correct sizes, unlike local headers that
+ * may use data descriptors with size=0).
  */
 function extractFitFromZip(buf, activityId) {
-    // ZIP local file header signature: PK\x03\x04
-    let offset = 0;
-    while (offset < buf.length - 30) {
-        if (buf[offset] === 0x50 &&
-            buf[offset + 1] === 0x4b &&
-            buf[offset + 2] === 0x03 &&
-            buf[offset + 3] === 0x04) {
-            const compressionMethod = buf.readUInt16LE(offset + 8);
-            const compressedSize = buf.readUInt32LE(offset + 18);
-            const uncompressedSize = buf.readUInt32LE(offset + 22);
-            const nameLength = buf.readUInt16LE(offset + 26);
-            const extraLength = buf.readUInt16LE(offset + 28);
-            const name = buf.toString("utf-8", offset + 30, offset + 30 + nameLength);
-            const dataStart = offset + 30 + nameLength + extraLength;
-            if (name.endsWith(".fit") && compressionMethod === 0) {
-                // Stored (no compression) — just slice the data
+    // Find End of Central Directory (EOCD): PK\x05\x06
+    let eocdOffset = buf.length - 22;
+    while (eocdOffset >= 0) {
+        if (buf[eocdOffset] === 0x50 &&
+            buf[eocdOffset + 1] === 0x4b &&
+            buf[eocdOffset + 2] === 0x05 &&
+            buf[eocdOffset + 3] === 0x06)
+            break;
+        eocdOffset--;
+    }
+    if (eocdOffset < 0)
+        return null;
+    const cdOffset = buf.readUInt32LE(eocdOffset + 16);
+    const cdEntries = buf.readUInt16LE(eocdOffset + 10);
+    // Walk central directory entries: PK\x01\x02
+    let pos = cdOffset;
+    for (let i = 0; i < cdEntries; i++) {
+        if (buf[pos] !== 0x50 ||
+            buf[pos + 1] !== 0x4b ||
+            buf[pos + 2] !== 0x01 ||
+            buf[pos + 3] !== 0x02)
+            break;
+        const method = buf.readUInt16LE(pos + 10);
+        const compressedSize = buf.readUInt32LE(pos + 20);
+        const uncompressedSize = buf.readUInt32LE(pos + 24);
+        const nameLength = buf.readUInt16LE(pos + 28);
+        const extraLength = buf.readUInt16LE(pos + 30);
+        const commentLength = buf.readUInt16LE(pos + 32);
+        const localHeaderOffset = buf.readUInt32LE(pos + 42);
+        const name = buf.toString("utf-8", pos + 46, pos + 46 + nameLength);
+        if (name.endsWith(".fit")) {
+            // Read local header to find data start
+            const localNameLen = buf.readUInt16LE(localHeaderOffset + 26);
+            const localExtraLen = buf.readUInt16LE(localHeaderOffset + 28);
+            const dataStart = localHeaderOffset + 30 + localNameLen + localExtraLen;
+            if (method === 0) {
                 const data = buf.subarray(dataStart, dataStart + uncompressedSize);
                 return { name: `${activityId}.fit`, data: Buffer.from(data) };
             }
-            if (name.endsWith(".fit") && compressionMethod === 8) {
-                // Deflate compressed — use Node's zlib
+            if (method === 8) {
                 const compressed = buf.subarray(dataStart, dataStart + compressedSize);
                 const data = inflateRawSync(compressed);
                 return { name: `${activityId}.fit`, data };
             }
-            // Skip to next file header
-            offset = dataStart + compressedSize;
-        }
-        else {
-            offset++;
         }
+        pos += 46 + nameLength + extraLength + commentLength;
     }
     return null;
 }
-import { inflateRawSync } from "node:zlib";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@etweisberg/garmin-connect-mcp",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "description": "MCP server for Garmin Connect — access activities, metrics, and FIT files via Claude Code",
   "type": "module",
   "bin": {