edfcore 0.4.254 → 0.4.255
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/cli-run.d.ts +4 -4
- package/dist/cli-run.js +4 -4
- package/dist/cli.d.ts +3 -2
- package/dist/cli.d.ts.map +1 -1
- package/dist/cli.js +3 -2
- package/dist/cli.js.map +1 -1
- package/dist/constants.d.ts +1 -1
- package/dist/constants.js +1 -1
- package/dist/diagnostics/summary.d.ts +3 -0
- package/dist/diagnostics/summary.d.ts.map +1 -1
- package/dist/diagnostics/summary.js +3 -0
- package/dist/diagnostics/summary.js.map +1 -1
- package/dist/format-report.d.ts +4 -2
- package/dist/format-report.d.ts.map +1 -1
- package/dist/format-report.js +4 -2
- package/dist/format-report.js.map +1 -1
- package/docs/CHANGELOG.md +12 -0
- package/package.json +1 -1
- package/src/cli-run.ts +4 -4
- package/src/cli.ts +3 -2
- package/src/constants.ts +1 -1
- package/src/diagnostics/summary.ts +3 -0
- package/src/format-report.ts +4 -2
package/dist/cli-run.d.ts
CHANGED
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* What `npx edfcore` actually does, with the process factored out.
|
|
3
3
|
*
|
|
4
|
-
* `cli.ts` is a shell that supplies real `node:fs` and `node:process`; everything
|
|
5
|
-
* here, behind an injected `CliIo`. That is not ceremony — a CLI tested by
|
|
6
|
-
* can only be tested once the package is built, so the tests either skip in
|
|
7
|
-
* binary. This way the exit codes and the output are ordinary unit tests.
|
|
4
|
+
* Layer 7. `cli.ts` is a shell that supplies real `node:fs` and `node:process`; everything
|
|
5
|
+
* decidable lives here, behind an injected `CliIo`. That is not ceremony — a CLI tested by
|
|
6
|
+
* spawning a subprocess can only be tested once the package is built, so the tests either skip in
|
|
7
|
+
* CI or test a stale binary. This way the exit codes and the output are ordinary unit tests.
|
|
8
8
|
*/
|
|
9
9
|
/**
|
|
10
10
|
* A mistake in the command line, as opposed to a mistake in the file.
|
package/dist/cli-run.js
CHANGED
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* What `npx edfcore` actually does, with the process factored out.
|
|
3
3
|
*
|
|
4
|
-
* `cli.ts` is a shell that supplies real `node:fs` and `node:process`; everything
|
|
5
|
-
* here, behind an injected `CliIo`. That is not ceremony — a CLI tested by
|
|
6
|
-
* can only be tested once the package is built, so the tests either skip in
|
|
7
|
-
* binary. This way the exit codes and the output are ordinary unit tests.
|
|
4
|
+
* Layer 7. `cli.ts` is a shell that supplies real `node:fs` and `node:process`; everything
|
|
5
|
+
* decidable lives here, behind an injected `CliIo`. That is not ceremony — a CLI tested by
|
|
6
|
+
* spawning a subprocess can only be tested once the package is built, so the tests either skip in
|
|
7
|
+
* CI or test a stale binary. This way the exit codes and the output are ordinary unit tests.
|
|
8
8
|
*/
|
|
9
9
|
import { countAnnotationsByText } from './annotations-query.js';
|
|
10
10
|
import { trimEdfField } from './bytes/latin1.js';
|
package/dist/cli.d.ts
CHANGED
|
@@ -2,8 +2,9 @@
|
|
|
2
2
|
/**
|
|
3
3
|
* `npx edfcore` — looking at a file without writing code.
|
|
4
4
|
*
|
|
5
|
-
* A separate entry point, never re-exported from the barrel,
|
|
6
|
-
* `node:process`. The universal entry must stay free of Node
|
|
5
|
+
* Layer 7, and the package's `bin`. A separate entry point, never re-exported from the barrel,
|
|
6
|
+
* because it imports `node:fs` and `node:process`. The universal entry must stay free of Node
|
|
7
|
+
* built-ins, and a test asserts it.
|
|
7
8
|
*
|
|
8
9
|
* This file is only the wiring. The decisions live in `cli-run.ts`, so they can be tested without
|
|
9
10
|
* spawning a process against a build that may not exist yet.
|
package/dist/cli.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA
|
|
1
|
+
{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA;;;;;;;;;;;;;GAaG"}
|
package/dist/cli.js
CHANGED
|
@@ -2,8 +2,9 @@
|
|
|
2
2
|
/**
|
|
3
3
|
* `npx edfcore` — looking at a file without writing code.
|
|
4
4
|
*
|
|
5
|
-
* A separate entry point, never re-exported from the barrel,
|
|
6
|
-
* `node:process`. The universal entry must stay free of Node
|
|
5
|
+
* Layer 7, and the package's `bin`. A separate entry point, never re-exported from the barrel,
|
|
6
|
+
* because it imports `node:fs` and `node:process`. The universal entry must stay free of Node
|
|
7
|
+
* built-ins, and a test asserts it.
|
|
7
8
|
*
|
|
8
9
|
* This file is only the wiring. The decisions live in `cli-run.ts`, so they can be tested without
|
|
9
10
|
* spawning a process against a build that may not exist yet.
|
package/dist/cli.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA
|
|
1
|
+
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA;;;;;;;;;;;;;GAaG;AAEH,8FAA8F;AAC9F,2FAA2F;AAC3F,OAAO,KAAK,cAAc,MAAM,kBAAkB,CAAC;AACnD,qDAAqD;AACrD,uFAAuF;AACvF,OAAO,KAAK,WAAW,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAc,aAAa,EAAE,SAAS,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AAC5E,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAezC,MAAM,EAAE,GAAW,cAAmC,CAAC;AACvD,MAAM,IAAI,GACP,WAAoD,CAAC,OAAO;IAC5D,WAAsC,CAAC;AAE1C;;;;;;;;;;;;GAYG;AACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,KAAK,EAAE,EAAE;IAChC,IAAI,KAAK,EAAE,IAAI,KAAK,OAAO;QAAE,MAAM,KAAK,CAAC;AAC3C,CAAC,CAAC,CAAC;AAEH,MAAM,EAAE,GAAU;IAChB;;;;;;OAMG;IACH,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE;QACvB,IAAI,CAAC;YACH,OAAO,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QACjC,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,IAAK,KAAkC,EAAE,IAAI,KAAK,QAAQ;gBAAE,MAAM,KAAK,CAAC;YACxE,MAAM,IAAI,KAAK,CACb,GAAG,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,2DAA2D;gBAChF,iBAAiB,CACpB,CAAC;QACJ,CAAC;IACH,CAAC;IACD,GAAG,EAAE,CAAC,IAAI,EAAE,EAAE;QACZ,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC1B,CAAC;IACD,GAAG,EAAE,CAAC,IAAI,EAAE,EAAE;QACZ,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC1B,CAAC;CACF,CAAC;AAEF,KAAK,UAAU,IAAI;IACjB,IAAI,CAAC;QACH,IAAI,CAAC,QAAQ,GAAG,MAAM,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;IAClE,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,6FAA6F;QAC7F,4CAA4C;QAC5C,MAAM,OAAO,GAAG,UAAU,CAAC,KAAK,CAAC,IAAI,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QAC5F,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,YAAY,OAAO,IAAI,CAAC,CAAC;QAC3C,4FAA4F;QAC5F,0FAA0F;QAC1F,gBAAgB;QAChB,IAAI,CAAC,QAAQ,GAAG,KAAK,YAAY,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACzD,CAAC;AACH,CAAC;AAED,8FAA8F;AAC9F,+FAA+F;AAC/F,6DAA6D;AAC7D,KAAK,IAAI,EAAE,CAAC"}
|
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.255";
|
|
113
113
|
//# sourceMappingURL=constants.d.ts.map
|
package/dist/constants.js
CHANGED
|
@@ -1,6 +1,9 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* Diagnostics, counted.
|
|
3
3
|
*
|
|
4
|
+
* Layer 1. Imports one type module and nothing else, which is what lets any layer summarise a
|
|
5
|
+
* diagnostics array without taking on a dependency.
|
|
6
|
+
*
|
|
4
7
|
* `formatDiagnostics` turns a list into text for a human. This turns the same list into numbers
|
|
5
8
|
* for a program: the question "is anything wrong with this header, and how wrong" has no answer
|
|
6
9
|
* on `EdfHeader` — `validateRecording` produces a `report.ok`, but that needs a full scan, and a
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"summary.d.ts","sourceRoot":"","sources":["../../src/diagnostics/summary.ts"],"names":[],"mappings":"AAAA
|
|
1
|
+
{"version":3,"file":"summary.d.ts","sourceRoot":"","sources":["../../src/diagnostics/summary.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,iBAAiB,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAEjF;;;;GAIG;AACH,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,IAAI,EAAE,iBAAiB,CAAC;IACjC,QAAQ,CAAC,QAAQ,EAAE,WAAW,CAAC;IAC/B,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;CACxB;AAED;;;;GAIG;AACH,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,yFAAyF;IACzF,QAAQ,CAAC,KAAK,EAAE,WAAW,GAAG,SAAS,CAAC;IACxC,uEAAuE;IACvE,QAAQ,CAAC,MAAM,EAAE,SAAS,YAAY,EAAE,CAAC;CAC1C;AAKD;;;;;;;;;;GAUG;AACH,wBAAgB,oBAAoB,CAAC,WAAW,EAAE,SAAS,aAAa,EAAE,GAAG,oBAAoB,CAiChG"}
|
|
@@ -1,6 +1,9 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* Diagnostics, counted.
|
|
3
3
|
*
|
|
4
|
+
* Layer 1. Imports one type module and nothing else, which is what lets any layer summarise a
|
|
5
|
+
* diagnostics array without taking on a dependency.
|
|
6
|
+
*
|
|
4
7
|
* `formatDiagnostics` turns a list into text for a human. This turns the same list into numbers
|
|
5
8
|
* for a program: the question "is anything wrong with this header, and how wrong" has no answer
|
|
6
9
|
* on `EdfHeader` — `validateRecording` produces a `report.ok`, but that needs a full scan, and a
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"summary.js","sourceRoot":"","sources":["../../src/diagnostics/summary.ts"],"names":[],"mappings":"AAAA
|
|
1
|
+
{"version":3,"file":"summary.js","sourceRoot":"","sources":["../../src/diagnostics/summary.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AA+BH,qEAAqE;AACrE,MAAM,aAAa,GAAgC,EAAE,KAAK,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,CAAC;AAErF;;;;;;;;;;GAUG;AACH,MAAM,UAAU,oBAAoB,CAAC,WAAqC;IACxE,IAAI,MAAM,GAAG,CAAC,CAAC;IACf,IAAI,QAAQ,GAAG,CAAC,CAAC;IACjB,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,KAA8B,CAAC;IAEnC,MAAM,MAAM,GAAG,IAAI,GAAG,EAAmC,CAAC;IAC1D,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;QACrC,IAAI,UAAU,CAAC,QAAQ,KAAK,OAAO;YAAE,MAAM,IAAI,CAAC,CAAC;aAC5C,IAAI,UAAU,CAAC,QAAQ,KAAK,SAAS;YAAE,QAAQ,IAAI,CAAC,CAAC;;YACrD,KAAK,IAAI,CAAC,CAAC;QAEhB,IAAI,KAAK,KAAK,SAAS,IAAI,aAAa,CAAC,UAAU,CAAC,QAAQ,CAAC,GAAG,aAAa,CAAC,KAAK,CAAC,EAAE,CAAC;YACrF,KAAK,GAAG,UAAU,CAAC,QAAQ,CAAC;QAC9B,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC;QAC7C,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,IAAI,EAAE;YAC1B,IAAI,EAAE,UAAU,CAAC,IAAI;YACrB,qFAAqF;YACrF,QAAQ,EAAE,QAAQ,EAAE,QAAQ,IAAI,UAAU,CAAC,QAAQ;YACnD,KAAK,EAAE,CAAC,QAAQ,EAAE,KAAK,IAAI,CAAC,CAAC,GAAG,CAAC;SAClC,CAAC,CAAC;IACL,CAAC;IAED,OAAO;QACL,KAAK,EAAE,WAAW,CAAC,MAAM;QACzB,MAAM;QACN,QAAQ;QACR,KAAK;QACL,KAAK;QACL,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,KAAK,CAAC,CAAC;KAC9E,CAAC;AACJ,CAAC"}
|
package/dist/format-report.d.ts
CHANGED
|
@@ -1,8 +1,10 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* A validation report as text.
|
|
3
3
|
*
|
|
4
|
-
* Part of `edfcore/validate`.
|
|
5
|
-
*
|
|
4
|
+
* Layer 7, and pure. Part of `edfcore/validate`.
|
|
5
|
+
*
|
|
6
|
+
* `validateRecording` returns a report a program can branch on; this turns it into the thing a
|
|
7
|
+
* person reads at the end of a CI job or a conformance sweep.
|
|
6
8
|
*
|
|
7
9
|
* The one judgement it makes is what to lead with. A sweep over a damaged file can produce six
|
|
8
10
|
* figures of diagnostics — `TIMEKEEPING_TAL_MISSING` is per record — and a wall of them buries
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"format-report.d.ts","sourceRoot":"","sources":["../src/format-report.ts"],"names":[],"mappings":"AAAA
|
|
1
|
+
{"version":3,"file":"format-report.d.ts","sourceRoot":"","sources":["../src/format-report.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAKH,OAAO,KAAK,EAAE,SAAS,EAAE,mBAAmB,EAAE,gBAAgB,EAAE,MAAM,YAAY,CAAC;AASnF;;;;;GAKG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,gBAAgB,EACxB,OAAO,CAAC,EAAE,mBAAmB,GAC5B,MAAM,CAoER;AAED,qFAAqF;AACrF,YAAY,EAAE,SAAS,EAAE,CAAC"}
|
package/dist/format-report.js
CHANGED
|
@@ -1,8 +1,10 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* A validation report as text.
|
|
3
3
|
*
|
|
4
|
-
* Part of `edfcore/validate`.
|
|
5
|
-
*
|
|
4
|
+
* Layer 7, and pure. Part of `edfcore/validate`.
|
|
5
|
+
*
|
|
6
|
+
* `validateRecording` returns a report a program can branch on; this turns it into the thing a
|
|
7
|
+
* person reads at the end of a CI job or a conformance sweep.
|
|
6
8
|
*
|
|
7
9
|
* The one judgement it makes is what to lead with. A sweep over a damaged file can produce six
|
|
8
10
|
* figures of diagnostics — `TIMEKEEPING_TAL_MISSING` is per record — and a wall of them buries
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"format-report.js","sourceRoot":"","sources":["../src/format-report.ts"],"names":[],"mappings":"AAAA
|
|
1
|
+
{"version":3,"file":"format-report.js","sourceRoot":"","sources":["../src/format-report.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAC5D,OAAO,EAAE,oBAAoB,EAAE,MAAM,0BAA0B,CAAC;AAChE,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAGhD,+EAA+E;AAC/E,MAAM,iBAAiB,GAAG,EAAE,CAAC;AAE7B,SAAS,SAAS,CAAC,KAAa,EAAE,IAAY;IAC5C,OAAO,GAAG,KAAK,IAAI,IAAI,GAAG,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,EAAE,CAAC;AACrD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB,CACpC,MAAwB,EACxB,OAA6B;IAE7B,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,MAAM,MAAM,GAAG,OAAO,EAAE,MAAM,CAAC;IAE/B,8EAA8E;IAC9E,MAAM,OAAO,GAAG,oBAAoB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;IAEzD,MAAM,OAAO,GAAG,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC;IAC5C,MAAM,UAAU,GACd,OAAO,CAAC,KAAK,KAAK,CAAC;QACjB,CAAC,CAAC,gBAAgB;QAClB,CAAC,CACG;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;IACpB,KAAK,CAAC,IAAI,CAAC,GAAG,OAAO,MAAM,UAAU,EAAE,CAAC,CAAC;IACzC,KAAK,CAAC,IAAI,CACR,WAAW,SAAS,CAAC,MAAM,CAAC,cAAc,EAAE,QAAQ,CAAC,IAAI;QACvD,QAAQ,MAAM,CAAC,SAAS,CAAC,cAAc,CAAC,OAAO,CAAC,QAAQ,CAC3D,CAAC;IAEF,IAAI,OAAO,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC9B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACvB,gFAAgF;QAChF,KAAK,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;YAC7C,KAAK,CAAC,IAAI,CAAC,KAAK,MAAM,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;QACxD,CAAC;IACH,CAAC;IAED,IAAI,MAAM,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAC;QACtC,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YACvC,MAAM,MAAM,GAAG,MAAM,EAAE,OAAO,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;YAClD,wFAAwF;YACxF,0FAA0F;YAC1F,wFAAwF;YACxF,MAAM,IAAI,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,UAAU,KAAK,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;YAC5F,MAAM,QAAQ,GACZ,KAAK,CAAC,sBAAsB,GAAG,CAAC;gBAC9B,CAAC,CAAC,KAAK,KAAK,CAAC,sBAAsB,6BAA6B;gBAChE,CAAC,CAAC,EAAE,CAAC;YACT,KAAK,CAAC,IAAI,CACR,KAAK,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,kBAAkB,KAAK,KAAK,CAAC,kBAAkB,EAAE;gBACzF,SAAS,KAAK,CAAC,WAAW,CAAC,cAAc,CAAC,OAAO,CAAC,WAAW,QAAQ,EAAE,CAC1E,CAAC;QACJ,CAAC;IACH,CAAC;IAED,IAAI,MAAM,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAClC,MAAM,QAAQ,GAAG,OAAO,EAAE,QAAQ,IAAI,iBAAiB,CAAC;QACxD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CACR,iBAAiB,CAAC,MAAM,CAAC,WAAW,EAAE;YACpC,QAAQ;YACR,GAAG,CAAC,OAAO,EAAE,YAAY,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,YAAY,EAAE,OAAO,CAAC,YAAY,EAAE,CAAC;SACvF,CAAC,CACH,CAAC;IACJ,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}
|
package/docs/CHANGELOG.md
CHANGED
|
@@ -6,6 +6,18 @@ 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.255
|
|
10
|
+
|
|
11
|
+
- **Gave four modules the layer declaration every other one has.** `AGENTS.md` states this
|
|
12
|
+
project's single architectural rule — `bytes`/`text` → `diagnostics` → `header`/`decode`/`tal` →
|
|
13
|
+
`time` → `io` → entry points, and a module may only import from a lower layer — and each file
|
|
14
|
+
repeats its own position on the first line of its docblock. Forty-eight of fifty-two did.
|
|
15
|
+
`cli.ts`, `cli-run.ts`, `diagnostics/summary.ts` and `format-report.ts` did not, so the one
|
|
16
|
+
invariant the codebase has was stated everywhere except where someone had skipped it, and
|
|
17
|
+
nothing noticed because nothing read the declarations. A missing one is now a failing test.
|
|
18
|
+
`removeComments: false` ships these docblocks in `dist/*.d.ts`, so a layer is also what an
|
|
19
|
+
editor shows on hover.
|
|
20
|
+
|
|
9
21
|
## 0.4.254
|
|
10
22
|
|
|
11
23
|
- **Wrote down** that the `archive/pre-squash-2026-08-16` branch cannot be deleted, in the "things
|
package/package.json
CHANGED
package/src/cli-run.ts
CHANGED
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* What `npx edfcore` actually does, with the process factored out.
|
|
3
3
|
*
|
|
4
|
-
* `cli.ts` is a shell that supplies real `node:fs` and `node:process`; everything
|
|
5
|
-
* here, behind an injected `CliIo`. That is not ceremony — a CLI tested by
|
|
6
|
-
* can only be tested once the package is built, so the tests either skip in
|
|
7
|
-
* binary. This way the exit codes and the output are ordinary unit tests.
|
|
4
|
+
* Layer 7. `cli.ts` is a shell that supplies real `node:fs` and `node:process`; everything
|
|
5
|
+
* decidable lives here, behind an injected `CliIo`. That is not ceremony — a CLI tested by
|
|
6
|
+
* spawning a subprocess can only be tested once the package is built, so the tests either skip in
|
|
7
|
+
* CI or test a stale binary. This way the exit codes and the output are ordinary unit tests.
|
|
8
8
|
*/
|
|
9
9
|
|
|
10
10
|
import { countAnnotationsByText } from './annotations-query.js';
|
package/src/cli.ts
CHANGED
|
@@ -2,8 +2,9 @@
|
|
|
2
2
|
/**
|
|
3
3
|
* `npx edfcore` — looking at a file without writing code.
|
|
4
4
|
*
|
|
5
|
-
* A separate entry point, never re-exported from the barrel,
|
|
6
|
-
* `node:process`. The universal entry must stay free of Node
|
|
5
|
+
* Layer 7, and the package's `bin`. A separate entry point, never re-exported from the barrel,
|
|
6
|
+
* because it imports `node:fs` and `node:process`. The universal entry must stay free of Node
|
|
7
|
+
* built-ins, and a test asserts it.
|
|
7
8
|
*
|
|
8
9
|
* This file is only the wiring. The decisions live in `cli-run.ts`, so they can be tested without
|
|
9
10
|
* spawning a process against a build that may not exist yet.
|
package/src/constants.ts
CHANGED
|
@@ -1,6 +1,9 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* Diagnostics, counted.
|
|
3
3
|
*
|
|
4
|
+
* Layer 1. Imports one type module and nothing else, which is what lets any layer summarise a
|
|
5
|
+
* diagnostics array without taking on a dependency.
|
|
6
|
+
*
|
|
4
7
|
* `formatDiagnostics` turns a list into text for a human. This turns the same list into numbers
|
|
5
8
|
* for a program: the question "is anything wrong with this header, and how wrong" has no answer
|
|
6
9
|
* on `EdfHeader` — `validateRecording` produces a `report.ok`, but that needs a full scan, and a
|
package/src/format-report.ts
CHANGED
|
@@ -1,8 +1,10 @@
|
|
|
1
1
|
/**
|
|
2
2
|
* A validation report as text.
|
|
3
3
|
*
|
|
4
|
-
* Part of `edfcore/validate`.
|
|
5
|
-
*
|
|
4
|
+
* Layer 7, and pure. Part of `edfcore/validate`.
|
|
5
|
+
*
|
|
6
|
+
* `validateRecording` returns a report a program can branch on; this turns it into the thing a
|
|
7
|
+
* person reads at the end of a CI job or a conformance sweep.
|
|
6
8
|
*
|
|
7
9
|
* The one judgement it makes is what to lead with. A sweep over a damaged file can produce six
|
|
8
10
|
* figures of diagnostics — `TIMEKEEPING_TAL_MISSING` is per record — and a wall of them buries
|