Skip to content

feat: TanStack Query Migration Phase 2 - Cleanup and Test Reorganization #588

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Wirasm merged 19 commits into from
Sep 6, 2025
Merged

feat: TanStack Query Migration Phase 2 - Cleanup and Test Reorganization #588

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
6f88689
refactor: migrate layouts to TanStack Query and Radix UI patterns
Wirasm Sep 4, 2025
5bcee1a
test: reorganize test infrastructure with colocated tests in subdirec…
Wirasm Sep 4, 2025
1dcecad
fix: use error boundary wrapper for ProjectPage
Wirasm Sep 4, 2025
5dc7a8b
cleanup: remove unused MCP client components
Wirasm Sep 4, 2025
6b50877
cleanup: remove unused mcpService.ts
Wirasm Sep 4, 2025
bfdc2fd
cleanup: remove unused mcpClientService and update deprecation comments
Wirasm Sep 4, 2025
4d9f53c
fix: correct test directory exclusion in coverage config
Wirasm Sep 5, 2025
02cc372
docs: fix ArchonChatPanel import path in agent-chat.mdx
Wirasm Sep 5, 2025
1687d70
refactor: improve backend health hook and types
Wirasm Sep 5, 2025
84f470f
fix: remove .d.ts exclusion from production TypeScript config
Wirasm Sep 5, 2025
f355c42
feat: implement modern MCP feature architecture
Wirasm Sep 5, 2025
9404bf4
refactor: complete MCP modernization and cleanup
Wirasm Sep 5, 2025
f4a8ecb
fix: correct MCP transport mode display and complete cleanup
Wirasm Sep 5, 2025
ca45ed4
simplify MCP API to status-only endpoints
Wirasm Sep 5, 2025
2b42f90
feat: complete MCP feature migration to TanStack Query
Wirasm Sep 5, 2025
949ff35
refactor: clean up unused CSS and unify Tron-themed scrollbars
Wirasm Sep 5, 2025
db2dc0b
Merge remote-tracking branch 'origin/main' into feat/tanstack-migrati…
Wirasm Sep 6, 2025
8550727
fix: address CodeRabbit CSS review feedback
Wirasm Sep 6, 2025
8b9dfa3
fix: properly close Docker client to prevent resource leak
Wirasm Sep 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions archon-ui-main/.dockerignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,16 @@ docker-compose.yml
# Tests
coverage
test-results
tests/
**/*.test.ts
**/*.test.tsx
**/*.spec.ts
**/*.spec.tsx
**/__tests__
**/*.e2e.test.ts
**/*.integration.test.ts
vitest.config.ts
tsconfig.prod.json

# Documentation
README.md
Expand Down
296 changes: 0 additions & 296 deletions archon-ui-main/__mocks__/lucide-react.tsx
View file
Open in desktop

This file was deleted.

2 changes: 1 addition & 1 deletion archon-ui-main/biome.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"$schema": "https://biomejs.dev/schemas/2.2.2/schema.json",
"files": {
"includes": ["src/features/**"]
"includes": ["src/features/**", "src/components/layout/**"]
},
"formatter": {
"enabled": true,
Expand Down
2 changes: 1 addition & 1 deletion archon-ui-main/src/App.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { KnowledgeBasePage } from './pages/KnowledgeBasePage';
import { SettingsPage } from './pages/SettingsPage';
import { MCPPage } from './pages/MCPPage';
import { OnboardingPage } from './pages/OnboardingPage';
import { MainLayout } from './components/layouts/MainLayout';
import { MainLayout } from './components/layout/MainLayout';
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Sep 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

Import path update looks correct; verify no remaining references to old layouts path

Ensure there are no lingering imports from ./components/layouts/MainLayout or SideNavigation.

🏁 Script executed:

#!/bin/bash
# Find any remaining references to old layout paths
rg -nP -C2 $'/components/layouts/(MainLayout|SideNavigation)\\.(tsx|ts)'

Length of output: 72

🏁 Script executed:

#!/bin/bash
# Search entire repo for any references to the old 'components/layouts' path
rg -n "components/layouts" -C2 .

# Check if the 'components/layouts' directory still exists
fd --type d 'components/layouts'

Length of output: 313

🏁 Script executed:

#!/bin/bash
# Locate the ArchonChatPanel component file to confirm its new path
rg -n "ArchonChatPanel" -C2 src/components
fd --type f --extension tsx ArchonChatPanel

Length of output: 204

🏁 Script executed:

#!/bin/bash
# Search for any lingering SideNavigation imports or references
rg -n "SideNavigation" -C2 .

Length of output: 743

Update MDX docs to reference new layout and agent-chat paths

  • docs/docs/agent-chat.mdx (line 48): change
    import { ArchonChatPanel } from '@/components/layouts/ArchonChatPanel'
    →
    import { ArchonChatPanel } from '@/components/agent-chat/ArchonChatPanel'
  • docs/docs/ui-components.mdx (line 92): update <SideNavigation /> import to
    @/components/layout/SideNavigation
  • docs/docs/ui.mdx (line 25): in the component tree, replace components/layouts with components/layout for both MainLayout and SideNavigation
🤖 Prompt for AI Agents 
In archon-ui-main/src/App.tsx around line 9 and the docs mentioned
(docs/docs/agent-chat.mdx line 48, docs/docs/ui-components.mdx line 92,
docs/docs/ui.mdx line 25), update MDX import paths and component-tree references
to match the new folder structure: change the agent-chat import to
"@/components/agent-chat/ArchonChatPanel", update the SideNavigation import to
"@/components/layout/SideNavigation", and replace any occurrences of
"components/layouts" with "components/layout" for MainLayout and SideNavigation
in the component tree; ensure path casing and aliases match the project’s
tsconfig/webpack aliases.

import { ThemeProvider } from './contexts/ThemeContext';
import { ToastProvider } from './contexts/ToastContext';
import { ToastProvider as FeaturesToastProvider } from './features/ui/components/ToastProvider';
Expand Down
0 ...rc/components/layouts/ArchonChatPanel.tsx → ...components/agent-chat/ArchonChatPanel.tsx
View file
Open in desktop
File renamed without changes.
Loading