straight_to_video 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7f9374ddbb488a573260822bdfc2b8a5a183f0c1f8c10e4e6428b7ae8c3e7f86
-  data.tar.gz: 3fb96670c80e91c46c0214d4153e2a28b96d3f5cddb368875568978bf2421912
+  metadata.gz: 57371915cd38c1ecac814d16a43244275216661f31b639a80022685672984e22
+  data.tar.gz: bb04ab7274edbb61031a270c8f650872b0b2338a7a1fc93a54948c8f3a3fcd04
 SHA512:
-  metadata.gz: 54b446d8179dd83c5604902b741c15c07b388b0fbada671527faf07441d11d73af5c1ba6729128897b43cf2232b7f9dc8f8271e0037362a94155736a28309089
-  data.tar.gz: bbe19415c4fa54b522fefbcf8aaaa208d72674aacf6e29db2830c06190076b45955b04a4e51ad75baa161abfe1ce1feb1a075970c439a863158b88459cfe26e1
+  metadata.gz: 92b3e2d474ba435ac6679c26755a9d62bb5388cd3fbb984fc6b31506287b4ced1be3a7c3621efdaf1812a9feae7f60287caa9e91e3601731c60c0a450e4d8824
+  data.tar.gz: 550b54b8fb004e7b6fea54282f00053fe47fd4bdefd20a5e87817c91495756adfe38c40d72888de883b0fbc049ad9615eb3f4f1783b05d01d92fc418356645cb

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.0.6
+
+- fix iOS Safari Stimulus controller hang by making `seeked` waits robust
+
 ## 0.0.5
 
 - unscrew up the extension in the importmap 🤦‍♂️

data/app/assets/javascripts/straight-to-video.js

@@ -1,4 +1,4 @@
-// straight-to-video@0.0.5 vendored by the straight_to_video gem
+// straight-to-video@0.0.6 vendored by the straight_to_video gem
 // straight-to-video - https://github.com/searlsco/straight-to-video
 
 // ----- External imports -----
@@ -149,6 +149,20 @@ async function waitForFrameReady (video, budgetMs) {
   })
 }
 
