@hyperframes/lint 0.8.50 → 0.8.53-desktop.0

package/dist/browser.js

@@ -1538,6 +1538,7 @@ function findRuntimeMediaSrcMutationFindings(ctx) {
 }
 var mediaRules = [
   findNestedMediaStartBasisFindings,
+  findSpeedRampOnNonMediaFindings,
   // duplicate_media_id + duplicate_media_discovery_risk
   ({ tags }) => {
     const findings = [];
@@ -1959,6 +1960,18 @@ function findVolumeDoubleAutomationFindings(ctx) {
   }
   return findings;
 }
+function findSpeedRampOnNonMediaFindings(ctx) {
+  return ctx.tags.filter((tag) => tag.name !== "video" && tag.name !== "audio").filter(
+    (tag) => /"target"\s*:\s*"rate"/.test(readDecodedAttr(tag.raw, "data-automation") ?? "")
+  ).map((tag) => ({
+    code: "speed_ramp_on_non_media",
+    severity: "warning",
+    message: `<${tag.name}> has a speed-ramp lane, but only <video> and <audio> clips can be retimed. The lane does nothing here.`,
+    elementId: readAttr(tag.raw, "id") || void 0,
+    fixHint: "Move the rate lane onto the <video> or <audio> clip, or retime an animation with a GSAP timeline instead.",
+    snippet: truncateSnippet(tag.raw)
+  }));
+}
 function findCarveUngroupedSourcesFindings(ctx) {
   const groupIds = new Set(
     ctx.tags.filter((tag) => tag.name === "hf-audio-group").map((tag) => readAttr(tag.raw, "id"))
@@ -2065,6 +2078,11 @@ function findAudioGroupCarveAttrFindings(ctx) {
 // src/rules/composition.ts
 import { COMPOSITION_VARIABLE_TYPES, isSafeMediaUrl } from "@hyperframes/parsers/composition";
 import { COMPOSITION_ATTRIBUTES, readClipTiming } from "@hyperframes/parsers/composition-contract";
+import { resolveCompositionDuration } from "@hyperframes/parsers/composition-duration";
+import {
+  readAuthoredDurationSeconds,
+  resolveMediaDuration
+} from "@hyperframes/parsers/media-duration";
 var MAX_COMPOSITION_LINES = 300;
 var MAX_TIMED_ELEMENTS_PER_TRACK = 3;
 var TRACK_DENSITY_EXEMPT_TAGS = /* @__PURE__ */ new Set(["audio", "script", "style", "video"]);
@@ -2840,6 +2858,18 @@ ${allInlineStyles}`.replace(/\/\*[\s\S]*?\*\//g, "");
     const hasInfiniteCssAnimation = /\banimation(?:-iteration-count)?\s*:[^;{}]*(?<![\w-])infinite(?![\w-])/.test(combinedCss);
     const hasAnyNonGsapSignal = usesLottie || usesThree || usesWaapi || hasCssAnimationName;
     if (!hasAnyNonGsapSignal) {
+      const derived = deriveDurationFromClips(tags, rootTag);
+      if (derived.source === "derived") {
+        return [
+          {
+            code: "root_composition_duration_derived",
+            severity: "warning",
+            message: `Root composition has no data-duration and no GSAP timeline, so its length is taken from its clips: at least ${derived.seconds}s` + (derived.pendingClips > 0 ? `, and ${derived.pendingClips} clip(s) whose length is only known at runtime` : "") + ".",
+            fixHint: 'Add data-duration="<seconds>" to the root element to set the length yourself.',
+            snippet: truncateSnippet(rootTag.raw)
+          }
+        ];
+      }
       return [
         {
           code: "root_composition_missing_duration_source",
@@ -2924,6 +2954,35 @@ ${allInlineStyles}`.replace(/\/\*[\s\S]*?\*\//g, "");
     ];
   }
 ];
+var CLIP_MEDIA_TAGS = /* @__PURE__ */ new Set(["img", "video", "audio"]);
+function deriveDurationFromClips(tags, rootTag) {
+  const clipEnds = [];
+  for (const tag of tags) {
+    if (tag === rootTag) continue;
+    const startRaw = readAttr(tag.raw, "data-start");
+    if (startRaw === null) continue;
+    const start = Number(startRaw);
+    if (!Number.isFinite(start)) {
+      clipEnds.push(null);
+      continue;
+    }
+    const getAttr = (name) => readAttr(tag.raw, name);
+    const authored = readAuthoredDurationSeconds(getAttr, start);
+    if (CLIP_MEDIA_TAGS.has(tag.name)) {
+      const { seconds } = resolveMediaDuration({
+        tag: tag.name,
+        authoredDurationSeconds: authored,
+        sourceDurationSeconds: null,
+        mediaStartSeconds: 0,
+        playbackRate: 1
+      });
+      clipEnds.push(seconds === null ? null : start + seconds);
+    } else if (authored !== null && authored > 0) {
+      clipEnds.push(start + authored);
+    }
+  }
+  return resolveCompositionDuration({ authoredDurationSeconds: null, clipEndsSeconds: clipEnds });
+}
 
 // src/rules/gsap.ts
 async function loadParseGsapScript() {
@@ -5355,7 +5414,7 @@ var OPAQUE_TAGS = /* @__PURE__ */ new Set([
   "object"
 ]);
 var NON_LAYOUT_TAGS = /* @__PURE__ */ new Set(["style", "script", "template", "noscript"]);
-var MEDIA_TAGS = /* @__PURE__ */ new Set(["img", "video", "audio"]);
+var MEDIA_TAGS = /* @__PURE__ */ new Set(["video", "audio", "img"]);
 var NODE_ATTRS = [
   "id",
   "class",
@@ -5421,19 +5480,15 @@ function nestedStructureFindings(rows, severity) {
   });
 }
 function missingDurationFindings(rows, severity) {
-  const intendedClip = (row) => !MEDIA_TAGS.has(row.tag) || row.attrs["data-start"] !== void 0 || row.attrs["data-track-index"] !== void 0;
   return rows.filter(
-    (row) => row.attrs["data-duration"] === void 0 && row.attrs["data-end"] === void 0 && !isSubCompositionHost(row) && intendedClip(row)
-  ).map((row) => {
-    const isMedia = MEDIA_TAGS.has(row.tag);
-    return {
-      code: isMedia ? "media_missing_duration" : "timeline_element_missing_timing",
-      severity,
-      message: isMedia ? `${describe(row)} is media on the timeline without data-duration, so its clip has no length.` : `${describe(row)} is a timeline element without data-duration, so the timeline cannot draw where it ends.`,
-      elementId: row.attrs.id,
-      fixHint: `Add data-duration (in seconds) to ${describe(row)}.`
-    };
-  });
+    (row) => !MEDIA_TAGS.has(row.tag) && row.attrs["data-duration"] === void 0 && row.attrs["data-end"] === void 0 && !isSubCompositionHost(row)
+  ).map((row) => ({
+    code: "timeline_element_missing_timing",
+    severity,
+    message: `${describe(row)} is a timeline element without data-duration, so the timeline cannot draw where it ends.`,
+    elementId: row.attrs.id,
+    fixHint: `Add data-duration (in seconds) to ${describe(row)}.`
+  }));
 }
 function captionFindings(rows, severity) {
   const captionRows = rows.filter((row) => trackKindOf(row).kind === "captions");