@hyperframes/lint 0.8.37 → 0.8.39

package/dist/index.js

@@ -14,12 +14,24 @@ function parseHtmlStructure(source) {
   const blocks = { script: [], style: [] };
   const openTagsByName = /* @__PURE__ */ new Map();
   const openBlocks = [];
+  let explicitOpenTag = null;
   const parser = new Parser(
     {
-      onopentag(name) {
-        const index = parser.startIndex;
+      onopentagname(name) {
+        let tokenStart = parser.endIndex - 1;
+        while (tokenStart >= parser.startIndex && !/[\t\n\f\r />]/.test(source.charAt(tokenStart))) {
+          tokenStart -= 1;
+        }
+        const index = source.indexOf("<", tokenStart + 1);
+        explicitOpenTag = index >= 0 && index < parser.endIndex && source.slice(index + 1, parser.endIndex).toLowerCase() === name ? { index, nameEnd: parser.endIndex } : null;
+      },
+      onopentag(name, _attrs, isImplied) {
+        const origin = !isImplied ? explicitOpenTag : null;
+        const index = origin?.index ?? parser.startIndex;
+        explicitOpenTag = null;
         const raw = source.slice(index, parser.endIndex + 1);
-        const attrs = raw.slice(name.length + 1, -1).replace(/\s*\/$/, "");
+        const rawAttrs = origin ? source.slice(origin.nameEnd, parser.endIndex) : raw.slice(name.length + 1, -1);
+        const attrs = rawAttrs.replace(/\s*\/$/, "");
         const tag = { raw, name, attrs, index };
         tags.push(tag);
         const sameNameStack = openTagsByName.get(name) ?? [];
@@ -2843,10 +2855,6 @@ function targetHasNoStableIdentity(selector, identity) {
   if (identity) return false;
   return selector === UNRESOLVED_TARGET || selector === "dwell/hold" || selector.startsWith("proxy \u2192 ");
 }
-function readRegisteredTimelineCompositionId(script) {
-  const match = script.match(WINDOW_TIMELINE_ASSIGN_PATTERN);
-  return match?.[1] || match?.[2] || null;
-}
 function unwrapRaw(value) {
   if (typeof value === "number") return value;
   if (typeof value !== "string") return void 0;
@@ -3014,21 +3022,6 @@ function makesOverlayVisible(win) {
   if (win.method === "from" && isHiddenGsapState(win.propertyValues)) return true;
   return isVisibleGsapState(win.propertyValues);
 }
-function isSceneBoundaryExit(win) {
-  if (win.end <= win.position) return false;
-  if (win.method !== "to" && win.method !== "fromTo") return false;
-  return isHiddenGsapState(win.propertyValues);
-}
-function isHardKillSet(win, selector, boundary) {
-  return win.method === "set" && win.targetSelector === selector && Math.abs(win.position - boundary) <= SCENE_BOUNDARY_EPSILON_SECONDS && isHiddenGsapState(win.propertyValues);
-}
-function hiddenStateLiteral(values) {
-  if (zeroValue(values.autoAlpha)) return "{ autoAlpha: 0 }";
-  if (zeroValue(values.opacity)) return "{ opacity: 0 }";
-  if (stringValue(values.visibility)?.toLowerCase() === "hidden") return '{ visibility: "hidden" }';
-  if (stringValue(values.display)?.toLowerCase() === "none") return '{ display: "none" }';
-  return "{ opacity: 0 }";
-}
 function findTagEnd(source, tag) {
   const escapedTagName = tag.name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
   const pattern = new RegExp(`<\\/?${escapedTagName}\\b[^>]*>`, "gi");
@@ -3045,56 +3038,11 @@ function findTagEnd(source, tag) {
   }
   return source.length;
 }
-function collectCompositionRanges(source, tags) {
-  return tags.map((tag) => {
-    const id = readDecodedAttr(tag.raw, "data-composition-id");
-    if (!id) return null;
-    return {
-      id,
-      start: tag.index,
-      end: findTagEnd(source, tag)
-    };
-  }).filter((range) => range !== null);
-}
-function findContainingCompositionId(tag, ranges) {
-  let match = null;
-  for (const range of ranges) {
-    if (tag.index < range.start || tag.index >= range.end) continue;
-    if (!match || range.start >= match.start) match = range;
-  }
-  return match?.id || null;
-}
 function getClipTagClasses(tag) {
   const classAttr = readAttr(tag.raw, "class") || "";
   const classes = classAttr.split(/\s+/).filter(Boolean);
   return classes.includes("clip") ? { classAttr, classes } : null;
 }
-function collectClipStartBoundariesByComposition(source, tags) {
-  const ranges = collectCompositionRanges(source, tags);
-  const boundaries = /* @__PURE__ */ new Map();
-  for (const tag of tags) {
-    if (!getClipTagClasses(tag)) continue;
-    const compositionId = findContainingCompositionId(tag, ranges);
-    if (!compositionId) continue;
-    const start = numberValue(readAttr(tag.raw, "data-start") ?? void 0);
-    if (start == null || start <= 0) continue;
-    const compositionBoundaries = boundaries.get(compositionId) ?? /* @__PURE__ */ new Set();
-    compositionBoundaries.add(start);
-    boundaries.set(compositionId, compositionBoundaries);
-  }
-  return new Map(
-    [...boundaries.entries()].map(([compositionId, values]) => [
-      compositionId,
-      [...values].sort((a, b) => a - b)
-    ])
-  );
-}
-function findMatchingSceneBoundary(time, boundaries) {
-  for (const boundary of boundaries) {
-    if (Math.abs(time - boundary) <= SCENE_BOUNDARY_EPSILON_SECONDS) return boundary;
-  }
-  return null;
-}
 function readStyleProperty(style, property) {
   const escapedProperty = property.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
   const match = style.match(new RegExp(`(?:^|;)\\s*${escapedProperty}\\s*:\\s*([^;]+)`, "i"));
@@ -3593,7 +3541,7 @@ var gsapRules = [
   },
   // overlapping_gsap_tweens + gsap_animates_clip_element
   // fallow-ignore-next-line complexity
-  async ({ source, tags, scripts, styles, rootCompositionId }) => {
+  async ({ tags, scripts, styles }) => {
     const findings = [];
     const authoredHiddenSelectors = new Set(
       scripts.flatMap((script) => [...extractStandaloneHiddenSelectors(script.content)])
@@ -3614,13 +3562,10 @@ var gsapRules = [
         if (cls !== "clip") clipClasses.set(`.${cls}`, info);
       }
     }
-    const clipStartBoundariesByComposition = collectClipStartBoundariesByComposition(source, tags);
     const styleRules = collectSimpleStyleRules(styles);
     const reportedVisibleOverlayKeys = /* @__PURE__ */ new Set();
     for (const script of scripts) {
-      const localTimelineCompId = readRegisteredTimelineCompositionId(script.content);
       const gsapWindows = await cachedExtractGsapWindows(script.content);
-      const clipStartBoundaries = clipStartBoundariesByComposition.get(localTimelineCompId || rootCompositionId || "") ?? [];
       for (let i = 0; i < gsapWindows.length; i++) {
         const left = gsapWindows[i];
         if (!left) continue;
@@ -3682,28 +3627,6 @@ ${right.raw}`)
           snippet: truncateSnippet(fromToWindows.map((win) => win.raw).join("\n"))
         });
       }
-      if (clipStartBoundaries.length > 0) {
-        for (const win of gsapWindows) {
-          if (win.targetSelector === UNRESOLVED_TARGET) continue;
-          if (!isSceneBoundaryExit(win)) continue;
-          const boundary = findMatchingSceneBoundary(win.end, clipStartBoundaries);
-          if (boundary == null) continue;
-          const hasHardKill = gsapWindows.some(
-            (candidate) => isHardKillSet(candidate, win.targetSelector, boundary)
-          );
-          if (hasHardKill) continue;
-          const exitClipInfo = clipIds.get(win.targetSelector) || clipClasses.get(win.targetSelector);
-          const fixHint = exitClipInfo ? `"${win.targetSelector}" is a clip element \u2014 the framework already manages its visibility. Wrap the scene's content in an inner non-clip <div>, move the exit tween and the hard kill (\`tl.set("<inner-selector>", ${hiddenStateLiteral(win.propertyValues)}, ${boundary.toFixed(2)})\`) onto that wrapper instead.` : `Add \`tl.set("${win.targetSelector}", ${hiddenStateLiteral(win.propertyValues)}, ${boundary.toFixed(2)})\` after the exit tween.`;
-          findings.push({
-            code: "gsap_exit_missing_hard_kill",
-            severity: "error",
-            message: `GSAP exit on "${win.targetSelector}" ends at the ${boundary.toFixed(2)}s clip start boundary without a matching tl.set hard kill. Non-linear seeking can land after the fade and leave stale visibility state.`,
-            selector: win.targetSelector,
-            fixHint,
-            snippet: truncateSnippet(win.raw)
-          });
-        }
-      }
       for (const tag of tags) {
         const selectors = tagSimpleSelectors(tag);
         if (selectors.length === 0) continue;