[2025-11-04] [LIVE] [General] Assorted Improvements

[angelux]

Fixes & Improvements

Refactoring & Performance (New)

• Modularize Vendor Logic: Centralized provider logic into a mapped structure, removing scattered regex and conditions.
• Modern API Requests: Replaced XMLHttpRequest with modern fetch API and async/await.
• Improve Observer Performance: Optimized element lookup in IntersectionObserver using a Map (O(1)) instead of array traversal (O(n)).
• Data Attribute Parsing: Improved safety and readability of boolean attribute parsing.

Features & Options

• Option to prevent thumbnail from loading
• Option to prevent play button from rendering

TypeScript Migration

• Refactor into TS
• Update rollup config for TS

Other Updates

Testing

• Updated and modernized browser.js tests.
• Fixed async environment configuration for tests (added Babel runtime).
• Added mocks for fetch and IntersectionObserver.

Maintenance

• Updated README
• Add new packages and script option
• Remove non-TS version file

Reference Links

For more details, you can check: https://app.clickup.com/t/86b44hm62

[Digznav]

The fixes so far look good! But I still have a few comments:

• Since you upgraded node to version 20, I strongly suggest to regenerate the lock file so it has the correct structure that's recommended for the version.

  rm -rf node_modules/ package-lock.json && npm install
  

• When I ran npm test I got the following error:

  Uncaught exception in test/browser.js
  
  test/browser.js:9
  
  8: const virtualConsole = new VirtualConsole();
  9: virtualConsole.sendTo(console);
  10:
  
  TypeError: virtualConsole.sendTo is not a function
  
  › Object.<anonymous> (test/browser.js:9:16)
  
  ✘ test/browser.js exited with a non-zero exit code: 1
  ✔ server › should expose lazyframe()
  ─
  
  1 test passed
  1 uncaught exception
  

  So I checked the jsdom documentation and it seems that the correct method is forwardTo(). But after I replace it, I got even more errors. Not sure if the tests worked before all the updates. Can you take a look?

• After I ran npm run build the dist/lazyframe.css file was modified, can you please commit the change?

[Digznav]

All good, just please resolve the merge conflicts!