edfcore 0.4.515 → 0.4.517
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/constants.d.ts +1 -1
- package/dist/constants.js +1 -1
- package/docs/CHANGELOG.md +42 -0
- package/package.json +1 -1
- package/src/constants.ts +1 -1
package/dist/constants.d.ts
CHANGED
|
@@ -109,5 +109,5 @@ export declare const SIGNAL_FIELD_BLOCK_OFFSETS: {
|
|
|
109
109
|
readonly reserved: 224;
|
|
110
110
|
};
|
|
111
111
|
/** Published package version. Kept in sync with package.json by a test. */
|
|
112
|
-
export declare const VERSION = "0.4.
|
|
112
|
+
export declare const VERSION = "0.4.517";
|
|
113
113
|
//# sourceMappingURL=constants.d.ts.map
|
package/dist/constants.js
CHANGED
package/docs/CHANGELOG.md
CHANGED
|
@@ -6,6 +6,48 @@ alone does not tell you whether you were affected.
|
|
|
6
6
|
edfcore is pre-1.0. Patch releases have carried behaviour changes where the old behaviour was a
|
|
7
7
|
defect; those are called out below.
|
|
8
8
|
|
|
9
|
+
## 0.4.517
|
|
10
|
+
|
|
11
|
+
- **Added** the convention `api-types.md` states in bold above every table on the page: "A field
|
|
12
|
+
that may be absent is `T | undefined`, and the key is always there." It is what lets a caller
|
|
13
|
+
write `header.startTime.resolvedDate === undefined` instead of `'resolvedDate' in
|
|
14
|
+
header.startTime`, and destructure a result without guarding each name. It was prose.
|
|
15
|
+
- The type half is read out of `src/types.ts`: every optional member must live in an interface the
|
|
16
|
+
caller constructs — the ten `…Options` and `…Selection` types. An optional member on a result is
|
|
17
|
+
not a compile error anywhere. It quietly makes one field a name TypeScript will not let you read
|
|
18
|
+
without a guard.
|
|
19
|
+
- The runtime half is the one a type cannot catch. A result assembled with a conditional spread —
|
|
20
|
+
`...(date === undefined ? {} : { resolvedDate: date })` — satisfies `T | undefined` and omits the
|
|
21
|
+
key, so `Object.keys` comes up short, `in` is false, and a JSON round-trip loses a field the
|
|
22
|
+
table says is always there. Twenty result interfaces are now instantiated and every declared
|
|
23
|
+
member checked with `Object.hasOwn`; writing that spread into `resolveStartTime` fails the run.
|
|
24
|
+
- The fixture is built to leave fields ABSENT rather than present: an unreadable start date, a
|
|
25
|
+
signal with a degenerate digital range so it has no scale, annotations with no duration, and a
|
|
26
|
+
discontinuous file so one segment has a gap before it and one has none. Twenty fields come back
|
|
27
|
+
`undefined`, and the run asserts that count — on a file where everything happened to be defined
|
|
28
|
+
the check would pass while testing nothing.
|
|
29
|
+
|
|
30
|
+
## 0.4.516
|
|
31
|
+
|
|
32
|
+
- **Added** the third column of the `EdfHeader` and `EdfSignal` tables on `api-types.md`, executed.
|
|
33
|
+
`type-tables.test.ts` already checks that both tables list every member of their interface in
|
|
34
|
+
declaration order, which settles the first column. The third is where the tables do their work:
|
|
35
|
+
most rows do not describe a field, they derive it — `dataByteLength` is
|
|
36
|
+
`recordCount * recordByteLength`, `sampleCount` is `samplesPerRecord * header.recordCount`,
|
|
37
|
+
`recordByteOffset` is an offset "within one data record" rather than within the file, and
|
|
38
|
+
`sampleRateHz` is `undefined` "exactly when that duration is `0`". None of it had been run.
|
|
39
|
+
- A wrong rule there is worse than a missing one, because these are the sentences a reader uses
|
|
40
|
+
instead of measuring. Someone who takes `recordByteOffset` for a file offset writes a seek that
|
|
41
|
+
lands in the header; someone who divides by `sampleRateHz` because the table did not warn them
|
|
42
|
+
gets `Infinity` on a legal annotations-only recording.
|
|
43
|
+
- Each rule is asserted twice: that the page still states it, quoting the row's own words, and that
|
|
44
|
+
the library obeys it. The quote is what makes the pair worth anything — a rule reworded on the
|
|
45
|
+
page fails here instead of drifting away from a test that only knew the behaviour.
|
|
46
|
+
- Reading the rows needs two things the obvious parser gets wrong. `recordByteLength` is a row in
|
|
47
|
+
BOTH tables and means different things in each, so a row is looked up under its `###` heading;
|
|
48
|
+
and a type cell writes a union as `number \| undefined`, so cells split on an unescaped pipe or
|
|
49
|
+
the rest of that row lands in the wrong column.
|
|
50
|
+
|
|
9
51
|
## 0.4.515
|
|
10
52
|
|
|
11
53
|
- **Added** the thirteen pairs behind one sentence on `cli.md`: "Each is accepted and ignored by
|
package/package.json
CHANGED
package/src/constants.ts
CHANGED