mirror of
https://github.com/siteboon/claudecodeui.git
synced 2026-03-11 08:57:38 +00:00
* refactor: remove unused TasksSettings component
* refactor: migrate TodoList component to a new file with improved structure and normalization logic
* refactor: Move Tooltip and DarkModeToggle to shared/ui
* refactor: Move Tooltip and DarkModeToggle to shared/view/ui
* refactor: move GeminiLogo to llm-logo-provider and update imports
* refactor: remove unused GeminiStatus component
* refactor: move components in src/components/ui to src/shared/view/ui
* refactor: move ErrorBoundary component to main-content/view and update imports
* refactor: move VersionUpgradeModal to its own module
* refactor(wizard): rebuild project creation flow as modular TypeScript components
Replace the monolithic `ProjectCreationWizard.jsx` with a feature-based TS
implementation under `src/components/project-creation-wizard`, while preserving
existing behavior and improving readability, maintainability, and state isolation.
Why:
- The previous wizard mixed API logic, flow state, folder browsing, and UI in one file.
- Refactoring and testing were difficult due to tightly coupled concerns.
- We needed stronger type safety and localized component state.
What changed:
- Deleted:
- `src/components/ProjectCreationWizard.jsx`
- Added new modular structure:
- `src/components/project-creation-wizard/index.ts`
- `src/components/project-creation-wizard/ProjectCreationWizard.tsx`
- `src/components/project-creation-wizard/types.ts`
- `src/components/project-creation-wizard/data/workspaceApi.ts`
- `src/components/project-creation-wizard/hooks/useGithubTokens.ts`
- `src/components/project-creation-wizard/utils/pathUtils.ts`
- `src/components/project-creation-wizard/components/*`
- `WizardProgress`, `WizardFooter`, `ErrorBanner`
- `StepTypeSelection`, `StepConfiguration`, `StepReview`
- `WorkspacePathField`, `GithubAuthenticationCard`, `FolderBrowserModal`
- Updated import usage:
- `src/components/sidebar/view/subcomponents/SidebarModals.tsx`
now imports from `../../../project-creation-wizard`.
Implementation details:
- Migrated wizard logic to TypeScript using `type` aliases only.
- Kept component prop types colocated in each component file.
- Split responsibilities by feature:
- container/orchestration in `ProjectCreationWizard.tsx`
- API/SSE and request parsing in `data/workspaceApi.ts`
- GitHub token loading/caching behavior in `useGithubTokens`
- path/URL helpers in `utils/pathUtils.ts`
- Localized UI-only state to child components:
- folder browser modal state (current path, hidden folders, create-folder input)
- path suggestion dropdown state with debounced lookup
- Preserved existing UX flows:
- step navigation and validation
- existing/new workspace modes
- optional GitHub clone + auth modes
- clone progress via SSE
- folder browsing + folder creation
- Added focused comments for non-obvious logic (debounce, SSE auth constraint, path edge cases).
* refactor(quick-settings): migrate panel to typed feature-based modules
Refactor QuickSettingsPanel from a single JSX component into a modular TypeScript feature structure while preserving behavior and translations.
Highlights:
- Replace legacy src/components/QuickSettingsPanel.jsx with a typed entrypoint (src/components/QuickSettingsPanel.tsx).
- Introduce src/components/quick-settings-panel/ with clear separation of concerns:
- view/: panel shell, header, handle, section wrappers, toggle rows, and content sections.
- hooks/: drag interactions and whisper mode persistence.
- constants.ts and types.ts for shared config and strict local typing.
- Move drag logic into useQuickSettingsDrag with explicit touch/mouse handling, drag threshold detection, click suppression after drag, position clamping, and localStorage persistence.
- Keep user-visible behavior intact:
- same open/close panel interactions.
- same mobile/desktop drag behavior and persisted handle position.
- same quick preference toggles and wiring to useUiPreferences.
- same hidden whisper section behavior and localStorage/event updates.
- Improve readability and maintainability by extracting repetitive setting rows and section scaffolding into reusable components.
- Add focused comments around non-obvious behavior (drag click suppression, touch scroll lock, hidden whisper section intent).
- Keep files small and reviewable (all new/changed files are under 300 lines).
Validation:
- npm run typecheck
- npm run build
* refactor(quick-settings-panel): restructure QuickSettingsPanel import and create index file
* refactor(shared): move shared ui components to share/view/ui without subfolders
* refactor(LanguageSelector): move LanguageSelector to shared UI components
* refactor(prd-editor): modularize PRD editor with typed feature modules
Break the legacy PRDEditor.jsx monolith into a feature-based TypeScript architecture under src/components/prd-editor while keeping behavior parity and readability.
Key changes:
- Replace PRDEditor.jsx with a typed orchestrator component and a compatibility export bridge at src/components/PRDEditor.tsx.
- Split responsibilities into dedicated hooks: document loading/init, existing PRD registry fetching, save workflow with overwrite detection, and keyboard shortcuts.
- Split UI into focused view components: header, editor/preview body, footer stats, loading state, generate-tasks modal, and overwrite-confirm modal.
- Move filename concerns into utility helpers (sanitize, extension handling, default naming) and centralize template/constants.
- Keep component-local state close to the UI that owns it (workspace controls/modal toggles), while shared workflow state remains in the feature container.
- Reuse the existing MarkdownPreview component for safer markdown rendering instead of ad-hoc HTML conversion.
- Update TaskMasterPanel integration to consume typed PRDEditor directly (remove any-cast) and pass isExisting metadata for correct overwrite behavior.
- Keep all new/changed files below 300 lines and add targeted comments where behavior needs clarification.
Validation:
- npm run typecheck
- npm run build
* refactor(TaskMasterPanel): update PRDEditor import path to match new structure
* refactor(TaskMaster): Remove unused TaskMasterSetupWizard and TaskMasterStatus components
* refactor(TaskDetail): remove unused TaskIndicator import
* refactor(task-master): migrate tasks to a typed feature module
- introduce a new feature-oriented TaskMaster domain under src/components/task-master
- add typed TaskMaster context/provider with explicit project, task, MCP, and loading state handling
- split task UI into focused components (panel, board, toolbar, content, card, detail modal, setup/help modals, banner)
- move task board filtering/sorting/kanban derivation into dedicated hooks and utilities
- relocate CreateTaskModal into the feature module and keep task views modular/readable
- remove legacy monolithic TaskList/TaskDetail/TaskCard files and route main task panel to the new feature panel
- replace contexts/TaskMasterContext.jsx with a typed contexts/TaskMasterContext.ts re-export to the feature context
- update MainContent project sync logic to compare by project name to avoid state churn
- validation: npm run typecheck, npm run build
* refactor(MobileNav): remove unused React import and TaskMasterContext
* refactor(auth): migrate login and setup flows to typed feature module
- Introduce a new feature-based auth module under src/components/auth with clear separation of concerns:\n - context/AuthContext.tsx for session lifecycle, onboarding status checks, token persistence, and auth actions\n - view/* components for loading, route guarding, form layout, input fields, and error display\n - shared auth constants, utility helpers, and type aliases (no interfaces)\n- Convert login and setup UIs to TypeScript and keep form state local to each component for readability and component-level ownership\n- Add explicit API payload typing and safe JSON parsing helpers to improve resilience when backend responses are malformed or incomplete\n- Centralize error fallback handling for auth requests to reduce repeated logic
- Replace legacy auth entrypoints with the new feature module in app wiring:\n - App now imports AuthProvider and ProtectedRoute from src/components/auth\n - WebSocketContext, TaskMasterContext, and Onboarding now consume useAuth from the new typed auth context\n- Remove duplicated legacy auth screens (LoginForm.jsx, SetupForm.jsx, ProtectedRoute.jsx)\n- Keep backward compatibility by turning src/contexts/AuthContext.jsx into a thin re-export of the new provider/hook
Result: auth code now follows a feature/domain structure, is fully typed, easier to navigate, and cleaner to extend without touching unrelated UI areas.
* refactor(AppContent): update MobileNav import path and add MobileNav component
* refactor(DiffViewer): rename different diff viewers and place them in different components
* refactor(components): reorganize onboarding/provider auth/sidebar indicator into domain features
- Move onboarding out of root-level components into a dedicated feature module:
- add src/components/onboarding/view/Onboarding.tsx
- split onboarding UI into focused subcomponents:
- OnboardingStepProgress
- GitConfigurationStep
- AgentConnectionsStep
- AgentConnectionCard
- add onboarding-local types and utils for provider status and validation helpers
- Move multi-provider login modal into a dedicated provider-auth feature:
- add src/components/provider-auth/view/ProviderLoginModal.tsx
- add src/components/provider-auth/types.ts
- keep provider-specific command/title behavior and Gemini setup guidance
- preserve compatibility for both onboarding flow and settings login flow
- Move TaskIndicator into the sidebar domain:
- add src/components/sidebar/view/subcomponents/TaskIndicator.tsx
- update SidebarProjectItem to consume local sidebar TaskIndicator
- Update integration points to the new structure:
- ProtectedRoute now imports onboarding from onboarding feature
- Settings now imports ProviderLoginModal directly (remove legacy cast wrapper)
- git panel consumers now import shared GitDiffViewer by explicit name
- Rename git shared diff view to clearer domain naming:
- replace shared DiffViewer with shared GitDiffViewer
- update FileChangeItem and CommitHistoryItem imports accordingly
- Remove superseded root-level legacy components:
- delete src/components/LoginModal.jsx
- delete src/components/Onboarding.jsx
- delete src/components/TaskIndicator.jsx
- delete old src/components/git-panel/view/shared/DiffViewer.tsx
- Result:
- clearer feature boundaries (auth vs onboarding vs provider-auth vs sidebar)
- easier navigation and ownership by domain
- preserved runtime behavior with improved readability and modularity
* refactor(MainContent): remove TaskMasterPanel import and relocate to task-master component
* fix: update import paths for Input component in FileTree and FileTreeNode
* refactor(FileTree): make file tree context menu a typescript component and move it inside the file tree view
* refactor(FileTree): remove unused ScrollArea import
* feat: setup eslint with typescript and react rules, add unused imports plugin
* fix: remove unused imports, functions, and types after discovering using `npm run lint`
* feat: setup eslint-plugin-react, react-refresh, import-x, and tailwindcss plugins with recommended rules and configurations
* chore: reformat files after running `npm run lint:fix`
* chore: add omments about eslint config plugin uses
* feat: add husky and lint-staged for pre-commit linting
* feat: setup commitlint with conventional config
* fix: i18n translations
---------
Co-authored-by: Haileyesus <something@gmail.com>
Co-authored-by: viper151 <simosmik@gmail.com>
264 lines
10 KiB
TypeScript
264 lines
10 KiB
TypeScript
import { Check, ChevronDown, Download, GitBranch, Plus, RefreshCw, Upload } from 'lucide-react';
|
|
import { useEffect, useRef, useState } from 'react';
|
|
import type { ConfirmationRequest, GitRemoteStatus } from '../types/types';
|
|
import NewBranchModal from './modals/NewBranchModal';
|
|
|
|
type GitPanelHeaderProps = {
|
|
isMobile: boolean;
|
|
currentBranch: string;
|
|
branches: string[];
|
|
remoteStatus: GitRemoteStatus | null;
|
|
isLoading: boolean;
|
|
isCreatingBranch: boolean;
|
|
isFetching: boolean;
|
|
isPulling: boolean;
|
|
isPushing: boolean;
|
|
isPublishing: boolean;
|
|
onRefresh: () => void;
|
|
onSwitchBranch: (branchName: string) => Promise<boolean>;
|
|
onCreateBranch: (branchName: string) => Promise<boolean>;
|
|
onFetch: () => Promise<void>;
|
|
onPull: () => Promise<void>;
|
|
onPush: () => Promise<void>;
|
|
onPublish: () => Promise<void>;
|
|
onRequestConfirmation: (request: ConfirmationRequest) => void;
|
|
};
|
|
|
|
export default function GitPanelHeader({
|
|
isMobile,
|
|
currentBranch,
|
|
branches,
|
|
remoteStatus,
|
|
isLoading,
|
|
isCreatingBranch,
|
|
isFetching,
|
|
isPulling,
|
|
isPushing,
|
|
isPublishing,
|
|
onRefresh,
|
|
onSwitchBranch,
|
|
onCreateBranch,
|
|
onFetch,
|
|
onPull,
|
|
onPush,
|
|
onPublish,
|
|
onRequestConfirmation,
|
|
}: GitPanelHeaderProps) {
|
|
const [showBranchDropdown, setShowBranchDropdown] = useState(false);
|
|
const [showNewBranchModal, setShowNewBranchModal] = useState(false);
|
|
const dropdownRef = useRef<HTMLDivElement | null>(null);
|
|
|
|
useEffect(() => {
|
|
const handleClickOutside = (event: MouseEvent) => {
|
|
if (dropdownRef.current && !dropdownRef.current.contains(event.target as Node)) {
|
|
setShowBranchDropdown(false);
|
|
}
|
|
};
|
|
|
|
document.addEventListener('mousedown', handleClickOutside);
|
|
return () => document.removeEventListener('mousedown', handleClickOutside);
|
|
}, []);
|
|
|
|
const aheadCount = remoteStatus?.ahead || 0;
|
|
const behindCount = remoteStatus?.behind || 0;
|
|
const remoteName = remoteStatus?.remoteName || 'remote';
|
|
const shouldShowFetchButton = aheadCount > 0 && behindCount > 0;
|
|
|
|
const requestPullConfirmation = () => {
|
|
onRequestConfirmation({
|
|
type: 'pull',
|
|
message: `Pull ${behindCount} commit${behindCount !== 1 ? 's' : ''} from ${remoteName}?`,
|
|
onConfirm: onPull,
|
|
});
|
|
};
|
|
|
|
const requestPushConfirmation = () => {
|
|
onRequestConfirmation({
|
|
type: 'push',
|
|
message: `Push ${aheadCount} commit${aheadCount !== 1 ? 's' : ''} to ${remoteName}?`,
|
|
onConfirm: onPush,
|
|
});
|
|
};
|
|
|
|
const requestPublishConfirmation = () => {
|
|
onRequestConfirmation({
|
|
type: 'publish',
|
|
message: `Publish branch "${currentBranch}" to ${remoteName}?`,
|
|
onConfirm: onPublish,
|
|
});
|
|
};
|
|
|
|
const handleSwitchBranch = async (branchName: string) => {
|
|
try {
|
|
const success = await onSwitchBranch(branchName);
|
|
if (success) {
|
|
setShowBranchDropdown(false);
|
|
}
|
|
} catch (error) {
|
|
console.error('[GitPanelHeader] Failed to switch branch:', error);
|
|
}
|
|
};
|
|
|
|
const handleFetch = async () => {
|
|
try {
|
|
await onFetch();
|
|
} catch (error) {
|
|
console.error('[GitPanelHeader] Failed to fetch remote changes:', error);
|
|
}
|
|
};
|
|
|
|
return (
|
|
<>
|
|
<div className={`flex items-center justify-between border-b border-border/60 ${isMobile ? 'px-3 py-2' : 'px-4 py-3'}`}>
|
|
<div className="relative" ref={dropdownRef}>
|
|
<button
|
|
onClick={() => setShowBranchDropdown((previous) => !previous)}
|
|
className={`flex items-center rounded-lg transition-colors hover:bg-accent ${isMobile ? 'space-x-1 px-2 py-1' : 'space-x-2 px-3 py-1.5'}`}
|
|
>
|
|
<GitBranch className={`text-muted-foreground ${isMobile ? 'h-3 w-3' : 'h-4 w-4'}`} />
|
|
<span className="flex items-center gap-1">
|
|
<span className={`font-medium ${isMobile ? 'text-xs' : 'text-sm'}`}>{currentBranch}</span>
|
|
{remoteStatus?.hasRemote && (
|
|
<span className="flex items-center gap-1 text-xs">
|
|
{aheadCount > 0 && (
|
|
<span
|
|
className="text-green-600 dark:text-green-400"
|
|
title={`${aheadCount} commit${aheadCount !== 1 ? 's' : ''} ahead`}
|
|
>
|
|
{'\u2191'}
|
|
{aheadCount}
|
|
</span>
|
|
)}
|
|
{behindCount > 0 && (
|
|
<span
|
|
className="text-primary"
|
|
title={`${behindCount} commit${behindCount !== 1 ? 's' : ''} behind`}
|
|
>
|
|
{'\u2193'}
|
|
{behindCount}
|
|
</span>
|
|
)}
|
|
{remoteStatus.isUpToDate && (
|
|
<span className="text-muted-foreground" title="Up to date with remote">
|
|
{'\u2713'}
|
|
</span>
|
|
)}
|
|
</span>
|
|
)}
|
|
</span>
|
|
<ChevronDown className={`h-3 w-3 text-muted-foreground transition-transform ${showBranchDropdown ? 'rotate-180' : ''}`} />
|
|
</button>
|
|
|
|
{showBranchDropdown && (
|
|
<div className="absolute left-0 top-full z-50 mt-1 w-64 overflow-hidden rounded-xl border border-border bg-card shadow-lg">
|
|
<div className="max-h-64 overflow-y-auto py-1">
|
|
{branches.map((branch) => (
|
|
<button
|
|
key={branch}
|
|
onClick={() => void handleSwitchBranch(branch)}
|
|
className={`w-full px-4 py-2 text-left text-sm transition-colors hover:bg-accent ${
|
|
branch === currentBranch ? 'bg-accent/50 text-foreground' : 'text-muted-foreground'
|
|
}`}
|
|
>
|
|
<span className="flex items-center space-x-2">
|
|
{branch === currentBranch && <Check className="h-3 w-3 text-primary" />}
|
|
<span className={branch === currentBranch ? 'font-medium' : ''}>{branch}</span>
|
|
</span>
|
|
</button>
|
|
))}
|
|
</div>
|
|
<div className="border-t border-border py-1">
|
|
<button
|
|
onClick={() => {
|
|
setShowNewBranchModal(true);
|
|
setShowBranchDropdown(false);
|
|
}}
|
|
className="flex w-full items-center space-x-2 px-4 py-2 text-left text-sm transition-colors hover:bg-accent"
|
|
>
|
|
<Plus className="h-3 w-3" />
|
|
<span>Create new branch</span>
|
|
</button>
|
|
</div>
|
|
</div>
|
|
)}
|
|
</div>
|
|
|
|
<div className={`flex items-center ${isMobile ? 'gap-1' : 'gap-2'}`}>
|
|
{remoteStatus?.hasRemote && (
|
|
<>
|
|
{!remoteStatus.hasUpstream && (
|
|
<button
|
|
onClick={requestPublishConfirmation}
|
|
disabled={isPublishing}
|
|
className="flex items-center gap-1 rounded-lg bg-purple-600 px-2.5 py-1 text-sm text-white transition-colors hover:bg-purple-700 disabled:opacity-50"
|
|
title={`Publish branch "${currentBranch}" to ${remoteName}`}
|
|
>
|
|
<Upload className={`h-3 w-3 ${isPublishing ? 'animate-pulse' : ''}`} />
|
|
<span>{isPublishing ? 'Publishing...' : 'Publish'}</span>
|
|
</button>
|
|
)}
|
|
|
|
{remoteStatus.hasUpstream && !remoteStatus.isUpToDate && (
|
|
<>
|
|
{behindCount > 0 && (
|
|
<button
|
|
onClick={requestPullConfirmation}
|
|
disabled={isPulling}
|
|
className="flex items-center gap-1 rounded-lg bg-green-600 px-2.5 py-1 text-sm text-white transition-colors hover:bg-green-700 disabled:opacity-50"
|
|
title={`Pull ${behindCount} commit${behindCount !== 1 ? 's' : ''} from ${remoteName}`}
|
|
>
|
|
<Download className={`h-3 w-3 ${isPulling ? 'animate-pulse' : ''}`} />
|
|
<span>{isPulling ? 'Pulling...' : `Pull ${behindCount}`}</span>
|
|
</button>
|
|
)}
|
|
|
|
{aheadCount > 0 && (
|
|
<button
|
|
onClick={requestPushConfirmation}
|
|
disabled={isPushing}
|
|
className="flex items-center gap-1 rounded-lg bg-orange-600 px-2.5 py-1 text-sm text-white transition-colors hover:bg-orange-700 disabled:opacity-50"
|
|
title={`Push ${aheadCount} commit${aheadCount !== 1 ? 's' : ''} to ${remoteName}`}
|
|
>
|
|
<Upload className={`h-3 w-3 ${isPushing ? 'animate-pulse' : ''}`} />
|
|
<span>{isPushing ? 'Pushing...' : `Push ${aheadCount}`}</span>
|
|
</button>
|
|
)}
|
|
|
|
{shouldShowFetchButton && (
|
|
<button
|
|
onClick={() => void handleFetch()}
|
|
disabled={isFetching}
|
|
className="flex items-center gap-1 rounded-lg bg-primary px-2.5 py-1 text-sm text-primary-foreground transition-colors hover:bg-primary/90 disabled:opacity-50"
|
|
title={`Fetch from ${remoteName}`}
|
|
>
|
|
<RefreshCw className={`h-3 w-3 ${isFetching ? 'animate-spin' : ''}`} />
|
|
<span>{isFetching ? 'Fetching...' : 'Fetch'}</span>
|
|
</button>
|
|
)}
|
|
</>
|
|
)}
|
|
</>
|
|
)}
|
|
|
|
<button
|
|
onClick={onRefresh}
|
|
disabled={isLoading}
|
|
className={`rounded-lg transition-colors hover:bg-accent ${isMobile ? 'p-1' : 'p-1.5'}`}
|
|
title="Refresh git status"
|
|
>
|
|
<RefreshCw className={`text-muted-foreground ${isLoading ? 'animate-spin' : ''} ${isMobile ? 'h-3 w-3' : 'h-4 w-4'}`} />
|
|
</button>
|
|
</div>
|
|
</div>
|
|
|
|
<NewBranchModal
|
|
isOpen={showNewBranchModal}
|
|
currentBranch={currentBranch}
|
|
isCreatingBranch={isCreatingBranch}
|
|
onClose={() => setShowNewBranchModal(false)}
|
|
onCreateBranch={onCreateBranch}
|
|
/>
|
|
</>
|
|
);
|
|
}
|