Public scan — anyone with this URL can view this analysis. Sign up to track your own repos privately, run scheduled re-scans, and get AI fix prompts via your dashboard.

alibaba/open-code-review

https://github.com/alibaba/open-code-review · scanned 2026-07-23 19:36 UTC (4 days, 20 hours ago)

56 raw signals (0 security + 56 graph)

File as issue Image

UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

Last scanned 4 days, 20 hours ago · v3 · last Δ +0.3 (diff) · 53 actionable findings from 1 signal source. 3 repeated signals grouped for readability. Security checks, system graph analysis, and verified AI-agent feedback are merged into one review queue.

JSON

70.0% cov · 53 findings

Severity distribution — click a segment to filter

Active filters: excluding tests × Reset all

Severity: Critical 0 High 0 Medium 30 Low 19 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Security checks 0 System graph 53 Crowd 0 Layer: Software 1 Frontend 2 Security 16 Quality 16 Cicd 5 Dependencies 13

Exclude dismissed / FP Exclude test files

Scan summary Repository scanned at 57.1/100 with 70.0% coverage. It contains 1186 nodes across 12 cross-layer flows, written primarily in mixed languages. Engine surfaced 56 findings — concentrated in quality (17), security (16), dependencies (13). Risk profile is low: 0 critical, 0 high, 31 medium. Recommended next step: open the quality layer findings first — that's where the highest-impact wins live.

Showing 49 of 53 actionable findings. 56 raw detector signals were grouped into reader-sized issues. Click TP / FP to vote on a finding's accuracy — votes adjust the confidence weighting and improve detection across the platform.

medium System graph frontend Frontend quality conf 0.80 `dangerouslySetInnerHTML` used in a React component — pages/src/components/MarkdownRenderer.tsx:171

Open XSS surface unless the input is provably trusted. Replace with explicit JSX or sanitize via a vetted library. Why: OWASP basics. Already partially flagged by the security analyzer. Rule id: fq.dangerous-html

pages/src/components/MarkdownRenderer.tsx:171 Fq dangerous html

medium System graph quality Agent instructions conf 1.00 Agent authority lacks a verifier contract: .claude/commands/open-code-review.md

This agent instruction grants code or shell authority but does not state the verification gate that decides promotion. The recurring safe pattern is: LLM proposes; deterministic tests/build/security checks verify; only verified code promotes.

.claude/commands/open-code-review.md VerificationClaude instruction

medium System graph quality Agent instructions conf 1.00 Agent authority lacks a verifier contract: .claude/commands/tag.md

.claude/commands/tag.md VerificationClaude instruction

medium System graph quality Agent instructions conf 1.00 Agent authority lacks a verifier contract: plugins/open-code-review/skills/open-code-review-delegate/SKILL.md

plugins/open-code-review/skills/open-code-review-delegate/SKILL.md VerificationSkill file

medium System graph quality Agent instructions conf 1.00 Agent authority lacks a verifier contract: skills/open-code-review-delegate/SKILL.md

skills/open-code-review-delegate/SKILL.md VerificationSkill file

medium System graph security Agent instructions conf 1.00 Agent instruction contains unpinned remote install: plugins/open-code-review/skills/open-code-review-delegate/SKILL.md

Remote install commands in agent instructions are a supply-chain risk, especially when an agent can execute shell commands.

plugins/open-code-review/skills/open-code-review-delegate/SKILL.md:11 Supply chainSkill file

medium System graph security Agent instructions conf 1.00 Agent instruction contains unpinned remote install: plugins/open-code-review/skills/open-code-review/SKILL.md

Remote install commands in agent instructions are a supply-chain risk, especially when an agent can execute shell commands.

plugins/open-code-review/skills/open-code-review/SKILL.md:13 Supply chainSkill file

medium System graph security Agent instructions conf 1.00 Agent instruction contains unpinned remote install: skills/open-code-review-delegate/SKILL.md

