infiniflow/ragflow

Component	Sub-score	Weight	Contribution
`structure_score`	85.0	0.15	12.75
`security_score`	100.0	0.25	25.00
`testing_score`	96.0	0.20	19.20
`documentation_score`	75.0	0.15	11.25
`practices_score`	90.0	0.15	13.50
`code_quality`	48.0	0.10	4.80
Overall		1.00	86.5

critical Security checks quality Quality conf 1.00 ✓ Repobility [MINED030] Python Pickle Loads: pickle.loads() can execute arbitrary code via __reduce__.

Review and fix per the pattern semantics. See CWE-502 / for context.

api/utils/configs.py:28

critical Security checks quality Quality conf 1.00 ✓ Repobility [MINED030] Python Pickle Loads: pickle.loads() can execute arbitrary code via __reduce__.

Review and fix per the pattern semantics. See CWE-502 / for context.

agent/sandbox/executor_manager/services/security.py:54

high Security checks quality Quality conf 1.00 ✓ Repobility 8 occurrences [MINED107] Missing import: `queue` used but not imported: The file uses `queue.something(...)` but never imports `queue`. This raises NameError at runtime the first time the line executes.

Add `import queue` at the top of the file.

8 files, 8 locations

api/db/services/dialog_service.py:695

api/utils/common.py:21

common/data_source/gitlab_connector.py:238

common/data_source/google_drive/connector.py:55

common/data_source/google_util/util.py:36

common/data_source/jira/connector.py:112

common/data_source/jira/utils.py:100

common/data_source/utils.py:751

critical Security checks security secrets conf 1.00 [SEC019] Raw Authorization Token in Example: A real-looking API token appears in an Authorization-style header or service-key example. Use placeholders in docs and CI snippets; never paste live tokens into source, comments, or README files.

Replace the value with a placeholder, revoke or rotate the exposed token, and store live values only in a masked secret store.

example/http/dataset_example.sh:22

critical Security checks quality Quality conf 1.00 [SEC081] Python: pickle.loads / marshal.loads on untrusted data: pickle.load(s) and marshal.load(s) execute arbitrary code on untrusted input. Ported from dlint DUO103 / DUO120 (BSD-3).

Use json, msgpack, or protobuf for untrusted data. If pickle is required, sign the payload with HMAC.

api/utils/configs.py:40

critical Security checks cicd CI/CD security conf 0.98 Compose service mounts the Docker socket

Avoid mounting docker.sock. Use a narrow proxy, rootless build service, or provider-native deployment credentials.

agent/sandbox/docker-compose.yml:1 CI/CD securitycontainers

critical Security checks cicd CI/CD security conf 0.98 Compose service runs privileged

Remove privileged mode. Add the single capability, device, or mount that is actually required.

agent/sandbox/docker-compose.yml:1 CI/CD securitycontainers

critical System graph security Secrets conf 1.00 Possible secret in admin/client/ragflow_client.py

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

admin/client/ragflow_client.py:74

critical System graph security Secrets conf 1.00 Possible secret in agent/tools/exesql.py

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

agent/tools/exesql.py:142

critical System graph security Secrets conf 1.00 7 occurrences Possible secret in conf/service_conf.yaml

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 10, 18, 25, 29, 39, 45, 59

conf/service_conf.yaml:10, 18, 25, 29, 39, 45, 59 (7 hits)

critical System graph security Secrets conf 1.00 Possible secret in internal/cli/cli.go

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

internal/cli/cli.go:511

critical System graph security Secrets conf 1.00 Possible secret in internal/cli/common_command.go

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

internal/cli/common_command.go:518

critical System graph security Secrets conf 1.00 Possible secret in web/src/components/dynamic-form.tsx

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

web/src/components/dynamic-form.tsx:50

critical System graph security Secrets conf 1.00 Possible secret in web/src/constants/setting.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

web/src/constants/setting.ts:5

critical System graph security Secrets conf 1.00 13 occurrences Possible secret in web/src/locales/ar.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1045, 1069, 1071, 1072, 1075, 1199, 1710, 2206, +5 more

web/src/locales/ar.ts:1045, 1069, 1071, 1072, 1075, 1199, 1710, 2206, +5 more (13 hits)

critical System graph security Secrets conf 1.00 13 occurrences Possible secret in web/src/locales/bg.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1098, 1123, 1125, 1126, 1130, 1255, 1763, 2293, +5 more

web/src/locales/bg.ts:1098, 1123, 1125, 1126, 1130, 1255, 1763, 2293, +5 more (13 hits)

critical System graph security Secrets conf 1.00 12 occurrences Possible secret in web/src/locales/de.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1136, 1162, 1164, 1168, 1297, 1819, 2351, 2591, +4 more

web/src/locales/de.ts:1136, 1162, 1164, 1168, 1297, 1819, 2351, 2591, +4 more (12 hits)

critical System graph security Secrets conf 1.00 13 occurrences Possible secret in web/src/locales/en.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1489, 1513, 1515, 1516, 1520, 1649, 2214, 2766, +5 more

web/src/locales/en.ts:1489, 1513, 1515, 1516, 1520, 1649, 2214, 2766, +5 more (13 hits)

critical System graph security Secrets conf 1.00 6 occurrences Possible secret in web/src/locales/es.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 314, 339, 341, 345, 444, 799

web/src/locales/es.ts:314, 339, 341, 345, 444, 799 (6 hits)

critical System graph security Secrets conf 1.00 13 occurrences Possible secret in web/src/locales/fr.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1144, 1169, 1171, 1172, 1176, 1277, 1973, 2616, +5 more

web/src/locales/fr.ts:1144, 1169, 1171, 1172, 1176, 1277, 1973, 2616, +5 more (13 hits)

critical System graph security Secrets conf 1.00 6 occurrences Possible secret in web/src/locales/id.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 488, 511, 513, 517, 643, 1001

web/src/locales/id.ts:488, 511, 513, 517, 643, 1001 (6 hits)

