Skip to content

fix(cli): preserve media after the last transcript word - #5325

Open
user-github-me wants to merge 7 commits into
heygen-com:mainfrom
user-github-me:fix/transcript-cut-source-duration
Open

user-github-me wants to merge 7 commits into
heygen-com:mainfrom
user-github-me:fix/transcript-cut-source-duration

Conversation

@user-github-me

@user-github-me user-github-me commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Transcript cuts use the input media's measured duration when planning kept segments. Removing a filler from a six-second source whose last word ends at four seconds now preserves the two-second ending. Explicit kept ranges after the last word also remain valid. Transcript-only planning still works without media tools and uses the last word's end.

The media guide and skill reference explain when to include the source while reviewing a cut plan.

Validation:

  • Updated against current main and regenerated the seven-skill manifest.
  • CLI rebuild and typecheck passed; the workspace was built before testing.
  • All 18 cut-list and transcript-cut tests passed, including real FFmpeg output with a retained ending and an explicit post-transcript range.
  • Lint, formatting, test reachability, manifest freshness, normal commit hooks and both comment checks passed.
  • Mint documentation validation and broken-link checks passed.

@jrusso1020

Copy link
Copy Markdown
Collaborator

Thanks for this, @user-github-me. This PR now has merge conflicts with main. Could you rebase it onto the latest main? Once it's conflict-free we'll review it and get it into the merge queue.

@mintlify

mintlify Bot commented Oct 24, 2026

Copy link
Copy Markdown

Preview approval required

This pull request comes from a fork, so Mintlify won't build a preview until someone with write access approves commit 3a3991b.

Approve preview

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.

2 participants