Remote install commands in agent instructions are a supply-chain risk, especially when an agent can execute shell commands.

skills/open-code-review-delegate/SKILL.md:11 Supply chainSkill file

medium System graph security Agent instructions conf 1.00 Agent instruction contains unpinned remote install: skills/open-code-review/SKILL.md

Remote install commands in agent instructions are a supply-chain risk, especially when an agent can execute shell commands.

skills/open-code-review/SKILL.md:13 Supply chainSkill file

medium System graph quality Agent instructions conf 1.00 Agent instructions exist but release-hardening basics are missing

AI-coder instruction files were found, but the repo is missing lockfile. Treat this as a contract gap: the agent is guided, but the generated output is not yet guarded by the controls that make it repeatable.

Repo hardeningGenerated repo pattern

medium System graph quality Placeholder conf 1.00 Critical user flow still appears backed by mock or placeholder data

A payment/auth/admin/order/billing-style flow contains mock, fake, TODO, dummy, or placeholder markers in runtime source. In the Fable corpus this is a high-leverage completeness smell: the app can look finished while the money, identity, or tenant flow is still scaffolded.

Mock dataCritical flowGenerated repo pattern

medium System graph security Semgrep conf 0.55 detect child process — extensions/vscode/src/extension/services/CliService.ts:35

Detected calls to child_process from a function argument `bin`. This could lead to a command injection if the input is user controllable. Try to avoid calls to child_process, and if it is needed ensure user input is correctly sanitized or sandboxed. Rule: javascript.lang.security.detect-child-proc…

extensions/vscode/src/extension/services/CliService.ts:35 SecurityJavascript

medium System graph security Semgrep conf 0.55 detect child process — scripts/update.js:61

Detected calls to child_process from a function argument `binPath`. This could lead to a command injection if the input is user controllable. Try to avoid calls to child_process, and if it is needed ensure user input is correctly sanitized or sandboxed. Rule: javascript.lang.security.detect-child-…

scripts/update.js:61 SecurityJavascript

medium System graph security Semgrep conf 0.55 dynamic urllib use detected — examples/gerrit_ci/post_review.py:241

Detected a dynamic value being used with urllib. urllib supports 'file://' schemes, so a dynamic value controlled by a malicious actor may allow them to read arbitrary files. Audit uses of urllib calls to ensure user data cannot control the URLs, or consider using the 'requests' library instead. R…

examples/gerrit_ci/post_review.py:241 SecurityPythonNon production context

medium System graph security Semgrep conf 0.55 dynamic urllib use detected — examples/gitflic_ci/post_review.py:341

examples/gitflic_ci/post_review.py:341 SecurityPythonNon production context

medium System graph quality Integrity conf 1.00 2 occurrences Frontend route `/docs/:slug` has no Link/navigate to it — pages/src/App.tsx

The route is registered but no `<Link to=…>` or `navigate(…)` in the codebase navigates here. Either it's reachable only via direct URL (intentional), it's dead, or the link broke during a refactor.

2 occurrences

repo-level (2 hits)

Orphan pageWiring

medium System graph cicd CI/CD security conf 1.00 GitHub Action is tag-pinned rather than SHA-pinned

softprops/action-gh-release@v2 can move without a code change in this repo. Pin third-party actions to a reviewed 40-character commit SHA.

.github/workflows/release.yml:147 CI/CD securitySupply chainGithub actions

medium System graph cicd CI/CD security conf 1.00 GitHub Actions workflow grants broad write permissions

CI tokens with write permissions increase blast radius when an action, dependency, or PR workflow is compromised. Prefer job-level least-privilege permissions.

.github/workflows/deploy-pages.yml CI/CD securitySupply chainGithub actions

medium System graph cicd CI/CD security conf 1.00 GitHub Actions workflow grants broad write permissions

CI tokens with write permissions increase blast radius when an action, dependency, or PR workflow is compromised. Prefer job-level least-privilege permissions.

.github/workflows/release.yml CI/CD securitySupply chainGithub actions

