Squashed 'docs/standards/playbook/' changes from e504a68..b529012

b529012 ✅ test(tests): avoid hardcoded missing path 07583c8 🐛 fix(tests): make doc link check awk-portable 6244aa3 🔧 chore(ci): relax gitattributes check 31dd0c5 🐛 fix(scripts): require existing project root 15d4d63 🔧 chore(ci): install python3-pip 90c6313 🔧 chore(ci): run tests in a single job d84eff0 🔧 chore(ci): make actions base url configurable 395598d 🔧 chore(ci): fix yaml step names 4ae2733 ✨ feat(sync_standards): auto-detect existing languages e97fb00 ✨ feat(sync_standards): default gitattributes to append da0ef2b ✅ test: add automated tests and ci workflow 99bef30 🎨 style(markdown): format all markdown files with prettier 7e96bc8 ♻️ refactor(tsl): split function.md into 44 modular files 0d6b2a0 📝 docs(readme): add quick decision table and TL;DR for distribution methods 3b2188f 🎨 style(markdown): format markdown files with prettier 41fc43b 📝 docs(tsl): document {Unit.}Type source annotation 3dceaf7 📝 docs(tsl): clarify tsf-only top-level rules and type annotations 3a63829 📝 docs(skills): add create-plan skill f02a707 🐛 fix(scripts): escape markdown backticks in vendor_playbook 064aa92 🐛 fix(scripts): correct bat scripts 4881feb 🔧 chore(gitattributes): enforce crlf for bat files 1fa3e2a 🐛 fix(scripts): escape parentheses in sync_standards output 3958cad 📝 docs(vendor_playbook): mention ci templates 9d059cf ✨ feat(templates): add gitea ci example a52bb24 📝 docs(codex_skills): normalize markdown formatting cc8ad4c 📝 docs(skills): slim commit-message 2a98e15 🐛 fix(skills): quote YAML descriptions 8f78d22 🐛 fix(sync_standards): generate minimal AGENTS.md 5547665 ✨ feat(skills): add commit-message suggestion skill 3fe8bd7 🔧 chore(sync_standards): create AGENTS.md on sync 283d311 ✨ feat(playbook): add syntax book and codex skills tooling 5b97ed5 📝 docs(tsl): clarify syntax references 27e0700 ✨ feat(skills): add pdf/docx/pptx/xlsx wrapper workflows 5551363 ✨ feat(skills): add debugging and bulk refactor workflows 1d4f548 ✨ feat(skills): add built-in workflow skills c0895dd 📝 docs(skills): add anthropics document-skills integration cf9f80d 🔧 chore(playbook): add Claude Code skills guide and align python templates git-subtree-dir: docs/standards/playbook git-subtree-split: b529012c59c5d67c3073884d7b617763647417fd
2026-01-08 15:56:36 +08:00
parent eada742d75
commit ac794a6a70
113 changed files with 254751 additions and 176 deletions
@@ -0,0 +1,64 @@
+---
+name: code-review-workflow
+description:
+  "Structured expert code review for TSL/C++/Python diffs or patches. Triggers:
+  code review, review PR, diff, 评审, 审查, 安全评审, 性能评审."
+---
+
+# Code Review Workflow
+
+## When to Use This Skill
+
+- Review a PR / `git diff` / patch
+- Pre-merge quality gate (correctness/security/perf/tests)
+- Risky refactor, behavior change, auth/data path changes
+
+## Inputs (required)
+
+- Change set: PR link or `git diff ...` output (must include context)
+- Goal: expected behavior / acceptance criteria (1–3 sentences)
+- Risk level: low|med|high (default: med)
+- Verification: test commands / repro steps (if unknown, ask first)
+
+## Procedure
+
+1. **Triage**
+   - Identify touched areas, public APIs, behavior changes, data/auth paths
+   - Classify risk (blast radius, rollback difficulty)
+
+2. **Correctness**
+   - Invariants, edge cases, error handling, null/empty, concurrency
+   - Backward compatibility (inputs/outputs, wire formats, config)
+
+3. **Security**
+   - AuthZ/AuthN boundaries, least privilege
+   - Input validation, injection surfaces, secrets/log redaction
+
+4. **Maintainability**
+   - Naming/structure/style aligned with Playbook docs
+   - Complexity hotspots, duplication, clarity of intent
+
+5. **Performance**
+   - Hot paths, algorithmic complexity, allocations/IO, N+1 patterns
+
+6. **Tests & Verification**
+   - Map changes → tests; identify missing coverage
+   - Provide minimal verification plan (commands + expected signals)
+
+## Review Standards (Playbook as authority)
+
+- Commit message: `docs/common/commit_message.md`
+- TSL: `docs/tsl/code_style.md`, `docs/tsl/naming.md`, `docs/tsl/toolchain.md`
+- C++: `docs/cpp/code_style.md`, `docs/cpp/naming.md`, `docs/cpp/toolchain.md`
+- Python: `docs/python/style_guide.md`, `docs/python/tooling.md`,
+  `docs/python/configuration.md`
+
+## Output Contract (stable)
+
+- Summary: what changed & why
+- Risk: low|med|high + reasoning
+- Blockers: must-fix before merge (with file/line references when possible)
+- Non-blocking: Major / Minor / Nit
+- Questions: missing context / assumptions
+- Suggested verification: exact commands + what success looks like
+- Optional patch: minimal diff-style suggestions (only when unambiguous)