Skills: clarify defaultOutName must omit file extension#6984
Skills: clarify defaultOutName must omit file extension#6984JonnyBurger wants to merge 1 commit intomainfrom
Conversation
Remotion appends the codec extension in getDefaultOutLocation; documenting this avoids duplicated suffixes like video.mp4.mp4. Made-with: Cursor
|
Reviewed PR #6984 — no actionable issues. The documentation changes accurately reflect the behavior of
|
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
There was a problem hiding this comment.
Verified against getDefaultOutLocation in packages/studio-shared/src/get-default-out-name.ts — the extension is unconditionally appended to compositionDefaultOutName (line 31), so including one in defaultOutName does indeed produce double extensions. Changes are accurate and concise.
Big Pickle (free) ｜ 𝕏
JonnyBurger
changed the title
|
Closing: upstream skill example already omitted the extension; the drift was in the vendored copy in the consuming repo, not a gap in remotion-dev/skills. |
1 participant
Summary
The agent skill for
calculateMetadatashoweddefaultOutNamewithout an extension in the code sample, but the prose and return-value list did not explain that the renderer appends the codec extension (seegetDefaultOutLocationin@remotion/studio-shared). Including.mp4or.movindefaultOutNameproduces paths likeCaption-First.mov.mov.Changes
defaultOutNamebullet in the return value section.Made with Cursor