v0.8.8 (4/4): why-comments on three non-obvious functions

Three functions did clever things without explaining why. Comments
added in the spirit of existing ones on exerciseSeries / ldShifts.

- rankBodyNames: documents the score tiers so the next reader
  doesn't have to guess what 0/1/2/3 mean.
- reconstructInterval: explains that it's the inverse of
  unrollIntervalBlocks and the assumption that all work/rest
  rounds are identical (because the UI only lets the user enter
  one work template + one rest template).
- openExerciseInfo: explains the synchronous-window.open trick to
  dodge popup blockers, the placeholder + fill pattern, and the
  fallback to a direct deep-link when the popup is blocked.

No code changes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: bgeurten

gym-tracker.html

@@ -593,8 +593,11 @@
   for (const arr of groups.values()) arr.sort((a, b) => a.date.localeCompare(b.date));
   return groups;
 }
-// Autocomplete source for the name field: user's own names first, then the
-// starter list (anything not already in user's set).
+// Autocomplete source for the body-metric name field: user's own names first,
+// then the starter list (anything not already in the user's set).
+// Score tiers (mirroring rankExercises): startsWith=2, word-boundary prefix=1,
+// includes=0, no-match=-1. Empty query returns score 3 so a freshly-opened
+// dropdown shows everything before the user types.
 function rankBodyNames(q) {
   const query = q.trim().toLowerCase();
   const user = new Set();
@@ -833,6 +836,12 @@
   return byPreset;
 }
 
+// Inverse of unrollIntervalBlocks: given a flat list of imported CSV rows
+// for one cardio session (warmup, work×N, rest×N, cooldown), rebuild the
+// nested interval shape used in-app. We assume every work round has the
+// same minutes/level (and likewise for rest), since that's what the UI lets
+// the user enter — first work row sets the work template, count is
+// derived from how many work rows arrived.
 function reconstructInterval(blocks) {
   const c = { mode: "interval" };
   const warmup = blocks.find(b => b.phase === "warmup");
@@ -1042,10 +1051,15 @@ <h2 id="flatgym-instr-heading" style="display:none">Instructions</h2>
 </body></html>`;
 }
 
+// Open the reference widget in a new tab. The tab is opened SYNCHRONOUSLY
+// inside the click event so the popup blocker treats it as user-initiated;
+// we then write a placeholder document (so users see something immediately)
+// and asynchronously fetch the full dataset to fill in instructions and chips.
+// If the popup is blocked outright (return value falsy), we degrade to a
+// plain deep-link to the source repo folder.
 async function openExerciseInfo(id, name) {
   const w = window.open("", "_blank");
   if (!w) {
-    // Popup blocked — fall back to a direct deep-link.
     window.open(exerciseTreeUrl(id), "_blank", "noopener");
     return;
   }