Files

Luong NGUYEN c7261394b0 fix(i18n): re-point translations to code-review-specialist rename (#128 )

* fix(i18n): re-point translations to code-review-specialist rename

The English code-review skill was renamed to code-review-specialist in #127
to avoid shadowing the new built-in /code-review command. Mirror that rename
across the uk, vi, ja, zh locales: rename each 03-skills/code-review/ directory
to code-review-specialist/, update the i18n-source markers in the moved ja
files, and fix path references in INDEX.md, QUICK_REFERENCE.md, and the vi
TRANSLATION_QUEUE.md.

* chore: ignore .asm-improver and .gitissue tooling state

* fix(i18n): re-point remaining code-review install paths to code-review-specialist

2026-05-26 10:29:51 +02:00

3.9 KiB

Raw Permalink Blame History

Шаблон знахідки код-рев'ю

Використовуйте цей шаблон для документування кожної знайденої проблеми під час код-рев'ю.

Проблема: [НАЗВА]

Серйозність

Критична (блокує деплой)
Висока (потрібно виправити перед мерджем)
Середня (потрібно виправити незабаром)
Низька (бажано виправити)

Категорія

Безпека
Продуктивність
Якість коду
Підтримуваність
Тестування
Патерн проєктування
Документація

Розташування

Файл: src/components/UserCard.tsx

Рядки: 45-52

Функція/Метод: renderUserDetails()

Опис проблеми

Що: Опишіть суть проблеми.

Чому це важливо: Поясніть вплив та чому це потрібно виправити.

Поточна поведінка: Покажіть проблемний код або поведінку.

Очікувана поведінка: Опишіть, що повинно відбуватися замість цього.

Приклад коду

Поточний (проблемний)

// Shows the N+1 query problem
const users = fetchUsers();
users.forEach(user => {
  const posts = fetchUserPosts(user.id); // Query per user!
  renderUserPosts(posts);
});

Запропоноване виправлення

// Optimized with JOIN query
const usersWithPosts = fetchUsersWithPosts();
usersWithPosts.forEach(({ user, posts }) => {
  renderUserPosts(posts);
});

Аналіз впливу

Аспект	Вплив	Серйозність
Продуктивність	100+ запитів для 20 користувачів	Висока
Досвід користувача	Повільне завантаження сторінки	Висока
Масштабованість	Ламається при масштабуванні	Критична
Підтримуваність	Складно дебажити	Середня

Пов'язані проблеми

Аналогічна проблема в AdminUserList.tsx рядок 120
Пов'язаний PR: #456
Пов'язана issue: #789

Додаткові ресурси

Нотатки рецензента

Це поширений патерн у цій кодовій базі
Варто додати це до гайду зі стилю коду
Можливо, варто створити допоміжну функцію

Відповідь автора (для зворотного зв'язку)

Заповнюється автором коду:

Виправлення реалізовано в коміті: abc123
Статус виправлення: Завершено / В процесі / Потребує обговорення
Питання або зауваження: (опишіть)

Статистика знахідок (для рецензента)

При рев'ю кількох знахідок, відстежуйте:

Всього знайдено проблем: X
Критичних: X
Високих: X
Середніх: X
Низьких: X

Рекомендація: ✅ Затвердити / ⚠️ Запросити зміни / 🔄 Потребує обговорення

Загальна якість коду: 1-5 зірок

3.9 KiB Raw Permalink Blame History Unescape Escape