Skip to main content

review

Review code changes against WooCommerce iOS standards

跳到安装

来源信息

仓库
woocommerce/woocommerce-ios
最近来源活动
2026年2月16日 18:58
检测到的 SKILL.md 语言
英语
星标
358
分支
131

安装方式

默认使用会先检查来源的 Prompt;你也可以切换为直接命令,或下载本地副本。

检查来源文件

决定是否安装前,请先阅读 SKILL.md,以及 SkillsMP 当前展示的配套文件。

正在显示 SKILL.md

SKILL.md
来源说明 · 只读预览
name
review
description
Review code changes against WooCommerce iOS standards
user-invocable
true
allowed-tools
Bash, Read, Grep, Glob
Review the current changes for compliance with WooCommerce iOS conventions. 1. Get the changes to review: ```bash git diff trunk...HEAD ``` If no diff against trunk, fall back to staged changes: `git diff --cached` 2. Check each modified file against these criteria: **Architecture** (see `.claude/rules/architecture.md`): - UI code does not import Networking or Storage directly - New actions follow the Yosemite dispatch pattern - Dependencies injected via constructor with protocol types - Coordinators for navigation, ViewModels for state **Swift Style** (see `.claude/rules/swift-style.md`): - No force unwraps or force casts - Error handling uses do-catch, not try? - Line length under 163 characters - No parentheses around conditionals **Localization** (see `.claude/rules/localization.md`): - NSLocalizedString uses reverse-DNS keys with value: and comment: - No LocalizedStringKey - No string interpolation in localized strings - Positional placeholders (%1$@) - Strings grouped in enum Localization { } **Analytics** (see `.claude/rules/analytics.md`): - New events added to WooAnalyticsStat - Properties use WooAnalyticsEvent factory pattern - Analytics dependency injected **Testing** (see `.claude/rules/testing.md`): - New code has corresponding tests - Tests use snake_case naming - Given/When/Then structure - Hand-written mocks with Mock prefix 3. Report findings organized by severity: - **Blockers**: Must be fixed before merge - **Suggestions**: Recommended improvements - **Positives**: Things done well
在 GitHub 查看