critical System graph security Secrets conf 1.00 11 occurrences Possible secret in web/src/locales/it.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 729, 754, 756, 757, 761, 1043, 1279, 1280, +3 more

web/src/locales/it.ts:729, 754, 756, 757, 761, 1043, 1279, 1280, +3 more (11 hits)

critical System graph security Secrets conf 1.00 4 occurrences Possible secret in web/src/locales/ja.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 550, 552, 556, 654

web/src/locales/ja.ts:550, 552, 556, 654 (4 hits)

critical System graph security Secrets conf 1.00 4 occurrences Possible secret in web/src/locales/pt-br.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 506, 508, 511, 610

web/src/locales/pt-br.ts:506, 508, 511, 610 (4 hits)

critical System graph security Secrets conf 1.00 13 occurrences Possible secret in web/src/locales/ru.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1205, 1230, 1232, 1233, 1237, 1363, 1882, 2425, +5 more

web/src/locales/ru.ts:1205, 1230, 1232, 1233, 1237, 1363, 1882, 2425, +5 more (13 hits)

critical System graph security Secrets conf 1.00 9 occurrences Possible secret in web/src/locales/tr.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 1362, 1364, 1365, 1368, 1487, 2843, 2844, 2845, +1 more

web/src/locales/tr.ts:1362, 1364, 1365, 1368, 1487, 2843, 2844, 2845, +1 more (9 hits)

critical System graph security Secrets conf 1.00 6 occurrences Possible secret in web/src/locales/vi.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 536, 559, 561, 564, 683, 1041

web/src/locales/vi.ts:536, 559, 561, 564, 683, 1041 (6 hits)

critical System graph security Secrets conf 1.00 Possible secret in web/src/locales/zh-traditional.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

web/src/locales/zh-traditional.ts:708

critical System graph security Secrets conf 1.00 Possible secret in web/src/locales/zh.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

web/src/locales/zh.ts:1341

critical System graph security Secrets conf 1.00 Possible secret in web/src/pages/user-setting/profile/hooks/use-profile.ts

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

web/src/pages/user-setting/profile/hooks/use-profile.ts:23

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: DELETE /:id.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:209

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: DELETE /:tenant_id/users.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:197

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: GET /:dataset_id.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:227

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: GET /:id.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:207

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: GET /:tenant_id/users.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:195

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: GET /sandbox/providers/:provider_id/schema.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/admin/router.go:117

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: PATCH /:tenant_id.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:194

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: POST /:tenant_id/users.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:196

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: PUT /:id.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/router/router.go:208

high Security checks security auth conf 0.70 [AUC003] Object-level route lacks visible authorization: A route with an object id-like parameter does not show nearby authentication or authorization evidence. This is a BOLA/IDOR review target. Endpoint: PUT /services/:service_id.

Add ownership, tenant, relationship, or policy checks before reading or mutating the target object.

internal/admin/router.go:100

low Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED006] Overcatch Baseexception: except BaseException: ... — prevents Ctrl+C and SystemExit from working.

Review and fix per the pattern semantics. See CWE-705 / for context.

3 files, 3 locations

common/constants.py:33

common/data_source/google_util/oauth_flow.py:39

rag/svr/discord_svr.py:77

high Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED020] Logging Credential Via Fstring: logger.error(f"failed for {api_key}") — secrets end up in log aggregators / sentry.

Review and fix per the pattern semantics. See CWE-532 / A09:2021 for context.

3 files, 3 locations

admin/server/auth.py:70

api/db/services/user_service.py:57

common/data_source/google_util/util.py:131

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED037] Insecure Random: random.random/randint/choice for tokens/IDs/keys instead of secrets/os.urandom.

Review and fix per the pattern semantics. See CWE-330,CWE-338 / A02:2021 for context.

rag/flow/tokenizer/tokenizer.py:131

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED040] Python Yaml Load Unsafe: yaml.load(stream) without SafeLoader can deserialize arbitrary classes.

Review and fix per the pattern semantics. See CWE-502 / for context.

common/config_utils.py:34

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED106] Phantom test coverage: test_connector: Test function `test_connector` runs code but contains no assert / expect / should call — it passes regardless of behaviour. Adds line coverage without verifying anything.

Add an explicit assertion that captures the test's intent, or remove the test.

api/apps/restful_apis/connector_api.py:183

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED106] Phantom test coverage: test_db_connection: Test function `test_db_connection` runs code but contains no assert / expect / should call — it passes regardless of behaviour. Adds line coverage without verifying anything.

Add an explicit assertion that captures the test's intent, or remove the test.

api/apps/restful_apis/agent_api.py:1052

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED106] Phantom test coverage: test_jira: Test function `test_jira` runs code but contains no assert / expect / should call — it passes regardless of behaviour. Adds line coverage without verifying anything.

Add an explicit assertion that captures the test's intent, or remove the test.

common/data_source/jira/connector.py:889

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED106] Phantom test coverage: test_mcp: Test function `test_mcp` runs code but contains no assert / expect / should call — it passes regardless of behaviour. Adds line coverage without verifying anything.

Add an explicit assertion that captures the test's intent, or remove the test.

api/apps/restful_apis/mcp_api.py:320

high Security checks quality Quality conf 1.00 ✓ Repobility 25 occurrences [MINED108] `self._find` used but never assigned in __init__: Method `_find` of class `_PsiUnionFind` reads `self._find`, but no assignment to it exists in __init__ (and no class-level fallback). This raises AttributeError the first time the method runs against an instance.

Initialize `self._find = <default>` in __init__, or add a class-level default.

2 files, 25 locations

run_tests.py:164, 167, 168, 169, 170, 171, 172, 174, +6 more (14 hits)

rag/raptor.py:83, 85, 117, 118, 126, 130, 131, 136, +1 more (11 hits)

high Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED110] Blocking call `requests.get` inside async function `delete_files`: `requests.get` is a synchronous (blocking) call. When invoked inside an `async def` it stalls the event loop, preventing every other coroutine in the process from making progress.

