Commit Graph
5 Commits
Author SHA1 Message Date
tt-a1iandtt-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
xronocode d367294bac fix: eliminate DRY violations in common-bugs-checklist, fix stale CONTRIBUTING.md, improve pr-analyzer.py
- Refactor common-bugs-checklist.md from 1227 lines to 217 lines:
  replace duplicated code examples with concise checklist items
  linking to the dedicated language-specific guides
- Fix CONTRIBUTING.md: stale directory name (ai-code-review-guide ->
  code-review-skill), missing files (go.md, qt.md, css-less-sass.md,
  architecture-review-guide.md, performance-review-guide.md)
- Fix pr-analyzer.py detect_language(): use os.path.splitext instead of
  fragile .endswith(), add missing extensions (.vue, .kt, .swift, .php,
  .cs, .svelte, .zig, .ex, .scala, .lua)
- Fix pr-analyzer.py generate_suggestions(): replace fragile string
  matching on risk messages with RISK_NO_TESTS constant
2026-05-08 16:09:36 +06:00
seekskyworld 3a76efb984 feat: add C/C++ guides and update indexes 2026-01-11 21:00:07 +08:00
Tu Shaokun 53c5f735f5 Initial commit: AI Code Review Guide skill
- Comprehensive code review skill for Claude Code
- Support for React 19, Vue 3, Rust, TypeScript
- TanStack Query v5, Suspense & Streaming patterns
- ~3000 lines of review guidelines and checklists
2025-11-29 10:06:40 +08:00