ds2api

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.

Scan timing: clone 2.23s · analysis 4.88s · 4.0 MB · GitHub API rate-limit (preflight)

https://github.com/CJackHwang/ds2api.git · scanned 2026-05-24 01:22 UTC (1 week, 5 days ago) · 10 languages

191 findings (128 legacy + 63 scanner) 64th percentile · Go · medium (20-100K LoC)

File as issue Image

UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

Last scanned 1 week, 5 days ago · v1 · 191 findings from 2 sources. Findings combine the legacy security pipeline AND the multi-layer engine (atlas, wiring, flows, ranked) AND verified AI agent contributions.

JSON

Score breakdown â 2026-05-18-v5

Component	Sub-score	Weight	Contribution
`structure_score`	85.0	0.15	12.75
`security_score`	59.5	0.25	14.88
`testing_score`	85.0	0.20	17.00
`documentation_score`	60.0	0.15	9.00
`practices_score`	97.0	0.15	14.55
`code_quality`	60.7	0.10	6.07
Overall		1.00	74.2

Severity distribution — click a segment to filter

Active filters: excluding tests × Reset all

Severity: Critical 0 High 91 Medium 25 Low 51 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Legacy 128 9-layer 63 Crowd 0 Layer: Security 39 Quality 67 Cicd 38 Software 33 Api 1 Frontend 8 Hardware 5

Exclude dismissed / FP Exclude test files

Bug-class explainers. Each card groups findings of the same shape — these are the patterns most likely to ship to prod and reappear in future scans unless you systematically fix the cause, not just the instance.

Fragile runtime 61 findings

What it is: Code that runs but breaks under predictable input — division by zero, missing keys, unbounded loops, off-by-one slicing.

Why it matters: Reaches production undetected because happy-path tests pass. First user with a weird input crashes the request.

How AI causes it: AI loves writing the happy path; doesn't probe edge cases unless explicitly asked.

Fix approach: Add property-based tests. Wrap external inputs with explicit validators. Use the framework's typed deserializer (Pydantic, attrs).

12 matching findings on this repo

medium [SEC134] AI scaffold leftover — Lorem ipsum / example.com / John Doe in code: L… webui/src/features/account/AddAccountModal.jsx:51
medium [SEC091] Go: net/http server without timeouts: HTTP server without ReadHeaderTi… cmd/ds2api/main.go:37
medium [ERR002] Empty Catch Block: Empty catch blocks hide errors. webui/src/features/apiTester/useChatStreamClient.…:210
low [SEC132] String concat where the language has interpolation (AI style drift): S… internal/httpapi/claude/handler_utils.go:131
low [SEC132] String concat where the language has interpolation (AI style drift): S… internal/httpapi/admin/vercel/handler_vercel.go:169
low [ERR003] Ignored Error (Go): Ignoring error return values. internal/auth/admin.go:215
low [ERR003] Ignored Error (Go): Ignoring error return values. cmd/ds2api/main.go:26
low [ERR003] Ignored Error (Go): Ignoring error return values. cmd/ds2api-tests/main.go:33
low Duplicated implementation block across source files internal/responsehistory/session.go:77
low Duplicated implementation block across source files internal/prompt/tool_calls.go:54
low Duplicated implementation block across source files internal/js/helpers/stream-tool-sieve/state.js:79
low Duplicated implementation block across source files internal/js/helpers/stream-tool-sieve/format.js:200

View all fragile runtime findings →

Legacy markers 1 finding

What it is: TODO, FIXME, XXX, HACK comments. Often indicate a known-broken path the author meant to fix.

Why it matters: Each marker is an unfinished thought. Production code shouldn't ship with debt that's documented but not tracked.

How AI causes it: AI mirrors the style of the codebase, so existing TODOs propagate into new code.

Fix approach: Convert each into a ticket. Delete the comment when the ticket lands. Use a pre-commit hook to block new TODOs without an issue link.

1 matching finding on this repo

low Legacy-named symbol `ds2api_env_config_draft_v1` in webui/src/app/useAdminConfi…

View all legacy markers findings →

Config drift 7 findings

What it is: Settings duplicated across env files, Docker compose, K8s, and code defaults, all with slightly different values.

Why it matters: Production behaviour depends on whichever copy your loader reads first. Subtle bugs in staging that don't reproduce in dev.

How AI causes it: AI writes new config from memory rather than reading the existing source.

Fix approach: Pick one source of truth (env vars + a settings module). Have every other place import from there. Lint for duplicates in CI.

7 matching findings on this repo

low File has no detected symbols: webui/vite.config.js
low File has no detected symbols: webui/tailwind.config.js
low File has no detected symbols: webui/postcss.config.js
low Stray `console.log` in TS/JS — webui/vite.config.js:28
info `truncate` class without `title=` for hover reveal — webui/src/features/apiTest…
low Legacy-named symbol `ds2api_env_config_draft_v1` in webui/src/app/useAdminConfi…
low 7 env vars used in code but missing from .env.example

View all config drift findings →

For AI agents: Voting guide (TP/FP) MCP manifest Stdio wrapper SARIF Integrate Findings queue Vote TP/FP on findings to calibrate the engine.

For AI agents + API integrations

Email me when this repo regresses

Free. We re-scan periodically; new criticals → your inbox. No signup required for the scan itself.

API access

This page is publicly accessible at: https://repobility.com/scan/7754021f-50b9-4416-9f65-4f7e38264c3b/

To check status programmatically (no auth required):

curl -s https://repobility.com/api/v1/public/scan/7754021f-50b9-4416-9f65-4f7e38264c3b/

Important — please don't re-submit the same URL repeatedly. The submission endpoint is idempotent: re-submitting the same git URL returns this same scan_token, not a new one. To re-scan this repo, sign up free and use the dashboard.