@likecoin/epubcheck-ts 0.3.3 → 0.3.4

package/dist/index.js

@@ -1756,9 +1756,14 @@ var ContentValidator = class {
       } else if (item.mediaType === "text/css" && refValidator) {
         const fullPath = opfDir ? `${opfDir}/${item.href}` : item.href;
         this.validateCSSDocument(context, fullPath, opfDir, refValidator);
-      } else if (item.mediaType === "image/svg+xml" && registry) {
+      } else if (item.mediaType === "image/svg+xml") {
         const fullPath = opfDir ? `${opfDir}/${item.href}` : item.href;
-        this.extractSVGIDs(context, fullPath, registry);
+        if (registry) {
+          this.extractSVGIDs(context, fullPath, registry);
+        }
+        if (context.version.startsWith("3")) {
+          this.validateSVGDocument(context, fullPath, item);
+        }
       }
     }
   }
@@ -1772,11 +1777,75 @@ var ContentValidator = class {
     try {
       doc = XmlDocument.fromString(svgContent);
       this.extractAndRegisterIDs(path, doc.root, registry);
-    } catch {
+    } catch (e) {
+      pushMessage(context.messages, {
+        id: MessageId.RSC_016,
+        message: e instanceof Error ? e.message : "SVG parsing failed",
+        location: { path }
+      });
     } finally {
       doc?.dispose();
     }
   }
+  validateSVGDocument(context, path, manifestItem) {
+    const svgData = context.files.get(path);
+    if (!svgData) return;
+    const svgContent = new TextDecoder().decode(svgData);
+    let doc;
+    try {
+      doc = XmlDocument.fromString(svgContent);
+    } catch {
+      return;
+    }
+    try {
+      const root = doc.root;
+      const hasRemote = this.detectSVGRemoteResources(root);
+      if (hasRemote && !manifestItem.properties?.includes("remote-resources")) {
+        pushMessage(context.messages, {
+          id: MessageId.OPF_014,
+          message: 'SVG document references remote resources but manifest item is missing "remote-resources" property',
+          location: { path }
+        });
+      }
+    } finally {
+      doc.dispose();
+    }
+  }
+  detectSVGRemoteResources(root) {
+    try {
+      const fontFaceUris = root.find(".//svg:font-face-uri", {
+        svg: "http://www.w3.org/2000/svg"
+      });
+      for (const uri of fontFaceUris) {
+        const href = this.getAttribute(uri, "xlink:href") ?? this.getAttribute(uri, "href");
+        if (href && (href.startsWith("http://") || href.startsWith("https://"))) {
+          return true;
+        }
+      }
+    } catch {
+    }
+    try {
+      const images = root.find(".//svg:image", { svg: "http://www.w3.org/2000/svg" });
+      for (const img of images) {
+        const href = this.getAttribute(img, "xlink:href") ?? this.getAttribute(img, "href");
+        if (href && (href.startsWith("http://") || href.startsWith("https://"))) {
+          return true;
+        }
+      }
+    } catch {
+    }
+    try {
+      const styles = root.find(".//svg:style", { svg: "http://www.w3.org/2000/svg" });
+      for (const style of styles) {
+        const cssContent = style.content;
+        if (this.cssContainsRemoteUrl(cssContent)) {
+          return true;
+        }
+      }
+    } catch {
+    }
+    return false;
+  }
   validateCSSDocument(context, path, opfDir, refValidator) {
     const cssData = context.files.get(path);
     if (!cssData) {
@@ -1937,6 +2006,13 @@ var ContentValidator = class {
             location: { path }
           });
         }
+        if (!hasScripts && manifestItem?.properties?.includes("scripted")) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_015,
+            message: 'The property "scripted" should not be declared in the OPF file',
+            location: { path }
+          });
+        }
         const hasMathML = this.detectMathML(context, path, root);
         if (hasMathML && !manifestItem?.properties?.includes("mathml")) {
           pushMessage(context.messages, {
@@ -1953,6 +2029,21 @@ var ContentValidator = class {
             location: { path }
           });
         }
+        if (!hasSVG && manifestItem?.properties?.includes("svg")) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_015,
+            message: 'The property "svg" should not be declared in the OPF file',
+            location: { path }
+          });
+        }
+        const hasSwitch = this.detectSwitch(root);
+        if (hasSwitch && !manifestItem?.properties?.includes("switch")) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_014,
+            message: 'Content document contains epub:switch but manifest item is missing "switch" property',
+            location: { path }
+          });
+        }
         const hasRemoteResources = this.detectRemoteResources(context, path, root);
         if (hasRemoteResources && !manifestItem?.properties?.includes("remote-resources")) {
           pushMessage(context.messages, {
@@ -1961,6 +2052,13 @@ var ContentValidator = class {
             location: { path }
           });
         }
+        if (!hasRemoteResources && manifestItem?.properties?.includes("remote-resources")) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_018,
+            message: 'The "remote-resources" property was declared in the Package Document, but no reference to remote resources has been found',
+            location: { path }
+          });
+        }
       }
       this.checkDiscouragedElements(context, path, root);
       this.checkAccessibility(context, path, root);
