reviewing-dotnet-code
Ревью кода .NET
При ревью C# часто упускают проблемы со стилем, async и обработкой исключений. Этот навык применяет соглашения Microsoft и современные паттерны .NET к ревью и рефакторингу.
Скачать ZIP навыка
Загрузить в Claude
Перейдите в Settings → Capabilities → Skills → Upload skill
Включите и начните использовать
Ресурсы для AI-агентов
Используйте эти ссылки, когда AI-агенту, crawler или script нужен чистый контекст вместо полной страницы.
Протестировать
Использование «reviewing-dotnet-code». Класс сервиса блокируется на async-вызове repository и перехватывает все исключения.
Ожидаемый результат:
- High: Замените блокирующий async-доступ на путь с async-методом, чтобы избежать deadlocks.
- Medium: Перехватывайте конкретные исключения и повторно выбрасывайте неожиданные сбои после логирования.
- Low: Переименуйте поля и параметры в соответствии с соглашениями C#.
Использование «reviewing-dotnet-code». Класс DTO вручную реализует equality и изменяемые свойства.
Ожидаемый результат:
- Suggestion: Преобразуйте DTO в record, когда требуется immutable value equality.
- Benefit: Результат будет короче, понятнее и будет соответствовать современным паттернам C# для передачи данных.
Использование «reviewing-dotnet-code». Ревьюер спрашивает, должна ли команда использовать String или string в новом коде.
Ожидаемый результат:
Используйте ключевые слова языка C#, такие как string, int и bool, если соглашение проекта не предписывает иное.
Аудит безопасности
Низкий рискStatic analysis reported command execution and weak-crypto patterns, but manual review found Markdown backticks, headings, and C# examples rather than executable shell or Ruby code. No prompt injection, secret exfiltration, network callback, or malicious intent was found. The remaining risk is normal filesystem access for reviewing and editing code.
Проблемы низкого риска (2)
Факторы риска
📁 Доступ к файловой системе (1)
Оценка качества
Что вы можете построить
Ревью Pull Request
Найти проблемы с именованием, async, обработкой исключений и LINQ перед слиянием изменений C#.
Рефакторинг Legacy C#
Модернизировать старый код .NET с помощью records, file-scoped namespaces, target-typed new и более понятного именования.
Стандартизация командных рекомендаций
Применять согласованные соглашения в стиле Microsoft при генерации сервисов, DTOs, interfaces и repositories.
Попробуйте эти промпты
Проверь этот файл C# на именование, использование async, обработку исключений и современный стиль .NET. Приоритизируй понятные замечания с исправлениями.
Предложи сфокусированный рефакторинг для этого класса .NET. Сохрани поведение и объясни каждое улучшение именования, LINQ, async или освобождения ресурсов.
Проверь эти изменения .NET как pull request. Сгруппируй замечания по серьезности и добавь краткие рекомендации по замене.
Проверь эту функцию .NET с учетом соглашений проекта. Сначала проверь .editorconfig, затем оцени async-границы, DTOs, repositories и обработку ошибок.
Лучшие практики
- Читайте .editorconfig проекта и существующие соглашения перед предложением изменений только по стилю.
- Отделяйте риски корректности от улучшений читаемости, чтобы ревьюеры могли приоритизировать исправления.
- Запрашивайте окружающий код, когда поведение async, освобождение ресурсов или поток исключений зависят от контекста вызова.
Избегать
- Не применяйте современный синтаксис C#, если проект ориентирован на более старую версию языка.
- Не рекомендуйте масштабный рефакторинг, когда пользователь попросил узкое ревью.
- Не воспринимайте примеры стиля как доказательство того, что production code имеет ту же проблему.
Часто задаваемые вопросы
Какие языки охватывает этот навык?
Может ли он редактировать код напрямую?
Запускает ли он тесты?
Следует ли он соглашениям Microsoft?
Будет ли он учитывать командные правила стиля?
Это навык ревью безопасности?
Сведения для разработчиков
Автор
BrendanShieldsЛицензия
MIT
Репозиторий
https://github.com/BrendanShields/spec-flow/tree/main/plugins/spec/skills/reviewing-dotnet-codeСсылка
main
Структура файлов