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.
Sign up free Scan another repo

Scan timing: clone 23.93s · analysis 12.82s · 36.1 MB · GitHub API rate-limit (preflight)

holaboss-ai/holaOS

https://github.com/holaboss-ai/holaOS · scanned 2026-05-31 01:26 UTC (5 days, 7 hours ago) · 10 languages

915 findings (167 legacy + 748 scanner) 11/13 scanners ran 37th percentile · Typescript · large (100-500K LoC) Scanner says 67 (higher by 6)

File as issue Image v2 schema
UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

Last scanned 5 days, 7 hours ago · v2 · last Δ +4.9 (diff) · 609 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
Combined
72.3
/100
Repobility
78
167 legacy
9-layer
67
100.0% cov · 442 gaps
Critical
28
click to filter
Agents
6
0 votes
Crowd
0
reported
Score breakdown â 2026-05-18-v5
Component Sub-score Weight Contribution
structure_score 60.0 0.15 9.00
security_score 100.0 0.25 25.00
testing_score 58.0 0.20 11.60
documentation_score 65.0 0.15 9.75
practices_score 70.0 0.15 10.50
code_quality 70.0 0.10 7.00
Overall 1.00 72.8
security_score may be inflated — optional security scanners were skipped on this fast scan
Severity distribution — click a segment to filter
Active filters: excluding tests × Reset all
Severity: Critical 28 High 106 Medium 51 Low 233 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Legacy 167 9-layer 442 Crowd 0 Layer: Quality 278 Security 19 Cicd 48 Software 106 Frontend 101 Hardware 2 Api 55

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 116 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 Agent control bridge may listen on a network interface without visible auth runtime/api-server/src/runner-worker.ts:219
  • medium localStorage write failures are swallowed silently apps/desktop/src/lib/workspaceSelection.tsx:26
  • medium localStorage write failures are swallowed silently apps/desktop/src/lib/chat/useChatComposerModelSel…:197
  • medium localStorage write failures are swallowed silently apps/desktop/src/features/workspace-onboarding/pr…:30
  • medium localStorage write failures are swallowed silently apps/desktop/src/components/publish/usePublishDra…:97
  • medium localStorage write failures are swallowed silently apps/desktop/src/components/panes/ChatPane/index.…:5320
  • medium localStorage write failures are swallowed silently apps/desktop/src/components/layout/new-shell/useS…:97
  • medium localStorage write failures are swallowed silently apps/desktop/src/components/layout/SettingsScreen…:915
  • medium localStorage write failures are swallowed silently apps/desktop/src/components/layout/AppShell.tsx:1441
  • medium [SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographicall… sdk/app-builder-sdk/src/runtime/state.ts:103
  • medium [SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographicall… runtime/harnesses/src/embedded-skills/app-builder…:103
  • medium [ERR002] Empty Catch Block: Empty catch blocks hide errors. runtime/api-server/src/session-scratchpad.ts:147
View all fragile runtime findings →
Legacy markers 20 findings
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.
12 matching findings on this repo
  • low Legacy-named symbol `workspace_copy` in apps/desktop/electron/preload.ts:483
  • low Legacy-named symbol `runtime_installation_state_legacy` in apps/desktop/electro…
  • low Legacy-named symbol `workspace_copy` in apps/desktop/electron/browser-pane/impo…
  • low Legacy-named symbol `workspace_copy` in apps/desktop/electron/browser-pane/type…
  • low Legacy-named symbol `workspace_copy` in apps/desktop/src/types/electron.d.ts:594
  • low Legacy-named symbol `certIssuerDNLegacy` in website/docs/worker-configuration.d…
  • low Legacy-named symbol `system__copy` in website/docs/app/components/DiagramSystem…
  • low Legacy-named symbol `ca_tw_v1` in runtime/state-store/src/store.test.ts:1718
  • low Legacy-named symbol `_sandbox_runtime_push_v1` in runtime/api-server/src/ts-run…
  • low Legacy-named symbol `_sandbox_runtime_exec_v1` in runtime/api-server/src/claime…
  • low Legacy-named symbol `_sandbox_runtime_exec_v1` in runtime/api-server/src/ts-run…
  • low Legacy-named symbol `_sandbox_runtime_exec_v1` in runtime/api-server/src/agent-…
View all legacy markers findings →
Commented-out code 95 findings
What it is: Lines of source that were intentionally disabled but never deleted.
Why it matters: Git already remembers history — commented code rots, becomes wrong, and adds noise to diffs.
How AI causes it: AI sometimes comments out broken code instead of fixing it. Reviewers approve out of inertia.
Fix approach: Delete. Trust `git log`. If you really need to remember, save it in a notes file under `docs/`.
12 matching findings on this repo
  • info Commented-code block (7 lines) in apps/desktop/electron/workspace-packager.ts:3…
  • info Commented-code block (5 lines) in apps/desktop/electron/preload.ts:1823
  • info Commented-code block (9 lines) in apps/desktop/electron/main.ts:14
  • info Commented-code block (8 lines) in apps/desktop/electron/browser-pane/index.ts:70
  • info Commented-code block (6 lines) in apps/desktop/electron/browser-pane/tab-state.…
  • info Commented-code block (5 lines) in apps/desktop/src/App.tsx:53
  • info Commented-code block (8 lines) in apps/desktop/src/types/electron.d.ts:429
  • info Commented-code block (5 lines) in apps/desktop/src/components/layout/AppShell.t…
  • info Commented-code block (8 lines) in apps/desktop/src/components/layout/WorkspaceC…
  • info Commented-code block (5 lines) in apps/desktop/src/components/layout/Notificati…
  • info Commented-code block (5 lines) in apps/desktop/src/components/layout/new-shell/…
  • info Commented-code block (6 lines) in apps/desktop/src/components/layout/new-shell/…
View all commented-out code findings →
Config drift 29 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.
12 matching findings on this repo
  • high [MINED113] Express DELETE /api/v1/integrations/oauth/configs/:providerId has no… runtime/api-server/src/app.ts:5557
  • high [MINED113] Express PUT /api/v1/integrations/oauth/configs/:providerId has no au… runtime/api-server/src/app.ts:5532
  • high [MINED113] Express PUT /api/v1/runtime/config has no auth: Express route PUT /a… runtime/api-server/src/app.ts:4818
  • low File has no detected symbols: apps/desktop/vite.config.ts
  • low File has no detected symbols: apps/desktop/tsup.config.ts
  • low File has no detected symbols: website/docs/vite.config.ts
  • low File has no detected symbols: website/docs/react-router.config.ts
  • low File has no detected symbols: website/docs/source.config.ts
  • low File has no detected symbols: runtime/state-store/tsup.config.ts
  • low File has no detected symbols: runtime/harness-host/tsup.config.ts
  • low File has no detected symbols: runtime/api-server/tsup.config.ts
  • low Stray `console.log` in TS/JS — website/docs/worker-configuration.d.ts:179
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/2bb252b3-baf7-4896-a58a-4c45dc20c51c/

To check status programmatically (no auth required): 

curl -s https://repobility.com/api/v1/public/scan/2bb252b3-baf7-4896-a58a-4c45dc20c51c/

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.