Repository navigation
fix(video-agent-kit): escape concat input paths - #89
Open
Ricardo-M-L wants to merge 1 commit into
Open
Ricardo-M-L wants to merge 1 commit into
Ricardo-M-L wants to merge 1 commit into
Conversation
Collaborator
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
9 of 12 tasks
This branch has not been deployed
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.
Plugin type (required) / 插件类型(必填)
Affected plugin name(s) / 涉及的插件名称:
video-agent-kit(Standard plugin)What / 改动内容
FFmpeg concat demuxer list files have their own quoting rules. The plugin previously wrote raw single-quoted paths in eight render and recap paths, so a valid project or media path such as
Editor's project/clip.wavwas parsed incorrectly and failed with “No such file or directory”.This change:
ffproc.py;video-agent-kitfrom0.4.3to0.4.4in both manifests andmarketplace.json.No dependency, network, permission, API, or file-write surface is added. The helper only changes serialization of the existing temporary concat-list files. No third-party code or assets are included.
The official plugin currently has a single Chinese
README.mdand noREADME_CN.md; this PR updates that existing user-facing document without inventing a large unrelated documentation migration.Open PR #38 changes Python launcher resolution in the same plugin but does not overlap this runtime fix. It currently reserves version
0.4.5; if it merges first, this branch will be rebased and bumped to the next free plugin version.Validation
python3 -m unittest plugins/video-agent-kit/tests/test_ffconcat_paths.py -v— 3 passed, including the real FFmpeg apostrophe-path smoke test.python3 -m compileall plugins/video-agent-kit/mcp plugins/video-agent-kit/tests— passed.pnpm install --frozen-lockfile— passed.pnpm build— passed with Node.js 24.20.0.pnpm test— passed (31 root TypeScript tests and all UI-plugin suites).python3 -m unittest discover -s tests— 122 passed, 19 skipped.python3 scripts/validate.py— 34 plugins validated.python3 scripts/build_dist.py— 34 artifacts and 292 assets built.git diff --check— passed.The affected FFmpeg behavior was exercised directly; a full ZCode desktop session was not available.
Self-check / 自查清单
external_plugins/<name>/; existing fixes retain ownership / 插件名唯一且 kebab-case,外部新增插件位于external_plugins/<name>/,已有插件修复保留原归属categoryis set independently of publisher ownership / 用途分类category已填写,与维护归属分开versionbumped in BOTHplugin.jsonandmarketplace.json(artifacts are immutable) /plugin.json与marketplace.json两处版本号已同步升级(制品不可变)README.md(EN) andREADME_CN.md(中文) both updated and equivalent / 中英文 README 均已更新且语义一致 — legacy official plugin has noREADME_CN.md; its existingREADME.mdwas updateddescription_i18nhas bothenandzh-CN/description_i18n包含en与zh-CNpython3 scripts/validate.pyandpython3 scripts/build_dist.pypass locally / 本地校验与构建均通过