fix: clean up dead renderSeek config, tool-call fixHints, and unnecessary comments

Author: miguel-heygen

packages/core/src/lint/hyperframeLinter.ts

@@ -369,8 +369,7 @@ export function lintHyperframeHtml(
   }
 
   // #3.6: Placeholder/fake media URLs — CRITICAL
-  // Agents sometimes fabricate URLs (placehold.co, placeholder.com, example.com)
-  // instead of using fetch_media or generate_image. These 404 at render time.
+  // Placeholder URLs (placehold.co, placeholder.com, example.com) will 404 at render time.
   {
     const PLACEHOLDER_DOMAINS = /\b(placehold\.co|placeholder\.com|placekitten\.com|picsum\.photos|example\.com|via\.placeholder\.com|dummyimage\.com)\b/i;
     for (const tag of tags) {
@@ -384,16 +383,15 @@ export function lintHyperframeHtml(
           severity: "error",
           message: `<${tag.name}${elementId ? ` id="${elementId}"` : ""}> uses a placeholder URL that will 404 at render time: ${src.slice(0, 80)}`,
           elementId,
-          fixHint: "Use fetch_media to find real stock media, or generate_image/generate_video for AI-generated content.",
+          fixHint: "Replace with a real media URL. Placeholder domains will 404 at render time.",
           snippet: truncateSnippet(tag.raw),
         });
       }
     }
   }
 
   // #3.7: Fabricated inline base64 media — CRITICAL
-  // Agents sometimes embed fake base64 audio/video data instead of using fetch_media.
-  // Even small base64 data URIs for audio are almost always fabricated garbage that
+  // Inline base64 audio/video data is almost always fabricated garbage that
   // won't play. Real audio files are 100KB+ when base64-encoded.
   {
     const base64MediaRe = /src\s*=\s*["'](data:(?:audio|video)\/[^;]+;base64,([A-Za-z0-9+/=]{100,}))["']/gi;
@@ -407,9 +405,11 @@ export function lintHyperframeHtml(
       // Any embedded base64 audio is suspicious — real audio should be a file
       pushFinding({
         code: "fabricated_inline_media",
-        severity: "error",
-        message: `Embedded base64 ${isSuspicious ? "FABRICATED" : ""} media detected (${(dataSize / 1024).toFixed(0)} KB). Inline base64 audio/video is almost always fake data that won't play. Use fetch_media or extract_audio to get real audio files.`,
-        fixHint: "Remove the data: URI. Use fetch_media to search for stock music, or extract_audio to get audio from a video.",
+        severity: isSuspicious ? "error" : "warning",
+        message: isSuspicious
+          ? `Fabricated base64 media detected (${(dataSize / 1024).toFixed(0)} KB). This is almost certainly fake data that won't play.`
+          : `Embedded base64 audio/video detected (${(dataSize / 1024).toFixed(0)} KB). Consider using a file URL instead.`,
+        fixHint: "Remove the data: URI and use a real media file URL instead.",
         snippet: truncateSnippet((b64Match[1] ?? "").slice(0, 80) + "..."),
       });
     }
@@ -858,7 +858,7 @@ export async function lintMediaUrls(
           severity: "error",
           message: `<${tagName}${elementId ? ` id="${elementId}"` : ""}> references a URL that returned HTTP ${resp.status}: ${url.slice(0, 100)}`,
           elementId,
-          fixHint: "This URL is not accessible. Use fetch_media to find real stock media, or check the URL is correct.",
+          fixHint: "This URL is not accessible. Replace with a valid, reachable media URL.",
           snippet,
         });
       }
@@ -869,7 +869,7 @@ export async function lintMediaUrls(
         severity: "error",
         message: `<${tagName}${elementId ? ` id="${elementId}"` : ""}> references an unreachable URL (${reason}): ${url.slice(0, 100)}`,
         elementId,
-        fixHint: "This URL is not accessible. Use fetch_media to find real stock media, or check the URL is correct.",
+        fixHint: "This URL is not accessible. Replace with a valid, reachable media URL.",
         snippet,
       });
     }

packages/engine/src/config.ts

@@ -66,16 +66,6 @@ export interface EngineConfig {
   /** Custom manifest path for Hyperframe runtime. */
   runtimeManifestPath?: string;
 
-  // ── Render Seek ────────────────────────────────────────────────────
-  /** Seek strategy for frame-accurate rendering. */
-  renderSeekMode: "preview-phase" | "strict-boundary";
-  /** Emit seek diagnostics to console. */
-  renderSeekDiagnostics: boolean;
-  /** Time step (seconds) for seek resolution. */
-  renderSeekStep: number;
-  /** Fractional offset within a frame for seek positioning. */
-  renderSeekOffsetFraction: number;
-
   // ── Debug ────────────────────────────────────────────────────────────
   debug: boolean;
 }
