@hyperframes/lint 0.8.81 → 0.8.83

package/dist/index.js

@@ -5822,6 +5822,9 @@ import {
   topLevelElements,
   trackKindOf
 } from "@hyperframes/parsers/top-level-elements";
+import { readClipTiming as readClipTiming2 } from "@hyperframes/parsers/composition-contract";
+import { readDataDurationSeconds } from "@hyperframes/parsers/media-duration";
+import { TIMING_TOLERANCE_SECONDS } from "@hyperframes/parsers/composition-duration";
 var INLINE_TEXT_TAGS = /* @__PURE__ */ new Set([
   "br",
   "wbr",
@@ -5950,6 +5953,27 @@ function captionFindings(rows, severity) {
   }
   return findings;
 }
+var hundredths = (seconds) => Math.round(seconds * 100) / 100;
+function clipsPastRootFindings(root, rows) {
+  if (root.attrs["data-composition-id"] === void 0) return [];
+  const rootDuration = readDataDurationSeconds((name) => root.attrs[name]);
+  if (rootDuration === null) return [];
+  const limit = hundredths(rootDuration);
+  return rows.flatMap((row) => {
+    const { start, end } = readClipTiming2({ getAttribute: (name) => row.attrs[name] ?? null });
+    if (start === null || end === null || end <= rootDuration + TIMING_TOLERANCE_SECONDS) return [];
+    return [
+      {
+        code: "clip_ends_past_root_duration",
+        severity: "warning",
+        message: `${describe(row)} runs from ${hundredths(start)}s to ${hundredths(end)}s, past the root composition's data-duration of ${limit}s, so it is cut off in previews, posters and renders.`,
+        elementId: row.attrs.id,
+        fixHint: `Extend the root data-duration to ${hundredths(end)}, or make ${describe(row)} end at or before ${limit}s.`,
+        snippet: truncateSnippet(row.open.raw)
+      }
+    ];
+  });
+}
 var structureRules = [
   (ctx) => {
     if (ctx.options.isSubComposition) return [];
@@ -5960,7 +5984,8 @@ var structureRules = [
     return [
       ...nestedStructureFindings(rows, severity),
       ...missingDurationFindings(rows, severity),
-      ...captionFindings(rows, severity)
+      ...captionFindings(rows, severity),
+      ...clipsPastRootFindings(root, rows)
     ];
   }
 ];
@@ -6378,6 +6403,7 @@ async function lintHevcPreviewCodec(candidates) {
 function parseSubCompHtml(html) {
   return parseHTML3(html).document;
 }
+var sourceFile = (projectDir, compSrcPath) => resolve(projectDir, compSrcPath ?? "index.html");
 function querySelectorAllIncludingTemplates(root, selector) {
   const matches = [...root.querySelectorAll(selector)];
   for (const template of root.querySelectorAll("template")) {
@@ -6529,20 +6555,20 @@ async function lintProject(projectDir, entryFile, hostOptions = {}) {
     ...await lintVideoMediaStartPastEof(projectDir, allHtmlSources),
     ...await lintHevcPreviewCodec(collectLocalVideoCandidates(projectDir, allHtmlSources))
   ];
-  if (projectFindings.length > 0) {
-    for (const finding of projectFindings) {
-      rootResult.findings.push(finding);
-      if (finding.severity === "error") {
-        rootResult.errorCount++;
-        rootResult.ok = false;
-        totalErrors++;
-      } else if (finding.severity === "warning") {
-        rootResult.warningCount++;
-        totalWarnings++;
-      } else {
-        rootResult.infoCount++;
-        totalInfos++;
-      }
+  for (const finding of projectFindings) {
+    const ownFile = finding.file && resolve(projectDir, finding.file);
+    const owner = results.find((entry) => resolve(projectDir, entry.file) === ownFile)?.result ?? rootResult;
+    owner.findings.push(finding);
+    if (finding.severity === "error") {
+      owner.errorCount++;
+      owner.ok = false;
+      totalErrors++;
+    } else if (finding.severity === "warning") {
+      owner.warningCount++;
+      totalWarnings++;
+    } else {
+      owner.infoCount++;
+      totalInfos++;
     }
   }
   return { results, totalErrors, totalWarnings, totalInfos };
@@ -6602,7 +6628,7 @@ function lintProjectAudioFiles(projectDir, htmlSources) {
 function lintAudioSrcNotFound(projectDir, htmlSources) {
   const findings = [];
   const audioSrcRe = mediaSrcTagRe("audio");
-  const missingSrcs = [];
+  const missingByFile = /* @__PURE__ */ new Map();
   for (const { html, compSrcPath } of htmlSources) {
     let match;
     while ((match = audioSrcRe.exec(html)) !== null) {
@@ -6611,15 +6637,17 @@ function lintAudioSrcNotFound(projectDir, htmlSources) {
       if (isUnresolvedAssetPlaceholder2(src)) continue;
       const rootRelative = compSrcPath ? rewriteAssetPath2(compSrcPath, src, (path) => existsSync2(join2(projectDir, path))) : src;
       if (!existsSync2(resolveProjectRelativeSrc(rootRelative, projectDir))) {
-        missingSrcs.push(src);
+        const file = sourceFile(projectDir, compSrcPath);
+        missingByFile.set(file, (missingByFile.get(file) ?? /* @__PURE__ */ new Set()).add(src));
       }
     }
   }
-  if (missingSrcs.length > 0) {
-    const unique = [...new Set(missingSrcs)];
+  for (const [file, srcs] of missingByFile) {
+    const unique = [...srcs];
     findings.push({
       code: "audio_src_not_found",
       severity: "error",
+      file,
       message: `<audio> element references file(s) not found in the project: ${unique.join(", ")}. The rendered video will be silent.`,
       fixHint: unique.length === 1 ? `Add the file "${unique[0]}" to the project directory, or update the src attribute to point to an existing file.` : `Add the missing files to the project directory, or update the src attributes to point to existing files.`
     });
@@ -6645,19 +6673,22 @@ function lintMissingLocalAsset(projectDir, htmlSources) {
       const resolvedAsset = resolveExistingLocalAsset2(projectDir, rootRelative);
       if (resolvedAsset) continue;
       const resolvedKey = resolve(projectDir, rootRelative);
-      let bucket = missingByTag.get(tagName);
+      const file = sourceFile(projectDir, compSrcPath);
+      const bucketKey = `${file}\0${tagName}`;
+      let bucket = missingByTag.get(bucketKey);
       if (!bucket) {
-        bucket = /* @__PURE__ */ new Map();
-        missingByTag.set(tagName, bucket);
+        bucket = { file, tagName, byResolved: /* @__PURE__ */ new Map() };
+        missingByTag.set(bucketKey, bucket);
       }
-      if (!bucket.has(resolvedKey)) bucket.set(resolvedKey, src);
+      if (!bucket.byResolved.has(resolvedKey)) bucket.byResolved.set(resolvedKey, src);
     }
   }
-  for (const [tagName, byResolved] of missingByTag) {
+  for (const { file, tagName, byResolved } of missingByTag.values()) {
     const unique = [...byResolved.values()];
     findings.push({
       code: "missing_local_asset",
       severity: "error",
+      file,
       message: `<${tagName}> element references local file(s) not found in the project: ${unique.join(", ")}. The renderer will silently skip these and produce a video with missing visuals.`,
       fixHint: unique.length === 1 ? `Add "${unique[0]}" to the project directory, or update the src attribute to point to an existing file. Common cause: captured asset filenames are unreliable (heygen-logo.svg often contains Google, nvidia-logo.svg may contain Autodesk, etc.). Open the contact sheets and verify the file actually exists at this path before referencing it.` : "Add the missing files to the project directory, or update the src attributes to point to existing files. Captured asset filenames are unreliable \u2014 verify against capture/contact-sheets/ and capture/extracted/asset-descriptions.md."
     });
@@ -6665,8 +6696,9 @@ function lintMissingLocalAsset(projectDir, htmlSources) {
   return findings;
 }
 function lintTextureMaskAssetNotFound(projectDir, htmlSources) {
-  const missing = /* @__PURE__ */ new Map();
+  const missingByFile = /* @__PURE__ */ new Map();
   for (const { html, compSrcPath } of htmlSources) {
+    const file = sourceFile(projectDir, compSrcPath);
     for (const cssSource of collectCssSources(projectDir, html, compSrcPath)) {
       let match;
       const pattern = new RegExp(MASK_IMAGE_URL_RE.source, MASK_IMAGE_URL_RE.flags);
@@ -6682,20 +6714,20 @@ function lintTextureMaskAssetNotFound(projectDir, htmlSources) {
           cssSource.rootRelativePath
         );
         if (candidates.some(existsSync2)) continue;
-        missing.set(url, candidates[0] ?? resolve(projectDir, url));
+        missingByFile.set(file, (missingByFile.get(file) ?? /* @__PURE__ */ new Set()).add(url));
       }
     }
   }
-  if (missing.size === 0) return [];
-  const urls = [...missing.keys()];
-  return [
-    {
+  return [...missingByFile].map(([file, found]) => {
+    const urls = [...found];
+    return {
       code: "texture_mask_asset_not_found",
       severity: "error",
+      file,
       message: `CSS mask-image references file(s) not found in the project: ${urls.join(", ")}.`,
       fixHint: urls.length === 1 ? `Add "${urls[0]}" to the project, or update the mask-image URL to point to an existing texture mask.` : "Add the missing texture mask files to the project, or update the mask-image URLs to point to existing files."
-    }
-  ];
+    };
+  });
 }
 function lintMultipleRootCompositions(projectDir) {
   const findings = [];