comprehensive-code-review

Review OpenHarmony ArkUI C++ and ArkTS code for stability, security, and architecture violations.

31|6|Updated Jan 29, 2026
One-click install
npx skills add https://github.com/openharmonyinsight/openharmony-skills --skill comprehensive-code-review-openharmonyinsight
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: comprehensive-code-review
Source: https://github.com/openharmonyinsight/openharmony-skills/tree/main/skills/comprehensive-code-review
Command: npx skills add https://github.com/openharmonyinsight/openharmony-skills --skill comprehensive-code-review-openharmonyinsight

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes scripts (resource) and references (resource) and assets (resource) components.

What problem does it solve?

This Skill reduces the risk of regressions and production incidents by systematically reviewing OpenHarmony ArkUI (ACE Engine) code across critical quality dimensions such as memory, security, stability, threading, architecture, and design correctness.

Core Features & Use Cases

  • 19+ Dimension Review: Evaluates stability, performance, threading, security, memory, modern C++ practices, SOLID principles, code smells, maintainability, observability, API design, technical debt, backward compatibility, and ACE Engine-specific architecture.
  • ACE Engine Specific Guardrails: Enforces ACE Engine four-layer architecture and checks for framework rules like Pattern/Model/Property separation, component lifecycle method usage, and RefPtr/WeakPtr practices (including safe callback captures).
  • Actionable Severity-Tiered Reports: Produces Markdown review output with severity levels and concrete recommendations mapped to reference documents and dimensions.
  • Use Cases: PR quality gates, security and memory audits, architecture compliance validation, and generating repeatable review reports for fast team alignment.

Quick Start

Ask the AI to review the code in frameworks/core/components_ng for memory management, threading safety, and ACE Engine four-layer architecture compliance.

Frequently Asked Questions about comprehensive-code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I perform a static code review on OpenHarmony ArkUI pull requests?▼

To perform static code review on OpenHarmony ArkUI pull requests, execute the provided analysis workflow to identify stability, security, and ACE architecture violations. It acts as a PR quality gate, generating severity-tiered Markdown reports with actionable merge recommendations.

What is ACE Engine four-layer architecture compliance validation?▼

ACE Engine four-layer architecture compliance validation checks OpenHarmony ArkUI code against framework rules like Pattern/Model/Property separation and component lifecycle usage. It enforces ACE-specific guardrails including RefPtr/WeakPtr practices and safe callback captures to prevent regressions.

Can I audit memory management and thread safety in C++ and ArkTS codebases?▼

Yes, you can audit memory management and thread safety across C++ and ArkTS codebases. The review evaluates memory leaks, threading issues, modern C++ practices, and SOLID principles, producing severity-tiered reports mapped to reference guidelines for targeted dimension checks.

Does the code review support directory-wide security audits?▼

The code review supports directory-wide security audits alongside pull request reviews and targeted dimension checks. It systematically evaluates OpenHarmony ArkUI code across security, stability, performance, and design dimensions to reduce the risk of production incidents.

What's the best way to catch critical ACE Engine issues before merge?▼

The best way to catch critical ACE Engine issues before merge is running a comprehensive code review that identifies architecture violations, memory safety risks, and threading defects. It produces actionable severity-tiered Markdown reports mapped to reference documents for fast team alignment.

Why does my OpenHarmony ArkUI code review need to check technical debt and backward compatibility?▼

Your OpenHarmony ArkUI code review needs to check technical debt and backward compatibility because evaluating these dimensions prevents regressions and maintains production stability. The review systematically assesses maintainability, API design, and code smells alongside ACE Engine-specific architecture rules.