Use the async equivalent: `aiohttp` instead of `requests`, `asyncio.sleep` instead of `time.sleep`, `aiofiles` instead of `open`.

2 files, 3 locations

api/apps/services/file_api_service.py:249, 287 (2 hits)

rag/svr/discord_svr.py:53

high Security checks software dependencies conf 0.90 ✓ Repobility 5 occurrences [MINED118] Dockerfile FROM `python:3.11-slim-bookworm` not pinned by digest: `FROM python:3.11-slim-bookworm` resolves the tag at build time. The registry CAN re-push a different image for the same tag, so every build is potentially different. Production images should pin to `image@sha256:...` for reproducibility + supply-chain integrity.

Replace with: `FROM python:3.11-slim-bookworm@sha256:<digest>`. Get the digest from `docker manifest inspect`. Re-pin via a scheduled bot (Renovate, Dependabot).

5 files, 5 locations

Dockerfile:2

Dockerfile.scratch.oc9:1

agent/sandbox/executor_manager/Dockerfile:1

agent/sandbox/sandbox_base_image/nodejs/Dockerfile:1

agent/sandbox/sandbox_base_image/python/Dockerfile:1

high Security checks software dependencies conf 0.90 ✓ Repobility [MINED130] Lockfile pulls package from off-canonical host `registry.npmmirror.com`: `package-lock.json` resolved URL for `node_modules/@adobe/css-tools` is `https://registry.npmmirror.com/@adobe/css-tools/-/css-tools-4.4.4.tgz...` — host `registry.npmmirror.com` is not the canonical registry. Could be a mirror compromise, dependency confusion attack, or a forgotten private registry.

Verify the host is intentional. If your org uses a private registry, add it to your scanner's allowlist (CANONICAL_NPM_HOSTS). Otherwise, regenerate the lockfile against the canonical registry.

web/package-lock.json:1

high Security checks software dependencies conf 0.90 ✓ Repobility [MINED130] Lockfile pulls package from off-canonical host `registry.npmmirror.com`: `package-lock.json` resolved URL for `node_modules/proxy-from-env` is `https://registry.npmmirror.com/proxy-from-env/-/proxy-from-env-1.1.0.tgz...` — host `registry.npmmirror.com` is not the canonical registry. Could be a mirror compromise, dependency confusion attack, or a forgotten private registry.

Verify the host is intentional. If your org uses a private registry, add it to your scanner's allowlist (CANONICAL_NPM_HOSTS). Otherwise, regenerate the lockfile against the canonical registry.

agent/sandbox/sandbox_base_image/nodejs/package-lock.json:1

high Security checks software dependencies conf 0.90 ✓ Repobility 2 occurrences [MINED131] pre-commit hook `https://github.com/pre-commit/pre-commit-hooks` pinned to mutable rev `v4.6.0`: `.pre-commit-config.yaml` references `https://github.com/pre-commit/pre-commit-hooks` at `rev: v4.6.0`. If `{rev}` is a branch or version tag, the repo owner can push new code there and `pre-commit install --install-hooks` will fetch it on every developer's machine.

Pin to a commit SHA: `rev: <40-char-sha>` and bump it through `pre-commit autoupdate` (which writes to PRs that are reviewed).

lines 2, 13

.pre-commit-config.yaml:2, 13 (2 hits)

high Security checks quality Quality conf 1.00 [SEC088] Go: TLS InsecureSkipVerify=true: tls.Config{InsecureSkipVerify:true} disables certificate verification — MITM risk. Ported from gosec G402 (Apache-2.0).

Remove the option. If self-signed certs are required, pin via RootCAs.

internal/cli/http_client.go:46

high Security checks cicd CI/CD security conf 0.95 Docker final stage runs as root

Create an application user after package installation and switch to it with USER appuser or USER 10001.

Dockerfile.scratch.oc9:2 CI/CD securitycontainers

high Security checks cicd CI/CD security conf 0.95 Docker final stage runs as root

Create an application user after package installation and switch to it with USER appuser or USER 10001.

Dockerfile:169 CI/CD securitycontainers

high Security checks cicd CI/CD security conf 0.92 Dockerfile copies the entire context without .dockerignore

Create .dockerignore before using broad context copies, or copy only the required files and directories.

agent/sandbox/executor_manager/Dockerfile:36 CI/CD securitycontainers

high Security checks cicd CI/CD security conf 0.92 Dockerfile pipes a remote script into a shell

Download the artifact, verify its checksum or signature, pin the version, and then execute it.

Dockerfile:89 CI/CD securitycontainers

medium Security checks cicd CI/CD security conf 0.90 ✓ Repobility 6 occurrences GitHub Action is tag-pinned rather than SHA-pinned

[MINED115] Action `softprops/action-gh-release` pinned to mutable ref `@v2`: `uses: softprops/action-gh-release@v2` resolves at workflow-run time. Tags and branches can be re-pushed by the action owner; that made the tj-actions/changed-files compromise (2025) instantly affect ~23K repos. Pin to a 4…

2 files, 6 locations

.github/workflows/tests.yml:95, 459 (4 hits)

.github/workflows/release.yml:77 (2 hits)

CI/CD securitySupply chainGitHub Actions

low Security checks cicd CI/CD security conf 0.90 ✓ Repobility 4 occurrences GitHub Action is tag-pinned rather than SHA-pinned