@@ -2017,8 +2115,9 @@ var ContentValidator = class {
       });
     }
   }
-  checkNavDocument(context, path, doc, root) {
-    const navElements = root.find(".//html:nav", { html: "http://www.w3.org/1999/xhtml" });
+  checkNavDocument(context, path, _doc, root) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const navElements = root.find(".//html:nav", HTML_NS);
     if (navElements.length === 0) {
       pushMessage(context.messages, {
         id: MessageId.NAV_001,
@@ -2027,18 +2126,25 @@ var ContentValidator = class {
       });
       return;
     }
-    let tocNav;
-    let tocEpubTypeValue = "";
-    for (const nav of navElements) {
-      if (!("attrs" in nav)) continue;
+    const getNavTypes = (nav) => {
+      if (!("attrs" in nav)) return [];
       const epubTypeAttr = nav.attrs.find(
         (attr) => attr.name === "type" && attr.prefix === "epub" && attr.namespaceUri === "http://www.idpf.org/2007/ops"
       );
-      if (epubTypeAttr?.value.includes("toc")) {
+      return epubTypeAttr ? epubTypeAttr.value.trim().split(/\s+/) : [];
+    };
+    let tocNav;
+    let tocEpubTypeValue = "";
+    let pageListCount = 0;
+    let landmarksCount = 0;
+    for (const nav of navElements) {
+      const types = getNavTypes(nav);
+      if (types.includes("toc") && !tocNav) {
         tocNav = nav;
-        tocEpubTypeValue = epubTypeAttr.value;
-        break;
+        tocEpubTypeValue = types.join(" ");
       }
+      if (types.includes("page-list")) pageListCount++;
+      if (types.includes("landmarks")) landmarksCount++;
     }
     if (!tocNav) {
       pushMessage(context.messages, {
@@ -2048,7 +2154,7 @@ var ContentValidator = class {
       });
       return;
     }
-    const ol = tocNav.get(".//html:ol", { html: "http://www.w3.org/1999/xhtml" });
+    const ol = tocNav.get(".//html:ol", HTML_NS);
     if (!ol) {
       pushMessage(context.messages, {
         id: MessageId.NAV_002,
@@ -2056,8 +2162,218 @@ var ContentValidator = class {
         location: { path }
       });
     }
+    if (pageListCount > 1) {
+      pushMessage(context.messages, {
+        id: MessageId.RSC_005,
+        message: 'Multiple occurrences of the "page-list" nav element',
+        location: { path }
+      });
+    }
+    if (landmarksCount > 1) {
+      pushMessage(context.messages, {
+        id: MessageId.RSC_005,
+        message: 'Multiple occurrences of the "landmarks" nav element',
+        location: { path }
+      });
+    }
+    for (const nav of navElements) {
+      const navElem = nav;
+      const types = getNavTypes(navElem);
+      if (types.length === 0) continue;
+      const isStandard = types.includes("toc") || types.includes("page-list") || types.includes("landmarks");
+      if (!isStandard) {
+        this.checkNavFirstChildHeading(context, path, navElem);
+      }
+      if (types.includes("landmarks")) {
+        this.checkNavLandmarks(context, path, navElem);
+      }
+      this.checkNavLabels(context, path, navElem);
+      this.checkNavContentModel(context, path, navElem);
+    }
+    this.checkNavHeadingContent(context, path, root);
+    this.checkNavHiddenAttribute(context, path, root);
     this.checkNavRemoteLinks(context, path, root, tocEpubTypeValue);
   }
+  checkNavFirstChildHeading(context, path, navElem) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const headingTags = /* @__PURE__ */ new Set(["h1", "h2", "h3", "h4", "h5", "h6"]);
+    const children = navElem.find("./html:*", HTML_NS);
+    if (children.length === 0) return;
+    const firstChild = children[0];
+    const localName = firstChild.name.split(":").pop() ?? firstChild.name;
+    if (!headingTags.has(localName)) {
+      pushMessage(context.messages, {
+        id: MessageId.RSC_005,
+        message: 'nav elements other than "toc", "page-list" and "landmarks" must have a heading as their first child',
+        location: { path }
+      });
+    }
+  }
+  checkNavLandmarks(context, path, navElem) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const EPUB_NS = "http://www.idpf.org/2007/ops";
+    const anchors = navElem.find(".//html:ol//html:a", HTML_NS);
+    const seenLandmarks = [];
+    for (const anchor of anchors) {
+      const aElem = anchor;
+      const epubTypeAttr = "attrs" in aElem ? aElem.attrs.find((attr) => attr.name === "type" && attr.namespaceUri === EPUB_NS) : void 0;
+      if (!epubTypeAttr) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: 'Missing epub:type attribute on anchor inside "landmarks" nav element',
+          location: { path }
+        });
+        continue;
+      }
+      const href = this.getAttribute(aElem, "href");
+      const typeTokens = epubTypeAttr.value.toLowerCase().trim().split(/\s+/);
+      const normalizedHref = (href ?? "").toLowerCase().trim();
+      for (const typeToken of typeTokens) {
+        const isDuplicate = seenLandmarks.some(
+          (seen) => seen.type === typeToken && seen.href === normalizedHref
+        );
+        if (isDuplicate) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: `Another landmark was found with the same epub:type and same reference to "${normalizedHref}"`,
+            location: { path }
+          });
+        }
+        seenLandmarks.push({ type: typeToken, href: normalizedHref });
+      }
+    }
+  }
+  checkNavLabels(context, path, navElem) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const anchors = navElem.find(".//html:ol//html:a", HTML_NS);
+    for (const anchor of anchors) {
+      if (!this.hasNavLabelContent(anchor)) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: "Anchors within nav elements must contain text",
+          location: { path }
+        });
+      }
+    }
+    const spans = navElem.find(".//html:ol//html:span", HTML_NS);
+    for (const span of spans) {
+      if (!this.hasNavLabelContent(span)) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: "Spans within nav elements must contain text",
+          location: { path }
+        });
+      }
+    }
+  }
+  hasNavLabelContent(element) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const textContent = element.content;
+    if (textContent && textContent.trim().length > 0) return true;
+    const imgs = element.find("./html:img[@alt]", HTML_NS);
+    for (const img of imgs) {
+      const alt = this.getAttribute(img, "alt");
+      if (alt && alt.trim().length > 0) return true;
+    }
+    const ariaLabel = this.getAttribute(element, "aria-label");
+    if (ariaLabel && ariaLabel.trim().length > 0) return true;
+    const ariaLabelElements = element.find(".//*[@aria-label]");
+    for (const el of ariaLabelElements) {
+      const label = this.getAttribute(el, "aria-label");
+      if (label && label.trim().length > 0) return true;
+    }
+    return false;
+  }
+  checkNavContentModel(context, path, navElem) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const headingTags = /* @__PURE__ */ new Set(["h1", "h2", "h3", "h4", "h5", "h6", "hgroup"]);
+    const navChildren = navElem.find("./html:*", HTML_NS);
+    for (const child of navChildren) {
+      const localName = child.name.split(":").pop() ?? child.name;
+      if (!headingTags.has(localName) && localName !== "ol") {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: `element "${localName}" not allowed here; expected element "h1", "h2", "h3", "h4", "h5", "h6", "hgroup" or "ol"`,
+          location: { path }
+        });
+      }
+    }
+    const olElements = navElem.find(".//html:ol", HTML_NS);
+    for (const ol of olElements) {
+      const liChildren = ol.find("./html:li", HTML_NS);
+      if (liChildren.length === 0) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: 'element "ol" incomplete; missing required element "li"',
+          location: { path }
+        });
+      }
+    }
+    const liElements = navElem.find(".//html:ol//html:li", HTML_NS);
+    for (const li of liElements) {
+      const liElem = li;
+      const hasOl = liElem.get("./html:ol", HTML_NS);
+      const hasAnchor = liElem.get("./html:a", HTML_NS);
+      const hasSpan = liElem.get("./html:span", HTML_NS);
+      if (!hasAnchor && !hasSpan) {
+        if (hasOl) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: 'element "ol" not allowed yet; expected element "a" or "span"',
+            location: { path }
+          });
+        } else {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: 'element "li" incomplete; missing required element "ol"',
+            location: { path }
+          });
+        }
+      } else if (hasSpan && !hasAnchor && !hasOl) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: 'element "li" incomplete; missing required element "ol"',
+          location: { path }
+        });
+      }
+    }
+  }
+  checkNavHeadingContent(context, path, root) {
+    const HTML_NS = { html: "http://www.w3.org/1999/xhtml" };
+    const headingSelectors = [
+      ".//html:h1",
+      ".//html:h2",
+      ".//html:h3",
+      ".//html:h4",
+      ".//html:h5",
+      ".//html:h6"
+    ];
+    for (const selector of headingSelectors) {
+      const headings = root.find(selector, HTML_NS);
+      for (const heading of headings) {
+        if (!this.hasNavLabelContent(heading)) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: "Heading elements must contain text",
+            location: { path }
+          });
+        }
+      }
+    }
+  }
+  checkNavHiddenAttribute(context, path, root) {
+    const hiddenElements = root.find(".//*[@hidden]");
+    for (const elem of hiddenElements) {
+      const hiddenValue = this.getAttribute(elem, "hidden");
+      if (hiddenValue !== null && hiddenValue !== "" && hiddenValue !== "hidden" && hiddenValue !== "until-found") {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: `value of attribute "hidden" is invalid; must be equal to "", "hidden" or "until-found"`,
+          location: { path }
+        });
+      }
+    }
+  }
   checkNavRemoteLinks(context, path, root, epubTypeValue) {
     const navTypes = epubTypeValue.split(/\s+/);
     const isToc = navTypes.includes("toc");
@@ -2129,6 +2445,10 @@ var ContentValidator = class {
     ]);
     return jsTypes.has(type.toLowerCase());
   }