medium System graph security security conf 0.65 Insecure pattern 'dangerous_innerhtml' in pages/src/components/MarkdownRenderer.tsx:171

Found a known-risky pattern (dangerous_innerhtml). Review and replace if possible.

pages/src/components/MarkdownRenderer.tsx:171 Dangerous innerhtml

medium System graph security security conf 0.65 Insecure pattern 'direct_innerhtml_assignment' in pages/src/components/MarkdownRenderer.tsx:123

Found a known-risky pattern (direct_innerhtml_assignment). Review and replace if possible.

pages/src/components/MarkdownRenderer.tsx:123 Direct innerhtml assignment

medium System graph quality Integrity conf 0.85 Network/subprocess call without timeout or try/except — examples/gitflic_ci/post_review.py:378

`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.

examples/gitflic_ci/post_review.py:378 runtime safetyRobustness

medium System graph quality Placeholder conf 1.00 Placeholder or mock-heavy implementation detected

Found 88 placeholder/mock markers across 27 source files. This often means the repo looks complete while core flows still use generated scaffolding or fake data.

Mock dataIncompleteGenerated repo pattern

medium System graph security Semgrep conf 0.70 react dangerouslysetinnerhtml — pages/src/components/MarkdownRenderer.tsx:171

Detection of dangerouslySetInnerHTML from non-constant definition. This can inadvertently expose users to cross-site scripting (XSS) attacks if this comes from user-provided input. If you have to use dangerouslySetInnerHTML, consider using a sanitization library such as DOMPurify to sanitize your H…

pages/src/components/MarkdownRenderer.tsx:171 SecurityReact

medium System graph security Skillspector conf 0.50 SkillSpector E1 (data-exfil) in plugins/open-code-review/skills/open-code-review/SKILL.md

https://api.anthropic.com/ Data is being sent to an external URL. This could be legitimate telemetry or data exfiltration. Manual review is recommended. Skill: open-code-review Rule: E1 Category: data-exfil Severity: MEDIUM Confidence: 0.50 Remediation: Verify the destination URL is trusted an…

plugins/open-code-review/skills/open-code-review/SKILL.md:54 Mcp skillData exfilE1

medium System graph security Skillspector conf 0.50 SkillSpector E1 (data-exfil) in skills/open-code-review/SKILL.md

skills/open-code-review/SKILL.md:49 Mcp skillData exfilE1

medium System graph dependencies dependencies conf 0.70 Vulnerable dependency webpack 5.89.0: GHSA-4vvj-4cpr-p986

OSV.dev reports `webpack` at version `5.89.0` (declared in `pages/package.json`) is affected by GHSA-4vvj-4cpr-p986 (aka CVE-2024-43788). Note: `5.89.0` is the declared floor of a range — the installed version may be newer. Webpack's AutoPublicPathRuntimeModule has a DOM Clobbering Gadget that lea…

pages/package.json ScaOsvGhsa 4vvj 4cpr p986

medium System graph dependencies dependencies conf 0.70 Vulnerable dependency webpack-dev-server 5.2.4: GHSA-f5vj-f2hx-8m93

OSV.dev reports `webpack-dev-server` at version `5.2.4` (declared in `pages/package.json`) is affected by GHSA-f5vj-f2hx-8m93 (aka CVE-2026-14620). Note: `5.2.4` is the declared floor of a range — the installed version may be newer. webpack-dev-server vulnerable to cross-site request forgery via i…

pages/package.json ScaOsvGhsa f5vj f2hx 8m93

medium System graph dependencies dependencies conf 0.70 Vulnerable dependency webpack-dev-server 5.2.4: GHSA-m28w-2pqf-7qgj

OSV.dev reports `webpack-dev-server` at version `5.2.4` (declared in `pages/package.json`) is affected by GHSA-m28w-2pqf-7qgj (aka CVE-2026-14631). Note: `5.2.4` is the declared floor of a range — the installed version may be newer. webpack-dev-server vulnerable to denial of service via a malforme…

pages/package.json ScaOsvGhsa m28w 2pqf 7qgj

medium System graph dependencies dependencies conf 0.70 Vulnerable dependency webpack-dev-server 5.2.4: GHSA-mx8g-39q3-5c79

OSV.dev reports `webpack-dev-server` at version `5.2.4` (declared in `pages/package.json`) is affected by GHSA-mx8g-39q3-5c79 (aka CVE-2026-9595). Note: `5.2.4` is the declared floor of a range — the installed version may be newer. webpack-dev-server vulnerable to HMR WebSocket interception via pe…

pages/package.json ScaOsvGhsa mx8g 39q3 5c79

low System graph quality Debug conf 1.00 Debug logging residue appears in source files

Found 14 console/debugger/print-style debug statements in non-test source. This is a common fast-generation residue before production cleanup.

CleanupRepo hardeningGenerated repo pattern

low System graph dependencies dependencies conf 0.90 Dependency react is a major version behind

`react` is pinned at `18.2.0` in `pages/package.json` while the latest release on the npm registry is `19.2.8` — 1 major version(s) behind. Old majors stop receiving security backports and accumulate known CVEs. Review the upstream changelog / migration guide and upgrade `react` to `19.2.8`.

pages/package.json FreshnessOutdated

low System graph dependencies dependencies conf 0.90 Dependency react-dom is a major version behind

`react-dom` is pinned at `18.2.0` in `pages/package.json` while the latest release on the npm registry is `19.2.8` — 1 major version(s) behind. Old majors stop receiving security backports and accumulate known CVEs. Review the upstream changelog / migration guide and upgrade `react-dom` to `19.2.8`.

pages/package.json FreshnessOutdated

low System graph dependencies dependencies conf 0.90 Dependency react-router-dom is a major version behind

`react-router-dom` is pinned at `6.8.0` in `pages/package.json` while the latest release on the npm registry is `7.18.1` — 1 major version(s) behind. Old majors stop receiving security backports and accumulate known CVEs. Review the upstream changelog / migration guide and upgrade `react-router-dom…

