Coleam00, you turned in a sprawling metropolis of code—634 files, over 140k lines—like you were trying to break a world record for Most Lines Without Refactoring. Those behemoth files in the settings and layouts scream “see me after class” louder than a fire alarm. Test coverage and security are the rare bright spots here, so at least you’re showing up and locking the doors, but the nine error handling slip-ups and 76 code structure issues look like you didn’t even do the homework twice. If you want to move beyond this chaos, start slicing those giant files and flattening those nesting nightmares before the parent-teacher conference.
Error Handling:Nine issues flagged—your error handling is basically a “try and hope” approach, which doesn’t inspire confidence.
Code Structure:Seventy-six problems and zero points. This is a code spaghetti nightmare; large files and eight-level deep nesting read like a cry for help.
Test Coverage:At 94 and just one issue, someone actually ran their tests—reluctant thumbs up for effort and follow-through.
Security:Perfect score here. I’m impressed; you locked down your code better than Fort Knox.
Dependencies:Clean as a whistle and fully approved. You didn’t bring any unwanted guests to this party.
Documentation:Spotless and complete—well, someone actually read the syllabus and took notes for once.
archon-ui-main/src/components/settings/OllamaConfigurationPanel.tsx is 877 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/settings/OllamaModelDiscoveryModal.tsx is 893 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/settings/OllamaModelSelectionModal.tsx is 1141 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/settings/RAGSettings.tsx is 2507 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/features/knowledge/hooks/useKnowledgeQueries.ts is 809 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/features/projects/tasks/hooks/useTaskEditor.ts has 8 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
archon-ui-main/src/features/style-guide/layouts/AgentWorkOrderLayoutExample.tsx is 1320 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/features/style-guide/layouts/ProjectsLayoutExample.tsx is 939 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
high·migration/complete_setup.sql
Large file (1375 lines) — this is a real problem
1375 LOC, threshold is 400
migration/complete_setup.sql is 1375 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
high·python/src/agent_work_orders/api/routes.py
Large file (948 lines) — this is a real problem
948 LOC, threshold is 400
python/src/agent_work_orders/api/routes.py is 948 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
high·python/src/agents/document_agent.py
Large file (858 lines) — this is a real problem
858 LOC, threshold is 400
python/src/agents/document_agent.py is 858 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/mcp_server/features/projects/project_tools.py has 8 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
high·python/src/server/api_routes/ollama_api.py
Large file (1331 lines) — this is a real problem
1331 LOC, threshold is 400
python/src/server/api_routes/ollama_api.py is 1331 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
high·python/src/server/api_routes/projects_api.py
Large file (1278 lines) — this is a real problem
1278 LOC, threshold is 400
python/src/server/api_routes/projects_api.py is 1278 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
high·python/src/server/api_routes/projects_api.py
Deep nesting detected (8 levels) — refactor this
Max nesting depth: 8
python/src/server/api_routes/projects_api.py has 8 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/crawling/crawling_service.py is 1074 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/server/services/crawling/crawling_service.py has 9 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/llm_provider_service.py is 1250 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/server/services/ollama/model_discovery_service.py is 1122 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/server/services/storage/code_storage_service.py is 1410 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/server/services/storage/code_storage_service.py has 8 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/storage/document_storage_service.py has 8 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
catch (localError) {
console.warn('Failed to backup to localStorage:', localError);
}
In archon-ui-main/src/components/settings/OllamaConfigurationPanel.tsx at line 96, the catch block only console.logs the error. Add proper error handling — return an error response, show a user-facing message, or re-throw.
catch (error) {
console.warn('Failed to load saved model preferences:', error);
}
In archon-ui-main/src/components/settings/OllamaConfigurationPanel.tsx at line 453, the catch block only console.logs the error. Add proper error handling — return an error response, show a user-facing message, or re-throw.
High console.log density (36 instances) — likely debug code left in production
36 console.log calls in 893 lines
In archon-ui-main/src/components/settings/OllamaModelDiscoveryModal.tsx, there are 36 console.log statements. Replace with a proper logging library or remove debug logs before shipping.
High console.log density (22 instances) — likely debug code left in production
22 console.log calls in 1141 lines
In archon-ui-main/src/components/settings/OllamaModelSelectionModal.tsx, there are 22 console.log statements. Replace with a proper logging library or remove debug logs before shipping.
High console.log density (14 instances) — likely debug code left in production
14 console.log calls in 2507 lines
In archon-ui-main/src/components/settings/RAGSettings.tsx, there are 14 console.log statements. Replace with a proper logging library or remove debug logs before shipping.
catch (error) {
console.warn("Clipboard API failed, trying fallback:", error);
}
In archon-ui-main/src/features/shared/utils/clipboard.ts at line 23, the catch block only console.logs the error. Add proper error handling — return an error response, show a user-facing message, or re-throw.
medium·archon-ui-main/src/utils/onboarding.ts
High console.log density (5 instances) — likely debug code left in production
5 console.log calls in 79 lines
In archon-ui-main/src/utils/onboarding.ts, there are 5 console.log statements. Replace with a proper logging library or remove debug logs before shipping.
High console.log density (28 instances) — likely debug code left in production
28 console.log calls in 110 lines
In archon-ui-main/tests/manual/test-knowledge-api.ts, there are 28 console.log statements. Replace with a proper logging library or remove debug logs before shipping.
medium·archon-ui-main/vite.config.ts
High console.log density (12 instances) — likely debug code left in production
12 console.log calls in 443 lines
In archon-ui-main/vite.config.ts, there are 12 console.log statements. Replace with a proper logging library or remove debug logs before shipping.
medium
Low test-to-source ratio (26.4%)
96 test files, 481 source files
Test coverage is below average (26.4%). Consider adding more test coverage, especially for edge cases.
Large file (679 lines) — should probably split this
679 LOC, threshold is 400
archon-ui-main/src/components/settings/ButtonPlayground.tsx is 679 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (6 levels) — getting hard to follow
Max nesting depth: 6
archon-ui-main/src/features/knowledge/services/knowledgeService.ts has 6 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Large file (604 lines) — should probably split this
604 LOC, threshold is 400
archon-ui-main/src/services/credentialsService.ts is 604 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
medium·python/src/mcp_server/mcp_server.py
Large file (630 lines) — should probably split this
630 LOC, threshold is 400
python/src/mcp_server/mcp_server.py is 630 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (7 levels) — getting hard to follow
Max nesting depth: 7
python/src/server/api_routes/knowledge_api.py has 7 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
medium·python/src/server/api_routes/ollama_api.py
Deep nesting detected (7 levels) — getting hard to follow
Max nesting depth: 7
python/src/server/api_routes/ollama_api.py has 7 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Large file (558 lines) — should probably split this
558 LOC, threshold is 400
python/src/server/services/crawling/discovery_service.py is 558 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (6 levels) — getting hard to follow
Max nesting depth: 6
python/src/server/services/crawling/discovery_service.py has 6 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Large file (707 lines) — should probably split this
707 LOC, threshold is 400
python/src/server/services/crawling/helpers/url_handler.py is 707 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (6 levels) — getting hard to follow
Max nesting depth: 6
python/src/server/services/crawling/strategies/recursive.py has 6 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Large file (632 lines) — should probably split this
632 LOC, threshold is 400
python/src/server/services/credential_service.py is 632 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Large file (533 lines) — should probably split this
533 LOC, threshold is 400
python/src/server/services/embeddings/embedding_service.py is 533 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (7 levels) — getting hard to follow
Max nesting depth: 7
python/src/server/services/embeddings/embedding_service.py has 7 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (7 levels) — getting hard to follow
Max nesting depth: 7
python/src/server/services/llm_provider_service.py has 7 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (7 levels) — getting hard to follow
Max nesting depth: 7
python/src/server/services/ollama/model_discovery_service.py has 7 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Large file (526 lines) — should probably split this
526 LOC, threshold is 400
python/src/server/services/projects/task_service.py is 526 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Large file (557 lines) — should probably split this
557 LOC, threshold is 400
python/src/server/services/provider_discovery_service.py is 557 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (6 levels) — getting hard to follow
Max nesting depth: 6
python/src/server/services/provider_discovery_service.py has 6 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (6 levels) — getting hard to follow
Max nesting depth: 6
python/src/server/services/search/agentic_rag_strategy.py has 6 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Large file (658 lines) — should probably split this
658 LOC, threshold is 400
python/src/server/services/source_management_service.py is 658 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Large file (542 lines) — should probably split this
542 LOC, threshold is 400
python/src/server/services/storage/document_storage_service.py is 542 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Large file (585 lines) — should probably split this
585 LOC, threshold is 400
python/src/server/services/threading_service.py is 585 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (6 levels) — getting hard to follow
Max nesting depth: 6
python/src/server/utils/document_processing.py has 6 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
archon-ui-main/src/components/agent-chat/ArchonChatPanel.tsx is 458 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/bug-report/BugReportModal.tsx is 426 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/code/CodeViewerModal.tsx is 447 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/settings/APIKeysSection.tsx is 403 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/components/settings/FeaturesSection.tsx is 436 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
archon-ui-main/src/features/projects/views/ProjectsView.tsx is 413 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
archon-ui-main/src/features/style-guide/layouts/ProjectsLayoutExample.tsx has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
low·archon-ui-main/src/services/ollamaService.ts
Large file (485 lines) — barely over threshold
485 LOC, threshold is 400
archon-ui-main/src/services/ollamaService.ts is 485 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
low·archon-ui-main/vite.config.ts
Large file (443 lines) — barely over threshold
443 LOC, threshold is 400
archon-ui-main/vite.config.ts is 443 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
low·python/src/agent_work_orders/api/routes.py
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/agent_work_orders/api/routes.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/agent_work_orders/state_manager/supabase_repository.py is 484 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/agent_work_orders/workflow_engine/workflow_operations.py is 461 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
low·python/src/agents/base_agent.py
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/agents/base_agent.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
low·python/src/agents/document_agent.py
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/agents/document_agent.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
low·python/src/agents/rag_agent.py
Large file (420 lines) — barely over threshold
420 LOC, threshold is 400
python/src/agents/rag_agent.py is 420 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/crawling/code_extraction_service.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/crawling/document_storage_operations.py is 473 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/crawling/document_storage_operations.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/crawling/helpers/url_handler.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/crawling/strategies/batch.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/crawling/strategies/single_page.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/credential_service.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/knowledge/knowledge_item_service.py is 479 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
python/src/server/services/ollama/embedding_router.py is 451 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/projects/project_service.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/search/agentic_rag_strategy.py is 408 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/search/base_search_strategy.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
python/src/server/services/search/rag_service.py is 491 lines long. Break it into smaller, focused modules. Each file should have one clear responsibility.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/search/rag_service.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/source_management_service.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.
Deep nesting detected (5 levels) — common in real code
Max nesting depth: 5
python/src/server/services/threading_service.py has 5 levels of nesting. Use early returns, extract helper functions, or restructure conditionals to flatten the code.