Repository navigation
Fix Windows active timeline writes, compile metadata, and fixture redaction - #135
Merged
renezander030 merged 1 commit intoOct 3, 2026
Merged
Conversation
This was referenced Oct 3, 2026
renezander030
marked this pull request as ready for review
October 3, 2026 11:57
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CapCut 8.7.0 on Windows loads an existing project's selected nested timeline, while the CLI previously edited only the root mirrors. Those edits disappeared when the app reopened or closed the project. The supplied field fixture now drives active-timeline selection and regression coverage.
This patch also fixes two independent pipeline defects: compile's ratio-only canvas override and source-duration metadata (#133), and fixture redaction of JSON-escaped Windows home paths (#134).
Changes
--nested. Existing root-only edits appear as divergence; the plan retains the newer-mirror refusal rather than silently choosing newer roots.--checkand actual writes. Separate full probed video/audio duration from segment duration in materials and sidecar entries. Validate speed-adjusted source ranges before initialization. Without probe evidence, an explicit duration supplies a lower bound covering the requested source range.Validation
npm test: 1,017 tests passed, zero failures or skips on Node 22.22.0.npm run lint,npm run docs:commands, andgit diff --checkpassed.The subprocess sandbox times out on the existing compile stdin test even on untouched master. The complete suite passes when run outside that sandbox.
Desktop verification
The field fixture establishes on-disk shape and divergence. The reporter observed nested content winning in CapCut 8.7.0.3685 on Windows; the patched CLI still needs an app open/close round-trip. Follow the review guide for caption survival, media staging, restore, compile cuts, and fixture redaction.
Keep #50 open until that round-trip passes. The 7.x and Mac 9.2.8-beta4 reports remain separately unverified. This PR prepares the patch for review; it does not publish a release.
Fixes #133.
Fixes #134.
Refs #50.