zzos/code-review-skill
1
0
Fork 0
mirror of https://github.com/awesome-skills/code-review-skill.git synced 2026-09-28 15:04:40 +08:00
Code Issues Packages Projects Releases Wiki Activity
55 Commits 6 Branches 0 Tags
main
Commit Graph
4 Commits
Author SHA1 Message Date
tt-a1iCursortt-a1i
 		f2fd4e57d9 feat: add Java 8 / Spring Boot 2 legacy review guide (#24) 
* feat: add Java 8 / Spring Boot 2 legacy review guide

Support reviewing long-lived Java 8 and Boot 2 (javax.*) codebases without
forcing Java 17/21 patterns. Closes #23.

Co-authored-by: Cursor <cursoragent@cursor.com>

* docs: rewrite Java 8 legacy guide in English

Make the guide accessible to international contributors; keep examples
and review checklists in English like other language guides (e.g. php.md).

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: tt-a1i <tt-a1i@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
 2026-07-09 14:50:14 +08:00
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
google-labs-jules[bot] e72fafef25 feat: Add Java 17+ and Spring Boot 3 code review guide 
- Create `reference/java.md` with guidelines for modern Java (Records, Switch, Text Blocks), Spring Boot 3 (DI, Configuration), JPA (Performance, N+1), and Concurrency (Virtual Threads).
- Update `SKILL.md` to register the Java guide for on-demand loading.
- Update `README.md` to include Java in supported languages and key topics.
 2025-12-04 14:50:08 +00:00