Coleam00, you’ve definitely shown up to class but skipped the parts about neatness and clarity. Four giant files topping a thousand lines each is less "masterpiece" and more "please save me from scrolling." Nine error handling issues and seventy-six code structure problems? That’s like turning in a paper with every paragraph in a different language. On the bright side, you aced test coverage, security, dependencies, and documentation — it’s like you did your homework but left the desk a disaster zone.
Error Handling:Nine issues mean exceptions are probably throwing more tantrums than your code.
Code Structure:Seventy-six issues and eight levels deep of nesting? The code’s more tangled than last year’s Christmas lights.
Test Coverage:94% — Well. Someone actually read the syllabus and did the work here.
Security:Perfect score — at least the gates are locked even if the house is a mess.
Dependencies:Clean as a whistle; no extra baggage hitching a ride.
Documentation:Spot-on, which makes the chaos just a bit more bearable for anyone brave enough to look.
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.