3 Commits

Author SHA1 Message Date
tt-a1i 28e918267d docs: expand review guides and sync release metadata (#21)
Co-authored-by: tt-a1i <tt-a1i@users.noreply.github.com>
2026-06-28 17:16:10 +08:00
tt-a1i 17267ff26b fix: correct review-guide errors and sync skill metadata (#16)
Correctness fixes in the reference guides:
- django: len()/count() caching explanation was reversed; fix slicing/index
  cache example; replace nonexistent __acall__ async middleware with the
  documented markcoroutinefunction pattern; drop removed
  SECURE_BROWSER_XSS_FILTER; REFERRER_POLICY -> SECURE_REFERRER_POLICY
- security: path-traversal guard compared a relative path to an absolute one
  (rejected every valid file); compare absolute-vs-absolute
- nestjs: e2e ValidationPipe needs forbidNonWhitelisted/transform for the
  "extra field -> 400" test to pass
- rust: select! cancel-safety example used read on both sides; bad case now
  uses non-cancel-safe read_exact
- code-quality: "read less" example still read the whole file; use readline()
- svelte: drop nonexistent unstate() (use $state.snapshot); fix devalue/Date
  note; comma-operator each-key -> template literal
- c/cpp/qt: restore mangled markers that rendered as a literal ?
- csharp: drop fabricated perf numbers
- java: scope HashMap infinite-loop note to Java 7 and earlier
- kotlin: closeableScope -> built-in viewModelScope
- css: deprecated darken()/clip:rect() -> color.adjust/clip-path
- typescript: legacy .eslintrc -> flat config (typescript-eslint v8)

Tooling and metadata:
- pr-analyzer.py: filename regex corrupted lib//web//db/ paths; parse the diff
  header via backreference and add utf-8/error handling; add test
- SKILL.md: canonical name code-review-skill; document severity tiers; wire in
  pr-analyzer.py
- README/CONTRIBUTING: fix stale skill name and line counts; complete the guide
  tree; add a conventions section
2026-06-05 09:45:40 +08:00
seeksky c687f500b7 feat: add C and C++ code review guides 2026-01-11 17:54:54 +08:00