@hyperframes/lint 0.8.166 → 0.8.168

package/dist/browser.js

@@ -2740,8 +2740,9 @@ function stylesheetCanvasZoomHits(styles, rootId, rootClasses) {
   const hits = [];
   for (const style of styles) {
     const ruleWithBody = /([^{}]+)\{([^{}]*)\}/g;
+    const content = stripCssComments(style.content);
     let match;
-    while ((match = ruleWithBody.exec(stripCssComments(style.content))) !== null) {
+    while ((match = ruleWithBody.exec(content)) !== null) {
       const hit = canvasZoomInRule(match[1] ?? "", match[2] ?? "", rootId, rootClasses);
       if (hit) hits.push(hit);
     }
@@ -3387,37 +3388,6 @@ var compositionRules = [
   // defaults aren't applied.
   // Checked on every element that declares: <html>, or the composition root.
   ({ tags }) => tags.filter((tag) => readJsonAttr(tag.raw, "data-composition-variables")).flatMap((tag) => variablesDeclarationFindings(tag, tags)),
-  // html_dir_attribute_breaks_render — valid non-LTR dir values on
-  // <html> renders correctly in preview/snapshot but produces a fully
-  // blank/black video from render, with no other lint/validate/inspect
-  // check catching it (output file size, far smaller than expected, is the
-  // only tell). Confirmed independently by two separate reports, both
-  // diagnosing the same exact trigger and the same fix: drop dir from
-  // <html>, keep lang, and scope `direction: rtl` to individual
-  // text-containing elements via CSS instead (text still bidi-shapes
-  // correctly). Advisory-only — this does not attempt to fix the render
-  // pipeline's own root cause (suspected to be a capture step that clips a
-  // fixed top-left-origin screenshot region, which RTL layout can shift the
-  // actual content away from), only surfaces the already-confirmed footgun
-  // before someone hits it blind.
-  ({ tags }) => {
-    const htmlTag = findHtmlTag(tags);
-    if (!htmlTag) return [];
-    const dir = readAttr(htmlTag.raw, "dir");
-    if (!dir) return [];
-    const normalizedDir = dir.toLowerCase();
-    if (normalizedDir !== "rtl" && normalizedDir !== "auto") return [];
-    const scopedDirection = normalizedDir === "auto" ? 'dir="auto"' : `direction: ${normalizedDir}`;
-    return [
-      {
-        code: "html_dir_attribute_breaks_render",
-        severity: "error",
-        message: `<html dir="${dir}"> renders correctly in preview/snapshot but produces a fully blank/black video from render. This is a confirmed, silent failure.`,
-        fixHint: `Remove dir="${dir}" from <html>. Keep lang, and scope ${scopedDirection} to individual text-containing elements instead. Text still shapes correctly via the browser's own bidi algorithm.`,
-        snippet: truncateSnippet(htmlTag.raw)
-      }
-    ];
-  },
   // subcomposition_blanks_before_host
   // Warns when a full-bleed sub-composition slot ends before the host composition
   // does, leaving the slot blank for the remainder (issue #1540). Scoped narrowly to
@@ -6243,7 +6213,8 @@ var NODE_ATTRS = [
   "data-track-index",
   "data-track-kind",
   "data-composition-id",
-  "data-composition-src"
+  "data-composition-src",
+  "data-hf-media-start-basis"
 ];
 function toNode(open) {
   const attrs = {};
@@ -6351,6 +6322,33 @@ function clipsPastRootFindings(root, rows) {
     ];
   });
 }
+function followsParentVisibility(clip) {
+  return clip.tag !== "audio" && clip.attrs["data-hf-media-start-basis"] !== "global";
+}
+function isTimed(timing) {
+  return timing.start !== null && timing.end !== null;
+}
+function neverVisibleNestedFindings(parent, severity) {
+  const window = readClipTiming2({ getAttribute: (name) => parent.attrs[name] ?? null });
+  return topLevelElements(parent).flatMap((clip) => {
+    const inner = isSubCompositionHost(clip) ? [] : neverVisibleNestedFindings(clip, severity);
+    const own = readClipTiming2({ getAttribute: (name) => clip.attrs[name] ?? null });
+    if (!followsParentVisibility(clip) || !isTimed(window) || !isTimed(own)) return inner;
+    if (own.end > window.start && own.start < window.end) return inner;
+    const shows = `${hundredths(window.start)}s to ${hundredths(window.end)}s`;
+    return [
+      {
+        code: "nested_clip_never_visible",
+        severity,
+        message: `${describe(clip)} plays from ${hundredths(own.start)}s to ${hundredths(own.end)}s, but it sits inside ${describe(parent)}, which only shows from ${shows}, so it never appears. A nested data-start counts from the composition's start, not from ${describe(parent)}.`,
+        elementId: clip.attrs.id,
+        fixHint: `Give ${describe(clip)} a data-start between ${shows} in composition time, or move it out of ${describe(parent)}.`,
+        snippet: truncateSnippet(clip.open.raw)
+      },
+      ...inner
+    ];
+  });
+}
 var structureRules = [
   (ctx) => {
     if (ctx.options.isSubComposition) return [];
@@ -6364,6 +6362,12 @@ var structureRules = [
       ...captionFindings(rows, severity),
       ...clipsPastRootFindings(root, rows)
     ];
+  },
+  (ctx) => {
+    const root = buildTree(ctx);
+    if (!root) return [];
+    const severity = ctx.options.host === "studio" ? "error" : "warning";
+    return topLevelElements(root).filter((row) => !isSubCompositionHost(row)).flatMap((row) => neverVisibleNestedFindings(row, severity));
   }
 ];
 

package/dist/index.js

@@ -2746,8 +2746,9 @@ function stylesheetCanvasZoomHits(styles, rootId, rootClasses) {
   const hits = [];
   for (const style of styles) {
     const ruleWithBody = /([^{}]+)\{([^{}]*)\}/g;
+    const content = stripCssComments(style.content);
     let match;
-    while ((match = ruleWithBody.exec(stripCssComments(style.content))) !== null) {
+    while ((match = ruleWithBody.exec(content)) !== null) {
       const hit = canvasZoomInRule(match[1] ?? "", match[2] ?? "", rootId, rootClasses);
       if (hit) hits.push(hit);
     }
@@ -3393,37 +3394,6 @@ var compositionRules = [
   // defaults aren't applied.
   // Checked on every element that declares: <html>, or the composition root.
   ({ tags }) => tags.filter((tag) => readJsonAttr(tag.raw, "data-composition-variables")).flatMap((tag) => variablesDeclarationFindings(tag, tags)),
-  // html_dir_attribute_breaks_render — valid non-LTR dir values on
-  // <html> renders correctly in preview/snapshot but produces a fully
-  // blank/black video from render, with no other lint/validate/inspect
-  // check catching it (output file size, far smaller than expected, is the
-  // only tell). Confirmed independently by two separate reports, both
-  // diagnosing the same exact trigger and the same fix: drop dir from
-  // <html>, keep lang, and scope `direction: rtl` to individual
-  // text-containing elements via CSS instead (text still bidi-shapes
-  // correctly). Advisory-only — this does not attempt to fix the render
-  // pipeline's own root cause (suspected to be a capture step that clips a
-  // fixed top-left-origin screenshot region, which RTL layout can shift the
-  // actual content away from), only surfaces the already-confirmed footgun
-  // before someone hits it blind.
-  ({ tags }) => {
-    const htmlTag = findHtmlTag(tags);
-    if (!htmlTag) return [];
-    const dir = readAttr(htmlTag.raw, "dir");
-    if (!dir) return [];
-    const normalizedDir = dir.toLowerCase();
-    if (normalizedDir !== "rtl" && normalizedDir !== "auto") return [];
-    const scopedDirection = normalizedDir === "auto" ? 'dir="auto"' : `direction: ${normalizedDir}`;
-    return [
-      {
-        code: "html_dir_attribute_breaks_render",
-        severity: "error",
-        message: `<html dir="${dir}"> renders correctly in preview/snapshot but produces a fully blank/black video from render. This is a confirmed, silent failure.`,
-        fixHint: `Remove dir="${dir}" from <html>. Keep lang, and scope ${scopedDirection} to individual text-containing elements instead. Text still shapes correctly via the browser's own bidi algorithm.`,
-        snippet: truncateSnippet(htmlTag.raw)
-      }
-    ];
-  },
   // subcomposition_blanks_before_host
   // Warns when a full-bleed sub-composition slot ends before the host composition
   // does, leaving the slot blank for the remainder (issue #1540). Scoped narrowly to
@@ -6249,7 +6219,8 @@ var NODE_ATTRS = [
   "data-track-index",
   "data-track-kind",
   "data-composition-id",
-  "data-composition-src"
+  "data-composition-src",
+  "data-hf-media-start-basis"
 ];
 function toNode(open) {
   const attrs = {};
@@ -6357,6 +6328,33 @@ function clipsPastRootFindings(root, rows) {
     ];
   });
 }
+function followsParentVisibility(clip) {
+  return clip.tag !== "audio" && clip.attrs["data-hf-media-start-basis"] !== "global";
+}
+function isTimed(timing) {
+  return timing.start !== null && timing.end !== null;
+}
+function neverVisibleNestedFindings(parent, severity) {
+  const window = readClipTiming2({ getAttribute: (name) => parent.attrs[name] ?? null });
+  return topLevelElements(parent).flatMap((clip) => {
+    const inner = isSubCompositionHost(clip) ? [] : neverVisibleNestedFindings(clip, severity);
+    const own = readClipTiming2({ getAttribute: (name) => clip.attrs[name] ?? null });
+    if (!followsParentVisibility(clip) || !isTimed(window) || !isTimed(own)) return inner;
+    if (own.end > window.start && own.start < window.end) return inner;
+    const shows = `${hundredths(window.start)}s to ${hundredths(window.end)}s`;
+    return [
+      {
+        code: "nested_clip_never_visible",
+        severity,
+        message: `${describe(clip)} plays from ${hundredths(own.start)}s to ${hundredths(own.end)}s, but it sits inside ${describe(parent)}, which only shows from ${shows}, so it never appears. A nested data-start counts from the composition's start, not from ${describe(parent)}.`,
+        elementId: clip.attrs.id,
+        fixHint: `Give ${describe(clip)} a data-start between ${shows} in composition time, or move it out of ${describe(parent)}.`,
+        snippet: truncateSnippet(clip.open.raw)
+      },
+      ...inner
+    ];
+  });
+}
 var structureRules = [
   (ctx) => {
     if (ctx.options.isSubComposition) return [];
@@ -6370,6 +6368,12 @@ var structureRules = [
       ...captionFindings(rows, severity),
       ...clipsPastRootFindings(root, rows)
     ];
+  },
+  (ctx) => {
+    const root = buildTree(ctx);
+    if (!root) return [];
+    const severity = ctx.options.host === "studio" ? "error" : "warning";
+    return topLevelElements(root).filter((row) => !isSubCompositionHost(row)).flatMap((row) => neverVisibleNestedFindings(row, severity));
   }
 ];
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperframes/lint",
-  "version": "0.8.166",
+  "version": "0.8.168",
   "repository": {
     "type": "git",
     "url": "https://github.com/heygen-com/hyperframes",
@@ -35,7 +35,7 @@
     "linkedom": "^0.18.12",
     "postcss": "^8.5.8",
     "postcss-selector-parser": "^7.1.4",
-    "@hyperframes/parsers": "0.8.166"
+    "@hyperframes/parsers": "0.8.168"
   },
   "devDependencies": {
     "@types/node": "^25.0.10",