feat: hierarchical notebooks (tree), remove all list view code, delete 22 unused files
All checks were successful
Deploy to Production / Build and Deploy (push) Successful in 2m3s

- Add parentId to Notebook model (tree structure)
- Update sidebar to render parent/child notebooks with expand/collapse
- Add sub-notebook creation from parent notebook
- Remove 'list' from NotesViewMode type everywhere
- Delete 22 unused components, hooks, and UI files
- Wrap revalidatePath in try-catch to prevent save 500
- Update notebook API to support parentId in creation
This commit is contained in:
Antigravity
2026-05-09 21:02:23 +00:00
parent 5a6ec4808f
commit d90b29b34f
30 changed files with 155 additions and 4280 deletions

View File

@@ -13,9 +13,7 @@ export default async function HomePage() {
? ('masonry' as const)
: settings?.notesViewMode === 'tabs'
? ('tabs' as const)
: settings?.notesViewMode === 'list'
? ('list' as const)
: ('masonry' as const)
: ('masonry' as const)
return (
<HomeClient

View File

@@ -14,7 +14,7 @@ export type UserAISettingsData = {
preferredLanguage?: 'auto' | 'en' | 'fr' | 'es' | 'de' | 'fa' | 'it' | 'pt' | 'ru' | 'zh' | 'ja' | 'ko' | 'ar' | 'hi' | 'nl' | 'pl'
demoMode?: boolean
showRecentNotes?: boolean
notesViewMode?: 'masonry' | 'tabs' | 'list'
notesViewMode?: 'masonry' | 'tabs'
emailNotifications?: boolean
desktopNotifications?: boolean
anonymousAnalytics?: boolean
@@ -64,8 +64,7 @@ function pickUserAISettingsForDb(input: UserAISettingsData): Partial<Record<User
if (
out.notesViewMode != null &&
out.notesViewMode !== 'masonry' &&
out.notesViewMode !== 'tabs' &&
out.notesViewMode !== 'list'
out.notesViewMode !== 'tabs'
) {
delete out.notesViewMode
}
@@ -170,9 +169,7 @@ const getCachedAISettings = unstable_cache(
? ('masonry' as const)
: raw === 'tabs'
? ('tabs' as const)
: raw === 'list'
? ('list' as const)
: ('masonry' as const)
: ('masonry' as const)
return {
titleSuggestions: settings.titleSuggestions,

View File

@@ -6,7 +6,23 @@ import { revalidatePath } from 'next/cache'
const DEFAULT_COLORS = ['#3B82F6', '#8B5CF6', '#EC4899', '#F59E0B', '#10B981', '#06B6D4']
const DEFAULT_ICONS = ['📁', '📚', '💼', '🎯', '📊', '🎨', '💡', '🔧']
// GET /api/notebooks - Get all notebooks for current user
function buildTree(notebooks: any[]): any[] {
const map = new Map<string, any>()
const roots: any[] = []
for (const nb of notebooks) {
map.set(nb.id, { ...nb, children: [] })
}
for (const nb of notebooks) {
const node = map.get(nb.id)!
if (nb.parentId && map.has(nb.parentId)) {
map.get(nb.parentId)!.children.push(node)
} else {
roots.push(node)
}
}
return roots
}
export async function GET(request: NextRequest) {
const session = await auth()
if (!session?.user?.id) {
@@ -17,9 +33,7 @@ export async function GET(request: NextRequest) {
const notebooks = await prisma.notebook.findMany({
where: { userId: session.user.id },
include: {
labels: {
orderBy: { name: 'asc' }
},
labels: { orderBy: { name: 'asc' } },
_count: {
select: { notes: { where: { isArchived: false, trashedAt: null } } }
}
@@ -27,23 +41,20 @@ export async function GET(request: NextRequest) {
orderBy: { order: 'asc' }
})
return NextResponse.json({
success: true,
notebooks: notebooks.map(nb => ({
...nb,
notesCount: nb._count.notes
}))
})
const flat = notebooks.map(nb => ({
...nb,
notesCount: nb._count.notes
}))
const tree = buildTree(flat)
return NextResponse.json({ success: true, notebooks: flat, tree })
} catch (error) {
console.error('Error fetching notebooks:', error)
return NextResponse.json(
{ success: false, error: 'Failed to fetch notebooks' },
{ status: 500 }
)
return NextResponse.json({ success: false, error: 'Failed to fetch notebooks' }, { status: 500 })
}
}
// POST /api/notebooks - Create a new notebook
export async function POST(request: NextRequest) {
const session = await auth()
if (!session?.user?.id) {
@@ -52,42 +63,49 @@ export async function POST(request: NextRequest) {
try {
const body = await request.json()
const { name, icon, color } = body
const { name, icon, color, parentId } = body
if (!name || typeof name !== 'string') {
return NextResponse.json(
{ success: false, error: 'Notebook name is required' },
{ status: 400 }
)
return NextResponse.json({ success: false, error: 'Notebook name is required' }, { status: 400 })
}
// Get the highest order value for this user
if (parentId) {
const parent = await prisma.notebook.findFirst({
where: { id: parentId, userId: session.user.id }
})
if (!parent) {
return NextResponse.json({ success: false, error: 'Parent notebook not found' }, { status: 400 })
}
}
const whereClause: any = { userId: session.user.id }
if (parentId) whereClause.parentId = parentId
else whereClause.parentId = null
const highestOrder = await prisma.notebook.findFirst({
where: { userId: session.user.id },
where: whereClause,
orderBy: { order: 'desc' },
select: { order: true }
})
const nextOrder = (highestOrder?.order ?? -1) + 1
// Create notebook
const notebook = await prisma.notebook.create({
data: {
name: name.trim(),
icon: icon || DEFAULT_ICONS[Math.floor(Math.random() * DEFAULT_ICONS.length)],
color: color || DEFAULT_COLORS[Math.floor(Math.random() * DEFAULT_COLORS.length)],
order: nextOrder,
parentId: parentId || null,
userId: session.user.id
},
include: {
labels: true,
_count: {
select: { notes: { where: { isArchived: false, trashedAt: null } } }
}
_count: { select: { notes: { where: { isArchived: false, trashedAt: null } } } }
}
})
revalidatePath('/')
try { revalidatePath('/') } catch {}
return NextResponse.json({
success: true,
@@ -96,9 +114,6 @@ export async function POST(request: NextRequest) {
}, { status: 201 })
} catch (error) {
console.error('Error creating notebook:', error)
return NextResponse.json(
{ success: false, error: 'Failed to create notebook' },
{ status: 500 }
)
return NextResponse.json({ success: false, error: 'Failed to create notebook' }, { status: 500 })
}
}