docs: remove stale translated docs for retired commands#2534
Conversation
11 commands (tdd, eval, verify, e2e, docs, claw, context-budget,
devfleet, orchestrate, prompt-optimize, rules-distill) were retired to
legacy-command-shims/ in favor of skills, but their translated
per-command doc pages were still committed under
docs/{ja-JP,zh-CN,zh-TW,ko-KR,pt-BR,es,tr}/commands/. Removed the 47
stale files (count differs per locale — not all locales had all 11
translated).
docs/ja-JP/commands/README.md also listed 4 of them directly; updated
those bullets to point at the replacement skills instead.
📝 WalkthroughSummary by CodeRabbit
WalkthroughLocalized documentation for legacy commands was removed across Spanish, Japanese, Korean, Portuguese, Turkish, Simplified Chinese, and Traditional Chinese. The Japanese command index now points readers from deprecated commands to replacement skills and workflows. ChangesLocalized command documentation migration
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/ja-JP/commands/README.md`:
- Line 53: Update the command example in the documentation so it no longer
replaces /tdd with the unrelated /test-coverage command; document the
tdd-workflow skill instead, or explicitly revise the example to describe
coverage inspection.
- Around line 18-22: Update the retired-command references in the README,
including the repeated entries at the other listed locations, to use Markdown
links instead of plain code spans. Link tdd-workflow, e2e-testing,
verification-loop, dmux-workflows, and autonomous-agent-harness to their
corresponding skills/*/SKILL.md documentation while preserving the existing
Japanese descriptions and command names.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: fd3bdfda-f9d4-4c1f-8a12-3bc3d54e3798
📒 Files selected for processing (48)
docs/es/commands/e2e.mddocs/es/commands/eval.mddocs/es/commands/orchestrate.mddocs/es/commands/tdd.mddocs/es/commands/verify.mddocs/ja-JP/commands/README.mddocs/ja-JP/commands/claw.mddocs/ja-JP/commands/context-budget.mddocs/ja-JP/commands/devfleet.mddocs/ja-JP/commands/docs.mddocs/ja-JP/commands/e2e.mddocs/ja-JP/commands/eval.mddocs/ja-JP/commands/orchestrate.mddocs/ja-JP/commands/prompt-optimize.mddocs/ja-JP/commands/rules-distill.mddocs/ja-JP/commands/tdd.mddocs/ja-JP/commands/verify.mddocs/ko-KR/commands/e2e.mddocs/ko-KR/commands/eval.mddocs/ko-KR/commands/orchestrate.mddocs/ko-KR/commands/tdd.mddocs/ko-KR/commands/verify.mddocs/pt-BR/commands/e2e.mddocs/pt-BR/commands/eval.mddocs/pt-BR/commands/orchestrate.mddocs/pt-BR/commands/tdd.mddocs/pt-BR/commands/verify.mddocs/tr/commands/e2e.mddocs/tr/commands/eval.mddocs/tr/commands/orchestrate.mddocs/tr/commands/tdd.mddocs/tr/commands/verify.mddocs/zh-CN/commands/claw.mddocs/zh-CN/commands/context-budget.mddocs/zh-CN/commands/devfleet.mddocs/zh-CN/commands/docs.mddocs/zh-CN/commands/e2e.mddocs/zh-CN/commands/eval.mddocs/zh-CN/commands/orchestrate.mddocs/zh-CN/commands/prompt-optimize.mddocs/zh-CN/commands/rules-distill.mddocs/zh-CN/commands/tdd.mddocs/zh-CN/commands/verify.mddocs/zh-TW/commands/e2e.mddocs/zh-TW/commands/eval.mddocs/zh-TW/commands/orchestrate.mddocs/zh-TW/commands/tdd.mddocs/zh-TW/commands/verify.md
💤 Files with no reviewable changes (47)
- docs/pt-BR/commands/verify.md
- docs/ko-KR/commands/verify.md
- docs/ko-KR/commands/eval.md
- docs/ja-JP/commands/orchestrate.md
- docs/es/commands/eval.md
- docs/pt-BR/commands/e2e.md
- docs/ja-JP/commands/prompt-optimize.md
- docs/zh-CN/commands/orchestrate.md
- docs/ja-JP/commands/docs.md
- docs/tr/commands/verify.md
- docs/ja-JP/commands/devfleet.md
- docs/ko-KR/commands/orchestrate.md
- docs/zh-CN/commands/eval.md
- docs/ja-JP/commands/claw.md
- docs/ja-JP/commands/eval.md
- docs/ko-KR/commands/tdd.md
- docs/ja-JP/commands/context-budget.md
- docs/zh-CN/commands/rules-distill.md
- docs/ja-JP/commands/verify.md
- docs/zh-CN/commands/verify.md
- docs/zh-TW/commands/eval.md
- docs/es/commands/e2e.md
- docs/es/commands/verify.md
- docs/ja-JP/commands/rules-distill.md
- docs/zh-CN/commands/claw.md
- docs/zh-TW/commands/tdd.md
- docs/pt-BR/commands/eval.md
- docs/es/commands/tdd.md
- docs/es/commands/orchestrate.md
- docs/zh-CN/commands/devfleet.md
- docs/zh-TW/commands/orchestrate.md
- docs/zh-CN/commands/tdd.md
- docs/zh-TW/commands/e2e.md
- docs/ja-JP/commands/tdd.md
- docs/zh-CN/commands/context-budget.md
- docs/ja-JP/commands/e2e.md
- docs/pt-BR/commands/orchestrate.md
- docs/tr/commands/orchestrate.md
- docs/zh-CN/commands/prompt-optimize.md
- docs/zh-TW/commands/verify.md
- docs/pt-BR/commands/tdd.md
- docs/tr/commands/tdd.md
- docs/zh-CN/commands/docs.md
- docs/zh-CN/commands/e2e.md
- docs/ko-KR/commands/e2e.md
- docs/tr/commands/eval.md
- docs/tr/commands/e2e.md
📜 Review details
⏰ Context from checks skipped due to timeout. (1)
- GitHub Check: Greptile Review
🧰 Additional context used
📓 Path-based instructions (3)
docs/ja-JP/commands/**/*.md
📄 CodeRabbit inference engine (docs/ja-JP/CLAUDE.md)
Format commands as Markdown with description frontmatter
Files:
docs/ja-JP/commands/README.md
docs/ja-JP/**/*.{md,json}
📄 CodeRabbit inference engine (docs/ja-JP/CLAUDE.md)
Use lowercase hyphen-separated naming convention for files (e.g.,
python-reviewer.md,tdd-workflow.md)
Files:
docs/ja-JP/commands/README.md
docs/ja-JP/**/README.md
📄 CodeRabbit inference engine (docs/ja-JP/CLAUDE.md)
Use
/readmeskill when working with README.md file
Files:
docs/ja-JP/commands/README.md
| - `/go-test`, `/kotlin-test`, `/rust-test`, `/cpp-test`, `/flutter-test`, `/react-test` - 言語別 TDD ワークフロー | ||
| - `/test-coverage` - テストカバレッジを確認 | ||
| - `/verify` - 実装を検証 | ||
| - （`/tdd` は廃止 → `tdd-workflow` スキルを使用） | ||
| - （`/e2e` は廃止 → `e2e-testing` スキルを使用） | ||
| - （`/verify` は廃止 → `verification-loop` スキルを使用） |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Link the replacement skills instead of displaying plain text.
The retired commands currently point to non-clickable code spans, so readers cannot navigate to the canonical documentation. Add Markdown links to skills/tdd-workflow/SKILL.md, skills/e2e-testing/SKILL.md, skills/verification-loop/SKILL.md, skills/dmux-workflows/SKILL.md, and skills/autonomous-agent-harness/SKILL.md at each listed reference.
Also applies to: 37-37, 69-69, 72-72, 76-76
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/ja-JP/commands/README.md` around lines 18 - 22, Update the
retired-command references in the README, including the repeated entries at the
other listed locations, to use Markdown links instead of plain code spans. Link
tdd-workflow, e2e-testing, verification-loop, dmux-workflows, and
autonomous-agent-harness to their corresponding skills/*/SKILL.md documentation
while preserving the existing Japanese descriptions and command names.
| ```bash | ||
| /plan | ||
| /tdd | ||
| /test-coverage |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Do not replace /tdd with the unrelated /test-coverage command.
The documented replacement for /tdd is the tdd-workflow skill, while /test-coverage changes the example from test-driven development to coverage inspection. Either document tdd-workflow in this example or explicitly revise the example’s purpose.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/ja-JP/commands/README.md` at line 53, Update the command example in the
documentation so it no longer replaces /tdd with the unrelated /test-coverage
command; document the tdd-workflow skill instead, or explicitly revise the
example to describe coverage inspection.
|
| Filename | Overview |
|---|---|
| docs/ja-JP/commands/README.md | Commands README updated to replace retired-command bullets with skill-redirect notices; replacement skill names look correct. |
| docs/es/commands/e2e.md | Deleted — stale Spanish translation of the retired /e2e command doc. |
| docs/zh-CN/commands/claw.md | Deleted — stale Simplified Chinese translation of the retired /claw command doc. |
| docs/ja-JP/commands/docs.md | Deleted — stale Japanese translation of the retired /docs command doc. |
| docs/ja-JP/commands/orchestrate.md | Deleted — stale Japanese translation of the retired /orchestrate command doc. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Retired Commands] --> B[legacy-command-shims/\nshim wrappers retained]
C[Translated per-command docs\n47 files across 7 locales] -->|DELETED by this PR| D[Files removed]
E[docs/ja-JP/commands/README.md] -->|4 bullets updated| F[Redirect to replacement skills\ntdd-workflow, e2e-testing\nverification-loop, dmux-workflows]
G[Locale-level README.md and CONTRIBUTING.md] -->|NOT updated| H[Stale directory tree entries\nstill mention deleted files]
style D fill:#f66,color:#fff
style F fill:#6b6,color:#fff
style H fill:#fa6,color:#000
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[Retired Commands] --> B[legacy-command-shims/\nshim wrappers retained]
C[Translated per-command docs\n47 files across 7 locales] -->|DELETED by this PR| D[Files removed]
E[docs/ja-JP/commands/README.md] -->|4 bullets updated| F[Redirect to replacement skills\ntdd-workflow, e2e-testing\nverification-loop, dmux-workflows]
G[Locale-level README.md and CONTRIBUTING.md] -->|NOT updated| H[Stale directory tree entries\nstill mention deleted files]
style D fill:#f66,color:#fff
style F fill:#6b6,color:#fff
style H fill:#fa6,color:#000
Reviews (1): Last reviewed commit: "docs: remove stale translated docs for r..." | Re-trigger Greptile
daltino
left a comment
daltino
left a comment
There was a problem hiding this comment.
DECISION: APPROVE
This PR removes stale, translated documentation for retired commands, which aligns with maintaining an up-to-date and relevant documentation set. The changes appear straightforward, consisting only of file deletions, and comply with the contribution guidelines under "Cross-Harness and Translations" about maintaining relevant content. No further action is needed.
2 participants
Summary
/tdd,/eval,/verify,/e2e,/docs,/claw,/context-budget,/devfleet,/orchestrate,/prompt-optimize,/rules-distill) were retired tolegacy-command-shims/in favor of skills, but their translated per-command doc pages were still committed underdocs/{ja-JP,zh-CN,zh-TW,ko-KR,pt-BR,es,tr}/commands/.docs/ja-JP/commands/README.mdalso listed 4 of them directly; updated those bullets to point at the replacement skills instead.Split out of #2532 into its own PR (COMMANDS-QUICK-REF.md fix moved to a separate PR: #2533).
Test plan
node tests/lib/locale-install.test.js— 7/7 pass (translated-docs install paths unaffected by removing these stale files)node tests/ci/catalog.test.js— 4/4 passnode tests/run-all.js— 2688/2690 pass (2 pre-existing failures unrelated to this change — from untracked local scaffold dirs under.agents/skills/source-command-*)