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

ccedacero/nyc-property-intel

https://github.com/ccedacero/nyc-property-intel · scanned 2026-06-17 01:46 UTC (1 month, 1 week ago)

43 raw signals (0 security + 43 graph)

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

Complete repo analysis

Last scanned 1 month, 1 week ago · v2 · last Δ +9.0 (diff) · 43 actionable findings from 1 signal source. Security checks, system graph analysis, and verified AI-agent feedback are merged into one review queue.

JSON
Combined
85.7
/100
Security checks
86
0 findings
System graph
86
66.7% cov · 43 findings
Critical
0
click to filter
Agents
10
0 votes
Crowd
0
reported
Severity distribution — click a segment to filter
Active filters: excluding tests × Reset all
Severity: Critical 0 High 0 Medium 10 Low 18 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Security checks 0 System graph 43 Crowd 0 Layer: Software 9 Security 3 Quality 31
Corpus Intelligence Cross-corpus context (cohort percentile, top patterns, fix plan) is shown only on repositories you own. Sign up and connect your repo to view it.
Scan summary Repository scanned at 85.7/100 with 66.7% coverage. It contains 1204 nodes across 0 cross-layer flows, written primarily in mixed languages. Engine surfaced 43 findings — concentrated in quality (31), software (9), security (3). Risk profile is low: 0 critical, 0 high, 10 medium. Recommended next step: open the quality layer findings first — that's where the highest-impact wins live.

Showing 28 of 43 actionable findings. Click TP / FP to vote on a finding's accuracy — votes adjust the confidence weighting and improve detection across the platform.

medium System graph quality Integrity conf 1.00 `fetch()` without try/.catch or AbortSignal — site/js/main.js:173
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 — site/js/report.js:69
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 Agent instructions conf 1.00 Agent authority lacks a verifier contract: .mcp.json
This agent instruction grants code or shell authority but does not state the verification gate that decides promotion. The recurring safe pattern is: LLM proposes; deterministic tests/build/security checks verify; only verified code promotes.
.mcp.json VerificationMcp config
medium System graph quality Placeholder conf 1.00 Critical user flow still appears backed by mock or placeholder data
A payment/auth/admin/order/billing-style flow contains mock, fake, TODO, dummy, or placeholder markers in runtime source. In the Fable corpus this is a high-leverage completeness smell: the app can look finished while the money, identity, or tenant flow is still scaffolded.
Mock dataCritical flowGenerated repo pattern
medium System graph security security conf 1.00 Insecure pattern 'direct_innerhtml_assignment' in site/js/chat.js:310
Found a known-risky pattern (direct_innerhtml_assignment). Review and replace if possible.
site/js/chat.js:310 Direct innerhtml assignment
medium System graph security security conf 1.00 Insecure pattern 'direct_innerhtml_assignment' in site/js/report.js:36
Found a known-risky pattern (direct_innerhtml_assignment). Review and replace if possible.
site/js/report.js:36 Direct innerhtml assignment
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — download_and_verify.py:56
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — scripts/load_local_datasets.py:28
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — scripts/run_backfill.py:48
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph security Coverage conf 1.00 No auth library detected
The scanner did not find any standard auth library (JWT, OAuth, NextAuth, Auth0, etc.). The repo has auth/admin/session surface indicators, so auth may live in custom code, in a separate service, or be missing.
auth
low System graph quality Integrity conf 1.00 12 env vars used in code but missing from .env.example
Drift between code and config docs. The first few: `ALERT_EMAIL_TO`, `ALERT_FROM_EMAIL`, `ANON_IP_HASH_SECRET`, `BACKFILL_DATASETS`, `BACKFILL_RESET`, `MCP_ALLOWED_HOSTS`, `MCP_TRANSPORT`, `PORT` + 4 more. Add them (with a placeholder/comment) to .env.example so onboarding doesn't break.
config drift
low System graph quality Debug conf 1.00 Debug logging residue appears in source files
Found 236 console/debugger/print-style debug statements in non-test source. This is a common fast-generation residue before production cleanup.
CleanupRepo hardeningGenerated repo pattern
low System graph quality Integrity conf 1.00 Near-duplicate function bodies in 2 places
Functions with the same first-5-line body hash: scripts/sync_delta.py:upsert_page, scripts/sync_delta.py:refresh_page_by_doc This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hygiene). Consolidate or document why they're separate.
duplicatesduplication
low System graph quality Integrity conf 1.00 Near-duplicate function bodies in 4 places
Functions with the same first-5-line body hash: scripts/column_null_audit.py:main, scripts/dashboard.py:run, scripts/signup_dashboard.py:run, scripts/coverage_audit.py:main This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hygiene). Consolidat…
duplicatesduplication
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `_dev_unused` in tests/test_analysis_unit.py:329
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `meta_deprecated` in scripts/coverage_audit.py:114
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `skipped_old` in scripts/load_rolling_sales.py:109
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph software Dead code conf 1.00 Possibly dead Python function: buttonize
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
scripts/build_blog_page.py:101
low System graph software Dead code conf 1.00 Possibly dead Python function: close_client
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
src/nyc_property_intel/socrata.py:61
low System graph software Dead code conf 1.00 Possibly dead Python function: close_client
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
src/nyc_property_intel/geoclient.py:53
low System graph software Dead code conf 1.00 Possibly dead Python function: ext_links
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
scripts/build_blog_page.py:112
low System graph software Dead code conf 1.00 Possibly dead Python function: invalidate_cache
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
src/nyc_property_intel/auth.py:176
low System graph software Dead code conf 1.00 Possibly dead Python function: probe_db_all
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
scripts/manual_qa.py:65
low System graph software Dead code conf 1.00 Possibly dead Python function: repl
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
scripts/clean_blog_citations.py:78
low System graph software Dead code conf 1.00 Possibly dead Python function: send_wrapper
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
src/nyc_property_intel/server.py:298
low System graph software Dead code conf 1.00 Possibly dead Python function: signup_endpoint
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
src/nyc_property_intel/chat.py:1422
low System graph quality Complexity conf 1.00 Very large file: scripts/manual_qa.py (1379 lines)
Files with >800 lines often hide complexity hotspots and discourage tests.
low System graph quality Complexity conf 1.00 Very large file: src/nyc_property_intel/chat.py (1553 lines)
Files with >800 lines often hide complexity hotspots and discourage tests.
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/d7d4b239-4e5c-4c4f-b4cf-1c6cdb54c996/

To check status programmatically (no auth required): 

curl -s https://repobility.com/api/v1/public/scan/d7d4b239-4e5c-4c4f-b4cf-1c6cdb54c996/

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.