HarmonyOS Code Review

Audit HarmonyOS ArkTS projects against official Huawei guidelines. Generate prioritized fix reports.

Review Process

1. Quick Scan

Run in parallel to identify critical issues:

# Hardcoded credentials
grep -r "password\|secret\|key\|token" --include="*.json5" --include="*.ets"

# console instead of hilog
grep -r "console\." --include="*.ets" | grep -v "hilog"

# async forEach anti-pattern
grep -r "forEach.*await" --include="*.ets"

# API version check
grep -r "compileSdkVersion\|targetSdkVersion" --include="*.json5"

# Deprecated API usage
grep -r "@Deprecated\|deprecated" --include="*.ets"

2. Deep Analysis

Apply checklist from references/checklist.md per category.

3. Generate Report

Use references/report-template.md as base structure. Include:

• Executive summary (issue counts by priority)
• Detailed findings with file:line references
• Prioritized fix recommendations
• Overall grade (A-F)

References

• Review checklist: references/checklist.md – All review criteria by category
• Official docs: references/official-docs.md – Huawei guidelines, Kit usage, API patterns
• Report template: references/report-template.md – Output format

Issue Priority

• Critical: Blocks release, fix immediately
• High: Fix soon, affects quality
• Medium: Technical debt, consider fixing
• Low: Optional optimization

Exit Criteria

• All checklist categories reviewed
• Report generated at docs/YYYY-MM-DD-review.md
• Critical/high issues have fix suggestions with file:line references