[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout@v6` resolves at workflow-run time. Tags and branches can be re-pushed by the action owner; that made the tj-actions/changed-files compromise (2025) instantly affect ~23K repos. Pin to a 40-char commit SHA + lo…

2 files, 4 locations

.github/workflows/release.yml:33 (2 hits)

.github/workflows/tests.yml:46 (2 hits)

CI/CD securitySupply chainGitHub Actions

high System graph security Secrets conf 1.00 .env file present in repo: docker/.env

A raw .env file is in the working tree. Verify it isn't committed and that secrets are in a vault.

Config

high System graph security Secrets conf 1.00 .env file present in repo: web/.env

A raw .env file is in the working tree. Verify it isn't committed and that secrets are in a vault.

Config

high System graph quality Integrity conf 1.00 Blocking `requests.delete(...)` inside `async def delete_files` — api/apps/services/file_api_service.py:287

Sync I/O inside an async function blocks the event loop. While `requests.delete(...)` is running, *all* other coroutines on this loop are paused — silent throughput collapse under concurrency. Use the async equivalent (`httpx.AsyncClient`, `asyncio.sleep`, `aiofiles`) or wrap with `await asyncio.to…

api/apps/services/file_api_service.py:287 Sync io in asyncPerformance

high System graph quality Integrity conf 1.00 Blocking `requests.get(...)` inside `async def delete_files` — api/apps/services/file_api_service.py:249

Sync I/O inside an async function blocks the event loop. While `requests.get(...)` is running, *all* other coroutines on this loop are paused — silent throughput collapse under concurrency. Use the async equivalent (`httpx.AsyncClient`, `asyncio.sleep`, `aiofiles`) or wrap with `await asyncio.to_th…

api/apps/services/file_api_service.py:249 Sync io in asyncPerformance

high System graph quality Integrity conf 1.00 Blocking `requests.post(...)` inside `async def on_message` — rag/svr/discord_svr.py:53

Sync I/O inside an async function blocks the event loop. While `requests.post(...)` is running, *all* other coroutines on this loop are paused — silent throughput collapse under concurrency. Use the async equivalent (`httpx.AsyncClient`, `asyncio.sleep`, `aiofiles`) or wrap with `await asyncio.to_t…

rag/svr/discord_svr.py:53 Sync io in asyncPerformance

high System graph api Wiring conf 1.00 Dangling fetch: GET /api/v1/skills/status (web/src/pages/skills/hooks.ts:1540)

`web/src/pages/skills/hooks.ts:1540` calls `GET /api/v1/skills/status` but no backend route matches that path. This is a runtime 404 waiting to happen. Tool: fetch Normalized path used for matching: `/v1/skills/status` If this points at an external API, prefix it with `https://` so the matcher ski…

Dangling fetchFetch

high System graph api Wiring conf 1.00 Dangling fetch: GET /conf.json (web/src/hooks/logic-hooks.ts:165)

`web/src/hooks/logic-hooks.ts:165` calls `GET /conf.json` but no backend route matches that path. This is a runtime 404 waiting to happen. Tool: axios Normalized path used for matching: `/conf.json` If this points at an external API, prefix it with `https://` so the matcher skips it.

Dangling fetchAxios

high System graph api Wiring conf 1.00 Dangling fetch: GET https://github.com/infiniflow/ragflow (web/src/constants/agent.tsx:22)

`web/src/constants/agent.tsx:22` calls `GET https://github.com/infiniflow/ragflow` but no backend route matches that path. This is a runtime 404 waiting to happen. Tool: axios Normalized path used for matching: `/https:/github.com/infiniflow/ragflow` If this points at an external API, prefix it wi…

Dangling fetchAxios

high System graph api Wiring conf 1.00 Dangling fetch: POST /api/v1/skills/index (web/src/pages/skills/hooks.ts:932)

`web/src/pages/skills/hooks.ts:932` calls `POST /api/v1/skills/index` but no backend route matches that path. This is a runtime 404 waiting to happen. Tool: fetch Normalized path used for matching: `/v1/skills/index` If this points at an external API, prefix it with `https://` so the matcher skips…

Dangling fetchFetch

high System graph api Wiring conf 1.00 Dangling fetch: POST /api/v1/skills/search (web/src/pages/skills/hooks.ts:623)

`web/src/pages/skills/hooks.ts:623` calls `POST /api/v1/skills/search` but no backend route matches that path. This is a runtime 404 waiting to happen. Tool: fetch Normalized path used for matching: `/v1/skills/search` If this points at an external API, prefix it with `https://` so the matcher ski…

Dangling fetchFetch

high System graph security security conf 1.00 Insecure pattern 'eval_used' in internal/cli/filesystem/skill_hub/security/patterns.go:236

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

internal/cli/filesystem/skill_hub/security/patterns.go:236 Eval used

high System graph security security conf 1.00 Insecure pattern 'eval_used' in internal/cli/filesystem/skill_install.go:404

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

internal/cli/filesystem/skill_install.go:404 Eval used

high System graph security security conf 1.00 Insecure pattern 'eval_used' in rag/graphrag/entity_resolution.py:281

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

rag/graphrag/entity_resolution.py:281 Eval used

medium Security checks security auth conf 0.92 [AUC001] No Repobility access matrix policy found: The repository uses web/API frameworks but does not define .repobility/access.yml or equivalent authorization documentation.

Add .repobility/access.yml mapping routes to anonymous, authenticated, owner, admin, and super_admin. Keep business-specific rules in the repo so CI can enforce them.

high Security checks security auth conf 0.74 [AUC002] Low visible authorization coverage in route inventory: Only 30.3% of discovered routes show nearby authentication, authorization, middleware, or public-route evidence.

Review the access matrix and add explicit framework auth declarations or policy-file exceptions for intentionally public routes.

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: GET /auth.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:56

high Security checks security auth conf 0.66 2 occurrences [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: GET /email.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

lines 366, 389

internal/admin/handler.go:366, 389 (2 hits)

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: GET /health.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:38

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: GET /logout.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:54

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: GET /ping.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:44

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: GET /users.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:62

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: POST /login.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:45

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: POST /reports.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:47

high Security checks security auth conf 0.66 [AUC004] Admin route does not show super_admin separation: An administrative route was detected without nearby evidence that platform super_admin access is separated from tenant/application admin access. Endpoint: POST /users.

Define whether this endpoint is admin-only or super_admin-only, then enforce that distinction in code and .repobility/access.yml.

internal/admin/router.go:63

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: ANY /kbID.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/service/chunk.go:415

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: ANY /oauth2callback.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

common/data_source/box_connector.py:188

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: ANY /recover.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/service/skill_space.go:516

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: DELETE /users/:username/keys/:token.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:78

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: DELETE /users/:username/tokens/:token.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:79

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: GET /roles.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:82

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: GET /tasks.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:59

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: POST /roles.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:83

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: POST /users/:username/keys.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:76

high Security checks security auth conf 0.68 [AUC009] Sensitive function route lacks elevated authorization evidence: A route appears to perform a sensitive function such as export, invite, role, token, billing, or destructive action without elevated policy evidence. Endpoint: POST /users/:username/tokens.

Require an explicit admin, maintainer, super_admin, or scoped service role in code and .repobility/access.yml.

internal/admin/router.go:77

low Security checks quality Error handling conf 1.00 3 occurrences [ERR001] Silent Exception Swallowing: Silently swallowing all exceptions hides bugs. Even in cleanup code, log at DEBUG level.

Log the error: `except Exception: logger.debug('cleanup failed', exc_info=True)`. Or handle specific exception types.

3 files, 3 locations

agent/component/fillup.py:56

agent/component/string_transform.py:104

agent/tools/searxng.py:60

medium Security checks quality Error handling conf 1.00 [ERR002] Empty Catch Block: Empty catch blocks hide errors.

Log the error or rethrow it. Use console.error() at minimum.

web/src/pages/agent/hooks/use-download-output.ts:18

medium Security checks quality Quality conf 1.00 ✓ Repobility 5 occurrences [MINED109] Mutable default argument in `init_database_tables` (list): `def init_database_tables(... = []/{}/set())` — Python's default value is constructed ONCE at function definition time and shared across all calls. Mutating it in one call mutates it for every future call too.

Use None as the default and create the collection inside the function: `def init_database_tables(x=None): x = x or []`

4 files, 5 locations

rag/prompts/generator.py:245, 281 (2 hits)

api/db/db_models.py:674

api/db/services/document_service.py:342

api/db/services/task_service.py:76

medium Security checks quality Quality conf 1.00 ✓ Repobility 19 occurrences [MINED109] Mutable default argument in `stream_output_with_tools_async` (dict): `def stream_output_with_tools_async(... = []/{}/set())` — Python's default value is constructed ONCE at function definition time and shared across all calls. Mutating it in one call mutates it for every future call too.

Use None as the default and create the collection inside the function: `def stream_output_with_tools_async(x=None): x = x or []`

8 files, 19 locations

rag/prompts/generator.py:205, 402, 421, 443, 474, 485, 543, 784 (8 hits)

api/db/services/llm_service.py:401, 437, 483 (3 hits)

api/db/services/dialog_service.py:1540, 1656 (2 hits)

api/utils/web_utils.py:132, 142 (2 hits)

agent/component/agent_with_tools.py:263

agent/component/llm.py:472

agent/component/message.py:153

agent/component/string_transform.py:93

medium Security checks software dependencies conf 0.90 ✓ Repobility 8 occurrences [MINED124] requirements.txt: `fastapi` has no version pin: Unpinned pip requirement means every fresh install may resolve a different version. Newer releases can introduce malicious code (typosquats, account compromises). Reproducible installs need exact pins.

Replace `fastapi` with `fastapi==<version>` and manage upgrades through PRs / Dependabot.

3 files, 8 locations

agent/sandbox/sandbox_base_image/python/requirements.txt:1, 2, 3, 4 (4 hits)

agent/sandbox/executor_manager/requirements.txt:1, 2, 3 (3 hits)

tools/chatgpt-on-wechat/plugins/requirements.txt:1

low Security checks security Deserialization conf 1.00 [SEC007] Unsafe Deserialization: Unsafe deserialization can execute arbitrary code.

Use yaml.safe_load() instead of yaml.load(). Avoid pickle for untrusted data.

common/config_utils.py:34

low Security checks security Deserialization conf 1.00 [SEC007] Unsafe Deserialization: Unsafe deserialization can execute arbitrary code.

Use yaml.safe_load() instead of yaml.load(). Avoid pickle for untrusted data.

api/utils/configs.py:40

medium Security checks security Crypto conf 1.00 [SEC014] SSL Verification Disabled: SSL certificate verification is disabled, allowing man-in-the-middle attacks.

Enable SSL verification. Use verify=True (default) for requests. Pin certificates if needed.

rag/utils/minio_conn.py:38

medium Security checks software Open redirect conf 1.00 [SEC046] Client-side open redirect — window.location = server-supplied URL: Assigning window.location from a server-supplied URL trusts the server endpoint to never return a hostile destination. If that endpoint is ever subverted (compromised admin, JSON injection, MITM on a webhook), users get redirected to a phishing site they trust because the original page is yours. CWE-601 (server-side OR client-side). Complement to server-side SEC030.

Validate the URL is same-origin or on an explicit allowlist before assignment: const u = new URL(serverUrl, location.href); if (u.origin !== location.origin && !ALLOWED.includes(u.host)) return; location.assign(u); Even better: have the server return a path (/checkout/done) instead of a full …

web/src/services/user-service.ts:83

low Security checks quality Quality conf 1.00 3 occurrences [SEC136] AI-typical over-broad exception handler swallowing all errors: Catch-all exception block that silently returns success or no-ops. AI agents reach for this pattern when a flaky test or an unfamiliar API throws — wrap, swallow, return success. Real bugs are masked, observability is destroyed, and callers think the operation worked. CWE-396 (improperly-generalized exception). Distinct from intentional fallback because there's no log line and the success value is fabricated.

Catch the specific exception type, log at error level with full exception info, and return a failure-shaped result. If the operation is genuinely best-effort, log at warning and document why in a comment so the next reader (or scanner) knows.

3 files, 3 locations

agent/component/switch.py:117

api/utils/file_utils.py:129

deepdoc/parser/utils.py:40

high Security checks quality Quality conf 0.72 3 occurrences Agent control bridge may listen on a network interface without visible auth

Bind local agent bridges to 127.0.0.1 by default. If remote access is required, require a bearer token or mTLS, enforce origin/CSRF checks for browser clients, and document the threat model.

3 files, 3 locations

api/ragflow_server.py:12

api/utils/health_utils.py:12

docker/docker-compose.yml:3

low Security checks quality Error handling conf 0.55 ✓ Repobility 25 occurrences Broad exception handler needs review

This handler catches Exception/BaseException. It is actionable when it swallows errors without logging, re-raising, or returning a structured error. Handlers that intentionally convert exceptions into typed error results should not be treated as high risk.

12 files, 21 locations

common/metadata_es_filter.py:384, 431, 455, 488 (4 hits)

common/mcp_tool_call_conn.py:100, 157, 254 (3 hits)

agent/canvas.py:223, 234 (2 hits)

common/crypto_utils.py:347, 373 (2 hits)

common/tag_feature_utils.py:35, 40 (2 hits)

run_tests.py:209, 291 (2 hits)

api/validation.py:48

common/connection_utils.py:44

Error handlingquality

high Security checks security auth conf 0.82 2 occurrences Browser storage is used for session token material

Prefer httpOnly, Secure, SameSite cookies or short-lived in-memory tokens. Avoid persistent browser storage for access, refresh, ID, or partner session tokens.

lines 10, 23

web/src/utils/authorization-util.ts:10, 23 (2 hits)

medium Security checks cicd CI/CD security conf 0.94 Compose service `sandbox-executor-manager` image uses the latest tag

Pin to a maintained version tag or digest and update it deliberately through dependency automation.

agent/sandbox/docker-compose.yml:1 CI/CD securitycontainers

medium Security checks cicd CI/CD security conf 0.90 Docker build context has no .dockerignore

Add .dockerignore with at least .git, .env, private keys, dependency folders, build outputs, and local databases.

.dockerignore CI/CD securitycontainers

high Security checks cicd CI/CD security conf 0.82 3 occurrences Docker final stage has no non-root USER

Add a non-root USER in the final runtime stage after files and permissions are prepared.

3 files, 3 locations

agent/sandbox/executor_manager/Dockerfile:1

agent/sandbox/sandbox_base_image/nodejs/Dockerfile:1

agent/sandbox/sandbox_base_image/python/Dockerfile:1

CI/CD securitycontainers

medium Security checks cicd CI/CD security conf 0.90 Dockerfile installs dependencies after copying the full source tree

Copy dependency manifests first, install dependencies in a cached layer, then copy the rest of the source tree.

agent/sandbox/executor_manager/Dockerfile:38 CI/CD securitycontainers

high Security checks quality Quality conf 0.74 15 occurrences Frontend API reference is not matched by discovered backend routes

Add the backend route, update the frontend constant to the implemented endpoint, or document that the route is served by another service and exclude it with .repobilityignore.

7 files, 15 locations

web/src/utils/llm-util.ts:87, 88, 91, 92, 93 (5 hits)

web/src/pages/skills/hooks.ts:623, 932, 1540 (3 hits)

web/vite.config.ts:61, 90, 95 (3 hits)

web/src/components/next-markdown-content/index.tsx:49

web/src/interfaces/database/dataset.ts:2

web/src/pages/agent/hooks/use-send-shared-message.ts:62

web/src/utils/api.ts:2

medium Security checks quality Quality conf 0.68 Ollama audio payload path may mislead users about direct model audio

Gate direct audio sending on a verified runtime capability check. Until supported, show a one-time notice that voice is transcribed in the browser and only text is sent to the model.

rag/llm/tts_model.py:365

medium Security checks quality Quality conf 0.70 Public web app has no Content Security Policy

Add a Content-Security-Policy header through the web framework or hosting config. For static apps, add a CSP meta tag that restricts default-src, script-src, connect-src, img-src, and frame-ancestors.

index.html

medium Security checks quality Quality conf 0.78 Public web service has no security.txt

Add /.well-known/security.txt with Contact, Expires, Canonical, Preferred-Languages, and Policy fields. Keep the contact endpoint monitored.

.well-known/security.txt

medium Security checks quality Quality conf 0.78 Suspicious implementation file appears unreferenced

Confirm whether this file is reachable. If not, delete it; if yes, wire it through explicit imports, routes, or entry points and add a test that proves the path executes.

internal/common/kg_query_rewrite.go:1

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/components/document-preview/doc-preview.tsx:131

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/components/floating-chat-widget-markdown.tsx:188

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/components/markdown-content/index.tsx:178

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/components/next-markdown-content/index.tsx:290

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/components/originui/select-with-search.tsx:214

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/components/parse-configuration/graph-rag-form-fields.tsx:170

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/hooks/use-warn-empty-model.tsx:30

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/chunk/parsed-result/add-knowledge/components/knowledge-chunk/components/chunk-card/index.tsx:130

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataflow-result/components/chunk-card/index.tsx:104

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataflow-result/components/rerun-button/index.tsx:24

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataflow-result/index.tsx:102

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataset/dataset-setting/category-panel.tsx:42

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataset/dataset-setting/components/link-data-pipeline.tsx:35

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataset/dataset-setting/components/tag-item.tsx:48

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataset/dataset/generate-button/generate.tsx:292

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/dataset/dataset/set-meta-dialog.tsx:96

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/next-search/markdown-content/index.tsx:176

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

Fq dangerous html

medium System graph frontend Frontend quality conf 1.00 `dangerouslySetInnerHTML` used in a React component — web/src/pages/user-setting/data-source/component/delete-source-modal.tsx:27

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

Fq dangerous html

medium System graph quality Integrity conf 1.00 `fetch()` without try/.catch or AbortSignal — web/src/components/ui/audio-button.tsx:257

Bare `fetch(...)` will throw an unhandled rejection on network failure. Wrap in try/catch, attach a `.catch(...)`, or pass an AbortSignal with a timeout.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 `fetch()` without try/.catch or AbortSignal — web/src/pages/skills/components/upload-modal.tsx:496

Bare `fetch(...)` will throw an unhandled rejection on network failure. Wrap in try/catch, attach a `.catch(...)`, or pass an AbortSignal with a timeout.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 `fetch()` without try/.catch or AbortSignal — web/src/pages/skills/hooks.ts:932

Bare `fetch(...)` will throw an unhandled rejection on network failure. Wrap in try/catch, attach a `.catch(...)`, or pass an AbortSignal with a timeout.

runtime safetyRobustness

medium System graph hardware Security conf 1.00 Dockerfile runs as root: agent/sandbox/executor_manager/Dockerfile

No non-root USER set. Containers running as root expand the blast radius of any vulnerability inside the image.

Container

medium System graph hardware Security conf 1.00 Dockerfile runs as root: agent/sandbox/sandbox_base_image/nodejs/Dockerfile

No non-root USER set. Containers running as root expand the blast radius of any vulnerability inside the image.

Container

medium System graph hardware Security conf 1.00 Dockerfile runs as root: agent/sandbox/sandbox_base_image/python/Dockerfile

No non-root USER set. Containers running as root expand the blast radius of any vulnerability inside the image.

Container

medium System graph hardware Security conf 1.00 Dockerfile runs as root: Dockerfile

No non-root USER set. Containers running as root expand the blast radius of any vulnerability inside the image.

Container

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 1.00 Insecure pattern 'cors_wildcard' in api/apps/__init__.py:62

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

api/apps/__init__.py:62 Cors wildcard

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/components/document-preview/doc-preview.tsx:131

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

web/src/components/document-preview/doc-preview.tsx:131 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/components/floating-chat-widget-markdown.tsx:188

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

web/src/components/floating-chat-widget-markdown.tsx:188 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/components/markdown-content/index.tsx:178

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

web/src/components/markdown-content/index.tsx:178 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/components/next-markdown-content/index.tsx:290

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

web/src/components/next-markdown-content/index.tsx:290 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/components/originui/select-with-search.tsx:214

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

web/src/components/originui/select-with-search.tsx:214 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/components/parse-configuration/graph-rag-form-fields.tsx:170

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

web/src/components/parse-configuration/graph-rag-form-fields.tsx:170 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/hooks/use-warn-empty-model.tsx:30

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

web/src/hooks/use-warn-empty-model.tsx:30 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/chunk/parsed-result/add-knowledge/components/knowledge-chunk/components/chunk-card/index.tsx:130

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

web/src/pages/chunk/parsed-result/add-knowledge/components/knowledge-chunk/components/chunk-card/index.tsx:130 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataflow-result/components/chunk-card/index.tsx:104

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

web/src/pages/dataflow-result/components/chunk-card/index.tsx:104 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataflow-result/components/rerun-button/index.tsx:24

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

web/src/pages/dataflow-result/components/rerun-button/index.tsx:24 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataflow-result/index.tsx:102

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

web/src/pages/dataflow-result/index.tsx:102 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataset/dataset-setting/category-panel.tsx:42

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

web/src/pages/dataset/dataset-setting/category-panel.tsx:42 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataset/dataset-setting/components/link-data-pipeline.tsx:35

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

web/src/pages/dataset/dataset-setting/components/link-data-pipeline.tsx:35 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataset/dataset-setting/components/tag-item.tsx:48

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

web/src/pages/dataset/dataset-setting/components/tag-item.tsx:48 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataset/dataset/generate-button/generate.tsx:292

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

web/src/pages/dataset/dataset/generate-button/generate.tsx:292 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/dataset/dataset/set-meta-dialog.tsx:96

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

web/src/pages/dataset/dataset/set-meta-dialog.tsx:96 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/next-search/markdown-content/index.tsx:176

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

web/src/pages/next-search/markdown-content/index.tsx:176 Dangerous innerhtml

medium System graph security security conf 1.00 Insecure pattern 'dangerous_innerhtml' in web/src/pages/user-setting/data-source/component/delete-source-modal.tsx:27

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

web/src/pages/user-setting/data-source/component/delete-source-modal.tsx:27 Dangerous innerhtml

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — agent/sandbox/providers/local.py:126

`subprocess.Popen(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — agent/tools/github.py:78

`requests.get(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — agent/tools/jin10.py:73

`requests.get(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — agent/tools/qweather.py:87

`requests.get(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — agent/tools/tushare.py:65

`requests.post(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — common/data_source/utils.py:992

`requests.post(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — common/data_source/zendesk_connector.py:55

`requests.get(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — deepdoc/parser/tcadp_parser.py:181

`requests.get(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — download_deps.py:72

`urllib.request.urlretrieve(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — rag/svr/discord_svr.py:53

`requests.post(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — sdk/python/ragflow_sdk/ragflow.py:37

`requests.post(...)` 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.

runtime safetyRobustness

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — test/testcases/utils/engine_utils.py:41

`requests.get(...)` 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.

runtime safetyRobustness

medium System graph network Security conf 1.00 Privileged port 13 in use

Port 13 is privileged (<1024). Make sure the service runs with the right caps or front it with a non-privileged port via a load balancer.

.github/workflows/release.yml Ports

medium System graph network Security conf 1.00 Privileged port 21 in use

Port 21 is privileged (<1024). Make sure the service runs with the right caps or front it with a non-privileged port via a load balancer.

.github/workflows/release.yml Ports

low Security checks quality Error handling conf 1.00 3 occurrences [ERR003] Ignored Error (Go): Ignoring error return values.

Handle the error or use errcheck linter.

3 files, 3 locations

internal/common/logger.go:150

internal/entity/models/mineru_local.go:135

internal/entity/models/paddleocr.go:164

high Security checks cicd CI/CD security conf 0.56 3 occurrences Compose service does not declare a runtime user

Set a non-root `user:` in Compose or ensure the final image stage has a non-root USER directive.

2 files, 3 locations

docker/docker-compose.yml:4, 55 (2 hits)

agent/sandbox/docker-compose.yml:1

CI/CD securitycontainers

high Security checks cicd CI/CD security conf 0.62 2 occurrences Compose service lacks no-new-privileges hardening

Add `security_opt: ["no-new-privileges:true"]` unless the service has a documented need for privilege escalation.

lines 4, 55

docker/docker-compose.yml:4, 55 (2 hits)

CI/CD securitycontainers

low Security checks cicd CI/CD security conf 0.72 3 occurrences Dockerfile installs recommended OS packages

Add `--no-install-recommends` and explicitly list only packages the image needs.

2 files, 3 locations

Dockerfile:89, 99 (2 hits)

agent/sandbox/executor_manager/Dockerfile:5

CI/CD securitycontainers

high Security checks cicd CI/CD security conf 0.72 5 occurrences Dockerfile keeps pip download cache

Use `pip install --no-cache-dir ...` in container builds.

3 files, 5 locations

Dockerfile.scratch.oc9:43, 44, 50 (3 hits)

agent/sandbox/executor_manager/Dockerfile:38

agent/sandbox/sandbox_base_image/python/Dockerfile:13

CI/CD securitycontainers

low Security checks cicd CI/CD security conf 0.74 3 occurrences Dockerfile leaves apt package indexes in the image layer

End the apt install layer with `rm -rf /var/lib/apt/lists/*`.

lines 34, 89, 99

Dockerfile:34, 89, 99 (3 hits)

CI/CD securitycontainers

low Security checks quality Quality conf 0.60 29 occurrences Duplicated implementation block across source files

Duplicate implementation blocks are maintenance debt. Keep them visible, but they are not a high-severity defect unless the duplicated logic is security-sensitive or drifting.

12 files, 16 locations

common/data_source/interfaces.py:131, 238 (2 hits)

common/data_source/outlook_connector.py:5, 7 (2 hits)

common/data_source/salesforce_connector.py:17, 18 (2 hits)

common/data_source/teams_connector.py:16, 21 (2 hits)

agent/tools/__init__.py:1

agent/tools/retrieval.py:118

api/apps/services/provider_api_service.py:317

api/db/init_data.py:42

duplicationquality

low Security checks quality Quality conf 0.68 Multiple AI-agent scaffold marker files are present

Keep one current agent instruction file if it helps contributors, remove stale progress/completion markers, and make sure the README, tests, and CI describe the real supported behavior.

.github/copilot-instructions.md:1

low Security checks quality Quality conf 0.64 Public docs site has no llms.txt

Add llms.txt with the product summary, canonical docs, API endpoints, security guidance, and preferred CLI workflow for AI agents.

llms.txt

low Security checks quality Quality conf 0.50 Public web app has no humans.txt

Add humans.txt with team ownership, contact URL, key documentation links, and the last-updated date.

humans.txt

low Security checks quality Quality conf 0.74 Public web app has no robots.txt

Add robots.txt at the web root or a framework-native robots route. Include an explicit Sitemap directive and disallow only private paths.

robots.txt

low Security checks quality Quality conf 0.72 Public web app has no sitemap

Add sitemap.xml, a sitemap index, or a framework-native sitemap route and reference it from robots.txt.

sitemap.xml

high Security checks quality Quality conf 0.62 Source file name looks like an AI patch artifact

Rename it to the domain concept it implements or merge it into the existing module it was meant to change.

internal/common/kg_query_rewrite.go:1

low System graph quality Maintenance conf 1.00 126 TODO/FIXME markers

High count of TODO/FIXME/HACK markers — track them as issues so they're not forgotten.

low System graph quality Integrity conf 1.00 235 env vars used in code but missing from .env.example

Drift between code and config docs. The first few: `ACCOUNT_URL`, `AGENTRUN_ACCESS_KEY_ID`, `AGENTRUN_ACCESS_KEY_SECRET`, `AGENTRUN_ACCOUNT_ID`, `AGENTRUN_REGION`, `AIRTABLE_ACCESS_TOKEN`, `AIRTABLE_CONNECTOR_SIZE_THRESHOLD`, `API_PROXY_SCHEME` + 227 more. Add them (with a placeholder/comment) to .…

config drift

low System graph hardware Supply chain conf 1.00 Docker base image is tag-pinned but not digest-pinned: node:24.13-bookworm-slim

Container tags can be retagged upstream. Pin production base images to a reviewed digest (`image@sha256:...`) when reproducibility and supply-chain integrity matter.

agent/sandbox/sandbox_base_image/nodejs/Dockerfile:1 containersPinned dependencies

low System graph hardware Supply chain conf 1.00 Docker base image is tag-pinned but not digest-pinned: python:3.11-slim-bookworm

Container tags can be retagged upstream. Pin production base images to a reviewed digest (`image@sha256:...`) when reproducibility and supply-chain integrity matter.

agent/sandbox/executor_manager/Dockerfile:1 containersPinned dependencies

low System graph hardware Supply chain conf 1.00 Docker base image is tag-pinned but not digest-pinned: python:3.11-slim-bookworm

Container tags can be retagged upstream. Pin production base images to a reviewed digest (`image@sha256:...`) when reproducibility and supply-chain integrity matter.

agent/sandbox/sandbox_base_image/python/Dockerfile:1 containersPinned dependencies

low System graph hardware Supply chain conf 1.00 Docker base image is tag-pinned but not digest-pinned: ubuntu:24.04

Container tags can be retagged upstream. Pin production base images to a reviewed digest (`image@sha256:...`) when reproducibility and supply-chain integrity matter.

Dockerfile:2 containersPinned dependencies

low System graph software Dead code candidate conf 1.00 File has no detected symbols: admin/server/admin_server.py