Feature/hyperframes video

[ceilf6]

Linked Issue Or Context

Summary

Impact Scope

GitNexus Impact Summary

• Risk level: -
• Critical skeleton changes: -
• GitNexus impact: -
• Verification: -

Verification

Checklist

• I have linked an issue or explained why this PR stands alone.
• I have kept the diff focused on the stated change.
• I have run pnpm quality:precommit, or explained why it could not run.
• I have run pnpm quality:local for critical skeleton changes, or explained why it could not run.
• I have updated docs or tests when behavior, public APIs, or Harness contracts changed.
• For critical skeleton changes, I have filled the GitNexus impact summary with concrete results.

[github-actions]

🛡️ ceilf6/repo-guard

代码评审报告: Feature/hyperframes video

风险等级: 中
处理建议: 请求修改
决策摘要: 目前不建议合并：PR 新增 HyperFrames 可执行渲染项目但没有关联 issue/验收结果/渲染验证证据，且核心 index.html 已膨胀到 1191 行并偏离自身设计文档建议的可拆分结构，合并前至少需要补齐验证记录并拆分或说明单文件设计理由。

级联分析

• 变更符号:
  • 新增营销文档：docs/marketing/frontagent-douyin-video.md
  • 新增 Superpowers 设计/计划文档
  • 新增 HyperFrames 项目：marketing/hyperframes/frontagent-douyin-promo/{meta.json,index.html,README.md,assets/*}
  • 新增媒体资产：assets/icon.png, assets/bgm.m4a, assets/bgm-captions.vtt
• 受影响流程:
  • HyperFrames lint / preview / render
  • 营销视频源码维护流程
  • 仓库二进制资产与媒体版权/来源管理
• 变更集外调用方:
  • 未看到产品 runtime、CLI、VS Code、Desktop、MCP 或 GitNexus 代码调用方受影响；主要影响是新增独立营销项目。
• 置信度: degraded

问题发现

1. [中] 新增可渲染 HyperFrames 项目缺少验收验证结果

   • 证据:
     • PR 描述中的 Verification 为空，Checklist 中 pnpm quality:precommit、quality:local、docs/tests 更新项均未勾选或解释。
     • 设计文档 Acceptance Criteria 明确要求“local render command has been attempted and its result is reported”，并在 Verification Plan 中要求 lint/preview/render/ffprobe/代表帧检查；当前 PR 没有提供任何命令结果或失败原因。
     • 新增项目包含 meta.json、index.html、音频/图片资产和 README 中的 render 命令，属于可执行渲染源，而不仅是静态文档。
   • 受影响调用方/流程:
     • npx hyperframes lint
     • npx hyperframes render --output dist/frontagent-douyin-promo.mp4
     • 后续维护者无法判断 1080x1920、30fps、30s、非空画面、音频轨、字幕 cue 是否真实可渲染。
   • 最小可行修复:
     • 在 PR 描述或提交文档中补充实际运行结果：Node/FFmpeg 版本、npx hyperframes lint、render 命令、ffprobe 输出；如果环境缺失，记录确切失败信息并说明源码已静态检查到什么程度。
     • 若 render 成功，至少报告 MP4 路径、宽高、帧率、时长，并说明抽帧检查结果。

2. [中] index.html 单文件 1191 行，偏离设计文档中自然拆分的 composition 边界

   • 证据:
     • marketing/hyperframes/frontagent-douyin-promo/index.html 新增 1191 行，集中包含全局样式、六个 scene 的 HTML、所有 keyframes 和自定义 timeline JS。
     • 新增设计文档的 “HyperFrames Project Design” 明确给出 compositions/01-hook.html 到 06-cta.html 的结构，并说明 root index.html 应 orchestrate six beats as timed sub-compositions、每个 sub-composition one visual responsibility。
     • 当前实现把六个场景和渲染控制逻辑全部塞进一个大文件，后续修改某一镜头 copy、动画或 timing 时需要扫描整文件，验证风险和冲突面明显增加。
   • 受影响调用方/流程:
     • 营销视频迭代、单场景修改、review diff 可读性、HyperFrames 渲染问题定位。
   • 最小可行修复:
     • 优先按设计文档拆出每个 scene 的 HTML/CSS/局部样式，保留 index.html 只负责 composition 元数据、全局 token 和 orchestration。
     • 如果 HyperFrames 当前不支持子 composition include，则至少把 CSS/JS 拆到项目本地文件，或在 PR 中明确说明为什么必须单文件，并补充后续拆分计划。

3. [中] 提交了本地 BGM 二进制资产但缺少来源/授权证明

   • 证据:
     • 设计文档 Non-Goals 明确要求 “Do not bundle copyrighted music or third-party media without a clear license.”
     • README 和营销文档称 assets/bgm.m4a 是 “original local BGM / original local synth bed”，但 PR 没有提供生成方式、作者声明、许可证或可复现命令。
     • assets/bgm.m4a 是新增二进制媒体文件，diff 无法审查内容或来源。
   • 受影响调用方/流程:
     • 仓库资产合规、后续视频发布、维护者判断能否长期保留该媒体文件。
   • 最小可行修复:
     • 在 README 或资产旁补充 BGM 的来源、作者/生成方式、许可证/使用授权；若无法证明，移除 bgm.m4a，改为静音或使用明确可分发授权的生成音频，并记录生成命令。

行级发现

• [marketing/hyperframes/frontagent-douyin-promo/index.html:1] 这个新增文件一次性承载 1191 行样式、六个场景和 timeline 控制，已经越过可扫描边界；建议按设计文档拆出 compositions/01-hook 到 06-cta 或至少拆分本地 CSS/JS，降低后续单镜头修改的验证成本。
• [marketing/hyperframes/frontagent-douyin-promo/README.md:17] 这里声明 assets/bgm.m4a 是 original local BGM，但没有给出生成方式、作者/许可证或可复现命令；请补充资产来源/授权说明，或移除二进制 BGM。
• [docs/superpowers/specs/2026-06-17-frontagent-hyperframes-douyin-video-design.md:249] Acceptance Criteria 要求报告本地 render attempt 结果，但 PR 描述没有提供 lint/render/ffprobe/抽帧结果或失败原因；请在合并前补齐实际验证记录。
• [marketing/hyperframes/frontagent-douyin-promo/README.md:13] README 给出了 render 命令，但没有记录本 PR 是否已运行及结果；建议增加 “Verification result” 小节，写明 lint/render/ffprobe 的实际输出或环境缺失原因。

Karpathy 评审

• 假设:
  • 关联 Issue 缺失，产品验收主要只能依赖本 PR 新增的设计/计划文档；级联置信度降级。
  • BGM 被假设为原创且可提交，但没有可审查来源证据。
• 简洁性:
  • 单个 1191 行 HTML 把场景、样式和时间线混在一起；更自然的结构是按六个 beat 拆分，直接减少扫描范围和冲突面。
• 结构质量:
  • 新增项目虽然隔离于 runtime 代码，但核心可维护边界偏弱；后续任何 copy/timing/动画改动都需要理解整份文件。
• 变更范围:
  • 变更基本隔离在 docs/marketing/hyperframes 下，没有修改产品 runtime；范围本身可接受。
  • 但 PR 描述为空、Checklist 未完成，使维护者无法判断是否满足设计文档的完成定义。
• 验证:
  • 缺少 HyperFrames lint/render、媒体属性、抽帧检查、资产来源验证；对于新增视频项目，这是主要合并风险。

缺失覆盖

• npx hyperframes lint 的实际结果。
• npx hyperframes render --output dist/frontagent-douyin-promo.mp4 的实际结果或失败原因。
• ffprobe 对输出视频宽高、帧率、时长的结果。
• hook/midpoint/CTA 代表帧的人工检查记录。
• assets/bgm.m4a 的来源、授权或生成方式说明。

[github-actions]

这个新增文件一次性承载 1191 行样式、六个场景和 timeline 控制，已经越过可扫描边界；建议按设计文档拆出 compositions/01-hook 到 06-cta 或至少拆分本地 CSS/JS，降低后续单镜头修改的验证成本。

[github-actions]

这里声明 assets/bgm.m4a 是 original local BGM，但没有给出生成方式、作者/许可证或可复现命令；请补充资产来源/授权说明，或移除二进制 BGM。

[github-actions]

Acceptance Criteria 要求报告本地 render attempt 结果，但 PR 描述没有提供 lint/render/ffprobe/抽帧结果或失败原因；请在合并前补齐实际验证记录。

[github-actions]

README 给出了 render 命令，但没有记录本 PR 是否已运行及结果；建议增加 “Verification result” 小节，写明 lint/render/ffprobe 的实际输出或环境缺失原因。