Repository navigation
Conversation
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
|
The current verification notes mainly cover DeepSeek Harness. Could you also document testing in ZCode? Has the full flow been tested successfully in ZCode: plugin installation → Skill invocation → recording → video generation and verification? Please include the ZCode version, operating system, and results for each step, and explicitly note any steps that have not yet been verified. |
|
Added the requested ZCode acceptance matrix to the PR body. The tested environment was ZCode CLI 0.17.1 on macOS 27.0.1 (26A434, arm64), Node v24.15.0, Python 3.14.4 and FFmpeg 8.1.1.
The full recording → render → verify chain ran in one ZCode Agent session. Not verified in ZCode: installation/update from the public marketplace index, Windows/Linux, an independent One ZCode-specific usability note is documented: when |
ZCode acceptance v2: full lifecycle, failure paths, and sample rendersTested on 2026-11-02 with ZCode CLI 0.17.1 on macOS 27.0.1 (26A434, arm64), Node v24.15.0, Python 3.14.4 and FFmpeg 8.1.1, plus one Linux run inside Install lifecycle
Skill selection
Recording → render → verify (real ZCode Agent session)One headless ZCode session ran the installed plugin end to end in
Failure paths
Linux hostThe same episode was rendered inside
Sample renders for reviewBoth files come from the same three-segment episode and were gated at score 100:
These are attached as a pre-release on my fork purely as review evidence, not as a plugin release. Both URLs were re-downloaded and hashed after upload; the hashes above match the rendered files. ZCode CLI observations (not plugin defects)
Still not verified in ZCodeInstall or update from the public marketplace index (this round used local development marketplaces only), Windows, the desktop GUI flow (all runs above used the CLI headless runner), an independent |
Session IDs behind the matrix aboveZCode persists every CLI run as a session. These are the exact session IDs behind the acceptance table, recorded on 2026-11-02 (times Asia/Shanghai) in
Prior round (the earlier single-session evidence, recorded 18:08 in Full index with the same table: zcode-session-index.md. To inspect locally, ZCode stores sessions in Two caveats: session IDs are local to the machine that ran the test ( |
|
@jacksonliuyn the ZCode verification you asked for is in the two comments above; linking them here so they are easy to find:
Short version: yes, the full chain passes in ZCode CLI 0.17.1 on macOS 27.0.1 (26A434, arm64) — plugin installation → Skill invocation → recording → video generation → verification, plus a Linux container run. Install/update/disable/enable/uninstall and the failure paths (missing ffmpeg, tampered artifact, Not verified: install/update from the public marketplace index, Windows, the desktop GUI flow, an independent No code changes were needed; the head is still |
Case-driven delivery demo (1.0.6 guidance, real run)1.0.6 adds optional case-driven recording guidance: when a video covers two or more acceptance cases, write the case list first, record one real run per case, and give each case its own result segment. It is guidance only — no new gate. We exercised this flow on a real local project (an after-sales ticket handover desk, iterated v1 → v2: bulk assignment, closed-ticket guard, audit log) and recorded two acceptance cases with the installed plugin:
Both renders passed the built-in delivery gate at score 100 and Videos and the case list (prerelease release assets, not committed to git):
Both SHA256 values were re-verified against the downloaded release assets. Demo data is synthetic; no independent visual critic was run for these two renders. |
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
a7307ec to
37ee64c
Compare
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
1.1.1 feedback report videos — ZCode verification (2026-11-04)Updated this PR in place to 1.1.1: full payload re-sync, Environment: ZCode CLI 0.17.1, macOS arm64, Python 3.14.4, FFmpeg 8.1.1. Installed with
In both sessions the agent resolved the scripts from the installed cache Feedback artifacts: Repo-side gates for the same commit: Not covered: uploading/publishing to a customer channel (none was performed), external |
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
ca69516 to
f1e0601
Compare
f1e0601 to
a3377d7
Compare
Third round — review & simplification (2026-11-05)No new features. Most of what this round fixed is documentation that described behaviour the code does not have — worth flagging because a reviewer reading only the docs would draw the wrong conclusions:
Also removed: the Verification: 18 Python + 3 Node tests pass (re-run), Payload: re-synced, 18 files, verified blob-by-blob against Not verified this round: no new recording was produced; the quality-gate logic and thresholds are byte-identical to 1.1.2 as reviewed above. |
A real render was run on 1.1.2 (2026-11-05)Not a new recording for delivery, but a live exercise of the whole pipeline with a genuine 1920×1080@30 take captured from a real browser session:
The gate rejected my footage, correctly. Four attempts against the same static source:
What this confirms: the render path, TTS, subtitle timing and both gate implementations run end-to-end on 1.1.2; the hard-failure thresholds fire and are not advisory. What it does not confirm: a passing sample on 1.1.2. Producing one needs footage with sustained motion (real interaction, scrolling, typing), which this round did not have. |
|
✅ Internal validation passed. A ZCode maintainer will review the change next; after acceptance, release checks run separately and this PR is closed automatically. |
a3377d7 to
ac6daaf
Compare
|
Re-pushed this branch as 1. The PR now follows 2. The entry named the repo-root Re-verified against the branch itself (not a local checkout):
Request: the earlier 中文摘要:本轮只补元数据与载荷完整性 —— 恢复了模板要求的插件类型声明与自查清单、标题加上 |
Plugin type (required) / 插件类型(必填)
Affected plugin name(s) / 涉及的插件名称:
delivery-videoWhat / 改动内容
EN — Adds
delivery-video, which turns a real screen recording of working software into a delivery video with narration, word-synced subtitles and a deterministic quality gate — plus a feedback mode that binds a defect report to one specific cut of that video.preflight.py/render.py/verify_delivery.pyare real commands; the gate rejects truncated video, stripped audio, a re-encoded resolution, and a tampered or missing feedback report.feedback.jsonfields plus the exact markdown bytes are written intoquality_gate/report.jsonand recomputed byverify_delivery.py.DELIVERY=PASS.中文 — 新增
delivery-video:把真实录屏做成带旁白、逐字字幕与确定性质量闸的交付视频,并支持把缺陷反馈绑定到某个具体成片。1.1.2 把反馈报告的内容哈希与 markdown 字节哈希纳入证据链,改标题/改正文/删 markdown 都会判 FAIL。Visible behavior and testing / 可见行为与测试
python3 -m unittest discover -s tests→ 18 passed;node --test tests/test_plugin.mjs→ 3 passed (Python 3.14.4, Node 24.15.0).tests/test_feedback_binding.py: clean pass, editedissue.title, editedissue.actual, edited markdown, deleted markdown — the last four exit non-zero with a specific message instead ofDELIVERY=PASS.Not covered by this round / 本轮未覆盖:
Version and marketplace registration / 版本与市场注册
external_plugins/delivery-video/(18 files, 128 KiB), manifest andmarketplace.jsonentry both at 1.1.2.productivity; icon added at the repo-root path the entry uses (assets/delivery-video/icon.png, 512×512).description_i18ncarriesenandzh-CN, identical in both files;README.md/README_CN.mdboth present.Dependencies, network, permissions, side effects / 依赖、网络、权限与副作用
ffmpeg/ffprobefor rendering and the video checks.edge-ttsfor narration uses Microsoft's public TTS endpoint over the network — this is the only network dependency, and it is documented in both READMEs.quality_gate/report.json,feedback.json,feedback.generated.mdand evidence files under the project's output directory.Licensing / provenance / 许可与来源
LICENSEships in the plugin directory); code written for this submission.edge-tts(MIT) at run time; no voice assets are redistributed.Self-check / 自查清单
external_plugins/delivery-video/; existing fixes retain ownership / 插件名唯一且 kebab-case,外部新增插件位于external_plugins/delivery-video/,已有插件修复保留原归属categoryis set independently of publisher ownership / 用途分类category已填写,与维护归属分开:productivityversionbumped in BOTHplugin.jsonandmarketplace.json(artifacts are immutable) /plugin.json与marketplace.json两处版本号已同步升级:1.1.2README.md(EN) andREADME_CN.md(中文) both updated and equivalent / 中英文 README 均已更新且语义一致description_i18nhas bothenandzh-CN/description_i18n包含en与zh-CNpython3 scripts/validate.pypasses on this branch, and the plugin artifact was built through the samebuild_zip()code path CI uses (size and sha256 in the comment below). The fullpython3 scripts/build_dist.pyneeds the repo's UI-pluginpnpm buildand fails identically on upstreammain(excalidraw: incomplete build), so CI runs it here — see the note above. / 本地validate.py通过,本插件产物用 CI 同一条build_zip()路径打出(体积与 sha256 见下方评论);完整build_dist.py依赖仓库自带 UI 插件的pnpm build,在main上同样失败,交由 CI 执行