+  detectSwitch(root) {
+    const switchElem = root.get(".//epub:switch", { epub: "http://www.idpf.org/2007/ops" });
+    return !!switchElem;
+  }
   detectMathML(_context, _path, root) {
     const mathMLElements = root.find(".//math:*", { math: "http://www.w3.org/1998/Math/MathML" });
     return mathMLElements.length > 0;
@@ -2190,8 +2510,19 @@ var ContentValidator = class {
         }
       }
     }
+    const styleElements = root.find(".//html:style", { html: "http://www.w3.org/1999/xhtml" });
+    for (const style of styleElements) {
+      const cssContent = style.content;
+      if (this.cssContainsRemoteUrl(cssContent)) {
+        return true;
+      }
+    }
     return false;
   }
+  cssContainsRemoteUrl(css) {
+    const urlRegex = /url\s*\(\s*["']?(https?:\/\/[^"')]+)["']?\s*\)/gi;
+    return urlRegex.test(css);
+  }
   checkDiscouragedElements(context, path, root) {
     for (const elemName of DISCOURAGED_ELEMENTS) {
       const element = root.get(`.//html:${elemName}`, { html: "http://www.w3.org/1999/xhtml" });
@@ -2455,9 +2786,9 @@ var ContentValidator = class {
     const docDir = path.includes("/") ? path.substring(0, path.lastIndexOf("/")) : "";
     const links = root.find(".//html:a[@href]", { html: "http://www.w3.org/1999/xhtml" });
     for (const link of links) {
-      const href = this.getAttribute(link, "href");
+      const href = this.getAttribute(link, "href")?.trim() ?? null;
       if (href === null) continue;
-      if (href.trim() === "") {
+      if (href === "") {
         pushMessage(context.messages, {
           id: MessageId.HTM_045,
           message: "Encountered empty href",
@@ -3056,7 +3387,7 @@ function toJSONReport(result) {
   );
 }
 var NCXValidator = class {
-  validate(context, ncxContent, ncxPath) {
+  validate(context, ncxContent, ncxPath, registry) {
     let doc = null;
     try {
       doc = XmlDocument.fromString(ncxContent);
@@ -3090,7 +3421,7 @@ var NCXValidator = class {
       }
       this.checkUid(context, root, ncxPath);
       this.checkNavMap(context, root, ncxPath);
-      this.checkContentSrc(context, root, ncxPath);
+      this.checkContentSrc(context, root, ncxPath, registry);
     } finally {
       doc.dispose();
     }
@@ -3126,7 +3457,7 @@ var NCXValidator = class {
       });
     }
   }
-  checkContentSrc(context, root, ncxPath) {
+  checkContentSrc(context, root, ncxPath, registry) {
     const contentElements = root.find(".//ncx:content[@src]", {
       ncx: "http://www.daisy.org/z3986/2005/ncx/"
     });
@@ -3135,7 +3466,9 @@ var NCXValidator = class {
       const srcAttr = contentElem.attr("src");
       const src = srcAttr?.value;
       if (!src) continue;
-      const srcBase = src.split("#")[0] ?? src;
+      const hashIdx = src.indexOf("#");
+      const srcBase = hashIdx >= 0 ? src.substring(0, hashIdx) : src;
+      const fragment = hashIdx >= 0 ? src.substring(hashIdx + 1) : "";
       let fullPath = srcBase;
       if (ncxDir) {
         if (srcBase.startsWith("/")) {
@@ -3160,6 +3493,15 @@ var NCXValidator = class {
           message: `NCX content src references missing file: ${src}`,
           location: { path: ncxPath, line }
         });
+      } else if (fragment && registry) {
+        if (!registry.hasID(fullPath, fragment)) {
+          const line = contentElem.line;
+          pushMessage(context.messages, {
+            id: MessageId.RSC_012,
+            message: `Fragment identifier is not defined`,
+            location: { path: ncxPath, line }
+          });
+        }
       }
     }
   }
@@ -3757,6 +4099,15 @@ function parseOPF(xml) {
   const guideSection = extractSection(xml, "guide");
   const guide = parseGuide(guideSection);
   const collections = parseCollections(xml);
+  const hasBindings = /<bindings[\s>]/.test(xml);
+  const xmlLangs = [];
+  const xmlLangRegex = /xml:lang=["']([^"']*?)["']/g;
+  let langMatch;
+  while ((langMatch = xmlLangRegex.exec(xml)) !== null) {
+    if (langMatch[1] !== void 0) {
+      xmlLangs.push(langMatch[1]);
+    }
+  }
   const result = {
     version,
     uniqueIdentifier,
@@ -3780,6 +4131,12 @@ function parseOPF(xml) {
   if (spineResult.pageProgressionDirection) {
     result.pageProgressionDirection = spineResult.pageProgressionDirection;
   }
+  if (hasBindings) {
+    result.hasBindings = true;
+  }
+  if (xmlLangs.length > 0) {
+    result.xmlLangs = xmlLangs;
+  }
   return result;
 }
 function normalizeVersion(versionStr) {
@@ -3904,6 +4261,9 @@ function parseLinkElements(metadataXml) {
       if (attrs.id) {
         element.id = attrs.id;
       }
+      if (attrs.hreflang !== void 0) {
+        element.hreflang = attrs.hreflang;
+      }
       elements.push(element);
     }
   }
@@ -4050,6 +4410,7 @@ var ITEM_PROPERTIES = /* @__PURE__ */ new Set([
   "svg",
   "switch"
 ]);
+var LINK_PROPERTIES = /* @__PURE__ */ new Set(["onix", "marc21xml-record", "mods-record", "xmp-record"]);
 var SPINE_PROPERTIES = /* @__PURE__ */ new Set([
   "page-spread-left",
   "page-spread-right",
@@ -4117,6 +4478,31 @@ var OPFValidator = class {
     }
     if (this.packageDoc.version.startsWith("3.")) {
       this.validateCollections(context, opfPath);
+      if (this.packageDoc.hasBindings) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_017,
+          message: "Use of the bindings element is deprecated",
+          location: { path: opfPath }
+        });
+      }
+    }
+    if (this.packageDoc.xmlLangs) {
+      for (const lang of this.packageDoc.xmlLangs) {
+        if (lang === "") continue;
+        if (lang !== lang.trim()) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_092,
+            message: `Language tag "${lang}" is not well-formed`,
+            location: { path: opfPath }
+          });
+        } else if (!isValidLanguageTag(lang)) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_092,
+            message: `Language tag "${lang}" is not well-formed`,
+            location: { path: opfPath }
+          });
+        }
+      }
     }
   }
   /**
@@ -4155,6 +4541,11 @@ var OPFValidator = class {
         (dc) => dc.name === "identifier" && dc.id === refId
       );
       if (!matchingDc) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: `package element unique-identifier attribute does not resolve to a dc:identifier element (given reference was "${refId}")`,
+          location: { path: opfPath }
+        });
         pushMessage(context.messages, {
           id: MessageId.OPF_030,
           message: `unique-identifier "${refId}" does not reference an existing dc:identifier`,
@@ -4220,6 +4611,19 @@ var OPFValidator = class {
           });
         }
       }
+      if (dc.name === "identifier" && dc.value) {
+        const val = dc.value.trim();
+        if (val.startsWith("urn:uuid:")) {
+          const uuid = val.substring(9);
+          if (!/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(uuid)) {
+            pushMessage(context.messages, {
+              id: MessageId.OPF_085,
+              message: `Invalid UUID value "${uuid}"`,
+              location: { path: opfPath }
+            });
+          }
+        }
+      }
       if (dc.name === "creator" && dc.attributes) {
         const opfRole = dc.attributes["opf:role"];
         if (opfRole?.startsWith("marc:")) {
@@ -4329,22 +4733,143 @@ var OPFValidator = class {
         }
       }
     }
+    if (this.packageDoc.version !== "2.0") {
+      for (const meta of this.packageDoc.metaElements) {
+        if (meta.property && /\s/.test(meta.property.trim())) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_025,
+            message: `Property value must be a single value, not a list: "${meta.property}"`,
+            location: { path: opfPath }
+          });
+        }
+        if (meta.scheme && /\s/.test(meta.scheme.trim())) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_025,
+            message: `Scheme value must be a single value, not a list: "${meta.scheme}"`,
+            location: { path: opfPath }
+          });
+        }
+        if (meta.property && !/\s/.test(meta.property.trim())) {
+          const prop = meta.property.trim();
+          if (prop.includes(":") && /:\s*$/.test(prop)) {
+            pushMessage(context.messages, {
+              id: MessageId.OPF_026,
+              message: `Malformed property name: "${prop}"`,
+              location: { path: opfPath }
+            });
+          }
+        }
+        if (meta.scheme) {
+          const scheme = meta.scheme.trim();
+          if (scheme && !scheme.includes(":")) {
+            pushMessage(context.messages, {
+              id: MessageId.OPF_027,
+              message: `Undefined property: "${scheme}"`,
+              location: { path: opfPath }
+            });
+          }
+        }
+      }
+    }
+    if (this.packageDoc.version !== "2.0") {
+      const allIds = /* @__PURE__ */ new Set();
+      for (const dc of dcElements) {
+        if (dc.id) allIds.add(dc.id);
+      }
+      for (const meta of this.packageDoc.metaElements) {
+        if (meta.id) allIds.add(meta.id);
+      }
+      for (const link of this.packageDoc.linkElements) {
+        if (link.id) allIds.add(link.id);
+      }
+      for (const item of this.packageDoc.manifest) {
+        allIds.add(item.id);
+      }
+      const seenGlobalIds = /* @__PURE__ */ new Set();
+      const allIdSources = [];
+      for (const dc of dcElements) {
+        if (dc.id) allIdSources.push({ id: dc.id, normalized: dc.id.trim() });
+      }
+      for (const meta of this.packageDoc.metaElements) {
+        if (meta.id) allIdSources.push({ id: meta.id, normalized: meta.id.trim() });
+      }
+      for (const link of this.packageDoc.linkElements) {
+        if (link.id) allIdSources.push({ id: link.id, normalized: link.id.trim() });
+      }
+      for (const item of this.packageDoc.manifest) {
+        allIdSources.push({ id: item.id, normalized: item.id.trim() });
+      }
+      for (const src of allIdSources) {
+        if (seenGlobalIds.has(src.normalized)) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: `Duplicate "${src.normalized}"`,
+            location: { path: opfPath }
+          });
+        }
+        seenGlobalIds.add(src.normalized);
+      }
+      for (const meta of this.packageDoc.metaElements) {
+        if (!meta.refines) continue;
+        const refines = meta.refines;
+        if (/^[a-zA-Z][a-zA-Z0-9+\-.]*:/.test(refines)) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: "@refines must be a relative URL",
+            location: { path: opfPath }
+          });
+          continue;
+        }
+        if (!refines.startsWith("#")) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_017,
+            message: `@refines should instead refer to "${refines}" using a fragment identifier pointing to its manifest item`,
+            location: { path: opfPath }
+          });
+          continue;
+        }
+        const targetId = refines.substring(1);
+        if (!allIds.has(targetId)) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: `@refines missing target id: "${targetId}"`,
+            location: { path: opfPath }
+          });
+        }
+      }
+      this.detectRefinesCycles(context, opfPath);
+    }
     if (this.packageDoc.version !== "2.0") {
       const modifiedMeta = this.packageDoc.metaElements.find(
         (meta) => meta.property === "dcterms:modified"
       );
       if (!modifiedMeta) {
         pushMessage(context.messages, {
-          id: MessageId.OPF_054,
-          message: "EPUB 3 metadata must include a dcterms:modified meta element",
+          id: MessageId.RSC_005,
+          message: "package dcterms:modified meta element must occur exactly once",
           location: { path: opfPath }
         });
-      } else if (modifiedMeta.value && !isValidW3CDateFormat(modifiedMeta.value)) {
         pushMessage(context.messages, {
           id: MessageId.OPF_054,
-          message: `Invalid dcterms:modified date format "${modifiedMeta.value}"; must be W3C date format (ISO 8601)`,
+          message: "EPUB 3 metadata must include a dcterms:modified meta element",
           location: { path: opfPath }
         });
+      } else if (modifiedMeta.value) {
+        const strictModifiedPattern = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z$/;
+        if (!strictModifiedPattern.test(modifiedMeta.value.trim())) {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_005,
+            message: `dcterms:modified illegal syntax (expecting: "CCYY-MM-DDThh:mm:ssZ")`,
+            location: { path: opfPath }
+          });
+        }
+        if (!isValidW3CDateFormat(modifiedMeta.value)) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_054,
+            message: `Invalid dcterms:modified date format "${modifiedMeta.value}"; must be W3C date format (ISO 8601)`,
+            location: { path: opfPath }
+          });
+        }
       }
     }
   }
