harmonyos-code-review
Warn
Audited by Gen Agent Trust Hub on Sep 7, 2026
Risk Level: MEDIUMCOMMAND_EXECUTIONDATA_EXFILTRATIONDYNAMIC_EXECUTIONINDIRECT_PROMPT_INJECTION
Full Analysis
- [COMMAND_EXECUTION]: The skill implementation uses
child_process.execSyncto run shell commands constructed via string interpolation of user-supplied variables. Parameters likebranchandrepoPathare directly embedded into command strings insrc/utils/commit-collector.tsandsrc/utils/batch-reviewer.ts, creating a command injection surface if the input is attacker-controlled. - [DATA_EXFILTRATION]: The skill constructs API request URLs based on user-provided repository links and fetches data using
fetch. The parsing logic insrc/analyzers/diff-parser.tsallows for arbitrary hostnames in GitLab-formatted URLs, which could lead to network operations targeting non-whitelisted or malicious domains. - [DYNAMIC_EXECUTION]: The skill spawns subprocesses to interact with the Git CLI for core functionality, including collecting commits, fetching file contents, and calculating diffs. This frequent use of
child_process.execSyncto execute external binaries based on external repository state increases the attack surface. - [INDIRECT_PROMPT_INJECTION]: The skill analyzes untrusted code content from external repositories. While it uses a static engine, it lacks explicit boundary markers for the data it ingests (via
git showand file reads), creating a surface where malicious code comments or identifiers could influence the agent's review output.
Audit Metadata