docs(onboarding): 将 WorkBuddy 列为受支持的 Agent Skill - #178
Conversation
tt-a1i
left a comment
There was a problem hiding this comment.
Thanks for adding this integration. The scope is appropriately narrow, and the proposed manual-install model is reasonable: Tencent documentation identifies .workbuddy/skills/ as WorkBuddy's Skill directory, while this PR keeps WorkBuddy out of the generated agent switcher and reuses the existing portable Skill.
I verified the following at this exact head:
- the branch is current with
main; git diff --checkpasses;- the two targeted test files pass (6/6);
docs/start.htmlis reproducible fromscripts/start-template.html;- the full local suite completes with 720 passed and 30 environment-gated skips.
However, listing a platform as supported is a stronger claim than showing that its documented directory convention is compatible. I need the following evidence and contract coverage before this can merge:
-
Add a real WorkBuddy installation and invocation receipt. Please install the complete
archify/directory into a fresh or clearly identified WorkBuddy environment using the exact documented path, reload/restart WorkBuddy, show thatarchifyis discovered, and invoke it to produce and validate at least one minimal diagram. Record the WorkBuddy version, OS, installed path, prompt/command, and Archify success receipt. The check must not resolve files or dependencies from the repository checkout. -
Make the tests cover the contract introduced by this PR. The current
cursor-onboarding.test.mjschecks that the READMEs and landing metadata mention WorkBuddy, but it never asserts that either Start surface contains the WorkBuddy instructions or the exact~/.workbuddy/skills/archifydestination. It also checks that the four existing buttons exist, but does not prove thatdata-agent="workbuddy"is absent or that the switcher remains exactly four agents. Please add assertions for these behaviors on both the canonical template and generated page. As written, the new WorkBuddy paragraph could be removed or the switcher boundary could regress while the claimed coverage still passes. -
Run the repository-required CI on the resulting exact head. This PR currently has zero attached checks. Local tests are useful evidence, but they do not replace the protected Node/package lanes or any environment-gated checks.
No archify.zip rebuild is needed for this diff because the packaged runtime is unchanged. Once the real WorkBuddy receipt, exact contract tests, and required green CI are attached to the same head, this should be straightforward to re-review.
e14f4f2 to
70967bd
Compare
Mirrors the Raven path: WorkBuddy is installed by copying archify/ into ~/.workbuddy/skills/archify, so it is documented in README, the start page, and the landing meta description, but is intentionally not added as a generated agent-switcher command or a data-agent tab. Updates the bilingual onboarding assertions to match the new agent list.
…boundary The previous change documented WorkBuddy on the Start surface, but nothing asserted that documentation: the bilingual WorkBuddy paragraph and the exact ~/.workbuddy/skills/archify destination could be deleted, or the switcher could grow a fifth agent, without any test failing. Adds assertions that run against both the canonical template and the checked-in generated page: - the WorkBuddy manual-install instruction stays present in English and Chinese - ~/.workbuddy/skills/archify appears in the English copy, the Chinese copy, and the visible text - WorkBuddy never appears as data-agent="workbuddy" and the switcher stays exactly four agents, both as tabs and in KNOWN_AGENTS - no surface claims a generated --agent workbuddy command, so listing WorkBuddy does not overstate what the agent switcher supports Verified by mutation: deleting the paragraph, adding a WorkBuddy tab, appending to KNOWN_AGENTS, and claiming a switcher command each turn the suite red.
70967bd to
e3d3687
Compare
|
Thanks for the detailed review. All three points are addressed on the new head \e3d3687. 1. Real WorkBuddy installation & invocation receipt
2. Contract coverage (commit e3d3687) \cursor-onboarding.test.mjs\ now pins, on both \scripts/start-template.html\ and the checked-in \docs/start.html:
Mutation-verified: deleting the paragraph, adding a WorkBuddy tab, appending to \KNOWN_AGENTS, and claiming a switcher command each turn the suite red. 3. CI on the exact head The branch was rebased onto current \main\ (\4ac500a, PR is mergeable) and force-pushed as \e3d3687. The \CI\ workflow run (id 33296592469) is queued on this head and awaiting approval for the fork PR - please approve so the Node 18/20/22/24 matrix, WebM/Chrome lanes, zip-freshness, and package smoke can attach to this head. Local verification on this Windows host: the two targeted test files pass 6/7 - the only local failure is the \unzip-dependent archive test (no POSIX \unzip\ on Windows), which passes in the ubuntu CI lane. A full local suite run on the exact head (1009 tests: 949 pass, 27 fail, 33 skip) confines every remaining failure to Windows-environment-sensitive lanes - symlink creation (EPERM without Developer Mode), \unzip, Node \kill('SIGTERM')\ semantics (the \cli: preview . exits cleanly\ assertion fails identically on \main\ on this host), git filemode/index handling, and git-spawn tests when \git\ is absent from PATH. None of these are touched by this diff; the onboarding/contract tests added here pass. |
|
你好我已经根据你给出的简易修改并上传 等你 还有哪里需要修改再联系
原始邮件
发件人:tt-a1i ***@***.***>
发件时间:2026年8月30日 00:39
收件人:tt-a1i/archify ***@***.***>
抄送:Tearimi ***@***.***>, Author ***@***.***>
主题:Re: [tt-a1i/archify] docs(onboarding): 将 WorkBuddy 列为受支持的 Agent Skill (PR #178)
@tt-a1i requested changes on this pull request.
Thanks for adding this integration. The scope is appropriately narrow, and the proposed manual-install model is reasonable: Tencent documentation identifies .workbuddy/skills/ as WorkBuddy's Skill directory, while this PR keeps WorkBuddy out of the generated agent switcher and reuses the existing portable Skill.
I verified the following at this exact head:
the branch is current with main;
git diff --check passes;
the two targeted test files pass (6/6);
docs/start.html is reproducible from scripts/start-template.html;
the full local suite completes with 720 passed and 30 environment-gated skips.
However, listing a platform as supported is a stronger claim than showing that its documented directory convention is compatible. I need the following evidence and contract coverage before this can merge:
Add a real WorkBuddy installation and invocation receipt. Please install the complete archify/ directory into a fresh or clearly identified WorkBuddy environment using the exact documented path, reload/restart WorkBuddy, show that archify is discovered, and invoke it to produce and validate at least one minimal diagram. Record the WorkBuddy version, OS, installed path, prompt/command, and Archify success receipt. The check must not resolve files or dependencies from the repository checkout.
Make the tests cover the contract introduced by this PR. The current cursor-onboarding.test.mjs checks that the READMEs and landing metadata mention WorkBuddy, but it never asserts that either Start surface contains the WorkBuddy instructions or the exact ~/.workbuddy/skills/archify destination. It also checks that the four existing buttons exist, but does not prove that data-agent="workbuddy" is absent or that the switcher remains exactly four agents. Please add assertions for these behaviors on both the canonical template and generated page. As written, the new WorkBuddy paragraph could be removed or the switcher boundary could regress while the claimed coverage still passes.
Run the repository-required CI on the resulting exact head. This PR currently has zero attached checks. Local tests are useful evidence, but they do not replace the protected Node/package lanes or any environment-gated checks.
No archify.zip rebuild is needed for this diff because the packaged runtime is unchanged. Once the real WorkBuddy receipt, exact contract tests, and required green CI are attached to the same head, this should be straightforward to re-review.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you authored the thread.
|
Problem
Archify 官方列出的受支持 Agent 为 Cursor、Claude Code、Codex CLI、OpenCode(中文 README 另含 Raven)。WorkBuddy 可以运行该 Skill(复制 archify/ 目录到 ~/.workbuddy/skills/archify 即可),但 README、落地页与 Start 页均未将其列为受支持 Agent,WorkBuddy 用户找不到对应的安装说明。
Solution
参照 Raven 的手动安装思路,为 WorkBuddy 补充「手动复制安装」说明,不新增 Agent 切换器命令或标签页,保持「同一份 Skill、零依赖渲染器」的交付一致性;同时同步更新三语 README、落地页 meta 描述与对应断言测试。
Changes
Testing
Notes for Reviewer