pages/package.json FreshnessOutdated

low System graph cicd CI/CD security conf 1.00 3 occurrences GitHub Action is tag-pinned rather than SHA-pinned

actions/deploy-pages@v4 can move without a code change in this repo. Pin third-party actions to a reviewed 40-character commit SHA.

3 files, 3 locations

.github/workflows/deploy-pages.yml:61

.github/workflows/ocr-review.yml:48

.github/workflows/vscode-ext.yml:33

CI/CD securitySupply chainGithub actions

low System graph dependencies dependencies conf 1.00 Node manifest has dependencies but no lockfile: package.json

`package.json` declares dependencies, but no same-directory npm/pnpm/yarn/bun lockfile was found. Generated projects without lockfiles are less reproducible and harder to secure-scan precisely.

package.json LockfileReproducibilityGenerated repo pattern

low System graph dependencies dependencies conf 1.00 Node manifest has dependencies but no lockfile: pages/package.json

`package.json` declares dependencies, but no same-directory npm/pnpm/yarn/bun lockfile was found. Generated projects without lockfiles are less reproducible and harder to secure-scan precisely.

pages/package.json LockfileReproducibilityGenerated repo pattern

low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `last_old` in examples/gitflic_ci/post_review.py:195

Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.

old markerDead code

low System graph cicd CI/CD security conf 1.00 package.json defines install-time lifecycle scripts

preinstall/install/postinstall/prepare scripts execute during dependency installation. Review them carefully for network calls, obfuscation, shell execution, or credential access.

package.json CI/CD securitySupply chainNpm

low System graph software Dead code conf 1.00 Possibly dead Python function: positive_float

No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.

examples/gerrit_ci/post_review.py:316

low System graph quality Complexity conf 1.00 Very large file: cmd/opencodereview/provider_tui.go (2953 lines)