@@ -114,11 +104,6 @@ export const DEFAULT_CONFIG: EngineConfig = {
 
   verifyRuntime: true,
 
-  renderSeekMode: "preview-phase",
-  renderSeekDiagnostics: false,
-  renderSeekStep: 1 / 120,
-  renderSeekOffsetFraction: 0.5,
-
   debug: false,
 };
 
@@ -202,16 +187,6 @@ export function resolveConfig(overrides?: Partial<EngineConfig>): EngineConfig {
 
     verifyRuntime: env("PRODUCER_VERIFY_HYPERFRAME_RUNTIME") !== "false",
     runtimeManifestPath: env("PRODUCER_HYPERFRAME_MANIFEST_PATH"),
-
-    renderSeekMode: env("PRODUCER_RUNTIME_RENDER_SEEK_MODE") === "strict-boundary"
-      ? "strict-boundary" as const
-      : DEFAULT_CONFIG.renderSeekMode,
-    renderSeekDiagnostics: envBool("PRODUCER_DEBUG_SEEK_DIAGNOSTICS", DEFAULT_CONFIG.renderSeekDiagnostics),
-    renderSeekStep: Math.max(1 / 600, envNum("PRODUCER_RENDER_SEEK_STEP", DEFAULT_CONFIG.renderSeekStep)),
-    renderSeekOffsetFraction: Math.max(
-      0,
-      Math.min(0.95, envNum("PRODUCER_RUNTIME_RENDER_SEEK_OFFSET_FRACTION", DEFAULT_CONFIG.renderSeekOffsetFraction)),
-    ),
   };
 
   // Remove undefined values so they don't override defaults

packages/producer/src/services/fileServer.ts

@@ -40,32 +40,7 @@ const MIME_TYPES: Record<string, string> = {
  * Render mode extension -- adds renderSeek() for frame-accurate seeking
  * without media sync (videos are replaced with frame images during render).
  */
-const RENDER_SEEK_MODE =
-  process.env.PRODUCER_RUNTIME_RENDER_SEEK_MODE === "strict-boundary"
-    ? "strict-boundary"
-    : "preview-phase";
-const RENDER_SEEK_DIAGNOSTICS = process.env.PRODUCER_DEBUG_SEEK_DIAGNOSTICS === "true";
-const RENDER_SEEK_STEP = Math.max(
-  1 / 600,
-  Number(process.env.PRODUCER_RENDER_SEEK_STEP || 1 / 120),
-);
-const RENDER_SEEK_OFFSET_FRACTION = Math.max(
-  0,
-  Math.min(0.95, Number(process.env.PRODUCER_RUNTIME_RENDER_SEEK_OFFSET_FRACTION || 0.5)),
-);
-
 const RENDER_MODE_SCRIPT = `(function() {
-  var __seekMode = ${JSON.stringify(RENDER_SEEK_MODE)};
-  var __seekDiagnostics = ${RENDER_SEEK_DIAGNOSTICS ? "true" : "false"};
-  var __seekStep = ${RENDER_SEEK_STEP};
-  var __seekOffsetFraction = ${RENDER_SEEK_OFFSET_FRACTION};
-  window.__HF_EXPORT_RENDER_SEEK_CONFIG = {
-    mode: __seekMode,
-    diagnostics: __seekDiagnostics,
-    step: __seekStep,
-    offsetFraction: __seekOffsetFraction,
-    owner: "runtime",
-  };
   function installMediaFallbackPlayer() {
     if (document.querySelector('[data-composition-id]')) return false;
     var mediaEls = Array.from(document.querySelectorAll('video, audio'));

packages/producer/src/services/htmlCompiler.ts

@@ -580,9 +580,6 @@ function inlineSubCompositions(
       const existing = host.getAttribute("style") || "";
       const needsWidth = !existing.includes("width");
       const needsHeight = !existing.includes("height");
-      // Don't inject position — the host's CSS class determines positioning.
-      // Injecting position:relative breaks overlay compositions (captions,
-      // foreground-ui) that use position:absolute via their CSS class.
       const additions = [
         needsWidth ? `width:${hostW}px` : "",
         needsHeight ? `height:${hostH}px` : "",