Repository navigation
docs(guides): one page per kind of video, with its prompts folded in - #6359
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
jrusso1020
left a comment
There was a problem hiding this comment.
Looks good. All six redirects land on real prompt sections, the workflow list and both comparison players render, and mint validate and broken-links pass. The app labels and skill rules I sampled match source. Two small notes inline. The comparison's timing, line counts and install size are as reported in the run, not re-measured.
| to seconds. Roughly 80% of a typical composition translates mechanically. | ||
| to seconds. | ||
|
|
||
| The other 20% is the point of the skill. It refuses to translate what does not |
There was a problem hiding this comment.
Nit: the 80% claim above was removed, so "The other 20%" no longer refers to anything. Maybe "The harder cases are the point of the skill."
| @@ -110,7 +140,7 @@ export const WorkflowChooser = () => { | |||
| <div className="hf-workflow-routes" ref={gridRef}> | |||
There was a problem hiding this comment.
The description says large playing tiles, two per row, but this is still one route per row (no grid on .hf-workflow-routes in custom.css). Either update the description or add the layout.
…pt pages as agreed deletions
jrusso1020
left a comment
There was a problem hiding this comment.
Both notes from our last pass are fixed: the chooser is two columns (one on phones) with no horizontal overflow, and the migration paragraph reads "The harder cases." The six deletion exceptions match the folded pages, and the guard plus its 9 tests pass. Nothing blocking from us.
miguel-heygen
left a comment
There was a problem hiding this comment.
Independent review of 90655e9da72b76919a375cfe7b9a1ac02e212122 against 0ae48c9f33d20ab5a92f999e6e25f6bbc3d108ab. No blocking finding.
docs/snippets/workflow-chooser.jsx keeps route data inside the named export, preserves reduced-motion cleanup, and gives every tile an accessible link. The six docs/docs.json redirects point at existing prompt sections, and the deletion-guard additions enumerate exactly those six folded files. Rames's two earlier notes are resolved at this head.
Audited: the ten guides, chooser, workflow page, CSS additions, navigation/redirects, changed cross-links and deletion exceptions; git/API file census agrees (30 files). Source checks include film/PR credit and release rules, media-use sign-in and free/paid contracts, slideshow controls, and Desktop labels at locally available HFI 66f122de (source evidence, not native acceptance).
Executed: 9 deletion-guard tests; independent exact-file/deletion/redirect checks, including a removed-exception negative control and the disjoint seven-redirect union with #6360. Published Mintlify PR preview: all 11 changed guide/chooser routes render; all six old URLs land on their real anchors. Chromium verified two desktop columns, one mobile column without overflow, keyboard activation of a tile, 13 paused/unloaded tiles under reduced motion, and loaded comparison players with manual playback. Captured page-error/console output is empty; no captured HTTP >=400 for this preview. Hosted exact-head Validate docs log independently records successful mint validate and mint broken-links --check-redirects.
Trusting / limits: comparison runtime/file-count/install-size figures and generation results remain attributed to the author's run; I did not repeat a paid/model run or native Studio workflow. Binary posters were checked through preview rendering. The environment was a public PR preview, not deployed production or a signed-in backend session. No install, product edit, worktree, merge or deployment.
Approval covers this exact head only. Both PRs edit the redirect tail and deletion allowlist; a subsequent rebase/composition needs a fresh check.
CI snapshot: 45 successful, 19 skipped checks; Windows engine-cli still running. No failed check. These are executed results at this head, not a claim that the remaining test has passed.
Verdict: APPROVE
Reasoning: The folded content's routes, responsive chooser and relevant documentation gates work at the reviewed head, and source checks found no blocking instruction or integration defect.
— Magi (desktop-review)
What
The "make a video" guides, rewritten to the docs decisions of 22–23 Oct (from the approved sitemap and page review):
/prompting/product-launch,explainers,code-and-prs,captions-and-talking-heads,music-and-slideshows,motion-graphics. The deck prompt moves to the presentation guide./hyperframesis named once per page, for people using their own agent.Why
Today the same kind of video has two pages (the guide, and a Prompt Guide page for what to type) that compete in search and split readers. People arrive on the guides (the launch guide has 62k Search Console impressions), so the prompts move there. A reference is the fastest start: a video you half like beats a blank page. The titles use the words people type (docs search, the docs assistant and Google autocomplete, pulled 22 Oct).
How we'll know: the old prompting URLs fold into the guides in Search Console within a few weeks; clicks per impression on the retitled guides; Studio app downloads that start from these pages (
srctags already tracked).Related work
Docs PR 4a, reviewed with 4b (media pages). Earlier: #6223, #6303, #6352. The Remotion migration page moves to its own page in the Prompt Guide PR (decided 22 Oct). "Studio" meaning the browser editor gets renamed with the browser editor pages.
How
docs/content-samples, 22–23 Oct), then every claim re-checked against today'smainof hyperframes and hyperframes-internal:packages/desktopsource today; the Script tab is public from build b463 (feature flagdesktop-script-tab).docs/studio-app/reference-video.mdx.skills/hyperframes/references/film.md).packages/player/src/slideshow).heygen auth login --oauth(skills/media-use).node_modulesvs none, and a music bed plus five sound effects in the HyperFrames result.docs/films/*-v1.mp4); the GPS film re-encoded at VMAF 96.5.Test plan
mint broken-links --check-redirects: no broken links;mint validatepasses;bun run lintand prettier pass