From 435e0af5228c69b8a621b5a0e9f8877c6ac4d37f Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Sun, 7 Dec 2025 23:54:21 +0000
Subject: [PATCH] cleanup: remove debug console.log statements from route
 components

Phase 6 partial cleanup:
- Removed [DockerRoute], [HostsRoute], [DashboardView] console logs
- Updated architecture doc to reflect completed cleanup

Remaining Phase 6 tasks (future):
- Remove legacy arrays from State (keep fallback for safety)
- Remove legacy AI context fallback
- Simplify route components to use resources directly
---
 .gemini/docs/unified-resource-architecture.md | 5 +++--
 frontend-modern/src/App.tsx                   | 3 ---
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/.gemini/docs/unified-resource-architecture.md b/.gemini/docs/unified-resource-architecture.md
index 61bbeac..aa87657 100644
--- a/.gemini/docs/unified-resource-architecture.md
+++ b/.gemini/docs/unified-resource-architecture.md
@@ -545,9 +545,10 @@ The dedicated `/resources` unified view was abandoned in favor of migrating exis
    - All pages can consume from unified model with legacy fallback
 
 4. **Future Cleanup (Phase 6):**
-   - [ ] Remove legacy arrays from State once migration is verified
+   - [x] Remove debug console.log statements from frontend routes
+   - [ ] Remove legacy arrays from State once migration is verified stable (keep fallback for safety)
    - [ ] Remove legacy AI context fallback
-   - [ ] Simplify route components once adapters are proven stable
+   - [ ] Simplify route components to use resources directly (remove adapter layer)
 
 ---
 
diff --git a/frontend-modern/src/App.tsx b/frontend-modern/src/App.tsx
index e576c8b..2689971 100644
--- a/frontend-modern/src/App.tsx
+++ b/frontend-modern/src/App.tsx
@@ -107,7 +107,6 @@ function DockerRoute() {
 
     // If we have unified resources, convert and reconstruct hierarchy
     if (dockerHostResources.length > 0) {
-      console.log('[DockerRoute] Using unified resources:', dockerHostResources.length, 'hosts,', dockerContainerResources.length, 'containers');
       return dockerHostResources.map(h => {
         const platformData = h.platformData as Record<string, unknown> | undefined;
 
@@ -207,7 +206,6 @@ function HostsRoute() {
     // If we have unified resources, convert them to legacy Host format
     // (This will be simplified once HostsOverview is migrated to use resources directly)
     if (unifiedHosts.length > 0) {
-      console.log('[HostsRoute] Using unified resources:', unifiedHosts.length, 'hosts');
       return unifiedHosts.map(r => {
         const platformData = r.platformData as Record<string, unknown> | undefined;
         return {
@@ -878,7 +876,6 @@ function App() {
     const vms = createMemo(() => {
       const vmResources = state().resources?.filter(r => r.type === 'vm') ?? [];
       if (vmResources.length > 0) {
-        console.log('[DashboardView] Using unified resources: VMs:', vmResources.length);
         return vmResources.map(r => {
           const platformData = r.platformData as Record<string, unknown> | undefined;
           return {