edfcore 0.2.65 → 0.2.67

package/CHANGELOG.md

@@ -6,7 +6,46 @@ alone does not tell you whether you were affected.
 edfcore is pre-1.0. Patch releases have carried behaviour changes where the old behaviour was a
 defect; those are called out below.
 
-## 0.2.64
+## 0.2.67
+
+Three defects in `formatHeader`, found by an adversarial sweep of the modules no earlier pass had
+covered.
+
+- **Fixed** the duration line losing a whole second. It computed `recordCount *
+  recordDurationSeconds` in float64 and truncated, and a record duration with no exact binary form
+  makes that product land just under the true value: 100 records of 0.29 s is exactly 29 s,
+  computes as 28.999999999999996, and printed `00:00:28`. It is now computed from
+  `recordDurationTicks`, which is exact. A genuine fraction still truncates rather than rounding —
+  7 × 0.7 s is 4.9 and prints `00:00:04`, because rounding would name a time the file never reaches.
+- **Fixed** control characters in a signal label being printed verbatim. EDF pads labels with
+  spaces and says nothing about what else may be in them, so a writer can put a newline there and
+  the label renders as TWO rows — forging a signal the file does not contain — while a tab shifts
+  every column after it. They are replaced with a dot in the rendering only; `signal.raw.label`
+  still holds the bytes.
+- **Fixed** the diagnostic severity summary being ordered by arrival, so two files with the same
+  diagnostics could summarise them differently. Now error-warning-info, matching
+  `formatValidationReport` since 0.2.15 and sharing its counting.
+
+## 0.2.66
+
+- **Fixed** the changelog numbering, which had drifted a second time, and **fixed the cause** so it
+  cannot drift a third.
+- The mechanism, both times: the entry is written by hand before the release runs, against the
+  version the author expects. When a release fails AFTER bumping — a lint error, a flaky test, an
+  agent's scratch file in the tree — that number is consumed, the next run produces a different
+  one, and every heading from there on inherits the drift. `0.2.29` and `0.2.36` went that way in
+  the last round; `0.2.59` went that way in this one, consumed by its own coverage guard when a
+  corpus file was added without regenerating its parity golden.
+- `scripts/release.mjs` now refuses to release unless the top `## <version>` heading in
+  CHANGELOG.md equals the version being tagged, and its message says what to do — including
+  recording a skipped number as never released. One file read, and a silent documentation defect
+  becomes a message before anything is committed.
+- Headings `0.2.59` through `0.2.64` are shifted to the releases that actually carried them,
+  verified against `git show <tag>:CHANGELOG.md` rather than reasoned about, and `0.2.59` is
+  recorded as never released. Prose references in the README, the docs and one test are shifted to
+  match.
+
+## 0.2.65
 
 - **Added** a migration guide for 0.3.0, published before the release rather than after it:
   [Migrating to 0.3](https://edfcore.vercel.app/docs/migrating-to-0-3). Three functions are
@@ -20,7 +59,7 @@ defect; those are called out below.
 - Includes the find-and-replace, with word boundaries, and a note on why: `sampleStartTicks` and
   `sampleStartTicksOf` are distinct names and a substring replace would damage the second.
 
-## 0.2.63
+## 0.2.64
 
 - **Added** a test that drives essentially the whole public barrel over every corpus file — six
   files written by five pieces of software across twenty-one years — and asserts the results are
@@ -34,7 +73,7 @@ defect; those are called out below.
   nowhere written down, and the sleep-edfx hypnogram is a real file of exactly that shape. Now
   stated in the source, in the docs, and pinned by the test that found it.
 
-## 0.2.62
+## 0.2.63
 
 - **Added** `formatAnnotations`, the third formatter beside `formatHeader` and
   `formatDiagnostics`, and the one a hypnogram or an event list actually needs.
@@ -49,7 +88,7 @@ defect; those are called out below.
   the header start time, a recording may begin after its first annotation, and clamping to zero
   would silently move it.
 
-## 0.2.61
+## 0.2.62
 
 - **Deprecated** `sampleIndexAt`, `sampleStartTicks` and `sampleStartSeconds`, which are renamed
   to `gridSampleIndexAt`, `gridSampleStartTicks` and `gridSampleStartSeconds` in **0.3.0**. The
@@ -62,7 +101,7 @@ defect; those are called out below.
 - Nothing is removed here. An editor will point at the replacement a release before the rename
   lands, and for a contiguous file the rename is the only thing that affects a caller.
 
-## 0.2.60
+## 0.2.61
 
 - **Added** `sampleAt`, `sampleStartTicksOf` and `sampleStartSecondsOf` — the recording-aware
   counterpart to the sample-grid family, and the groundwork for 0.3.0.
@@ -79,7 +118,7 @@ defect; those are called out below.
   follows. A round-trip test pins the pair together across the gap: the sample at a sample's start
   is that sample, for every sample in the file.
 
-## 0.2.59
+## 0.2.60
 
 - **Added** `calib.rec` from edfplus.info — the last corpus the README named. It was written by
   **Bob Kemp, who wrote the EDF specification**, expressly to check that a reader gets amplitude
@@ -99,6 +138,12 @@ defect; those are called out below.
   the new corpus file had no parity golden yet. That is exactly the drift it exists to catch, and
   it caught it before the release rather than after.
 
+## 0.2.59
+
+Never released. The 0.2.59 release run failed its own coverage guard — a corpus file had been added
+without regenerating its parity golden — which consumed the number before a tag was cut. The
+calibration-file work that carried this heading while it was being written shipped in `0.2.60`.
+
 ## 0.2.58
 
 - **Added** CHB-MIT to the corpus, closing a gap the README has named since 0.1. It is chosen to be

package/README.md

@@ -341,7 +341,7 @@ also supplies something no synthetic fixture here had — a montage that names o
 which is what `EdfAmbiguousChannelError` exists for and had until now only ever been raised against
 a fixture written to raise it.
 
-**The format author's own calibration file, as of 0.2.59.** `calib.rec` was written by Bob Kemp —
+**The format author's own calibration file, as of 0.2.60.** `calib.rec` was written by Bob Kemp —
 who wrote the EDF specification — expressly to check that a reader gets amplitude *and polarity*
 right. It is the only fixture here whose expected values come from neither edfcore nor another
 library but from the file's own design: its gain is 25/1024, exactly representable in binary, so

package/dist/constants.d.ts

@@ -109,5 +109,5 @@ export declare const SIGNAL_FIELD_BLOCK_OFFSETS: {
     readonly reserved: 224;
 };
 /** Published package version. Kept in sync with package.json by a test. */
-export declare const VERSION = "0.2.65";
+export declare const VERSION = "0.2.67";
 //# sourceMappingURL=constants.d.ts.map

package/dist/constants.js

@@ -79,5 +79,5 @@ export const SIGNAL_FIELD_BLOCK_OFFSETS = {
     reserved: 224,
 };
 /** Published package version. Kept in sync with package.json by a test. */
-export const VERSION = '0.2.65';
+export const VERSION = '0.2.67';
 //# sourceMappingURL=constants.js.map

package/dist/format-header.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"format-header.d.ts","sourceRoot":"","sources":["../src/format-header.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,KAAK,EAAmB,SAAS,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAwBlF;;;;;;GAMG;AACH,wBAAgB,YAAY,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,mBAAmB,GAAG,MAAM,CA0DrF"}
+{"version":3,"file":"format-header.d.ts","sourceRoot":"","sources":["../src/format-header.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAIH,OAAO,KAAK,EAAmB,SAAS,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AA0ClF;;;;;;GAMG;AACH,wBAAgB,YAAY,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAsErF"}

package/dist/format-header.js

@@ -11,6 +11,8 @@
  * identification unless asked — a header carries a name and a birth date, and a summary that
  * lands in a chat log or an issue tracker should not carry them by default.
  */
+import { TICKS_PER_SECOND } from './constants.js';
+import { summarizeDiagnostics } from './diagnostics/summary.js';
 function formatDate(date) {
     if (date === undefined)
         return 'unknown';
@@ -18,16 +20,33 @@ function formatDate(date) {
     const day = String(date.day).padStart(2, '0');
     return `${date.year}-${month}-${day}`;
 }
-function formatDuration(seconds) {
-    if (!Number.isFinite(seconds) || seconds < 0)
+/**
+ * `hh:mm:ss` from an exact tick count.
+ *
+ * Ticks, not `recordCount * recordDurationSeconds`. That product is float64, and a record duration
+ * with no exact binary representation makes it land just under the true value: 100 records of
+ * 0.29 s is exactly 29 s and computes as 28.999999999999996, which floors to 28. The header line
+ * then reports a recording a whole second shorter than it is (fixed in 0.2.67).
+ */
+function formatDurationTicks(ticks) {
+    if (ticks < 0n)
         return 'unknown';
-    const whole = Math.floor(seconds);
+    const whole = Number(ticks / TICKS_PER_SECOND);
     const hours = Math.floor(whole / 3600);
     const minutes = Math.floor((whole % 3600) / 60);
     const rest = whole % 60;
     const pad = (n) => String(n).padStart(2, '0');
     return `${pad(hours)}:${pad(minutes)}:${pad(rest)}`;
 }
+/** Control characters become a dot, so one field can never become two rows or shift a column. */
+function printable(text) {
+    let out = '';
+    for (const character of text) {
+        const code = character.codePointAt(0) ?? 0;
+        out += code < 0x20 || code === 0x7f ? '.' : character;
+    }
+    return out;
+}
 function formatRate(signal) {
     // undefined is the honest answer for a zero record duration, which is legal EDF.
     return signal.sampleRateHz === undefined ? '—' : `${signal.sampleRateHz} Hz`;
@@ -48,7 +67,7 @@ export function formatHeader(header, options) {
         ` (local, no timezone)`);
     lines.push(`record       ${header.recordDurationSeconds} s · ${header.recordByteLength} bytes · ` +
         `${header.bytesPerSample} bytes/sample`);
-    lines.push(`duration     ${formatDuration(header.recordCount * header.recordDurationSeconds)} ` +
+    lines.push(`duration     ${formatDurationTicks(header.recordDurationTicks * BigInt(header.recordCount))} ` +
         `(${header.recordCount} × ${header.recordDurationSeconds} s)`);
     if (header.recordCountSource === 'sourceByteLength') {
         // Worth saying out loud: the count came from the file size, not from the header field.
@@ -62,7 +81,11 @@ export function formatHeader(header, options) {
     lines.push('  #  label                 kind         rate      range');
     for (const signal of header.signals) {
         const index = String(signal.index).padStart(3);
-        const label = signal.label.slice(0, 20).padEnd(21);
+        // Control characters are replaced, not printed. A label holding a newline would otherwise
+        // render as two rows and forge a signal the file does not contain; a tab would shift every
+        // column after it. EDF pads labels with spaces and says nothing about what else may be in
+        // them, so a writer can put anything there and a reader must not be steered by it.
+        const label = printable(signal.label).slice(0, 20).padEnd(21);
         const kind = signal.kind.padEnd(12);
         const rate = formatRate(signal).padEnd(9);
         const range = signal.kind === 'annotations'
@@ -74,11 +97,17 @@ export function formatHeader(header, options) {
     }
     if (header.diagnostics.length > 0) {
         lines.push('');
-        const counts = new Map();
-        for (const diagnostic of header.diagnostics) {
-            counts.set(diagnostic.severity, (counts.get(diagnostic.severity) ?? 0) + 1);
-        }
-        const summary = [...counts].map(([severity, count]) => `${count} ${severity}`).join(', ');
+        // Fixed error-warning-info order, matching `formatValidationReport` since 0.2.15. Ordering by
+        // arrival meant two files with the same diagnostics could summarise them differently.
+        const counted = summarizeDiagnostics(header.diagnostics);
+        const summary = [
+            [counted.errors, 'error'],
+            [counted.warnings, 'warning'],
+            [counted.infos, 'info'],
+        ]
+            .filter(([count]) => count > 0)
+            .map(([count, severity]) => `${count} ${severity}`)
+            .join(', ');
         lines.push(`${header.diagnostics.length} diagnostic(s): ${summary}`);
         lines.push('Call formatDiagnostics(header.diagnostics) for the detail.');
     }

package/dist/format-header.js.map

@@ -1 +1 @@
-{"version":3,"file":"format-header.js","sourceRoot":"","sources":["../src/format-header.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAIH,SAAS,UAAU,CAAC,IAAiC;IACnD,IAAI,IAAI,KAAK,SAAS;QAAE,OAAO,SAAS,CAAC;IACzC,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAClD,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9C,OAAO,GAAG,IAAI,CAAC,IAAI,IAAI,KAAK,IAAI,GAAG,EAAE,CAAC;AACxC,CAAC;AAED,SAAS,cAAc,CAAC,OAAe;IACrC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,OAAO,GAAG,CAAC;QAAE,OAAO,SAAS,CAAC;IAC/D,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAClC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,GAAG,IAAI,CAAC,CAAC;IACvC,MAAM,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC;IAChD,MAAM,IAAI,GAAG,KAAK,GAAG,EAAE,CAAC;IACxB,MAAM,GAAG,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,OAAO,GAAG,GAAG,CAAC,KAAK,CAAC,IAAI,GAAG,CAAC,OAAO,CAAC,IAAI,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;AACtD,CAAC;AAED,SAAS,UAAU,CAAC,MAAoC;IACtD,iFAAiF;IACjF,OAAO,MAAM,CAAC,YAAY,KAAK,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,MAAM,CAAC,YAAY,KAAK,CAAC;AAC/E,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,YAAY,CAAC,MAAiB,EAAE,OAA6B;IAC3E,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,MAAM,CAAC,SAAS,CAAC;IAE/B,KAAK,CAAC,IAAI,CACR,GAAG,MAAM,CAAC,OAAO,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,cAAc,MAAM,CAAC,WAAW,UAAU,CACvF,CAAC;IACF,KAAK,CAAC,IAAI,CACR,gBAAgB,UAAU,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG;QAC5F,GAAG,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE;QAC/F,uBAAuB,CAC1B,CAAC;IACF,KAAK,CAAC,IAAI,CACR,gBAAgB,MAAM,CAAC,qBAAqB,QAAQ,MAAM,CAAC,gBAAgB,WAAW;QACpF,GAAG,MAAM,CAAC,cAAc,eAAe,CAC1C,CAAC;IACF,KAAK,CAAC,IAAI,CACR,gBAAgB,cAAc,CAAC,MAAM,CAAC,WAAW,GAAG,MAAM,CAAC,qBAAqB,CAAC,GAAG;QAClF,IAAI,MAAM,CAAC,WAAW,MAAM,MAAM,CAAC,qBAAqB,KAAK,CAChE,CAAC;IACF,IAAI,MAAM,CAAC,iBAAiB,KAAK,kBAAkB,EAAE,CAAC;QACpD,uFAAuF;QACvF,KAAK,CAAC,IAAI,CAAC,4DAA4D,CAAC,CAAC;IAC3E,CAAC;IAED,IAAI,OAAO,EAAE,gBAAgB,KAAK,IAAI,EAAE,CAAC;QACvC,KAAK,CAAC,IAAI,CAAC,gBAAgB,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,SAAS,EAAE,CAAC,CAAC;QACrE,KAAK,CAAC,IAAI,CAAC,gBAAgB,MAAM,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,SAAS,EAAE,CAAC,CAAC;IACzE,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,yDAAyD,CAAC,CAAC;IACtE,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACpC,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;QAC/C,MAAM,KAAK,GAAG,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACnD,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACpC,MAAM,IAAI,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;QAC1C,MAAM,KAAK,GACT,MAAM,CAAC,IAAI,KAAK,aAAa;YAC3B,CAAC,CAAC,GAAG;YACL,CAAC,CAAC,MAAM,CAAC,KAAK,KAAK,SAAS;gBAC1B,CAAC,CAAC,iBAAiB;gBACnB,CAAC,CAAC,GAAG,MAAM,CAAC,eAAe,KAAK,MAAM,CAAC,eAAe,IAAI,MAAM,CAAC,iBAAiB,EAAE,CAAC;QAC3F,KAAK,CAAC,IAAI,CAAC,GAAG,KAAK,KAAK,KAAK,GAAG,IAAI,GAAG,IAAI,IAAI,KAAK,EAAE,CAAC,CAAC;IAC1D,CAAC;IAED,IAAI,MAAM,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,MAAM,MAAM,GAAG,IAAI,GAAG,EAAkB,CAAC;QACzC,KAAK,MAAM,UAAU,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YAC5C,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,QAAQ,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;QAC9E,CAAC;QACD,MAAM,OAAO,GAAG,CAAC,GAAG,MAAM,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,IAAI,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1F,KAAK,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,WAAW,CAAC,MAAM,mBAAmB,OAAO,EAAE,CAAC,CAAC;QACrE,KAAK,CAAC,IAAI,CAAC,4DAA4D,CAAC,CAAC;IAC3E,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}
+{"version":3,"file":"format-header.js","sourceRoot":"","sources":["../src/format-header.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,gBAAgB,EAAE,MAAM,gBAAgB,CAAC;AAClD,OAAO,EAAE,oBAAoB,EAAE,MAAM,0BAA0B,CAAC;AAGhE,SAAS,UAAU,CAAC,IAAiC;IACnD,IAAI,IAAI,KAAK,SAAS;QAAE,OAAO,SAAS,CAAC;IACzC,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAClD,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9C,OAAO,GAAG,IAAI,CAAC,IAAI,IAAI,KAAK,IAAI,GAAG,EAAE,CAAC;AACxC,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,mBAAmB,CAAC,KAAa;IACxC,IAAI,KAAK,GAAG,EAAE;QAAE,OAAO,SAAS,CAAC;IACjC,MAAM,KAAK,GAAG,MAAM,CAAC,KAAK,GAAG,gBAAgB,CAAC,CAAC;IAC/C,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,GAAG,IAAI,CAAC,CAAC;IACvC,MAAM,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC;IAChD,MAAM,IAAI,GAAG,KAAK,GAAG,EAAE,CAAC;IACxB,MAAM,GAAG,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,OAAO,GAAG,GAAG,CAAC,KAAK,CAAC,IAAI,GAAG,CAAC,OAAO,CAAC,IAAI,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;AACtD,CAAC;AAED,iGAAiG;AACjG,SAAS,SAAS,CAAC,IAAY;IAC7B,IAAI,GAAG,GAAG,EAAE,CAAC;IACb,KAAK,MAAM,SAAS,IAAI,IAAI,EAAE,CAAC;QAC7B,MAAM,IAAI,GAAG,SAAS,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC3C,GAAG,IAAI,IAAI,GAAG,IAAI,IAAI,IAAI,KAAK,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC;IACxD,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED,SAAS,UAAU,CAAC,MAAoC;IACtD,iFAAiF;IACjF,OAAO,MAAM,CAAC,YAAY,KAAK,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,MAAM,CAAC,YAAY,KAAK,CAAC;AAC/E,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,YAAY,CAAC,MAAiB,EAAE,OAA6B;IAC3E,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,MAAM,CAAC,SAAS,CAAC;IAE/B,KAAK,CAAC,IAAI,CACR,GAAG,MAAM,CAAC,OAAO,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,cAAc,MAAM,CAAC,WAAW,UAAU,CACvF,CAAC;IACF,KAAK,CAAC,IAAI,CACR,gBAAgB,UAAU,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG;QAC5F,GAAG,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE;QAC/F,uBAAuB,CAC1B,CAAC;IACF,KAAK,CAAC,IAAI,CACR,gBAAgB,MAAM,CAAC,qBAAqB,QAAQ,MAAM,CAAC,gBAAgB,WAAW;QACpF,GAAG,MAAM,CAAC,cAAc,eAAe,CAC1C,CAAC;IACF,KAAK,CAAC,IAAI,CACR,gBAAgB,mBAAmB,CAAC,MAAM,CAAC,mBAAmB,GAAG,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,GAAG;QAC7F,IAAI,MAAM,CAAC,WAAW,MAAM,MAAM,CAAC,qBAAqB,KAAK,CAChE,CAAC;IACF,IAAI,MAAM,CAAC,iBAAiB,KAAK,kBAAkB,EAAE,CAAC;QACpD,uFAAuF;QACvF,KAAK,CAAC,IAAI,CAAC,4DAA4D,CAAC,CAAC;IAC3E,CAAC;IAED,IAAI,OAAO,EAAE,gBAAgB,KAAK,IAAI,EAAE,CAAC;QACvC,KAAK,CAAC,IAAI,CAAC,gBAAgB,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,SAAS,EAAE,CAAC,CAAC;QACrE,KAAK,CAAC,IAAI,CAAC,gBAAgB,MAAM,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,SAAS,EAAE,CAAC,CAAC;IACzE,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,yDAAyD,CAAC,CAAC;IACtE,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACpC,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;QAC/C,0FAA0F;QAC1F,2FAA2F;QAC3F,0FAA0F;QAC1F,mFAAmF;QACnF,MAAM,KAAK,GAAG,SAAS,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC9D,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACpC,MAAM,IAAI,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;QAC1C,MAAM,KAAK,GACT,MAAM,CAAC,IAAI,KAAK,aAAa;YAC3B,CAAC,CAAC,GAAG;YACL,CAAC,CAAC,MAAM,CAAC,KAAK,KAAK,SAAS;gBAC1B,CAAC,CAAC,iBAAiB;gBACnB,CAAC,CAAC,GAAG,MAAM,CAAC,eAAe,KAAK,MAAM,CAAC,eAAe,IAAI,MAAM,CAAC,iBAAiB,EAAE,CAAC;QAC3F,KAAK,CAAC,IAAI,CAAC,GAAG,KAAK,KAAK,KAAK,GAAG,IAAI,GAAG,IAAI,IAAI,KAAK,EAAE,CAAC,CAAC;IAC1D,CAAC;IAED,IAAI,MAAM,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,8FAA8F;QAC9F,sFAAsF;QACtF,MAAM,OAAO,GAAG,oBAAoB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QACzD,MAAM,OAAO,GACX;YACE,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC;YACzB,CAAC,OAAO,CAAC,QAAQ,EAAE,SAAS,CAAC;YAC7B,CAAC,OAAO,CAAC,KAAK,EAAE,MAAM,CAAC;SAE1B;aACE,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,CAAC;aAC9B,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,QAAQ,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,IAAI,QAAQ,EAAE,CAAC;aAClD,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,KAAK,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,WAAW,CAAC,MAAM,mBAAmB,OAAO,EAAE,CAAC,CAAC;QACrE,KAAK,CAAC,IAAI,CAAC,4DAA4D,CAAC,CAAC;IAC3E,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edfcore",
-  "version": "0.2.65",
+  "version": "0.2.67",
   "description": "Modern, typed, zero-dependency reader for EDF, EDF+, BDF and BDF+ biosignal files. Works in browsers and Node with true random access.",
   "keywords": [
     "edf",

package/src/constants.ts

@@ -93,4 +93,4 @@ export const SIGNAL_FIELD_BLOCK_OFFSETS = {
 } as const;
 
 /** Published package version. Kept in sync with package.json by a test. */
-export const VERSION = '0.2.65';
+export const VERSION = '0.2.67';

package/src/format-header.ts

@@ -12,6 +12,8 @@
  * lands in a chat log or an issue tracker should not carry them by default.
  */
 
+import { TICKS_PER_SECOND } from './constants.js';
+import { summarizeDiagnostics } from './diagnostics/summary.js';
 import type { EdfCalendarDate, EdfHeader, FormatHeaderOptions } from './types.js';
 
 function formatDate(date: EdfCalendarDate | undefined): string {
@@ -21,9 +23,17 @@ function formatDate(date: EdfCalendarDate | undefined): string {
   return `${date.year}-${month}-${day}`;
 }
 
-function formatDuration(seconds: number): string {
-  if (!Number.isFinite(seconds) || seconds < 0) return 'unknown';
-  const whole = Math.floor(seconds);
+/**
+ * `hh:mm:ss` from an exact tick count.
+ *
+ * Ticks, not `recordCount * recordDurationSeconds`. That product is float64, and a record duration
+ * with no exact binary representation makes it land just under the true value: 100 records of
+ * 0.29 s is exactly 29 s and computes as 28.999999999999996, which floors to 28. The header line
+ * then reports a recording a whole second shorter than it is (fixed in 0.2.67).
+ */
+function formatDurationTicks(ticks: bigint): string {
+  if (ticks < 0n) return 'unknown';
+  const whole = Number(ticks / TICKS_PER_SECOND);
   const hours = Math.floor(whole / 3600);
   const minutes = Math.floor((whole % 3600) / 60);
   const rest = whole % 60;
@@ -31,6 +41,16 @@ function formatDuration(seconds: number): string {
   return `${pad(hours)}:${pad(minutes)}:${pad(rest)}`;
 }
 
+/** Control characters become a dot, so one field can never become two rows or shift a column. */
+function printable(text: string): string {
+  let out = '';
+  for (const character of text) {
+    const code = character.codePointAt(0) ?? 0;
+    out += code < 0x20 || code === 0x7f ? '.' : character;
+  }
+  return out;
+}
+
 function formatRate(signal: EdfHeader['signals'][number]): string {
   // undefined is the honest answer for a zero record duration, which is legal EDF.
   return signal.sampleRateHz === undefined ? '—' : `${signal.sampleRateHz} Hz`;
@@ -60,7 +80,7 @@ export function formatHeader(header: EdfHeader, options?: FormatHeaderOptions):
       `${header.bytesPerSample} bytes/sample`,
   );
   lines.push(
-    `duration     ${formatDuration(header.recordCount * header.recordDurationSeconds)} ` +
+    `duration     ${formatDurationTicks(header.recordDurationTicks * BigInt(header.recordCount))} ` +
       `(${header.recordCount} × ${header.recordDurationSeconds} s)`,
   );
   if (header.recordCountSource === 'sourceByteLength') {
@@ -77,7 +97,11 @@ export function formatHeader(header: EdfHeader, options?: FormatHeaderOptions):
   lines.push('  #  label                 kind         rate      range');
   for (const signal of header.signals) {
     const index = String(signal.index).padStart(3);
-    const label = signal.label.slice(0, 20).padEnd(21);
+    // Control characters are replaced, not printed. A label holding a newline would otherwise
+    // render as two rows and forge a signal the file does not contain; a tab would shift every
+    // column after it. EDF pads labels with spaces and says nothing about what else may be in
+    // them, so a writer can put anything there and a reader must not be steered by it.
+    const label = printable(signal.label).slice(0, 20).padEnd(21);
     const kind = signal.kind.padEnd(12);
     const rate = formatRate(signal).padEnd(9);
     const range =
@@ -91,11 +115,19 @@ export function formatHeader(header: EdfHeader, options?: FormatHeaderOptions):
 
   if (header.diagnostics.length > 0) {
     lines.push('');
-    const counts = new Map<string, number>();
-    for (const diagnostic of header.diagnostics) {
-      counts.set(diagnostic.severity, (counts.get(diagnostic.severity) ?? 0) + 1);
-    }
-    const summary = [...counts].map(([severity, count]) => `${count} ${severity}`).join(', ');
+    // Fixed error-warning-info order, matching `formatValidationReport` since 0.2.15. Ordering by
+    // arrival meant two files with the same diagnostics could summarise them differently.
+    const counted = summarizeDiagnostics(header.diagnostics);
+    const summary = (
+      [
+        [counted.errors, 'error'],
+        [counted.warnings, 'warning'],
+        [counted.infos, 'info'],
+      ] as const
+    )
+      .filter(([count]) => count > 0)
+      .map(([count, severity]) => `${count} ${severity}`)
+      .join(', ');
     lines.push(`${header.diagnostics.length} diagnostic(s): ${summary}`);
     lines.push('Call formatDiagnostics(header.diagnostics) for the detail.');
   }