← Back to scan
File as GitHub Issue repo: modu-ai/moai-adk

Push this scan report to modu-ai/moai-adk

Click the green button below to open GitHub’s new-issue form, pre-filled with the report title, summary table, top findings, and an embedded score-card image. No authentication needed — you review on GitHub before submitting. Repobility is credited as the scanner.

Embedded score card image

This image will render at the top of the issue body. Hosted on Repobility, refreshes automatically after re-scans.

Repobility score card

Issue title

Python: yaml.load without SafeLoader

Curate findings to include

Pick exactly which findings appear in the issue body. By default the top 5 are included. Uncheck noise, check what matters.

Top 5 (default)
Severity Rule Title File:line
CRIT MINED018 [MINED018] Unsafe Deserialization Pickle: pickle.loads / yaml.load (without Loader=SafeLo… internal/hook/security/rules.go:199
CRIT SEC116 [SEC116] Ruby YAML.load / Marshal.load on untrusted input: `YAML.load` (pre-3.1) and `Mar… internal/hook/security/rules.go:200
CRIT SEC084 [SEC084] JS: require() with non-literal: require(<variable>) loads arbitrary modules — eq… internal/hook/security/rules.go:192
CRIT SEC081 [SEC081] Python: pickle.loads / marshal.loads on untrusted data: pickle.load(s) and marsh… internal/hook/security/rules.go:199
CRIT SEC079 [SEC079] Python: yaml.load without SafeLoader: yaml.load() without explicit SafeLoader ca… internal/hook/security/rules.go:200
CRIT MINED123 [MINED123] Trojan Source bidi character (LRE) in source: Line 1 contains a Unicode bidire… docs-site/themes/hugo-geekdoc/static/js…:1
CRIT MINED116 [MINED116] Workflow uses `secrets.CODECOV_TOKEN` on a `pull_request` trigger: This workfl… .github/workflows/ci.yml:133
CRIT MINED116 [MINED116] Workflow uses `secrets.CLAUDE_CODE_OAUTH_TOKEN` on a `pull_request` trigger: T… .github/workflows/claude-code-review.yml:48
HIGH MINED012 [MINED012] Curl Pipe Bash: curl ... | sh / bash — runs unverified network code. internal/update/updater.go:299
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … internal/lsp/gopls/handler.go:49
HIGH MINED033 [MINED033] Go Recover Without Log: defer func() { recover() }() that silently swallows pa… internal/hook/trace/writer.go:66
HIGH SEC100 [SEC100] CORS permissive Access-Control-Allow-Origin: *: Permissive CORS policy (`*` orig… internal/hook/security/rules.go:218
HIGH SEC114 [SEC114] path.join / Path() on user-controlled segment without containment check: filepat… internal/hook/instructions_loaded.go:46
HIGH SEC085 [SEC085] JS: child_process.exec with non-literal: child_process.exec with user-derived in… internal/github/issue_closer.go:124
HIGH SEC085 [SEC085] JS: child_process.exec with non-literal: child_process.exec with user-derived in… internal/cli/cg.go:27
HIGH SEC085 [SEC085] JS: child_process.exec with non-literal: child_process.exec with user-derived in… internal/cli/cc.go:27
HIGH SEC093 [SEC093] Go: exec.Command with non-literal: exec.Command(<var>) — variable command name a… internal/cli/worktree/guard.go:285
HIGH SEC093 [SEC093] Go: exec.Command with non-literal: exec.Command(<var>) — variable command name a… internal/cli/branch_protection.go:69
HIGH SEC093 [SEC093] Go: exec.Command with non-literal: exec.Command(<var>) — variable command name a… internal/cli/astgrep.go:214
HIGH MINED016 [MINED016] Go Error Ignored: _, err := fn() with err not checked. Go anti-pattern. internal/cli/doctor_permission.go:160
HIGH MINED016 [MINED016] Go Error Ignored: _, err := fn() with err not checked. Go anti-pattern. internal/cli/astgrep.go:196
HIGH MINED016 [MINED016] Go Error Ignored: _, err := fn() with err not checked. Go anti-pattern. internal/bodp/audit_trail.go:58
HIGH SEC029 [SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT… internal/cli/worktree/project.go:31
HIGH SEC029 [SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT… internal/cli/wizard/types.go:27
HIGH SEC029 [SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT… docs-site/api/i18n-detect.ts:109
HIGH SEC006 [SEC006] XSS Risk: Direct HTML injection without sanitization. internal/hook/security/rules.go:143
HIGH MINED115 [MINED115] Action `actions/setup-go` pinned to mutable ref `@v6`: `uses: actions/setup-go… .github/workflows/ci.yml:177
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/ci.yml:172
HIGH MINED115 [MINED115] Action `codecov/codecov-action` pinned to mutable ref `@v6`: `uses: codecov/co… .github/workflows/ci.yml:129
HIGH MINED115 [MINED115] Action `actions/setup-go` pinned to mutable ref `@v6`: `uses: actions/setup-go… .github/workflows/ci.yml:85
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/ci.yml:80
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/ci.yml:39
HIGH MINED115 [MINED115] Action `actions/upload-artifact` pinned to mutable ref `@v4`: `uses: actions/u… .github/workflows/docs-i18n-check.yml:165
HIGH MINED115 [MINED115] Action `actions/github-script` pinned to mutable ref `@v7`: `uses: actions/git… .github/workflows/docs-i18n-check.yml:99
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout… .github/workflows/docs-i18n-check.yml:53
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/test-install.yml:202
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/test-install.yml:167
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/test-install.yml:121
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/test-install.yml:72
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/test-install.yml:37
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout… .github/workflows/claude-code-review.yml:19
HIGH MINED115 [MINED115] Action `EndBug/label-sync` pinned to mutable ref `@v2`: `uses: EndBug/label-sy… .github/workflows/label-sync.yml:45
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout… .github/workflows/label-sync.yml:42
HIGH MINED115 [MINED115] Action `actions/setup-go` pinned to mutable ref `@v6`: `uses: actions/setup-go… .github/workflows/release-pr-multi-os.y…:57
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/release-pr-multi-os.y…:52
HIGH MINED115 [MINED115] Action `github/codeql-action/analyze` pinned to mutable ref `@v4`: `uses: gith… .github/workflows/codeql.yml:97
HIGH MINED115 [MINED115] Action `github/codeql-action/autobuild` pinned to mutable ref `@v4`: `uses: gi… .github/workflows/codeql.yml:94
HIGH MINED115 [MINED115] Action `github/codeql-action/init` pinned to mutable ref `@v4`: `uses: github/… .github/workflows/codeql.yml:89
HIGH MINED115 [MINED115] Action `actions/setup-go` pinned to mutable ref `@v6`: `uses: actions/setup-go… .github/workflows/codeql.yml:83
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/codeql.yml:80
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v5`: `uses: actions/checkout… .github/workflows/codeql.yml:30
MED SEC119 [SEC119] World-writable / world-readable file permissions: World-writable files let any l… internal/update/updater.go:138
MED SEC119 [SEC119] World-writable / world-readable file permissions: World-writable files let any l… internal/update/rollback.go:53
MED SEC123 [SEC123] Production stack trace / debug output exposed: Debug mode left on in production … internal/hook/security/rules.go:208
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … internal/github/issue_closer.go:124
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … internal/cli/cg.go:27
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … internal/cli/cc.go:27
MED SEC112 [SEC112] Go html/template bypass — text/template used for HTML output, or template.HTML o… internal/cli/doctor_sandbox.go:141
MED SEC112 [SEC112] Go html/template bypass — text/template used for HTML output, or template.HTML o… internal/cli/doctor_hook.go:102
MED SEC112 [SEC112] Go html/template bypass — text/template used for HTML output, or template.HTML o… internal/cli/astgrep.go:164
MED SEC087 [SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographically secure; … .moai/brain/IDEA-002/claude-design-hand…:152
MED SEC007 [SEC007] Unsafe Deserialization: Unsafe deserialization can execute arbitrary code. internal/hook/security/rules.go:199
MED AGT007 localStorage write failures are swallowed silently .moai/design/SPEC-V3R3-CLI-TUI-001/sour…:270
MED AGT015 Remote install command pipes network code directly to a shell .moai/marketing/blog-posts/velog-ko.md:59
MED AGT015 Remote install command pipes network code directly to a shell .moai/marketing/blog-posts/okky-ko.md:158
MED SEC005 [SEC005] Command Injection Risk: Unsafe shell execution or eval of user input. internal/hook/security/rules.go:181
LOW ERR003 [ERR003] Ignored Error (Go): Ignoring error return values. internal/cli/astgrep.go:130
LOW ERR003 [ERR003] Ignored Error (Go): Ignoring error return values. internal/ciwatch/state.go:57
LOW ERR003 [ERR003] Ignored Error (Go): Ignoring error return values. internal/astgrep/rules.go:36
LOW AIC003 Duplicated implementation block across source files internal/harness/tier/tier.go:7
LOW AIC003 Duplicated implementation block across source files .moai/scripts/status-drift-cleanup.go:110
LOW AIC003 Duplicated implementation block across source files internal/lsp/hook/tracker.go:123
LOW AIC003 Duplicated implementation block across source files internal/lsp/aggregator/aggregator.go:173
LOW AIC003 Duplicated implementation block across source files internal/loop/storage.go:68
LOW AIC003 Duplicated implementation block across source files internal/hook/user_prompt_submit.go:69
LOW AIC003 Duplicated implementation block across source files internal/harness/retention.go:63
LOW AIC003 Duplicated implementation block across source files internal/design/dtcg/categories/typogra…:10
LOW AIC003 Duplicated implementation block across source files internal/cli/migrate_agency_disk_window…:30
LOW AIC002 Source file name looks like an AI patch artifact internal/hook/auto_update.go:1
INFO MINED069 [MINED069] Debug True Prod: Django/Flask DEBUG=True or app.debug=True in non-test files. internal/hook/security/rules.go:208
INFO MINED071 [MINED071] Go Panic Call: panic() crashes the process. Should return error in most cases. internal/tui/progress_line.go:175
INFO MINED071 [MINED071] Go Panic Call: panic() crashes the process. Should return error in most cases. internal/core/project/root.go:95
INFO MINED071 [MINED071] Go Panic Call: panic() crashes the process. Should return error in most cases. internal/cli/harness_route.go:146
INFO MINED060 [MINED060] Go Context No Cancel: context.Background() at request handler boundary leaks g… internal/cli/mcp.go:44
INFO MINED060 [MINED060] Go Context No Cancel: context.Background() at request handler boundary leaks g… internal/cli/github.go:143
INFO MINED060 [MINED060] Go Context No Cancel: context.Background() at request handler boundary leaks g… internal/cli/astgrep.go:212
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … docs-site/scripts/generate-favicons.js:27
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … docs-site/scripts/fix-mdx-formatting.js:70
INFO MINED056 [MINED056] React Key As Index: key={index} in map() — re-renders the wrong elements on re… .moai/brain/IDEA-002/claude-design-hand…:64
INFO MINED098 [MINED098] Global Scope Pollution: Attaching libraries/objects directly to the global win… .moai/brain/IDEA-002/claude-design-hand…:35
Reset to top 5 90 findings available (after auto-suppression of test files + won't-fix)

Issue body (markdown)

## Code-quality scan: `modu-ai/moai-adk`

**Score: 84/100 (A-)**  ·  90 findings  ·  scanned 2026-05-31 01:24 UTC  ·  334,560 LOC

| Severity | Count |
|---|---|
| CRITICAL | 8 |
| HIGH | 43 |
| MEDIUM | 15 |
| LOW | 13 |

📊 [Full filterable report](https://repobility.com/scan/b17cb6f7-f398-498c-b7bc-c52164018838/)  ·  ![scorecard](https://repobility.com/scan/b17cb6f7-f398-498c-b7bc-c52164018838/report.png?v=1780190670-s2)

### Top findings

1. **CRITICAL** `MINED018` — Unsafe Deserialization Pickle
   `internal/hook/security/rules.go:199` · CWE-502 · ✓ Repobility
2. **CRITICAL** `SEC116` — Ruby YAML.load / Marshal.load on untrusted input
   `internal/hook/security/rules.go:200` · A08:2021 Software & Data Integrity Failures
3. **CRITICAL** `SEC084` — JS: require() with non-literal
   `internal/hook/security/rules.go:192`
4. **CRITICAL** `SEC081` — Python: pickle.loads / marshal.loads on untrusted data
   `internal/hook/security/rules.go:199` · A05:2021 Security Misconfiguration
5. **CRITICAL** `SEC079` — Python: yaml.load without SafeLoader
   `internal/hook/security/rules.go:200` · A05:2021 Security Misconfiguration

---

**Security note**: this issue is public. If any flagged finding is a real, exploitable vulnerability, please redirect to your `SECURITY.md` policy or open a [private security advisory](https://docs.github.com/en/code-security/security-advisories/guidance-on-reporting-and-writing-information-about-vulnerabilities/privately-reporting-a-security-vulnerability) instead. We're happy to close this and re-submit privately.

---

_Filed automatically. Close this issue if not useful — we won't refile. Full report: https://repobility.com/scan/b17cb6f7-f398-498c-b7bc-c52164018838/_
Already filed
This repo publishes a SECURITY.md policy and the scan contains 17 Critical/High security finding(s). Public issue filing would violate coordinated disclosure. Submit privately via the project's security reporting channel.
Megaproject â high spam risk
Could not determine 'modu-ai/moai-adk' star count (GitHub API rate-limited or unreachable). When in doubt about repo size, prefer opening a focused PR or a discussion rather than an issue.

The button opens GitHubâs new-issue page in a new tab. You will see the title + body pre-filled â review, edit if you want, then click GitHubâs "Submit new issue" button. Repobility never posts anything on your behalf.

For real security findings on big repos: use the project's SECURITY.md or private advisory flow instead of a public issue.