fix: address second review round

- searchTimeEntries pagination termination now uses explicit
  undefined/NaN checks instead of truthy (`!nextId || !nextRowNumber`),
  so a legitimate 0 cursor value is not mistaken for end-of-results
  (Copilot).
- Comment on `toggl_get_time_entries` post-filters said "on hydrated
  entries" but the implementation applies them before hydration.
  Update the comment to match the code (Copilot).
- README: "Free-plan" -> "Free plan" (CodeRabbit).

Author: salmon-21

README.md

@@ -161,7 +161,7 @@ Dashboard-parity search backed by **Toggl Reports API v3**. Per-workspace, auto-
 }
 ```
 Notes:
-- `billable` is a premium feature; Free-plan workspaces will surface a 402 error.
+- `billable` is a premium feature; Free plan workspaces will surface a 402 error.
 - Date range is required (either `start_date`+`end_date` or a `period`).
 - Pagination is automatic via `X-Next-ID` / `X-Next-Row-Number`; cap traversal with `max_pages`.
 

src/index.ts

@@ -163,7 +163,7 @@ const tools: Tool[] = [
         end_date: { type: 'string', description: 'End date (YYYY-MM-DD format)' },
         workspace_id: { type: 'number', description: 'Filter by workspace ID' },
         project_id: { type: 'number', description: 'Filter by project ID' },
-        // Post-filter additions (applied after fetch, on hydrated entries).
+        // Post-filter additions (applied client-side after fetch, before hydration).
         description: { type: 'string', description: 'Case-insensitive substring match on entry description' },
         billable: { type: 'boolean', description: 'Filter by billable status' },
         user_ids: { type: 'array', items: { type: 'number' }, description: 'Filter by user IDs (client-side post-filter)' },

src/toggl-api.ts

@@ -363,7 +363,14 @@ export class TogglAPI {
         }
       }
 
-      if (!nextId || !nextRowNumber) break;
+      // Use explicit undefined/NaN checks so a legitimate 0 cursor value
+      // (possible if Toggl ever uses it) is not mistaken for end-of-results.
+      if (
+        nextId === undefined || Number.isNaN(nextId) ||
+        nextRowNumber === undefined || Number.isNaN(nextRowNumber)
+      ) {
+        break;
+      }
       payload.first_id = nextId;
       payload.first_row_number = nextRowNumber;
     }