Skip to content

docs(skills): check render tools with doctor before design work - #7208

Open
vincent65 wants to merge 1 commit into
heygen-com:mainfrom
vincent65:docs/hyperframes-doctor-step
Open

vincent65 wants to merge 1 commit into
heygen-com:mainfrom
vincent65:docs/hyperframes-doctor-step

Conversation

@vincent65

Copy link
Copy Markdown
Contributor

What

Adds one step to the start of the hyperframes entry skill's work loop:

  • Before design work on a new or resumed project: npx hyperframes doctor --json. Only render needs FFmpeg and FFprobe; if either is not ok, pass its hint to the person now, not at the render.

The hyperframes budget in scripts/check-core-skills.mjs goes from 7360 to 7552 B to make room, with a comment saying what the bytes pay for, the same way the hyperframes-cli and beacon raises are recorded.

Why

An agent following the skill could plan, design and get approval before render reported FFmpeg missing. This is item 3 of #5232, which @miguel-heygen asked for as its own PR so the skill wording gets reviewed separately.

Related work

Refs #5232. Items 1 and 2 (the init warning and the macOS non-Homebrew hint) are in #7142. The two PRs don't depend on each other. With #7142 merged, the hint the agent passes on also names HYPERFRAMES_FFMPEG_PATH / HYPERFRAMES_FFPROBE_PATH and .hyperframes/bin.

How

  • The step checks the FFmpeg and FFprobe entries, not the top-level .ok. .ok is also false for checks that don't matter here, like Docker not running or a newer CLI version being available.
  • I raised the budget instead of trimming existing lines, so no current guidance changes. The step is 196 B. If you'd rather keep the budget and cut something, tell me which line and I'll rework it.

Test plan

  • Unit tests added/updated: not applicable, this is skill text and a budget constant.
  • Manual testing performed
    • HYPERFRAMES_FFMPEG_PATH=/nope npx hyperframes doctor --json returns checks[] entries { name: "FFmpeg", ok: false, hint: "..." } and { name: "FFprobe", ok: true }. This is the shape the step relies on.
  • node scripts/check-core-skills.mjs (7550 B against the 7552 B budget; commands and paths resolve), node --test scripts/check-core-skills.test.mjs (11 pass), tsx scripts/lint-skills.ts, check-skill-mirror, skills manifest in sync, format:check, check-comment-citations.
  • Documentation updated: this PR is the documentation change.
  • Comments follow CONTRIBUTING.md "Comments"

🤖 Generated with Claude Code

Only render needs FFmpeg and FFprobe, so an agent following the hyperframes
skill learned they were missing at the last step. The work loop now starts a
new or resumed project with `npx hyperframes doctor --json` and has the agent
pass on the FFmpeg/FFprobe hint before design work.

The step costs 196 B, so the hyperframes budget in check-core-skills.mjs goes
from 7360 to 7552 B, with a comment saying what the bytes pay for.

Refs heygen-com#5232
@vincent65
vincent65 force-pushed the docs/hyperframes-doctor-step branch from d222125 to 4a889bd Compare November 8, 2026 23:04
@vincent65

Copy link
Copy Markdown
Contributor Author

@miguel-heygen @jrusso1020 rebased onto current main (2927fe6). The only conflict was the generated skills-manifest.json, which I regenerated with gen-skills-manifest.ts; the skill text is unchanged. skills/hyperframes/SKILL.md is 7544 B against the raised 7552 B budget, and check-core-skills, the skill lint, the mirror check and the manifest are all in sync locally. This is item 3 of #5232, as agreed on the issue; items 1 and 2 are in #7142. Could you approve the fork workflow runs and review when you get a chance? Thanks!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant