Skip to content

Commit 3cf3fc0

Browse files
committed
Clean up frontend parsing errors and remove redundant MCP status page
- Fix console statement wrapping in InteractiveGraphVisualization.tsx - Remove redundant /mcp-status route and McpStatus.tsx component - Remove "View MCP Server Health Status" link from Agents page - Keep MCP status functionality available in AI & Agents page - All console statements now properly wrapped with DEV environment checks
1 parent 965b74f commit 3cf3fc0

4 files changed

Lines changed: 9 additions & 275 deletions

File tree

packages/web/src/App.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import { Agents } from './pages/Agents';
66
import { Analytics } from './pages/Analytics';
77
import { Settings } from './pages/Settings';
88
import { Backend } from './pages/Backend';
9-
import { McpStatus } from './pages/McpStatus';
109
import { Login } from './pages/Login';
1110
import { GraphVisualization } from './components/GraphVisualization';
1211
import { AuthProvider, useAuth } from './contexts/AuthContext';
@@ -59,7 +58,6 @@ function AuthenticatedApp() {
5958
<Route path="/analytics" element={<Analytics />} />
6059
<Route path="/settings" element={<Settings />} />
6160
<Route path="/backend" element={<Backend />} />
62-
<Route path="/mcp-status" element={<McpStatus />} />
6361
</Routes>
6462
</Layout>
6563
</GraphProvider>

packages/web/src/components/InteractiveGraphVisualization.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -780,7 +780,9 @@ export function InteractiveGraphVisualization() {
780780
g.attr('transform', event.transform);
781781
});
782782

783-
console.log('✅ Visualization initialized with', nodes.length, 'nodes');
783+
if (import.meta.env.DEV) {
784+
console.log('✅ Visualization initialized with', nodes.length, 'nodes');
785+
}
784786
}, [nodes, validatedEdges, handleNodeClick]); // Include handleNodeClick to get fresh connection state
785787

786788
// Store simulation reference for resize handling
@@ -789,7 +791,9 @@ export function InteractiveGraphVisualization() {
789791
// Initialization effect - NOW with access to nodes data
790792
useEffect(() => {
791793
if (nodes.length > 0) {
792-
console.log('useEffect: Initializing visualization with', nodes.length, 'nodes');
794+
if (import.meta.env.DEV) {
795+
console.log('useEffect: Initializing visualization with', nodes.length, 'nodes');
796+
}
793797
initializeVisualization();
794798
}
795799

@@ -813,7 +817,9 @@ export function InteractiveGraphVisualization() {
813817
.alpha(0.3) // Restart simulation with some energy
814818
.restart();
815819

816-
console.log('🔄 Resized visualization to', newWidth, 'x', newHeight);
820+
if (import.meta.env.DEV) {
821+
console.log('🔄 Resized visualization to', newWidth, 'x', newHeight);
822+
}
817823
};
818824

819825
window.addEventListener('resize', handleResize);

packages/web/src/pages/Agents.tsx

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -118,13 +118,6 @@ export function Agents() {
118118
<p className="text-sm text-gray-400 mt-1">
119119
Collaborate with AI agents and configure MCP servers for your work graph
120120
</p>
121-
<Link
122-
to="/mcp-status"
123-
className="inline-flex items-center mt-2 text-xs text-blue-400 hover:text-blue-300 transition-colors"
124-
>
125-
<Server className="w-3 h-3 mr-1" />
126-
View MCP Server Health Status →
127-
</Link>
128121
</div>
129122

130123
<div className="flex items-center space-x-3">

packages/web/src/pages/McpStatus.tsx

Lines changed: 0 additions & 263 deletions
This file was deleted.

0 commit comments

Comments
 (0)