@lde/distribution-probe 0.2.4 → 0.2.6

package/dist/probe.js

@@ -734,12 +734,12 @@ async function validateBody(body, contentType, baseIRI, budgetSignal, truncated)
  * Parse an RDF body just far enough to tell whether it carries any triples:
  * resolve on the first triple (presence is all we need, not a full count), on a
  * clean end with none ('empty'), or on a parse error. The parse is bounded by
- * `budgetSignal` – the caller's validation-budget {@link AbortController} –
+ * `budgetSignal` – the caller’s validation-budget {@link AbortController} –
  * because a JSON-LD `@context` is fetched from its origin, and a slow or hanging
  * context host would otherwise stall the probe past its budget; on abort – and
  * likewise when a remote `@context` is unreachable – the outcome is
  * 'inconclusive', so a valid distribution is never flagged faulty for a context
- * host's failure. Sharing the caller's single budget (rather than starting a
+ * host’s failure. Sharing the caller’s single budget (rather than starting a
  * second, identical timer) keeps that timeout deterministic: it fires once, when
  * the budget elapses, instead of leaving an orphan timer to settle on its own
  * schedule. `baseIRI` resolves any relative IRIs in the document.
@@ -765,20 +765,25 @@ function classifyRdfBody(body, contentType, baseIRI, budgetSignal, truncated) {
             quads.destroy();
             resolve(outcome);
         }
-        // The budget may already have elapsed while the body was being read; treat an
-        // already-aborted signal as an immediate expiry rather than waiting for an
-        // 'abort' event that will never come.
-        if (budgetSignal.aborted) {
-            onBudgetElapsed();
-            return;
-        }
-        budgetSignal.addEventListener('abort', onBudgetElapsed, { once: true });
+        // Attach the parser listeners before consulting the budget: settle() calls
+        // quads.destroy(), which can make the parser emit a late 'error', so the
+        // error sink must already be in place – including on the already-aborted
+        // path below – or that error goes unhandled and crashes the process.
         quads
             .on('data', () => settle({ type: 'hasTriples' }))
             .on('error', (error) => settle(truncated || isRemoteContextError(error)
             ? { type: 'inconclusive' }
             : { type: 'parseError', message: error.message }))
             .on('end', () => settle(truncated ? { type: 'inconclusive' } : { type: 'empty' }));
+        // The budget may already have elapsed while the body was being read; an
+        // already-aborted signal never emits another 'abort', so settle immediately
+        // rather than wait for an event that will not come.
+        if (budgetSignal.aborted) {
+            onBudgetElapsed();
+        }
+        else {
+            budgetSignal.addEventListener('abort', onBudgetElapsed, { once: true });
+        }
     });
 }
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lde/distribution-probe",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "repository": {
     "url": "git+https://github.com/ldelements/lde.git",
     "directory": "packages/distribution-probe"