@@ -4355,11 +4880,43 @@ var OPFValidator = class {
     if (!this.packageDoc) return;
     const opfDir = opfPath.includes("/") ? opfPath.substring(0, opfPath.lastIndexOf("/")) : "";
     for (const link of this.packageDoc.linkElements) {
+      if (link.hreflang !== void 0 && link.hreflang !== "") {
+        const lang = link.hreflang;
+        if (lang !== lang.trim()) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_092,
+            message: `Language tag must not have leading or trailing whitespace: "${lang}"`,
+            location: { path: opfPath }
+          });
+        } else if (!isValidLanguageTag(lang)) {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_092,
+            message: `Invalid language tag: "${lang}"`,
+            location: { path: opfPath }
+          });
+        }
+      }
+      if (link.properties) {
+        for (const prop of link.properties) {
+          if (!LINK_PROPERTIES.has(prop) && !prop.includes(":")) {
+            pushMessage(context.messages, {
+              id: MessageId.OPF_027,
+              message: `Undefined property: "${prop}"`,
+              location: { path: opfPath }
+            });
+          }
+        }
+      }
       const href = link.href;
       const decodedHref = tryDecodeUriComponent(href);
       const basePath = href.includes("#") ? href.substring(0, href.indexOf("#")) : href;
       const basePathDecoded = decodedHref.includes("#") ? decodedHref.substring(0, decodedHref.indexOf("#")) : decodedHref;
       if (href.startsWith("#")) {
+        pushMessage(context.messages, {
+          id: MessageId.OPF_098,
+          message: `The "href" attribute must reference resources, not elements in the package document, but found URL "${href}".`,
+          location: { path: opfPath }
+        });
         continue;
       }
       const isRemote = /^[a-zA-Z][a-zA-Z0-9+\-.]*:/.test(href);
