hammer/todo-app-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Compare commits

base: hammer:5a4d7e0ba9296f7da6712693720b8940a0d9e986
Branches Tags
hammer:main
..
compare: hammer:160d6014df2489de3f500f03bee76c2e1d7cc185
Branches Tags
hammer:main

1 Commits
5a4d7e0ba9 ... 160d6014df

Author SHA1 Message Date
hammer
160d6014df Add CI/CD workflow
Some checks failed
CI/CD / test (push) Failing after 32s
Details
CI/CD / deploy (push) Has been skipped
Details
2026-01-30 01:55:01 +00:00
10 changed files with 27 additions and 26 deletions
Expand all files Collapse all files

2
dist/index.html vendored
View File

@@ -6,7 +6,7 @@
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> <meta name="viewport" content="width=device-width, initial-scale=1.0" />
<meta name="description" content="Todo App - Task management made simple" /> <meta name="description" content="Todo App - Task management made simple" />
<title>Todo App</title> <title>Todo App</title>
<script type="module" crossorigin src="/assets/index-BJQCGGBb.js"></script> <script type="module" crossorigin src="/assets/index-BYeSixg2.js"></script>
<link rel="stylesheet" crossorigin href="/assets/index-C44jgAFE.css"> <link rel="stylesheet" crossorigin href="/assets/index-C44jgAFE.css">
</head> </head>
<body> <body>

2
src/components/AddTask.tsx
View File

@@ -1,5 +1,5 @@
import { useState, useRef, useEffect } from 'react'; import { useState, useRef, useEffect } from 'react';
import { Plus, Calendar, X } from 'lucide-react'; import { Plus, Calendar, Flag, Tag, X } from 'lucide-react';
import type { Priority } from '@/types'; import type { Priority } from '@/types';
import { cn, getPriorityColor } from '@/lib/utils'; import { cn, getPriorityColor } from '@/lib/utils';
import { useTasksStore } from '@/stores/tasks'; import { useTasksStore } from '@/stores/tasks';

1
src/components/CompletedSection.tsx
View File

