Compare commits

..
31 Commits
Author SHA1 Message Date
Linus Rath 705b942800 Merge dev into main - version 1.4.5 2026-03-20 18:10:08 +01:00
Linus Rath d68b81e6b8 chore: bump version to 1.4.5 2026-03-20 18:10:00 +01:00
Linus Rath 616e4d018d fix: expand recurring events in CalendarEvent/query (closes #65)
Add expandRecurrences: true to CalendarEvent/query when a date range
filter is provided, so the JMAP server returns individual occurrences
of recurring events instead of only the master event.
2026-03-20 17:53:30 +01:00
Linus Rath 68e141b787 feat: add mobile visibility toggle for sidebar apps and update related components 2026-03-20 17:44:26 +01:00
Linus Rath 1e6f5e2c8c fix: correct JSX syntax in CalendarToolbar component 2026-03-20 17:25:01 +01:00
Linus Rath 9495b34430 feat: add prev/next navigation buttons and date label to desktop calendar toolbar
Closes #59
2026-03-20 17:23:22 +01:00
Linus Rath 65fc489b9c feat: add pending event preview functionality to calendar views and event modal 2026-03-20 17:21:44 +01:00
Linus Rath bd686c092c fix: validate event start field when fetching calendar events 2026-03-20 17:11:36 +01:00
Linus Rath 6cff98ddb8 feat: implement pagination for fetching contacts and add maxObjectsInGet capability 2026-03-20 16:47:27 +01:00
Linus Rath dcc35335f5 feat: add setting to show event start time in month view 2026-03-20 16:42:07 +01:00
Linus Rath 8a54ae2456 feat: add NotFound component to handle 404 errors and redirect unauthenticated users 2026-03-20 16:31:57 +01:00
Linus Rath e26654a005 fix: enhance account switching logic and clear stores on account change 2026-03-20 16:29:32 +01:00
Linus Rath c1c06c68bb fix: improve draft handling in email composer and enhance session cookie verification logic 2026-03-20 15:45:25 +01:00
Linus Rath 74cf642182 Merge branch 'dev' 2026-03-20 00:16:41 +01:00
Linus Rath c5b1731a63 feat: add attachment position setting in email settings
- Introduced a new setting for attachment position in email settings, allowing users to choose between displaying attachments beside the sender or below the header.
- Updated the settings store to include the new attachment position type and default value.
- Added translations for the new setting in multiple languages (de, en, es, fr, it, ja, nl, pt).
2026-03-19 21:14:18 +01:00
Linus Rath 40cf164df3 fix: update calendar agenda view to auto-scroll to today's events and ensure today's date is included in the groups 2026-03-19 19:25:34 +01:00
Linus Rath ff56245db8 fix: improve account restoration logic and handle stale accounts in auth store 2026-03-19 19:21:08 +01:00
Linus Rath 9b4de4d152 fix: remove claude directory from .dockerignore and .gitignore 2026-03-19 19:06:31 +01:00
Linus Rath 0c9e60db8b fix: update flatted to 3.4.2 2026-03-19 19:01:40 +01:00
Linus Rath def8ee89fa Merge branch 'dev' 2026-03-19 18:57:57 +01:00
Linus Rath e7e07a38d7 fix: use native ARM runners instead of QEMU for Docker builds 2026-03-19 18:57:17 +01:00
Linus Rath a009e5ae32 fix: enhance health check functionality with detailed memory diagnostics and stable liveness probe 2026-03-19 18:50:59 +01:00
Linus Rath b141240fa3 Merge dev into main - version 1.4.4 2026-03-19 18:18:23 +01:00
Linus Rath 44896dee3e chore: bump version to 1.4.4 2026-03-19 18:12:16 +01:00
Linus Rath a5c5fa6669 fix: improve mailbox role management by ensuring roles are cleared from all mailboxes when reassigning 2026-03-19 18:00:09 +01:00
Linus Rath 95af61c4be fix: enhance account management by updating existing accounts and improving session handling 2026-03-19 17:47:18 +01:00
Linus Rath 34e495dde3 Fix email signature rendering 2026-03-19 17:20:39 +01:00
Linus Rath 0b721661e9 Fix logout redirects and unauthenticated home rendering 2026-03-19 17:01:27 +01:00
Linus Rath 9fa851a674 feat: implement CalDAV discovery API and enhance calendar ID handling 2026-03-19 16:56:00 +01:00
Linus Rath 41f91244d9 Fix duplicate calendar edits and prevent double-save submissions 2026-03-19 14:40:16 +01:00
Linus Rath 77514bd054 fix: RFC 9553 compliance for contacts (birthday, addresses) 2026-03-19 13:33:53 +01:00
61 changed files with 2346 additions and 561 deletions
-2
View File
@@ -5,9 +5,7 @@ node_modules
.env* .env*
!.env.example !.env.example
!.env.dev.example !.env.dev.example
.claude/
scripts/ scripts/
TODO.md TODO.md
CLAUDE.md
*.md *.md
!README.md !README.md
+79 -17
View File
@@ -20,9 +20,20 @@ on:
tags: ["v*.*.*"] tags: ["v*.*.*"]
workflow_dispatch: workflow_dispatch:
env:
IMAGE_NAME: ghcr.io/${{ github.repository }}
jobs: jobs:
build-and-push: build:
runs-on: ubuntu-latest strategy:
fail-fast: false
matrix:
include:
- platform: linux/amd64
runner: ubuntu-latest
- platform: linux/arm64
runner: ubuntu-24.04-arm
runs-on: ${{ matrix.runner }}
permissions: permissions:
contents: read contents: read
packages: write packages: write
@@ -31,9 +42,6 @@ jobs:
- name: Checkout - name: Checkout
uses: actions/checkout@v4 uses: actions/checkout@v4
- name: Set up QEMU
uses: docker/setup-qemu-action@v3
- name: Set up Docker Buildx - name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3 uses: docker/setup-buildx-action@v3
@@ -48,21 +56,75 @@ jobs:
id: meta id: meta
uses: docker/metadata-action@v5 uses: docker/metadata-action@v5
with: with:
images: | images: ${{ env.IMAGE_NAME }}
ghcr.io/${{ github.repository }}
- name: Build and push by digest
id: build
uses: docker/build-push-action@v6
with:
context: .
platforms: ${{ matrix.platform }}
labels: ${{ steps.meta.outputs.labels }}
outputs: type=image,name=${{ env.IMAGE_NAME }},push-by-digest=true,name-canonical=true,push=true
cache-from: type=gha,scope=${{ matrix.platform }}
cache-to: type=gha,mode=max,scope=${{ matrix.platform }}
- name: Export digest
run: |
mkdir -p /tmp/digests
digest="${{ steps.build.outputs.digest }}"
touch "/tmp/digests/${digest#sha256:}"
- name: Upload digest
uses: actions/upload-artifact@v4
with:
name: digests-${{ matrix.platform == 'linux/amd64' && 'amd64' || 'arm64' }}
path: /tmp/digests/*
if-no-files-found: error
retention-days: 1
merge:
runs-on: ubuntu-latest
needs: build
permissions:
contents: read
packages: write
steps:
- name: Download digests
uses: actions/download-artifact@v4
with:
path: /tmp/digests
pattern: digests-*
merge-multiple: true
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
- name: Log in to GHCR
uses: docker/login-action@v3
with:
registry: ghcr.io
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}
- name: Extract metadata
id: meta
uses: docker/metadata-action@v5
with:
images: ${{ env.IMAGE_NAME }}
tags: | tags: |
type=raw,value=latest,enable={{is_default_branch}} type=raw,value=latest,enable={{is_default_branch}}
type=semver,pattern={{version}} type=semver,pattern={{version}}
type=semver,pattern={{major}}.{{minor}} type=semver,pattern={{major}}.{{minor}}
type=sha,prefix= type=sha,prefix=
- name: Build and push - name: Create manifest list and push
uses: docker/build-push-action@v6 working-directory: /tmp/digests
with: run: |
context: . docker buildx imagetools create $(jq -cr '.tags | map("-t " + .) | join(" ")' <<< "$DOCKER_METADATA_OUTPUT_JSON") \
platforms: linux/amd64,linux/arm64 $(printf '${{ env.IMAGE_NAME }}@sha256:%s ' *)
push: true
tags: ${{ steps.meta.outputs.tags }} - name: Inspect image
labels: ${{ steps.meta.outputs.labels }} run: |
cache-from: type=gha docker buildx imagetools inspect ${{ env.IMAGE_NAME }}:${{ steps.meta.outputs.version }}
cache-to: type=gha,mode=max
-3
View File
@@ -42,9 +42,6 @@ yarn-error.log*
*.tsbuildinfo *.tsbuildinfo
next-env.d.ts next-env.d.ts
# claude code
.claude/
# settings sync data # settings sync data
/data/ /data/
+43
View File
@@ -1,5 +1,48 @@
# Changelog # Changelog
## 1.4.5 (2026-03-20)
### Features
- **Calendar**: Add prev/next navigation buttons and date label to desktop calendar toolbar
- **Calendar**: Add pending event preview functionality to calendar views and event modal
- **Calendar**: Add setting to show event start time in month view
- **Contacts**: Implement pagination for fetching contacts with maxObjectsInGet capability
- **Email**: Add attachment position setting in email settings
- **Layout**: Add mobile visibility toggle for sidebar apps
- **Error**: Add NotFound component to handle 404 errors and redirect unauthenticated users
### Fixes
- **Auth**: Enhance account switching logic and clear stores on account change
- **Auth**: Improve account restoration logic and handle stale accounts
- **Auth**: Improve draft handling in email composer and enhance session cookie verification
- **Calendar**: Expand recurring events in CalendarEvent/query so individual occurrences are returned (#65)
- **Calendar**: Validate event start field when fetching calendar events
- **Calendar**: Auto-scroll agenda view to today's events and include today's date in groups
- **Calendar**: Correct JSX syntax in CalendarToolbar component
- **Dependencies**: Update flatted to 3.4.2
- **DevOps**: Use native ARM runners instead of QEMU for Docker builds
- **DevOps**: Enhance health check with detailed memory diagnostics and stable liveness probe
## 1.4.4 (2026-03-19)
### Features
- **Calendar**: Implement CalDAV discovery API with automatic calendar home resolution for multi-account setups
- **Calendar**: Enhance calendar management settings with mailbox role reassignment controls
- **Email**: Add signature rendering utilities with HTML-to-text conversion and sanitization
### Fixes
- **Auth**: Fix account session handling to update existing accounts instead of duplicating entries
- **Auth**: Fix logout redirects and unauthenticated home page rendering
- **Calendar**: Fix duplicate calendar edits and prevent double-save submissions in event modal
- **Calendar**: Remove stale calendar ID references in favor of CalDAV-discovered IDs
- **Contacts**: Improve RFC 9553 compliance for contact birthdays and address formatting
- **Email**: Fix email signature rendering for identity signatures
- **Folders**: Improve mailbox role management by clearing roles from all mailboxes before reassigning
## 1.4.3 (2026-03-19) ## 1.4.3 (2026-03-19)
### Features ### Features
+46 -46
View File
@@ -3,7 +3,7 @@
<picture> <picture>
<source media="(prefers-color-scheme: dark)" srcset="public/branding/Bulwark_Logo_with_Lettering_White_and_Color.svg" /> <source media="(prefers-color-scheme: dark)" srcset="public/branding/Bulwark_Logo_with_Lettering_White_and_Color.svg" />
<source media="(prefers-color-scheme: light)" srcset="public/branding/Bulwark_Logo_with_Lettering_Dark_Color.svg" /> <source media="(prefers-color-scheme: light)" srcset="public/branding/Bulwark_Logo_with_Lettering_Dark_Color.svg" />
<img src="public/branding/Bulwark_Logo_with_Lettering_Dark_Color.svg" alt="Bulwark Webmail" width="480" /> <img src="public/branding/Bulwark_Logo_with_Lettering_Dark_Color.svg" alt="Bulwark Webmail" width="280" />
</picture> </picture>
# Bulwark Webmail # Bulwark Webmail
@@ -25,16 +25,16 @@ Built with Next.js and the JMAP protocol.
<tr> <tr>
<td width="50%"> <td width="50%">
<img src="screenshots/inbox.png" width="100%" alt="Inbox - three-pane layout with sidebar, email list, and viewer (dark mode)"> <img src="screenshots/inbox.png" width="100%" alt="Inbox three-pane layout with sidebar, email list, and viewer (dark mode)">
**Mail** - Three-pane layout with sidebar, email list, and viewer **Mail** Three-pane layout with sidebar, email list, and viewer
</td> </td>
<td width="50%"> <td width="50%">
<img src="screenshots/calendar.png" width="100%" alt="Calendar"> <img src="screenshots/calendar.png" width="100%" alt="Calendar">
**Calendar** - Month, week, day, and agenda views with event management **Calendar** Month, week, day, and agenda views with event management
</td> </td>
</tr> </tr>
@@ -43,14 +43,14 @@ Built with Next.js and the JMAP protocol.
<img src="screenshots/contacts.png" width="100%" alt="Contacts"> <img src="screenshots/contacts.png" width="100%" alt="Contacts">
**Contacts** - Contact management with groups and vCard support **Contacts** Contact management with groups and vCard support
</td> </td>
<td width="50%"> <td width="50%">
<img src="screenshots/files.png" width="100%" alt="File browser"> <img src="screenshots/files.png" width="100%" alt="File browser">
**Files** - Cloud file browser with upload, preview, and folder navigation **Files** Cloud file browser with upload, preview, and folder navigation
</td> </td>
</tr> </tr>
@@ -62,16 +62,16 @@ Built with Next.js and the JMAP protocol.
<tr> <tr>
<td width="50%"> <td width="50%">
<img src="screenshots/inbox%20whitemode.png" width="100%" alt="Inbox - light mode"> <img src="screenshots/inbox%20whitemode.png" width="100%" alt="Inbox light mode">
**Light mode** - Full theme support with intelligent color transformation **Light mode** Full theme support with intelligent color transformation
</td> </td>
<td width="50%"> <td width="50%">
<img src="screenshots/settings.png" width="100%" alt="Settings"> <img src="screenshots/settings.png" width="100%" alt="Settings">
**Settings** - Appearance, identities, filters, templates, and more **Settings** Appearance, identities, filters, templates, and more
</td> </td>
</tr> </tr>
@@ -80,7 +80,7 @@ Built with Next.js and the JMAP protocol.
<img src="screenshots/login.png" width="100%" alt="Login page"> <img src="screenshots/login.png" width="100%" alt="Login page">
**Login** - Configurable branding with OAuth2/OIDC and 2FA support **Login** Configurable branding with OAuth2/OIDC and 2FA support
</td> </td>
<td width="50%"> <td width="50%">
@@ -94,21 +94,21 @@ Built with Next.js and the JMAP protocol.
### Mail ### Mail
- **Read, compose, reply, reply-all, forward** with rich HTML rendering - **Read, compose, reply, reply-all, forward** with rich HTML rendering
- **Threading** - Gmail-style inline expansion with thread navigation - **Threading** Gmail-style inline expansion with thread navigation
- **Draft auto-save** with discard confirmation - **Draft auto-save** with discard confirmation
- **Attachments** - upload, download, and inline preview - **Attachments** upload, download, and inline preview
- **Search** - full-text with JMAP filter panel, search chips, cross-mailbox queries, wildcard support, and OR conditions - **Search** full-text with JMAP filter panel, search chips, cross-mailbox queries, wildcard support, and OR conditions
- **Batch operations** - multi-select with checkboxes, archive, delete, move, tag - **Batch operations** multi-select with checkboxes, archive, delete, move, tag
- **Archive modes** - archive directly or organize archived mail by year or month - **Archive modes** archive directly or organize archived mail by year or month
- **Print** emails directly from the viewer - **Print** emails directly from the viewer
- **Color tags/labels** and star/unstar - **Color tags/labels** and star/unstar
- **Virtual scrolling** for large mailboxes - **Virtual scrolling** for large mailboxes
- **Quick reply** from the viewer - **Quick reply** from the viewer
- **Sender avatars** - favicon-based with negative caching for performance - **Sender avatars** favicon-based with negative caching for performance
- **Recipient popover** for quick contact interaction - **Recipient popover** for quick contact interaction
- **Folder management** - create, rename, delete folders with icon picker and subfolder support - **TNEF support** — extract Outlook `winmail.dat` message bodies and attachments automatically
- **Tag counts** - unread and total counts displayed in sidebar - **Folder management** — create, rename, delete folders with icon picker and subfolder support
- **TNEF support** - extract Outlook `winmail.dat` message bodies and attachments automatically - **Tag counts** — unread and total counts displayed in sidebar
### Calendar ### Calendar
@@ -116,8 +116,8 @@ Built with Next.js and the JMAP protocol.
- **Event hover preview** popover with details - **Event hover preview** popover with details
- **Drag-and-drop rescheduling**, click-drag creation, edge-resize (15-min snap) - **Drag-and-drop rescheduling**, click-drag creation, edge-resize (15-min snap)
- **Recurring events** with edit/delete scope (this / this and following / all) - **Recurring events** with edit/delete scope (this / this and following / all)
- **Participant scheduling** - iTIP invitations, organizer/attendee UI, RSVP - **Participant scheduling** iTIP invitations, organizer/attendee UI, RSVP
- **Inline calendar invitations** in email viewer - auto-detect `.ics`, RSVP, import - **Inline calendar invitations** in email viewer auto-detect `.ics`, RSVP, import
- **iCalendar import** with preview and bulk create - **iCalendar import** with preview and bulk create
- **Notifications** with configurable sound and alert persistence - **Notifications** with configurable sound and alert persistence
- **Real-time sync** via JMAP push - **Real-time sync** via JMAP push
@@ -128,15 +128,15 @@ Built with Next.js and the JMAP protocol.
- **Contact groups** with group expansion and member management - **Contact groups** with group expansion and member management
- **vCard import/export** (RFC 6350) with duplicate detection - **vCard import/export** (RFC 6350) with duplicate detection
- **Autocomplete** in composer (To/Cc/Bcc) - **Autocomplete** in composer (To/Cc/Bcc)
- **Bulk operations** - multi-select, delete, group add, export - **Bulk operations** multi-select, delete, group add, export
### Filters & Automation ### Filters & Automation
- **Server-side email filters** via JMAP Sieve Scripts (RFC 9661) - **Server-side email filters** via JMAP Sieve Scripts (RFC 9661)
- **Visual rule builder** - conditions (From, To, Subject, Size, Body…) and actions (Move, Forward, Star, Discard…) - **Visual rule builder** conditions (From, To, Subject, Size, Body…) and actions (Move, Forward, Star, Discard…)
- **Raw Sieve editor** with syntax validation - **Raw Sieve editor** with syntax validation
- **Vacation responder** with date range scheduling and sidebar indicator - **Vacation responder** with date range scheduling and sidebar indicator
- **Email templates** - reusable, categorized, with placeholder auto-fill (`{{recipientName}}`, `{{date}}`, etc.) - **Email templates** reusable, categorized, with placeholder auto-fill (`{{recipientName}}`, `{{date}}`, etc.)
### Files ### Files
@@ -144,39 +144,39 @@ Built with Next.js and the JMAP protocol.
- **Upload and download** files with progress tracking and folder upload support - **Upload and download** files with progress tracking and folder upload support
- **Folder navigation** with breadcrumb path and tree sidebar - **Folder navigation** with breadcrumb path and tree sidebar
- **Grid and list views** with sorting by name, size, or date - **Grid and list views** with sorting by name, size, or date
- **Clipboard operations** - cut, copy, paste, duplicate files - **Clipboard operations** cut, copy, paste, duplicate files
- **File preview** for images, text, audio, video, and more - **File preview** for images, text, audio, video, and more
- **Favorites and recent files** for quick access - **Favorites and recent files** for quick access
- **Bulk operations** - multi-select, delete, move, download - **Bulk operations** multi-select, delete, move, download
### Security & Privacy ### Security & Privacy
- **External content blocked** by default - trusted senders list for auto-load - **External content blocked** by default trusted senders list for auto-load
- **HTML sanitization** via DOMPurify with XSS prevention - **HTML sanitization** via DOMPurify with XSS prevention
- **S/MIME** - manage certificates, sign outgoing mail, encrypt to recipients, decrypt messages, and verify signatures - **S/MIME** manage certificates, sign outgoing mail, encrypt to recipients, decrypt messages, and verify signatures
- **SPF/DKIM/DMARC** status indicators - **SPF/DKIM/DMARC** status indicators
- **OAuth2/OIDC with PKCE** for SSO (Keycloak, Authentik, or built-in), with OAuth-only mode - **OAuth2/OIDC with PKCE** for SSO (Keycloak, Authentik, or built-in), with OAuth-only mode
- **TOTP two-factor authentication** - **TOTP two-factor authentication**
- **Account security panel** - manage passwords and 2FA via Stalwart admin API - **Account security panel** manage passwords and 2FA via Stalwart admin API
- **"Remember me"** - AES-256-GCM encrypted httpOnly cookie (opt-in) - **"Remember me"** AES-256-GCM encrypted httpOnly cookie (opt-in)
- **Security headers** - CSP with per-request nonce, X-Frame-Options, Referrer-Policy - **Security headers** CSP with per-request nonce, X-Frame-Options, Referrer-Policy
- **Newsletter unsubscribe** (RFC 2369) - **Newsletter unsubscribe** (RFC 2369)
### Interface ### Interface
- **Three-pane layout** - sidebar, email list, viewer with resizable columns - **Three-pane layout** sidebar, email list, viewer with resizable columns
- **Dark and light themes** with intelligent email color transformation - **Dark and light themes** with intelligent email color transformation
- **Responsive** - desktop sidebar + mobile bottom tab bar with tablet support - **Always-light email rendering** option for problematic HTML messages in dark theme
- **Keyboard shortcuts** - full navigation without a mouse - **Responsive** — desktop sidebar + mobile bottom tab bar with tablet support
- **Keyboard shortcuts** — full navigation without a mouse
- **Drag-and-drop** email organization between mailboxes and tag assignment - **Drag-and-drop** email organization between mailboxes and tag assignment
- **Right-click context menus**, toast notifications with undo, form validation with shake feedback - **Right-click context menus**, toast notifications with undo, form validation with shake feedback
- **Always-light email rendering** option for problematic HTML messages in dark theme
- **Customizable toolbar** position, custom favicon, sidebar/login logos, and login page branding - **Customizable toolbar** position, custom favicon, sidebar/login logos, and login page branding
- **Sidebar apps** - pin custom tools to the navigation rail and open them inline or in a new tab - **Sidebar apps** pin custom tools to the navigation rail and open them inline or in a new tab
- **Settings sync** - preferences synchronized with the server (encrypted) - **Settings sync** preferences synchronized with the server (encrypted)
- **Storage quota** display - **Storage quota** display
- **Shared folders** - multi-account access - **Shared folders** multi-account access
- **Accessibility** - WCAG AA contrast, reduced-motion support, focus trap, screen reader live regions - **Accessibility** WCAG AA contrast, reduced-motion support, focus trap, screen reader live regions
### Internationalization ### Internationalization
@@ -187,13 +187,13 @@ Automatic browser detection with persistent preference.
### Identity Management ### Identity Management
- **Multiple sender identities** with per-identity signatures - **Multiple sender identities** with per-identity signatures
- **Sub-addressing** - `user+tag@domain.com` with contextual tag suggestions - **Identity refresh** — keep the identity manager aligned with server-side changes after edits
- **Identity refresh** - keep the identity manager aligned with server-side changes after edits - **Sub-addressing** — `user+tag@domain.com` with contextual tag suggestions
- **Identity badges** in viewer and email list - **Identity badges** in viewer and email list
### Operations ### Operations
- **Automatic update check** - server logs when a newer release is available - **Automatic update check** server logs when a newer release is available
--- ---
@@ -211,7 +211,7 @@ Or with Docker Compose:
```bash ```bash
cp .env.example .env.local cp .env.example .env.local
# Edit .env.local - set JMAP_SERVER_URL # Edit .env.local set JMAP_SERVER_URL
docker compose up -d docker compose up -d
``` ```
@@ -222,7 +222,7 @@ git clone https://github.com/bulwarkmail/webmail.git
cd webmail cd webmail
npm install npm install
cp .env.example .env.local cp .env.example .env.local
# Edit .env.local - set JMAP_SERVER_URL # Edit .env.local set JMAP_SERVER_URL
npm run build && npm start npm run build && npm start
``` ```
@@ -246,7 +246,7 @@ JMAP_SERVER_URL=https://mail.example.com
APP_NAME=My Webmail APP_NAME=My Webmail
``` ```
All variables are **runtime** - Docker deployments can be configured without rebuilding. All variables are **runtime** Docker deployments can be configured without rebuilding.
<details> <details>
<summary>Server Listen Address</summary> <summary>Server Listen Address</summary>
@@ -313,7 +313,7 @@ Credentials encrypted with AES-256-GCM, stored in an httpOnly cookie (30-day exp
## Why Stalwart? ## Why Stalwart?
[Stalwart](https://github.com/stalwartlabs/mail-server) is a mail server written in Rust with **native JMAP support** - not IMAP/SMTP with JMAP bolted on. It handles JMAP, IMAP, SMTP, and ManageSieve in a single binary. Self-hosted, no third-party dependencies. [Stalwart](https://github.com/stalwartlabs/mail-server) is a mail server written in Rust with **native JMAP support** not IMAP/SMTP with JMAP bolted on. It handles JMAP, IMAP, SMTP, and ManageSieve in a single binary. Self-hosted, no third-party dependencies.
## Contributing ## Contributing
+1 -1
View File
@@ -1 +1 @@
1.4.3 1.4.5
+29 -17
View File
@@ -7,7 +7,7 @@ import { Plus } from "lucide-react";
import { import {
startOfMonth, endOfMonth, startOfWeek, endOfWeek, startOfMonth, endOfMonth, startOfWeek, endOfWeek,
addMonths, subMonths, addWeeks, subWeeks, addDays, subDays, addMonths, subMonths, addWeeks, subWeeks, addDays, subDays,
format, parseISO, startOfDay, format, parseISO,
} from "date-fns"; } from "date-fns";
import { useCalendarStore } from "@/stores/calendar-store"; import { useCalendarStore } from "@/stores/calendar-store";
import { isCalendarViewMode } from "@/stores/calendar-store"; import { isCalendarViewMode } from "@/stores/calendar-store";
@@ -25,7 +25,7 @@ import { CalendarDayView } from "@/components/calendar/calendar-day-view";
import { CalendarAgendaView } from "@/components/calendar/calendar-agenda-view"; import { CalendarAgendaView } from "@/components/calendar/calendar-agenda-view";
import { MiniCalendar } from "@/components/calendar/mini-calendar"; import { MiniCalendar } from "@/components/calendar/mini-calendar";
import { CalendarSidebarPanel } from "@/components/calendar/calendar-sidebar-panel"; import { CalendarSidebarPanel } from "@/components/calendar/calendar-sidebar-panel";
import { EventModal } from "@/components/calendar/event-modal"; import { EventModal, type PendingEventPreview } from "@/components/calendar/event-modal";
import { EventDetailPopover } from "@/components/calendar/event-detail-popover"; import { EventDetailPopover } from "@/components/calendar/event-detail-popover";
import { ICalImportModal } from "@/components/calendar/ical-import-modal"; import { ICalImportModal } from "@/components/calendar/ical-import-modal";
import { ICalSubscriptionModal } from "@/components/calendar/ical-subscription-modal"; import { ICalSubscriptionModal } from "@/components/calendar/ical-subscription-modal";
@@ -82,6 +82,7 @@ export default function CalendarPage() {
const [pendingScopeAction, setPendingScopeAction] = useState<PendingScopeAction | null>(null); const [pendingScopeAction, setPendingScopeAction] = useState<PendingScopeAction | null>(null);
const [detailEvent, setDetailEvent] = useState<CalendarEvent | null>(null); const [detailEvent, setDetailEvent] = useState<CalendarEvent | null>(null);
const [detailAnchorRect, setDetailAnchorRect] = useState<DOMRect | null>(null); const [detailAnchorRect, setDetailAnchorRect] = useState<DOMRect | null>(null);
const [pendingPreview, setPendingPreview] = useState<PendingEventPreview | null>(null);
const hasFetched = useRef(false); const hasFetched = useRef(false);
// Sidebar resize state // Sidebar resize state
@@ -156,11 +157,15 @@ export default function CalendarPage() {
start: format(d, "yyyy-MM-dd'T'00:00:00"), start: format(d, "yyyy-MM-dd'T'00:00:00"),
end: format(d, "yyyy-MM-dd'T'23:59:59"), end: format(d, "yyyy-MM-dd'T'23:59:59"),
}; };
case "agenda": case "agenda": {
// Agenda always starts from today at the earliest
const today = startOfDay(new Date());
const agendaStart = d >= today ? d : today;
return { return {
start: format(d, "yyyy-MM-dd'T'00:00:00"), start: format(agendaStart, "yyyy-MM-dd'T'00:00:00"),
end: format(addDays(d, 30), "yyyy-MM-dd'T'23:59:59"), end: format(addDays(agendaStart, 30), "yyyy-MM-dd'T'23:59:59"),
}; };
}
} }
}, [selectedDate, normalizedViewMode, firstDayOfWeek]); }, [selectedDate, normalizedViewMode, firstDayOfWeek]);
@@ -264,12 +269,13 @@ export default function CalendarPage() {
}, [closeDetail, openEditModal]); }, [closeDetail, openEditModal]);
const handleHoverEvent = useCallback((event: CalendarEvent, anchorRect: DOMRect) => { const handleHoverEvent = useCallback((event: CalendarEvent, anchorRect: DOMRect) => {
if (isMobile) return;
if (hoverTimerRef.current) { clearTimeout(hoverTimerRef.current); hoverTimerRef.current = null; } if (hoverTimerRef.current) { clearTimeout(hoverTimerRef.current); hoverTimerRef.current = null; }
// Don't show hover popover if the sidebar is already open for this event // Don't show hover popover if the sidebar is already open for this event
if (showEventModal && editEvent?.id === event.id) return; if (showEventModal && editEvent?.id === event.id) return;
setDetailEvent(event); setDetailEvent(event);
setDetailAnchorRect(anchorRect); setDetailAnchorRect(anchorRect);
}, [showEventModal, editEvent]); }, [isMobile, showEventModal, editEvent]);
const handleHoverLeave = useCallback(() => { const handleHoverLeave = useCallback(() => {
hoverTimerRef.current = setTimeout(() => { hoverTimerRef.current = setTimeout(() => {
@@ -611,7 +617,7 @@ export default function CalendarPage() {
const visibleEvents = useMemo(() => const visibleEvents = useMemo(() =>
events.filter((e) => { events.filter((e) => {
if (!e.calendarIds) return false; if (!e.start || !e.calendarIds) return false;
const calIds = Object.keys(e.calendarIds); const calIds = Object.keys(e.calendarIds);
return calIds.some((id) => selectedCalendarIds.includes(id)); return calIds.some((id) => selectedCalendarIds.includes(id));
}), }),
@@ -644,6 +650,7 @@ export default function CalendarPage() {
onCreateAtTime={openCreateModal} onCreateAtTime={openCreateModal}
firstDayOfWeek={firstDayOfWeek} firstDayOfWeek={firstDayOfWeek}
isMobile={isMobile} isMobile={isMobile}
pendingPreview={pendingPreview}
/> />
); );
case "week": case "week":
@@ -660,6 +667,7 @@ export default function CalendarPage() {
firstDayOfWeek={firstDayOfWeek} firstDayOfWeek={firstDayOfWeek}
timeFormat={timeFormat} timeFormat={timeFormat}
isMobile={isMobile} isMobile={isMobile}
pendingPreview={pendingPreview}
/> />
); );
case "day": case "day":
@@ -674,6 +682,7 @@ export default function CalendarPage() {
onCreateAtTime={openCreateModal} onCreateAtTime={openCreateModal}
timeFormat={timeFormat} timeFormat={timeFormat}
isMobile={isMobile} isMobile={isMobile}
pendingPreview={pendingPreview}
/> />
); );
case "agenda": case "agenda":
@@ -704,7 +713,7 @@ export default function CalendarPage() {
}; };
return ( return (
<div className="flex h-dvh bg-background overflow-hidden"> <div className={cn("flex h-dvh bg-background overflow-hidden", isMobile && "flex-col")}>
{/* Left Navigation Rail */} {/* Left Navigation Rail */}
{!isMobile && ( {!isMobile && (
<div className="w-14 bg-secondary flex flex-col flex-shrink-0" style={{ borderRight: '1px solid rgba(128, 128, 128, 0.3)' }}> <div className="w-14 bg-secondary flex flex-col flex-shrink-0" style={{ borderRight: '1px solid rgba(128, 128, 128, 0.3)' }}>
@@ -767,7 +776,7 @@ export default function CalendarPage() {
)} )}
{!inlineApp && ( {!inlineApp && (
<div className="flex flex-col flex-1 min-w-0"> <div className="flex flex-col flex-1 min-w-0 min-h-0">
<CalendarToolbar <CalendarToolbar
selectedDate={selectedDate} selectedDate={selectedDate}
viewMode={normalizedViewMode} viewMode={normalizedViewMode}
@@ -804,7 +813,8 @@ export default function CalendarPage() {
onDelete={handleDeleteEvent} onDelete={handleDeleteEvent}
onDuplicate={handleDuplicateEvent} onDuplicate={handleDuplicateEvent}
onRsvp={handleRsvp} onRsvp={handleRsvp}
onClose={() => { setShowEventModal(false); setEditEvent(null); }} onClose={() => { setShowEventModal(false); setEditEvent(null); setPendingPreview(null); }}
onPreviewChange={setPendingPreview}
currentUserEmails={currentUserEmails} currentUserEmails={currentUserEmails}
isMobile={false} isMobile={false}
/> />
@@ -827,13 +837,15 @@ export default function CalendarPage() {
{/* Mobile Bottom Navigation */} {/* Mobile Bottom Navigation */}
{isMobile && ( {isMobile && (
<NavigationRail <div className="shrink-0">
orientation="horizontal" <NavigationRail
onManageApps={handleManageApps} orientation="horizontal"
onInlineApp={handleInlineApp} onManageApps={handleManageApps}
onCloseInlineApp={closeInlineApp} onInlineApp={handleInlineApp}
activeAppId={inlineApp?.id ?? null} onCloseInlineApp={closeInlineApp}
/> activeAppId={inlineApp?.id ?? null}
/>
</div>
)} )}
{detailEvent && detailAnchorRect && ( {detailEvent && detailAnchorRect && (
+1 -1
View File
@@ -544,7 +544,7 @@ export default function ContactsPage() {
}; };
return ( return (
<div className="flex h-dvh bg-background overflow-hidden"> <div className={cn("flex h-dvh bg-background overflow-hidden", isMobile && "flex-col")}>
{/* Navigation Rail - desktop only */} {/* Navigation Rail - desktop only */}
{!isMobile && ( {!isMobile && (
<div className="w-14 bg-secondary flex flex-col flex-shrink-0" style={{ borderRight: '1px solid rgba(128, 128, 128, 0.3)' }}> <div className="w-14 bg-secondary flex flex-col flex-shrink-0" style={{ borderRight: '1px solid rgba(128, 128, 128, 0.3)' }}>
+8 -6
View File
@@ -41,6 +41,7 @@ import { useSidebarApps } from "@/hooks/use-sidebar-apps";
import { Input } from "@/components/ui/input"; import { Input } from "@/components/ui/input";
import { FilePreviewModal } from "@/components/files/file-preview-modal"; import { FilePreviewModal } from "@/components/files/file-preview-modal";
import { isFilePreviewable } from "@/lib/file-preview"; import { isFilePreviewable } from "@/lib/file-preview";
import { appendPlainTextSignature } from "@/lib/signature-utils";
import { Search, Filter, ChevronDown, X, Paperclip, Star, Mail, MailOpen, RotateCcw, PenSquare, PenLine, CheckSquare, Square } from "lucide-react"; import { Search, Filter, ChevronDown, X, Paperclip, Star, Mail, MailOpen, RotateCcw, PenSquare, PenLine, CheckSquare, Square } from "lucide-react";
import { ResizeHandle } from "@/components/layout/resize-handle"; import { ResizeHandle } from "@/components/layout/resize-handle";
import { Button } from "@/components/ui/button"; import { Button } from "@/components/ui/button";
@@ -815,13 +816,13 @@ export default function Home() {
}; };
}, []); }, []);
const handleDownloadAttachment = async (blobId: string, name: string, type?: string) => { const handleDownloadAttachment = async (blobId: string, name: string, type?: string, forceDownload?: boolean) => {
if (!client) return; if (!client) return;
try { try {
const { mailAttachmentAction } = useSettingsStore.getState(); const { mailAttachmentAction } = useSettingsStore.getState();
if (mailAttachmentAction === 'preview' && isFilePreviewable(name, type)) { if (!forceDownload && mailAttachmentAction === 'preview' && isFilePreviewable(name, type)) {
setPreviewAttachment({ blobId, name, type }); setPreviewAttachment({ blobId, name, type });
return; return;
} }
@@ -862,10 +863,7 @@ export default function Home() {
const primaryIdentity = identities[0]; const primaryIdentity = identities[0];
// Append signature from the primary identity // Append signature from the primary identity
let finalBody = body; const finalBody = appendPlainTextSignature(body, primaryIdentity);
if (primaryIdentity?.textSignature) {
finalBody = body + '\n\n-- \n' + primaryIdentity.textSignature;
}
// Send reply with just the body text // Send reply with just the body text
await sendEmail( await sendEmail(
@@ -1022,6 +1020,10 @@ export default function Home() {
</button> </button>
); );
if (!isAuthenticated) {
return null;
}
return ( return (
<DragDropProvider> <DragDropProvider>
<div className="flex flex-col h-dvh bg-background overflow-hidden"> <div className="flex flex-col h-dvh bg-background overflow-hidden">
+103
View File
@@ -0,0 +1,103 @@
import { NextRequest, NextResponse } from 'next/server';
import { logger } from '@/lib/logger';
import { getStalwartCredentials } from '@/lib/stalwart/credentials';
interface DiscoveryAccountRequest {
key: string;
candidates: string[];
}
interface DiscoveryResult {
url: string | null;
resolvedAccount: string | null;
}
function buildPublicUrl(serverUrl: string, path: string): string {
return new URL(path, serverUrl).toString();
}
async function probeCalendarHome(serverUrl: string, authHeader: string, accountName: string): Promise<string | null> {
const targetUrl = buildPublicUrl(serverUrl, `/dav/cal/${encodeURIComponent(accountName)}`);
const response = await fetch(targetUrl, {
method: 'PROPFIND',
headers: {
Authorization: authHeader,
Depth: '0',
'Content-Type': 'application/xml; charset=utf-8',
},
body: `<?xml version="1.0" encoding="utf-8"?>
<D:propfind xmlns:D="DAV:">
<D:prop>
<D:resourcetype/>
<D:displayname/>
</D:prop>
</D:propfind>`,
redirect: 'manual',
});
if (response.status === 207) {
return targetUrl;
}
if (response.status >= 300 && response.status < 400) {
const location = response.headers.get('Location');
if (location) {
return new URL(location, targetUrl).toString();
}
}
return null;
}
export async function POST(request: NextRequest) {
try {
const creds = await getStalwartCredentials(request);
if (!creds) {
return NextResponse.json({ error: 'Not authenticated' }, { status: 401 });
}
const body = await request.json().catch(() => ({}));
const accounts = Array.isArray(body.accounts) ? body.accounts as DiscoveryAccountRequest[] : [];
const wellKnownUrl = buildPublicUrl(creds.serverUrl, '/.well-known/caldav');
const discovered: Record<string, DiscoveryResult> = {};
for (const account of accounts) {
if (!account?.key) continue;
const candidates = Array.from(new Set(
(account.candidates || [])
.map((candidate) => candidate?.trim())
.filter((candidate): candidate is string => Boolean(candidate))
));
let url: string | null = null;
let resolvedAccount: string | null = null;
for (const candidate of candidates) {
try {
url = await probeCalendarHome(creds.serverUrl, creds.authHeader, candidate);
if (url) {
resolvedAccount = candidate;
break;
}
} catch (error) {
logger.warn('CalDAV discovery probe failed', {
accountKey: account.key,
candidate,
error: error instanceof Error ? error.message : 'Unknown',
});
}
}
discovered[account.key] = { url, resolvedAccount };
}
return NextResponse.json({
wellKnownUrl,
accounts: discovered,
});
} catch (error) {
logger.error('CalDAV discovery failed', { error: error instanceof Error ? error.message : 'Unknown' });
return NextResponse.json({ error: 'Internal server error' }, { status: 500 });
}
}
+26 -35
View File
@@ -1,10 +1,21 @@
import v8 from 'node:v8';
import { NextResponse } from 'next/server'; import { NextResponse } from 'next/server';
import { NextRequest } from 'next/server'; import { NextRequest } from 'next/server';
import { logger } from '@/lib/logger'; import { logger } from '@/lib/logger';
// Health check thresholds const MEMORY_WARNING_THRESHOLD = 0.85;
const MEMORY_WARNING_THRESHOLD = 0.85; // 85% heap usage const MEMORY_CRITICAL_THRESHOLD = 0.95;
const MEMORY_CRITICAL_THRESHOLD = 0.95; // 95% heap usage
function getHeapUsagePercent(heapUsed: number, heapTotal: number): number {
const heapSizeLimit = v8.getHeapStatistics().heap_size_limit;
const denominator = heapSizeLimit > 0 ? heapSizeLimit : heapTotal;
if (denominator <= 0) {
return 0;
}
return (heapUsed / denominator) * 100;
}
interface HealthStatus { interface HealthStatus {
status: 'healthy' | 'degraded' | 'unhealthy'; status: 'healthy' | 'degraded' | 'unhealthy';
@@ -14,6 +25,7 @@ interface HealthStatus {
memory?: { memory?: {
heapUsed: number; heapUsed: number;
heapTotal: number; heapTotal: number;
heapSizeLimit: number;
rss: number; rss: number;
external: number; external: number;
heapUsagePercent: number; heapUsagePercent: number;
@@ -27,14 +39,9 @@ interface HealthStatus {
/** /**
* Health check endpoint for container orchestration * Health check endpoint for container orchestration
* *
* GET /api/health - Basic health check (returns 200 OK or 503 Service Unavailable) * GET /api/health - Liveness probe for container orchestration
* GET /api/health?detailed=true - Detailed diagnostics with memory stats * GET /api/health?detailed=true - Diagnostics with advisory memory warnings
* HEAD /api/health - Lightweight health check (status code only) * HEAD /api/health - Lightweight liveness probe (status code only)
*
* Health status based on Node.js heap usage:
* - Healthy (200): < 85% heap usage
* - Degraded (200): 85-95% heap usage (warnings in detailed mode)
* - Unhealthy (503): > 95% heap usage
*/ */
export async function GET(request: NextRequest) { export async function GET(request: NextRequest) {
const searchParams = request.nextUrl.searchParams; const searchParams = request.nextUrl.searchParams;
@@ -43,38 +50,31 @@ export async function GET(request: NextRequest) {
try { try {
const timestamp = new Date().toISOString(); const timestamp = new Date().toISOString();
const memUsage = process.memoryUsage(); const memUsage = process.memoryUsage();
const heapUsagePercent = (memUsage.heapUsed / memUsage.heapTotal) * 100; const heapSizeLimit = v8.getHeapStatistics().heap_size_limit;
const heapUsagePercent = getHeapUsagePercent(memUsage.heapUsed, memUsage.heapTotal);
// Determine health status based on memory usage
let status: 'healthy' | 'degraded' | 'unhealthy' = 'healthy'; let status: 'healthy' | 'degraded' | 'unhealthy' = 'healthy';
const warnings: string[] = []; const warnings: string[] = [];
let httpStatus = 200;
if (heapUsagePercent >= MEMORY_CRITICAL_THRESHOLD * 100) { if (heapUsagePercent >= MEMORY_CRITICAL_THRESHOLD * 100) {
status = 'unhealthy'; status = 'degraded';
httpStatus = 503; warnings.push(`V8 heap usage is very high: ${heapUsagePercent.toFixed(1)}% of heap limit`);
} else if (heapUsagePercent >= MEMORY_WARNING_THRESHOLD * 100) { } else if (heapUsagePercent >= MEMORY_WARNING_THRESHOLD * 100) {
status = 'degraded'; status = 'degraded';
warnings.push(`Memory usage high: ${heapUsagePercent.toFixed(1)}%`); warnings.push(`V8 heap usage is high: ${heapUsagePercent.toFixed(1)}% of heap limit`);
} }
// Build response
const response: HealthStatus = { const response: HealthStatus = {
status, status: detailed ? status : 'healthy',
timestamp, timestamp,
}; };
if (status === 'unhealthy') {
response.reason = `Memory usage critical: ${heapUsagePercent.toFixed(1)}%`;
}
// Add detailed information if requested
if (detailed) { if (detailed) {
response.uptime = process.uptime(); response.uptime = process.uptime();
response.version = process.env.npm_package_version || '0.1.0'; response.version = process.env.npm_package_version || '0.1.0';
response.memory = { response.memory = {
heapUsed: memUsage.heapUsed, heapUsed: memUsage.heapUsed,
heapTotal: memUsage.heapTotal, heapTotal: memUsage.heapTotal,
heapSizeLimit,
rss: memUsage.rss, rss: memUsage.rss,
external: memUsage.external, external: memUsage.external,
heapUsagePercent: Number(heapUsagePercent.toFixed(2)), heapUsagePercent: Number(heapUsagePercent.toFixed(2)),
@@ -87,10 +87,8 @@ export async function GET(request: NextRequest) {
} }
} }
logger.info('Health check', { status, detailed });
return NextResponse.json(response, { return NextResponse.json(response, {
status: httpStatus, status: 200,
headers: { headers: {
'Cache-Control': 'no-store, no-cache, must-revalidate', 'Cache-Control': 'no-store, no-cache, must-revalidate',
'Pragma': 'no-cache', 'Pragma': 'no-cache',
@@ -116,13 +114,6 @@ export async function GET(request: NextRequest) {
*/ */
export async function HEAD() { export async function HEAD() {
try { try {
const memUsage = process.memoryUsage();
const heapUsagePercent = (memUsage.heapUsed / memUsage.heapTotal) * 100;
if (heapUsagePercent >= MEMORY_CRITICAL_THRESHOLD * 100) {
return new Response(null, { status: 503 });
}
return new Response(null, { status: 200 }); return new Response(null, { status: 200 });
} catch { } catch {
return new Response(null, { status: 503 }); return new Response(null, { status: 503 });
+20 -9
View File
@@ -2,7 +2,7 @@ import { NextRequest, NextResponse } from 'next/server';
import { cookies } from 'next/headers'; import { cookies } from 'next/headers';
import { logger } from '@/lib/logger'; import { logger } from '@/lib/logger';
import { decryptSession } from '@/lib/auth/crypto'; import { decryptSession } from '@/lib/auth/crypto';
import { SESSION_COOKIE } from '@/lib/auth/session-cookie'; import { sessionCookieName } from '@/lib/auth/session-cookie';
import { saveUserSettings, loadUserSettings, deleteUserSettings } from '@/lib/settings-sync'; import { saveUserSettings, loadUserSettings, deleteUserSettings } from '@/lib/settings-sync';
function isEnabled(): boolean { function isEnabled(): boolean {
@@ -10,19 +10,30 @@ function isEnabled(): boolean {
} }
/** /**
* Verify identity against the session cookie if available. * Verify identity against session cookies across all account slots.
* Returns true if no session cookie exists (can't verify) or if identity matches. * With multi-account, the requesting account may be on any slot (0-4).
* Returns false if session cookie exists but identity doesn't match. * Returns true if any slot matches OR if no session cookies exist at all.
*/ */
async function verifyIdentity(username: string, serverUrl: string): Promise<boolean> { async function verifyIdentity(username: string, serverUrl: string): Promise<boolean> {
const cookieStore = await cookies(); const cookieStore = await cookies();
const sessionToken = cookieStore.get(SESSION_COOKIE)?.value; let hasAnyCookie = false;
if (!sessionToken) return true; // No session cookie, can't verify (same-origin protection applies)
const session = decryptSession(sessionToken); for (let slot = 0; slot <= 4; slot++) {
if (!session) return true; // Invalid session cookie, skip verification const token = cookieStore.get(sessionCookieName(slot))?.value;
if (!token) continue;
hasAnyCookie = true;
return session.username === username && session.serverUrl === serverUrl; const session = decryptSession(token);
if (session && session.username === username && session.serverUrl === serverUrl) {
return true; // Found a matching slot
}
}
// No cookies at all → can't verify, allow (same-origin protection applies)
if (!hasAnyCookie) return true;
// Cookies exist but none matched → identity mismatch
return false;
} }
export async function GET(request: NextRequest) { export async function GET(request: NextRequest) {
+6
View File
@@ -20,6 +20,8 @@
--color-accent-foreground: #1e40af; --color-accent-foreground: #1e40af;
--color-destructive: #ef4444; --color-destructive: #ef4444;
--color-destructive-foreground: #ffffff; --color-destructive-foreground: #ffffff;
--color-popover: #ffffff;
--color-popover-foreground: #0f172a;
/* Settings variables */ /* Settings variables */
--font-size-base: 16px; --font-size-base: 16px;
@@ -50,6 +52,8 @@
--color-accent-foreground: #dbeafe; --color-accent-foreground: #dbeafe;
--color-destructive: #ef4444; --color-destructive: #ef4444;
--color-destructive-foreground: #fafafa; --color-destructive-foreground: #fafafa;
--color-popover: #1c1c1c;
--color-popover-foreground: #fafafa;
} }
@theme inline { @theme inline {
@@ -68,6 +72,8 @@
--color-accent-foreground: var(--color-accent-foreground); --color-accent-foreground: var(--color-accent-foreground);
--color-destructive: var(--color-destructive); --color-destructive: var(--color-destructive);
--color-destructive-foreground: var(--color-destructive-foreground); --color-destructive-foreground: var(--color-destructive-foreground);
--color-popover: var(--color-popover);
--color-popover-foreground: var(--color-popover-foreground);
} }
* { * {
+33
View File
@@ -0,0 +1,33 @@
"use client";
import { useEffect } from "react";
import { useAuthStore } from "@/stores/auth-store";
export default function NotFound() {
const isAuthenticated = useAuthStore((s) => s.isAuthenticated);
useEffect(() => {
if (!isAuthenticated) {
window.location.href = "/login";
}
}, [isAuthenticated]);
if (!isAuthenticated) {
return null;
}
return (
<div className="min-h-screen flex items-center justify-center bg-background">
<div className="text-center max-w-md px-4">
<h1 className="text-4xl font-bold text-foreground mb-2">404</h1>
<p className="text-muted-foreground mb-6">This page could not be found.</p>
<a
href="/"
className="inline-flex items-center px-4 py-2 bg-primary text-primary-foreground rounded-lg hover:opacity-90 transition-opacity"
>
Go home
</a>
</div>
</div>
);
}
+45 -16
View File
@@ -1,12 +1,12 @@
"use client"; "use client";
import { useMemo } from "react"; import { useMemo, useRef, useEffect, useCallback } from "react";
import { useTranslations, useFormatter } from "next-intl"; import { useTranslations, useFormatter } from "next-intl";
import { format, parseISO, isToday, isTomorrow } from "date-fns"; import { format, parseISO, isToday, isTomorrow, startOfDay } from "date-fns";
import { Calendar as CalendarIcon, MapPin, Users } from "lucide-react"; import { Calendar as CalendarIcon, MapPin, Users } from "lucide-react";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
import { parseDuration, getEventColor } from "./event-card"; import { parseDuration, getEventColor } from "./event-card";
import { getEventDayBounds } from "@/lib/calendar-utils"; import { getEventDayBounds, getPrimaryCalendarId } from "@/lib/calendar-utils";
import { getParticipantCount } from "@/lib/calendar-participants"; import { getParticipantCount } from "@/lib/calendar-participants";
import type { CalendarEvent, Calendar } from "@/lib/jmap/types"; import type { CalendarEvent, Calendar } from "@/lib/jmap/types";
@@ -27,6 +27,7 @@ interface DayGroup {
} }
export function CalendarAgendaView({ export function CalendarAgendaView({
selectedDate,
events, events,
calendars, calendars,
onSelectEvent, onSelectEvent,
@@ -43,6 +44,9 @@ export function CalendarAgendaView({
return map; return map;
}, [calendars]); }, [calendars]);
const todayRef = useRef<HTMLDivElement>(null);
const scrollContainerRef = useRef<HTMLDivElement>(null);
const grouped = useMemo(() => { const grouped = useMemo(() => {
const sorted = [...events].sort((a, b) => const sorted = [...events].sort((a, b) =>
new Date(a.start).getTime() - new Date(b.start).getTime() new Date(a.start).getTime() - new Date(b.start).getTime()
@@ -69,10 +73,38 @@ export function CalendarAgendaView({
} catch { /* skip invalid dates */ } } catch { /* skip invalid dates */ }
}); });
// Always include today's date in the groups so the view has a "Today" anchor
const todayKey = format(new Date(), "yyyy-MM-dd");
if (!groupMap.has(todayKey)) {
const todayGroup = { date: startOfDay(new Date()), dateKey: todayKey, events: [] as CalendarEvent[] };
groupMap.set(todayKey, todayGroup);
groups.push(todayGroup);
}
groups.sort((a, b) => a.date.getTime() - b.date.getTime()); groups.sort((a, b) => a.date.getTime() - b.date.getTime());
return groups; return groups;
}, [events]); }, [events]);
// Auto-scroll to today's section on mount and when selectedDate changes to today
const scrollToToday = useCallback(() => {
if (todayRef.current) {
todayRef.current.scrollIntoView({ block: "start" });
}
}, []);
useEffect(() => {
// Scroll to today on mount
const frame = requestAnimationFrame(scrollToToday);
return () => cancelAnimationFrame(frame);
}, [scrollToToday]);
useEffect(() => {
// Scroll to today when selectedDate changes to today
if (isToday(selectedDate)) {
scrollToToday();
}
}, [selectedDate, scrollToToday]);
const formatDateHeader = (date: Date): string => { const formatDateHeader = (date: Date): string => {
if (isToday(date)) return t("events.today_header"); if (isToday(date)) return t("events.today_header");
if (isTomorrow(date)) return t("events.tomorrow_header"); if (isTomorrow(date)) return t("events.tomorrow_header");
@@ -86,19 +118,10 @@ export function CalendarAgendaView({
return format(date, "HH:mm"); return format(date, "HH:mm");
}; };
if (grouped.length === 0) {
return (
<div className="flex flex-col items-center justify-center flex-1 text-muted-foreground">
<CalendarIcon className="w-12 h-12 mb-3 opacity-30" />
<p className="text-sm">{t("events.no_events")}</p>
</div>
);
}
return ( return (
<div className="flex-1 overflow-y-auto"> <div className="flex-1 overflow-y-auto" ref={scrollContainerRef}>
{grouped.map((group) => ( {grouped.map((group) => (
<div key={group.dateKey}> <div key={group.dateKey} ref={isToday(group.date) ? todayRef : undefined}>
<div className="sticky top-0 bg-muted/80 backdrop-blur-sm px-4 py-2 border-b border-border"> <div className="sticky top-0 bg-muted/80 backdrop-blur-sm px-4 py-2 border-b border-border">
<span className={cn( <span className={cn(
"text-sm font-medium", "text-sm font-medium",
@@ -111,10 +134,15 @@ export function CalendarAgendaView({
</span> </span>
</div> </div>
{group.events.length === 0 ? (
<div className="px-4 py-6 text-center text-sm text-muted-foreground">
{t("events.no_events")}
</div>
) : (
<div className="divide-y divide-border"> <div className="divide-y divide-border">
{group.events.map((ev) => { {group.events.map((ev) => {
const calId = Object.keys(ev.calendarIds)[0]; const calId = getPrimaryCalendarId(ev);
const calendar = calendarMap.get(calId); const calendar = calId ? calendarMap.get(calId) : undefined;
const color = getEventColor(ev, calendar); const color = getEventColor(ev, calendar);
const start = parseISO(ev.start); const start = parseISO(ev.start);
const durMin = parseDuration(ev.duration); const durMin = parseDuration(ev.duration);
@@ -176,6 +204,7 @@ export function CalendarAgendaView({
); );
})} })}
</div> </div>
)}
</div> </div>
))} ))}
</div> </div>
+37 -6
View File
@@ -2,13 +2,14 @@
import { useMemo, useEffect, useRef, useState } from "react"; import { useMemo, useEffect, useRef, useState } from "react";
import { useTranslations, useFormatter } from "next-intl"; import { useTranslations, useFormatter } from "next-intl";
import { format, isToday, parseISO } from "date-fns"; import { format, isSameDay, isToday, parseISO } from "date-fns";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
import { EventCard, parseDuration } from "./event-card"; import { EventCard, parseDuration } from "./event-card";
import { QuickEventInput } from "./quick-event-input"; import { QuickEventInput } from "./quick-event-input";
import { getEventDayBounds, layoutOverlappingEvents, formatSnapTime } from "@/lib/calendar-utils"; import { formatSnapTime, getEventDayBounds, getPrimaryCalendarId, layoutOverlappingEvents } from "@/lib/calendar-utils";
import type { CalendarEvent, Calendar } from "@/lib/jmap/types"; import type { CalendarEvent, Calendar } from "@/lib/jmap/types";
import { useTimeGridInteractions } from "@/hooks/use-time-grid-interactions"; import { useTimeGridInteractions } from "@/hooks/use-time-grid-interactions";
import type { PendingEventPreview } from "./event-modal";
interface CalendarDayViewProps { interface CalendarDayViewProps {
selectedDate: Date; selectedDate: Date;
@@ -20,6 +21,7 @@ interface CalendarDayViewProps {
onCreateAtTime: (date: Date, endDate?: Date) => void; onCreateAtTime: (date: Date, endDate?: Date) => void;
timeFormat?: "12h" | "24h"; timeFormat?: "12h" | "24h";
isMobile?: boolean; isMobile?: boolean;
pendingPreview?: PendingEventPreview | null;
} }
const HOUR_HEIGHT = 64; const HOUR_HEIGHT = 64;
@@ -35,6 +37,7 @@ export function CalendarDayView({
onCreateAtTime, onCreateAtTime,
timeFormat = "24h", timeFormat = "24h",
isMobile, isMobile,
pendingPreview,
}: CalendarDayViewProps) { }: CalendarDayViewProps) {
const t = useTranslations("calendar"); const t = useTranslations("calendar");
const intlFormatter = useFormatter(); const intlFormatter = useFormatter();
@@ -127,12 +130,12 @@ export function CalendarDayView({
<div className="text-[10px] text-muted-foreground mb-1">{t("events.all_day")}</div> <div className="text-[10px] text-muted-foreground mb-1">{t("events.all_day")}</div>
<div className="space-y-1"> <div className="space-y-1">
{allDayEvents.map((ev) => { {allDayEvents.map((ev) => {
const calId = Object.keys(ev.calendarIds)[0]; const calId = getPrimaryCalendarId(ev);
return ( return (
<EventCard <EventCard
key={ev.id} key={ev.id}
event={ev} event={ev}
calendar={calendarMap.get(calId)} calendar={calId ? calendarMap.get(calId) : undefined}
variant="chip" variant="chip"
onClick={(rect) => onSelectEvent(ev, rect)} onClick={(rect) => onSelectEvent(ev, rect)}
onMouseEnter={(rect) => onHoverEvent?.(ev, rect)} onMouseEnter={(rect) => onHoverEvent?.(ev, rect)}
@@ -192,7 +195,7 @@ export function CalendarDayView({
const top = (startMin / 60) * HOUR_HEIGHT; const top = (startMin / 60) * HOUR_HEIGHT;
const baseHeight = Math.max(24, (durMin / 60) * HOUR_HEIGHT); const baseHeight = Math.max(24, (durMin / 60) * HOUR_HEIGHT);
const height = resizeVisual?.eventId === ev.id ? resizeVisual.heightPx : baseHeight; const height = resizeVisual?.eventId === ev.id ? resizeVisual.heightPx : baseHeight;
const calId = Object.keys(ev.calendarIds)[0]; const calId = getPrimaryCalendarId(ev);
const leftPct = (column / totalColumns) * 100; const leftPct = (column / totalColumns) * 100;
const widthPct = (1 / totalColumns) * 100; const widthPct = (1 / totalColumns) * 100;
@@ -205,7 +208,7 @@ export function CalendarDayView({
> >
<EventCard <EventCard
event={ev} event={ev}
calendar={calendarMap.get(calId)} calendar={calId ? calendarMap.get(calId) : undefined}
variant="block" variant="block"
onClick={(rect) => onSelectEvent(ev, rect)} onClick={(rect) => onSelectEvent(ev, rect)}
onMouseEnter={(rect) => onHoverEvent?.(ev, rect)} onMouseEnter={(rect) => onHoverEvent?.(ev, rect)}
@@ -274,6 +277,34 @@ export function CalendarDayView({
</div> </div>
</div> </div>
)} )}
{pendingPreview && !pendingPreview.allDay && isSameDay(pendingPreview.start, selectedDate) && (
(() => {
const startMin = pendingPreview.start.getHours() * 60 + pendingPreview.start.getMinutes();
const endMin = pendingPreview.end.getHours() * 60 + pendingPreview.end.getMinutes();
const durationMin = Math.max(15, endMin - startMin);
const cal = calendars.find(c => c.id === pendingPreview.calendarId);
const color = cal?.color || "hsl(var(--primary))";
return (
<div
className="absolute left-2 right-2 z-10 rounded-md pointer-events-none border-2 border-dashed overflow-hidden"
style={{
top: (startMin / 60) * HOUR_HEIGHT,
height: Math.max(24, (durationMin / 60) * HOUR_HEIGHT),
borderColor: color,
backgroundColor: `${color}10`,
}}
>
<div className="text-[10px] font-medium px-1.5 py-0.5 truncate" style={{ color }}>
{pendingPreview.title}
</div>
<div className="text-[9px] px-1.5 opacity-70" style={{ color }}>
{formatSnapTime(startMin, timeFormat)} {formatSnapTime(startMin + durationMin, timeFormat)}
</div>
</div>
);
})()
)}
</div> </div>
</div> </div>
</div> </div>
+57 -22
View File
@@ -8,10 +8,11 @@ import {
} from "date-fns"; } from "date-fns";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
import { EventCard } from "./event-card"; import { EventCard } from "./event-card";
import { buildWeekSegments, getEventDayBounds } from "@/lib/calendar-utils"; import { buildWeekSegments, getEventDayBounds, getPrimaryCalendarId } from "@/lib/calendar-utils";
import type { CalendarEvent, Calendar } from "@/lib/jmap/types"; import type { CalendarEvent, Calendar } from "@/lib/jmap/types";
import { useAuthStore } from "@/stores/auth-store"; import { useAuthStore } from "@/stores/auth-store";
import { useCalendarStore } from "@/stores/calendar-store"; import { useCalendarStore } from "@/stores/calendar-store";
import type { PendingEventPreview } from "./event-modal";
import { toast } from "@/stores/toast-store"; import { toast } from "@/stores/toast-store";
interface CalendarMonthViewProps { interface CalendarMonthViewProps {
@@ -25,6 +26,7 @@ interface CalendarMonthViewProps {
onCreateAtTime?: (date: Date) => void; onCreateAtTime?: (date: Date) => void;
firstDayOfWeek?: number; firstDayOfWeek?: number;
isMobile?: boolean; isMobile?: boolean;
pendingPreview?: PendingEventPreview | null;
} }
export function CalendarMonthView({ export function CalendarMonthView({
@@ -38,6 +40,7 @@ export function CalendarMonthView({
onCreateAtTime, onCreateAtTime,
firstDayOfWeek = 1, firstDayOfWeek = 1,
isMobile, isMobile,
pendingPreview,
}: CalendarMonthViewProps) { }: CalendarMonthViewProps) {
const t = useTranslations("calendar"); const t = useTranslations("calendar");
const intlFormatter = useFormatter(); const intlFormatter = useFormatter();
@@ -194,35 +197,67 @@ export function CalendarMonthView({
</span> </span>
</div> </div>
{isMobile ? ( {isMobile ? (
dayEvents.length > 0 && ( <div className="flex items-center justify-center gap-0.5 flex-wrap">
<div className="flex items-center justify-center gap-0.5 flex-wrap"> {dayEvents.slice(0, 3).map((ev) => {
{dayEvents.slice(0, 3).map((ev) => { const calId = getPrimaryCalendarId(ev);
const calId = Object.keys(ev.calendarIds)[0]; const cal = calId ? calendarMap.get(calId) : undefined;
const cal = calendarMap.get(calId); const evColor = ev.color || cal?.color || "#3b82f6";
const evColor = ev.color || cal?.color || "#3b82f6"; return (
return ( <span
<span key={ev.id}
key={ev.id} className="w-1.5 h-1.5 rounded-full"
className="w-1.5 h-1.5 rounded-full" style={{ backgroundColor: evColor }}
style={{ backgroundColor: evColor }} />
/> );
); })}
})} {dayEvents.length > 3 && (
{dayEvents.length > 3 && ( <span className="w-1.5 h-1.5 rounded-full bg-muted-foreground/40" />
<span className="w-1.5 h-1.5 rounded-full bg-muted-foreground/40" /> )}
)} {pendingPreview && isSameDay(pendingPreview.start, day) && (
</div> <span
) className="w-1.5 h-1.5 rounded-full border border-dashed"
style={{ borderColor: calendarMap.get(pendingPreview.calendarId)?.color || "#3b82f6" }}
/>
)}
</div>
) : null} ) : null}
</div> </div>
); );
})} })}
</div> </div>
{!isMobile && pendingPreview && (() => {
const previewDayIdx = week.findIndex(d => isSameDay(d, pendingPreview.start));
if (previewDayIdx === -1) return null;
const previewRow = rowCount;
const cal = calendarMap.get(pendingPreview.calendarId);
const color = cal?.color || "#3b82f6";
return (
<div className="absolute inset-x-0 pointer-events-none" style={{ top: 30 }}>
<div
className="absolute px-0.5"
style={{
left: `calc(${(previewDayIdx / 7) * 100}% + 1px)`,
width: `calc(${(1 / 7) * 100}% - 2px)`,
top: previewRow * 22,
height: 20,
}}
>
<div
className="h-full rounded text-[10px] leading-[20px] font-medium px-1.5 truncate border-2 border-dashed"
style={{ borderColor: color, color, backgroundColor: `${color}10` }}
>
{pendingPreview.title}
</div>
</div>
</div>
);
})()}
{!isMobile && segments.length > 0 && ( {!isMobile && segments.length > 0 && (
<div className="absolute inset-x-0 pointer-events-none" style={{ top: 30 }}> <div className="absolute inset-x-0 pointer-events-none" style={{ top: 30 }}>
{segments.map((segment) => { {segments.map((segment) => {
const calId = Object.keys(segment.event.calendarIds)[0]; const calId = getPrimaryCalendarId(segment.event);
return ( return (
<div <div
key={`${segment.event.id}-${segment.startIndex}-${segment.row}`} key={`${segment.event.id}-${segment.startIndex}-${segment.row}`}
@@ -236,7 +271,7 @@ export function CalendarMonthView({
> >
<EventCard <EventCard
event={segment.event} event={segment.event}
calendar={calendarMap.get(calId)} calendar={calId ? calendarMap.get(calId) : undefined}
variant="span" variant="span"
continuesBefore={segment.continuesBefore} continuesBefore={segment.continuesBefore}
continuesAfter={segment.continuesAfter} continuesAfter={segment.continuesAfter}
@@ -47,7 +47,7 @@ export function CalendarSidebarPanel({
const shared = calendars.filter(c => c.isShared); const shared = calendars.filter(c => c.isShared);
const groups = new Map<string, { accountName: string; calendars: Calendar[] }>(); const groups = new Map<string, { accountName: string; calendars: Calendar[] }>();
for (const cal of shared) { for (const cal of shared) {
const key = cal.accountId!; const key = cal.accountId || cal.accountName || cal.id;
if (!groups.has(key)) { if (!groups.has(key)) {
groups.set(key, { accountName: cal.accountName || key, calendars: [] }); groups.set(key, { accountName: cal.accountName || key, calendars: [] });
} }
+15 -1
View File
@@ -136,6 +136,20 @@ export function CalendarToolbar({
{t("views.today")} {t("views.today")}
</Button> </Button>
{!isMobile && (
<div className="flex items-center gap-1">
<Button variant="ghost" size="icon" className="h-8 w-8" onClick={onPrev} aria-label={t("nav_prev")}>
<ChevronLeft className="w-4 h-4" />
</Button>
<Button variant="ghost" size="icon" className="h-8 w-8" onClick={onNext} aria-label={t("nav_next")}>
<ChevronRight className="w-4 h-4" />
</Button>
<span className="text-base font-semibold ml-2 select-none">
{getDateLabel()}
</span>
</div>
)}
{isMobile && calendars && selectedCalendarIds && onToggleVisibility && ( {isMobile && calendars && selectedCalendarIds && onToggleVisibility && (
<div className="relative" ref={dropdownRef}> <div className="relative" ref={dropdownRef}>
<Button <Button
@@ -183,7 +197,7 @@ export function CalendarToolbar({
const shared = calendars.filter(c => c.isShared); const shared = calendars.filter(c => c.isShared);
const groups = new Map<string, { accountName: string; cals: typeof shared }>(); const groups = new Map<string, { accountName: string; cals: typeof shared }>();
for (const c of shared) { for (const c of shared) {
const key = c.accountId!; const key = c.accountId || c.accountName || c.id;
if (!groups.has(key)) groups.set(key, { accountName: c.accountName || key, cals: [] }); if (!groups.has(key)) groups.set(key, { accountName: c.accountName || key, cals: [] });
groups.get(key)!.cals.push(c); groups.get(key)!.cals.push(c);
} }
+36 -5
View File
@@ -8,9 +8,10 @@ import {
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
import { EventCard, parseDuration } from "./event-card"; import { EventCard, parseDuration } from "./event-card";
import { QuickEventInput } from "./quick-event-input"; import { QuickEventInput } from "./quick-event-input";
import { buildWeekSegments, getEventDayBounds, layoutOverlappingEvents, formatSnapTime } from "@/lib/calendar-utils"; import { buildWeekSegments, formatSnapTime, getEventDayBounds, getPrimaryCalendarId, layoutOverlappingEvents } from "@/lib/calendar-utils";
import type { CalendarEvent, Calendar } from "@/lib/jmap/types"; import type { CalendarEvent, Calendar } from "@/lib/jmap/types";
import { useTimeGridInteractions } from "@/hooks/use-time-grid-interactions"; import { useTimeGridInteractions } from "@/hooks/use-time-grid-interactions";
import type { PendingEventPreview } from "./event-modal";
interface CalendarWeekViewProps { interface CalendarWeekViewProps {
selectedDate: Date; selectedDate: Date;
@@ -24,6 +25,7 @@ interface CalendarWeekViewProps {
firstDayOfWeek?: number; firstDayOfWeek?: number;
timeFormat?: "12h" | "24h"; timeFormat?: "12h" | "24h";
isMobile?: boolean; isMobile?: boolean;
pendingPreview?: PendingEventPreview | null;
} }
const HOUR_HEIGHT = 60; const HOUR_HEIGHT = 60;
@@ -41,6 +43,7 @@ export function CalendarWeekView({
firstDayOfWeek = 1, firstDayOfWeek = 1,
timeFormat = "24h", timeFormat = "24h",
isMobile, isMobile,
pendingPreview,
}: CalendarWeekViewProps) { }: CalendarWeekViewProps) {
const t = useTranslations("calendar"); const t = useTranslations("calendar");
const intlFormatter = useFormatter(); const intlFormatter = useFormatter();
@@ -162,7 +165,7 @@ export function CalendarWeekView({
<div className="absolute inset-0 pointer-events-none"> <div className="absolute inset-0 pointer-events-none">
{allDaySegments.map((segment) => { {allDaySegments.map((segment) => {
const calId = Object.keys(segment.event.calendarIds)[0]; const calId = getPrimaryCalendarId(segment.event);
return ( return (
<div <div
key={`${segment.event.id}-${segment.startIndex}-${segment.row}`} key={`${segment.event.id}-${segment.startIndex}-${segment.row}`}
@@ -176,7 +179,7 @@ export function CalendarWeekView({
> >
<EventCard <EventCard
event={segment.event} event={segment.event}
calendar={calendarMap.get(calId)} calendar={calId ? calendarMap.get(calId) : undefined}
variant="span" variant="span"
continuesBefore={segment.continuesBefore} continuesBefore={segment.continuesBefore}
continuesAfter={segment.continuesAfter} continuesAfter={segment.continuesAfter}
@@ -284,7 +287,7 @@ export function CalendarWeekView({
const top = (startMin / 60) * HOUR_HEIGHT; const top = (startMin / 60) * HOUR_HEIGHT;
const baseHeight = Math.max(20, (durMin / 60) * HOUR_HEIGHT); const baseHeight = Math.max(20, (durMin / 60) * HOUR_HEIGHT);
const height = resizeVisual?.eventId === ev.id ? resizeVisual.heightPx : baseHeight; const height = resizeVisual?.eventId === ev.id ? resizeVisual.heightPx : baseHeight;
const calId = Object.keys(ev.calendarIds)[0]; const calId = getPrimaryCalendarId(ev);
const leftPct = (column / totalColumns) * 100; const leftPct = (column / totalColumns) * 100;
const widthPct = (1 / totalColumns) * 100; const widthPct = (1 / totalColumns) * 100;
@@ -297,7 +300,7 @@ export function CalendarWeekView({
> >
<EventCard <EventCard
event={ev} event={ev}
calendar={calendarMap.get(calId)} calendar={calId ? calendarMap.get(calId) : undefined}
variant="block" variant="block"
onClick={(rect) => onSelectEvent(ev, rect)} onClick={(rect) => onSelectEvent(ev, rect)}
onMouseEnter={(rect) => onHoverEvent?.(ev, rect)} onMouseEnter={(rect) => onHoverEvent?.(ev, rect)}
@@ -366,6 +369,34 @@ export function CalendarWeekView({
</div> </div>
</div> </div>
)} )}
{pendingPreview && !pendingPreview.allDay && isSameDay(pendingPreview.start, day) && (
(() => {
const startMin = pendingPreview.start.getHours() * 60 + pendingPreview.start.getMinutes();
const endMin = pendingPreview.end.getHours() * 60 + pendingPreview.end.getMinutes();
const durationMin = Math.max(15, endMin - startMin);
const cal = calendars.find(c => c.id === pendingPreview.calendarId);
const color = cal?.color || "hsl(var(--primary))";
return (
<div
className="absolute left-1 right-1 z-10 rounded-md pointer-events-none border-2 border-dashed overflow-hidden"
style={{
top: (startMin / 60) * HOUR_HEIGHT,
height: Math.max(20, (durationMin / 60) * HOUR_HEIGHT),
borderColor: color,
backgroundColor: `${color}10`,
}}
>
<div className="text-[10px] font-medium px-1.5 py-0.5 truncate" style={{ color }}>
{pendingPreview.title}
</div>
<div className="text-[9px] px-1.5 opacity-70" style={{ color }}>
{formatSnapTime(startMin, timeFormat)} {formatSnapTime(startMin + durationMin, timeFormat)}
</div>
</div>
);
})()
)}
</div> </div>
); );
})} })}
+4
View File
@@ -70,6 +70,7 @@ export function EventCard({ event, calendar, variant, onClick, onMouseEnter, onM
const color = getEventColor(event, calendar); const color = getEventColor(event, calendar);
const startDate = parseISO(event.start); const startDate = parseISO(event.start);
const timeFormat = useSettingsStore((state) => state.timeFormat); const timeFormat = useSettingsStore((state) => state.timeFormat);
const showTimeInMonthView = useSettingsStore((state) => state.showTimeInMonthView);
const timeFmt = timeFormat === "12h" ? "h:mm a" : "HH:mm"; const timeFmt = timeFormat === "12h" ? "h:mm a" : "HH:mm";
const calendarName = calendar?.name || ""; const calendarName = calendar?.name || "";
@@ -156,6 +157,9 @@ export function EventCard({ event, calendar, variant, onClick, onMouseEnter, onM
style={{ backgroundColor: `${color}24`, borderLeft: `3px solid ${color}`, color, ...style }} style={{ backgroundColor: `${color}24`, borderLeft: `3px solid ${color}`, color, ...style }}
> >
<div className="flex items-center gap-1 min-w-0"> <div className="flex items-center gap-1 min-w-0">
{showTimeInMonthView && !event.showWithoutTime && (
<span className="flex-shrink-0 opacity-80">{format(startDate, timeFmt)}</span>
)}
<span className="truncate font-medium">{event.title || t("events.no_title")}</span> <span className="truncate font-medium">{event.title || t("events.no_title")}</span>
</div> </div>
</button> </button>
+36 -8
View File
@@ -8,7 +8,7 @@ import { X, Trash2, Check, Users, CalendarDays, Copy, Pencil, Clock, MapPin, Vid
import { format, parseISO, addHours, addDays } from "date-fns"; import { format, parseISO, addHours, addDays } from "date-fns";
import type { CalendarEvent, Calendar, CalendarParticipant } from "@/lib/jmap/types"; import type { CalendarEvent, Calendar, CalendarParticipant } from "@/lib/jmap/types";
import { parseDuration, getEventColor } from "./event-card"; import { parseDuration, getEventColor } from "./event-card";
import { buildAllDayDuration, getEventDisplayEndDate } from "@/lib/calendar-utils"; import { buildAllDayDuration, getEventDisplayEndDate, getPrimaryCalendarId } from "@/lib/calendar-utils";
import { ParticipantInput } from "./participant-input"; import { ParticipantInput } from "./participant-input";
import { import {
isOrganizer, isOrganizer,
@@ -20,16 +20,25 @@ import {
} from "@/lib/calendar-participants"; } from "@/lib/calendar-participants";
import { useSettingsStore } from "@/stores/settings-store"; import { useSettingsStore } from "@/stores/settings-store";
export interface PendingEventPreview {
start: Date;
end: Date;
title: string;
allDay: boolean;
calendarId: string;
}
interface EventModalProps { interface EventModalProps {
event?: CalendarEvent | null; event?: CalendarEvent | null;
calendars: Calendar[]; calendars: Calendar[];
defaultDate?: Date; defaultDate?: Date;
defaultEndDate?: Date; defaultEndDate?: Date;
onSave: (data: Partial<CalendarEvent>, sendSchedulingMessages?: boolean) => void; onSave: (data: Partial<CalendarEvent>, sendSchedulingMessages?: boolean) => void | Promise<void>;
onDelete?: (id: string, sendSchedulingMessages?: boolean) => void; onDelete?: (id: string, sendSchedulingMessages?: boolean) => void;
onDuplicate?: (data: Partial<CalendarEvent>) => void; onDuplicate?: (data: Partial<CalendarEvent>) => void;
onRsvp?: (eventId: string, participantId: string, status: CalendarParticipant['participationStatus']) => void; onRsvp?: (eventId: string, participantId: string, status: CalendarParticipant['participationStatus']) => void;
onClose: () => void; onClose: () => void;
onPreviewChange?: (preview: PendingEventPreview | null) => void;
currentUserEmails?: string[]; currentUserEmails?: string[];
isMobile?: boolean; isMobile?: boolean;
} }
@@ -105,6 +114,7 @@ export function EventModal({
onDuplicate, onDuplicate,
onRsvp, onRsvp,
onClose, onClose,
onPreviewChange,
currentUserEmails = [], currentUserEmails = [],
isMobile = false, isMobile = false,
}: EventModalProps) { }: EventModalProps) {
@@ -184,7 +194,7 @@ export function EventModal({
const [endTime, setEndTime] = useState(formatTimeInput(getInitialEnd())); const [endTime, setEndTime] = useState(formatTimeInput(getInitialEnd()));
const [allDay, setAllDay] = useState(event?.showWithoutTime || false); const [allDay, setAllDay] = useState(event?.showWithoutTime || false);
const [calendarId, setCalendarId] = useState<string>(() => { const [calendarId, setCalendarId] = useState<string>(() => {
if (event?.calendarIds) return Object.keys(event.calendarIds)[0] || calendars[0]?.id || ""; if (event?.calendarIds) return getPrimaryCalendarId(event) || calendars[0]?.id || "";
const defaultCal = calendars.find(c => c.isDefault); const defaultCal = calendars.find(c => c.isDefault);
return defaultCal?.id || calendars[0]?.id || ""; return defaultCal?.id || calendars[0]?.id || "";
}); });
@@ -209,6 +219,7 @@ export function EventModal({
return "none"; return "none";
}); });
const [showDeleteConfirm, setShowDeleteConfirm] = useState(false); const [showDeleteConfirm, setShowDeleteConfirm] = useState(false);
const [isSaving, setIsSaving] = useState(false);
const [attendees, setAttendees] = useState<{ name: string; email: string }[]>(() => { const [attendees, setAttendees] = useState<{ name: string; email: string }[]>(() => {
if (!event?.participants) return []; if (!event?.participants) return [];
@@ -218,6 +229,18 @@ export function EventModal({
}); });
const [sendInvitations, setSendInvitations] = useState(true); const [sendInvitations, setSendInvitations] = useState(true);
// Report live preview to parent for grid outline
useEffect(() => {
if (!onPreviewChange || isEdit) return;
const startStr = allDay ? `${startDate}T00:00:00` : `${startDate}T${startTime}:00`;
const endStr = allDay ? `${endDate}T23:59:59` : `${endDate}T${endTime}:00`;
const s = new Date(startStr);
const e = new Date(endStr);
if (isNaN(s.getTime()) || isNaN(e.getTime())) return;
onPreviewChange({ start: s, end: e, title: title || "(No title)", allDay, calendarId });
return () => onPreviewChange(null);
}, [startDate, startTime, endDate, endTime, allDay, title, calendarId, isEdit, onPreviewChange]);
const statusCounts = useMemo(() => { const statusCounts = useMemo(() => {
if (!event?.participants) return null; if (!event?.participants) return null;
return getStatusCounts(event); return getStatusCounts(event);
@@ -231,9 +254,9 @@ export function EventModal({
setAttendees(prev => prev.filter(a => a.email.toLowerCase() !== email.toLowerCase())); setAttendees(prev => prev.filter(a => a.email.toLowerCase() !== email.toLowerCase()));
}, []); }, []);
const handleSave = useCallback(() => { const handleSave = useCallback(async () => {
const trimmedTitle = title.trim(); const trimmedTitle = title.trim();
if (!trimmedTitle) return; if (!trimmedTitle || isSaving) return;
if (trimmedTitle.length > 500 || description.trim().length > 10000 || location.trim().length > 500) return; if (trimmedTitle.length > 500 || description.trim().length > 10000 || location.trim().length > 500) return;
const startStr = allDay const startStr = allDay
@@ -343,8 +366,13 @@ export function EventModal({
} }
const shouldSendScheduling = attendees.length > 0 && sendInvitations; const shouldSendScheduling = attendees.length > 0 && sendInvitations;
onSave(data, shouldSendScheduling); setIsSaving(true);
}, [title, description, location, startDate, startTime, endDate, endTime, allDay, calendarId, recurrence, alert, attendees, sendInvitations, currentUserEmails, existingParticipants, event, onSave]); try {
await onSave(data, shouldSendScheduling);
} finally {
setIsSaving(false);
}
}, [title, description, location, startDate, startTime, endDate, endTime, allDay, calendarId, recurrence, alert, attendees, sendInvitations, currentUserEmails, existingParticipants, event, onSave, isSaving]);
const handleRsvp = useCallback((status: CalendarParticipant['participationStatus']) => { const handleRsvp = useCallback((status: CalendarParticipant['participationStatus']) => {
if (!event || !userParticipantId || !onRsvp) return; if (!event || !userParticipantId || !onRsvp) return;
@@ -945,7 +973,7 @@ export function EventModal({
<Button variant="outline" onClick={isEdit ? () => setMode("view") : onClose}> <Button variant="outline" onClick={isEdit ? () => setMode("view") : onClose}>
{t("form.cancel")} {t("form.cancel")}
</Button> </Button>
<Button onClick={handleSave} disabled={!title.trim()}> <Button onClick={handleSave} disabled={!title.trim() || isSaving}>
{t("form.save")} {t("form.save")}
</Button> </Button>
</div> </div>
+21 -8
View File
@@ -6,7 +6,7 @@ import { Mail, Phone, Building, MapPin, StickyNote, Pencil, Trash2, BookUser, Co
import { Avatar } from "@/components/ui/avatar"; import { Avatar } from "@/components/ui/avatar";
import { Button } from "@/components/ui/button"; import { Button } from "@/components/ui/button";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
import type { ContactCard } from "@/lib/jmap/types"; import type { ContactCard, AnniversaryDate, PartialDate } from "@/lib/jmap/types";
import { getContactDisplayName, getContactPrimaryEmail } from "@/stores/contact-store"; import { getContactDisplayName, getContactPrimaryEmail } from "@/stores/contact-store";
import { useSmimeStore } from "@/stores/smime-store"; import { useSmimeStore } from "@/stores/smime-store";
import { parseCertificatePemOrDer, extractCertificateInfo } from "@/lib/smime/certificate-utils"; import { parseCertificatePemOrDer, extractCertificateInfo } from "@/lib/smime/certificate-utils";
@@ -26,12 +26,23 @@ function formatPhoneFeatures(features?: Record<string, boolean>): string {
return Object.keys(features).filter(k => features[k]).join(", "); return Object.keys(features).filter(k => features[k]).join(", ");
} }
function formatDate(dateInput: string | Record<string, unknown>): string { function formatDate(dateInput: AnniversaryDate): string {
// Handle RFC 9553 PartialDate objects: { year?, month?, day?, calendarScale? } // Handle RFC 9553 PartialDate objects: { year?, month?, day?, calendarScale? }
// Handle RFC 9553 Timestamp objects: { "@type": "Timestamp", utc: "..." }
if (typeof dateInput === 'object' && dateInput !== null) { if (typeof dateInput === 'object' && dateInput !== null) {
const year = dateInput.year as number | undefined; if (dateInput['@type'] === 'Timestamp' && typeof dateInput.utc === 'string') {
const month = dateInput.month as number | undefined; try {
const day = dateInput.day as number | undefined; const d = new Date(dateInput.utc as string);
if (!isNaN(d.getTime())) {
return d.toLocaleDateString(undefined, { year: "numeric", month: "long", day: "numeric" });
}
} catch { /* fallback */ }
return String(dateInput.utc);
}
const pd = dateInput as PartialDate;
const year = pd.year;
const month = pd.month;
const day = pd.day;
const monthNames = ["Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"]; const monthNames = ["Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"];
const parts: string[] = []; const parts: string[] = [];
if (month && monthNames[month - 1]) parts.push(monthNames[month - 1]); if (month && monthNames[month - 1]) parts.push(monthNames[month - 1]);
@@ -282,9 +293,11 @@ export function ContactDetail({ contact, onEdit, onDelete, isMobile, className }
{addresses.map((a, i) => ( {addresses.map((a, i) => (
<div key={i} className="text-sm space-y-0.5 rounded-md border border-border/60 bg-muted/30 p-3"> <div key={i} className="text-sm space-y-0.5 rounded-md border border-border/60 bg-muted/30 p-3">
<div> <div>
{a.fullAddress {a.full || a.fullAddress
? a.fullAddress ? (a.full || a.fullAddress)
: [a.street, a.locality, a.region, a.postcode, a.country].filter(Boolean).join(", ")} : a.components && a.components.length > 0
? a.components.filter(c => c.kind !== 'separator').map(c => c.value).filter(Boolean).join(", ")
: [a.street, a.locality, a.region, a.postcode, a.country].filter(Boolean).join(", ")}
{a.contexts && <ContextBadge contexts={a.contexts} />} {a.contexts && <ContextBadge contexts={a.contexts} />}
</div> </div>
{a.timeZone && ( {a.timeZone && (
+72 -17
View File
@@ -6,7 +6,7 @@ import { X, Plus, ChevronDown, ChevronRight, User, Building, MapPin, Globe, Cake
import { Button } from "@/components/ui/button"; import { Button } from "@/components/ui/button";
import { Input } from "@/components/ui/input"; import { Input } from "@/components/ui/input";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
import type { ContactCard, ContactOnlineService, ContactAnniversary, ContactPersonalInfo, AddressBook } from "@/lib/jmap/types"; import type { ContactCard, ContactOnlineService, ContactAnniversary, ContactPersonalInfo, AddressBook, AnniversaryDate, PartialDate, ContactAddress } from "@/lib/jmap/types";
interface EmailEntry { interface EmailEntry {
address: string; address: string;
@@ -129,6 +129,67 @@ export function ContactForm({ contact, addressBooks, onSave, onCancel }: Contact
const findComponent = (kind: string) => contact?.name?.components?.find(c => c.kind === kind)?.value || ""; const findComponent = (kind: string) => contact?.name?.components?.find(c => c.kind === kind)?.value || "";
// Convert RFC 9553 AnniversaryDate to ISO date string for HTML date input
function anniversaryDateToString(date: AnniversaryDate): string {
if (typeof date === 'string') return date;
if (date && typeof date === 'object') {
if ('@type' in date && date['@type'] === 'Timestamp' && 'utc' in date) {
return (date as { utc: string }).utc.split('T')[0];
}
const pd = date as PartialDate;
if (pd.year && pd.month && pd.day) {
return `${String(pd.year).padStart(4, '0')}-${String(pd.month).padStart(2, '0')}-${String(pd.day).padStart(2, '0')}`;
}
if (pd.month && pd.day) {
return `--${String(pd.month).padStart(2, '0')}-${String(pd.day).padStart(2, '0')}`;
}
if (pd.year && pd.month) {
return `${String(pd.year).padStart(4, '0')}-${String(pd.month).padStart(2, '0')}`;
}
if (pd.year) return String(pd.year);
}
return String(date);
}
// Convert ISO date string back to RFC 9553 PartialDate for the server
function stringToPartialDate(str: string): PartialDate {
if (str.startsWith('--')) {
const parts = str.substring(2).split('-');
const pd: PartialDate = { month: parseInt(parts[0], 10) };
if (parts[1]) pd.day = parseInt(parts[1], 10);
return pd;
}
const parts = str.split('-');
const pd: PartialDate = {};
if (parts[0]) pd.year = parseInt(parts[0], 10);
if (parts[1]) pd.month = parseInt(parts[1], 10);
if (parts[2]) pd.day = parseInt(parts[2], 10);
return pd;
}
// Extract flat address fields from RFC 9553 components format
function addressToFlat(a: ContactAddress): AddressEntry {
if (a.components && a.components.length > 0) {
const findComp = (kind: string) => a.components!.filter(c => c.kind === kind).map(c => c.value).join(' ');
return {
street: findComp('name') || findComp('number') ? [findComp('number'), findComp('name')].filter(Boolean).join(' ') : '',
locality: findComp('locality'),
region: findComp('region'),
postcode: findComp('postcode'),
country: findComp('country'),
context: a.contexts?.work ? 'work' : a.contexts?.private ? 'private' : '',
};
}
return {
street: a.street || '',
locality: a.locality || '',
region: a.region || '',
postcode: a.postcode || '',
country: a.country || '',
context: a.contexts?.work ? 'work' : a.contexts?.private ? 'private' : '',
};
}
const [prefix, setPrefix] = useState(findComponent("prefix")); const [prefix, setPrefix] = useState(findComponent("prefix"));
const [givenName, setGivenName] = useState(findComponent("given")); const [givenName, setGivenName] = useState(findComponent("given"));
const [additionalName, setAdditionalName] = useState(findComponent("additional")); const [additionalName, setAdditionalName] = useState(findComponent("additional"));
@@ -184,14 +245,7 @@ export function ContactForm({ contact, addressBooks, onSave, onCancel }: Contact
const [addresses, setAddresses] = useState<AddressEntry[]>(() => { const [addresses, setAddresses] = useState<AddressEntry[]>(() => {
if (contact?.addresses) { if (contact?.addresses) {
return Object.values(contact.addresses).map(a => ({ return Object.values(contact.addresses).map(a => addressToFlat(a));
street: a.street || "",
locality: a.locality || "",
region: a.region || "",
postcode: a.postcode || "",
country: a.country || "",
context: a.contexts?.work ? "work" : a.contexts?.private ? "private" : "",
}));
} }
return []; return [];
}); });
@@ -210,7 +264,7 @@ export function ContactForm({ contact, addressBooks, onSave, onCancel }: Contact
const [anniversaries, setAnniversaries] = useState<AnniversaryEntry[]>(() => { const [anniversaries, setAnniversaries] = useState<AnniversaryEntry[]>(() => {
if (contact?.anniversaries) { if (contact?.anniversaries) {
return Object.values(contact.anniversaries).map(a => ({ return Object.values(contact.anniversaries).map(a => ({
date: a.date, date: anniversaryDateToString(a.date),
kind: a.kind, kind: a.kind,
})); }));
} }
@@ -336,12 +390,13 @@ export function ContactForm({ contact, addressBooks, onSave, onCancel }: Contact
const addressesMap: Record<string, ContactCard["addresses"] extends Record<string, infer V> ? V : never> = {}; const addressesMap: Record<string, ContactCard["addresses"] extends Record<string, infer V> ? V : never> = {};
addresses.filter(a => a.street.trim() || a.locality.trim() || a.country.trim()).forEach((a, i) => { addresses.filter(a => a.street.trim() || a.locality.trim() || a.country.trim()).forEach((a, i) => {
const obj: Record<string, unknown> = {}; const components: Array<{ kind: string; value: string }> = [];
if (a.street.trim()) obj.street = a.street.trim(); if (a.street.trim()) components.push({ kind: "name", value: a.street.trim() });
if (a.locality.trim()) obj.locality = a.locality.trim(); if (a.locality.trim()) components.push({ kind: "locality", value: a.locality.trim() });
if (a.region.trim()) obj.region = a.region.trim(); if (a.region.trim()) components.push({ kind: "region", value: a.region.trim() });
if (a.postcode.trim()) obj.postcode = a.postcode.trim(); if (a.postcode.trim()) components.push({ kind: "postcode", value: a.postcode.trim() });
if (a.country.trim()) obj.country = a.country.trim(); if (a.country.trim()) components.push({ kind: "country", value: a.country.trim() });
const obj: Record<string, unknown> = { components, isOrdered: true, defaultSeparator: ", " };
if (a.context) obj.contexts = { [a.context]: true }; if (a.context) obj.contexts = { [a.context]: true };
// @ts-expect-error - dynamic build // @ts-expect-error - dynamic build
addressesMap[`a${i}`] = obj; addressesMap[`a${i}`] = obj;
@@ -357,7 +412,7 @@ export function ContactForm({ contact, addressBooks, onSave, onCancel }: Contact
const anniversariesMap: Record<string, ContactAnniversary> = {}; const anniversariesMap: Record<string, ContactAnniversary> = {};
anniversaries.filter(a => a.date.trim()).forEach((a, i) => { anniversaries.filter(a => a.date.trim()).forEach((a, i) => {
anniversariesMap[`an${i}`] = { date: a.date.trim(), kind: a.kind }; anniversariesMap[`an${i}`] = { date: stringToPartialDate(a.date.trim()), kind: a.kind };
}); });
const personalInfoMap: Record<string, ContactPersonalInfo> = {}; const personalInfoMap: Record<string, ContactPersonalInfo> = {};
+17 -12
View File
@@ -27,6 +27,7 @@ import { substitutePlaceholders } from "@/lib/template-utils";
import { TemplatePicker } from "@/components/templates/template-picker"; import { TemplatePicker } from "@/components/templates/template-picker";
import { TemplateForm } from "@/components/templates/template-form"; import { TemplateForm } from "@/components/templates/template-form";
import type { EmailTemplate } from "@/lib/template-types"; import type { EmailTemplate } from "@/lib/template-types";
import { appendPlainTextSignature, getPlainTextSignature } from "@/lib/signature-utils";
export interface ComposerDraftData { export interface ComposerDraftData {
to: string; to: string;
@@ -199,6 +200,14 @@ export function EmailComposer({
const { client } = useAuthStore(); const { client } = useAuthStore();
const identities = useIdentityStore((s) => s.identities); const identities = useIdentityStore((s) => s.identities);
const primaryIdentity = identities[0] ?? null; const primaryIdentity = identities[0] ?? null;
const currentIdentity = selectedIdentityId
? identities.find((identity) => identity.id === selectedIdentityId) || primaryIdentity
: primaryIdentity;
const composerSignatureHtml = currentIdentity?.htmlSignature
? `<div>${sanitizeEmailHtml(currentIdentity.htmlSignature)}</div>`
: currentIdentity?.textSignature
? `<div>${getPlainTextSignature(currentIdentity).replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;').replace(/\n/g, '<br>')}</div>`
: '';
const getAutocomplete = useContactStore((s) => s.getAutocomplete); const getAutocomplete = useContactStore((s) => s.getAutocomplete);
const addTemplate = useTemplateStore((s) => s.addTemplate); const addTemplate = useTemplateStore((s) => s.addTemplate);
const sendRawEmail = useEmailStore((s) => s.sendRawEmail); const sendRawEmail = useEmailStore((s) => s.sendRawEmail);
@@ -527,10 +536,6 @@ export function EmailComposer({
setSaveStatus('saving'); setSaveStatus('saving');
// Get the selected identity or primary identity // Get the selected identity or primary identity
const currentIdentity = selectedIdentityId
? identities.find(id => id.id === selectedIdentityId)
: primaryIdentity;
// Generate sub-addressed email if tag is set // Generate sub-addressed email if tag is set
const fromEmail = currentIdentity?.email const fromEmail = currentIdentity?.email
? subAddressTag ? subAddressTag
@@ -649,10 +654,6 @@ export function EmailComposer({
} }
} }
const currentIdentity = selectedIdentityId
? identities.find(id => id.id === selectedIdentityId)
: primaryIdentity;
const fromEmail = currentIdentity?.email const fromEmail = currentIdentity?.email
? subAddressTag ? subAddressTag
? generateSubAddress(currentIdentity.email, subAddressTag) ? generateSubAddress(currentIdentity.email, subAddressTag)
@@ -660,10 +661,7 @@ export function EmailComposer({
: undefined; : undefined;
// Append signature from the selected identity // Append signature from the selected identity
let finalBody = body; let finalBody = appendPlainTextSignature(body, currentIdentity);
if (currentIdentity?.textSignature) {
finalBody = body + '\n\n-- \n' + currentIdentity.textSignature;
}
// Append quoted original text for the plain text part in reply/forward // Append quoted original text for the plain text part in reply/forward
if (replyTo && (mode === 'reply' || mode === 'replyAll' || mode === 'forward')) { if (replyTo && (mode === 'reply' || mode === 'replyAll' || mode === 'forward')) {
@@ -1126,6 +1124,13 @@ export function EmailComposer({
/> />
</div> </div>
{composerSignatureHtml && (
<div
className="px-4 pb-3 text-sm leading-6 text-foreground break-words [&_a]:text-primary [&_a]:underline-offset-2 [&_a:hover]:underline"
dangerouslySetInnerHTML={{ __html: `<div>-- </div>${composerSignatureHtml}` }}
/>
)}
{/* Quoted original HTML */} {/* Quoted original HTML */}
{replyTo?.htmlBody && (mode === 'reply' || mode === 'replyAll' || mode === 'forward') && ( {replyTo?.htmlBody && (mode === 'reply' || mode === 'replyAll' || mode === 'forward') && (
<div className="border-t border-border"> <div className="border-t border-border">
+437 -139
View File
@@ -5,6 +5,7 @@ import ReactDOM from "react-dom";
import DOMPurify from "dompurify"; import DOMPurify from "dompurify";
import { Email, ContactCard, Mailbox } from "@/lib/jmap/types"; import { Email, ContactCard, Mailbox } from "@/lib/jmap/types";
import { EMAIL_SANITIZE_CONFIG, collapseBlockedImageContainers } from "@/lib/email-sanitization"; import { EMAIL_SANITIZE_CONFIG, collapseBlockedImageContainers } from "@/lib/email-sanitization";
import { hasMeaningfulHtmlBody } from "@/lib/signature-utils";
import { Button } from "@/components/ui/button"; import { Button } from "@/components/ui/button";
import { Avatar } from "@/components/ui/avatar"; import { Avatar } from "@/components/ui/avatar";
import { formatFileSize, cn, buildMailboxTree, MailboxNode, formatDateTime } from "@/lib/utils"; import { formatFileSize, cn, buildMailboxTree, MailboxNode, formatDateTime } from "@/lib/utils";
@@ -104,7 +105,7 @@ interface EmailViewerProps {
onToggleStar?: () => void; onToggleStar?: () => void;
onMarkAsRead?: (emailId: string, read: boolean) => void; onMarkAsRead?: (emailId: string, read: boolean) => void;
onSetColorTag?: (emailId: string, color: string | null) => void; onSetColorTag?: (emailId: string, color: string | null) => void;
onDownloadAttachment?: (blobId: string, name: string, type?: string) => void; onDownloadAttachment?: (blobId: string, name: string, type?: string, forceDownload?: boolean) => void;
onQuickReply?: (body: string) => Promise<void>; onQuickReply?: (body: string) => Promise<void>;
onMarkAsSpam?: () => void; onMarkAsSpam?: () => void;
onUndoSpam?: () => void; onUndoSpam?: () => void;
@@ -148,6 +149,42 @@ const getFileIcon = (name?: string, type?: string) => {
return File; return File;
}; };
const MIME_TYPE_LABELS: Record<string, string> = {
'application/pdf': 'Document.pdf',
'application/zip': 'Archive.zip',
'application/x-zip-compressed': 'Archive.zip',
'application/gzip': 'Archive.gz',
'application/x-rar-compressed': 'Archive.rar',
'application/x-7z-compressed': 'Archive.7z',
'application/msword': 'Document.doc',
'application/vnd.openxmlformats-officedocument.wordprocessingml.document': 'Document.docx',
'application/vnd.ms-excel': 'Spreadsheet.xls',
'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet': 'Spreadsheet.xlsx',
'application/vnd.ms-powerpoint': 'Presentation.ppt',
'application/vnd.openxmlformats-officedocument.presentationml.presentation': 'Presentation.pptx',
'text/plain': 'Text.txt',
'text/html': 'Document.html',
'text/csv': 'Data.csv',
'application/json': 'Data.json',
'application/xml': 'Data.xml',
'application/octet-stream': 'Attachment',
'message/rfc822': 'Email.eml',
};
const getAttachmentDisplayName = (name: string | null | undefined, mimeType?: string): string => {
if (name) return name;
if (mimeType) {
const label = MIME_TYPE_LABELS[mimeType.toLowerCase()];
if (label) return label;
const sub = mimeType.split('/')[1];
if (sub) {
const clean = sub.replace(/^x-/, '').replace(/^vnd\./, '');
return `Attachment.${clean}`;
}
}
return 'Attachment';
};
const getCurrentColor = (keywords: Record<string, boolean> | undefined) => { const getCurrentColor = (keywords: Record<string, boolean> | undefined) => {
if (!keywords) return null; if (!keywords) return null;
for (const key of Object.keys(keywords)) { for (const key of Object.keys(keywords)) {
@@ -706,7 +743,11 @@ function ContactSidebarPanel({
<SidebarSection icon={MapPin} title="Addresses"> <SidebarSection icon={MapPin} title="Addresses">
{addresses.map((a, i) => ( {addresses.map((a, i) => (
<div key={i} className="text-sm text-muted-foreground"> <div key={i} className="text-sm text-muted-foreground">
{[a.street, a.locality, a.region, a.postcode, a.country].filter(Boolean).join(", ")} {a.full || a.fullAddress
? (a.full || a.fullAddress)
: a.components && a.components.length > 0
? a.components.filter(c => c.kind !== 'separator').map(c => c.value).filter(Boolean).join(", ")
: [a.street, a.locality, a.region, a.postcode, a.country].filter(Boolean).join(", ")}
</div> </div>
))} ))}
</SidebarSection> </SidebarSection>
@@ -832,6 +873,7 @@ export function EmailViewer({
const tFiles = useTranslations('files'); const tFiles = useTranslations('files');
const externalContentPolicy = useSettingsStore((state) => state.externalContentPolicy); const externalContentPolicy = useSettingsStore((state) => state.externalContentPolicy);
const mailAttachmentAction = useSettingsStore((state) => state.mailAttachmentAction); const mailAttachmentAction = useSettingsStore((state) => state.mailAttachmentAction);
const attachmentPosition = useSettingsStore((state) => state.attachmentPosition);
const addTrustedSender = useSettingsStore((state) => state.addTrustedSender); const addTrustedSender = useSettingsStore((state) => state.addTrustedSender);
const isSenderTrusted = useSettingsStore((state) => state.isSenderTrusted); const isSenderTrusted = useSettingsStore((state) => state.isSenderTrusted);
const emailKeywords = useSettingsStore((state) => state.emailKeywords); const emailKeywords = useSettingsStore((state) => state.emailKeywords);
@@ -859,6 +901,8 @@ export function EmailViewer({
const { identities, client } = useAuthStore(); const { identities, client } = useAuthStore();
const resolvedTheme = useThemeStore((state) => state.resolvedTheme); const resolvedTheme = useThemeStore((state) => state.resolvedTheme);
const [showFullHeaders, setShowFullHeaders] = useState(false); const [showFullHeaders, setShowFullHeaders] = useState(false);
const [showAllBesideAttachments, setShowAllBesideAttachments] = useState(false);
const [showAllMobileAttachments, setShowAllMobileAttachments] = useState(false);
const [allowExternalContent, setAllowExternalContent] = useState(false); const [allowExternalContent, setAllowExternalContent] = useState(false);
const [hasBlockedContent, setHasBlockedContent] = useState(false); const [hasBlockedContent, setHasBlockedContent] = useState(false);
const [cidBlobUrls, setCidBlobUrls] = useState<Record<string, string>>({}); const [cidBlobUrls, setCidBlobUrls] = useState<Record<string, string>>({});
@@ -2144,9 +2188,7 @@ export function EmailViewer({
// Server-generated HTML from text/plain emails often lacks <br> tags, collapsing newlines. // Server-generated HTML from text/plain emails often lacks <br> tags, collapsing newlines.
const hasTextBody = email.textBody?.[0]?.partId && email.bodyValues[email.textBody[0].partId]; const hasTextBody = email.textBody?.[0]?.partId && email.bodyValues[email.textBody[0].partId];
if (hasTextBody && htmlContent) { if (hasTextBody && htmlContent) {
const stripped = htmlContent.replace(/<\/?(html|head|body|meta|!doctype|!DOCTYPE|br\s*\/?)[^>]*>/gi, '').trim(); useHtmlVersion = hasMeaningfulHtmlBody(htmlContent);
const hasRichContent = /<(table|tr|td|th|img|style|link|div\s+[^>]*class|span\s+[^>]*class|font|center|blockquote|ul|ol|li|h[1-6])\b/i.test(stripped);
useHtmlVersion = hasRichContent;
} else { } else {
useHtmlVersion = !!htmlContent; useHtmlVersion = !!htmlContent;
} }
@@ -2391,6 +2433,44 @@ export function EmailViewer({
setTimeout(() => URL.revokeObjectURL(objectUrl), 60_000); setTimeout(() => URL.revokeObjectURL(objectUrl), 60_000);
}, [mailAttachmentAction, onDownloadAttachment]); }, [mailAttachmentAction, onDownloadAttachment]);
const handleEffectiveAttachmentDownload = useCallback((attachment: EffectiveAttachment) => {
if (attachment.blobId && onDownloadAttachment) {
onDownloadAttachment(attachment.blobId, attachment.name || 'download', attachment.type, true);
return;
}
if (attachment.tnefData) {
const buffer = attachment.tnefData.buffer.slice(
attachment.tnefData.byteOffset,
attachment.tnefData.byteOffset + attachment.tnefData.byteLength,
) as ArrayBuffer;
const blob = new Blob([buffer], { type: attachment.type || 'application/octet-stream' });
const objectUrl = URL.createObjectURL(blob);
const anchor = document.createElement('a');
anchor.href = objectUrl;
anchor.download = attachment.name || 'download';
document.body.appendChild(anchor);
anchor.click();
anchor.remove();
setTimeout(() => URL.revokeObjectURL(objectUrl), 60000);
return;
}
if (!attachment.decryptedAttachment) return;
const bytes = getAttachmentContentBytes(attachment.decryptedAttachment);
if (!bytes || bytes.byteLength === 0) return;
const buffer = bytes.buffer.slice(bytes.byteOffset, bytes.byteOffset + bytes.byteLength) as ArrayBuffer;
const blob = new Blob([buffer], { type: attachment.type || 'application/octet-stream' });
const objectUrl = URL.createObjectURL(blob);
const anchor = document.createElement('a');
anchor.href = objectUrl;
anchor.download = attachment.name || 'download';
document.body.appendChild(anchor);
anchor.click();
anchor.remove();
setTimeout(() => URL.revokeObjectURL(objectUrl), 60_000);
}, [onDownloadAttachment]);
// Iframe for rendering HTML emails true-to-life // Iframe for rendering HTML emails true-to-life
const iframeRef = useRef<HTMLIFrameElement>(null); const iframeRef = useRef<HTMLIFrameElement>(null);
@@ -2910,6 +2990,21 @@ export function EmailViewer({
<Code className="w-4 h-4" /> <Code className="w-4 h-4" />
</Button> </Button>
{/* Dark/light mode toggle for HTML emails */}
{effectiveEmailContent.isHtml && (
<Button
variant="ghost"
size="sm"
onClick={() => setEmailViewDarkOverride(prev => prev === null ? !(resolvedTheme === 'dark') : !prev)}
data-overflow-item
data-overflow-priority="11"
className="hidden sm:inline-flex h-8 gap-1.5"
title={isDark ? 'View in light mode' : 'View in dark mode'}
>
{isDark ? <Sun className="w-4 h-4" /> : <Moon className="w-4 h-4" />}
</Button>
)}
{/* More menu — click-based */} {/* More menu — click-based */}
<div ref={moreMenuRef} className="relative"> <div ref={moreMenuRef} className="relative">
<Button <Button
@@ -3092,6 +3187,16 @@ export function EmailViewer({
<Code className="w-4 h-4" /> <Code className="w-4 h-4" />
{t('view_source')} {t('view_source')}
</button> </button>
{/* Overflow: dark/light mode toggle */}
{effectiveEmailContent.isHtml && (
<button
onClick={() => { setEmailViewDarkOverride(prev => prev === null ? !(resolvedTheme === 'dark') : !prev); setMoreMenuOpen(false); setMoreMenuSub(null); }}
className={cn("w-full px-3 py-1.5 text-sm text-left hover:bg-muted text-foreground flex items-center gap-2", hiddenPriorities.has(11) ? "" : "sm:hidden")}
>
{isDark ? <Sun className="w-4 h-4" /> : <Moon className="w-4 h-4" />}
{isDark ? 'View in light mode' : 'View in dark mode'}
</button>
)}
<div className="h-px bg-border my-1" /> <div className="h-px bg-border my-1" />
{/* Export email */} {/* Export email */}
<button <button
@@ -3265,6 +3370,15 @@ export function EmailViewer({
<Code className="w-5 h-5" /> <Code className="w-5 h-5" />
{t('view_source')} {t('view_source')}
</button> </button>
{effectiveEmailContent.isHtml && (
<button
onClick={() => { setEmailViewDarkOverride(prev => prev === null ? !(resolvedTheme === 'dark') : !prev); setMoreMenuOpen(false); }}
className="w-full px-4 py-3 min-h-[44px] text-sm text-left hover:bg-muted text-foreground flex items-center gap-3"
>
{isDark ? <Sun className="w-5 h-5" /> : <Moon className="w-5 h-5" />}
{isDark ? 'View in light mode' : 'View in dark mode'}
</button>
)}
<div className="h-px bg-border my-1" /> <div className="h-px bg-border my-1" />
<button <button
onClick={() => { handleExportEmail(); setMoreMenuOpen(false); }} onClick={() => { handleExportEmail(); setMoreMenuOpen(false); }}
@@ -3338,7 +3452,7 @@ export function EmailViewer({
)} )}
<div className="flex-1 min-w-0"> <div className="flex-1 min-w-0">
<div className="flex items-start gap-2"> <div className="flex items-start gap-2">
<h1 className="text-lg lg:text-2xl font-bold text-foreground tracking-tight break-words min-w-0"> <h1 className="text-lg lg:text-xl font-bold text-foreground tracking-tight break-words min-w-0">
{email.subject || t('no_subject')} {email.subject || t('no_subject')}
</h1> </h1>
{/* Star inline with subject (top toolbar mode) */} {/* Star inline with subject (top toolbar mode) */}
@@ -3362,19 +3476,24 @@ export function EmailViewer({
<span className={cn("w-2.5 h-2.5 rounded-full flex-shrink-0", dotClass)} title={kw!.label} /> <span className={cn("w-2.5 h-2.5 rounded-full flex-shrink-0", dotClass)} title={kw!.label} />
) : null; ) : null;
})()} })()}
</div>
<div className="flex items-center gap-2 lg:gap-3 mt-1 lg:mt-1.5 text-xs lg:text-sm text-muted-foreground">
<span className="flex items-center gap-1 lg:gap-1.5 whitespace-nowrap">
<Clock className="w-3.5 h-3.5 lg:w-4 lg:h-4" />
{formatDateTime(email.receivedAt, timeFormat, { weekday: 'short', year: 'numeric', month: 'short', day: 'numeric' })}
</span>
{isImportant && ( {isImportant && (
<span className="px-1.5 lg:px-2 py-0.5 bg-amber-50 dark:bg-amber-900/30 text-amber-700 dark:text-amber-400 rounded-full text-xs font-medium whitespace-nowrap"> <span className="px-1.5 lg:px-2 py-0.5 bg-amber-50 dark:bg-amber-900/30 text-amber-700 dark:text-amber-400 rounded-full text-xs font-medium whitespace-nowrap flex-shrink-0 self-center">
{t('important')} {t('important')}
</span> </span>
)} )}
</div> </div>
</div> </div>
{/* Date/time on the right of subject row */}
<div className="flex-shrink-0 text-right">
<span className="text-xs lg:text-sm text-muted-foreground whitespace-nowrap">
{formatDateTime(email.receivedAt, timeFormat, { weekday: 'short', year: 'numeric', month: 'short', day: 'numeric' })}
</span>
{email.size > 0 && (
<div className="text-xs text-muted-foreground/60">
{formatFileSize(email.size)}
</div>
)}
</div>
</div> </div>
</div> </div>
</div> </div>
@@ -3405,13 +3524,14 @@ export function EmailViewer({
name={sender?.name} name={sender?.name}
email={sender?.email} email={sender?.email}
size="lg" size="lg"
className="shadow-sm w-12 h-12 group-hover:ring-2 group-hover:ring-primary/30 transition-all" className="shadow-sm w-10 h-10 group-hover:ring-2 group-hover:ring-primary/30 transition-all"
/> />
</button> </button>
<div className="flex-1 min-w-0"> <div className="flex-1 min-w-0 flex gap-4">
{/* Sender line with email and badges */} <div className="flex-1 min-w-0">
<div className="flex items-start justify-between gap-4"> {/* Row 1: Sender name + badges */}
<div>
<div className="min-w-0"> <div className="min-w-0">
<div className="flex items-center gap-2 flex-wrap"> <div className="flex items-center gap-2 flex-wrap">
<button <button
@@ -3422,103 +3542,80 @@ export function EmailViewer({
{sender?.name || sender?.email || t('unknown_sender')} {sender?.name || sender?.email || t('unknown_sender')}
</button> </button>
<EmailIdentityBadge email={email} identities={identities} /> <EmailIdentityBadge email={email} identities={identities} />
{shouldShowUnsubBanner && listHeaders?.listUnsubscribe && (
<UnsubscribeBanner
listUnsubscribe={listHeaders.listUnsubscribe}
senderEmail={email?.from?.[0]?.email || ''}
onDismiss={() => {
const messageId = email?.messageId || '';
const newSet = new Set(dismissedUnsubBanners).add(messageId);
setDismissedUnsubBanners(newSet);
localStorage.setItem('dismissed-unsub-banners', JSON.stringify([...newSet]));
}}
/>
)}
</div> </div>
{sender?.email && ( {/* Email address under name */}
<div className="text-sm text-muted-foreground mt-0.5 flex items-center min-w-0"> {sender?.email && sender?.name && (
<span className="truncate">{sender.email}</span> <div className="text-sm text-muted-foreground mt-0.5 truncate">{sender.email}</div>
{shouldShowUnsubBanner && listHeaders?.listUnsubscribe && (
<UnsubscribeBanner
listUnsubscribe={listHeaders.listUnsubscribe}
senderEmail={email?.from?.[0]?.email || ''}
onDismiss={() => {
const messageId = email?.messageId || '';
const newSet = new Set(dismissedUnsubBanners).add(messageId);
setDismissedUnsubBanners(newSet);
localStorage.setItem('dismissed-unsub-banners', JSON.stringify([...newSet]));
}}
/>
)}
</div>
)}
</div>
{/* Date and size on the right */}
<div className="text-right flex-shrink-0">
<div className="text-sm text-muted-foreground whitespace-nowrap">
{formatDateTime(email.receivedAt, timeFormat, { weekday: 'short', year: 'numeric', month: 'short', day: 'numeric' })}
</div>
{email.size > 0 && (
<div className="text-xs text-muted-foreground/70 mt-0.5">
{formatFileSize(email.size)}
</div>
)}
{effectiveEmailContent.isHtml && (
<button
onClick={() => setEmailViewDarkOverride(prev => prev === null ? !(resolvedTheme === 'dark') : !prev)}
className="inline-flex items-center rounded-full p-1 mt-1 text-muted-foreground/70 hover:text-foreground transition-colors hover:bg-muted"
title={isDark ? 'View in light mode' : 'View in dark mode'}
>
{isDark ? <Sun className="w-4 h-4" /> : <Moon className="w-4 h-4" />}
</button>
)} )}
</div> </div>
</div> </div>
{/* Recipient section - separate line */} {/* Row 2: Recipients + Show details */}
<div className="mt-2 space-y-1"> <div className="mt-1 flex items-center gap-2 text-sm text-muted-foreground flex-wrap">
{email.to && email.to.length > 0 && ( {email.to && email.to.length > 0 && (
<div className="flex flex-wrap items-center gap-1 text-sm"> <>
<span className="text-muted-foreground">{t('recipient_to_prefix')}</span> <span>{t('recipient_to_prefix')}</span>
{renderClickableRecipients(email.to, currentUserEmail, t, handleViewContactSidebar)} {renderClickableRecipients(email.to, currentUserEmail, t, handleViewContactSidebar)}
{email.to.length > 2 && ( {email.to.length > 2 && (
<button <button
onClick={() => setShowFullHeaders(!showFullHeaders)} onClick={() => setShowFullHeaders(!showFullHeaders)}
className="ml-1 text-blue-600 dark:text-blue-400 hover:underline text-sm" className="text-blue-600 dark:text-blue-400 hover:underline text-sm"
> >
{t('more_count', { count: email.to.length - 2 })} {t('more_count', { count: email.to.length - 2 })}
</button> </button>
)} )}
</div> </>
)} )}
{email.cc && email.cc.length > 0 && ( {email.cc && email.cc.length > 0 && (
<div className="flex flex-wrap items-center gap-1 text-sm"> <>
<span className="text-muted-foreground">CC:</span> <span className="text-muted-foreground/50">|</span>
<span>CC:</span>
{renderClickableRecipients(email.cc, currentUserEmail, t, handleViewContactSidebar)} {renderClickableRecipients(email.cc, currentUserEmail, t, handleViewContactSidebar)}
{email.cc.length > 2 && ( {email.cc.length > 2 && (
<span className="text-muted-foreground text-sm">+{email.cc.length - 2}</span> <span className="text-muted-foreground">+{email.cc.length - 2}</span>
)} )}
</div> </>
)} )}
{email.bcc && email.bcc.length > 0 && ( {email.bcc && email.bcc.length > 0 && (
<div className="flex flex-wrap items-center gap-1 text-sm"> <>
<span className="text-muted-foreground">{t('bcc')}:</span> <span className="text-muted-foreground/50">|</span>
<span>{t('bcc')}:</span>
{renderClickableRecipients(email.bcc, currentUserEmail, t, handleViewContactSidebar)} {renderClickableRecipients(email.bcc, currentUserEmail, t, handleViewContactSidebar)}
{email.bcc.length > 2 && ( {email.bcc.length > 2 && (
<span className="text-muted-foreground text-sm">+{email.bcc.length - 2}</span> <span className="text-muted-foreground">+{email.bcc.length - 2}</span>
)} )}
</div> </>
)} )}
<button
onClick={() => setShowFullHeaders(!showFullHeaders)}
className="text-xs text-muted-foreground hover:text-foreground flex items-center gap-0.5 transition-colors ml-1"
>
{showFullHeaders ? (
<>
<ChevronUp className="w-3 h-3" />
{t('hide_details')}
</>
) : (
<>
<ChevronDown className="w-3 h-3" />
{t('show_details')}
</>
)}
</button>
</div> </div>
{/* Details toggle - stays in place when expanded */}
<button
onClick={() => setShowFullHeaders(!showFullHeaders)}
className="mt-3 text-xs text-muted-foreground hover:text-foreground flex items-center gap-1 transition-colors"
>
{showFullHeaders ? (
<>
<ChevronUp className="w-3 h-3" />
{t('hide_details')}
</>
) : (
<>
<ChevronDown className="w-3 h-3" />
{t('show_details')}
</>
)}
</button>
{/* Expandable Details */} {/* Expandable Details */}
{showFullHeaders && ( {showFullHeaders && (
<div className="mt-3 space-y-3"> <div className="mt-3 space-y-3">
@@ -3890,46 +3987,248 @@ export function EmailViewer({
</div> </div>
)} )}
</div>
{/* Attachments on the right (beside-sender mode) */}
{attachmentPosition === 'beside-sender' && effectiveAttachments.length > 0 && (
<div className="relative flex flex-col items-end justify-start gap-1 flex-shrink-0 max-w-[50%]">
{effectiveAttachments.slice(0, 2).map((attachment) => {
const FileIcon = getFileIcon(attachment.name || undefined, attachment.type);
const isPreviewable = isFilePreviewable(attachment.name || undefined, attachment.type);
const opensPreview = isPreviewable && mailAttachmentAction === 'preview';
return (
<div
key={attachment.id}
className="inline-flex items-center gap-1.5 px-2 py-1 bg-muted/60 rounded-md border border-border/50 group relative cursor-default"
>
<FileIcon className="w-3.5 h-3.5 text-muted-foreground flex-shrink-0" />
<span className="text-xs text-foreground truncate max-w-[140px]">
{getAttachmentDisplayName(attachment.name, attachment.type)}
</span>
<span className="text-[10px] text-muted-foreground">
{formatFileSize(attachment.size)}
</span>
<div className="absolute inset-y-0 right-0 rounded-r-md bg-background/95 opacity-0 group-hover:opacity-100 transition-opacity flex items-center justify-center gap-1 px-1.5">
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={t('download')}
onClick={() => handleEffectiveAttachmentDownload(attachment)}
>
<Download className="w-3.5 h-3.5 text-foreground" />
</button>
{opensPreview && (
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={tFiles('preview')}
onClick={() => handleEffectiveAttachmentOpen(attachment)}
>
<Eye className="w-3.5 h-3.5 text-foreground" />
</button>
)}
</div>
</div>
);
})}
{effectiveAttachments.length > 2 && (
<button
onClick={() => setShowAllBesideAttachments(!showAllBesideAttachments)}
className="text-xs text-muted-foreground hover:text-foreground transition-colors px-2 py-0.5"
>
+{effectiveAttachments.length - 2} {t('more')}
</button>
)}
{/* Floating popup for remaining attachments */}
{showAllBesideAttachments && effectiveAttachments.length > 2 && (
<>
<div className="fixed inset-0 z-40" onClick={() => setShowAllBesideAttachments(false)} />
<div className="absolute top-full right-0 mt-1 z-50 bg-background border border-border rounded-lg shadow-lg p-2 flex flex-col gap-1 min-w-[220px]">
{effectiveAttachments.slice(2).map((attachment) => {
const FileIcon = getFileIcon(attachment.name || undefined, attachment.type);
const isPreviewable = isFilePreviewable(attachment.name || undefined, attachment.type);
const opensPreview = isPreviewable && mailAttachmentAction === 'preview';
return (
<div
key={attachment.id}
className="flex items-center gap-1.5 px-2 py-1 rounded-md group relative cursor-default w-full"
>
<FileIcon className="w-3.5 h-3.5 text-muted-foreground flex-shrink-0" />
<span className="text-xs text-foreground truncate max-w-[180px]">
{getAttachmentDisplayName(attachment.name, attachment.type)}
</span>
<span className="text-[10px] text-muted-foreground ml-auto flex-shrink-0">
{formatFileSize(attachment.size)}
</span>
<div className="absolute inset-y-0 right-0 rounded-r-md bg-background/95 opacity-0 group-hover:opacity-100 transition-opacity flex items-center justify-center gap-1 px-1.5">
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={t('download')}
onClick={() => { handleEffectiveAttachmentDownload(attachment); setShowAllBesideAttachments(false); }}
>
<Download className="w-3.5 h-3.5 text-foreground" />
</button>
{opensPreview && (
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={tFiles('preview')}
onClick={() => { handleEffectiveAttachmentOpen(attachment); setShowAllBesideAttachments(false); }}
>
<Eye className="w-3.5 h-3.5 text-foreground" />
</button>
)}
</div>
</div>
);
})}
</div>
</>
)}
</div>
)}
</div> </div>
</div> </div>
</div> </div>
{/* === ATTACHMENTS (integrated into header) === */} {/* === ATTACHMENTS below header (below-header mode, desktop only) === */}
{effectiveAttachments.length > 0 && ( {attachmentPosition === 'below-header' && effectiveAttachments.length > 0 && (
<div className="bg-background border-b border-border px-4 lg:px-6 py-3"> <div className="hidden lg:block bg-background border-b border-border px-4 lg:px-6 py-2">
<div className="flex items-start gap-2 flex-wrap"> <div className="flex items-center gap-2 flex-wrap">
{effectiveAttachments.map((attachment) => { {effectiveAttachments.map((attachment) => {
const FileIcon = getFileIcon(attachment.name || undefined, attachment.type); const FileIcon = getFileIcon(attachment.name || undefined, attachment.type);
const isPreviewable = isFilePreviewable(attachment.name || undefined, attachment.type); const isPreviewable = isFilePreviewable(attachment.name || undefined, attachment.type);
const opensPreview = isPreviewable && mailAttachmentAction === 'preview'; const opensPreview = isPreviewable && mailAttachmentAction === 'preview';
return ( return (
<button <div
key={attachment.id} key={attachment.id}
className="inline-flex items-center gap-2 px-3 py-2 bg-muted/60 hover:bg-accent rounded-lg transition-colors group border border-border/50" className="inline-flex items-center gap-1.5 px-2.5 py-1.5 bg-muted/60 rounded-md border border-border/50 group relative cursor-default"
title={`${opensPreview ? tFiles('preview') : t('download')} ${attachment.name || 'Unnamed'} (${formatFileSize(attachment.size)})`}
onClick={() => handleEffectiveAttachmentOpen(attachment)}
> >
<FileIcon className="w-4 h-4 text-muted-foreground flex-shrink-0" /> <FileIcon className="w-4 h-4 text-muted-foreground flex-shrink-0" />
<div className="flex flex-col items-start min-w-0"> <span className="text-sm text-foreground truncate max-w-[200px]">
<span className="text-sm text-foreground truncate max-w-[200px]"> {getAttachmentDisplayName(attachment.name, attachment.type)}
{attachment.name || "Unnamed"} </span>
</span> <span className="text-xs text-muted-foreground">
<span className="text-xs text-muted-foreground"> {formatFileSize(attachment.size)}
{formatFileSize(attachment.size)} </span>
</span> <div className="absolute inset-y-0 right-0 rounded-r-md bg-background/95 opacity-0 group-hover:opacity-100 transition-opacity flex items-center justify-center gap-1 px-1.5">
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={t('download')}
onClick={() => handleEffectiveAttachmentDownload(attachment)}
>
<Download className="w-4 h-4 text-foreground" />
</button>
{opensPreview && (
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={tFiles('preview')}
onClick={() => handleEffectiveAttachmentOpen(attachment)}
>
<Eye className="w-4 h-4 text-foreground" />
</button>
)}
</div> </div>
{opensPreview ? ( </div>
<Eye className="w-3.5 h-3.5 text-muted-foreground opacity-0 group-hover:opacity-100 transition-opacity flex-shrink-0" />
) : (
<Download className="w-3.5 h-3.5 text-muted-foreground opacity-0 group-hover:opacity-100 transition-opacity flex-shrink-0" />
)}
</button>
); );
})} })}
</div> </div>
</div> </div>
)} )}
{/* Mobile/Tablet Attachments */}
{effectiveAttachments.length > 0 && (
<div className="lg:hidden bg-background border-b border-border px-4 py-2">
<div className="relative flex items-center gap-1.5 flex-wrap">
{effectiveAttachments.slice(0, 2).map((attachment) => {
const FileIcon = getFileIcon(attachment.name || undefined, attachment.type);
const isPreviewable = isFilePreviewable(attachment.name || undefined, attachment.type);
const opensPreview = isPreviewable && mailAttachmentAction === 'preview';
return (
<div
key={attachment.id}
className="inline-flex items-center gap-1.5 px-2.5 py-1.5 bg-muted/60 rounded-md border border-border/50 group relative cursor-default"
>
<FileIcon className="w-4 h-4 text-muted-foreground flex-shrink-0" />
<span className="text-sm text-foreground truncate max-w-[200px]">
{getAttachmentDisplayName(attachment.name, attachment.type)}
</span>
<span className="text-xs text-muted-foreground">
{formatFileSize(attachment.size)}
</span>
<div className="absolute inset-y-0 right-0 rounded-r-md bg-background/95 opacity-0 group-hover:opacity-100 transition-opacity flex items-center justify-center gap-1 px-1.5">
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={t('download')}
onClick={() => handleEffectiveAttachmentDownload(attachment)}
>
<Download className="w-4 h-4 text-foreground" />
</button>
{opensPreview && (
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={tFiles('preview')}
onClick={() => handleEffectiveAttachmentOpen(attachment)}
>
<Eye className="w-4 h-4 text-foreground" />
</button>
)}
</div>
</div>
);
})}
{effectiveAttachments.length > 2 && (
<button
onClick={() => setShowAllMobileAttachments(!showAllMobileAttachments)}
className="text-xs text-muted-foreground hover:text-foreground transition-colors px-2 py-0.5"
>
+{effectiveAttachments.length - 2} {t('more')}
</button>
)}
{showAllMobileAttachments && effectiveAttachments.length > 2 && (
<>
<div className="fixed inset-0 z-40" onClick={() => setShowAllMobileAttachments(false)} />
<div className="absolute top-full left-0 mt-1 z-50 bg-background border border-border rounded-lg shadow-lg p-2 flex flex-col gap-1 min-w-[220px]">
{effectiveAttachments.slice(2).map((attachment) => {
const FileIcon = getFileIcon(attachment.name || undefined, attachment.type);
const isPreviewable = isFilePreviewable(attachment.name || undefined, attachment.type);
const opensPreview = isPreviewable && mailAttachmentAction === 'preview';
return (
<div
key={attachment.id}
className="flex items-center gap-1.5 px-2 py-1 rounded-md group relative cursor-default w-full"
>
<FileIcon className="w-3.5 h-3.5 text-muted-foreground flex-shrink-0" />
<span className="text-xs text-foreground truncate max-w-[180px]">
{getAttachmentDisplayName(attachment.name, attachment.type)}
</span>
<span className="text-[10px] text-muted-foreground ml-auto flex-shrink-0">
{formatFileSize(attachment.size)}
</span>
<div className="absolute inset-y-0 right-0 rounded-r-md bg-background/95 opacity-0 group-hover:opacity-100 transition-opacity flex items-center justify-center gap-1 px-1.5">
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={t('download')}
onClick={() => { handleEffectiveAttachmentDownload(attachment); setShowAllMobileAttachments(false); }}
>
<Download className="w-3.5 h-3.5 text-foreground" />
</button>
{opensPreview && (
<button
className="p-1 hover:bg-accent rounded transition-colors"
title={tFiles('preview')}
onClick={() => { handleEffectiveAttachmentOpen(attachment); setShowAllMobileAttachments(false); }}
>
<Eye className="w-3.5 h-3.5 text-foreground" />
</button>
)}
</div>
</div>
);
})}
</div>
</>
)}
</div>
</div>
)}
{/* Mobile/Tablet Sender Info - scrolls with content */} {/* Mobile/Tablet Sender Info - scrolls with content */}
<div className="lg:hidden bg-background border-b border-border px-4" style={{ paddingBlock: 'var(--density-header-py)' }}> <div className="lg:hidden bg-background border-b border-border px-4" style={{ paddingBlock: 'var(--density-header-py)' }}>
<div className="flex items-start" style={{ gap: 'var(--density-item-gap)' }}> <div className="flex items-start" style={{ gap: 'var(--density-item-gap)' }}>
@@ -3946,8 +4245,8 @@ export function EmailViewer({
/> />
</button> </button>
<div className="flex-1 min-w-0"> <div className="flex-1 min-w-0">
{/* Mobile 2-line layout */} {/* Row 1: Sender name + badges */}
<div className="flex items-center gap-2 flex-wrap"> <div className="flex items-center gap-1.5 flex-wrap">
<button <button
onClick={() => sender?.email && handleViewContactSidebar(null, sender.email)} onClick={() => sender?.email && handleViewContactSidebar(null, sender.email)}
className="text-sm font-semibold text-foreground hover:text-primary hover:underline transition-colors cursor-pointer text-left" className="text-sm font-semibold text-foreground hover:text-primary hover:underline transition-colors cursor-pointer text-left"
@@ -3955,43 +4254,42 @@ export function EmailViewer({
{sender?.name || sender?.email || t('unknown_sender')} {sender?.name || sender?.email || t('unknown_sender')}
</button> </button>
<EmailIdentityBadge email={email} identities={identities} /> <EmailIdentityBadge email={email} identities={identities} />
</div> {shouldShowUnsubBanner && listHeaders?.listUnsubscribe && (
<div className="mt-1 flex items-center gap-1 text-sm text-muted-foreground flex-wrap"> <UnsubscribeBanner
{sender?.email && sender?.name && ( listUnsubscribe={listHeaders.listUnsubscribe}
<> senderEmail={email?.from?.[0]?.email || ''}
<span className="truncate">{sender.email}</span> onDismiss={() => {
{shouldShowUnsubBanner && listHeaders?.listUnsubscribe && ( const messageId = email?.messageId || '';
<UnsubscribeBanner const newSet = new Set(dismissedUnsubBanners).add(messageId);
listUnsubscribe={listHeaders.listUnsubscribe} setDismissedUnsubBanners(newSet);
senderEmail={email?.from?.[0]?.email || ''} localStorage.setItem('dismissed-unsub-banners', JSON.stringify([...newSet]));
onDismiss={() => { }}
const messageId = email?.messageId || ''; />
const newSet = new Set(dismissedUnsubBanners).add(messageId);
setDismissedUnsubBanners(newSet);
localStorage.setItem('dismissed-unsub-banners', JSON.stringify([...newSet]));
}}
/>
)}
<span>·</span>
</>
)} )}
</div>
{/* Email address under name */}
{sender?.email && sender?.name && (
<div className="text-xs text-muted-foreground mt-0.5 truncate">{sender.email}</div>
)}
{/* Row 2: Recipients */}
<div className="mt-0.5 flex items-center gap-1 text-sm text-muted-foreground flex-wrap">
{email.to && email.to.length > 0 && ( {email.to && email.to.length > 0 && (
<> <>
<span> {t('recipient_to_prefix')}</span> <span> {t('recipient_to_prefix')}</span>
{renderClickableRecipients(email.to, currentUserEmail, t, handleViewContactSidebar)} {renderClickableRecipients(email.to, currentUserEmail, t, handleViewContactSidebar)}
</> </>
)} )}
{email.cc && email.cc.length > 0 && (
<>
<span className="text-muted-foreground/50">|</span>
<span>CC:</span>
{renderClickableRecipients(email.cc, currentUserEmail, t, handleViewContactSidebar)}
{email.cc.length > 2 && (
<span>+{email.cc.length - 2}</span>
)}
</>
)}
</div> </div>
{/* CC line (mobile - only if present) */}
{email.cc && email.cc.length > 0 && (
<div className="mt-1 flex items-center gap-1 text-sm">
<span className="text-muted-foreground">CC:</span>
{renderClickableRecipients(email.cc, currentUserEmail, t, handleViewContactSidebar)}
{email.cc.length > 2 && (
<span className="text-muted-foreground">+{email.cc.length - 2}</span>
)}
</div>
)}
</div> </div>
</div> </div>
</div> </div>
@@ -4,6 +4,7 @@ import { useState, useEffect, useMemo } from "react";
import DOMPurify from "dompurify"; import DOMPurify from "dompurify";
import { Email, ThreadGroup } from "@/lib/jmap/types"; import { Email, ThreadGroup } from "@/lib/jmap/types";
import { EMAIL_SANITIZE_CONFIG, collapseBlockedImageContainers } from "@/lib/email-sanitization"; import { EMAIL_SANITIZE_CONFIG, collapseBlockedImageContainers } from "@/lib/email-sanitization";
import { hasMeaningfulHtmlBody } from "@/lib/signature-utils";
import { transformInlineStyles, transformColorForDarkMode, transformBgColorForDarkMode } from "@/lib/color-transform"; import { transformInlineStyles, transformColorForDarkMode, transformBgColorForDarkMode } from "@/lib/color-transform";
import { useThemeStore } from "@/stores/theme-store"; import { useThemeStore } from "@/stores/theme-store";
import { Avatar } from "@/components/ui/avatar"; import { Avatar } from "@/components/ui/avatar";
@@ -320,9 +321,7 @@ function EmailCard({
// Server-generated HTML from text/plain emails often lacks <br> tags, collapsing newlines. // Server-generated HTML from text/plain emails often lacks <br> tags, collapsing newlines.
const hasTextBody = email.textBody?.[0]?.partId && email.bodyValues[email.textBody[0].partId]; const hasTextBody = email.textBody?.[0]?.partId && email.bodyValues[email.textBody[0].partId];
if (hasTextBody && htmlContent) { if (hasTextBody && htmlContent) {
const stripped = htmlContent.replace(/<\/?(html|head|body|meta|!doctype|!DOCTYPE|br\s*\/?)[^>]*>/gi, '').trim(); useHtmlVersion = hasMeaningfulHtmlBody(htmlContent);
const hasRichContent = /<(table|tr|td|th|img|style|link|div\s+[^>]*class|span\s+[^>]*class|font|center|blockquote|ul|ol|li|h[1-6])\b/i.test(stripped);
useHtmlVersion = hasRichContent;
} else { } else {
useHtmlVersion = !!htmlContent; useHtmlVersion = !!htmlContent;
} }
+23 -12
View File
@@ -217,8 +217,8 @@ export function NavigationRail({
); );
})} })}
{/* Custom sidebar apps */} {/* Custom sidebar apps (per-app mobile visibility) */}
{sidebarApps.map((app) => { {sidebarApps.filter((app) => app.showOnMobile).map((app) => {
const AppIcon = lucideIcons[app.icon as keyof typeof lucideIcons] as LucideIcon | undefined; const AppIcon = lucideIcons[app.icon as keyof typeof lucideIcons] as LucideIcon | undefined;
const isActive = activeAppId === app.id; const isActive = activeAppId === app.id;
return ( return (
@@ -252,16 +252,27 @@ export function NavigationRail({
); );
})} })}
{/* Manage apps button */} {/* Settings */}
{onManageApps && ( <Link
<button href="/settings"
onClick={onManageApps} onClick={activeAppId ? () => onCloseInlineApp?.() : undefined}
className="flex flex-col items-center justify-center gap-1 py-2 px-3 min-w-[64px] min-h-[44px] transition-colors duration-150 text-muted-foreground hover:text-foreground" className={cn(
> "flex flex-col items-center justify-center gap-1 py-2 px-3 min-w-[64px] min-h-[44px]",
<Plus className="w-5 h-5" /> "transition-colors duration-150",
<span className="text-[10px] font-medium leading-tight">{t("add_app")}</span> isSettingsActive
</button> ? "text-primary"
)} : "text-muted-foreground hover:text-foreground"
)}
aria-current={isSettingsActive ? "page" : undefined}
>
<div className="relative">
<Settings className="w-5 h-5" />
{isSettingsActive && (
<span className="absolute -bottom-1 left-1/2 -translate-x-1/2 w-4 h-0.5 rounded-full bg-primary" />
)}
</div>
<span className="text-[10px] font-medium leading-tight">{t("settings")}</span>
</Link>
</nav> </nav>
); );
} }
@@ -153,6 +153,9 @@ export function CalendarManagementSettings() {
const { client, serverUrl, username } = useAuthStore(); const { client, serverUrl, username } = useAuthStore();
const { calendars, updateCalendar, createCalendar, removeCalendar, clearCalendarEvents, fetchCalendars, icalSubscriptions, removeICalSubscription, refreshICalSubscription, isSubscriptionCalendar } = useCalendarStore(); const { calendars, updateCalendar, createCalendar, removeCalendar, clearCalendarEvents, fetchCalendars, icalSubscriptions, removeICalSubscription, refreshICalSubscription, isSubscriptionCalendar } = useCalendarStore();
const [discoveredCalDavUrls, setDiscoveredCalDavUrls] = useState<Record<string, string | null>>({});
const [wellKnownCalDavUrl, setWellKnownCalDavUrl] = useState<string | null>(null);
const [isCreating, setIsCreating] = useState(false); const [isCreating, setIsCreating] = useState(false);
const [editingId, setEditingId] = useState<string | null>(null); const [editingId, setEditingId] = useState<string | null>(null);
const [deletingId, setDeletingId] = useState<string | null>(null); const [deletingId, setDeletingId] = useState<string | null>(null);
@@ -175,6 +178,60 @@ export function CalendarManagementSettings() {
} }
}, [client, calendars.length, fetchCalendars]); }, [client, calendars.length, fetchCalendars]);
useEffect(() => {
if (!client || !serverUrl || !username) {
setDiscoveredCalDavUrls({});
setWellKnownCalDavUrl(null);
return;
}
const primaryKey = username;
const accounts = new Map<string, string[]>();
accounts.set(primaryKey, [username]);
for (const calendar of calendars) {
if (!calendar.isShared) continue;
const key = calendar.accountId || calendar.accountName || calendar.id;
const candidates = accounts.get(key) || [];
if (calendar.accountId) candidates.push(calendar.accountId);
if (calendar.accountName) candidates.push(calendar.accountName);
accounts.set(key, candidates);
}
const controller = new AbortController();
fetch('/api/caldav/discover', {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({
accounts: Array.from(accounts.entries()).map(([key, candidates]) => ({ key, candidates })),
}),
signal: controller.signal,
})
.then(async (response) => {
if (!response.ok) throw new Error(`CalDAV discovery failed: ${response.status}`);
return response.json() as Promise<{
wellKnownUrl?: string;
accounts?: Record<string, { url: string | null }>;
}>;
})
.then((payload) => {
setWellKnownCalDavUrl(payload.wellKnownUrl || null);
const next: Record<string, string | null> = {};
for (const [key, value] of Object.entries(payload.accounts || {})) {
next[key] = value?.url || null;
}
setDiscoveredCalDavUrls(next);
})
.catch(() => {
const fallbackWellKnown = new URL('/.well-known/caldav', serverUrl).toString();
setDiscoveredCalDavUrls({});
setWellKnownCalDavUrl(fallbackWellKnown);
});
return () => controller.abort();
}, [client, calendars, serverUrl, username]);
const handleRefreshSubscription = async (subId: string) => { const handleRefreshSubscription = async (subId: string) => {
if (!client) return; if (!client) return;
setRefreshingSubId(subId); setRefreshingSubId(subId);
@@ -295,8 +352,10 @@ export function CalendarManagementSettings() {
const buildCalDavUrl = (calendarId: string) => { const buildCalDavUrl = (calendarId: string) => {
if (!serverUrl || !username) return null; if (!serverUrl || !username) return null;
const base = serverUrl.replace(/\/$/, ''); const calendar = calendars.find((entry) => entry.id === calendarId);
return `${base}/dav/cal/${encodeURIComponent(username)}/${encodeURIComponent(calendarId)}/`; if (!calendar) return null;
const accountKey = calendar.isShared ? (calendar.accountId || calendar.accountName || calendar.id) : username;
return discoveredCalDavUrls[accountKey] || wellKnownCalDavUrl;
}; };
const handleCopyUrl = async (url: string) => { const handleCopyUrl = async (url: string) => {
+11
View File
@@ -14,6 +14,7 @@ export function CalendarSettings() {
const { const {
timeFormat, timeFormat,
firstDayOfWeek, firstDayOfWeek,
showTimeInMonthView,
calendarNotificationsEnabled, calendarNotificationsEnabled,
calendarNotificationSound, calendarNotificationSound,
calendarInvitationParsingEnabled, calendarInvitationParsingEnabled,
@@ -57,6 +58,16 @@ export function CalendarSettings() {
/> />
</SettingItem> </SettingItem>
<SettingItem
label={t('show_time_in_month_view')}
description={t('show_time_in_month_view_desc')}
>
<ToggleSwitch
checked={showTimeInMonthView}
onChange={(checked) => updateSetting('showTimeInMonthView', checked)}
/>
</SettingItem>
<SettingItem <SettingItem
label={t('notifications_enabled')} label={t('notifications_enabled')}
description={t('notifications_enabled_desc')} description={t('notifications_enabled_desc')}
+12
View File
@@ -24,6 +24,7 @@ export function EmailSettings() {
emailsPerPage, emailsPerPage,
externalContentPolicy, externalContentPolicy,
mailAttachmentAction, mailAttachmentAction,
attachmentPosition,
emailAlwaysLightMode, emailAlwaysLightMode,
archiveMode, archiveMode,
trustedSenders, trustedSenders,
@@ -195,6 +196,17 @@ export function EmailSettings() {
/> />
</SettingItem> </SettingItem>
<SettingItem label={t('attachment_position.label')} description={t('attachment_position.description')}>
<Select
value={attachmentPosition}
onChange={(value) => updateSetting('attachmentPosition', value as 'beside-sender' | 'below-header')}
options={[
{ value: 'beside-sender', label: t('attachment_position.beside-sender') },
{ value: 'below-header', label: t('attachment_position.below-header') },
]}
/>
</SettingItem>
{/* Emails Per Page */} {/* Emails Per Page */}
<SettingItem label={t('emails_per_page.label')} description={t('emails_per_page.description')}> <SettingItem label={t('emails_per_page.label')} description={t('emails_per_page.description')}>
<Select <Select
+28 -15
View File
@@ -490,21 +490,34 @@ export function FolderSettings() {
{/* Standard Folder Roles — advanced section */} {/* Standard Folder Roles — advanced section */}
<SettingsSection title={t('standard_roles')} description={t('standard_roles_description')}> <SettingsSection title={t('standard_roles')} description={t('standard_roles_description')}>
{STANDARD_ROLES.map((role) => ( {STANDARD_ROLES.map((role) => {
<SettingItem key={role} label={t(`role_${role}`)}> // Disambiguate duplicate folder names by appending parent path
<Select const nameCounts = new Map<string, number>();
value={getRoleMailboxId(role)} ownMailboxes.forEach(mb => nameCounts.set(mb.name, (nameCounts.get(mb.name) || 0) + 1));
onChange={(value) => handleRoleChange(role, value)} const getParentPath = (mb: { parentId?: string; name: string }) => {
options={[ if (!mb.parentId) return '';
{ value: '', label: t('role_none') }, const parent = ownMailboxes.find(p => p.id === mb.parentId);
...ownMailboxes.map(mb => ({ return parent ? `${parent.name}/` : '';
value: mb.id, };
label: mb.name,
})), return (
]} <SettingItem key={role} label={t(`role_${role}`)}>
/> <Select
</SettingItem> value={getRoleMailboxId(role)}
))} onChange={(value) => handleRoleChange(role, value)}
options={[
{ value: '', label: t('role_none') },
...ownMailboxes.map(mb => ({
value: mb.id,
label: (nameCounts.get(mb.name) || 0) > 1
? `${getParentPath(mb)}${mb.name} (${mb.id.slice(-6)})`
: mb.name,
})),
]}
/>
</SettingItem>
);
})}
</SettingsSection> </SettingsSection>
</div> </div>
); );
@@ -18,6 +18,7 @@ interface SidebarAppFormData {
url: string; url: string;
icon: string; icon: string;
openMode: "tab" | "inline"; openMode: "tab" | "inline";
showOnMobile: boolean;
} }
function AppForm({ function AppForm({
@@ -37,6 +38,7 @@ function AppForm({
url: app?.url || "", url: app?.url || "",
icon: app?.icon || "Globe", icon: app?.icon || "Globe",
openMode: app?.openMode || "tab", openMode: app?.openMode || "tab",
showOnMobile: app?.showOnMobile ?? false,
}); });
const [errors, setErrors] = useState<Record<string, string>>({}); const [errors, setErrors] = useState<Record<string, string>>({});
@@ -144,6 +146,25 @@ function AppForm({
</div> </div>
</div> </div>
<div className="flex items-center justify-between">
<label className="text-sm font-medium">{t("show_on_mobile")}</label>
<button
type="button"
onClick={() => setFormData({ ...formData, showOnMobile: !formData.showOnMobile })}
className={cn(
"relative inline-flex h-5 w-9 items-center rounded-full transition-colors",
formData.showOnMobile ? "bg-primary" : "bg-muted-foreground/30"
)}
>
<span
className={cn(
"inline-block h-3.5 w-3.5 rounded-full bg-white transition-transform",
formData.showOnMobile ? "translate-x-4.5" : "translate-x-0.5"
)}
/>
</button>
</div>
<div className="flex gap-2 justify-end"> <div className="flex gap-2 justify-end">
<Button type="button" variant="ghost" size="sm" onClick={onCancel}> <Button type="button" variant="ghost" size="sm" onClick={onCancel}>
{t("cancel")} {t("cancel")}
+9 -5
View File
@@ -28,15 +28,18 @@ export default [
}, },
plugins: { plugins: {
"@typescript-eslint": tseslint, "@typescript-eslint": tseslint,
"react": reactPlugin, react: reactPlugin,
"react-hooks": reactHooksPlugin, "react-hooks": reactHooksPlugin,
}, },
rules: { rules: {
...tseslint.configs.recommended.rules, ...tseslint.configs.recommended.rules,
"@typescript-eslint/no-unused-vars": ["warn", { "@typescript-eslint/no-unused-vars": [
argsIgnorePattern: "^_", "warn",
varsIgnorePattern: "^_" {
}], argsIgnorePattern: "^_",
varsIgnorePattern: "^_",
},
],
"@typescript-eslint/no-explicit-any": "warn", "@typescript-eslint/no-explicit-any": "warn",
"@typescript-eslint/no-empty-object-type": "off", "@typescript-eslint/no-empty-object-type": "off",
"react-hooks/rules-of-hooks": "error", "react-hooks/rules-of-hooks": "error",
@@ -70,6 +73,7 @@ export default [
"*.config.js", "*.config.js",
"*.config.mjs", "*.config.mjs",
"e2e/**", "e2e/**",
"local-data/**/*.mjs",
], ],
}, },
]; ];
+98
View File
@@ -0,0 +1,98 @@
import { beforeEach, describe, expect, it, vi } from 'vitest';
const loggerError = vi.fn();
vi.mock('next/server', () => ({
NextResponse: {
json: (data: unknown, init?: { status?: number; headers?: unknown }) => ({
status: init?.status ?? 200,
headers: init?.headers,
json: async () => data,
}),
},
}));
vi.mock('@/lib/logger', () => ({
logger: {
error: loggerError,
},
}));
describe('health route', () => {
beforeEach(() => {
vi.restoreAllMocks();
loggerError.mockReset();
});
it('returns healthy for the basic liveness probe even when heap usage is high', async () => {
vi.spyOn(process, 'memoryUsage').mockReturnValue({
rss: 120_000_000,
heapTotal: 45_000_000,
heapUsed: 43_000_000,
external: 8_000_000,
arrayBuffers: 1_000_000,
});
const { GET } = await import('@/app/api/health/route');
const response = await GET({ nextUrl: new URL('http://localhost/api/health') } as never);
const payload = await response.json();
expect(response.status).toBe(200);
expect(payload).toMatchObject({
status: 'healthy',
});
expect(payload.warnings).toBeUndefined();
});
it('returns degraded diagnostics in detailed mode without failing the probe', async () => {
vi.spyOn(process, 'memoryUsage').mockReturnValue({
rss: 120_000_000,
heapTotal: 4_100_000_000,
heapUsed: 4_000_000_000,
external: 8_000_000,
arrayBuffers: 1_000_000,
});
vi.spyOn(process, 'uptime').mockReturnValue(123.45);
const { GET } = await import('@/app/api/health/route');
const response = await GET({ nextUrl: new URL('http://localhost/api/health?detailed=true') } as never);
const payload = await response.json();
expect(response.status).toBe(200);
expect(payload.status).toBe('degraded');
expect(payload.memory).toMatchObject({
heapUsed: 4_000_000_000,
heapTotal: 4_100_000_000,
rss: 120_000_000,
external: 8_000_000,
});
expect(payload.memory.heapSizeLimit).toBeGreaterThan(0);
expect(payload.warnings).toEqual([
expect.stringContaining('V8 heap usage is high'),
]);
});
it('keeps HEAD as a stable liveness probe', async () => {
const { HEAD } = await import('@/app/api/health/route');
const response = await HEAD();
expect(response.status).toBe(200);
});
it('returns 503 when collecting health diagnostics throws', async () => {
vi.spyOn(process, 'memoryUsage').mockImplementation(() => {
throw new Error('boom');
});
const { GET } = await import('@/app/api/health/route');
const response = await GET({ nextUrl: new URL('http://localhost/api/health') } as never);
const payload = await response.json();
expect(response.status).toBe(503);
expect(payload).toMatchObject({
status: 'unhealthy',
reason: 'boom',
});
expect(loggerError).toHaveBeenCalledWith('Health check failed', { error: 'boom' });
});
});
+39
View File
@@ -0,0 +1,39 @@
import { describe, expect, it } from 'vitest';
import {
appendPlainTextSignature,
getPlainTextSignature,
hasMeaningfulHtmlBody,
} from '../signature-utils';
describe('signature-utils', () => {
describe('getPlainTextSignature', () => {
it('prefers text signatures when present', () => {
expect(getPlainTextSignature({ textSignature: 'Regards,\nAlice', htmlSignature: '<p>Ignored</p>' })).toBe('Regards,\nAlice');
});
it('converts html-only signatures into plain text', () => {
expect(getPlainTextSignature({ htmlSignature: '<p>Alice Example<br><a href="mailto:alice@example.com">alice@example.com</a></p>' })).toBe('Alice Example\nalice@example.com');
});
});
describe('appendPlainTextSignature', () => {
it('appends a converted html signature to the text body', () => {
expect(appendPlainTextSignature('Hello there', { htmlSignature: '<p>Alice<br>Engineering</p>' })).toBe('Hello there\n\n-- \nAlice\nEngineering');
});
it('leaves the body untouched when no signature exists', () => {
expect(appendPlainTextSignature('Hello there', {})).toBe('Hello there');
});
});
describe('hasMeaningfulHtmlBody', () => {
it('prefers html bodies that preserve signature formatting', () => {
expect(hasMeaningfulHtmlBody('<div>Hello</div><br><p>Alice</p>')).toBe(true);
});
it('ignores minimal wrapper html with a single block', () => {
expect(hasMeaningfulHtmlBody('<div>Hello world</div>')).toBe(false);
});
});
});
+14
View File
@@ -8,6 +8,7 @@ import { useEmailStore } from '@/stores/email-store';
import { useContactStore } from '@/stores/contact-store'; import { useContactStore } from '@/stores/contact-store';
import { useCalendarStore } from '@/stores/calendar-store'; import { useCalendarStore } from '@/stores/calendar-store';
import { useFilterStore } from '@/stores/filter-store'; import { useFilterStore } from '@/stores/filter-store';
import { DEFAULT_SEARCH_FILTERS } from '@/lib/jmap/search-utils';
import { useIdentityStore } from '@/stores/identity-store'; import { useIdentityStore } from '@/stores/identity-store';
import { useVacationStore } from '@/stores/vacation-store'; import { useVacationStore } from '@/stores/vacation-store';
@@ -97,6 +98,19 @@ export function clearAllStores(): void {
error: null, error: null,
searchQuery: '', searchQuery: '',
quota: null, quota: null,
isPushConnected: false,
lastPushUpdate: null,
newEmailNotification: null,
selectedEmailIds: new Set<string>(),
hasMoreEmails: false,
totalEmails: 0,
expandedThreadIds: new Set<string>(),
threadEmailsCache: new Map(),
isLoadingThread: null,
selectedKeyword: null,
tagCounts: {},
searchFilters: { ...DEFAULT_SEARCH_FILTERS },
isAdvancedSearchOpen: false,
}); });
useIdentityStore.getState().clearIdentities(); useIdentityStore.getState().clearIdentities();
useContactStore.getState().clearContacts(); useContactStore.getState().clearContacts();
+7
View File
@@ -0,0 +1,7 @@
export function replaceWindowLocation(url: string): void {
if (typeof window === 'undefined') {
return;
}
window.location.replace(url);
}
+4
View File
@@ -142,3 +142,7 @@ export function formatSnapTime(minutes: number, timeFormat: "12h" | "24h"): stri
} }
return `${String(h).padStart(2, "0")}:${String(m).padStart(2, "0")}`; return `${String(h).padStart(2, "0")}:${String(m).padStart(2, "0")}`;
} }
export function getPrimaryCalendarId(event: Pick<CalendarEvent, 'calendarIds'>): string | undefined {
return Object.keys(event.calendarIds || {})[0];
}
+72 -37
View File
@@ -2066,6 +2066,11 @@ export class JMAPClient {
return coreCapability?.maxCallsInRequest || 50; return coreCapability?.maxCallsInRequest || 50;
} }
getMaxObjectsInGet(): number {
const coreCapability = this.capabilities["urn:ietf:params:jmap:core"] as { maxObjectsInGet?: number } | undefined;
return coreCapability?.maxObjectsInGet || 500;
}
getEventSourceUrl(): string | null { getEventSourceUrl(): string | null {
if (!this.session) return null; if (!this.session) return null;
@@ -2428,26 +2433,62 @@ export class JMAPClient {
} }
} }
async getContacts(addressBookId?: string): Promise<ContactCard[]> { private async fetchPaginatedContacts(
try { accountId: string,
const accountId = this.getContactsAccountId(); filter?: Record<string, unknown>,
const queryArgs: Record<string, unknown> = { accountId, limit: 1000 }; ): Promise<ContactCard[]> {
if (addressBookId) { const batchSize = this.getMaxObjectsInGet();
queryArgs.filter = { inAddressBook: addressBookId }; const allIds: string[] = [];
let position = 0;
// Paginate ContactCard/query to collect all IDs
for (;;) {
const queryArgs: Record<string, unknown> = { accountId, position, limit: batchSize };
if (filter) {
queryArgs.filter = filter;
} }
const response = await this.request([ const response = await this.request([
["ContactCard/query", queryArgs, "0"], ["ContactCard/query", queryArgs, "q"],
["ContactCard/get", {
accountId,
"#ids": { resultOf: "0", name: "ContactCard/query", path: "/ids" },
}, "1"],
], this.contactUsing()); ], this.contactUsing());
if (response.methodResponses?.[1]?.[0] === "ContactCard/get") { const queryResult = response.methodResponses?.[0];
return (response.methodResponses[1][1].list || []) as ContactCard[]; if (queryResult?.[0] !== "ContactCard/query") break;
const ids: string[] = queryResult[1].ids || [];
allIds.push(...ids);
const total: number = queryResult[1].total ?? -1;
if (ids.length < batchSize || (total > 0 && allIds.length >= total)) {
break;
} }
return []; position += ids.length;
}
if (allIds.length === 0) return [];
// Batch ContactCard/get to respect maxObjectsInGet
const allContacts: ContactCard[] = [];
for (let i = 0; i < allIds.length; i += batchSize) {
const chunk = allIds.slice(i, i + batchSize);
const response = await this.request([
["ContactCard/get", { accountId, ids: chunk }, "g"],
], this.contactUsing());
if (response.methodResponses?.[0]?.[0] === "ContactCard/get") {
const list = (response.methodResponses[0][1].list || []) as ContactCard[];
allContacts.push(...list);
}
}
return allContacts;
}
async getContacts(addressBookId?: string): Promise<ContactCard[]> {
try {
const accountId = this.getContactsAccountId();
const filter = addressBookId ? { inAddressBook: addressBookId } : undefined;
return await this.fetchPaginatedContacts(accountId, filter);
} catch (error) { } catch (error) {
console.error('Failed to get contacts:', error); console.error('Failed to get contacts:', error);
return []; return [];
@@ -2465,29 +2506,19 @@ export class JMAPClient {
const account = this.accounts[accountId]; const account = this.accounts[accountId];
try { try {
const response = await this.request([ const rawContacts = await this.fetchPaginatedContacts(accountId);
["ContactCard/query", { accountId, limit: 1000 }, "0"], const contacts = rawContacts.map((contact) => ({
["ContactCard/get", { ...contact,
accountId, id: isPrimary ? contact.id : `${accountId}:${contact.id}`,
"#ids": { resultOf: "0", name: "ContactCard/query", path: "/ids" }, originalId: contact.id,
}, "1"], addressBookIds: isPrimary ? contact.addressBookIds : (contact.addressBookIds ? Object.fromEntries(
], this.contactUsing()); Object.entries(contact.addressBookIds).map(([bookId, v]) => [`${accountId}:${bookId}`, v])
) : contact.addressBookIds),
if (response.methodResponses?.[1]?.[0] === "ContactCard/get") { accountId,
const rawContacts = (response.methodResponses[1][1].list || []) as ContactCard[]; accountName: account?.name || (isPrimary ? this.username : accountId),
const contacts = rawContacts.map((contact) => ({ isShared: !isPrimary,
...contact, }));
id: isPrimary ? contact.id : `${accountId}:${contact.id}`, allContacts.push(...contacts);
originalId: contact.id,
addressBookIds: isPrimary ? contact.addressBookIds : (contact.addressBookIds ? Object.fromEntries(
Object.entries(contact.addressBookIds).map(([bookId, v]) => [`${accountId}:${bookId}`, v])
) : contact.addressBookIds),
accountId,
accountName: account?.name || (isPrimary ? this.username : accountId),
isShared: !isPrimary,
}));
allContacts.push(...contacts);
}
} catch (error) { } catch (error) {
console.error(`Failed to fetch contacts for account ${accountId}:`, error); console.error(`Failed to fetch contacts for account ${accountId}:`, error);
} }
@@ -2888,6 +2919,10 @@ export class JMAPClient {
filter, filter,
limit: limit || 1000, limit: limit || 1000,
}; };
// Expand recurring events into individual occurrences when a date range is provided
if (filter.after || filter.before) {
queryArgs.expandRecurrences = true;
}
if (sort) { if (sort) {
queryArgs.sort = sort; queryArgs.sort = sort;
} }
+32 -1
View File
@@ -252,7 +252,20 @@ export interface ContactTitle {
organizationId?: string; organizationId?: string;
} }
// RFC 9553 AddressComponent
export interface AddressComponent {
kind: 'room' | 'apartment' | 'floor' | 'building' | 'number' | 'name' | 'block' | 'subDistrict' | 'district' | 'locality' | 'region' | 'postcode' | 'country' | 'direction' | 'landmark' | 'postOfficeBox' | 'separator' | string;
value: string;
phonetic?: string;
}
export interface ContactAddress { export interface ContactAddress {
// RFC 9553 format
components?: AddressComponent[];
full?: string;
isOrdered?: boolean;
defaultSeparator?: string;
// Legacy flat fields (from vCard import)
street?: string; street?: string;
locality?: string; locality?: string;
region?: string; region?: string;
@@ -284,9 +297,27 @@ export interface ContactMedia {
mediaType?: string; mediaType?: string;
} }
// RFC 9553 PartialDate
export interface PartialDate {
'@type'?: 'PartialDate';
year?: number;
month?: number;
day?: number;
calendarScale?: string;
}
// RFC 9553 Timestamp
export interface Timestamp {
'@type': 'Timestamp';
utc: string;
}
export type AnniversaryDate = string | PartialDate | Timestamp;
export interface ContactAnniversary { export interface ContactAnniversary {
'@type'?: 'Anniversary';
kind: 'birth' | 'death' | 'wedding' | 'other'; kind: 'birth' | 'death' | 'wedding' | 'other';
date: string; date: AnniversaryDate;
place?: ContactAddress; place?: ContactAddress;
} }
+158
View File
@@ -0,0 +1,158 @@
import { parseHtmlSafely, sanitizeSignatureHtml } from '@/lib/email-sanitization';
type SignatureSource = {
textSignature?: string;
htmlSignature?: string;
};
const BLOCK_TAGS = new Set([
'address',
'article',
'aside',
'blockquote',
'div',
'footer',
'header',
'li',
'nav',
'p',
'section',
'tr',
]);
function normalizeSignatureLineBreaks(value: string): string {
return value
.replace(/\r\n?/g, '\n')
.replace(/\u00a0/g, ' ')
.replace(/[ \t]+\n/g, '\n')
.replace(/\n{3,}/g, '\n\n')
.trim();
}
function htmlToPlainText(html: string): string {
const document = parseHtmlSafely(html);
const chunks: string[] = [];
const appendText = (value: string) => {
if (!value) return;
const normalized = value.replace(/\s+/g, ' ');
if (!normalized.trim()) return;
const previous = chunks[chunks.length - 1];
if (previous && !previous.endsWith('\n') && !previous.endsWith(' ')) {
chunks.push(' ');
}
chunks.push(normalized);
};
const appendNewline = () => {
const previous = chunks[chunks.length - 1];
if (previous === '\n') return;
if (previous?.endsWith('\n')) return;
chunks.push('\n');
};
const walk = (node: Node) => {
if (node.nodeType === Node.TEXT_NODE) {
appendText(node.textContent || '');
return;
}
if (node.nodeType !== Node.ELEMENT_NODE) {
return;
}
const element = node as HTMLElement;
const tagName = element.tagName.toLowerCase();
if (tagName === 'br') {
appendNewline();
return;
}
if (tagName === 'a') {
const text = element.textContent?.replace(/\s+/g, ' ').trim() || '';
const href = element.getAttribute('href')?.trim() || '';
const normalizedHref = href.replace(/^mailto:/i, '');
if (text && normalizedHref && text === normalizedHref) {
appendText(text);
return;
}
if (text && href && text !== href) {
appendText(`${text} <${href}>`);
return;
}
}
if (BLOCK_TAGS.has(tagName) && chunks.length > 0) {
appendNewline();
}
Array.from(element.childNodes).forEach(walk);
if (BLOCK_TAGS.has(tagName)) {
appendNewline();
}
};
Array.from(document.body.childNodes).forEach(walk);
return normalizeSignatureLineBreaks(chunks.join(''));
}
export function getPlainTextSignature(signature?: SignatureSource | null): string {
if (signature?.textSignature?.trim()) {
return normalizeSignatureLineBreaks(signature.textSignature);
}
if (signature?.htmlSignature?.trim()) {
return htmlToPlainText(sanitizeSignatureHtml(signature.htmlSignature));
}
return '';
}
export function appendPlainTextSignature(body: string, signature?: SignatureSource | null): string {
const plainTextSignature = getPlainTextSignature(signature);
if (!plainTextSignature) {
return body;
}
return `${body}\n\n-- \n${plainTextSignature}`;
}
export function hasMeaningfulHtmlBody(html: string): boolean {
if (!html.trim()) return false;
const document = parseHtmlSafely(html);
const richSelector = [
'table',
'img',
'style',
'b',
'strong',
'i',
'em',
'u',
'font',
'a[href]',
'div[style]',
'span[style]',
'p[style]',
'h1',
'h2',
'h3',
'h4',
'h5',
'h6',
'ul',
'ol',
'blockquote',
'br',
].join(', ');
if (document.querySelector(richSelector)) {
return true;
}
const blockElements = document.body.querySelectorAll('p, div, blockquote, li');
return blockElements.length > 1;
}
+10 -6
View File
@@ -97,17 +97,19 @@ const ROLE_PRIORITY: Record<string, number> = {
// Deduplicate mailboxes (e.g., "Sent" vs "Sent Mail") // Deduplicate mailboxes (e.g., "Sent" vs "Sent Mail")
function deduplicateMailboxes(mailboxes: Mailbox[]): Mailbox[] { function deduplicateMailboxes(mailboxes: Mailbox[]): Mailbox[] {
const roleMap = new Map<string, Mailbox>();
const result: Mailbox[] = []; const result: Mailbox[] = [];
// First pass: collect mailboxes with roles // Group role mailboxes by account so deduplication is scoped per-account
const rolesByAccount = new Map<string, Mailbox[]>();
mailboxes.forEach(mb => { mailboxes.forEach(mb => {
if (mb.role) { if (mb.role) {
roleMap.set(mb.role, mb); const key = mb.accountId || '';
if (!rolesByAccount.has(key)) rolesByAccount.set(key, []);
rolesByAccount.get(key)!.push(mb);
} }
}); });
// Second pass: filter out duplicates // Filter out duplicates scoped to the same account
mailboxes.forEach(mb => { mailboxes.forEach(mb => {
// If this mailbox has a role, always keep it // If this mailbox has a role, always keep it
if (mb.role) { if (mb.role) {
@@ -115,9 +117,11 @@ function deduplicateMailboxes(mailboxes: Mailbox[]): Mailbox[] {
return; return;
} }
// Check if this is a duplicate of a role-based mailbox // Check if this is a duplicate of a role-based mailbox in the SAME account
const accountKey = mb.accountId || '';
const accountRoles = rolesByAccount.get(accountKey) || [];
const lowerName = mb.name.toLowerCase(); const lowerName = mb.name.toLowerCase();
const isDuplicate = Array.from(roleMap.values()).some(roleMb => { const isDuplicate = accountRoles.some(roleMb => {
const roleLowerName = roleMb.name.toLowerCase(); const roleLowerName = roleMb.name.toLowerCase();
// Check for common duplicates: "Sent Mail" vs "Sent", etc. // Check for common duplicates: "Sent Mail" vs "Sent", etc.
return lowerName.includes(roleLowerName) || roleLowerName.includes(lowerName); return lowerName.includes(roleLowerName) || roleLowerName.includes(lowerName);
+48 -9
View File
@@ -1,4 +1,26 @@
import type { ContactCard, NameComponent, ContactMedia, ContactOnlineService } from "@/lib/jmap/types"; import type { ContactCard, NameComponent, ContactMedia, ContactOnlineService, AnniversaryDate, PartialDate } from "@/lib/jmap/types";
// Convert RFC 9553 AnniversaryDate (PartialDate|Timestamp|string) to vCard date string
function anniversaryDateToVcardString(date: AnniversaryDate): string {
if (typeof date === 'string') return date;
if (date && typeof date === 'object') {
if ('@type' in date && date['@type'] === 'Timestamp' && 'utc' in date) {
return (date as { utc: string }).utc.split('T')[0];
}
const pd = date as PartialDate;
if (pd.year && pd.month && pd.day) {
return `${String(pd.year).padStart(4, '0')}-${String(pd.month).padStart(2, '0')}-${String(pd.day).padStart(2, '0')}`;
}
if (pd.month && pd.day) {
return `--${String(pd.month).padStart(2, '0')}-${String(pd.day).padStart(2, '0')}`;
}
if (pd.year && pd.month) {
return `${String(pd.year).padStart(4, '0')}-${String(pd.month).padStart(2, '0')}`;
}
if (pd.year) return String(pd.year);
}
return String(date);
}
const VCARD_SEX_TO_GENDER: Record<string, string> = { const VCARD_SEX_TO_GENDER: Record<string, string> = {
M: "masculine", M: "masculine",
@@ -598,14 +620,30 @@ function generateSingleVCard(contact: ContactCard): string {
for (const addr of Object.values(contact.addresses)) { for (const addr of Object.values(contact.addresses)) {
const type = contextToType(addr.contexts); const type = contextToType(addr.contexts);
const typeParam = type ? `;TYPE=${type}` : ""; const typeParam = type ? `;TYPE=${type}` : "";
let street = addr.street || "";
let locality = addr.locality || "";
let region = addr.region || "";
let postcode = addr.postcode || "";
let country = addr.country || "";
// RFC 9553 components-based address: extract flat fields for vCard ADR
if (addr.components && addr.components.length > 0) {
const findComp = (kind: string) => addr.components!.filter(c => c.kind === kind).map(c => c.value).join(' ');
const number = findComp('number');
const name = findComp('name');
street = street || [number, name].filter(Boolean).join(' ');
locality = locality || findComp('locality');
region = region || findComp('region');
postcode = postcode || findComp('postcode');
country = country || findComp('country');
}
const parts = [ const parts = [
"", "",
"", "",
addr.street || "", street,
addr.locality || "", locality,
addr.region || "", region,
addr.postcode || "", postcode,
addr.country || "", country,
]; ];
lines.push(`ADR${typeParam}:${parts.map(encodeValue).join(";")}`); lines.push(`ADR${typeParam}:${parts.map(encodeValue).join(";")}`);
} }
@@ -613,12 +651,13 @@ function generateSingleVCard(contact: ContactCard): string {
if (contact.anniversaries) { if (contact.anniversaries) {
for (const ann of Object.values(contact.anniversaries)) { for (const ann of Object.values(contact.anniversaries)) {
const dateStr = anniversaryDateToVcardString(ann.date);
if (ann.kind === "birth") { if (ann.kind === "birth") {
lines.push(`BDAY:${ann.date}`); lines.push(`BDAY:${dateStr}`);
} else if (ann.kind === "wedding") { } else if (ann.kind === "wedding") {
lines.push(`ANNIVERSARY:${ann.date}`); lines.push(`ANNIVERSARY:${dateStr}`);
} else if (ann.kind === "death") { } else if (ann.kind === "death") {
lines.push(`DEATHDATE:${ann.date}`); lines.push(`DEATHDATE:${dateStr}`);
} }
} }
} }
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Anhänge", "attachments": "Anhänge",
"important": "Wichtig", "important": "Wichtig",
"download": "Herunterladen", "download": "Herunterladen",
"download_all": "Alle herunterladen",
"from": "Von", "from": "Von",
"to": "An", "to": "An",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "Zurück", "previous": "Zurück",
"next": "Weiter", "next": "Weiter",
"send": "Senden" "send": "Senden",
"more": "mehr"
}, },
"email_composer": { "email_composer": {
"new_message": "Neue Nachricht", "new_message": "Neue Nachricht",
@@ -753,6 +755,12 @@
"preview": "Wenn möglich in Vorschau öffnen", "preview": "Wenn möglich in Vorschau öffnen",
"download": "Sofort herunterladen" "download": "Sofort herunterladen"
}, },
"attachment_position": {
"label": "Anhangsposition",
"description": "Wo Anhänge im E-Mail-Header angezeigt werden",
"beside-sender": "Neben dem Absender",
"below-header": "Unter dem Header"
},
"emails_per_page": { "emails_per_page": {
"25": "25 E-Mails", "25": "25 E-Mails",
"50": "50 E-Mails", "50": "50 E-Mails",
@@ -1947,6 +1955,12 @@
"deleted": "Abonnement entfernt", "deleted": "Abonnement entfernt",
"delete_error": "Abonnement konnte nicht entfernt werden", "delete_error": "Abonnement konnte nicht entfernt werden",
"last_refreshed": "Zuletzt aktualisiert: {time}" "last_refreshed": "Zuletzt aktualisiert: {time}"
},
"tasks": {
"no_tasks": "Keine Aufgaben",
"no_title": "(Kein Titel)",
"mark_complete": "Als erledigt markieren",
"mark_incomplete": "Als unerledigt markieren"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Attachments", "attachments": "Attachments",
"important": "Important", "important": "Important",
"download": "Download", "download": "Download",
"download_all": "Download all",
"from": "From", "from": "From",
"to": "To", "to": "To",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
"event_status_tentative": "Tentative", "event_status_tentative": "Tentative",
"event_status_cancelled": "Cancelled" "event_status_cancelled": "Cancelled"
}, },
"send": "Send" "send": "Send",
"more": "more"
}, },
"email_composer": { "email_composer": {
"new_message": "New Message", "new_message": "New Message",
@@ -753,6 +755,12 @@
"preview": "Preview when possible", "preview": "Preview when possible",
"download": "Download immediately" "download": "Download immediately"
}, },
"attachment_position": {
"label": "Attachment Position",
"description": "Where to display attachments in the email header",
"beside-sender": "Next to sender",
"below-header": "Below header"
},
"emails_per_page": { "emails_per_page": {
"25": "25 emails", "25": "25 emails",
"50": "50 emails", "50": "50 emails",
@@ -1947,6 +1955,12 @@
"deleted": "Subscription removed", "deleted": "Subscription removed",
"delete_error": "Failed to remove subscription", "delete_error": "Failed to remove subscription",
"last_refreshed": "Last updated: {time}" "last_refreshed": "Last updated: {time}"
},
"tasks": {
"no_tasks": "No tasks",
"no_title": "(No title)",
"mark_complete": "Mark as complete",
"mark_incomplete": "Mark as incomplete"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Archivos adjuntos", "attachments": "Archivos adjuntos",
"important": "Importante", "important": "Importante",
"download": "Descargar", "download": "Descargar",
"download_all": "Descargar todo",
"from": "De", "from": "De",
"to": "Para", "to": "Para",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "Anterior", "previous": "Anterior",
"next": "Siguiente", "next": "Siguiente",
"send": "Enviar" "send": "Enviar",
"more": "más"
}, },
"email_composer": { "email_composer": {
"new_message": "Nuevo Mensaje", "new_message": "Nuevo Mensaje",
@@ -753,6 +755,12 @@
"preview": "Mostrar vista previa cuando sea posible", "preview": "Mostrar vista previa cuando sea posible",
"download": "Descargar inmediatamente" "download": "Descargar inmediatamente"
}, },
"attachment_position": {
"label": "Posición del adjunto",
"description": "Dónde mostrar los adjuntos en el encabezado del correo",
"beside-sender": "Junto al remitente",
"below-header": "Debajo del encabezado"
},
"emails_per_page": { "emails_per_page": {
"25": "25 correos", "25": "25 correos",
"50": "50 correos", "50": "50 correos",
@@ -1947,6 +1955,12 @@
"deleted": "Suscripción eliminada", "deleted": "Suscripción eliminada",
"delete_error": "No se pudo eliminar la suscripción", "delete_error": "No se pudo eliminar la suscripción",
"last_refreshed": "Última actualización: {time}" "last_refreshed": "Última actualización: {time}"
},
"tasks": {
"no_tasks": "Sin tareas",
"no_title": "(Sin título)",
"mark_complete": "Marcar como completada",
"mark_incomplete": "Marcar como incompleta"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Pièces jointes", "attachments": "Pièces jointes",
"important": "Important", "important": "Important",
"download": "Télécharger", "download": "Télécharger",
"download_all": "Tout télécharger",
"from": "De", "from": "De",
"to": "À", "to": "À",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "Précédent", "previous": "Précédent",
"next": "Suivant", "next": "Suivant",
"send": "Envoyer" "send": "Envoyer",
"more": "plus"
}, },
"email_composer": { "email_composer": {
"new_message": "Nouveau message", "new_message": "Nouveau message",
@@ -753,6 +755,12 @@
"preview": "Aperçu si possible", "preview": "Aperçu si possible",
"download": "Télécharger immédiatement" "download": "Télécharger immédiatement"
}, },
"attachment_position": {
"label": "Position des pièces jointes",
"description": "Où afficher les pièces jointes dans l'en-tête de l'email",
"beside-sender": "À côté de l'expéditeur",
"below-header": "Sous l'en-tête"
},
"emails_per_page": { "emails_per_page": {
"25": "25 emails", "25": "25 emails",
"50": "50 emails", "50": "50 emails",
@@ -1947,6 +1955,12 @@
"deleted": "Abonnement supprimé", "deleted": "Abonnement supprimé",
"delete_error": "Impossible de supprimer l'abonnement", "delete_error": "Impossible de supprimer l'abonnement",
"last_refreshed": "Dernière mise à jour : {time}" "last_refreshed": "Dernière mise à jour : {time}"
},
"tasks": {
"no_tasks": "Aucune tâche",
"no_title": "(Sans titre)",
"mark_complete": "Marquer comme terminée",
"mark_incomplete": "Marquer comme non terminée"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Allegati", "attachments": "Allegati",
"important": "Importante", "important": "Importante",
"download": "Scarica", "download": "Scarica",
"download_all": "Scarica tutto",
"from": "Da", "from": "Da",
"to": "A", "to": "A",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "Precedente", "previous": "Precedente",
"next": "Successivo", "next": "Successivo",
"send": "Invia" "send": "Invia",
"more": "altri"
}, },
"email_composer": { "email_composer": {
"new_message": "Nuovo messaggio", "new_message": "Nuovo messaggio",
@@ -753,6 +755,12 @@
"preview": "Anteprima quando possibile", "preview": "Anteprima quando possibile",
"download": "Scarica immediatamente" "download": "Scarica immediatamente"
}, },
"attachment_position": {
"label": "Posizione degli allegati",
"description": "Dove visualizzare gli allegati nell'intestazione dell'email",
"beside-sender": "Accanto al mittente",
"below-header": "Sotto l'intestazione"
},
"emails_per_page": { "emails_per_page": {
"25": "25 messaggi", "25": "25 messaggi",
"50": "50 messaggi", "50": "50 messaggi",
@@ -1947,6 +1955,12 @@
"deleted": "Abbonamento rimosso", "deleted": "Abbonamento rimosso",
"delete_error": "Impossibile rimuovere l'abbonamento", "delete_error": "Impossibile rimuovere l'abbonamento",
"last_refreshed": "Ultimo aggiornamento: {time}" "last_refreshed": "Ultimo aggiornamento: {time}"
},
"tasks": {
"no_tasks": "Nessuna attività",
"no_title": "(Senza titolo)",
"mark_complete": "Segna come completata",
"mark_incomplete": "Segna come non completata"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "添付ファイル", "attachments": "添付ファイル",
"important": "重要", "important": "重要",
"download": "ダウンロード", "download": "ダウンロード",
"download_all": "すべてダウンロード",
"from": "送信者", "from": "送信者",
"to": "宛先", "to": "宛先",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "前へ", "previous": "前へ",
"next": "次へ", "next": "次へ",
"send": "送信" "send": "送信",
"more": "他"
}, },
"email_composer": { "email_composer": {
"new_message": "新規メッセージ", "new_message": "新規メッセージ",
@@ -753,6 +755,12 @@
"preview": "可能ならプレビューを開く", "preview": "可能ならプレビューを開く",
"download": "すぐにダウンロード" "download": "すぐにダウンロード"
}, },
"attachment_position": {
"label": "添付ファイルの位置",
"description": "メールヘッダー内での添付ファイルの表示位置",
"beside-sender": "送信者の横",
"below-header": "ヘッダーの下"
},
"emails_per_page": { "emails_per_page": {
"25": "25件", "25": "25件",
"50": "50件", "50": "50件",
@@ -1947,6 +1955,12 @@
"deleted": "購読を解除しました", "deleted": "購読を解除しました",
"delete_error": "購読の解除に失敗しました", "delete_error": "購読の解除に失敗しました",
"last_refreshed": "最終更新: {time}" "last_refreshed": "最終更新: {time}"
},
"tasks": {
"no_tasks": "タスクなし",
"no_title": "(タイトルなし)",
"mark_complete": "完了にする",
"mark_incomplete": "未完了にする"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Bijlagen", "attachments": "Bijlagen",
"important": "Belangrijk", "important": "Belangrijk",
"download": "Downloaden", "download": "Downloaden",
"download_all": "Alles downloaden",
"from": "Van", "from": "Van",
"to": "Aan", "to": "Aan",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "Vorige", "previous": "Vorige",
"next": "Volgende", "next": "Volgende",
"send": "Verzenden" "send": "Verzenden",
"more": "meer"
}, },
"email_composer": { "email_composer": {
"new_message": "Nieuw bericht", "new_message": "Nieuw bericht",
@@ -753,6 +755,12 @@
"preview": "Voorbeeld tonen indien mogelijk", "preview": "Voorbeeld tonen indien mogelijk",
"download": "Direct downloaden" "download": "Direct downloaden"
}, },
"attachment_position": {
"label": "Positie van bijlagen",
"description": "Waar bijlagen in de e-mailkop worden weergegeven",
"beside-sender": "Naast de afzender",
"below-header": "Onder de kop"
},
"emails_per_page": { "emails_per_page": {
"25": "25 e-mails", "25": "25 e-mails",
"50": "50 e-mails", "50": "50 e-mails",
@@ -1947,6 +1955,12 @@
"deleted": "Abonnement verwijderd", "deleted": "Abonnement verwijderd",
"delete_error": "Kan abonnement niet verwijderen", "delete_error": "Kan abonnement niet verwijderen",
"last_refreshed": "Laatst bijgewerkt: {time}" "last_refreshed": "Laatst bijgewerkt: {time}"
},
"tasks": {
"no_tasks": "Geen taken",
"no_title": "(Geen titel)",
"mark_complete": "Markeren als voltooid",
"mark_incomplete": "Markeren als onvoltooid"
} }
}, },
"advanced_search": { "advanced_search": {
+15 -1
View File
@@ -210,6 +210,7 @@
"attachments": "Anexos", "attachments": "Anexos",
"important": "Importante", "important": "Importante",
"download": "Baixar", "download": "Baixar",
"download_all": "Baixar tudo",
"from": "De", "from": "De",
"to": "Para", "to": "Para",
"cc": "CC", "cc": "CC",
@@ -401,7 +402,8 @@
}, },
"previous": "Anterior", "previous": "Anterior",
"next": "Próximo", "next": "Próximo",
"send": "Enviar" "send": "Enviar",
"more": "mais"
}, },
"email_composer": { "email_composer": {
"new_message": "Nova Mensagem", "new_message": "Nova Mensagem",
@@ -753,6 +755,12 @@
"preview": "Visualizar quando possível", "preview": "Visualizar quando possível",
"download": "Baixar imediatamente" "download": "Baixar imediatamente"
}, },
"attachment_position": {
"label": "Posição dos anexos",
"description": "Onde exibir os anexos no cabeçalho do e-mail",
"beside-sender": "Ao lado do remetente",
"below-header": "Abaixo do cabeçalho"
},
"emails_per_page": { "emails_per_page": {
"25": "25 e-mails", "25": "25 e-mails",
"50": "50 e-mails", "50": "50 e-mails",
@@ -1947,6 +1955,12 @@
"deleted": "Assinatura removida", "deleted": "Assinatura removida",
"delete_error": "Falha ao remover assinatura", "delete_error": "Falha ao remover assinatura",
"last_refreshed": "Última atualização: {time}" "last_refreshed": "Última atualização: {time}"
},
"tasks": {
"no_tasks": "Sem tarefas",
"no_title": "(Sem título)",
"mark_complete": "Marcar como concluída",
"mark_incomplete": "Marcar como não concluída"
} }
}, },
"advanced_search": { "advanced_search": {
+5 -5
View File
@@ -1,12 +1,12 @@
{ {
"name": "bulwark-webmail", "name": "bulwark-webmail",
"version": "1.4.3", "version": "1.4.4",
"lockfileVersion": 3, "lockfileVersion": 3,
"requires": true, "requires": true,
"packages": { "packages": {
"": { "": {
"name": "bulwark-webmail", "name": "bulwark-webmail",
"version": "1.4.3", "version": "1.4.4",
"license": "AGPL-3.0-only", "license": "AGPL-3.0-only",
"dependencies": { "dependencies": {
"@tanstack/react-virtual": "^3.13.18", "@tanstack/react-virtual": "^3.13.18",
@@ -5480,9 +5480,9 @@
} }
}, },
"node_modules/flatted": { "node_modules/flatted": {
"version": "3.3.3", "version": "3.4.2",
"resolved": "https://registry.npmjs.org/flatted/-/flatted-3.3.3.tgz", "resolved": "https://registry.npmjs.org/flatted/-/flatted-3.4.2.tgz",
"integrity": "sha512-GX+ysw4PBCz0PzosHDepZGANEuFCMLrnRTiEy9McGjmkCQYwRq4A/X786G/fjM/+OjsWSU1ZrY5qyARZmO/uwg==", "integrity": "sha512-PjDse7RzhcPkIJwy5t7KPWQSZ9cAbzQXcafsetQoD7sOJRQlGikNbx7yZp2OotDnJyrDcbyRq3Ttb18iYOqkxA==",
"dev": true, "dev": true,
"license": "ISC" "license": "ISC"
}, },
+2 -1
View File
@@ -1,6 +1,6 @@
{ {
"name": "bulwark-webmail", "name": "bulwark-webmail",
"version": "1.4.3", "version": "1.4.5",
"description": "Bulwark Webmail — a modern webmail client built for Stalwart Mail Server", "description": "Bulwark Webmail — a modern webmail client built for Stalwart Mail Server",
"author": "Bulwark Webmail <bulwark@rbm.systems>", "author": "Bulwark Webmail <bulwark@rbm.systems>",
"license": "AGPL-3.0-only", "license": "AGPL-3.0-only",
@@ -77,6 +77,7 @@
}, },
"overrides": { "overrides": {
"elliptic": "^6.6.1", "elliptic": "^6.6.1",
"flatted": "^3.4.2",
"undici": "^7.24.0" "undici": "^7.24.0"
} }
} }
@@ -0,0 +1,97 @@
import { beforeEach, afterEach, describe, expect, it, vi } from 'vitest';
import * as browserNavigation from '@/lib/browser-navigation';
import { useAuthStore } from '../auth-store';
import { useAccountStore } from '../account-store';
type FetchInput = Parameters<typeof fetch>[0];
type FetchInit = Parameters<typeof fetch>[1];
describe('auth-store logout redirects', () => {
beforeEach(() => {
vi.restoreAllMocks();
sessionStorage.clear();
localStorage.clear();
window.history.pushState({}, '', '/en');
useAccountStore.setState({
accounts: [],
activeAccountId: null,
defaultAccountId: null,
});
useAuthStore.setState({
isAuthenticated: false,
isLoading: false,
error: null,
serverUrl: null,
username: null,
client: null,
identities: [],
primaryIdentity: null,
authMode: 'basic',
rememberMe: false,
accessToken: null,
tokenExpiresAt: null,
connectionLost: false,
activeAccountId: null,
});
});
afterEach(() => {
vi.useRealTimers();
});
it('redirects full logout to the locale login page', () => {
const fetchMock = vi.fn().mockResolvedValue({ ok: true, json: async () => ({}) });
vi.stubGlobal('fetch', fetchMock);
const replaceSpy = vi.spyOn(browserNavigation, 'replaceWindowLocation').mockImplementation(() => {});
window.history.pushState({}, '', '/fr/calendar');
useAuthStore.setState({ isAuthenticated: true, authMode: 'basic' });
useAuthStore.getState().logout();
expect(replaceSpy).toHaveBeenCalledWith('/fr/login');
expect(fetchMock).toHaveBeenCalledWith('/api/auth/session?slot=0', { method: 'DELETE', keepalive: true });
});
it('marks session expiry, preserves the current path, and redirects to login on refresh failure', async () => {
vi.useFakeTimers();
const fetchMock = vi.fn(async (input: FetchInput, init?: FetchInit) => {
const url = String(input);
const method = init?.method ?? 'GET';
if (url === '/api/auth/token?slot=0' && method === 'PUT') {
return { ok: false, json: async () => ({}) };
}
if (url === '/api/auth/token?slot=0' && method === 'DELETE') {
return { ok: true, json: async () => ({}) };
}
if (url === '/api/auth/session?slot=0' && method === 'DELETE') {
return { ok: true, json: async () => ({}) };
}
throw new Error(`Unexpected fetch call: ${method} ${url}`);
});
vi.stubGlobal('fetch', fetchMock);
const replaceSpy = vi.spyOn(browserNavigation, 'replaceWindowLocation').mockImplementation(() => {});
window.history.pushState({}, '', '/en/calendar?view=day');
useAuthStore.setState({
isAuthenticated: true,
authMode: 'oauth',
activeAccountId: null,
});
await useAuthStore.getState().refreshAccessToken();
await vi.runAllTimersAsync();
expect(sessionStorage.getItem('session_expired')).toBe('true');
expect(sessionStorage.getItem('redirect_after_login')).toBe('/en/calendar?view=day');
expect(replaceSpy).toHaveBeenCalledWith('/en/login');
});
});
@@ -226,6 +226,34 @@ describe('email-store folder management', () => {
expect(client.updateMailbox).toHaveBeenCalledWith('trash-1', { role: 'trash' }); expect(client.updateMailbox).toHaveBeenCalledWith('trash-1', { role: 'trash' });
}); });
it('should clear role from ALL mailboxes with that role when reassigning', async () => {
// Simulate server anomaly: two mailboxes with role "trash"
const extraTrash = makeMailbox({ id: 'trash-2', name: 'Deleted Items', role: 'trash' });
useEmailStore.setState({
mailboxes: [inbox, sent, trash, custom, extraTrash],
});
const newMailboxes = [inbox, sent, custom,
makeMailbox({ id: 'trash-1', name: 'Trash', role: undefined }),
makeMailbox({ id: 'trash-2', name: 'Deleted Items', role: undefined }),
];
// custom-1 gets the trash role
newMailboxes[2] = { ...newMailboxes[2], role: 'trash' };
const client = makeMockClient({
getAllMailboxes: vi.fn().mockResolvedValue(newMailboxes),
});
await useEmailStore.getState().setMailboxRole(client, 'custom-1', 'trash');
// Should clear trash role from BOTH trash-1 and trash-2
expect(client.updateMailbox).toHaveBeenCalledWith('trash-1', { role: null });
expect(client.updateMailbox).toHaveBeenCalledWith('trash-2', { role: null });
// Then set trash role on custom-1
expect(client.updateMailbox).toHaveBeenCalledWith('custom-1', { role: 'trash' });
expect(client.updateMailbox).toHaveBeenCalledTimes(3);
});
it('should set error on failure', async () => { it('should set error on failure', async () => {
const client = makeMockClient({ const client = makeMockClient({
updateMailbox: vi.fn().mockRejectedValue(new Error('Role update failed')), updateMailbox: vi.fn().mockRejectedValue(new Error('Role update failed')),
+21 -4
View File
@@ -58,13 +58,30 @@ export const useAccountStore = create<AccountState>()(
addAccount: (entry) => { addAccount: (entry) => {
const state = get(); const state = get();
if (state.accounts.length >= MAX_ACCOUNTS) {
throw new Error(`Maximum of ${MAX_ACCOUNTS} accounts reached`);
}
const id = generateAccountId(entry.username, entry.serverUrl); const id = generateAccountId(entry.username, entry.serverUrl);
if (state.accounts.some((a) => a.id === id)) { if (state.accounts.some((a) => a.id === id)) {
return id; // already exists, return existing id // Already exists — update mutable fields and return existing id
set((s) => ({
accounts: s.accounts.map((a) =>
a.id === id
? {
...a,
rememberMe: entry.rememberMe,
isConnected: entry.isConnected,
hasError: entry.hasError,
errorMessage: undefined,
lastLoginAt: entry.lastLoginAt,
authMode: entry.authMode,
}
: a
),
}));
return id;
}
if (state.accounts.length >= MAX_ACCOUNTS) {
throw new Error(`Maximum of ${MAX_ACCOUNTS} accounts reached`);
} }
const cookieSlot = state.getNextCookieSlot(); const cookieSlot = state.getNextCookieSlot();
+234 -56
View File
@@ -11,6 +11,7 @@ import { useAccountStore } from './account-store';
import { fetchConfig } from '@/hooks/use-config'; import { fetchConfig } from '@/hooks/use-config';
import { debug } from '@/lib/debug'; import { debug } from '@/lib/debug';
import { generateAccountId } from '@/lib/account-utils'; import { generateAccountId } from '@/lib/account-utils';
import { replaceWindowLocation } from '@/lib/browser-navigation';
import { snapshotAccount, restoreAccount, clearAllStores, evictAccount, evictAll } from '@/lib/account-state-manager'; import { snapshotAccount, restoreAccount, clearAllStores, evictAccount, evictAll } from '@/lib/account-state-manager';
import type { Identity } from '@/lib/jmap/types'; import type { Identity } from '@/lib/jmap/types';
@@ -33,7 +34,7 @@ interface AuthState {
login: (serverUrl: string, username: string, password: string, totp?: string, rememberMe?: boolean) => Promise<boolean>; login: (serverUrl: string, username: string, password: string, totp?: string, rememberMe?: boolean) => Promise<boolean>;
loginWithOAuth: (serverUrl: string, code: string, codeVerifier: string, redirectUri: string) => Promise<boolean>; loginWithOAuth: (serverUrl: string, code: string, codeVerifier: string, redirectUri: string) => Promise<boolean>;
refreshAccessToken: () => Promise<string | null>; refreshAccessToken: () => Promise<string | null>;
logout: () => void; logout: () => Promise<void>;
logoutAll: () => void; logoutAll: () => void;
switchAccount: (accountId: string) => Promise<void>; switchAccount: (accountId: string) => Promise<void>;
checkAuth: () => Promise<void>; checkAuth: () => Promise<void>;
@@ -98,8 +99,45 @@ function loadIdentities(rawIdentities: Identity[], username: string): { identiti
return { identities, primaryIdentity }; return { identities, primaryIdentity };
} }
function getLocaleLoginPath(): string {
if (typeof window === 'undefined') return '/en/login';
const segments = window.location.pathname.split('/').filter(Boolean);
const locale = segments[0] || 'en';
return `/${locale}/login`;
}
function saveRedirectAfterLogin(): void {
if (typeof window === 'undefined') return;
try {
const loginPath = getLocaleLoginPath();
const currentPath = `${window.location.pathname}${window.location.search}${window.location.hash}`;
if (currentPath !== loginPath) {
sessionStorage.setItem('redirect_after_login', currentPath);
}
} catch {
/* noop */
}
}
function redirectToLogin(): void {
if (typeof window === 'undefined') return;
const loginPath = getLocaleLoginPath();
if (window.location.pathname === loginPath) return;
replaceWindowLocation(loginPath);
}
function markSessionExpired(): void { function markSessionExpired(): void {
try { sessionStorage.setItem('session_expired', 'true'); } catch { /* noop */ } try {
sessionStorage.setItem('session_expired', 'true');
} catch {
/* noop */
}
saveRedirectAfterLogin();
} }
function initializeFeatureStores(client: JMAPClient): void { function initializeFeatureStores(client: JMAPClient): void {
@@ -226,10 +264,12 @@ export const useAuthStore = create<AuthState>()(
? (accountStore.getAccountById(accountId)?.cookieSlot ?? accountStore.getNextCookieSlot()) ? (accountStore.getAccountById(accountId)?.cookieSlot ?? accountStore.getNextCookieSlot())
: accountStore.getNextCookieSlot(); : accountStore.getNextCookieSlot();
// Snapshot current account if switching away // Snapshot current account if switching away and clear stores so
// the new account starts with a clean email/contact/calendar state.
const prevAccountId = get().activeAccountId; const prevAccountId = get().activeAccountId;
if (prevAccountId && prevAccountId !== accountId) { if (prevAccountId && prevAccountId !== accountId) {
snapshotAccount(prevAccountId); snapshotAccount(prevAccountId);
clearAllStores();
} }
// Store client in multi-account map // Store client in multi-account map
@@ -250,6 +290,33 @@ export const useAuthStore = create<AuthState>()(
}); });
accountStore.setActiveAccount(accountId); accountStore.setActiveAccount(accountId);
// Update account entry in case it already existed (addAccount is a no-op for existing accounts)
accountStore.updateAccount(accountId, {
rememberMe: !!rememberMe,
isConnected: true,
hasError: false,
errorMessage: undefined,
lastLoginAt: Date.now(),
});
// Store session cookie BEFORE setting isAuthenticated to avoid a race
// condition: setting isAuthenticated triggers navigation to the main page,
// whose checkAuth() would try to read the cookie before it was stored.
if (rememberMe) {
try {
const res = await fetch(`/api/auth/session?slot=${cookieSlot}`, {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ serverUrl, username, password: effectivePassword, slot: cookieSlot }),
});
if (!res.ok) {
debug.error('Failed to store session: server returned', res.status);
}
} catch (err) {
debug.error('Failed to store session:', err);
}
}
set({ set({
isAuthenticated: true, isAuthenticated: true,
isLoading: false, isLoading: false,
@@ -259,6 +326,7 @@ export const useAuthStore = create<AuthState>()(
identities, identities,
primaryIdentity, primaryIdentity,
authMode: 'basic', authMode: 'basic',
rememberMe: !!rememberMe,
accessToken: null, accessToken: null,
tokenExpiresAt: null, tokenExpiresAt: null,
connectionLost: false, connectionLost: false,
@@ -274,23 +342,6 @@ export const useAuthStore = create<AuthState>()(
}); });
}).catch(() => {}); }).catch(() => {});
if (rememberMe) {
try {
const res = await fetch(`/api/auth/session?slot=${cookieSlot}`, {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ serverUrl, username, password: effectivePassword, slot: cookieSlot }),
});
if (res.ok) {
set({ rememberMe: true });
} else {
debug.error('Failed to store session: server returned', res.status);
}
} catch (err) {
debug.error('Failed to store session:', err);
}
}
return true; return true;
} catch (error) { } catch (error) {
debug.error('Login error:', error); debug.error('Login error:', error);
@@ -341,10 +392,12 @@ export const useAuthStore = create<AuthState>()(
// Register in account store // Register in account store
const accountId = generateAccountId(username, serverUrl); const accountId = generateAccountId(username, serverUrl);
// Snapshot current account if switching away // Snapshot current account if switching away and clear stores so
// the new account starts with a clean email/contact/calendar state.
const prevAccountId = get().activeAccountId; const prevAccountId = get().activeAccountId;
if (prevAccountId && prevAccountId !== accountId) { if (prevAccountId && prevAccountId !== accountId) {
snapshotAccount(prevAccountId); snapshotAccount(prevAccountId);
clearAllStores();
} }
clients.set(accountId, client); clients.set(accountId, client);
@@ -457,7 +510,7 @@ export const useAuthStore = create<AuthState>()(
return promise; return promise;
}, },
logout: () => { logout: async () => {
const state = get(); const state = get();
const wasOAuth = state.authMode === 'oauth'; const wasOAuth = state.authMode === 'oauth';
const accountId = state.activeAccountId; const accountId = state.activeAccountId;
@@ -466,6 +519,11 @@ export const useAuthStore = create<AuthState>()(
const slot = account?.cookieSlot ?? 0; const slot = account?.cookieSlot ?? 0;
clearRefreshTimer(accountId ?? undefined); clearRefreshTimer(accountId ?? undefined);
// Null out the client BEFORE disconnecting so the page doesn't fire
// data-loading effects with the stale disconnected client while
// stores are being cleared.
set({ client: null });
state.client?.disconnect(); state.client?.disconnect();
// Remove client from multi-account map // Remove client from multi-account map
@@ -479,6 +537,7 @@ export const useAuthStore = create<AuthState>()(
// Check if there are remaining accounts to switch to // Check if there are remaining accounts to switch to
const remainingAccounts = accountStore.accounts; const remainingAccounts = accountStore.accounts;
const shouldRedirectToLogin = remainingAccounts.length === 0;
if (remainingAccounts.length > 0) { if (remainingAccounts.length > 0) {
// Switch to the next account // Switch to the next account
const nextAccount = remainingAccounts[0]; const nextAccount = remainingAccounts[0];
@@ -486,11 +545,56 @@ export const useAuthStore = create<AuthState>()(
clearAllStores(); clearAllStores();
// Restore next account // Restore next account
const nextClient = clients.get(nextAccount.id); let nextClient = clients.get(nextAccount.id);
// If the client isn't in memory, try to restore it from the session
if (!nextClient) {
try {
if (nextAccount.authMode === 'oauth') {
const res = await fetch(`/api/auth/token?slot=${nextAccount.cookieSlot}`, { method: 'PUT' });
if (res.ok) {
const { access_token, expires_in } = await res.json();
const refreshFn = get().refreshAccessToken;
nextClient = JMAPClient.withBearer(nextAccount.serverUrl, access_token, nextAccount.username, () => refreshFn());
nextClient.onConnectionChange((connected) => {
if (get().activeAccountId === nextAccount.id) {
set({ connectionLost: !connected });
}
accountStore.updateAccount(nextAccount.id, { isConnected: connected });
});
await nextClient.connect();
clients.set(nextAccount.id, nextClient);
scheduleRefresh(expires_in, get().refreshAccessToken, nextAccount.id);
}
} else if (nextAccount.authMode === 'basic' && nextAccount.rememberMe) {
const res = await fetch(`/api/auth/session?slot=${nextAccount.cookieSlot}`);
if (res.ok) {
const { serverUrl: sUrl, username: uName, password: pwd } = await res.json();
nextClient = new JMAPClient(sUrl, uName, pwd);
nextClient.onConnectionChange((connected) => {
if (get().activeAccountId === nextAccount.id) {
set({ connectionLost: !connected });
}
accountStore.updateAccount(nextAccount.id, { isConnected: connected });
});
await nextClient.connect();
clients.set(nextAccount.id, nextClient);
}
}
} catch (err) {
debug.error(`Failed to restore next account ${nextAccount.id} during logout:`, err);
nextClient = undefined;
}
}
if (nextClient) { if (nextClient) {
const restored = restoreAccount(nextAccount.id); const restored = restoreAccount(nextAccount.id);
accountStore.setActiveAccount(nextAccount.id); accountStore.setActiveAccount(nextAccount.id);
// Build identity state up front so the name updates atomically
const restoredIdentities = restored ? useIdentityStore.getState().identities : [];
const restoredPrimary = restoredIdentities[0] ?? null;
set({ set({
isAuthenticated: true, isAuthenticated: true,
isLoading: false, isLoading: false,
@@ -498,9 +602,12 @@ export const useAuthStore = create<AuthState>()(
username: nextAccount.username, username: nextAccount.username,
client: nextClient, client: nextClient,
authMode: nextAccount.authMode, authMode: nextAccount.authMode,
rememberMe: nextAccount.rememberMe,
connectionLost: false, connectionLost: false,
error: null, error: null,
activeAccountId: nextAccount.id, activeAccountId: nextAccount.id,
identities: restoredIdentities,
primaryIdentity: restoredPrimary,
}); });
if (!restored) { if (!restored) {
@@ -509,13 +616,32 @@ export const useAuthStore = create<AuthState>()(
const { identities, primaryIdentity } = loadIdentities(rawIds, nextAccount.username); const { identities, primaryIdentity } = loadIdentities(rawIds, nextAccount.username);
set({ identities, primaryIdentity }); set({ identities, primaryIdentity });
}).catch((err) => debug.error('Failed to load identities after switch:', err)); }).catch((err) => debug.error('Failed to load identities after switch:', err));
} else {
const identityState = useIdentityStore.getState();
set({
identities: identityState.identities,
primaryIdentity: identityState.identities[0] ?? null,
});
} }
} else {
// Could not restore the next account — remove it and do a full logout
debug.error(`Cannot restore next account ${nextAccount.id}, performing full logout`);
evictAccount(nextAccount.id);
accountStore.removeAccount(nextAccount.id);
set({
isAuthenticated: false,
serverUrl: null,
username: null,
client: null,
identities: [],
primaryIdentity: null,
authMode: 'basic',
rememberMe: false,
accessToken: null,
tokenExpiresAt: null,
connectionLost: false,
error: null,
activeAccountId: null,
});
localStorage.removeItem('auth-storage');
clearAllStores();
redirectToLogin();
} }
} else { } else {
// No accounts remaining — full logout // No accounts remaining — full logout
@@ -540,28 +666,51 @@ export const useAuthStore = create<AuthState>()(
} }
// Clean up cookies for the removed account // Clean up cookies for the removed account
fetch(`/api/auth/session?slot=${slot}`, { method: 'DELETE' }).catch((err) => { fetch(`/api/auth/session?slot=${slot}`, { method: 'DELETE', keepalive: shouldRedirectToLogin }).catch((err) => {
debug.error('Failed to clear session cookie:', err); debug.error('Failed to clear session cookie:', err);
}); });
if (wasOAuth) { if (wasOAuth && shouldRedirectToLogin) {
fetch(`/api/auth/token?slot=${slot}`, { method: 'DELETE' }) let redirectCommitted = false;
const commitLoginRedirect = () => {
if (redirectCommitted) return;
redirectCommitted = true;
redirectToLogin();
};
window.setTimeout(commitLoginRedirect, 0);
fetch(`/api/auth/token?slot=${slot}`, { method: 'DELETE', keepalive: true })
.then((res) => { .then((res) => {
if (!res.ok) throw new Error(`Revocation failed: ${res.status}`); if (!res.ok) throw new Error(`Revocation failed: ${res.status}`);
return res.json(); return res.json();
}) })
.then((data) => { .then((data) => {
if (data.end_session_url && remainingAccounts.length === 0) { if (redirectCommitted) return;
if (data.end_session_url) {
redirectCommitted = true;
const locale = window.location.pathname.split('/')[1] || 'en'; const locale = window.location.pathname.split('/')[1] || 'en';
const redirectUri = `${window.location.origin}/${locale}/login`; const redirectUri = `${window.location.origin}/${locale}/login`;
const url = new URL(data.end_session_url); const url = new URL(data.end_session_url);
url.searchParams.set('post_logout_redirect_uri', redirectUri); url.searchParams.set('post_logout_redirect_uri', redirectUri);
window.location.href = url.toString(); replaceWindowLocation(url.toString());
return;
} }
commitLoginRedirect();
}) })
.catch((err) => { .catch((err) => {
debug.error('OAuth logout cleanup failed:', err); debug.error('OAuth logout cleanup failed:', err);
commitLoginRedirect();
}); });
} else if (wasOAuth) {
fetch(`/api/auth/token?slot=${slot}`, { method: 'DELETE', keepalive: false })
.catch((err) => {
debug.error('OAuth logout cleanup failed:', err);
});
} else if (shouldRedirectToLogin) {
redirectToLogin();
} }
}, },
@@ -604,8 +753,9 @@ export const useAuthStore = create<AuthState>()(
} }
// Delete all cookies // Delete all cookies
fetch('/api/auth/session?all=true', { method: 'DELETE' }).catch(() => {}); fetch('/api/auth/session?all=true', { method: 'DELETE', keepalive: true }).catch(() => {});
fetch('/api/auth/token?all=true', { method: 'DELETE' }).catch(() => {}); fetch('/api/auth/token?all=true', { method: 'DELETE', keepalive: true }).catch(() => {});
redirectToLogin();
}, },
switchAccount: async (accountId: string) => { switchAccount: async (accountId: string) => {
@@ -616,7 +766,9 @@ export const useAuthStore = create<AuthState>()(
const targetAccount = accountStore.getAccountById(accountId); const targetAccount = accountStore.getAccountById(accountId);
if (!targetAccount) return; if (!targetAccount) return;
set({ isLoading: true }); // Null out the client immediately so the page doesn't fire data-loading
// effects with the old client while stores are being cleared.
set({ isLoading: true, client: null });
// Snapshot current account // Snapshot current account
if (state.activeAccountId) { if (state.activeAccountId) {
@@ -666,18 +818,39 @@ export const useAuthStore = create<AuthState>()(
} }
} catch (err) { } catch (err) {
debug.error(`Failed to restore client for ${accountId}:`, err); debug.error(`Failed to restore client for ${accountId}:`, err);
accountStore.updateAccount(accountId, {
isConnected: false,
hasError: true,
errorMessage: err instanceof Error ? err.message : 'Connection failed',
});
set({ isLoading: false });
return;
} }
} }
if (!targetClient) { if (!targetClient) {
// Cannot restore — remove the stale account and redirect to login
evictAccount(accountId);
accountStore.removeAccount(accountId);
fetch(`/api/auth/session?slot=${targetAccount.cookieSlot}`, { method: 'DELETE' }).catch(() => {});
// Restore the previous account if still available
if (state.activeAccountId && state.activeAccountId !== accountId) {
const prevClient = clients.get(state.activeAccountId);
const prevAccount = accountStore.getAccountById(state.activeAccountId);
if (prevClient && prevAccount) {
restoreAccount(state.activeAccountId);
accountStore.setActiveAccount(state.activeAccountId);
set({
isLoading: false,
serverUrl: prevAccount.serverUrl,
username: prevAccount.username,
client: prevClient,
authMode: prevAccount.authMode,
rememberMe: prevAccount.rememberMe,
connectionLost: false,
activeAccountId: state.activeAccountId,
});
return;
}
}
set({ isLoading: false }); set({ isLoading: false });
// Redirect to login so the user can re-authenticate
replaceWindowLocation(getLocaleLoginPath());
return; return;
} }
@@ -686,6 +859,10 @@ export const useAuthStore = create<AuthState>()(
accountStore.setActiveAccount(accountId); accountStore.setActiveAccount(accountId);
accountStore.updateAccount(accountId, { isConnected: true, hasError: false, errorMessage: undefined }); accountStore.updateAccount(accountId, { isConnected: true, hasError: false, errorMessage: undefined });
// Build identity state up front so the name updates atomically
const restoredIdentities = restored ? useIdentityStore.getState().identities : [];
const restoredPrimary = restoredIdentities[0] ?? null;
set({ set({
isAuthenticated: true, isAuthenticated: true,
isLoading: false, isLoading: false,
@@ -693,9 +870,12 @@ export const useAuthStore = create<AuthState>()(
username: targetAccount.username, username: targetAccount.username,
client: targetClient, client: targetClient,
authMode: targetAccount.authMode, authMode: targetAccount.authMode,
rememberMe: targetAccount.rememberMe,
connectionLost: false, connectionLost: false,
error: null, error: null,
activeAccountId: accountId, activeAccountId: accountId,
identities: restoredIdentities,
primaryIdentity: restoredPrimary,
}); });
if (!restored) { if (!restored) {
@@ -707,12 +887,6 @@ export const useAuthStore = create<AuthState>()(
} catch (err) { } catch (err) {
debug.error(`Failed to load data for ${accountId}:`, err); debug.error(`Failed to load data for ${accountId}:`, err);
} }
} else {
const identityState = useIdentityStore.getState();
set({
identities: identityState.identities,
primaryIdentity: identityState.identities[0] ?? null,
});
} }
// Sync settings // Sync settings
@@ -730,7 +904,9 @@ export const useAuthStore = create<AuthState>()(
// Multi-account restoration: restore all registered accounts // Multi-account restoration: restore all registered accounts
if (accounts.length > 0) { if (accounts.length > 0) {
set({ isLoading: true }); // Null out client so the page doesn't fire data-loading effects
// with a stale client reference while we're restoring accounts.
set({ isLoading: true, client: null });
// Determine which account to activate first // Determine which account to activate first
const defaultAccount = accountStore.getDefaultAccount(); const defaultAccount = accountStore.getDefaultAccount();
@@ -784,11 +960,11 @@ export const useAuthStore = create<AuthState>()(
} }
} catch (err) { } catch (err) {
debug.error(`Failed to restore account ${account.id}:`, err); debug.error(`Failed to restore account ${account.id}:`, err);
accountStore.updateAccount(account.id, { // Remove unrestorable accounts so the user is prompted to log in
isConnected: false, // again rather than seeing a stale error entry forever.
hasError: true, evictAccount(account.id);
errorMessage: err instanceof Error ? err.message : 'Restore failed', accountStore.removeAccount(account.id);
}); fetch(`/api/auth/session?slot=${account.cookieSlot}`, { method: 'DELETE' }).catch(() => {});
} }
} }
@@ -809,6 +985,7 @@ export const useAuthStore = create<AuthState>()(
identities, identities,
primaryIdentity, primaryIdentity,
authMode: targetAccount.authMode, authMode: targetAccount.authMode,
rememberMe: targetAccount.rememberMe,
connectionLost: false, connectionLost: false,
error: null, error: null,
activeAccountId: targetId, activeAccountId: targetId,
@@ -840,6 +1017,7 @@ export const useAuthStore = create<AuthState>()(
identities, identities,
primaryIdentity, primaryIdentity,
authMode: acc.authMode, authMode: acc.authMode,
rememberMe: acc.rememberMe,
connectionLost: false, connectionLost: false,
error: null, error: null,
activeAccountId: id, activeAccountId: id,
+4 -18
View File
@@ -110,10 +110,12 @@ export const useCalendarStore = create<CalendarStore>()(
fetchEvents: async (client, start, end) => { fetchEvents: async (client, start, end) => {
set({ isLoadingEvents: true, error: null }); set({ isLoadingEvents: true, error: null });
try { try {
const events = await client.queryAllCalendarEvents({ const rawEvents = await client.queryAllCalendarEvents({
after: start, after: start,
before: end, before: end,
}); });
// Filter out malformed events missing required 'start' field
const events = rawEvents.filter(e => typeof e.start === 'string' && e.start);
set({ events, isLoadingEvents: false, dateRange: { start, end } }); set({ events, isLoadingEvents: false, dateRange: { start, end } });
} catch (error) { } catch (error) {
debug.error('Failed to fetch events:', error); debug.error('Failed to fetch events:', error);
@@ -142,13 +144,6 @@ export const useCalendarStore = create<CalendarStore>()(
} }
const created = await client.createCalendarEvent(cleanEvent, sendSchedulingMessages, targetAccountId); const created = await client.createCalendarEvent(cleanEvent, sendSchedulingMessages, targetAccountId);
set((state) => ({ events: [...state.events, created] })); set((state) => ({ events: [...state.events, created] }));
if (sendSchedulingMessages && created.participants) {
try {
await client.sendImipInvitation(created);
} catch (e) {
debug.error('Failed to send invitation emails:', e);
}
}
return created; return created;
} catch (error) { } catch (error) {
debug.error('Failed to create event:', error); debug.error('Failed to create event:', error);
@@ -178,16 +173,6 @@ export const useCalendarStore = create<CalendarStore>()(
set((state) => ({ set((state) => ({
events: state.events.map(e => e.id === id ? { ...e, ...updates } : e), events: state.events.map(e => e.id === id ? { ...e, ...updates } : e),
})); }));
if (sendSchedulingMessages) {
try {
const updatedEvent = await client.getCalendarEvent(realId, targetAccountId);
if (updatedEvent?.participants) {
await client.sendImipInvitation(updatedEvent);
}
} catch (e) {
debug.error('Failed to send update notification emails:', e);
}
}
} catch (error) { } catch (error) {
debug.error('Failed to update event:', error); debug.error('Failed to update event:', error);
set({ error: 'Failed to update event' }); set({ error: 'Failed to update event' });
@@ -672,6 +657,7 @@ export const useCalendarStore = create<CalendarStore>()(
return { return {
...mergedState, ...mergedState,
selectedDate: new Date(),
viewMode: getSafeCalendarViewMode(mergedState.viewMode), viewMode: getSafeCalendarViewMode(mergedState.viewMode),
}; };
}, },
+9 -7
View File
@@ -239,15 +239,17 @@ export const useEmailStore = create<EmailStore>((set, get) => ({
try { try {
const mailboxes = await client.getAllMailboxes(); const mailboxes = await client.getAllMailboxes();
// Auto-select inbox if no mailbox is currently selected // Auto-select inbox if no mailbox is selected or the current selection
// doesn't exist in the fetched list (e.g. after an account switch)
const currentSelectedMailbox = get().selectedMailbox; const currentSelectedMailbox = get().selectedMailbox;
if (!currentSelectedMailbox) { const selectionValid = currentSelectedMailbox && mailboxes.some(m => m.id === currentSelectedMailbox);
if (!selectionValid) {
// Find inbox from PRIMARY account (not shared accounts) // Find inbox from PRIMARY account (not shared accounts)
const inboxMailbox = mailboxes.find(m => m.role === 'inbox' && !m.isShared); const inboxMailbox = mailboxes.find(m => m.role === 'inbox' && !m.isShared);
if (inboxMailbox) { if (inboxMailbox) {
set({ mailboxes, selectedMailbox: inboxMailbox.id, isLoading: false }); set({ mailboxes, selectedMailbox: inboxMailbox.id, isLoading: false });
} else { } else {
set({ mailboxes, isLoading: false }); set({ mailboxes, selectedMailbox: '', isLoading: false });
} }
} else { } else {
set({ mailboxes, isLoading: false }); set({ mailboxes, isLoading: false });
@@ -1298,11 +1300,11 @@ export const useEmailStore = create<EmailStore>((set, get) => ({
setMailboxRole: async (client, mailboxId, role) => { setMailboxRole: async (client, mailboxId, role) => {
try { try {
// If assigning a role, first clear that role from any other mailbox // If assigning a role, first clear that role from ALL other mailboxes that have it
if (role) { if (role) {
const existingMailbox = get().mailboxes.find(mb => mb.role === role && !mb.isShared); const existingMailboxes = get().mailboxes.filter(mb => mb.role === role && !mb.isShared && mb.id !== mailboxId);
if (existingMailbox && existingMailbox.id !== mailboxId) { for (const existing of existingMailboxes) {
await client.updateMailbox(existingMailbox.id, { role: null }); await client.updateMailbox(existing.id, { role: null });
} }
} }
await client.updateMailbox(mailboxId, { role }); await client.updateMailbox(mailboxId, { role });
+4
View File
@@ -29,6 +29,7 @@ export type TimeFormat = '12h' | '24h';
export type FirstDayOfWeek = 0 | 1; // 0 = Sunday, 1 = Monday export type FirstDayOfWeek = 0 | 1; // 0 = Sunday, 1 = Monday
export type ExternalContentPolicy = 'ask' | 'block' | 'allow'; export type ExternalContentPolicy = 'ask' | 'block' | 'allow';
export type MailAttachmentAction = 'preview' | 'download'; export type MailAttachmentAction = 'preview' | 'download';
export type AttachmentPosition = 'beside-sender' | 'below-header';
export type ToolbarPosition = 'top' | 'below-subject'; export type ToolbarPosition = 'top' | 'below-subject';
export type ArchiveMode = 'single' | 'year' | 'month'; export type ArchiveMode = 'single' | 'year' | 'month';
@@ -92,6 +93,7 @@ interface SettingsState {
emailsPerPage: number; emailsPerPage: number;
externalContentPolicy: ExternalContentPolicy; externalContentPolicy: ExternalContentPolicy;
mailAttachmentAction: MailAttachmentAction; mailAttachmentAction: MailAttachmentAction;
attachmentPosition: AttachmentPosition;
emailAlwaysLightMode: boolean; // Always render email content in light mode emailAlwaysLightMode: boolean; // Always render email content in light mode
archiveMode: ArchiveMode; // How to organize archived emails: single folder, by year, or by year+month archiveMode: ArchiveMode; // How to organize archived emails: single folder, by year, or by year+month
@@ -186,6 +188,7 @@ const DEFAULT_SETTINGS = {
emailsPerPage: 50, emailsPerPage: 50,
externalContentPolicy: 'ask' as ExternalContentPolicy, externalContentPolicy: 'ask' as ExternalContentPolicy,
mailAttachmentAction: 'preview' as MailAttachmentAction, mailAttachmentAction: 'preview' as MailAttachmentAction,
attachmentPosition: 'beside-sender' as AttachmentPosition,
emailAlwaysLightMode: false, emailAlwaysLightMode: false,
archiveMode: 'single' as ArchiveMode, archiveMode: 'single' as ArchiveMode,
@@ -271,6 +274,7 @@ export const useSettingsStore = create<SettingsState>()(
emailsPerPage: state.emailsPerPage, emailsPerPage: state.emailsPerPage,
externalContentPolicy: state.externalContentPolicy, externalContentPolicy: state.externalContentPolicy,
mailAttachmentAction: state.mailAttachmentAction, mailAttachmentAction: state.mailAttachmentAction,
attachmentPosition: state.attachmentPosition,
archiveMode: state.archiveMode, archiveMode: state.archiveMode,
trustedSenders: state.trustedSenders, trustedSenders: state.trustedSenders,
autoSaveDraftInterval: state.autoSaveDraftInterval, autoSaveDraftInterval: state.autoSaveDraftInterval,