@@ -4476,6 +5033,11 @@ var OPFValidator = class {
         }
         if (item.properties.includes("nav")) {
           if (item.mediaType !== "application/xhtml+xml") {
+            pushMessage(context.messages, {
+              id: MessageId.RSC_005,
+              message: `The manifest item representing the Navigation Document must be of the "application/xhtml+xml" type (given type was "${item.mediaType}")`,
+              location: { path: opfPath }
+            });
             pushMessage(context.messages, {
               id: MessageId.OPF_012,
               message: `Item with "nav" property must be XHTML, found: ${item.mediaType}`,
@@ -4483,6 +5045,22 @@ var OPFValidator = class {
             });
           }
         }
+        if (item.properties.includes("cover-image")) {
+          if (!item.mediaType.startsWith("image/")) {
+            pushMessage(context.messages, {
+              id: MessageId.OPF_012,
+              message: `Item with "cover-image" property must be an image, found: ${item.mediaType}`,
+              location: { path: opfPath }
+            });
+          }
+        }
+      }
+      if (item.href.includes(" ")) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_020,
+          message: `"${item.href}" is not a valid URL (Illegal character in path segment: space is not allowed)`,
+          location: { path: opfPath }
+        });
       }
       if (this.packageDoc.version !== "2.0" && item.href.includes("#")) {
         pushMessage(context.messages, {
@@ -4492,6 +5070,13 @@ var OPFValidator = class {
         });
       }
       if (this.packageDoc.version !== "2.0" && (item.href.startsWith("http://") || item.href.startsWith("https://"))) {
+        if (!item.mediaType.startsWith("audio/") && !item.mediaType.startsWith("video/") && !item.mediaType.startsWith("font/") && item.mediaType !== "application/font-sfnt" && item.mediaType !== "application/font-woff" && item.mediaType !== "application/vnd.ms-opentype") {
+          pushMessage(context.messages, {
+            id: MessageId.RSC_006,
+            message: `Remote resource reference is not allowed in this context; resource "${item.href}" must be located in the EPUB container`,
+            location: { path: opfPath }
+          });
+        }
         const inSpine = this.packageDoc.spine.some((s) => s.idref === item.id);
         if (inSpine && !item.properties?.includes("remote-resources")) {
           pushMessage(context.messages, {
@@ -4503,13 +5088,29 @@ var OPFValidator = class {
       }
     }
     if (this.packageDoc.version !== "2.0") {
-      const hasNav = this.packageDoc.manifest.some((item) => item.properties?.includes("nav"));
-      if (!hasNav) {
+      const navItems = this.packageDoc.manifest.filter((item) => item.properties?.includes("nav"));
+      if (navItems.length === 0) {
         pushMessage(context.messages, {
           id: MessageId.RSC_005,
           message: 'Exactly one manifest item must declare the "nav" property',
           location: { path: opfPath }
         });
+      } else if (navItems.length > 1) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: `Exactly one manifest item must declare the "nav" property (number of "nav" items: ${String(navItems.length)}).`,
+          location: { path: opfPath }
+        });
+      }
+      const coverItems = this.packageDoc.manifest.filter(
+        (item) => item.properties?.includes("cover-image")
+      );
+      if (coverItems.length > 1) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: `Multiple occurrences of the "cover-image" property (number of "cover-image" items: ${String(coverItems.length)}).`,
+          location: { path: opfPath }
+        });
       }
     }
   }