@@ -1,6 +1,7 @@
import { useState } from 'react'; import { useState } from 'react';
import { ChevronRight, ChevronDown, Check } from 'lucide-react'; import { ChevronRight, ChevronDown, Check } from 'lucide-react';
import type { Task } from '@/types'; import type { Task } from '@/types';
import { cn } from '@/lib/utils';
import { useTasksStore } from '@/stores/tasks'; import { useTasksStore } from '@/stores/tasks';
interface CompletedSectionProps { interface CompletedSectionProps {

5
src/components/Sidebar.tsx
View File

@@ -2,7 +2,7 @@ import { useState, useRef, useEffect } from 'react';
import { Link, useLocation, useNavigate } from 'react-router-dom'; import { Link, useLocation, useNavigate } from 'react-router-dom';
import { import {
Inbox, Calendar, CalendarDays, Plus, ChevronDown, ChevronRight, Inbox, Calendar, CalendarDays, Plus, ChevronDown, ChevronRight,
Settings, LogOut, Tag, X, Hash, Settings, LogOut, User, FolderPlus, Tag, X, Check,
PanelLeftClose, PanelLeftOpen PanelLeftClose, PanelLeftOpen
} from 'lucide-react'; } from 'lucide-react';
import { cn } from '@/lib/utils'; import { cn } from '@/lib/utils';
@@ -56,7 +56,7 @@ export function Sidebar({ mobileOpen, onMobileClose }: SidebarProps) {
setIsCollapsed(next); setIsCollapsed(next);
try { try {
localStorage.setItem(SIDEBAR_COLLAPSED_KEY, String(next)); localStorage.setItem(SIDEBAR_COLLAPSED_KEY, String(next));
} catch { /* ignore localStorage errors */ } } catch {}
}; };
const handleNavClick = () => { const handleNavClick = () => {
@@ -94,6 +94,7 @@ export function Sidebar({ mobileOpen, onMobileClose }: SidebarProps) {
navigate('/login'); navigate('/login');
}; };
const inbox = projects.find(p => p.isInbox);
const regularProjects = projects.filter(p => !p.isInbox); const regularProjects = projects.filter(p => !p.isInbox);
const navItems = [ const navItems = [

6
src/components/TaskDetail.tsx
View File

@@ -4,7 +4,7 @@ import {
Tag, MessageSquare, ChevronRight, AlertCircle, Layers Tag, MessageSquare, ChevronRight, AlertCircle, Layers
} from 'lucide-react'; } from 'lucide-react';
import type { Task, Priority, Comment, Section } from '@/types'; import type { Task, Priority, Comment, Section } from '@/types';
import { cn, getPriorityColor } from '@/lib/utils'; import { cn, formatDate, getPriorityColor, getPriorityLabel } from '@/lib/utils';
import { useTasksStore } from '@/stores/tasks'; import { useTasksStore } from '@/stores/tasks';
import { api } from '@/lib/api'; import { api } from '@/lib/api';
@@ -29,9 +29,7 @@ export function TaskDetail({ task, onClose }: TaskDetailProps) {
const titleRef = useRef<HTMLInputElement>(null); const titleRef = useRef<HTMLInputElement>(null);
const panelRef = useRef<HTMLDivElement>(null); const panelRef = useRef<HTMLDivElement>(null);
// Sync local state from task prop - intentional setState in effect for controlled form
useEffect(() => { useEffect(() => {
// eslint-disable-next-line react-hooks/set-state-in-effect -- syncing local form state from prop
setTitle(task.title); setTitle(task.title);
setDescription(task.description || ''); setDescription(task.description || '');
setDueDate(task.dueDate ? task.dueDate.slice(0, 10) : ''); setDueDate(task.dueDate ? task.dueDate.slice(0, 10) : '');
@@ -52,7 +50,7 @@ export function TaskDetail({ task, onClose }: TaskDetailProps) {
if (projectId) { if (projectId) {
const proj = projects.find(p => p.id === projectId); const proj = projects.find(p => p.id === projectId);
if (proj?.sections) { if (proj?.sections) {
setAvailableSections(proj.sections); // eslint-disable-line react-hooks/set-state-in-effect setAvailableSections(proj.sections);
} else { } else {
api.getProject(projectId).then((p) => { api.getProject(projectId).then((p) => {
setAvailableSections(p.sections || []); setAvailableSections(p.sections || []);

28
src/lib/__tests__/api.test.ts
View File

@@ -1,23 +1,25 @@
import { describe, it, expect, vi, beforeEach } from 'vitest'; import { describe, it, expect, vi, beforeEach } from 'vitest';
import type { Mock } from 'vitest';
// We need to test the ApiClient class, so we import the module fresh // We need to test the ApiClient class, so we import the module fresh
// The api.ts uses import.meta.env.PROD which is false in test // The api.ts uses import.meta.env.PROD which is false in test
// So API_BASE = '/api' and AUTH_BASE = '' // So API_BASE = '/api' and AUTH_BASE = ''
let api: Record<string, (...args: unknown[]) => unknown>; let ApiClient: any;
let api: any;
beforeEach(async () => { beforeEach(async () => {
vi.resetModules(); vi.resetModules();
vi.stubGlobal('fetch', vi.fn()); vi.stubGlobal('fetch', vi.fn());
const mod = await import('@/lib/api'); const mod = await import('@/lib/api');
api = mod.api as unknown as Record<string, (...args: unknown[]) => unknown>; api = mod.api;
// Get the class from the singleton
ApiClient = (api as any).constructor;
}); });
function mockFetchResponse(data: unknown, ok = true, _status = 200) { function mockFetchResponse(data: any, ok = true, status = 200) {
(fetch as Mock).mockResolvedValueOnce({ (fetch as any).mockResolvedValueOnce({
ok, ok,
status: _status, status,
json: () => Promise.resolve(data), json: () => Promise.resolve(data),
}); });
} }
@@ -40,7 +42,7 @@ describe('ApiClient', () => {
}); });
it('throws on failed login', async () => { it('throws on failed login', async () => {
(fetch as Mock).mockResolvedValueOnce({ (fetch as any).mockResolvedValueOnce({
ok: false, ok: false,
json: () => Promise.resolve({ message: 'Invalid credentials' }), json: () => Promise.resolve({ message: 'Invalid credentials' }),
}); });
@@ -76,7 +78,7 @@ describe('ApiClient', () => {
}); });
it('returns null when not authenticated', async () => { it('returns null when not authenticated', async () => {
(fetch as Mock).mockResolvedValueOnce({ (fetch as any).mockResolvedValueOnce({
ok: false, ok: false,
json: () => Promise.resolve({}), json: () => Promise.resolve({}),
}); });
@@ -86,7 +88,7 @@ describe('ApiClient', () => {
}); });
it('returns null on network error', async () => { it('returns null on network error', async () => {
(fetch as Mock).mockRejectedValueOnce(new Error('Network error')); (fetch as any).mockRejectedValueOnce(new Error('Network error'));
const result = await api.getSession(); const result = await api.getSession();
expect(result).toBeNull(); expect(result).toBeNull();
@@ -163,7 +165,7 @@ describe('ApiClient', () => {
await api.getTasks({ projectId: 'proj1', completed: false, today: true }); await api.getTasks({ projectId: 'proj1', completed: false, today: true });
const calledUrl = (fetch as Mock).mock.calls[0][0] as string; const calledUrl = (fetch as any).mock.calls[0][0] as string;
expect(calledUrl).toContain('/api/tasks?'); expect(calledUrl).toContain('/api/tasks?');
expect(calledUrl).toContain('projectId=proj1'); expect(calledUrl).toContain('projectId=proj1');
expect(calledUrl).toContain('completed=false'); expect(calledUrl).toContain('completed=false');
@@ -257,14 +259,14 @@ describe('ApiClient', () => {
await api.getProjects(); await api.getProjects();
const headers = (fetch as Mock).mock.calls[0][1].headers; const headers = (fetch as any).mock.calls[0][1].headers;
expect(headers['Authorization']).toBeUndefined(); expect(headers['Authorization']).toBeUndefined();
}); });
}); });
describe('Error handling', () => { describe('Error handling', () => {
it('throws error with message from non-200 response', async () => { it('throws error with message from non-200 response', async () => {
(fetch as Mock).mockResolvedValueOnce({ (fetch as any).mockResolvedValueOnce({
ok: false, ok: false,
json: () => Promise.resolve({ error: 'Not found' }), json: () => Promise.resolve({ error: 'Not found' }),
}); });
@@ -273,7 +275,7 @@ describe('ApiClient', () => {
}); });
it('throws "Request failed" when error body is unparseable', async () => { it('throws "Request failed" when error body is unparseable', async () => {
(fetch as Mock).mockResolvedValueOnce({ (fetch as any).mockResolvedValueOnce({
ok: false, ok: false,
json: () => Promise.reject(new Error('parse error')), json: () => Promise.reject(new Error('parse error')),
}); });

3
src/pages/Board.tsx
View File

@@ -16,7 +16,7 @@ interface BoardViewProps {
} }
function TaskCard({ task, muted }: { task: Task; muted?: boolean }) { function TaskCard({ task, muted }: { task: Task; muted?: boolean }) {
const { setSelectedTask } = useTasksStore(); const { toggleComplete, setSelectedTask } = useTasksStore();
const overdue = !task.isCompleted && isOverdue(task.dueDate); const overdue = !task.isCompleted && isOverdue(task.dueDate);
return ( return (
@@ -84,7 +84,6 @@ function EditableHeader({
title, title,
count, count,
sectionId, sectionId,
// eslint-disable-next-line @typescript-eslint/no-unused-vars
projectId, projectId,
onRename, onRename,
onDelete, onDelete,

2
src/pages/Login.tsx
View File

@@ -1,5 +1,5 @@
import { useState } from 'react'; import { useState } from 'react';
import { useNavigate } from 'react-router-dom'; import { useNavigate, Link } from 'react-router-dom';
import { useAuthStore } from '@/stores/auth'; import { useAuthStore } from '@/stores/auth';
export function LoginPage() { export function LoginPage() {

2
src/pages/Project.tsx
View File

@@ -1,6 +1,6 @@
import { useEffect, useState, useRef } from 'react'; import { useEffect, useState, useRef } from 'react';
import { useParams, useNavigate, useLocation } from 'react-router-dom'; import { useParams, useNavigate, useLocation } from 'react-router-dom';
import { LayoutList, LayoutGrid, Pencil, Trash2, AlertCircle } from 'lucide-react'; import { LayoutList, LayoutGrid, Plus, Pencil, Trash2, AlertCircle } from 'lucide-react';
import { cn } from '@/lib/utils'; import { cn } from '@/lib/utils';
import { useTasksStore } from '@/stores/tasks'; import { useTasksStore } from '@/stores/tasks';
import { TaskItem } from '@/components/TaskItem'; import { TaskItem } from '@/components/TaskItem';

2
src/stores/auth.ts
View File

@@ -17,7 +17,7 @@ interface AuthState {
export const useAuthStore = create<AuthState>()( export const useAuthStore = create<AuthState>()(
persist( persist(
(set) => ({ (set, get) => ({
user: null, user: null,
isLoading: true, isLoading: true,
isAuthenticated: false, isAuthenticated: false,