edf2csv 0.8.74 → 0.8.75

package/dist/convert/options.d.ts

@@ -93,6 +93,29 @@ export declare function assertPlanInput(input: {
     recordDuration?: unknown;
     recordCount?: unknown;
 }): void;
+/**
+ * The two numbers a window is measured against, apart from the channel list.
+ *
+ * `resolveRange` is exported on its own and has its own signature block on the api page, and
+ * `buildPlan` calls it — so it was covered only from above. Called directly it took both
+ * numbers unexamined and answered with a range:
+ *
+ *     resolveRange({ recordDuration: 1 })       // recordCount undefined
+ *     // { startSeconds: 0, endSeconds: null, startRecord: 0, endRecord: 0 }
+ *
+ * A range over no records, returned as a fact about a recording. Its own opening comment
+ * already says why that is the wrong answer — "no error, no warning, and a range read back as
+ * `startSeconds: null, startRecord: null`, which is the 'takes the whole recording without
+ * saying so'" — about the three fields it does check. `resolveRange(42)` went the same way,
+ * since reading `.start` off a number is `undefined` rather than a throw.
+ *
+ * Split out rather than calling `assertPlanInput`, which would demand a channel list this
+ * function never looks at.
+ */
+export declare function assertRecordShape(input: {
+    recordDuration?: unknown;
+    recordCount?: unknown;
+}): void;
 /**
  * The recording to read, checked before it is opened.
  *

package/dist/convert/options.js

@@ -256,7 +256,29 @@ export function assertSignals(signals) {
  */
 export function assertPlanInput(input) {
     assertSignals(input.signals);
-    const { recordDuration, recordCount } = input;
+    assertRecordShape(input);
+}
+/**
+ * The two numbers a window is measured against, apart from the channel list.
+ *
+ * `resolveRange` is exported on its own and has its own signature block on the api page, and
+ * `buildPlan` calls it — so it was covered only from above. Called directly it took both
+ * numbers unexamined and answered with a range:
+ *
+ *     resolveRange({ recordDuration: 1 })       // recordCount undefined
+ *     // { startSeconds: 0, endSeconds: null, startRecord: 0, endRecord: 0 }
+ *
+ * A range over no records, returned as a fact about a recording. Its own opening comment
+ * already says why that is the wrong answer — "no error, no warning, and a range read back as
+ * `startSeconds: null, startRecord: null`, which is the 'takes the whole recording without
+ * saying so'" — about the three fields it does check. `resolveRange(42)` went the same way,
+ * since reading `.start` off a number is `undefined` rather than a throw.
+ *
+ * Split out rather than calling `assertPlanInput`, which would demand a channel list this
+ * function never looks at.
+ */
+export function assertRecordShape(input) {
+    const { recordDuration, recordCount } = input ?? {};
     if (typeof recordDuration !== 'number' || !Number.isFinite(recordDuration) || recordDuration <= 0) {
         throw new OptionError(`recordDuration must be a positive number of seconds, got ${describeValue(recordDuration)}.`);
     }

package/dist/convert/options.js.map

@@ -1 +1 @@
-{"version":3,"file":"options.js","sourceRoot":"","sources":["../../src/convert/options.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,yFAAyF;AACzF,MAAM,OAAO,WAAY,SAAQ,KAAK;IACpC,YAAY,OAAe;QACzB,KAAK,CAAC,OAAO,CAAC,CAAC;QACf,IAAI,CAAC,IAAI,GAAG,aAAa,CAAC;IAC5B,CAAC;CACF;AAED;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,MAAM,YAAY,GAAG,EAAE,CAAC;AAE/B,MAAM,UAAU,aAAa,CAAC,OAe7B;IACC,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,IAAI,QAAQ,KAAK,SAAS,EAAE,CAAC;QAC3B,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,QAAQ,GAAG,CAAC,IAAI,QAAQ,GAAG,YAAY,EAAE,CAAC;YAC3E,MAAM,IAAI,WAAW,CACnB,iDAAiD,YAAY,SAAS,aAAa,CAAC,QAAQ,CAAC,GAAG,CACjG,CAAC;QACJ,CAAC;IACH,CAAC;IAED;;;;;;;;;;;MAWE;IACF,KAAK,MAAM,IAAI,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,KAAK,CAAU,EAAE,CAAC;QACzD,MAAM,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5B,IAAI,KAAK,KAAK,SAAS;YAAE,SAAS;QAClC,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;YACzD,MAAM,IAAI,WAAW,CAAC,GAAG,IAAI,qCAAqC,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC7F,CAAC;QACD;;;;;;;;;UASE;QACF,IAAI,IAAI,KAAK,UAAU,IAAI,KAAK,GAAG,CAAC,EAAE,CAAC;YACrC,MAAM,IAAI,WAAW,CAAC,iDAAiD,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAClG,CAAC;IACH,CAAC;IAED;;;;;;;;MAQE;IACF,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC;IAC3B,IAAI,MAAM,KAAK,SAAS,IAAI,MAAM,KAAK,MAAM,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QACnE,MAAM,IAAI,WAAW,CAAC,wCAAwC,aAAa,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IAC1F,CAAC;IAED;;;;;;;;;;;;;;;MAeE;IACF;;;;;;;;;;;;;;MAcE;IACF;;;;;;;;;;;;;;;MAeE;IACF,MAAM,EAAE,SAAS,EAAE,GAAG,OAAO,CAAC;IAC9B,IAAI,SAAS,KAAK,SAAS,IAAI,OAAO,SAAS,KAAK,QAAQ,EAAE,CAAC;QAC7D,MAAM,IAAI,WAAW,CAAC,iCAAiC,aAAa,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IACtF,CAAC;IACD,IAAI,SAAS,KAAK,EAAE,EAAE,CAAC;QACrB,MAAM,IAAI,WAAW,CAAC,kEAAkE,CAAC,CAAC;IAC5F,CAAC;IAED;;;;;;;;;;;;;;;MAeE;IACF,KAAK,MAAM,IAAI,IAAI,CAAC,iBAAiB,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,CAAU,EAAE,CAAC;QAChG,MAAM,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5B,IAAI,KAAK,KAAK,SAAS,IAAI,OAAO,KAAK,KAAK,SAAS,EAAE,CAAC;YACtD,MAAM,IAAI,WAAW,CAAC,GAAG,IAAI,+BAA+B,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QACvF,CAAC;IACH,CAAC;IAED;;;;;;;;;;;;;;;;;MAiBE;IACF,MAAM,EAAE,UAAU,EAAE,GAAG,OAAO,CAAC;IAC/B,IAAI,UAAU,KAAK,SAAS,IAAI,OAAO,UAAU,KAAK,UAAU,EAAE,CAAC;QACjE,MAAM,IAAI,WAAW,CAAC,sCAAsC,aAAa,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC;IAC5F,CAAC;IAED,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,IAAI,QAAQ,KAAK,SAAS,EAAE,CAAC;QAC3B;;;;;;;UAOE;QACF,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,OAAO,IAAI,KAAK,QAAQ,CAAC,EAAE,CAAC;YAClF,MAAM,IAAI,WAAW,CAAC,iDAAiD,aAAa,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;QACrG,CAAC;QACD,IAAI,QAAQ,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;YACjD,MAAM,IAAI,WAAW,CAAC,gDAAgD,CAAC,CAAC;QAC1E,CAAC;IACH,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,aAAa,CAAC,OAAgB;IAC5C,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5B,MAAM,IAAI,WAAW,CACnB,uDAAuD,aAAa,CAAC,OAAO,CAAC,GAAG,CACjF,CAAC;IACJ,CAAC;IACD,MAAM,KAAK,GAAG,OAAO,CAAC,SAAS,CAC7B,CAAC,MAAM,EAAE,EAAE,CACT,OAAO,MAAM,KAAK,QAAQ;QAC1B,MAAM,KAAK,IAAI;QACf,OAAQ,MAA8B,CAAC,KAAK,KAAK,QAAQ,CAC5D,CAAC;IACF,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;QACjB,MAAM,IAAI,WAAW,CACnB,WAAW,KAAK,yCAAyC,aAAa,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,GAAG,CAC1F,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,MAAM,UAAU,eAAe,CAAC,KAI/B;IACC,aAAa,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAC7B,MAAM,EAAE,cAAc,EAAE,WAAW,EAAE,GAAG,KAAK,CAAC;IAC9C,IAAI,OAAO,cAAc,KAAK,QAAQ,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,IAAI,cAAc,IAAI,CAAC,EAAE,CAAC;QAClG,MAAM,IAAI,WAAW,CACnB,4DAA4D,aAAa,CAAC,cAAc,CAAC,GAAG,CAC7F,CAAC;IACJ,CAAC;IACD,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,WAAW,CAAC,IAAK,WAAsB,GAAG,CAAC,EAAE,CAAC;QAClE,MAAM,IAAI,WAAW,CACnB,2DAA2D,aAAa,CAAC,WAAW,CAAC,GAAG,CACzF,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,eAAe,CAAC,KAAc;IAC5C,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,IAAI,WAAW,CAAC,4CAA4C,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC7F,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,aAAa,CAAC,KAAc;IAC1C,OAAO,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,CAAC;AAC5F,CAAC","sourcesContent":["/**\n * Checking the options a caller passed, before anything is written.\n *\n * The command line has always validated these — `--decimals 1.5` is a usage error and always\n * has been — and the library did not, so the same value behaved differently depending on how\n * it arrived. `convert(file, { decimals: NaN })` resolved successfully having written whole\n * numbers into a column the caller had asked for decimals in, which is the worst of the\n * three: no error, no warning, and output that looks like a deliberate choice. `decimals: -1`\n * reached `toFixed` and came back as a bare RangeError from deep inside the formatter, naming\n * nothing the caller had written. `start: NaN` created the output directory, wrote\n * signals.csv, and then failed with a message about the input being unreadable — a partial\n * conversion, blamed on the file.\n *\n * These run at the top of `buildPlan`, which every path goes through before a directory is\n * created or a stream is opened, so a rejected option leaves nothing behind.\n */\n\n/** A problem with the options a caller passed, as opposed to a problem with the file. */\nexport class OptionError extends Error {\n  constructor(message: string) {\n    super(message);\n    this.name = 'OptionError';\n  }\n}\n\n/**\n * The largest `--decimals` accepts, and what both documentation pages state.\n *\n * Not a limit of `toFixed`, whatever this comment used to say. `toFixed` takes 0 to 100 and\n * throws a RangeError at 101 — which is exactly the belief `MAX_DERIVED_DECIMALS` in\n * edf/scale.ts exists to correct, having once clamped the *derived* precision to 20 on the\n * same wrong grounds and rounded a magnetometer channel needing 23 places onto a grid three\n * digital codes wide, losing 69% of its samples in silence.\n *\n * Twenty is a bound on a number a person types by hand, not on what the format can express.\n * The derived precision, which nobody types, runs to 100 and says so.\n */\nexport const MAX_DECIMALS = 20;\n\nexport function assertOptions(options: {\n  decimals?: number | undefined;\n  start?: number | undefined;\n  duration?: number | undefined;\n  end?: number | undefined;\n  layout?: string | undefined;\n  channels?: readonly string[] | undefined;\n  outputDir?: string | undefined;\n  annotationsOnly?: boolean | undefined;\n  gzip?: boolean | undefined;\n  bom?: boolean | undefined;\n  force?: boolean | undefined;\n  checksum?: boolean | undefined;\n  toStdout?: boolean | undefined;\n  onProgress?: unknown;\n}): void {\n  const { decimals } = options;\n  if (decimals !== undefined) {\n    if (!Number.isInteger(decimals) || decimals < 0 || decimals > MAX_DECIMALS) {\n      throw new OptionError(\n        `decimals must be a whole number between 0 and ${MAX_DECIMALS}, got ${describeValue(decimals)}.`,\n      );\n    }\n  }\n\n  /*\n    `start` and `end` are positions on the recording's own clock, `duration` is a length.\n\n    All three were held above zero, which is right for a length and wrong for a position: a\n    recording timed from its first record's timekeeping annotation may sit before zero, and\n    -100 is then where its first sample is. So a caller could read `plan.range` back as\n    `recordingStartSeconds: -100` and not be allowed to ask for it — the same wall\n    `parseTimeSpec` put in front of the command line until 0.5.120.\n\n    Non-finite is still refused for all three, since NaN reaches a comparison as false and\n    would take the whole recording without saying so.\n  */\n  for (const name of ['start', 'duration', 'end'] as const) {\n    const value = options[name];\n    if (value === undefined) continue;\n    if (typeof value !== 'number' || !Number.isFinite(value)) {\n      throw new OptionError(`${name} must be a number of seconds, got ${describeValue(value)}.`);\n    }\n    /*\n      A length below zero is refused for what it is, rather than for not being a number.\n\n      `duration: -1` came back as \"duration must be a number of seconds, got -1\", which is\n      not the reason and not true: -1 is a number of seconds, and this same call accepts it\n      for `start` and for `end`, where a recording timed from before zero makes it an\n      ordinary position. What is wrong is that a duration is a length, and no length is\n      negative — which is what the command line says for the same value, and what the API\n      reference has always said this check enforces.\n    */\n    if (name === 'duration' && value < 0) {\n      throw new OptionError(`duration is a length of time, so it cannot be ${describeValue(value)}.`);\n    }\n  }\n\n  /*\n    Two words, and a caller who writes a third means something the tool cannot do.\n\n    The command line has always rejected `--layout tall`. The library took it, put it in\n    `plan.layout` for the caller to read back, and wrote the wide layout — so a programmatic\n    caller with a typo got a conversion that was not the one they asked for, described by a\n    plan that agreed with the typo. Every other option that has a shape is checked here; this\n    one was added in 0.5.0 and never joined them.\n  */\n  const { layout } = options;\n  if (layout !== undefined && layout !== 'wide' && layout !== 'long') {\n    throw new OptionError(`layout must be \"wide\" or \"long\", got ${describeValue(layout)}.`);\n  }\n\n  /*\n    A selection that names nothing is not the absence of a selection.\n\n    `buildPlan` asks `options.channels.length > 0` before selecting, so an empty array fell\n    through to the branch that means \"no channels option was given\" — and `convert(file, {\n    channels: [] })` wrote every channel in the recording, resolved, and said nothing. That is\n    the one shape of this the command line has always refused, in as many words: \"Returning\n    undefined here would mean 'no --channels given' and convert everything, which is the\n    opposite of what someone passing an empty list is asking for.\"\n\n    A list of blanks is the same request written differently — it is what `''.split(',')`\n    produces, which is how a caller building the array from user input arrives here — and it\n    reached `selectChannels` and came back \"No channels were selected\", a sentence about the\n    file rather than about the call. Both are the option being wrong, so both are refused\n    here, before a directory exists.\n  */\n  /*\n    An empty destination, refused here rather than by the filesystem.\n\n    `convert(file, { outputDir: '' })` went the whole way to `mkdir('')` and came back a\n    `ConversionError`: \"Cannot create \\\"\\\": part of the path does not exist. Check the path\n    exists and that you have permission to write there.\" Advice about a path and a permission\n    for a value that is neither, and a failure class that means the conversion went wrong\n    where the option did.\n\n    The command line refused this at 0.6.x, with the reasoning that `--out \"$DEST\"` and `DEST`\n    unset is how it gets written by accident — and left the library, which a caller building\n    the path in code reaches the same way. Not trimmed, for the reason given there: a\n    directory whose name is a space is a strange thing to ask for, but it is a thing the\n    filesystem has and a path is not a keyword.\n  */\n  /*\n    And its shape, which was the one option with a value and no check on it.\n\n    The empty string was refused and nothing else was, so `outputDir` failed two ways that the\n    paragraph above describes for the flags. A value of the wrong type reached `path.join` and\n    came back as a Node error about an argument this caller never passed:\n\n        convert('rec.edf', { outputDir: 42 })\n        TypeError: The \"path\" argument must be of type string. Received type number (42)\n\n    And `null` — which is what `JSON.parse` of a config file gives for a field left unset, the\n    same door `1` and `'true'` come through — was not an error at all. It is not `undefined`,\n    so it never meant \"use the default\", but every read of it is `?? default` or a truthiness\n    test, so that is what it did: the rows went to `<recording>_csv` beside the input, a\n    directory the caller had not named, and the run reported success.\n  */\n  const { outputDir } = options;\n  if (outputDir !== undefined && typeof outputDir !== 'string') {\n    throw new OptionError(`outputDir must be a path, got ${describeValue(outputDir)}.`);\n  }\n  if (outputDir === '') {\n    throw new OptionError('outputDir is empty. Give a directory, for example \"./converted\".');\n  }\n\n  /*\n    The six flags, every one of which is read as `=== true` where it is read.\n\n    Which means a value that is not a boolean is not merely tolerated: it is taken as the\n    opposite of what it says. `convert(file, { annotationsOnly: 'true' })` wrote every signal\n    the caller had asked to leave out; `{ gzip: 1 }` wrote plain CSVs under names ending\n    `.csv`, so a caller who then opened `signals.csv.gz` found nothing there. No error, no\n    warning, and output that looks like a deliberate choice — which is the sentence at the\n    top of this file, describing the case it was written for.\n\n    `1` and `'true'` are how a flag arrives from `JSON.parse` of a config file, from a query\n    string, or from a CLI wrapper that did not coerce; none of them is a caller being\n    careless in a way TypeScript would catch, since the callers this reaches are the ones not\n    using it. The layout check above states the rule these were missing from: \"Every other\n    option that has a shape is checked here.\"\n  */\n  for (const name of ['annotationsOnly', 'gzip', 'bom', 'force', 'checksum', 'toStdout'] as const) {\n    const value = options[name];\n    if (value !== undefined && typeof value !== 'boolean') {\n      throw new OptionError(`${name} must be true or false, got ${describeValue(value)}.`);\n    }\n  }\n\n  /*\n    The one option that is called rather than read, and the only one that was not checked.\n\n    `convert` invokes it as `options.onProgress?.(...)` once a record has been written, so a\n    value that is not a function passes every check here, opens the destination, writes rows\n    into it and then fails from inside the loop:\n\n        convert('rec.edf', { outputDir: 'out', onProgress: 'every record' })\n        ConversionError: The onProgress callback threw: options.onProgress is not a function\n\n    A callback that threw is what that sentence reports, and no callback was given; the text\n    after the colon names an expression inside this package. Worse is what it leaves: `out`\n    exists with a half-written signals.csv in it, which is the case the paragraph at the top\n    of this file describes — \"`start: NaN` created the output directory, wrote signals.csv,\n    and then failed with a message about the input being unreadable — a partial conversion,\n    blamed on the file\". Checked here, the same call writes nothing and says which argument\n    is wrong.\n  */\n  const { onProgress } = options;\n  if (onProgress !== undefined && typeof onProgress !== 'function') {\n    throw new OptionError(`onProgress must be a function, got ${describeValue(onProgress)}.`);\n  }\n\n  const { channels } = options;\n  if (channels !== undefined) {\n    /*\n      A list of strings, checked as one. `selectChannels` calls `.trim()` on every term, so a\n      caller who passed the string `'ECG'` had it iterated character by character and was told\n      `No channel named \"E\"`, and one who passed `[1]` — a position, reasonably enough — got\n      `TypeError: rawTerm.trim is not a function` out of the middle of the selector, naming\n      nothing they had written. Both are the option being the wrong shape, which is the case\n      this function exists for.\n    */\n    if (!Array.isArray(channels) || channels.some((term) => typeof term !== 'string')) {\n      throw new OptionError(`channels must be a list of channel names, got ${describeValue(channels)}.`);\n    }\n    if (channels.every((term) => term.trim() === '')) {\n      throw new OptionError('channels was given but lists no channel names.');\n    }\n  }\n}\n\n/**\n * The channel list two exported functions take, checked the way their other argument is.\n *\n * `selectChannels(signals, terms)` has checked `terms` since 0.6.x — \"`'ECG'` was iterated\n * character by character and answered `No channel named \\\"E\\\"`… naming nothing the caller had\n * written\" — and never checked `signals`, which is the argument in front of it. Passing one\n * signal where the list goes, or a header where its `signals` goes, came back as\n * `TypeError: signals.filter is not a function`: a local of this package, over a value the\n * caller did write.\n *\n * `buildColumnNames` is worse off, because a string is iterable. `buildColumnNames('ECG')`\n * returned `Map { null => 'undefined_chundefined' }` and no error at all — a column name for a\n * channel that does not exist, keyed by a position that is not one.\n *\n * The bad entry is named by position rather than the whole list being printed back: a header\n * may declare hundreds of channels, and a message is not the place for all of them.\n */\nexport function assertSignals(signals: unknown): void {\n  if (!Array.isArray(signals)) {\n    throw new OptionError(\n      `signals must be the channel list from a header, got ${describeValue(signals)}.`,\n    );\n  }\n  const wrong = signals.findIndex(\n    (signal) =>\n      typeof signal !== 'object' ||\n      signal === null ||\n      typeof (signal as { index?: unknown }).index !== 'number',\n  );\n  if (wrong !== -1) {\n    throw new OptionError(\n      `signals[${wrong}] is not a channel from a header, got ${describeValue(signals[wrong])}.`,\n    );\n  }\n}\n\n/**\n * What `buildPlan` is told about the recording, checked the way what it is asked for is.\n *\n * `assertOptions` runs at the top of `buildPlan` and covers the second argument completely.\n * The first was not looked at, and it is the one carrying the numbers every figure in the plan\n * is derived from. Two of them missing produced a plan rather than an error:\n *\n *     buildPlan({ signals, recordDuration: 1 }, {})\n *     // groups: 3, estimate.rows: 0, range.endSeconds: null\n *\n * A plan saying the conversion writes nothing, handed back as an answer — which is the \"takes\n * the whole recording without saying so\" this checker exists to stop, one field over. A record\n * count below zero was worse: it came back as\n *\n *     TimeRangeError: --start 0s is at or past the end of this -5s recording.\n *\n * a flag the caller never passed, about a recording that cannot exist, blaming the request for\n * the input. And `recordDuration: '1'` was coerced by the arithmetic and accepted, where the\n * same string is refused for `end` two functions down.\n *\n * A real header cannot produce any of them: the parser refuses a record duration that is not a\n * positive number, \"Infinity\" included.\n */\nexport function assertPlanInput(input: {\n  signals?: unknown;\n  recordDuration?: unknown;\n  recordCount?: unknown;\n}): void {\n  assertSignals(input.signals);\n  const { recordDuration, recordCount } = input;\n  if (typeof recordDuration !== 'number' || !Number.isFinite(recordDuration) || recordDuration <= 0) {\n    throw new OptionError(\n      `recordDuration must be a positive number of seconds, got ${describeValue(recordDuration)}.`,\n    );\n  }\n  if (!Number.isInteger(recordCount) || (recordCount as number) < 0) {\n    throw new OptionError(\n      `recordCount must be a whole number of data records, got ${describeValue(recordCount)}.`,\n    );\n  }\n}\n\n/**\n * The recording to read, checked before it is opened.\n *\n * `EdfFile.open` hands whatever it is given to `fs`, and the refusal comes back as an\n * `EdfError` coded `UNREADABLE`, hinted \"Check the path is spelled the way it is on disk and\n * that you have permission to read it\" — advice about a path, over a value that is not one,\n * filed as a problem with the recording rather than with the call. `convert({ input: 'a.edf' })`,\n * which is the option-bag shape the second parameter has, answered `Cannot read \"[object\n * Object]\"`; `convert(['a.edf', 'b.edf'])` answered `Cannot read \"a.edf,b.edf\"`, a path the\n * caller never wrote, because `String` of an array joins it with commas.\n *\n * The empty string is left to `fs`, which has no such file and says so truthfully — the same\n * reasoning `outputDir` states for not trimming: a path is not a keyword.\n */\nexport function assertInputPath(input: unknown): void {\n  if (typeof input !== 'string') {\n    throw new OptionError(`input must be a path to a recording, got ${describeValue(input)}.`);\n  }\n}\n\n/**\n * How a rejected value reads in the refusal: numbers bare, everything else quoted so its\n * type is visible.\n *\n * `JSON.stringify` has no text for a function or a symbol — it returns `undefined`, not a\n * string — so `layout: () => 'long'` came back as `layout must be \"wide\" or \"long\", got\n * undefined.`, which names the one value that does not raise this: every option here is\n * optional, and `undefined` is how a caller says they are not passing it. `input` was worse,\n * since `convert(undefined)` and `convert(someFunction)` then produced the same sentence, and\n * the first is a forgotten argument while the second is a wrong one.\n *\n * Exported because time-range.ts had the identical function, fixed there and not here — the\n * same two-copies-of-one-helper the derived precision and the pluraliser were each pulled\n * together for.\n */\nexport function describeValue(value: unknown): string {\n  return typeof value === 'number' ? String(value) : JSON.stringify(value) ?? String(value);\n}\n"]}
+{"version":3,"file":"options.js","sourceRoot":"","sources":["../../src/convert/options.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,yFAAyF;AACzF,MAAM,OAAO,WAAY,SAAQ,KAAK;IACpC,YAAY,OAAe;QACzB,KAAK,CAAC,OAAO,CAAC,CAAC;QACf,IAAI,CAAC,IAAI,GAAG,aAAa,CAAC;IAC5B,CAAC;CACF;AAED;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,MAAM,YAAY,GAAG,EAAE,CAAC;AAE/B,MAAM,UAAU,aAAa,CAAC,OAe7B;IACC,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,IAAI,QAAQ,KAAK,SAAS,EAAE,CAAC;QAC3B,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,QAAQ,GAAG,CAAC,IAAI,QAAQ,GAAG,YAAY,EAAE,CAAC;YAC3E,MAAM,IAAI,WAAW,CACnB,iDAAiD,YAAY,SAAS,aAAa,CAAC,QAAQ,CAAC,GAAG,CACjG,CAAC;QACJ,CAAC;IACH,CAAC;IAED;;;;;;;;;;;MAWE;IACF,KAAK,MAAM,IAAI,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,KAAK,CAAU,EAAE,CAAC;QACzD,MAAM,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5B,IAAI,KAAK,KAAK,SAAS;YAAE,SAAS;QAClC,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;YACzD,MAAM,IAAI,WAAW,CAAC,GAAG,IAAI,qCAAqC,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC7F,CAAC;QACD;;;;;;;;;UASE;QACF,IAAI,IAAI,KAAK,UAAU,IAAI,KAAK,GAAG,CAAC,EAAE,CAAC;YACrC,MAAM,IAAI,WAAW,CAAC,iDAAiD,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAClG,CAAC;IACH,CAAC;IAED;;;;;;;;MAQE;IACF,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC;IAC3B,IAAI,MAAM,KAAK,SAAS,IAAI,MAAM,KAAK,MAAM,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QACnE,MAAM,IAAI,WAAW,CAAC,wCAAwC,aAAa,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IAC1F,CAAC;IAED;;;;;;;;;;;;;;;MAeE;IACF;;;;;;;;;;;;;;MAcE;IACF;;;;;;;;;;;;;;;MAeE;IACF,MAAM,EAAE,SAAS,EAAE,GAAG,OAAO,CAAC;IAC9B,IAAI,SAAS,KAAK,SAAS,IAAI,OAAO,SAAS,KAAK,QAAQ,EAAE,CAAC;QAC7D,MAAM,IAAI,WAAW,CAAC,iCAAiC,aAAa,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IACtF,CAAC;IACD,IAAI,SAAS,KAAK,EAAE,EAAE,CAAC;QACrB,MAAM,IAAI,WAAW,CAAC,kEAAkE,CAAC,CAAC;IAC5F,CAAC;IAED;;;;;;;;;;;;;;;MAeE;IACF,KAAK,MAAM,IAAI,IAAI,CAAC,iBAAiB,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,CAAU,EAAE,CAAC;QAChG,MAAM,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5B,IAAI,KAAK,KAAK,SAAS,IAAI,OAAO,KAAK,KAAK,SAAS,EAAE,CAAC;YACtD,MAAM,IAAI,WAAW,CAAC,GAAG,IAAI,+BAA+B,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QACvF,CAAC;IACH,CAAC;IAED;;;;;;;;;;;;;;;;;MAiBE;IACF,MAAM,EAAE,UAAU,EAAE,GAAG,OAAO,CAAC;IAC/B,IAAI,UAAU,KAAK,SAAS,IAAI,OAAO,UAAU,KAAK,UAAU,EAAE,CAAC;QACjE,MAAM,IAAI,WAAW,CAAC,sCAAsC,aAAa,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC;IAC5F,CAAC;IAED,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,IAAI,QAAQ,KAAK,SAAS,EAAE,CAAC;QAC3B;;;;;;;UAOE;QACF,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,OAAO,IAAI,KAAK,QAAQ,CAAC,EAAE,CAAC;YAClF,MAAM,IAAI,WAAW,CAAC,iDAAiD,aAAa,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;QACrG,CAAC;QACD,IAAI,QAAQ,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;YACjD,MAAM,IAAI,WAAW,CAAC,gDAAgD,CAAC,CAAC;QAC1E,CAAC;IACH,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,aAAa,CAAC,OAAgB;IAC5C,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5B,MAAM,IAAI,WAAW,CACnB,uDAAuD,aAAa,CAAC,OAAO,CAAC,GAAG,CACjF,CAAC;IACJ,CAAC;IACD,MAAM,KAAK,GAAG,OAAO,CAAC,SAAS,CAC7B,CAAC,MAAM,EAAE,EAAE,CACT,OAAO,MAAM,KAAK,QAAQ;QAC1B,MAAM,KAAK,IAAI;QACf,OAAQ,MAA8B,CAAC,KAAK,KAAK,QAAQ,CAC5D,CAAC;IACF,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;QACjB,MAAM,IAAI,WAAW,CACnB,WAAW,KAAK,yCAAyC,aAAa,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,GAAG,CAC1F,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,MAAM,UAAU,eAAe,CAAC,KAI/B;IACC,aAAa,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAC7B,iBAAiB,CAAC,KAAK,CAAC,CAAC;AAC3B,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAGjC;IACC,MAAM,EAAE,cAAc,EAAE,WAAW,EAAE,GAAG,KAAK,IAAI,EAAE,CAAC;IACpD,IAAI,OAAO,cAAc,KAAK,QAAQ,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,IAAI,cAAc,IAAI,CAAC,EAAE,CAAC;QAClG,MAAM,IAAI,WAAW,CACnB,4DAA4D,aAAa,CAAC,cAAc,CAAC,GAAG,CAC7F,CAAC;IACJ,CAAC;IACD,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,WAAW,CAAC,IAAK,WAAsB,GAAG,CAAC,EAAE,CAAC;QAClE,MAAM,IAAI,WAAW,CACnB,2DAA2D,aAAa,CAAC,WAAW,CAAC,GAAG,CACzF,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,eAAe,CAAC,KAAc;IAC5C,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,IAAI,WAAW,CAAC,4CAA4C,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC7F,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,aAAa,CAAC,KAAc;IAC1C,OAAO,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,CAAC;AAC5F,CAAC","sourcesContent":["/**\n * Checking the options a caller passed, before anything is written.\n *\n * The command line has always validated these — `--decimals 1.5` is a usage error and always\n * has been — and the library did not, so the same value behaved differently depending on how\n * it arrived. `convert(file, { decimals: NaN })` resolved successfully having written whole\n * numbers into a column the caller had asked for decimals in, which is the worst of the\n * three: no error, no warning, and output that looks like a deliberate choice. `decimals: -1`\n * reached `toFixed` and came back as a bare RangeError from deep inside the formatter, naming\n * nothing the caller had written. `start: NaN` created the output directory, wrote\n * signals.csv, and then failed with a message about the input being unreadable — a partial\n * conversion, blamed on the file.\n *\n * These run at the top of `buildPlan`, which every path goes through before a directory is\n * created or a stream is opened, so a rejected option leaves nothing behind.\n */\n\n/** A problem with the options a caller passed, as opposed to a problem with the file. */\nexport class OptionError extends Error {\n  constructor(message: string) {\n    super(message);\n    this.name = 'OptionError';\n  }\n}\n\n/**\n * The largest `--decimals` accepts, and what both documentation pages state.\n *\n * Not a limit of `toFixed`, whatever this comment used to say. `toFixed` takes 0 to 100 and\n * throws a RangeError at 101 — which is exactly the belief `MAX_DERIVED_DECIMALS` in\n * edf/scale.ts exists to correct, having once clamped the *derived* precision to 20 on the\n * same wrong grounds and rounded a magnetometer channel needing 23 places onto a grid three\n * digital codes wide, losing 69% of its samples in silence.\n *\n * Twenty is a bound on a number a person types by hand, not on what the format can express.\n * The derived precision, which nobody types, runs to 100 and says so.\n */\nexport const MAX_DECIMALS = 20;\n\nexport function assertOptions(options: {\n  decimals?: number | undefined;\n  start?: number | undefined;\n  duration?: number | undefined;\n  end?: number | undefined;\n  layout?: string | undefined;\n  channels?: readonly string[] | undefined;\n  outputDir?: string | undefined;\n  annotationsOnly?: boolean | undefined;\n  gzip?: boolean | undefined;\n  bom?: boolean | undefined;\n  force?: boolean | undefined;\n  checksum?: boolean | undefined;\n  toStdout?: boolean | undefined;\n  onProgress?: unknown;\n}): void {\n  const { decimals } = options;\n  if (decimals !== undefined) {\n    if (!Number.isInteger(decimals) || decimals < 0 || decimals > MAX_DECIMALS) {\n      throw new OptionError(\n        `decimals must be a whole number between 0 and ${MAX_DECIMALS}, got ${describeValue(decimals)}.`,\n      );\n    }\n  }\n\n  /*\n    `start` and `end` are positions on the recording's own clock, `duration` is a length.\n\n    All three were held above zero, which is right for a length and wrong for a position: a\n    recording timed from its first record's timekeeping annotation may sit before zero, and\n    -100 is then where its first sample is. So a caller could read `plan.range` back as\n    `recordingStartSeconds: -100` and not be allowed to ask for it — the same wall\n    `parseTimeSpec` put in front of the command line until 0.5.120.\n\n    Non-finite is still refused for all three, since NaN reaches a comparison as false and\n    would take the whole recording without saying so.\n  */\n  for (const name of ['start', 'duration', 'end'] as const) {\n    const value = options[name];\n    if (value === undefined) continue;\n    if (typeof value !== 'number' || !Number.isFinite(value)) {\n      throw new OptionError(`${name} must be a number of seconds, got ${describeValue(value)}.`);\n    }\n    /*\n      A length below zero is refused for what it is, rather than for not being a number.\n\n      `duration: -1` came back as \"duration must be a number of seconds, got -1\", which is\n      not the reason and not true: -1 is a number of seconds, and this same call accepts it\n      for `start` and for `end`, where a recording timed from before zero makes it an\n      ordinary position. What is wrong is that a duration is a length, and no length is\n      negative — which is what the command line says for the same value, and what the API\n      reference has always said this check enforces.\n    */\n    if (name === 'duration' && value < 0) {\n      throw new OptionError(`duration is a length of time, so it cannot be ${describeValue(value)}.`);\n    }\n  }\n\n  /*\n    Two words, and a caller who writes a third means something the tool cannot do.\n\n    The command line has always rejected `--layout tall`. The library took it, put it in\n    `plan.layout` for the caller to read back, and wrote the wide layout — so a programmatic\n    caller with a typo got a conversion that was not the one they asked for, described by a\n    plan that agreed with the typo. Every other option that has a shape is checked here; this\n    one was added in 0.5.0 and never joined them.\n  */\n  const { layout } = options;\n  if (layout !== undefined && layout !== 'wide' && layout !== 'long') {\n    throw new OptionError(`layout must be \"wide\" or \"long\", got ${describeValue(layout)}.`);\n  }\n\n  /*\n    A selection that names nothing is not the absence of a selection.\n\n    `buildPlan` asks `options.channels.length > 0` before selecting, so an empty array fell\n    through to the branch that means \"no channels option was given\" — and `convert(file, {\n    channels: [] })` wrote every channel in the recording, resolved, and said nothing. That is\n    the one shape of this the command line has always refused, in as many words: \"Returning\n    undefined here would mean 'no --channels given' and convert everything, which is the\n    opposite of what someone passing an empty list is asking for.\"\n\n    A list of blanks is the same request written differently — it is what `''.split(',')`\n    produces, which is how a caller building the array from user input arrives here — and it\n    reached `selectChannels` and came back \"No channels were selected\", a sentence about the\n    file rather than about the call. Both are the option being wrong, so both are refused\n    here, before a directory exists.\n  */\n  /*\n    An empty destination, refused here rather than by the filesystem.\n\n    `convert(file, { outputDir: '' })` went the whole way to `mkdir('')` and came back a\n    `ConversionError`: \"Cannot create \\\"\\\": part of the path does not exist. Check the path\n    exists and that you have permission to write there.\" Advice about a path and a permission\n    for a value that is neither, and a failure class that means the conversion went wrong\n    where the option did.\n\n    The command line refused this at 0.6.x, with the reasoning that `--out \"$DEST\"` and `DEST`\n    unset is how it gets written by accident — and left the library, which a caller building\n    the path in code reaches the same way. Not trimmed, for the reason given there: a\n    directory whose name is a space is a strange thing to ask for, but it is a thing the\n    filesystem has and a path is not a keyword.\n  */\n  /*\n    And its shape, which was the one option with a value and no check on it.\n\n    The empty string was refused and nothing else was, so `outputDir` failed two ways that the\n    paragraph above describes for the flags. A value of the wrong type reached `path.join` and\n    came back as a Node error about an argument this caller never passed:\n\n        convert('rec.edf', { outputDir: 42 })\n        TypeError: The \"path\" argument must be of type string. Received type number (42)\n\n    And `null` — which is what `JSON.parse` of a config file gives for a field left unset, the\n    same door `1` and `'true'` come through — was not an error at all. It is not `undefined`,\n    so it never meant \"use the default\", but every read of it is `?? default` or a truthiness\n    test, so that is what it did: the rows went to `<recording>_csv` beside the input, a\n    directory the caller had not named, and the run reported success.\n  */\n  const { outputDir } = options;\n  if (outputDir !== undefined && typeof outputDir !== 'string') {\n    throw new OptionError(`outputDir must be a path, got ${describeValue(outputDir)}.`);\n  }\n  if (outputDir === '') {\n    throw new OptionError('outputDir is empty. Give a directory, for example \"./converted\".');\n  }\n\n  /*\n    The six flags, every one of which is read as `=== true` where it is read.\n\n    Which means a value that is not a boolean is not merely tolerated: it is taken as the\n    opposite of what it says. `convert(file, { annotationsOnly: 'true' })` wrote every signal\n    the caller had asked to leave out; `{ gzip: 1 }` wrote plain CSVs under names ending\n    `.csv`, so a caller who then opened `signals.csv.gz` found nothing there. No error, no\n    warning, and output that looks like a deliberate choice — which is the sentence at the\n    top of this file, describing the case it was written for.\n\n    `1` and `'true'` are how a flag arrives from `JSON.parse` of a config file, from a query\n    string, or from a CLI wrapper that did not coerce; none of them is a caller being\n    careless in a way TypeScript would catch, since the callers this reaches are the ones not\n    using it. The layout check above states the rule these were missing from: \"Every other\n    option that has a shape is checked here.\"\n  */\n  for (const name of ['annotationsOnly', 'gzip', 'bom', 'force', 'checksum', 'toStdout'] as const) {\n    const value = options[name];\n    if (value !== undefined && typeof value !== 'boolean') {\n      throw new OptionError(`${name} must be true or false, got ${describeValue(value)}.`);\n    }\n  }\n\n  /*\n    The one option that is called rather than read, and the only one that was not checked.\n\n    `convert` invokes it as `options.onProgress?.(...)` once a record has been written, so a\n    value that is not a function passes every check here, opens the destination, writes rows\n    into it and then fails from inside the loop:\n\n        convert('rec.edf', { outputDir: 'out', onProgress: 'every record' })\n        ConversionError: The onProgress callback threw: options.onProgress is not a function\n\n    A callback that threw is what that sentence reports, and no callback was given; the text\n    after the colon names an expression inside this package. Worse is what it leaves: `out`\n    exists with a half-written signals.csv in it, which is the case the paragraph at the top\n    of this file describes — \"`start: NaN` created the output directory, wrote signals.csv,\n    and then failed with a message about the input being unreadable — a partial conversion,\n    blamed on the file\". Checked here, the same call writes nothing and says which argument\n    is wrong.\n  */\n  const { onProgress } = options;\n  if (onProgress !== undefined && typeof onProgress !== 'function') {\n    throw new OptionError(`onProgress must be a function, got ${describeValue(onProgress)}.`);\n  }\n\n  const { channels } = options;\n  if (channels !== undefined) {\n    /*\n      A list of strings, checked as one. `selectChannels` calls `.trim()` on every term, so a\n      caller who passed the string `'ECG'` had it iterated character by character and was told\n      `No channel named \"E\"`, and one who passed `[1]` — a position, reasonably enough — got\n      `TypeError: rawTerm.trim is not a function` out of the middle of the selector, naming\n      nothing they had written. Both are the option being the wrong shape, which is the case\n      this function exists for.\n    */\n    if (!Array.isArray(channels) || channels.some((term) => typeof term !== 'string')) {\n      throw new OptionError(`channels must be a list of channel names, got ${describeValue(channels)}.`);\n    }\n    if (channels.every((term) => term.trim() === '')) {\n      throw new OptionError('channels was given but lists no channel names.');\n    }\n  }\n}\n\n/**\n * The channel list two exported functions take, checked the way their other argument is.\n *\n * `selectChannels(signals, terms)` has checked `terms` since 0.6.x — \"`'ECG'` was iterated\n * character by character and answered `No channel named \\\"E\\\"`… naming nothing the caller had\n * written\" — and never checked `signals`, which is the argument in front of it. Passing one\n * signal where the list goes, or a header where its `signals` goes, came back as\n * `TypeError: signals.filter is not a function`: a local of this package, over a value the\n * caller did write.\n *\n * `buildColumnNames` is worse off, because a string is iterable. `buildColumnNames('ECG')`\n * returned `Map { null => 'undefined_chundefined' }` and no error at all — a column name for a\n * channel that does not exist, keyed by a position that is not one.\n *\n * The bad entry is named by position rather than the whole list being printed back: a header\n * may declare hundreds of channels, and a message is not the place for all of them.\n */\nexport function assertSignals(signals: unknown): void {\n  if (!Array.isArray(signals)) {\n    throw new OptionError(\n      `signals must be the channel list from a header, got ${describeValue(signals)}.`,\n    );\n  }\n  const wrong = signals.findIndex(\n    (signal) =>\n      typeof signal !== 'object' ||\n      signal === null ||\n      typeof (signal as { index?: unknown }).index !== 'number',\n  );\n  if (wrong !== -1) {\n    throw new OptionError(\n      `signals[${wrong}] is not a channel from a header, got ${describeValue(signals[wrong])}.`,\n    );\n  }\n}\n\n/**\n * What `buildPlan` is told about the recording, checked the way what it is asked for is.\n *\n * `assertOptions` runs at the top of `buildPlan` and covers the second argument completely.\n * The first was not looked at, and it is the one carrying the numbers every figure in the plan\n * is derived from. Two of them missing produced a plan rather than an error:\n *\n *     buildPlan({ signals, recordDuration: 1 }, {})\n *     // groups: 3, estimate.rows: 0, range.endSeconds: null\n *\n * A plan saying the conversion writes nothing, handed back as an answer — which is the \"takes\n * the whole recording without saying so\" this checker exists to stop, one field over. A record\n * count below zero was worse: it came back as\n *\n *     TimeRangeError: --start 0s is at or past the end of this -5s recording.\n *\n * a flag the caller never passed, about a recording that cannot exist, blaming the request for\n * the input. And `recordDuration: '1'` was coerced by the arithmetic and accepted, where the\n * same string is refused for `end` two functions down.\n *\n * A real header cannot produce any of them: the parser refuses a record duration that is not a\n * positive number, \"Infinity\" included.\n */\nexport function assertPlanInput(input: {\n  signals?: unknown;\n  recordDuration?: unknown;\n  recordCount?: unknown;\n}): void {\n  assertSignals(input.signals);\n  assertRecordShape(input);\n}\n\n/**\n * The two numbers a window is measured against, apart from the channel list.\n *\n * `resolveRange` is exported on its own and has its own signature block on the api page, and\n * `buildPlan` calls it — so it was covered only from above. Called directly it took both\n * numbers unexamined and answered with a range:\n *\n *     resolveRange({ recordDuration: 1 })       // recordCount undefined\n *     // { startSeconds: 0, endSeconds: null, startRecord: 0, endRecord: 0 }\n *\n * A range over no records, returned as a fact about a recording. Its own opening comment\n * already says why that is the wrong answer — \"no error, no warning, and a range read back as\n * `startSeconds: null, startRecord: null`, which is the 'takes the whole recording without\n * saying so'\" — about the three fields it does check. `resolveRange(42)` went the same way,\n * since reading `.start` off a number is `undefined` rather than a throw.\n *\n * Split out rather than calling `assertPlanInput`, which would demand a channel list this\n * function never looks at.\n */\nexport function assertRecordShape(input: {\n  recordDuration?: unknown;\n  recordCount?: unknown;\n}): void {\n  const { recordDuration, recordCount } = input ?? {};\n  if (typeof recordDuration !== 'number' || !Number.isFinite(recordDuration) || recordDuration <= 0) {\n    throw new OptionError(\n      `recordDuration must be a positive number of seconds, got ${describeValue(recordDuration)}.`,\n    );\n  }\n  if (!Number.isInteger(recordCount) || (recordCount as number) < 0) {\n    throw new OptionError(\n      `recordCount must be a whole number of data records, got ${describeValue(recordCount)}.`,\n    );\n  }\n}\n\n/**\n * The recording to read, checked before it is opened.\n *\n * `EdfFile.open` hands whatever it is given to `fs`, and the refusal comes back as an\n * `EdfError` coded `UNREADABLE`, hinted \"Check the path is spelled the way it is on disk and\n * that you have permission to read it\" — advice about a path, over a value that is not one,\n * filed as a problem with the recording rather than with the call. `convert({ input: 'a.edf' })`,\n * which is the option-bag shape the second parameter has, answered `Cannot read \"[object\n * Object]\"`; `convert(['a.edf', 'b.edf'])` answered `Cannot read \"a.edf,b.edf\"`, a path the\n * caller never wrote, because `String` of an array joins it with commas.\n *\n * The empty string is left to `fs`, which has no such file and says so truthfully — the same\n * reasoning `outputDir` states for not trimming: a path is not a keyword.\n */\nexport function assertInputPath(input: unknown): void {\n  if (typeof input !== 'string') {\n    throw new OptionError(`input must be a path to a recording, got ${describeValue(input)}.`);\n  }\n}\n\n/**\n * How a rejected value reads in the refusal: numbers bare, everything else quoted so its\n * type is visible.\n *\n * `JSON.stringify` has no text for a function or a symbol — it returns `undefined`, not a\n * string — so `layout: () => 'long'` came back as `layout must be \"wide\" or \"long\", got\n * undefined.`, which names the one value that does not raise this: every option here is\n * optional, and `undefined` is how a caller says they are not passing it. `input` was worse,\n * since `convert(undefined)` and `convert(someFunction)` then produced the same sentence, and\n * the first is a forgotten argument while the second is a wrong one.\n *\n * Exported because time-range.ts had the identical function, fixed there and not here — the\n * same two-copies-of-one-helper the derived precision and the pluraliser were each pulled\n * together for.\n */\nexport function describeValue(value: unknown): string {\n  return typeof value === 'number' ? String(value) : JSON.stringify(value) ?? String(value);\n}\n"]}

package/dist/convert/time-range.js

@@ -7,7 +7,7 @@
  * a message that shows the forms that work.
  */
 import { fixed, formatDuration } from '../format/number.js';
-import { assertOptions, describeValue } from './options.js';
+import { assertOptions, assertRecordShape, describeValue, OptionError } from './options.js';
 export class TimeRangeError extends Error {
     constructor(message) {
         super(message);
@@ -327,7 +327,32 @@ export function resolveRange(options) {
       accepted; `{ start: '30' }` reached the past-the-end error and printed it with the value
       missing — `--start s is at or past the end of this 3s recording`.
     */
+    /*
+      The argument bag itself, before three properties are read off it.
+  
+      `resolveRange(42)` read `.start` off a number, which is `undefined` rather than a throw, and
+      went on to answer with a range; `resolveRange(null)` reached the same line and came back as
+      `TypeError: Cannot read properties of null (reading 'start')`, naming a property of this
+      function's own parameter.
+    */
+    if (options === null || typeof options !== 'object') {
+        throw new OptionError(`resolveRange takes one object — the recording's shape and the window asked of it — ` +
+            `got ${describeValue(options)}.`);
+    }
     assertOptions({ start: options.start, duration: options.duration, end: options.end });
+    /*
+      And the two numbers the window is measured against, which that call does not cover.
+  
+      `assertOptions` checks what was asked for; these are what it is asked of, and nothing
+      looked at them. `resolveRange({ start: 1, recordDuration: 1 })` — no `recordCount` —
+      answered `{ startSeconds: 0, endSeconds: null, startRecord: 0, endRecord: 0 }`: a window
+      over no records, handed back as a fact about a recording. `resolveRange(42)` reached the
+      same answer, since reading `.start` off a number is `undefined` rather than a throw.
+  
+      The check `buildPlan` has made since 0.8.64, made here too, because this is exported on
+      its own — the same sentence the line above this one is here for.
+    */
+    assertRecordShape(options);
     // For a continuous file the recording spans recordCount * recordDuration. A
     // discontinuous one does not: a 10-second recording with a 95-second gap in the
     // middle still ends at 105 seconds. Deriving the span from the records' real

package/dist/convert/time-range.js.map

@@ -1 +1 @@
-{"version":3,"file":"time-range.js","sourceRoot":"","sources":["../../src/convert/time-range.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAC5D,OAAO,EAAE,aAAa,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAE5D,MAAM,OAAO,cAAe,SAAQ,KAAK;IACvC,YAAY,OAAe;QACzB,KAAK,CAAC,OAAO,CAAC,CAAC;QACf,IAAI,CAAC,IAAI,GAAG,gBAAgB,CAAC;IAC/B,CAAC;CACF;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,YAAY,GAA2B;IAClD,CAAC,EAAE,IAAI;IACP,EAAE,EAAE,IAAI;IACR,GAAG,EAAE,IAAI;IACT,IAAI,EAAE,IAAI;IACV,KAAK,EAAE,IAAI;IACX,CAAC,EAAE,EAAE;IACL,GAAG,EAAE,EAAE;IACP,IAAI,EAAE,EAAE;IACR,MAAM,EAAE,EAAE;IACV,OAAO,EAAE,EAAE;IACX,CAAC,EAAE,CAAC;IACJ,GAAG,EAAE,CAAC;IACN,IAAI,EAAE,CAAC;IACP,MAAM,EAAE,CAAC;IACT,OAAO,EAAE,CAAC;IACV,EAAE,EAAE,KAAK;CACV,CAAC;AAEF,MAAM,KAAK,GAAG,6CAA6C,CAAC;AAC5D,MAAM,UAAU,GAAG,+BAA+B,CAAC;AAEnD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,UAAmB,EAAE,aAAa,GAAG,KAAK;IACrF;;;;;;;;;;;;;;;;;MAiBE;IACF,MAAM,KAAK,GACT,OAAO,UAAU,KAAK,QAAQ,IAAI,UAAU,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,WAAW,CAAC;IACxF;;;;;;;;;;MAUE;IACF,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,+BAA+B,aAAa,CAAC,KAAK,CAAC,IAAI;YAC7D,qFAAqF,CACxF,CAAC;IACJ,CAAC;IACD,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC3C,IAAI,OAAO,KAAK,EAAE,EAAE,CAAC;QACnB,MAAM,IAAI,cAAc,CAAC,GAAG,KAAK,mDAAmD,CAAC,CAAC;IACxF,CAAC;IAED;;;;;;;MAOE;IACF,MAAM,QAAQ,GAAG,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC;IACzC,MAAM,IAAI,GAAG,QAAQ,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;IAEnD;;;;;;;;;;;;;;;;;MAiBE;IACF,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QACzB,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,qDAAqD;YACrE,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,EAAE,CAC/C,CAAC;IACJ,CAAC;IACD,MAAM,MAAM,GAAG,CAAC,OAAe,EAAU,EAAE,CACzC,YAAY,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,aAAa,CAAC,CAAC;IAE3E,6BAA6B;IAC7B,IAAI,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC;IAE/D,iCAAiC;IACjC,MAAM,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,KAAK,EAAE,CAAC;QACV,MAAM,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAC5D,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACjC,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACjC,IAAI,OAAO,IAAI,EAAE,IAAI,OAAO,IAAI,EAAE,EAAE,CAAC;YACnC,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,iDAAiD,CACpE,CAAC;QACJ,CAAC;QACD,OAAO,MAAM,CAAC,KAAK,GAAG,IAAI,GAAG,OAAO,GAAG,EAAE,GAAG,OAAO,CAAC,CAAC;IACvD,CAAC;IAED,0FAA0F;IAC1F,UAAU,CAAC,SAAS,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,OAAO,GAAG,CAAC,CAAC;IAChB,IAAI,QAAQ,GAAG,CAAC,CAAC;IACjB,4FAA4F;IAC5F,IAAI,MAAM,GAAG,KAAK,CAAC;IACnB,IAAI,KAA6B,CAAC;IAClC,qFAAqF;IACrF,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,OAAO,CAAC,KAAK,GAAG,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAChD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAC5B,MAAM,KAAK,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC;QACjC,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,2BAA2B,IAAI,wBAAwB;gBACvE,+CAA+C,CAClD,CAAC;QACJ,CAAC;QAED,iFAAiF;QACjF,qFAAqF;QACrF,qFAAqF;QACrF,IAAI,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;YACpB,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,+BAA+B;gBAC/C,sCAAsC,CACzC,CAAC;QACJ,CAAC;QACD,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QAEhB,KAAK,IAAI,MAAM,GAAG,KAAK,CAAC;QACxB,OAAO,EAAE,CAAC;QACV,qFAAqF;QACrF,qFAAqF;QACrF,iEAAiE;QACjE,QAAQ,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC;QACjD,IAAI,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YAAE,MAAM,GAAG,IAAI,CAAC;IAC1C,CAAC;IAED,8DAA8D;IAC9D,IAAI,OAAO,KAAK,CAAC,IAAI,QAAQ,KAAK,IAAI,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC;QACnE,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,gCAAgC;YAChD,6DAA6D,CAChE,CAAC;IACJ,CAAC;IAED;;;;;;;;;;;;;MAaE;IACF,IAAI,MAAM,EAAE,CAAC;QACX,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,gDAAgD;YAChE,wBAAwB,KAAK,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,mBAAmB,EAAE,MAAM,CAAC,EAAE,CAC9E,CAAC;IACJ,CAAC;IAED,OAAO,MAAM,CAAC,KAAK,CAAC,CAAC;AACvB,CAAC;AAED,SAAS,YAAY,CACnB,KAAa,EACb,UAAkB,EAClB,KAAa,EACb,aAAsB;IAEtB;;;;;;;;;MASE;IACF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QAC5B,MAAM,IAAI,cAAc,CACtB,GAAG,UAAU,KAAK,KAAK,2DAA2D,CACnF,CAAC;IACJ,CAAC;IACD,IAAI,CAAC,aAAa,IAAI,KAAK,GAAG,CAAC,EAAE,CAAC;QAChC,MAAM,IAAI,cAAc,CAAC,GAAG,UAAU,KAAK,KAAK,qCAAqC,CAAC,CAAC;IACzF,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAmBD;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAG,IAAI,CAAC;AAEvC;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,YAAY,CAAC,IAAY;IACvC,IAAI,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC;QAAE,OAAO,kBAAkB,CAAC;IACrE,OAAO,IAAI,CAAC,GAAG,CAAC,kBAAkB,EAAE,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC;AACpD,CAAC;AAED,+EAA+E;AAC/E,MAAM,UAAU,mBAAmB,CACjC,IAAY,EACZ,YAAoB,EACpB,UAAkB,EAClB,YAAoB,kBAAkB;IAEtC,OAAO,IAAI,IAAI,YAAY,GAAG,SAAS,IAAI,IAAI,GAAG,UAAU,GAAG,SAAS,CAAC;AAC3E,CAAC;AAED,mFAAmF;AACnF,MAAM,UAAU,mBAAmB,CAAC,OAMnC;IACC;;;;;;;;MAQE;IACF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,CAAC;QAAE,OAAO,CAAC,CAAC;IACpD,MAAM,KAAK,GAAG,YAAY,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IACzC,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,CACrB,CAAC,OAAO,CAAC,YAAY,GAAG,KAAK,GAAG,OAAO,CAAC,WAAW,CAAC,GAAG,OAAO,CAAC,IAAI,CACpE,CAAC;IACF,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,CACrB,CAAC,OAAO,CAAC,UAAU,GAAG,KAAK,GAAG,OAAO,CAAC,WAAW,CAAC,GAAG,OAAO,CAAC,IAAI,CAClE,CAAC;IACF,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,gBAAgB,EAAE,KAAK,CAAC,CAAC,CAAC;IACrE,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,gBAAgB,EAAE,KAAK,CAAC,CAAC,CAAC;IACpE,OAAO,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,GAAG,KAAK,CAAC,CAAC;AACnC,CAAC;AAED;;;;;;;;GAQG;AACH,SAAS,WAAW,CAAC,CAAS,EAAE,CAAS;IACvC;;;;;;;;;;;;;;;;;;MAkBE;IACF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC;QAAE,OAAO,CAAC,KAAK,CAAC,CAAC;IAC/D,OAAO,IAAI,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,KAAK,CAAC;AACvE,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,YAAY,CAAC,OAiB5B;IACC;;;;;;;;;;MAUE;IACF,aAAa,CAAC,EAAE,KAAK,EAAE,OAAO,CAAC,KAAK,EAAE,QAAQ,EAAE,OAAO,CAAC,QAAQ,EAAE,GAAG,EAAE,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAEtF,4EAA4E;IAC5E,gFAAgF;IAChF,6EAA6E;IAC7E,4EAA4E;IAC5E,2EAA2E;IAC3E,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC,OAAO,CAAC,YAAY,EAAE,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,cAAc,CAAC,CAAC;IAErG,IAAI,OAAO,CAAC,QAAQ,KAAK,SAAS,IAAI,OAAO,CAAC,GAAG,KAAK,SAAS,EAAE,CAAC;QAChE,MAAM,IAAI,cAAc,CAAC,2CAA2C,CAAC,CAAC;IACxE,CAAC;IAED,MAAM,YAAY,GAAG,OAAO,CAAC,KAAK,IAAI,QAAQ,CAAC;IAE/C,IAAI,UAAkB,CAAC;IACvB,IAAI,OAAO,CAAC,QAAQ,KAAK,SAAS;QAAE,UAAU,GAAG,YAAY,GAAG,OAAO,CAAC,QAAQ,CAAC;SAC5E,IAAI,OAAO,CAAC,GAAG,KAAK,SAAS;QAAE,UAAU,GAAG,OAAO,CAAC,GAAG,CAAC;;QACxD,UAAU,GAAG,MAAM,CAAC;IAEzB;;;;;;;;;;;;;MAaE;IACF,IAAI,YAAY,IAAI,MAAM,IAAI,WAAW,CAAC,YAAY,EAAE,MAAM,CAAC,EAAE,CAAC;QAChE;;;;;;;;;UASE;QACF,MAAM,IAAI,cAAc;QACtB;;;;;;;;;;;;UAYE;QACF;;;;;;;UAOE;QACF,QAAQ,KAAK,CAAC;YACZ,CAAC,CAAC,WAAW,MAAM,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,iCAAiC;gBACnF,GAAG,cAAc,CAAC,MAAM,CAAC,aAAa;YACxC,CAAC,CAAC,WAAW,MAAM,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,iCAAiC;gBACnF,GAAG,cAAc,CAAC,MAAM,GAAG,QAAQ,CAAC,8BAA8B;gBAClE,GAAG,aAAa,CAAC,QAAQ,CAAC,OAAO,aAAa,CAAC,MAAM,CAAC,GAAG,CAC9D,CAAC;IACJ,CAAC;IACD,IAAI,UAAU,IAAI,YAAY,EAAE,CAAC;QAC/B,mFAAmF;QACnF,MAAM,QAAQ,GACZ,OAAO,CAAC,GAAG,KAAK,SAAS,IAAI,OAAO,CAAC,OAAO,KAAK,SAAS;YACxD,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,OAAO,EAAE,UAAU,CAAC;YACrC,CAAC,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC;QAChC,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC;QAC3D;;;;;;;;;;;;;;;;UAgBE;QACF,MAAM,IAAI,cAAc,CACtB,OAAO,CAAC,GAAG,KAAK,SAAS,IAAI,OAAO,CAAC,QAAQ,KAAK,SAAS;YACzD,CAAC,CAAC,cAAc,MAAM,CAAC,OAAO,CAAC,YAAY,EAAE,OAAO,CAAC,QAAQ,CAAC,sBAAsB;gBAClF,gDAAgD,UAAU,wBAAwB;gBAClF,sDAAsD;YACxD,CAAC,CAAC,gCAAgC,QAAQ,qCAAqC,UAAU,GAAG,CAC/F,CAAC;IACJ,CAAC;IAED,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;IAChD;;;;;;;;;;;;;;;;;;;;;;;;;;;MA2BE;IACF,MAAM,YAAY,GAAG,UAAU,IAAI,QAAQ,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,YAAY,EAAE,QAAQ,CAAC,CAAC;IAC9F,MAAM,EAAE,WAAW,EAAE,SAAS,EAAE,GAAG,aAAa,CAAC,OAAO,EAAE,YAAY,EAAE,UAAU,CAAC,CAAC;IAEpF,OAAO;QACL,YAAY,EAAE,YAAY;QAC1B,UAAU,EAAE,UAAU;QACtB,WAAW;QACX,SAAS;QACT;;;;;;;;UAQE;QACF,gBAAgB,EACd,CAAC,YAAY,IAAI,QAAQ,IAAI,WAAW,CAAC,YAAY,EAAE,QAAQ,CAAC,CAAC;YACjE,CAAC,UAAU,IAAI,MAAM,IAAI,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;QAC3D,qBAAqB,EAAE,QAAQ;QAC/B,mBAAmB,EAAE,MAAM;KAC5B,CAAC;AACJ,CAAC;AAED,SAAS,IAAI,CACX,YAA6C,EAC7C,WAAmB,EACnB,cAAsB;IAEtB,IAAI,CAAC,YAAY,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC/C,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,WAAW,GAAG,cAAc,EAAE,CAAC;IAC/D,CAAC;IACD,IAAI,QAAQ,GAAG,QAAQ,CAAC;IACxB,IAAI,MAAM,GAAG,CAAC,QAAQ,CAAC;IACvB,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;QACjC,IAAI,KAAK,GAAG,QAAQ;YAAE,QAAQ,GAAG,KAAK,CAAC;QACvC,IAAI,KAAK,GAAG,cAAc,GAAG,MAAM;YAAE,MAAM,GAAG,KAAK,GAAG,cAAc,CAAC;IACvE,CAAC;IACD,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3D,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,WAAW,GAAG,cAAc,EAAE,CAAC;IAC/D,CAAC;IACD,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;AAC9B,CAAC;AAED,sEAAsE;AACtE,SAAS,aAAa,CACpB,OAAwG,EACxG,YAAoB,EACpB,UAAkB;IAElB,MAAM,MAAM,GAAG,OAAO,CAAC,YAAY,CAAC;IACpC,IAAI,CAAC,MAAM,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACnC;;;;;;;;;;;UAWE;QACF,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,YAAY,GAAG,OAAO,CAAC,cAAc,CAAC,CAAC,CAAC;QAC7E,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,WAAW,EAAE,IAAI,CAAC,IAAI,CAAC,UAAU,GAAG,OAAO,CAAC,cAAc,CAAC,CAAC,CAAC;QAC3F,OAAO,KAAK,GAAG,IAAI,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC;IACnG,CAAC;IAED,IAAI,KAAK,GAAG,OAAO,CAAC,WAAW,CAAC;IAChC,IAAI,IAAI,GAAG,CAAC,CAAC;IACb,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACvC,MAAM,KAAK,GAAG,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC7B,IAAI,KAAK,GAAG,OAAO,CAAC,cAAc,GAAG,YAAY,IAAI,KAAK,GAAG,UAAU,EAAE,CAAC;YACxE,IAAI,CAAC,GAAG,KAAK;gBAAE,KAAK,GAAG,CAAC,CAAC;YACzB,IAAI,CAAC,GAAG,CAAC,GAAG,IAAI;gBAAE,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;QACjC,CAAC;IACH,CAAC;IACD,OAAO,KAAK,GAAG,IAAI,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC;AACnG,CAAC;AAED;;;;GAIG;AACH,SAAS,MAAM,CAAC,IAAwB,EAAE,OAAe;IACvD,OAAO,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,aAAa,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,IAAI,GAAG,CAAC;AACnE,CAAC;AAED,SAAS,aAAa,CAAC,OAAe;IACpC;;;;;;;;MAQE;IACF,MAAM,IAAI,GAAG,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC;IAC/B,OAAO,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC;AACvE,CAAC","sourcesContent":["/**\n * Parsing for the time-range options.\n *\n * Researchers write offsets in whatever form is natural for the recording in front\n * of them: seconds for a short ECG strip, `30m` into a sleep study, `01:23:45` when\n * reading off a clock. All three are accepted; anything ambiguous is rejected with\n * a message that shows the forms that work.\n */\n\nimport { fixed, formatDuration } from '../format/number.js';\nimport { assertOptions, describeValue } from './options.js';\n\nexport class TimeRangeError extends Error {\n  constructor(message: string) {\n    super(message);\n    this.name = 'TimeRangeError';\n  }\n}\n\n/**\n * Every spelling of a unit this accepts, and what one of it is worth in seconds.\n *\n * Exported so the spellings can be enumerated by a test rather than listed a second time:\n * twelve of these sixteen are named nowhere in the tool or its documentation, and none of\n * their values was checked by anything. `hrs: 360` would have converted `--start 2hrs` from\n * twelve minutes in and said nothing, which is the one kind of mistake a window may not make.\n */\nexport const UNIT_SECONDS: Record<string, number> = {\n  h: 3600,\n  hr: 3600,\n  hrs: 3600,\n  hour: 3600,\n  hours: 3600,\n  m: 60,\n  min: 60,\n  mins: 60,\n  minute: 60,\n  minutes: 60,\n  s: 1,\n  sec: 1,\n  secs: 1,\n  second: 1,\n  seconds: 1,\n  ms: 0.001,\n};\n\nconst CLOCK = /^(?:(\\d+):)?(\\d{1,2}):(\\d{1,2}(?:\\.\\d+)?)$/u;\nconst UNIT_TOKEN = /(\\d+(?:\\.\\d+)?)\\s*([a-z]+)/giu;\n\n/**\n * Parse a duration or offset into seconds.\n *\n * Accepted: `90`, `90s`, `5m`, `1h30m`, `1h 30m 15s`, `00:30:00`, `30:00`, `250ms`.\n *\n * `allowNegative` is for the two options that name a position rather than a length.\n *\n * A recording is timed from its first record's timekeeping annotation, and nothing obliges that\n * to sit at or after zero: a file whose records run from -100 s to -97 s is one this tool reads,\n * times from -100, and describes with\n *\n *     Timed from -100.000s  (first sample; --start and --end use this clock)\n *\n * That line says the number can be typed straight back in, and it could not be. Every offset\n * such a recording has came back as \"not a time I understand\", so its whole clock was\n * unreachable and no window of it could be converted at all — the one file shape where a\n * window is refused for naming a moment the recording actually contains.\n *\n * A length below zero is still a different thing, and `--duration` still refuses one.\n */\nexport function parseTimeSpec(input: string, optionName?: string, allowNegative = false): number {\n  /*\n    A name for the value, when the caller has not given one.\n\n    Every refusal below opens with it, and the CLI passes the flag — `--start \"1,5\" is not a\n    time I understand`. This is also the function the api page sends other people's users to,\n    and a caller who writes the one-argument call its neighbours take got\n\n        TimeRangeError: undefined is empty. Try a value like 30s, 5m, or 00:30:00.\n\n    on every one of the eleven messages in here. The paragraph below hardened `input` for\n    exactly that caller; the argument beside it could still put the word `undefined` at the\n    front of the sentence.\n\n    Coerced rather than refused, unlike `input`: this one names the value in a message rather\n    than being the value, so there is a right answer to fall back on and nothing is lost by\n    using it. An empty or non-string name takes the same route, since `\"\" is empty.` is the\n    same sentence with a different hole in it.\n  */\n  const named =\n    typeof optionName === 'string' && optionName.trim() !== '' ? optionName : 'The value';\n  /*\n    Text, because this is the function the API page points other people's users at: \"Use\n    `parseTimeSpec` if you want to accept those forms from your own users.\"\n\n    A value arriving from a JSON config or a form field is a number as often as a string, and\n    `{ \"start\": 30 }` came back as `TypeError: input.trim is not a function` — a variable name\n    from inside this file, thrown past a caller who had done exactly what the page told them\n    to. Every other refusal here is a TimeRangeError quoting the value and the option. The\n    number is not coerced, for the reason `channels: 'ECG'` is not iterated: a shape that\n    happens to work for 30 also swallows NaN, and the CLI already writes `String(raw)`.\n  */\n  if (typeof input !== 'string') {\n    throw new TimeRangeError(\n      `${named} must be given as text, not ${describeValue(input)}. ` +\n        `Seconds are written \"30\", and the other forms are \"30s\", \"5m\", \"1h30m\", \"00:30:00\".`,\n    );\n  }\n  const trimmed = input.trim().toLowerCase();\n  if (trimmed === '') {\n    throw new TimeRangeError(`${named} is empty. Try a value like 30s, 5m, or 00:30:00.`);\n  }\n\n  /*\n    The sign is read off the front and applied to the whole value, so `-1h30m` is an hour and a\n    half before the origin rather than an hour before it and half an hour after.\n\n    Nothing may sit between the sign and the number, and a leading `+` is still refused: every\n    other number this tool takes is refused when written in a form nobody types, and `+5` is\n    exactly what `--decimals` and `--jobs` reject one flag over.\n  */\n  const negative = trimmed.startsWith('-');\n  const text = negative ? trimmed.slice(1) : trimmed;\n\n  /*\n    A leading plus, named as such.\n\n    Refusing it is deliberate — see the paragraph above — but it came back as \"is not a time I\n    understand. Try 30s, 5m, 1h30m, 00:30:00, or a plain number of seconds\", which is the\n    message for input that could not be read at all. Every part of `+5s` was read: the number\n    parsed, the unit was looked up and found. The one thing wrong with it is a character the\n    reader put there on purpose, and they were sent to re-check their unit spellings with `5m`\n    sitting in the list of suggestions, differing from what they typed by the sign they cannot\n    see is the problem. 0.7.102 made this argument about a space; this is the same sentence\n    about the other character that reaches it. `--decimals` and `--jobs` both quote the value\n    back and say what is wrong with it.\n\n    Every plus, not the first one, and the leftover space with it — the same rule the spaced\n    refusal below applies to spaces, for the same reason: a hint whose command this function\n    refuses is worse than no hint. `+1h+30m` was answered with `1h+30m`, which comes straight\n    back as \"is not a time I understand\", and `+ 5s` with a value that opened on a space.\n  */\n  if (text.startsWith('+')) {\n    throw new TimeRangeError(\n      `${named} \"${input}\" begins with a plus. Write the number on its own: ` +\n        `${input.trim().replaceAll('+', '').trim()}`,\n    );\n  }\n  const signed = (seconds: number): number =>\n    assertFinite(negative ? -seconds : seconds, named, input, allowNegative);\n\n  // Bare number means seconds.\n  if (/^\\d+(?:\\.\\d+)?$/u.test(text)) return signed(Number(text));\n\n  // Clock form: hh:mm:ss or mm:ss.\n  const clock = CLOCK.exec(text);\n  if (clock) {\n    const hours = clock[1] === undefined ? 0 : Number(clock[1]);\n    const minutes = Number(clock[2]);\n    const seconds = Number(clock[3]);\n    if (minutes >= 60 || seconds >= 60) {\n      throw new TimeRangeError(\n        `${named} \"${input}\" has a minutes or seconds field of 60 or more.`,\n      );\n    }\n    return signed(hours * 3600 + minutes * 60 + seconds);\n  }\n\n  // Unit form: 1h30m, 1h30m 15s, 250ms. A number sits directly against its unit; see below.\n  UNIT_TOKEN.lastIndex = 0;\n  let total = 0;\n  let matched = 0;\n  let consumed = 0;\n  /** Whether any token had a space between its number and its unit. See the refusal below. */\n  let spaced = false;\n  let match: RegExpExecArray | null;\n  // Which units have already been seen, so a repeat can be rejected rather than added.\n  const seen = new Set<number>();\n  while ((match = UNIT_TOKEN.exec(text)) !== null) {\n    const amount = Number(match[1]);\n    const unit = match[2] ?? '';\n    const scale = UNIT_SECONDS[unit];\n    if (scale === undefined) {\n      throw new TimeRangeError(\n        `${named} \"${input}\" uses an unknown unit \"${unit}\". Use h, m, s or ms, ` +\n          `or their long forms: hours, minutes, seconds.`,\n      );\n    }\n\n    // \"1h1h\" is a typo, not a request for two hours. Summing repeated units silently\n    // turned a slip into a plausible window that was quietly the wrong length. Units are\n    // keyed by their scale so the aliases collapse together: \"1h30min20m\" is caught too.\n    if (seen.has(scale)) {\n      throw new TimeRangeError(\n        `${named} \"${input}\" gives the same unit twice. ` +\n          `Combine each unit once, as in 1h30m.`,\n      );\n    }\n    seen.add(scale);\n\n    total += amount * scale;\n    matched++;\n    // Counted with the whitespace taken out, so the comparison below is against the same\n    // string on both sides and a space inside a token reaches the refusal written for it\n    // rather than the one about input that could not be read at all.\n    consumed += match[0].replace(/\\s+/gu, '').length;\n    if (/\\s/u.test(match[0])) spaced = true;\n  }\n\n  // Reject partially-understood input like \"5x\" or \"1h banana\".\n  if (matched === 0 || consumed !== text.replace(/\\s+/gu, '').length) {\n    throw new TimeRangeError(\n      `${named} \"${input}\" is not a time I understand. ` +\n        `Try 30s, 5m, 1h30m, 00:30:00, or a plain number of seconds.`,\n    );\n  }\n\n  /*\n    A space between a number and its unit, named as such.\n\n    `5 min` is refused on purpose — cli-reference sets the rule out, and it is what keeps\n    `1 2h` from being read as anything — but it came back as \"is not a time I understand. Try\n    30s, 5m, 1h30m, 00:30:00, or a plain number of seconds\", which is the message for input\n    that could not be read at all. Every part of `5 min` was read: the number parsed, the unit\n    was looked up in the table and found. The one thing wrong with it is a space, and the\n    reader was sent to re-check their unit spellings instead — with `5m` sitting in the list of\n    suggestions, differing from what they typed by a character they cannot see is the problem.\n\n    Only reachable now that the count above ignores whitespace; before, the length check caught\n    these first and there was nothing left to tell them apart by.\n  */\n  if (spaced) {\n    throw new TimeRangeError(\n      `${named} \"${input}\" puts a space between a number and its unit. ` +\n        `Write them together: ${input.trim().replace(/(\\d)\\s+([a-z])/giu, '$1$2')}`,\n    );\n  }\n\n  return signed(total);\n}\n\nfunction assertFinite(\n  value: number,\n  optionName: string,\n  input: string,\n  allowNegative: boolean,\n): number {\n  /*\n    Overflow answers for itself, rather than borrowing the sentence about signs.\n\n    One branch covered two rejections. `--start` with four hundred nines is a positive value\n    that `Number` returns as `Infinity`, and the answer was `is not a valid non-negative\n    time` — which names the one thing about it that is not the problem. Worse on the two\n    options this is called with `allowNegative` for: `--start` and `--end` take a sign on\n    purpose, since a recording timed from before zero has no other way to be addressed, so\n    \"non-negative\" is not their rule at all and overflow is the only way they reach here.\n  */\n  if (!Number.isFinite(value)) {\n    throw new TimeRangeError(\n      `${optionName} \"${input}\" is further from zero than a number of seconds can hold.`,\n    );\n  }\n  if (!allowNegative && value < 0) {\n    throw new TimeRangeError(`${optionName} \"${input}\" is not a valid non-negative time.`);\n  }\n  return value;\n}\n\nexport interface ResolvedRange {\n  /** Inclusive start, in seconds from the beginning of the recording. */\n  startSeconds: number;\n  /** Exclusive end, in seconds. */\n  endSeconds: number;\n  /** First data record touching the window. */\n  startRecord: number;\n  /** One past the last data record touching the window. */\n  endRecord: number;\n  /** True when the window covers the whole recording. */\n  isWholeRecording: boolean;\n  /** Earliest record start, including EDF+D timing gaps. */\n  recordingStartSeconds: number;\n  /** End of the latest record, including EDF+D timing gaps. */\n  recordingEndSeconds: number;\n}\n\n/**\n * Slack for comparisons at sample/window boundaries.\n *\n * A nanosecond is far below any real sampling interval — 20 kHz is 50 microseconds — so it\n * absorbs the arithmetic error in `recordStart + sample / rate` without reaching a\n * neighbouring sample.\n */\nexport const BOUNDARY_TOLERANCE = 1e-9;\n\n/**\n * The slack to use for a channel sampled this often.\n *\n * Never past half a sample interval, because slack that reaches the next sample stops being\n * slack — half is the most that cannot, since the nearest sample below a bound sits a whole\n * interval away from it. `Math.min` takes exactly half at rates above 500 MHz, which is where\n * this said \"never as much as half\" of a formula that returns it. A fixed nanosecond was\n * applied whatever the rate, and the format does not\n * oblige the interval to be larger than it: EDF's record duration is an 8-character field\n * that accepts `1e-9`. A recording of two 1 ns records holding ten samples each wrote ten of\n * its twenty rows — the window ends at 2e-9, the comparison asked for `time < 2e-9 - 1e-9`,\n * and the entire second record failed it. Exit 0, no warning, half the samples gone.\n */\nexport function toleranceFor(rate: number): number {\n  if (!(rate > 0) || !Number.isFinite(rate)) return BOUNDARY_TOLERANCE;\n  return Math.min(BOUNDARY_TOLERANCE, 1 / rate / 2);\n}\n\n/** Match the exact half-open boundary rules used while writing signal rows. */\nexport function sampleTimeIsInRange(\n  time: number,\n  startSeconds: number,\n  endSeconds: number,\n  tolerance: number = BOUNDARY_TOLERANCE,\n): boolean {\n  return time >= startSeconds - tolerance && time < endSeconds - tolerance;\n}\n\n/** Count samples from one record that fall inside a half-open requested window. */\nexport function countSamplesInRange(options: {\n  recordStart: number;\n  rate: number;\n  samplesPerRecord: number;\n  startSeconds: number;\n  endSeconds: number;\n}): number {\n  /*\n    A record with no place on the clock contributes no rows, which is what the conversion\n    already does with one: its sample times are non-finite, so every one of them fails the\n    range test and none is written. The arithmetic below reached `Infinity - Infinity` and\n    answered `NaN`, which `--info` then printed — \"Would write NaN rows, roughly NaN B.\" on a\n    recording that goes on to write eight. Reachable from a header: a record duration of 1e308\n    is five characters in an eight-character field, and the third record of one is past what a\n    double can hold.\n  */\n  if (!Number.isFinite(options.recordStart)) return 0;\n  const slack = toleranceFor(options.rate);\n  const lower = Math.ceil(\n    (options.startSeconds - slack - options.recordStart) * options.rate,\n  );\n  const upper = Math.ceil(\n    (options.endSeconds - slack - options.recordStart) * options.rate,\n  );\n  const first = Math.max(0, Math.min(options.samplesPerRecord, lower));\n  const last = Math.max(0, Math.min(options.samplesPerRecord, upper));\n  return Math.max(0, last - first);\n}\n\n/**\n * Whether two instants differ only by the arithmetic that produced them.\n *\n * A relative epsilon, because the gap between doubles grows with magnitude — the same shape\n * the long layout uses to decide two sample times are one instant. Well below any real sample\n * interval, and well above the rounding that two routes to one quantity produce: a recording's\n * length is `recordCount * recordDuration`, which for 6003 records of 0.1s is not the 600.3 it\n * prints as.\n */\nfunction sameInstant(a: number, b: number): boolean {\n  /*\n    A relative tolerance has nothing to be relative to at infinity.\n\n    `Math.abs(a - Infinity)` is Infinity, and so is the allowance beside it, so\n    `Infinity <= Infinity` made every instant the same instant as an infinite one. A\n    recording whose end overflows a double — three records of 1e308 seconds, which an\n    8-character record-duration field can state and a timekeeping TAL can place — was\n    therefore refused the window nobody asked for:\n\n        $ edf2csv rec.edf --info\n        error: --start 0s is at or past the end of this unknown recording.\n\n    Exit 2 for a file, naming a flag the command did not carry, quoting a length that is\n    `formatDuration`'s word for a number it will not print. `--info` is the one command that\n    would have explained the file and it is the one that could not run.\n\n    Finite or equal, which is the only reading that means anything: two infinities are the\n    same non-instant, and a real offset is not one of them.\n  */\n  if (!Number.isFinite(a) || !Number.isFinite(b)) return a === b;\n  return Math.abs(a - b) <= Math.max(Math.abs(a), Math.abs(b)) * 1e-12;\n}\n\n/**\n * Turn a requested window into both an exact time span and the record range that\n * contains it. Records are the unit the file can be read in; the exact span is what\n * decides which samples inside those records are actually written.\n */\nexport function resolveRange(options: {\n  start?: number | undefined;\n  /** The `--start` value exactly as typed, quoted back in the past-the-end error. */\n  startText?: string | undefined;\n  duration?: number | undefined;\n  /** The `--duration` value exactly as typed, quoted back in the window error. */\n  durationText?: string | undefined;\n  end?: number | undefined;\n  /** The `--end` value exactly as typed, quoted back in the window error. */\n  endText?: string | undefined;\n  recordDuration: number;\n  recordCount: number;\n  /**\n   * True start time of each data record, for discontinuous files. When absent,\n   * records are assumed to sit end to end.\n   */\n  recordStarts?: Float64Array | null | undefined;\n}): ResolvedRange {\n  /*\n    The same check `buildPlan` makes, made here too, because this is exported on its own.\n\n    `assertOptions` sits at the top of `buildPlan`, so `convert` was covered and the function\n    underneath it was not — and this one has its own signature block on the API page.\n    `resolveRange({ start: NaN, ... })` resolved: no error, no warning, and a range read back\n    as `startSeconds: null, startRecord: null`, which is the \"takes the whole recording\n    without saying so\" that comment names. `{ end: '2' }` was coerced by the arithmetic and\n    accepted; `{ start: '30' }` reached the past-the-end error and printed it with the value\n    missing — `--start s is at or past the end of this 3s recording`.\n  */\n  assertOptions({ start: options.start, duration: options.duration, end: options.end });\n\n  // For a continuous file the recording spans recordCount * recordDuration. A\n  // discontinuous one does not: a 10-second recording with a 95-second gap in the\n  // middle still ends at 105 seconds. Deriving the span from the records' real\n  // positions is what stops a requested window from being clipped back to the\n  // amount of *data* in the file and silently discarding everything past it.\n  const { earliest, latest } = span(options.recordStarts, options.recordCount, options.recordDuration);\n\n  if (options.duration !== undefined && options.end !== undefined) {\n    throw new TimeRangeError('Use either --duration or --end, not both.');\n  }\n\n  const startSeconds = options.start ?? earliest;\n\n  let endSeconds: number;\n  if (options.duration !== undefined) endSeconds = startSeconds + options.duration;\n  else if (options.end !== undefined) endSeconds = options.end;\n  else endSeconds = latest;\n\n  /*\n    At the end, allowing for the arithmetic that produced the end.\n\n    `latest` is `recordCount * recordDuration`, and with a fractional duration that is not the\n    number it prints as: 6003 records of 0.1s is 600.3000000000001, not 600.3. So `--start\n    600.3` on a recording `--info` calls \"10m 0.3s\" was accepted by a hair, converted nothing,\n    and exited 0 with a signals.csv holding its header — which is the empty conversion this\n    error exists to prevent, and which the same command on a whole-second recording is refused\n    for.\n\n    A relative epsilon, the same shape the long layout uses to decide two sample times are one\n    instant: well below any real interval, and well above the rounding that two different\n    routes to the same quantity produce.\n  */\n  if (startSeconds >= latest || sameInstant(startSeconds, latest)) {\n    /*\n      Quote what was typed. Reporting the parsed seconds meant `--start 4h` came back as\n      \"--start 14400s is at or past the end\", which reads as a value the user never gave.\n\n      In quotation marks, which the parse errors above have always used and this did not.\n      Without them the value ran into the sentence: `--start \"  5s  \"` printed as\n      `--start   5s   is at or past the end`, where the value appears to be `5s   is` and\n      the surrounding spaces — the actual reason a shell-built argument went wrong — are\n      invisible.\n    */\n    throw new TimeRangeError(\n      /*\n        The recording's length in the same words --info uses for it.\n\n        `formatSeconds` renders a bare number of seconds, so this message and the Duration\n        line disagreed about one file in one session: --info said \"6m 40s\" and the error said\n        \"400s\". On an overnight recording it read \"7950s recording\", leaving the reader to\n        divide by 3600 to find out whether their --start was reasonable — which is the one\n        question this message exists to answer. cli-reference.md has always documented it\n        humanised (\"2h 12m 30s\"), a form no input could produce.\n\n        The typed value keeps `quoted`, since that is the user's own text and should come\n        back exactly as they wrote it.\n      */\n      /*\n        `latest` is where the recording ends on its own clock, which is its length only when\n        it starts at zero. A file timed from its first record's timekeeping TAL need not: one\n        whose records run 1000s to 1003s is three seconds long, and this called it \"this\n        16m 43s recording\" — while --info two lines away said \"Duration 3s\". Where the\n        recording sits is the useful thing to say in that case, and it is the number --start\n        has to be given.\n      */\n      earliest === 0\n        ? `--start ${quoted(options.startText, startSeconds)} is at or past the end of this ` +\n          `${formatDuration(latest)} recording.`\n        : `--start ${quoted(options.startText, startSeconds)} is at or past the end of this ` +\n          `${formatDuration(latest - earliest)} recording, which runs from ` +\n          `${formatSeconds(earliest)} to ${formatSeconds(latest)}.`,\n    );\n  }\n  if (endSeconds <= startSeconds) {\n    // Quote whatever the caller actually gave, for the same reason as the error above.\n    const endShown =\n      options.end !== undefined && options.endText !== undefined\n        ? quoted(options.endText, endSeconds)\n        : formatSeconds(endSeconds);\n    const startShown = quoted(options.startText, startSeconds);\n    /*\n      Name the option that caused it, when only one of them could have.\n\n      `end = start + duration`, and a negative duration is refused before this, so a window\n      that ends where it starts and was built from `--duration` was built from a duration of\n      zero — the one value in the command that is wrong. The message named neither it nor the\n      flag:\n\n          edf2csv rec.edf --start 5s --duration 0\n          error: The requested window ends at 5s, which is not after its start at \"5s\".\n\n      Two mentions of the value that is fine, none of the one that is not, in a file whose\n      other refusals quote what was typed on the reasoning that \"`--start 4h` came back as\n      '--start 14400s is at or past the end', which reads as a value the user never gave\".\n      `--duration` had no typed text to quote because nothing passed it; it does now, like\n      its two neighbours.\n    */\n    throw new TimeRangeError(\n      options.end === undefined && options.duration !== undefined\n        ? `--duration ${quoted(options.durationText, options.duration)} is not a length of ` +\n          `time, so the window ends where it starts, at ${startShown}. Give a length above ` +\n          `zero, or leave --duration out to convert to the end.`\n        : `The requested window ends at ${endShown}, which is not after its start at ${startShown}.`,\n    );\n  }\n\n  const clampedEnd = Math.min(endSeconds, latest);\n  /*\n    Both ends of the window, clamped the same way.\n\n    The end has been clamped to the recording since these fields existed — `--end 999h` on a\n    two-hour file is documented as converting to the end, silently — and the start was left as\n    typed. So a window asked for before the recording begins was recorded as one:\n\n        edf2csv two-second.edf --out csv --start=-500\n        metadata.json  \"start_seconds\": -500, \"end_seconds\": 2\n\n    over a signals.csv whose first row is 0.000, with `records_converted` correctly `[0, 2]`.\n    output-files calls the pair \"the resolved time window\", which the end is and the start was\n    not, and a script taking `end_seconds - start_seconds` for the span converted got 502 for\n    two seconds of data.\n\n    `earliest` and not zero, because a recording is timed from its first record and need not\n    begin there: negative-origin.edf runs from -100s, and -100 is where a start before the\n    recording belongs. Nothing downstream moves — every sample sits at or after `earliest`, so\n    the row filter and the record selection answer the same for either value, and the byte\n    estimate takes a width from the smaller magnitude, which is the one the column actually\n    holds.\n\n    Not when the whole window sits before the recording, where there is no overlap to clamp\n    to and raising the start past the end would describe the request backwards: `--start 0\n    --duration 0.001` on a file beginning at 30s is an empty window, and EMPTY_WINDOW quotes\n    the bounds — \"(30.000s to 0.001s)\" says nothing a reader can act on, where the bounds as\n    asked for are what its hint is already explaining.\n  */\n  const clampedStart = clampedEnd <= earliest ? startSeconds : Math.max(startSeconds, earliest);\n  const { startRecord, endRecord } = selectRecords(options, clampedStart, clampedEnd);\n\n  return {\n    startSeconds: clampedStart,\n    endSeconds: clampedEnd,\n    startRecord,\n    endRecord,\n    /*\n      The same rounding, one field over.\n\n      `latest` is `recordCount * recordDuration`, and 6003 records of 0.1s is\n      600.3000000000001. So `--end 600.3` on a recording of exactly that length wrote every\n      sample it has — byte-identical to a bare conversion — and metadata.json recorded\n      `whole_recording: false` for it, while the bare run recorded true. One conversion, two\n      answers, on the field a pipeline reads to decide whether it has the lot.\n    */\n    isWholeRecording:\n      (startSeconds <= earliest || sameInstant(startSeconds, earliest)) &&\n      (clampedEnd >= latest || sameInstant(clampedEnd, latest)),\n    recordingStartSeconds: earliest,\n    recordingEndSeconds: latest,\n  };\n}\n\nfunction span(\n  recordStarts: Float64Array | null | undefined,\n  recordCount: number,\n  recordDuration: number,\n): { earliest: number; latest: number } {\n  if (!recordStarts || recordStarts.length === 0) {\n    return { earliest: 0, latest: recordCount * recordDuration };\n  }\n  let earliest = Infinity;\n  let latest = -Infinity;\n  for (const start of recordStarts) {\n    if (start < earliest) earliest = start;\n    if (start + recordDuration > latest) latest = start + recordDuration;\n  }\n  if (!Number.isFinite(earliest) || !Number.isFinite(latest)) {\n    return { earliest: 0, latest: recordCount * recordDuration };\n  }\n  return { earliest, latest };\n}\n\n/** Every record whose own time span overlaps the requested window. */\nfunction selectRecords(\n  options: { recordDuration: number; recordCount: number; recordStarts?: Float64Array | null | undefined },\n  startSeconds: number,\n  endSeconds: number,\n): { startRecord: number; endRecord: number } {\n  const starts = options.recordStarts;\n  if (!starts || starts.length === 0) {\n    /*\n      An empty range is `[0, 0]` here as it is below, rather than whatever the arithmetic\n      produced. Each bound was clamped on one side only — the start up to zero, the end down\n      to the record count — so a window lying entirely before the recording came back as\n      `records_converted: [0, -1]` in metadata.json, over a `endRecord` documented as \"one\n      past the last data record touching the window\". A script differencing the pair read\n      minus one record converted, and `ConversionProgress.recordsTotal` reported the same.\n\n      The discontinuous branch below has always answered `[0, 0]` for exactly this request,\n      so the two halves of one function disagreed about how to say \"none\": the same window on\n      `annotations.edf` and on `discontinuous.edf` gave `[0, -1]` and `[0, 0]`.\n    */\n    const first = Math.max(0, Math.floor(startSeconds / options.recordDuration));\n    const last = Math.min(options.recordCount, Math.ceil(endSeconds / options.recordDuration));\n    return first < last ? { startRecord: first, endRecord: last } : { startRecord: 0, endRecord: 0 };\n  }\n\n  let first = options.recordCount;\n  let last = 0;\n  for (let i = 0; i < starts.length; i++) {\n    const begin = starts[i] ?? 0;\n    if (begin + options.recordDuration > startSeconds && begin < endSeconds) {\n      if (i < first) first = i;\n      if (i + 1 > last) last = i + 1;\n    }\n  }\n  return first < last ? { startRecord: first, endRecord: last } : { startRecord: 0, endRecord: 0 };\n}\n\n/**\n * The value as the caller typed it, in quotation marks, or the parsed seconds if they gave\n * none. The marks show where the value begins and ends, which matters most for the values\n * that went wrong because of what surrounds them.\n */\nfunction quoted(text: string | undefined, seconds: number): string {\n  return text === undefined ? formatSeconds(seconds) : `\"${text}\"`;\n}\n\nfunction formatSeconds(seconds: number): string {\n  /*\n    The same 1e21 cliff, in the bounds this message hands back.\n\n    `--start \"9e21\"` was answered with \"is at or past the end of this 3e+21s recording, which\n    runs from 1e+21s to 4e+21s\" — a sentence whose whole purpose is to say what window there\n    is to ask for, ending in two tokens the parser refuses: `--start 1e+21s` is \"uses an\n    unknown unit \\\"e\\\"\". `Number(...)` was here to drop the trailing zeros, and it also\n    re-introduced the exponent form that `toFixed` had produced.\n  */\n  const text = fixed(seconds, 3);\n  return `${text.includes('.') ? text.replace(/\\.?0+$/u, '') : text}s`;\n}\n"]}
+{"version":3,"file":"time-range.js","sourceRoot":"","sources":["../../src/convert/time-range.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAC5D,OAAO,EAAE,aAAa,EAAE,iBAAiB,EAAE,aAAa,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAE5F,MAAM,OAAO,cAAe,SAAQ,KAAK;IACvC,YAAY,OAAe;QACzB,KAAK,CAAC,OAAO,CAAC,CAAC;QACf,IAAI,CAAC,IAAI,GAAG,gBAAgB,CAAC;IAC/B,CAAC;CACF;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,YAAY,GAA2B;IAClD,CAAC,EAAE,IAAI;IACP,EAAE,EAAE,IAAI;IACR,GAAG,EAAE,IAAI;IACT,IAAI,EAAE,IAAI;IACV,KAAK,EAAE,IAAI;IACX,CAAC,EAAE,EAAE;IACL,GAAG,EAAE,EAAE;IACP,IAAI,EAAE,EAAE;IACR,MAAM,EAAE,EAAE;IACV,OAAO,EAAE,EAAE;IACX,CAAC,EAAE,CAAC;IACJ,GAAG,EAAE,CAAC;IACN,IAAI,EAAE,CAAC;IACP,MAAM,EAAE,CAAC;IACT,OAAO,EAAE,CAAC;IACV,EAAE,EAAE,KAAK;CACV,CAAC;AAEF,MAAM,KAAK,GAAG,6CAA6C,CAAC;AAC5D,MAAM,UAAU,GAAG,+BAA+B,CAAC;AAEnD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,UAAmB,EAAE,aAAa,GAAG,KAAK;IACrF;;;;;;;;;;;;;;;;;MAiBE;IACF,MAAM,KAAK,GACT,OAAO,UAAU,KAAK,QAAQ,IAAI,UAAU,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,WAAW,CAAC;IACxF;;;;;;;;;;MAUE;IACF,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,+BAA+B,aAAa,CAAC,KAAK,CAAC,IAAI;YAC7D,qFAAqF,CACxF,CAAC;IACJ,CAAC;IACD,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC3C,IAAI,OAAO,KAAK,EAAE,EAAE,CAAC;QACnB,MAAM,IAAI,cAAc,CAAC,GAAG,KAAK,mDAAmD,CAAC,CAAC;IACxF,CAAC;IAED;;;;;;;MAOE;IACF,MAAM,QAAQ,GAAG,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC;IACzC,MAAM,IAAI,GAAG,QAAQ,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;IAEnD;;;;;;;;;;;;;;;;;MAiBE;IACF,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QACzB,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,qDAAqD;YACrE,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,EAAE,CAC/C,CAAC;IACJ,CAAC;IACD,MAAM,MAAM,GAAG,CAAC,OAAe,EAAU,EAAE,CACzC,YAAY,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,aAAa,CAAC,CAAC;IAE3E,6BAA6B;IAC7B,IAAI,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC;IAE/D,iCAAiC;IACjC,MAAM,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,KAAK,EAAE,CAAC;QACV,MAAM,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAC5D,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACjC,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACjC,IAAI,OAAO,IAAI,EAAE,IAAI,OAAO,IAAI,EAAE,EAAE,CAAC;YACnC,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,iDAAiD,CACpE,CAAC;QACJ,CAAC;QACD,OAAO,MAAM,CAAC,KAAK,GAAG,IAAI,GAAG,OAAO,GAAG,EAAE,GAAG,OAAO,CAAC,CAAC;IACvD,CAAC;IAED,0FAA0F;IAC1F,UAAU,CAAC,SAAS,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,OAAO,GAAG,CAAC,CAAC;IAChB,IAAI,QAAQ,GAAG,CAAC,CAAC;IACjB,4FAA4F;IAC5F,IAAI,MAAM,GAAG,KAAK,CAAC;IACnB,IAAI,KAA6B,CAAC;IAClC,qFAAqF;IACrF,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,OAAO,CAAC,KAAK,GAAG,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAChD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAC5B,MAAM,KAAK,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC;QACjC,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,2BAA2B,IAAI,wBAAwB;gBACvE,+CAA+C,CAClD,CAAC;QACJ,CAAC;QAED,iFAAiF;QACjF,qFAAqF;QACrF,qFAAqF;QACrF,IAAI,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;YACpB,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,+BAA+B;gBAC/C,sCAAsC,CACzC,CAAC;QACJ,CAAC;QACD,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QAEhB,KAAK,IAAI,MAAM,GAAG,KAAK,CAAC;QACxB,OAAO,EAAE,CAAC;QACV,qFAAqF;QACrF,qFAAqF;QACrF,iEAAiE;QACjE,QAAQ,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC;QACjD,IAAI,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YAAE,MAAM,GAAG,IAAI,CAAC;IAC1C,CAAC;IAED,8DAA8D;IAC9D,IAAI,OAAO,KAAK,CAAC,IAAI,QAAQ,KAAK,IAAI,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC;QACnE,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,gCAAgC;YAChD,6DAA6D,CAChE,CAAC;IACJ,CAAC;IAED;;;;;;;;;;;;;MAaE;IACF,IAAI,MAAM,EAAE,CAAC;QACX,MAAM,IAAI,cAAc,CACtB,GAAG,KAAK,KAAK,KAAK,gDAAgD;YAChE,wBAAwB,KAAK,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,mBAAmB,EAAE,MAAM,CAAC,EAAE,CAC9E,CAAC;IACJ,CAAC;IAED,OAAO,MAAM,CAAC,KAAK,CAAC,CAAC;AACvB,CAAC;AAED,SAAS,YAAY,CACnB,KAAa,EACb,UAAkB,EAClB,KAAa,EACb,aAAsB;IAEtB;;;;;;;;;MASE;IACF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QAC5B,MAAM,IAAI,cAAc,CACtB,GAAG,UAAU,KAAK,KAAK,2DAA2D,CACnF,CAAC;IACJ,CAAC;IACD,IAAI,CAAC,aAAa,IAAI,KAAK,GAAG,CAAC,EAAE,CAAC;QAChC,MAAM,IAAI,cAAc,CAAC,GAAG,UAAU,KAAK,KAAK,qCAAqC,CAAC,CAAC;IACzF,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAmBD;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAG,IAAI,CAAC;AAEvC;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,YAAY,CAAC,IAAY;IACvC,IAAI,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC;QAAE,OAAO,kBAAkB,CAAC;IACrE,OAAO,IAAI,CAAC,GAAG,CAAC,kBAAkB,EAAE,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC;AACpD,CAAC;AAED,+EAA+E;AAC/E,MAAM,UAAU,mBAAmB,CACjC,IAAY,EACZ,YAAoB,EACpB,UAAkB,EAClB,YAAoB,kBAAkB;IAEtC,OAAO,IAAI,IAAI,YAAY,GAAG,SAAS,IAAI,IAAI,GAAG,UAAU,GAAG,SAAS,CAAC;AAC3E,CAAC;AAED,mFAAmF;AACnF,MAAM,UAAU,mBAAmB,CAAC,OAMnC;IACC;;;;;;;;MAQE;IACF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,CAAC;QAAE,OAAO,CAAC,CAAC;IACpD,MAAM,KAAK,GAAG,YAAY,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IACzC,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,CACrB,CAAC,OAAO,CAAC,YAAY,GAAG,KAAK,GAAG,OAAO,CAAC,WAAW,CAAC,GAAG,OAAO,CAAC,IAAI,CACpE,CAAC;IACF,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,CACrB,CAAC,OAAO,CAAC,UAAU,GAAG,KAAK,GAAG,OAAO,CAAC,WAAW,CAAC,GAAG,OAAO,CAAC,IAAI,CAClE,CAAC;IACF,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,gBAAgB,EAAE,KAAK,CAAC,CAAC,CAAC;IACrE,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,gBAAgB,EAAE,KAAK,CAAC,CAAC,CAAC;IACpE,OAAO,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,GAAG,KAAK,CAAC,CAAC;AACnC,CAAC;AAED;;;;;;;;GAQG;AACH,SAAS,WAAW,CAAC,CAAS,EAAE,CAAS;IACvC;;;;;;;;;;;;;;;;;;MAkBE;IACF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC;QAAE,OAAO,CAAC,KAAK,CAAC,CAAC;IAC/D,OAAO,IAAI,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,KAAK,CAAC;AACvE,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,YAAY,CAAC,OAiB5B;IACC;;;;;;;;;;MAUE;IACF;;;;;;;MAOE;IACF,IAAI,OAAO,KAAK,IAAI,IAAI,OAAO,OAAO,KAAK,QAAQ,EAAE,CAAC;QACpD,MAAM,IAAI,WAAW,CACnB,qFAAqF;YACnF,OAAO,aAAa,CAAC,OAAO,CAAC,GAAG,CACnC,CAAC;IACJ,CAAC;IACD,aAAa,CAAC,EAAE,KAAK,EAAE,OAAO,CAAC,KAAK,EAAE,QAAQ,EAAE,OAAO,CAAC,QAAQ,EAAE,GAAG,EAAE,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IACtF;;;;;;;;;;;MAWE;IACF,iBAAiB,CAAC,OAAO,CAAC,CAAC;IAE3B,4EAA4E;IAC5E,gFAAgF;IAChF,6EAA6E;IAC7E,4EAA4E;IAC5E,2EAA2E;IAC3E,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC,OAAO,CAAC,YAAY,EAAE,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,cAAc,CAAC,CAAC;IAErG,IAAI,OAAO,CAAC,QAAQ,KAAK,SAAS,IAAI,OAAO,CAAC,GAAG,KAAK,SAAS,EAAE,CAAC;QAChE,MAAM,IAAI,cAAc,CAAC,2CAA2C,CAAC,CAAC;IACxE,CAAC;IAED,MAAM,YAAY,GAAG,OAAO,CAAC,KAAK,IAAI,QAAQ,CAAC;IAE/C,IAAI,UAAkB,CAAC;IACvB,IAAI,OAAO,CAAC,QAAQ,KAAK,SAAS;QAAE,UAAU,GAAG,YAAY,GAAG,OAAO,CAAC,QAAQ,CAAC;SAC5E,IAAI,OAAO,CAAC,GAAG,KAAK,SAAS;QAAE,UAAU,GAAG,OAAO,CAAC,GAAG,CAAC;;QACxD,UAAU,GAAG,MAAM,CAAC;IAEzB;;;;;;;;;;;;;MAaE;IACF,IAAI,YAAY,IAAI,MAAM,IAAI,WAAW,CAAC,YAAY,EAAE,MAAM,CAAC,EAAE,CAAC;QAChE;;;;;;;;;UASE;QACF,MAAM,IAAI,cAAc;QACtB;;;;;;;;;;;;UAYE;QACF;;;;;;;UAOE;QACF,QAAQ,KAAK,CAAC;YACZ,CAAC,CAAC,WAAW,MAAM,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,iCAAiC;gBACnF,GAAG,cAAc,CAAC,MAAM,CAAC,aAAa;YACxC,CAAC,CAAC,WAAW,MAAM,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,iCAAiC;gBACnF,GAAG,cAAc,CAAC,MAAM,GAAG,QAAQ,CAAC,8BAA8B;gBAClE,GAAG,aAAa,CAAC,QAAQ,CAAC,OAAO,aAAa,CAAC,MAAM,CAAC,GAAG,CAC9D,CAAC;IACJ,CAAC;IACD,IAAI,UAAU,IAAI,YAAY,EAAE,CAAC;QAC/B,mFAAmF;QACnF,MAAM,QAAQ,GACZ,OAAO,CAAC,GAAG,KAAK,SAAS,IAAI,OAAO,CAAC,OAAO,KAAK,SAAS;YACxD,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,OAAO,EAAE,UAAU,CAAC;YACrC,CAAC,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC;QAChC,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC;QAC3D;;;;;;;;;;;;;;;;UAgBE;QACF,MAAM,IAAI,cAAc,CACtB,OAAO,CAAC,GAAG,KAAK,SAAS,IAAI,OAAO,CAAC,QAAQ,KAAK,SAAS;YACzD,CAAC,CAAC,cAAc,MAAM,CAAC,OAAO,CAAC,YAAY,EAAE,OAAO,CAAC,QAAQ,CAAC,sBAAsB;gBAClF,gDAAgD,UAAU,wBAAwB;gBAClF,sDAAsD;YACxD,CAAC,CAAC,gCAAgC,QAAQ,qCAAqC,UAAU,GAAG,CAC/F,CAAC;IACJ,CAAC;IAED,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;IAChD;;;;;;;;;;;;;;;;;;;;;;;;;;;MA2BE;IACF,MAAM,YAAY,GAAG,UAAU,IAAI,QAAQ,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,YAAY,EAAE,QAAQ,CAAC,CAAC;IAC9F,MAAM,EAAE,WAAW,EAAE,SAAS,EAAE,GAAG,aAAa,CAAC,OAAO,EAAE,YAAY,EAAE,UAAU,CAAC,CAAC;IAEpF,OAAO;QACL,YAAY,EAAE,YAAY;QAC1B,UAAU,EAAE,UAAU;QACtB,WAAW;QACX,SAAS;QACT;;;;;;;;UAQE;QACF,gBAAgB,EACd,CAAC,YAAY,IAAI,QAAQ,IAAI,WAAW,CAAC,YAAY,EAAE,QAAQ,CAAC,CAAC;YACjE,CAAC,UAAU,IAAI,MAAM,IAAI,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;QAC3D,qBAAqB,EAAE,QAAQ;QAC/B,mBAAmB,EAAE,MAAM;KAC5B,CAAC;AACJ,CAAC;AAED,SAAS,IAAI,CACX,YAA6C,EAC7C,WAAmB,EACnB,cAAsB;IAEtB,IAAI,CAAC,YAAY,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC/C,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,WAAW,GAAG,cAAc,EAAE,CAAC;IAC/D,CAAC;IACD,IAAI,QAAQ,GAAG,QAAQ,CAAC;IACxB,IAAI,MAAM,GAAG,CAAC,QAAQ,CAAC;IACvB,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;QACjC,IAAI,KAAK,GAAG,QAAQ;YAAE,QAAQ,GAAG,KAAK,CAAC;QACvC,IAAI,KAAK,GAAG,cAAc,GAAG,MAAM;YAAE,MAAM,GAAG,KAAK,GAAG,cAAc,CAAC;IACvE,CAAC;IACD,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3D,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,WAAW,GAAG,cAAc,EAAE,CAAC;IAC/D,CAAC;IACD,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;AAC9B,CAAC;AAED,sEAAsE;AACtE,SAAS,aAAa,CACpB,OAAwG,EACxG,YAAoB,EACpB,UAAkB;IAElB,MAAM,MAAM,GAAG,OAAO,CAAC,YAAY,CAAC;IACpC,IAAI,CAAC,MAAM,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACnC;;;;;;;;;;;UAWE;QACF,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,YAAY,GAAG,OAAO,CAAC,cAAc,CAAC,CAAC,CAAC;QAC7E,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,WAAW,EAAE,IAAI,CAAC,IAAI,CAAC,UAAU,GAAG,OAAO,CAAC,cAAc,CAAC,CAAC,CAAC;QAC3F,OAAO,KAAK,GAAG,IAAI,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC;IACnG,CAAC;IAED,IAAI,KAAK,GAAG,OAAO,CAAC,WAAW,CAAC;IAChC,IAAI,IAAI,GAAG,CAAC,CAAC;IACb,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACvC,MAAM,KAAK,GAAG,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC7B,IAAI,KAAK,GAAG,OAAO,CAAC,cAAc,GAAG,YAAY,IAAI,KAAK,GAAG,UAAU,EAAE,CAAC;YACxE,IAAI,CAAC,GAAG,KAAK;gBAAE,KAAK,GAAG,CAAC,CAAC;YACzB,IAAI,CAAC,GAAG,CAAC,GAAG,IAAI;gBAAE,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;QACjC,CAAC;IACH,CAAC;IACD,OAAO,KAAK,GAAG,IAAI,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC;AACnG,CAAC;AAED;;;;GAIG;AACH,SAAS,MAAM,CAAC,IAAwB,EAAE,OAAe;IACvD,OAAO,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,aAAa,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,IAAI,GAAG,CAAC;AACnE,CAAC;AAED,SAAS,aAAa,CAAC,OAAe;IACpC;;;;;;;;MAQE;IACF,MAAM,IAAI,GAAG,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC;IAC/B,OAAO,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC;AACvE,CAAC","sourcesContent":["/**\n * Parsing for the time-range options.\n *\n * Researchers write offsets in whatever form is natural for the recording in front\n * of them: seconds for a short ECG strip, `30m` into a sleep study, `01:23:45` when\n * reading off a clock. All three are accepted; anything ambiguous is rejected with\n * a message that shows the forms that work.\n */\n\nimport { fixed, formatDuration } from '../format/number.js';\nimport { assertOptions, assertRecordShape, describeValue, OptionError } from './options.js';\n\nexport class TimeRangeError extends Error {\n  constructor(message: string) {\n    super(message);\n    this.name = 'TimeRangeError';\n  }\n}\n\n/**\n * Every spelling of a unit this accepts, and what one of it is worth in seconds.\n *\n * Exported so the spellings can be enumerated by a test rather than listed a second time:\n * twelve of these sixteen are named nowhere in the tool or its documentation, and none of\n * their values was checked by anything. `hrs: 360` would have converted `--start 2hrs` from\n * twelve minutes in and said nothing, which is the one kind of mistake a window may not make.\n */\nexport const UNIT_SECONDS: Record<string, number> = {\n  h: 3600,\n  hr: 3600,\n  hrs: 3600,\n  hour: 3600,\n  hours: 3600,\n  m: 60,\n  min: 60,\n  mins: 60,\n  minute: 60,\n  minutes: 60,\n  s: 1,\n  sec: 1,\n  secs: 1,\n  second: 1,\n  seconds: 1,\n  ms: 0.001,\n};\n\nconst CLOCK = /^(?:(\\d+):)?(\\d{1,2}):(\\d{1,2}(?:\\.\\d+)?)$/u;\nconst UNIT_TOKEN = /(\\d+(?:\\.\\d+)?)\\s*([a-z]+)/giu;\n\n/**\n * Parse a duration or offset into seconds.\n *\n * Accepted: `90`, `90s`, `5m`, `1h30m`, `1h 30m 15s`, `00:30:00`, `30:00`, `250ms`.\n *\n * `allowNegative` is for the two options that name a position rather than a length.\n *\n * A recording is timed from its first record's timekeeping annotation, and nothing obliges that\n * to sit at or after zero: a file whose records run from -100 s to -97 s is one this tool reads,\n * times from -100, and describes with\n *\n *     Timed from -100.000s  (first sample; --start and --end use this clock)\n *\n * That line says the number can be typed straight back in, and it could not be. Every offset\n * such a recording has came back as \"not a time I understand\", so its whole clock was\n * unreachable and no window of it could be converted at all — the one file shape where a\n * window is refused for naming a moment the recording actually contains.\n *\n * A length below zero is still a different thing, and `--duration` still refuses one.\n */\nexport function parseTimeSpec(input: string, optionName?: string, allowNegative = false): number {\n  /*\n    A name for the value, when the caller has not given one.\n\n    Every refusal below opens with it, and the CLI passes the flag — `--start \"1,5\" is not a\n    time I understand`. This is also the function the api page sends other people's users to,\n    and a caller who writes the one-argument call its neighbours take got\n\n        TimeRangeError: undefined is empty. Try a value like 30s, 5m, or 00:30:00.\n\n    on every one of the eleven messages in here. The paragraph below hardened `input` for\n    exactly that caller; the argument beside it could still put the word `undefined` at the\n    front of the sentence.\n\n    Coerced rather than refused, unlike `input`: this one names the value in a message rather\n    than being the value, so there is a right answer to fall back on and nothing is lost by\n    using it. An empty or non-string name takes the same route, since `\"\" is empty.` is the\n    same sentence with a different hole in it.\n  */\n  const named =\n    typeof optionName === 'string' && optionName.trim() !== '' ? optionName : 'The value';\n  /*\n    Text, because this is the function the API page points other people's users at: \"Use\n    `parseTimeSpec` if you want to accept those forms from your own users.\"\n\n    A value arriving from a JSON config or a form field is a number as often as a string, and\n    `{ \"start\": 30 }` came back as `TypeError: input.trim is not a function` — a variable name\n    from inside this file, thrown past a caller who had done exactly what the page told them\n    to. Every other refusal here is a TimeRangeError quoting the value and the option. The\n    number is not coerced, for the reason `channels: 'ECG'` is not iterated: a shape that\n    happens to work for 30 also swallows NaN, and the CLI already writes `String(raw)`.\n  */\n  if (typeof input !== 'string') {\n    throw new TimeRangeError(\n      `${named} must be given as text, not ${describeValue(input)}. ` +\n        `Seconds are written \"30\", and the other forms are \"30s\", \"5m\", \"1h30m\", \"00:30:00\".`,\n    );\n  }\n  const trimmed = input.trim().toLowerCase();\n  if (trimmed === '') {\n    throw new TimeRangeError(`${named} is empty. Try a value like 30s, 5m, or 00:30:00.`);\n  }\n\n  /*\n    The sign is read off the front and applied to the whole value, so `-1h30m` is an hour and a\n    half before the origin rather than an hour before it and half an hour after.\n\n    Nothing may sit between the sign and the number, and a leading `+` is still refused: every\n    other number this tool takes is refused when written in a form nobody types, and `+5` is\n    exactly what `--decimals` and `--jobs` reject one flag over.\n  */\n  const negative = trimmed.startsWith('-');\n  const text = negative ? trimmed.slice(1) : trimmed;\n\n  /*\n    A leading plus, named as such.\n\n    Refusing it is deliberate — see the paragraph above — but it came back as \"is not a time I\n    understand. Try 30s, 5m, 1h30m, 00:30:00, or a plain number of seconds\", which is the\n    message for input that could not be read at all. Every part of `+5s` was read: the number\n    parsed, the unit was looked up and found. The one thing wrong with it is a character the\n    reader put there on purpose, and they were sent to re-check their unit spellings with `5m`\n    sitting in the list of suggestions, differing from what they typed by the sign they cannot\n    see is the problem. 0.7.102 made this argument about a space; this is the same sentence\n    about the other character that reaches it. `--decimals` and `--jobs` both quote the value\n    back and say what is wrong with it.\n\n    Every plus, not the first one, and the leftover space with it — the same rule the spaced\n    refusal below applies to spaces, for the same reason: a hint whose command this function\n    refuses is worse than no hint. `+1h+30m` was answered with `1h+30m`, which comes straight\n    back as \"is not a time I understand\", and `+ 5s` with a value that opened on a space.\n  */\n  if (text.startsWith('+')) {\n    throw new TimeRangeError(\n      `${named} \"${input}\" begins with a plus. Write the number on its own: ` +\n        `${input.trim().replaceAll('+', '').trim()}`,\n    );\n  }\n  const signed = (seconds: number): number =>\n    assertFinite(negative ? -seconds : seconds, named, input, allowNegative);\n\n  // Bare number means seconds.\n  if (/^\\d+(?:\\.\\d+)?$/u.test(text)) return signed(Number(text));\n\n  // Clock form: hh:mm:ss or mm:ss.\n  const clock = CLOCK.exec(text);\n  if (clock) {\n    const hours = clock[1] === undefined ? 0 : Number(clock[1]);\n    const minutes = Number(clock[2]);\n    const seconds = Number(clock[3]);\n    if (minutes >= 60 || seconds >= 60) {\n      throw new TimeRangeError(\n        `${named} \"${input}\" has a minutes or seconds field of 60 or more.`,\n      );\n    }\n    return signed(hours * 3600 + minutes * 60 + seconds);\n  }\n\n  // Unit form: 1h30m, 1h30m 15s, 250ms. A number sits directly against its unit; see below.\n  UNIT_TOKEN.lastIndex = 0;\n  let total = 0;\n  let matched = 0;\n  let consumed = 0;\n  /** Whether any token had a space between its number and its unit. See the refusal below. */\n  let spaced = false;\n  let match: RegExpExecArray | null;\n  // Which units have already been seen, so a repeat can be rejected rather than added.\n  const seen = new Set<number>();\n  while ((match = UNIT_TOKEN.exec(text)) !== null) {\n    const amount = Number(match[1]);\n    const unit = match[2] ?? '';\n    const scale = UNIT_SECONDS[unit];\n    if (scale === undefined) {\n      throw new TimeRangeError(\n        `${named} \"${input}\" uses an unknown unit \"${unit}\". Use h, m, s or ms, ` +\n          `or their long forms: hours, minutes, seconds.`,\n      );\n    }\n\n    // \"1h1h\" is a typo, not a request for two hours. Summing repeated units silently\n    // turned a slip into a plausible window that was quietly the wrong length. Units are\n    // keyed by their scale so the aliases collapse together: \"1h30min20m\" is caught too.\n    if (seen.has(scale)) {\n      throw new TimeRangeError(\n        `${named} \"${input}\" gives the same unit twice. ` +\n          `Combine each unit once, as in 1h30m.`,\n      );\n    }\n    seen.add(scale);\n\n    total += amount * scale;\n    matched++;\n    // Counted with the whitespace taken out, so the comparison below is against the same\n    // string on both sides and a space inside a token reaches the refusal written for it\n    // rather than the one about input that could not be read at all.\n    consumed += match[0].replace(/\\s+/gu, '').length;\n    if (/\\s/u.test(match[0])) spaced = true;\n  }\n\n  // Reject partially-understood input like \"5x\" or \"1h banana\".\n  if (matched === 0 || consumed !== text.replace(/\\s+/gu, '').length) {\n    throw new TimeRangeError(\n      `${named} \"${input}\" is not a time I understand. ` +\n        `Try 30s, 5m, 1h30m, 00:30:00, or a plain number of seconds.`,\n    );\n  }\n\n  /*\n    A space between a number and its unit, named as such.\n\n    `5 min` is refused on purpose — cli-reference sets the rule out, and it is what keeps\n    `1 2h` from being read as anything — but it came back as \"is not a time I understand. Try\n    30s, 5m, 1h30m, 00:30:00, or a plain number of seconds\", which is the message for input\n    that could not be read at all. Every part of `5 min` was read: the number parsed, the unit\n    was looked up in the table and found. The one thing wrong with it is a space, and the\n    reader was sent to re-check their unit spellings instead — with `5m` sitting in the list of\n    suggestions, differing from what they typed by a character they cannot see is the problem.\n\n    Only reachable now that the count above ignores whitespace; before, the length check caught\n    these first and there was nothing left to tell them apart by.\n  */\n  if (spaced) {\n    throw new TimeRangeError(\n      `${named} \"${input}\" puts a space between a number and its unit. ` +\n        `Write them together: ${input.trim().replace(/(\\d)\\s+([a-z])/giu, '$1$2')}`,\n    );\n  }\n\n  return signed(total);\n}\n\nfunction assertFinite(\n  value: number,\n  optionName: string,\n  input: string,\n  allowNegative: boolean,\n): number {\n  /*\n    Overflow answers for itself, rather than borrowing the sentence about signs.\n\n    One branch covered two rejections. `--start` with four hundred nines is a positive value\n    that `Number` returns as `Infinity`, and the answer was `is not a valid non-negative\n    time` — which names the one thing about it that is not the problem. Worse on the two\n    options this is called with `allowNegative` for: `--start` and `--end` take a sign on\n    purpose, since a recording timed from before zero has no other way to be addressed, so\n    \"non-negative\" is not their rule at all and overflow is the only way they reach here.\n  */\n  if (!Number.isFinite(value)) {\n    throw new TimeRangeError(\n      `${optionName} \"${input}\" is further from zero than a number of seconds can hold.`,\n    );\n  }\n  if (!allowNegative && value < 0) {\n    throw new TimeRangeError(`${optionName} \"${input}\" is not a valid non-negative time.`);\n  }\n  return value;\n}\n\nexport interface ResolvedRange {\n  /** Inclusive start, in seconds from the beginning of the recording. */\n  startSeconds: number;\n  /** Exclusive end, in seconds. */\n  endSeconds: number;\n  /** First data record touching the window. */\n  startRecord: number;\n  /** One past the last data record touching the window. */\n  endRecord: number;\n  /** True when the window covers the whole recording. */\n  isWholeRecording: boolean;\n  /** Earliest record start, including EDF+D timing gaps. */\n  recordingStartSeconds: number;\n  /** End of the latest record, including EDF+D timing gaps. */\n  recordingEndSeconds: number;\n}\n\n/**\n * Slack for comparisons at sample/window boundaries.\n *\n * A nanosecond is far below any real sampling interval — 20 kHz is 50 microseconds — so it\n * absorbs the arithmetic error in `recordStart + sample / rate` without reaching a\n * neighbouring sample.\n */\nexport const BOUNDARY_TOLERANCE = 1e-9;\n\n/**\n * The slack to use for a channel sampled this often.\n *\n * Never past half a sample interval, because slack that reaches the next sample stops being\n * slack — half is the most that cannot, since the nearest sample below a bound sits a whole\n * interval away from it. `Math.min` takes exactly half at rates above 500 MHz, which is where\n * this said \"never as much as half\" of a formula that returns it. A fixed nanosecond was\n * applied whatever the rate, and the format does not\n * oblige the interval to be larger than it: EDF's record duration is an 8-character field\n * that accepts `1e-9`. A recording of two 1 ns records holding ten samples each wrote ten of\n * its twenty rows — the window ends at 2e-9, the comparison asked for `time < 2e-9 - 1e-9`,\n * and the entire second record failed it. Exit 0, no warning, half the samples gone.\n */\nexport function toleranceFor(rate: number): number {\n  if (!(rate > 0) || !Number.isFinite(rate)) return BOUNDARY_TOLERANCE;\n  return Math.min(BOUNDARY_TOLERANCE, 1 / rate / 2);\n}\n\n/** Match the exact half-open boundary rules used while writing signal rows. */\nexport function sampleTimeIsInRange(\n  time: number,\n  startSeconds: number,\n  endSeconds: number,\n  tolerance: number = BOUNDARY_TOLERANCE,\n): boolean {\n  return time >= startSeconds - tolerance && time < endSeconds - tolerance;\n}\n\n/** Count samples from one record that fall inside a half-open requested window. */\nexport function countSamplesInRange(options: {\n  recordStart: number;\n  rate: number;\n  samplesPerRecord: number;\n  startSeconds: number;\n  endSeconds: number;\n}): number {\n  /*\n    A record with no place on the clock contributes no rows, which is what the conversion\n    already does with one: its sample times are non-finite, so every one of them fails the\n    range test and none is written. The arithmetic below reached `Infinity - Infinity` and\n    answered `NaN`, which `--info` then printed — \"Would write NaN rows, roughly NaN B.\" on a\n    recording that goes on to write eight. Reachable from a header: a record duration of 1e308\n    is five characters in an eight-character field, and the third record of one is past what a\n    double can hold.\n  */\n  if (!Number.isFinite(options.recordStart)) return 0;\n  const slack = toleranceFor(options.rate);\n  const lower = Math.ceil(\n    (options.startSeconds - slack - options.recordStart) * options.rate,\n  );\n  const upper = Math.ceil(\n    (options.endSeconds - slack - options.recordStart) * options.rate,\n  );\n  const first = Math.max(0, Math.min(options.samplesPerRecord, lower));\n  const last = Math.max(0, Math.min(options.samplesPerRecord, upper));\n  return Math.max(0, last - first);\n}\n\n/**\n * Whether two instants differ only by the arithmetic that produced them.\n *\n * A relative epsilon, because the gap between doubles grows with magnitude — the same shape\n * the long layout uses to decide two sample times are one instant. Well below any real sample\n * interval, and well above the rounding that two routes to one quantity produce: a recording's\n * length is `recordCount * recordDuration`, which for 6003 records of 0.1s is not the 600.3 it\n * prints as.\n */\nfunction sameInstant(a: number, b: number): boolean {\n  /*\n    A relative tolerance has nothing to be relative to at infinity.\n\n    `Math.abs(a - Infinity)` is Infinity, and so is the allowance beside it, so\n    `Infinity <= Infinity` made every instant the same instant as an infinite one. A\n    recording whose end overflows a double — three records of 1e308 seconds, which an\n    8-character record-duration field can state and a timekeeping TAL can place — was\n    therefore refused the window nobody asked for:\n\n        $ edf2csv rec.edf --info\n        error: --start 0s is at or past the end of this unknown recording.\n\n    Exit 2 for a file, naming a flag the command did not carry, quoting a length that is\n    `formatDuration`'s word for a number it will not print. `--info` is the one command that\n    would have explained the file and it is the one that could not run.\n\n    Finite or equal, which is the only reading that means anything: two infinities are the\n    same non-instant, and a real offset is not one of them.\n  */\n  if (!Number.isFinite(a) || !Number.isFinite(b)) return a === b;\n  return Math.abs(a - b) <= Math.max(Math.abs(a), Math.abs(b)) * 1e-12;\n}\n\n/**\n * Turn a requested window into both an exact time span and the record range that\n * contains it. Records are the unit the file can be read in; the exact span is what\n * decides which samples inside those records are actually written.\n */\nexport function resolveRange(options: {\n  start?: number | undefined;\n  /** The `--start` value exactly as typed, quoted back in the past-the-end error. */\n  startText?: string | undefined;\n  duration?: number | undefined;\n  /** The `--duration` value exactly as typed, quoted back in the window error. */\n  durationText?: string | undefined;\n  end?: number | undefined;\n  /** The `--end` value exactly as typed, quoted back in the window error. */\n  endText?: string | undefined;\n  recordDuration: number;\n  recordCount: number;\n  /**\n   * True start time of each data record, for discontinuous files. When absent,\n   * records are assumed to sit end to end.\n   */\n  recordStarts?: Float64Array | null | undefined;\n}): ResolvedRange {\n  /*\n    The same check `buildPlan` makes, made here too, because this is exported on its own.\n\n    `assertOptions` sits at the top of `buildPlan`, so `convert` was covered and the function\n    underneath it was not — and this one has its own signature block on the API page.\n    `resolveRange({ start: NaN, ... })` resolved: no error, no warning, and a range read back\n    as `startSeconds: null, startRecord: null`, which is the \"takes the whole recording\n    without saying so\" that comment names. `{ end: '2' }` was coerced by the arithmetic and\n    accepted; `{ start: '30' }` reached the past-the-end error and printed it with the value\n    missing — `--start s is at or past the end of this 3s recording`.\n  */\n  /*\n    The argument bag itself, before three properties are read off it.\n\n    `resolveRange(42)` read `.start` off a number, which is `undefined` rather than a throw, and\n    went on to answer with a range; `resolveRange(null)` reached the same line and came back as\n    `TypeError: Cannot read properties of null (reading 'start')`, naming a property of this\n    function's own parameter.\n  */\n  if (options === null || typeof options !== 'object') {\n    throw new OptionError(\n      `resolveRange takes one object — the recording's shape and the window asked of it — ` +\n        `got ${describeValue(options)}.`,\n    );\n  }\n  assertOptions({ start: options.start, duration: options.duration, end: options.end });\n  /*\n    And the two numbers the window is measured against, which that call does not cover.\n\n    `assertOptions` checks what was asked for; these are what it is asked of, and nothing\n    looked at them. `resolveRange({ start: 1, recordDuration: 1 })` — no `recordCount` —\n    answered `{ startSeconds: 0, endSeconds: null, startRecord: 0, endRecord: 0 }`: a window\n    over no records, handed back as a fact about a recording. `resolveRange(42)` reached the\n    same answer, since reading `.start` off a number is `undefined` rather than a throw.\n\n    The check `buildPlan` has made since 0.8.64, made here too, because this is exported on\n    its own — the same sentence the line above this one is here for.\n  */\n  assertRecordShape(options);\n\n  // For a continuous file the recording spans recordCount * recordDuration. A\n  // discontinuous one does not: a 10-second recording with a 95-second gap in the\n  // middle still ends at 105 seconds. Deriving the span from the records' real\n  // positions is what stops a requested window from being clipped back to the\n  // amount of *data* in the file and silently discarding everything past it.\n  const { earliest, latest } = span(options.recordStarts, options.recordCount, options.recordDuration);\n\n  if (options.duration !== undefined && options.end !== undefined) {\n    throw new TimeRangeError('Use either --duration or --end, not both.');\n  }\n\n  const startSeconds = options.start ?? earliest;\n\n  let endSeconds: number;\n  if (options.duration !== undefined) endSeconds = startSeconds + options.duration;\n  else if (options.end !== undefined) endSeconds = options.end;\n  else endSeconds = latest;\n\n  /*\n    At the end, allowing for the arithmetic that produced the end.\n\n    `latest` is `recordCount * recordDuration`, and with a fractional duration that is not the\n    number it prints as: 6003 records of 0.1s is 600.3000000000001, not 600.3. So `--start\n    600.3` on a recording `--info` calls \"10m 0.3s\" was accepted by a hair, converted nothing,\n    and exited 0 with a signals.csv holding its header — which is the empty conversion this\n    error exists to prevent, and which the same command on a whole-second recording is refused\n    for.\n\n    A relative epsilon, the same shape the long layout uses to decide two sample times are one\n    instant: well below any real interval, and well above the rounding that two different\n    routes to the same quantity produce.\n  */\n  if (startSeconds >= latest || sameInstant(startSeconds, latest)) {\n    /*\n      Quote what was typed. Reporting the parsed seconds meant `--start 4h` came back as\n      \"--start 14400s is at or past the end\", which reads as a value the user never gave.\n\n      In quotation marks, which the parse errors above have always used and this did not.\n      Without them the value ran into the sentence: `--start \"  5s  \"` printed as\n      `--start   5s   is at or past the end`, where the value appears to be `5s   is` and\n      the surrounding spaces — the actual reason a shell-built argument went wrong — are\n      invisible.\n    */\n    throw new TimeRangeError(\n      /*\n        The recording's length in the same words --info uses for it.\n\n        `formatSeconds` renders a bare number of seconds, so this message and the Duration\n        line disagreed about one file in one session: --info said \"6m 40s\" and the error said\n        \"400s\". On an overnight recording it read \"7950s recording\", leaving the reader to\n        divide by 3600 to find out whether their --start was reasonable — which is the one\n        question this message exists to answer. cli-reference.md has always documented it\n        humanised (\"2h 12m 30s\"), a form no input could produce.\n\n        The typed value keeps `quoted`, since that is the user's own text and should come\n        back exactly as they wrote it.\n      */\n      /*\n        `latest` is where the recording ends on its own clock, which is its length only when\n        it starts at zero. A file timed from its first record's timekeeping TAL need not: one\n        whose records run 1000s to 1003s is three seconds long, and this called it \"this\n        16m 43s recording\" — while --info two lines away said \"Duration 3s\". Where the\n        recording sits is the useful thing to say in that case, and it is the number --start\n        has to be given.\n      */\n      earliest === 0\n        ? `--start ${quoted(options.startText, startSeconds)} is at or past the end of this ` +\n          `${formatDuration(latest)} recording.`\n        : `--start ${quoted(options.startText, startSeconds)} is at or past the end of this ` +\n          `${formatDuration(latest - earliest)} recording, which runs from ` +\n          `${formatSeconds(earliest)} to ${formatSeconds(latest)}.`,\n    );\n  }\n  if (endSeconds <= startSeconds) {\n    // Quote whatever the caller actually gave, for the same reason as the error above.\n    const endShown =\n      options.end !== undefined && options.endText !== undefined\n        ? quoted(options.endText, endSeconds)\n        : formatSeconds(endSeconds);\n    const startShown = quoted(options.startText, startSeconds);\n    /*\n      Name the option that caused it, when only one of them could have.\n\n      `end = start + duration`, and a negative duration is refused before this, so a window\n      that ends where it starts and was built from `--duration` was built from a duration of\n      zero — the one value in the command that is wrong. The message named neither it nor the\n      flag:\n\n          edf2csv rec.edf --start 5s --duration 0\n          error: The requested window ends at 5s, which is not after its start at \"5s\".\n\n      Two mentions of the value that is fine, none of the one that is not, in a file whose\n      other refusals quote what was typed on the reasoning that \"`--start 4h` came back as\n      '--start 14400s is at or past the end', which reads as a value the user never gave\".\n      `--duration` had no typed text to quote because nothing passed it; it does now, like\n      its two neighbours.\n    */\n    throw new TimeRangeError(\n      options.end === undefined && options.duration !== undefined\n        ? `--duration ${quoted(options.durationText, options.duration)} is not a length of ` +\n          `time, so the window ends where it starts, at ${startShown}. Give a length above ` +\n          `zero, or leave --duration out to convert to the end.`\n        : `The requested window ends at ${endShown}, which is not after its start at ${startShown}.`,\n    );\n  }\n\n  const clampedEnd = Math.min(endSeconds, latest);\n  /*\n    Both ends of the window, clamped the same way.\n\n    The end has been clamped to the recording since these fields existed — `--end 999h` on a\n    two-hour file is documented as converting to the end, silently — and the start was left as\n    typed. So a window asked for before the recording begins was recorded as one:\n\n        edf2csv two-second.edf --out csv --start=-500\n        metadata.json  \"start_seconds\": -500, \"end_seconds\": 2\n\n    over a signals.csv whose first row is 0.000, with `records_converted` correctly `[0, 2]`.\n    output-files calls the pair \"the resolved time window\", which the end is and the start was\n    not, and a script taking `end_seconds - start_seconds` for the span converted got 502 for\n    two seconds of data.\n\n    `earliest` and not zero, because a recording is timed from its first record and need not\n    begin there: negative-origin.edf runs from -100s, and -100 is where a start before the\n    recording belongs. Nothing downstream moves — every sample sits at or after `earliest`, so\n    the row filter and the record selection answer the same for either value, and the byte\n    estimate takes a width from the smaller magnitude, which is the one the column actually\n    holds.\n\n    Not when the whole window sits before the recording, where there is no overlap to clamp\n    to and raising the start past the end would describe the request backwards: `--start 0\n    --duration 0.001` on a file beginning at 30s is an empty window, and EMPTY_WINDOW quotes\n    the bounds — \"(30.000s to 0.001s)\" says nothing a reader can act on, where the bounds as\n    asked for are what its hint is already explaining.\n  */\n  const clampedStart = clampedEnd <= earliest ? startSeconds : Math.max(startSeconds, earliest);\n  const { startRecord, endRecord } = selectRecords(options, clampedStart, clampedEnd);\n\n  return {\n    startSeconds: clampedStart,\n    endSeconds: clampedEnd,\n    startRecord,\n    endRecord,\n    /*\n      The same rounding, one field over.\n\n      `latest` is `recordCount * recordDuration`, and 6003 records of 0.1s is\n      600.3000000000001. So `--end 600.3` on a recording of exactly that length wrote every\n      sample it has — byte-identical to a bare conversion — and metadata.json recorded\n      `whole_recording: false` for it, while the bare run recorded true. One conversion, two\n      answers, on the field a pipeline reads to decide whether it has the lot.\n    */\n    isWholeRecording:\n      (startSeconds <= earliest || sameInstant(startSeconds, earliest)) &&\n      (clampedEnd >= latest || sameInstant(clampedEnd, latest)),\n    recordingStartSeconds: earliest,\n    recordingEndSeconds: latest,\n  };\n}\n\nfunction span(\n  recordStarts: Float64Array | null | undefined,\n  recordCount: number,\n  recordDuration: number,\n): { earliest: number; latest: number } {\n  if (!recordStarts || recordStarts.length === 0) {\n    return { earliest: 0, latest: recordCount * recordDuration };\n  }\n  let earliest = Infinity;\n  let latest = -Infinity;\n  for (const start of recordStarts) {\n    if (start < earliest) earliest = start;\n    if (start + recordDuration > latest) latest = start + recordDuration;\n  }\n  if (!Number.isFinite(earliest) || !Number.isFinite(latest)) {\n    return { earliest: 0, latest: recordCount * recordDuration };\n  }\n  return { earliest, latest };\n}\n\n/** Every record whose own time span overlaps the requested window. */\nfunction selectRecords(\n  options: { recordDuration: number; recordCount: number; recordStarts?: Float64Array | null | undefined },\n  startSeconds: number,\n  endSeconds: number,\n): { startRecord: number; endRecord: number } {\n  const starts = options.recordStarts;\n  if (!starts || starts.length === 0) {\n    /*\n      An empty range is `[0, 0]` here as it is below, rather than whatever the arithmetic\n      produced. Each bound was clamped on one side only — the start up to zero, the end down\n      to the record count — so a window lying entirely before the recording came back as\n      `records_converted: [0, -1]` in metadata.json, over a `endRecord` documented as \"one\n      past the last data record touching the window\". A script differencing the pair read\n      minus one record converted, and `ConversionProgress.recordsTotal` reported the same.\n\n      The discontinuous branch below has always answered `[0, 0]` for exactly this request,\n      so the two halves of one function disagreed about how to say \"none\": the same window on\n      `annotations.edf` and on `discontinuous.edf` gave `[0, -1]` and `[0, 0]`.\n    */\n    const first = Math.max(0, Math.floor(startSeconds / options.recordDuration));\n    const last = Math.min(options.recordCount, Math.ceil(endSeconds / options.recordDuration));\n    return first < last ? { startRecord: first, endRecord: last } : { startRecord: 0, endRecord: 0 };\n  }\n\n  let first = options.recordCount;\n  let last = 0;\n  for (let i = 0; i < starts.length; i++) {\n    const begin = starts[i] ?? 0;\n    if (begin + options.recordDuration > startSeconds && begin < endSeconds) {\n      if (i < first) first = i;\n      if (i + 1 > last) last = i + 1;\n    }\n  }\n  return first < last ? { startRecord: first, endRecord: last } : { startRecord: 0, endRecord: 0 };\n}\n\n/**\n * The value as the caller typed it, in quotation marks, or the parsed seconds if they gave\n * none. The marks show where the value begins and ends, which matters most for the values\n * that went wrong because of what surrounds them.\n */\nfunction quoted(text: string | undefined, seconds: number): string {\n  return text === undefined ? formatSeconds(seconds) : `\"${text}\"`;\n}\n\nfunction formatSeconds(seconds: number): string {\n  /*\n    The same 1e21 cliff, in the bounds this message hands back.\n\n    `--start \"9e21\"` was answered with \"is at or past the end of this 3e+21s recording, which\n    runs from 1e+21s to 4e+21s\" — a sentence whose whole purpose is to say what window there\n    is to ask for, ending in two tokens the parser refuses: `--start 1e+21s` is \"uses an\n    unknown unit \\\"e\\\"\". `Number(...)` was here to drop the trailing zeros, and it also\n    re-introduced the exponent form that `toFixed` had produced.\n  */\n  const text = fixed(seconds, 3);\n  return `${text.includes('.') ? text.replace(/\\.?0+$/u, '') : text}s`;\n}\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edf2csv",
-  "version": "0.8.74",
+  "version": "0.8.75",
   "description": "Convert EDF, EDF+ and BDF biosignal recordings (European Data Format) to CSV from the command line. Local, streaming, and never resamples or alters units.",
   "keywords": [
     "edf",