@@ -4541,6 +5142,17 @@ var OPFValidator = class {
         message: "EPUB 2 spine should have a toc attribute referencing the NCX",
         location: { path: opfPath }
       });
+    } else if (!ncxId && this.packageDoc.version !== "2.0") {
+      const hasNcxInManifest = this.packageDoc.manifest.some(
+        (item) => item.mediaType === "application/x-dtbncx+xml"
+      );
+      if (hasNcxInManifest) {
+        pushMessage(context.messages, {
+          id: MessageId.RSC_005,
+          message: "spine element toc attribute must be set when an NCX is included in the publication",
+          location: { path: opfPath }
+        });
+      }
     } else if (ncxId) {
       const ncxItem = this.manifestById.get(ncxId);
       if (!ncxItem) {
@@ -4650,21 +5262,74 @@ var OPFValidator = class {
       }
     }
   }
+  detectRefinesCycles(context, opfPath) {
+    if (!this.packageDoc) return;
+    const refinesGraph = /* @__PURE__ */ new Map();
+    for (const meta of this.packageDoc.metaElements) {
+      if (!meta.refines || !meta.id) continue;
+      if (!meta.refines.startsWith("#")) continue;
+      const targetId = meta.refines.substring(1);
+      const existing = refinesGraph.get(meta.id);
+      if (existing) {
+        existing.push(targetId);
+      } else {
+        refinesGraph.set(meta.id, [targetId]);
+      }
+    }
+    const visited = /* @__PURE__ */ new Set();
+    const inStack = /* @__PURE__ */ new Set();
+    const reportedCycles = /* @__PURE__ */ new Set();
+    const dfs = (node) => {
+      if (inStack.has(node)) return true;
+      if (visited.has(node)) return false;
+      visited.add(node);
+      inStack.add(node);
+      const neighbors = refinesGraph.get(node) ?? [];
+      for (const neighbor of neighbors) {
+        if (dfs(neighbor)) {
+          if (!reportedCycles.has(node)) {
+            reportedCycles.add(node);
+            pushMessage(context.messages, {
+              id: MessageId.OPF_065,
+              message: `Invalid metadata declaration, probably due to a cycle in "refines" metadata.`,
+              location: { path: opfPath }
+            });
+          }
+          return true;
+        }
+      }
+      inStack.delete(node);
+      return false;
+    };
+    for (const node of refinesGraph.keys()) {
+      dfs(node);
+    }
+  }
   validateCollections(context, opfPath) {
     if (!this.packageDoc) return;
     const collections = this.packageDoc.collections;
     if (collections.length === 0) {
       return;
     }
-    const validRoles = /* @__PURE__ */ new Set(["dictionary", "index", "preview", "recordings"]);
     for (const collection of collections) {
-      if (!validRoles.has(collection.role)) {
+      if (collection.role === "manifest") {
         pushMessage(context.messages, {
-          id: MessageId.OPF_071,
-          message: `Unknown collection role: "${collection.role}"`,
+          id: MessageId.RSC_005,
+          message: 'Collection with role "manifest" must be a child of another collection',
           location: { path: opfPath }
         });
       }
+      if (collection.role.includes(":")) {
+        try {
+          new URL(collection.role);
+        } catch {
+          pushMessage(context.messages, {
+            id: MessageId.OPF_070,
+            message: `Invalid collection role URL: "${collection.role}"`,
+            location: { path: opfPath }
+          });
+        }
+      }
       if (collection.role === "dictionary") {
         if (!collection.name || collection.name.trim() === "") {
           pushMessage(context.messages, {
@@ -4952,7 +5617,9 @@ function hasParentDirectoryReference(url) {
 function isMalformedURL(url) {
   if (!url) return true;
   try {
-    if (/[\s<>]/.test(url)) return true;
+    const trimmed = url.trim();
+    if (!trimmed) return true;
+    if (/[\s<>]/.test(trimmed)) return true;
     return false;
   } catch {
     return true;
@@ -4996,7 +5663,7 @@ var ReferenceValidator = class {
    * Validate a single reference
    */
   validateReference(context, reference) {
-    const url = reference.url;
+    const url = reference.url.trim();
     if (isMalformedURL(url)) {
       pushMessage(context.messages, {
         id: MessageId.RSC_020,
@@ -5604,8 +6271,8 @@ var EpubCheck = class _EpubCheck {
       }
       const contentValidator = new ContentValidator();
       contentValidator.validate(context, registry, refValidator);
-      if (context.version === "2.0" && context.packageDocument) {
-        this.validateNCX(context);
+      if (context.packageDocument) {
+        this.validateNCX(context, registry);
       }
       refValidator.validate(context);
       const schemaValidator = new SchemaValidator(context);
@@ -5646,9 +6313,9 @@ var EpubCheck = class _EpubCheck {
     return this.options.version;
   }
   /**
-   * Validate NCX navigation document (EPUB 2.0)
+   * Validate NCX navigation document (EPUB 2 always, EPUB 3 when NCX present)
    */
-  validateNCX(context) {
+  validateNCX(context, registry) {
     if (!context.packageDocument) {
       return;
     }
@@ -5669,7 +6336,7 @@ var EpubCheck = class _EpubCheck {
     }
     const ncxContent = new TextDecoder().decode(ncxData);
     const ncxValidator = new NCXValidator();
-    ncxValidator.validate(context, ncxContent, ncxPath);
+    ncxValidator.validate(context, ncxContent, ncxPath, registry);
     if (context.ncxUid && context.packageDocument.uniqueIdentifier) {
       const uniqueIdRef = context.packageDocument.uniqueIdentifier;
       const matchingIdentifier = context.packageDocument.dcElements.find(