reffy 10.0.2 → 10.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reffy",
-  "version": "10.0.2",
+  "version": "10.0.4",
   "description": "W3C/WHATWG spec dependencies exploration companion. Features a short set of tools to study spec references as well as WebIDL term definitions and references found in W3C specifications.",
   "repository": {
     "type": "git",
@@ -34,9 +34,9 @@
     "abortcontroller-polyfill": "1.7.3",
     "commander": "9.4.0",
     "fetch-filecache-for-crawling": "4.1.0",
-    "puppeteer": "17.1.3",
+    "puppeteer": "18.0.5",
     "semver": "^7.3.5",
-    "web-specs": "2.24.0",
+    "web-specs": "2.25.0",
     "webidl2": "24.2.2"
   },
   "devDependencies": {
@@ -45,7 +45,7 @@
     "nock": "13.2.9",
     "respec": "32.2.4",
     "respec-hljs": "2.1.1",
-    "rollup": "2.79.0"
+    "rollup": "2.79.1"
   },
   "scripts": {
     "test": "mocha --recursive tests/"

package/src/browserlib/extract-headings.mjs

@@ -13,6 +13,7 @@ export default function (spec, idToHeading) {
     const headingLevel = headingNumber ? headingNumber.split(".").length : undefined;
     return {
       id: n.parentNode.id,
+      href: getAbsoluteUrl(n.parentNode, { singlePage }),
       title: n.textContent.replace(headingNumber, '').trim(),
       level: headingLevel,
       number: headingNumber

package/src/browserlib/extract-references.mjs

@@ -101,7 +101,10 @@ function parseReferences(referenceList, options) {
     if (!desc || !ref.name) {
       return;
     }
-    ref.url = desc.querySelector('a[href^="http"]')?.href ?? "";
+    const url = desc.querySelector('a[href*="://"]')?.href;
+    if (url) {
+      ref.url = url;
+    }
     if (options.filterInformative &&
         desc.textContent.match(/non-normative/i)) {
       return informativeRef.push(ref);

package/src/browserlib/get-absolute-url.mjs

@@ -16,6 +16,9 @@ export default function (node, { singlePage, attribute } =
   const page = singlePage ? null :
     node.closest('[data-reffy-page]')?.getAttribute('data-reffy-page');
   const url = new URL(page ?? window.location.href);
-  url.hash = '#' + node.getAttribute(attribute);
+  const hashid = node.getAttribute(attribute);
+  if (hashid) {
+    url.hash = '#' + hashid;
+  }
   return url.toString();
 }

package/src/browserlib/map-ids-to-headings.mjs

@@ -79,11 +79,13 @@ export default function () {
       const match = trimmedText.match(reNumber);
       const number = match ? match[1] : null;
 
-      mappingTable[nodeid] = {
-        id,
-        href,
-        title: trimmedText.replace(reNumber, '').trim().replace(/\s+/g, ' ')
-      };
+      const mapping = {};
+      if (id) {
+        mapping.id = id;
+      }
+      mapping.href = href;
+      mapping.title = trimmedText.replace(reNumber, '').trim().replace(/\s+/g, ' ');
+      mappingTable[nodeid] = mapping;
 
       if (number) {
         // Store the number without the final "."
@@ -120,11 +122,13 @@ function esMapIdToHeadings() {
       const match = trimmedText.match(reNumber);
       const number = match ? match[1] : null;
 
-      mappingTable[nodeid] = {
-        id: section.id,
-        href,
-        title: trimmedText.replace(reNumber, '').trim().replace(/\s+/g, ' ')
-      };
+      const mapping = {};
+      if (section.id) {
+        mapping.id = section.id;
+      }
+      mapping.href = href;
+      mapping.title = trimmedText.replace(reNumber, '').trim().replace(/\s+/g, ' ');
+      mappingTable[nodeid] = mapping;
 
       if (number) {
         // Store the number without the final "."

package/src/cli/parse-webidl.js

@@ -362,6 +362,7 @@ function parseType(idltype, idlReport, contextName) {
             "short", "unsigned short",
             "long", "unsigned long", "long long", "unsigned long long",
             "float", "unrestricted float", "double", "unrestricted double",
+            "bigint",
             "DOMString", "ByteString", "USVString",
             "object"
     ];

package/src/lib/post-processor.js

@@ -249,6 +249,14 @@ function dependsOn(mod) {
   return mod.dependsOn;
 }
 
+/**
+ * Return the name of the property that will be set in the spec crawl result
+ * when the post-processing module runs, if any
+ */
+function getProperty(mod) {
+  mod = getModule(mod);
+  return mod.property ?? mod.name;
+}
 
 function appliesAtLevel(mod, level) {
   mod = getModule(mod);
@@ -266,5 +274,6 @@ module.exports = {
   run, save,
   extractsPerSeries,
   dependsOn,
+  getProperty,
   appliesAtLevel
 };

package/src/lib/specs-crawler.js

@@ -130,7 +130,7 @@ async function crawlSpec(spec, crawlOptions) {
             }
         });
         crawlOptions.post?.forEach(mod => {
-            const prop = mod.property ?? mod.name;
+            const prop = postProcessor.getProperty(mod);
             if (postProcessor.appliesAtLevel(mod, 'spec') && result[prop]) {
                 spec[prop] = result[prop];
             }