danvics/Pulse
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

cleanup: remove debug console.log statements from route components

Browse source 
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
This commit is contained in:
rcourtman 2025-12-07 23:54:21 +00:00
parent cc4b32ad4a
commit 435e0af522
2 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.gemini/docs/unified-resource-architecture.md
View file

@ -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)
---

3
frontend-modern/src/App.tsx
View file

@ -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 {