@ignix-social/editor 0.1.13 → 0.1.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ignix-social/editor",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "description": "Ignix Editor: render HTML video compositions (reels, edits, clips) on your own computer. Based on HyperFrames.",
   "license": "Apache-2.0",
   "type": "module",

package/skills/ignix-editor/SKILL.md

@@ -147,4 +147,4 @@ Only if the app offers no way, give the user ONE line to paste into Terminal, e.
 
 1. `ignix-editor check` → 0 errors.
 2. `ignix-editor snapshot --at <every ~0.5 s and every cue>` and LOOK at the frames (or tile them: `ignix-editor ffmpeg -pattern_type glob -i 'snapshots/frame-*.png' -vf "scale=270:480,tile=8x2" sheet.jpg`). Check: hook readable at frame 0, nothing still for >1.5 s, every named thing pointed at on its word, captions never over what is pointed at, nothing outside the safe zone.
-3. `ignix-editor render` → it reviews the MP4 by itself and prints `✓ Review: ready to show` or `✗ Review: NOT READY` with what to fix and where (stretches where nothing on the stage moves, a mostly empty frame, silences between voice lines, no change of scene, no sound effects). NOT READY means it is not finished: fix every ✗, render again, and only then show it. The `!` lines are worth fixing too when it is quick. (`ignix-editor review <mp4>` runs the same check on any video.) Then open the MP4's first frame and a few cues again, and deliver as the playbook says.
+3. `ignix-editor render` → it reviews the MP4 by itself and prints `✓ Review: ready to show` or `✗ Review: NOT READY` with what to fix and where (stretches where nothing on the stage moves, a mostly empty frame, silences between voice lines, no change of scene). NOT READY means it is not finished: fix every ✗, render again, and only then show it. The `!` lines are worth fixing too when it is quick. (`ignix-editor review <mp4>` runs the same check on any video.) Then open the MP4's first frame and a few cues again, and deliver as the playbook says.

package/src/analysis/review.mjs

@@ -13,7 +13,7 @@ import { measureVideo } from './style.mjs';
 //              encenderse. Los subtítulos solos NO cuentan como movimiento.
 //   lleno    — qué parte de la zona segura ocupa algo distinto del fondo.
 //   huecos   — silencios entre frases de la voz.
-//   cortes y efectos — los de `style measure` (cambios de escena, efectos oídos).
+//   cortes — los de `style measure` (cambios de escena).
 
 const FPS = 10;
 /** Fracción de píxeles que tiene que cambiar entre dos fotogramas para contar como movimiento. */
@@ -43,11 +43,13 @@ const SAFE = { x0: 0.12, x1: 0.88, y0: 0.13, y1: 0.765 };
 /**
  * Lo que cambia en un vídeo HORIZONTAL (YouTube, 2026-10-06): su manual pide un cambio cada
  * ~3-5 s (no 1-1,5), subtítulos abajo del todo y ~0,8 s de aire al cambiar de capítulo.
- * Sin calibrar con vídeos reales todavía: por eso más holgado.
+ * Calibrado con un youtuber que Fran da por bueno (36 min, 2026-10-06): plano al 89 %, quietud
+ * máxima 4,1 s, 8 cortes/min y ~26 pausas naturales de 1-2 s al hablar → los huecos avisan desde
+ * 2,5 s, y «sin efectos» no se dice (bajo una voz continua no se distinguen).
  */
 const WIDE = {
   stillFailSec: 6,
-  gapWarnSec: 1,
+  gapWarnSec: 2.5,
   captionBand: { y0: 0.8, y1: 0.98 },
   safe: { x0: 0.05, x1: 0.95, y0: 0.06, y1: 0.92 },
 };
@@ -202,7 +204,7 @@ export async function reviewVideo({ ffmpeg, ffprobe, path, log = () => {} }) {
       level: 'warn',
       code: 'speech_gap',
       at: [r1(gp.start), r1(gp.end)],
-      text: `${r1(gp.end - gp.start)} s of silence between two voice lines at ${r1(gp.start)} s. Lay the lines back to back (≤ 0.3 s${wide ? '; ~0.8 s only at a chapter change' : ''}).`,
+      text: `${r1(gp.end - gp.start)} s of silence between two voice lines at ${r1(gp.start)} s. ${wide ? 'Cut the pause, or keep it only where it lands a point or changes chapter.' : 'Lay the lines back to back (≤ 0.3 s).'}`,
     });
   }
   // Calibrado con los reels de Fran (2026-10-06): todos los buenos cortan 3,5-10 veces por
@@ -210,10 +212,10 @@ export async function reviewVideo({ ffmpeg, ffprobe, path, log = () => {} }) {
   if (m?.shots?.cutsPerMinute === 0 && info.duration > 10) {
     issues.push({ level: 'warn', code: 'no_cuts', text: 'No real change of scene anywhere: it plays as one slow sequence of fades. Change scene with cuts or quick whips, and move a camera inside each scene.' });
   }
-  const sfx = m?.audio?.sound?.sfxPerMinute;
-  if (sfx === 0 && info.duration > 8) {
-    issues.push({ level: 'warn', code: 'no_sfx', text: 'No sound effect is heard. A 20 s reel usually needs 4-8, on the moments that matter (hook, scene change, the key click or reveal, the CTA).' });
-  }
+  // Sin aviso de efectos de sonido (2026-10-07): bajo una voz continua no se
+  // distinguen. En un reel de Claude con 8 efectos dijo «no se oye ninguno»
+  // (estaban, a −18,6 dB en los huecos). Un aviso que falla en lo bueno enseña
+  // al agente a ignorar la revisión; los efectos siguen en el manual.
   return {
     path,
     durationSec: r1(info.duration),
@@ -223,7 +225,6 @@ export async function reviewVideo({ ffmpeg, ffprobe, path, log = () => {} }) {
       longestStillSec: r1(Math.max(0, ...stillRuns(steps, 0).map((s) => s.end - s.start))),
       fill: fill === null ? null : Math.round(fill * 100) / 100,
       textSize: m?.captions?.size ?? null,
-      sfxPerMinute: sfx ?? null,
       cutsPerMinute: m?.shots?.cutsPerMinute ?? null,
     },
     issues,