github.com / alibaba

alibaba/open-code-review

Go·945 files·commit 486022d·scanned 2h ago·cached ✓

15/100

LOW RISK

Minor findings consistent with the project type. Nothing reachable from install hooks.

score capped at 15 — 40,871+ stars — findings likely false positives

verdict accurate?

40,871 stars

Partial architecture graph

CLI tool detected

threat-state: lowlive

FINDINGS ░▒▓

criticalGo os/exec usageos/exec is used to run external commands. In malware it drops payloads, modifies the system, or opens shells.internal/config/rules/sniffer.go


	cmd := exec.CommandContext(ctx, "git", args...)
	cmd.Dir = s.repoDir
	var stdout bytes.Buffer

criticalGo os/exec usageos/exec is used to run external commands. In malware it drops payloads, modifies the system, or opens shells.internal/config/rules/sniffer_test.go

		t.Helper()
		cmd := exec.Command("git", args...)
		cmd.Dir = dir
		out, err := cmd.CombinedOutput()

criticalGo os/exec usageos/exec is used to run external commands. In malware it drops payloads, modifies the system, or opens shells.cmd/opencodereview/background_file_test.go

	run := func(args ...string) []byte {
		cmd := exec.Command("git", args...)
		cmd.Dir = repo
		out, err := cmd.CombinedOutput()

infoSuspicious files are not reachable from entry pointsFlagged files exist but are not imported by any entry point. They may be dead code, tests, or attack payloads triggered by another mechanism.+2

warningSSH/credential path accessAccessing .ssh, .aws/credentials, or .env files to steal authentication credentials.internal/config/allowlist/default_secret_patterns.json

[
  "**/.ssh/**",
  "**/id_rsa",
  "**/id_dsa",

warningExtremely long lines (>1000 chars)Very long lines in source files (not minified bundles) can hide malicious code.internal/config/template/scan_template.json+4

warningExtremely long lines (>1000 chars)Very long lines in source files (not minified bundles) can hide malicious code.internal/config/toolsconfig/tools.json+4

warningExtremely long lines (>1000 chars)Very long lines in source files (not minified bundles) can hide malicious code..opencodereview/rule.json+4

warningHigh-entropy string literalsFound 26 long strings with high Shannon entropy. This is common in obfuscated payloads that hide URLs, keys, or bytecode.extensions/frontend/src/shared/i18n.ts+3

warningFlattened or dead control flowDetected switch(true), dead if branches, or deeply nested ternaries — patterns used by obfuscators to hide execution order.extensions/frontend/src/webview/components/CustomProviderManager.tsx+3

warningHigh-entropy string literalsFound 9 long strings with high Shannon entropy. This is common in obfuscated payloads that hide URLs, keys, or bytecode.pages/src/i18n/en.ts+3

warningHigh-entropy string literalsFound 54 long strings with high Shannon entropy. This is common in obfuscated payloads that hide URLs, keys, or bytecode.pages/src/i18n/ja.ts+3

infoSuspicious file in repoExecutable file (gradlew.bat) in repositoryextensions/idea/gradlew.bat+2

infoSuspicious file in repoExecutable file (install.ps1) in repositoryinstall.ps1+2

warningPossible typosquat: preact"preact" is only 1 character(s) away from the popular package "react". This is a common supply-chain attack vector.package.json+4

warningHardcoded secret/credentialAPI keys, passwords, or tokens hardcoded in source code. Should be in environment variables.internal/llm/resolver.go

	envCCBaseURL = "ANTHROPIC_BASE_URL"
	envCCToken   = "ANTHROPIC_AUTH_TOKEN"
	envCCModel   = "ANTHROPIC_MODEL"
)

Scores are heuristics. A “safe” verdict means no known-malicious patterns were found — clever malware can look boring. Wrong verdict? Flag it above; confirmed false positives become regression tests.

544 files scanned @ 486022d | 9/25/2026 | heuristic scan — always review manually

risk by category

code execution25

network & exfiltration0

file system access5

obfuscation15

supply chain8

owasp / injection3

telemetry

files 544/945rules hit 18engine v6commit 486022d

github

alibaba/open-code-review

Secure, fast, efficient, battle-tested at Alibaba's scale. Hybrid architecture code review tool: deterministic pipelines + LLM Agent, precise line-level comments, built-in multi-language ruleset (NPE, thread-safety, XSS, SQL injection), OpenAI & Anthropic compatible.

40871

2936

130d

945 files

544 scanned(58%)

486022d

architecture░▒▓

entry (3) flagged (46) pkg (125)

525 nodes · 1366 edgesscroll to zoom · click node to jump to finding