+async function seekOnce (video, time) {
+  if (!video) return
+  const t = Number.isFinite(time) ? time : 0
+  if (Math.abs(video.currentTime - t) < 1e-6) return
+  await new Promise((resolve) => {
+    const onSeeked = () => {
+      video.removeEventListener('seeked', onSeeked)
+      resolve()
+    }
+    video.addEventListener('seeked', onSeeked, { once: true })
+    video.currentTime = t
+  })
+}
+
 async function encodeVideo ({ file, srcMeta, onProgress }) {
   const w = srcMeta.w
   const h = srcMeta.h
@@ -205,8 +219,22 @@ async function encodeVideo ({ file, srcMeta, onProgress }) {
   const url = URL.createObjectURL(file)
   const v = document.createElement('video')
   v.muted = true; v.preload = 'auto'; v.playsInline = true
-  v.src = url
-  await new Promise((resolve, reject) => { v.onloadedmetadata = resolve; v.onerror = () => reject(new Error('video load failed')) })
+  await new Promise((resolve, reject) => {
+    const onLoaded = () => {
+      v.removeEventListener('loadedmetadata', onLoaded)
+      v.removeEventListener('error', onError)
+      resolve()
+    }
+    const onError = () => {
+      v.removeEventListener('loadedmetadata', onLoaded)
+      v.removeEventListener('error', onError)
+      reject(new Error('video load failed'))
+    }
+    v.addEventListener('loadedmetadata', onLoaded)
+    v.addEventListener('error', onError)
+    v.src = url
+    try { v.load() } catch (_) {}
+  })
   const canvas = document.createElement('canvas'); canvas.width = targetWidth; canvas.height = targetHeight
   const ctx = canvas.getContext('2d', { alpha: false })
 
@@ -216,13 +244,15 @@ async function encodeVideo ({ file, srcMeta, onProgress }) {
     const drawTime = i === 0
       ? Math.min(Math.max(0, t + (step * 0.5)), Math.max(0.000001, durationCfr - 0.000001))
       : targetTime
-
-    await new Promise((resolve) => { v.currentTime = drawTime; v.onseeked = () => resolve() })
+    if (i === 0) {}
+    await seekOnce(v, drawTime)
+    if (i === 0) {}
     const budgetMs = Math.min(34, Math.max(17, Math.round(step * 1000)))
     const presented = await waitForFrameReady(v, budgetMs)
     if (!presented && i === 0) {
       const nudge = Math.min(step * 0.25, 0.004)
-      await new Promise((resolve) => { v.currentTime = Math.min(drawTime + nudge, Math.max(0.000001, durationCfr - 0.000001)); v.onseeked = () => resolve() })
+      const target = Math.min(drawTime + nudge, Math.max(0.000001, durationCfr - 0.000001))
+      await seekOnce(v, target)
     }
 
     ctx.drawImage(v, 0, 0, canvas.width, canvas.height)
@@ -255,7 +285,6 @@ async function encodeVideo ({ file, srcMeta, onProgress }) {
   const sample = new AudioSample({ format: 'f32', sampleRate: TARGET_AUDIO_SR, numberOfChannels: TARGET_AUDIO_CHANNELS, timestamp: 0, data: interleaved })
   await audioSource.add(sample)
   audioSource.close()
-
   await output.finalize()
   const { buffer } = output.target
   const payload = new Uint8Array(buffer)
@@ -323,6 +352,8 @@ function registerStraightToVideoController (app, opts = {}) {
     }
 
     async _processFileInput (fileInput) {
+      const ua = typeof navigator !== 'undefined' && navigator.userAgent ? navigator.userAgent : ''
+      const isIos = /iP(hone|ad|od)/.test(ua)
       this._markFlag(fileInput, 'processing')
       fileInput.disabled = true
       try {

data/index.html

@@ -169,8 +169,8 @@
 
       <section id="install">
         <h2>Install</h2>
-<pre class="language-bash"><code>npm install straight-to-video</code></pre>
-<pre class="language-bash"><code>bundle add straight_to_video</code></pre>
+          <pre class="language-bash"><code>npm install straight-to-video</code></pre>
+          <pre class="language-bash"><code>bundle add straight_to_video</code></pre>
         <p>
           <a class="btn ghost readme" href="https://github.com/searlsco/straight-to-video" target="_blank" rel="noopener">View README</a>
       </p>
@@ -179,7 +179,12 @@
 
     <script type="module">
       import { Application, Controller } from '@hotwired/stimulus'
-      import { registerStraightToVideoController } from 'straight-to-video'
+      import { canOptimizeVideo, optimizeVideo, registerStraightToVideoController } from 'straight-to-video'
+
+      // Declare as global so that people can play with them
+      window.canOptimizeVideo = canOptimizeVideo
+      window.optimizeVideo = optimizeVideo
+      window.registerStraightToVideoController = registerStraightToVideoController
 
       const app = Application.start()
       registerStraightToVideoController(app, { Controller })
@@ -193,8 +198,6 @@
       const dl = document.getElementById('download')
       const sizes = document.getElementById('sizes')
 
-      // Removed environment badge
-
       // Single‑button flow: open picker then auto‑submit
       tryBtn.addEventListener('click', () => {
         if (fileEl.showPicker) { try { fileEl.showPicker(); return } catch (_) {} }

data/index.js

@@ -148,6 +148,20 @@ async function waitForFrameReady (video, budgetMs) {
   })
 }
 
+async function seekOnce (video, time) {
+  if (!video) return
+  const t = Number.isFinite(time) ? time : 0
+  if (Math.abs(video.currentTime - t) < 1e-6) return
+  await new Promise((resolve) => {
+    const onSeeked = () => {
+      video.removeEventListener('seeked', onSeeked)
+      resolve()
+    }
+    video.addEventListener('seeked', onSeeked, { once: true })
+    video.currentTime = t
+  })
+}
+
 async function encodeVideo ({ file, srcMeta, onProgress }) {
   const w = srcMeta.w
   const h = srcMeta.h
@@ -204,8 +218,22 @@ async function encodeVideo ({ file, srcMeta, onProgress }) {
   const url = URL.createObjectURL(file)
   const v = document.createElement('video')
   v.muted = true; v.preload = 'auto'; v.playsInline = true
-  v.src = url
-  await new Promise((resolve, reject) => { v.onloadedmetadata = resolve; v.onerror = () => reject(new Error('video load failed')) })
+  await new Promise((resolve, reject) => {
+    const onLoaded = () => {
+      v.removeEventListener('loadedmetadata', onLoaded)
+      v.removeEventListener('error', onError)
+      resolve()
+    }
+    const onError = () => {
+      v.removeEventListener('loadedmetadata', onLoaded)
+      v.removeEventListener('error', onError)
+      reject(new Error('video load failed'))
+    }
+    v.addEventListener('loadedmetadata', onLoaded)
+    v.addEventListener('error', onError)
+    v.src = url
+    try { v.load() } catch (_) {}
+  })
   const canvas = document.createElement('canvas'); canvas.width = targetWidth; canvas.height = targetHeight
   const ctx = canvas.getContext('2d', { alpha: false })
 
@@ -215,13 +243,15 @@ async function encodeVideo ({ file, srcMeta, onProgress }) {
     const drawTime = i === 0
       ? Math.min(Math.max(0, t + (step * 0.5)), Math.max(0.000001, durationCfr - 0.000001))
       : targetTime
-
-    await new Promise((resolve) => { v.currentTime = drawTime; v.onseeked = () => resolve() })
+    if (i === 0) {}
+    await seekOnce(v, drawTime)
+    if (i === 0) {}
     const budgetMs = Math.min(34, Math.max(17, Math.round(step * 1000)))
     const presented = await waitForFrameReady(v, budgetMs)
     if (!presented && i === 0) {
       const nudge = Math.min(step * 0.25, 0.004)
-      await new Promise((resolve) => { v.currentTime = Math.min(drawTime + nudge, Math.max(0.000001, durationCfr - 0.000001)); v.onseeked = () => resolve() })
+      const target = Math.min(drawTime + nudge, Math.max(0.000001, durationCfr - 0.000001))
+      await seekOnce(v, target)
     }
 
     ctx.drawImage(v, 0, 0, canvas.width, canvas.height)
@@ -254,7 +284,6 @@ async function encodeVideo ({ file, srcMeta, onProgress }) {
   const sample = new AudioSample({ format: 'f32', sampleRate: TARGET_AUDIO_SR, numberOfChannels: TARGET_AUDIO_CHANNELS, timestamp: 0, data: interleaved })
   await audioSource.add(sample)
   audioSource.close()
-
   await output.finalize()
   const { buffer } = output.target
   const payload = new Uint8Array(buffer)
@@ -322,6 +351,8 @@ function registerStraightToVideoController (app, opts = {}) {
     }
 
     async _processFileInput (fileInput) {
+      const ua = typeof navigator !== 'undefined' && navigator.userAgent ? navigator.userAgent : ''
+      const isIos = /iP(hone|ad|od)/.test(ua)
       this._markFlag(fileInput, 'processing')
       fileInput.disabled = true
       try {

data/lib/straight_to_video/version.rb

@@ -1,3 +1,3 @@
 module StraightToVideo
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "straight-to-video",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "straight-to-video",
-      "version": "0.0.5",
+      "version": "0.0.6",
       "license": "MIT",
       "dependencies": {
         "mediabunny": "^1.24.4"

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "straight-to-video",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "description": "Browser-based, hardware-accelerated video upload optimization",
   "type": "module",
   "exports": {

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: straight_to_video
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Justin Searls