@rapidmx/web-client 0.5.0 → 0.6.0

package/dist/apps/shared/search/localIndexWorker.d.ts

@@ -0,0 +1,145 @@
+import type { ParsedSearchQuery } from "@rapidmx/react-shared/search/queryGrammar.js";
+import { LocalIndexEntity } from "./localIndexSchema.js";
+/** One request sent from the main thread. `id` correlates the eventual response - `postMessage` has no
+ * native request/response pairing, so every RPC call carries its own id (see `localIndexRpcClient.ts`,
+ * the main-thread counterpart that generates/awaits these). */
+export interface LocalIndexRequest {
+    id: number;
+    method: "init" | "indexEntities" | "removeEntity" | "moveEntity" | "indexedVersions" | "pruneEntities" | "search" | "coverage" | "setWindow" | "setBuilding" | "destroy" | "destroyAll" | "selfTest" | "ping";
+    params?: unknown;
+}
+export type LocalIndexResponse = {
+    id: number;
+    ok: true;
+    result: unknown;
+} | {
+    id: number;
+    ok: false;
+    error: string;
+};
+/**
+ * Carried by every call a build pass makes (`init`, `setWindow`, `setBuilding`, `indexEntities`,
+ * `pruneEntities`). Generations come from one monotonic counter on the main thread
+ * (`localIndexRpcClient.ts`), shared with `destroy`/`destroyAll`: a call whose generation is older than the
+ * last destroy of its mailbox is rejected (`StaleGenerationError`), so a build started before a lock/sign-out
+ * can't recreate the index with its captured keys or write into a rebuild that started after it. Calls
+ * without a generation (search, coverage, in-session moves) are never rejected.
+ */
+export interface GenerationParams {
+    generation?: number;
+}
+export interface InitParams extends GenerationParams {
+    mailboxUid: string;
+    /** The mailbox's already-derived local-index key (`localIndexKey.ts`'s `deriveLocalIndexKey()`) -
+     * this Worker never touches the master key itself, only this one purpose-derived value, matching
+     * every other MK-derived-key boundary already established elsewhere in this codebase. */
+    indexKey: Uint8Array;
+}
+export interface IndexEntitiesParams extends GenerationParams {
+    mailboxUid: string;
+    entities: LocalIndexEntity[];
+}
+export interface IndexEntitiesResult {
+    /** `true` when this call's eviction pass had to delete anything to get back under the byte budget. */
+    budgetReached: boolean;
+    /** The eviction watermark after this call - see `WindowState.evictedBefore`. */
+    evictedBefore?: string;
+}
+export interface WindowState {
+    /** Set once the byte budget has forced an eviction: messages dated strictly before this were evicted (or
+     * would be, on insert), so a build pass neither re-fetches them nor walks past them. Cleared by
+     * `setWindow` once the index has comfortably shrunk back under budget (or the budget was raised). */
+    evictedBefore?: string;
+}
+export interface DestroyParams extends GenerationParams {
+    mailboxUid: string;
+}
+export interface RemoveEntityParams {
+    mailboxUid: string;
+    entityUid: string;
+}
+export interface MoveEntityParams {
+    mailboxUid: string;
+    entityUid: string;
+    folderUid: string;
+}
+export interface IndexedVersionsParams {
+    mailboxUid: string;
+    entityUids: string[];
+}
+export interface PruneEntitiesParams extends GenerationParams {
+    mailboxUid: string;
+    /** Every entity uid a build pass saw on the server. */
+    keepEntityUids: string[];
+    /** Only rows at or after this `date_for_sort` are candidates (the walk's own time floor - anything
+     * older was never re-listed, so its absence proves nothing). `undefined` means no floor. */
+    since?: string;
+    /** Only rows currently filed in one of these folders are candidates - folders whose listing was reliable
+     * for the whole walk. `undefined` means every folder. */
+    folderUids?: string[];
+}
+export interface SearchParams {
+    mailboxUid: string;
+    parsed: ParsedSearchQuery;
+    limit: number;
+    /** How many matches to skip before returning `limit` more - §8's composite pagination cursor's own
+     * "the local index position consumed so far" component, threaded straight through to SQL `OFFSET`.
+     * Defaults to `0` for a first page. */
+    offset?: number;
+}
+export interface LocalSearchHit {
+    entityUid: string;
+    /** Raw `bm25()` score - more negative is a better match, per SQLite's own convention. Normalized by
+     * `searchTier2.ts` (main thread) before merging (spec §7). */
+    score: number;
+    snippet?: string;
+}
+export interface LocalSearchPage {
+    hits: LocalSearchHit[];
+    /** `true` when at least one more match exists beyond this page - detected by fetching `limit + 1`
+     * rows and trimming the extra one, rather than a separate `COUNT(*)` query. */
+    hasMore: boolean;
+}
+export interface Coverage {
+    /** The oldest date the index is known to cover. While `complete`, every indexable message at least
+     * this recent is present; otherwise it is only the oldest row that happens to be present. `undefined`
+     * for an empty index. */
+    indexedFrom?: string;
+    indexedCount: number;
+    building: boolean;
+    /** `true` only once a build pass *in this connection's lifetime* has walked every mail folder all the
+     * way back to its time floor with no errors and no budget cut-off - reset whenever the index is opened,
+     * so a completion persisted by an earlier session never counts. Callers MUST NOT treat `indexedFrom` as
+     * a coverage guarantee (e.g. to narrow Tier 3) unless this is `true` and `building` is `false`. */
+    complete: boolean;
+    /** Only set while `complete`: the guarantee ends here. Nothing indexes mail that arrives after a pass
+     * started, so everything dated after this (ISO) is NOT covered and must still be searched elsewhere. */
+    indexedUntil?: string;
+}
+export interface SetWindowParams extends GenerationParams {
+    mailboxUid: string;
+    timeFloorMonths: number;
+    byteBudgetBytes: number;
+}
+export interface SetBuildingParams extends GenerationParams {
+    mailboxUid: string;
+    building: boolean;
+    /** Only meaningful with `building: false` - whether the pass that just ended walked everything. */
+    complete?: boolean;
+    /** Only meaningful with `complete: true` - the pass's time floor (ISO), or `undefined` for none. */
+    coveredFrom?: string;
+    /** Only meaningful with `complete: true` - when the pass started (ISO, minus a clock-skew margin). */
+    coveredUntil?: string;
+}
+/** Runs `task` after every previously queued task for `mailboxUid` has settled. Exported for tests. */
+export declare function runExclusive<T>(mailboxUid: string, task: () => Promise<T>): Promise<T>;
+/** Rejects a build-pass call issued before its mailbox's index was last destroyed. */
+export declare class StaleGenerationError extends Error {
+    constructor(mailboxUid: string);
+}
+/** Thrown by `init` when another tab already holds this mailbox's index open. */
+export declare class LocalIndexBusyError extends Error {
+    constructor(mailboxUid: string);
+}
+/** Routes one request to its handler, inside the owning mailbox's serial queue. Exported for tests. */
+export declare function handleRequest(method: LocalIndexRequest["method"], params: unknown): Promise<unknown>;

package/dist/apps/shared/search/searchTier2.d.ts

@@ -0,0 +1,30 @@
+/**
+ * Tier 2 (`specs/search.md` §6) - the local encrypted index's public entry point, mirroring
+ * `@rapidmx/react-shared`'s `searchTier3.ts#searchEncryptedCandidates()` in shape: given a parsed query
+ * and this session's unlocked keys, return already-scored, already-sourced results plus how much of the
+ * mailbox the local index actually covers right now.
+ *
+ * Returns `{ results: [], coverage: undefined }` - never throws, never rejects - both when `unlocked` is
+ * absent (nothing is indexed without a master key to derive the index key from, the same "silently
+ * contributes nothing" degradation `searchTier3.ts`'s own doc comment describes for its identical case)
+ * and when anything about the local index itself fails (Worker/WASM/OPFS unsupported or unavailable in
+ * this browser, a corrupted index mid-rebuild, a query timeout). This tier's own storage engine is
+ * strictly best-effort infrastructure Tier 1 doesn't depend on - a caller `Promise.all()`-ing this
+ * alongside Tier 1/Tier 3 (`apps/www/index.tsx`'s own search orchestration) must never see the *whole
+ * search* fail just because this one, most novel piece had a bad moment.
+ */
+import type { UnlockedKeys } from "@rapidmx/react-shared/crypto/keySession.js";
+import type { ParsedSearchQuery } from "@rapidmx/react-shared/search/queryGrammar.js";
+import type { SearchResult } from "@rapidmx/react-shared/search/searchApi.js";
+import type { Coverage } from "./localIndexWorker.js";
+export interface Tier2SearchOutcome {
+    results: SearchResult[];
+    /** `undefined` when `unlocked` was absent - there's no local index to report coverage for at all in
+     * that case, distinct from a real, empty (just-built) index. */
+    coverage?: Coverage;
+    /** `true` when at least one more match exists beyond `results` - §8's composite pagination cursor's
+     * own per-tier continuation signal. Always `false` alongside the `unlocked`-absent/error
+     * degradations below - there is nothing more to page through either way. */
+    hasMore: boolean;
+}
+export declare function searchLocalIndex(mailboxUid: string, parsed: ParsedSearchQuery, unlocked: UnlockedKeys | undefined, limit?: number, offset?: number): Promise<Tier2SearchOutcome>;

package/dist/apps/www/_404.d.ts

@@ -0,0 +1,2 @@
+import React from "react";
+export default function NotFoundPage(): React.JSX.Element;

package/dist/apps/www/_500.d.ts

@@ -0,0 +1,2 @@
+import React from "react";
+export default function ErrorPage(): React.JSX.Element;

package/dist/apps/www/_layout.d.ts

@@ -0,0 +1,14 @@
+import React, { PropsWithChildren } from "react";
+import { Branding } from "@rapidmx/react-shared/branding/brandingApi.js";
+export interface LayoutProps {
+    /**
+     * Supplied by `WwwRoute`'s `fetchProps()` override (`src/mongo/routes/wwwRoute.ts`/
+     * `src/sql/routes/wwwRoute.ts`) - `@rapidrest/react` spreads the same merged page props onto this
+     * layout as onto the page it wraps, so it renders correctly server-side on the very first byte of the
+     * response: no client-side flash from the stock defaults below, and a crawler reading the raw HTML
+     * sees the real branding too. `undefined` only on the framework's unwrapped `_500` fallback path,
+     * which doesn't receive props at all - falling back to the stock defaults there is acceptable.
+     */
+    branding?: Branding;
+}
+export default function Layout({ children, branding }: PropsWithChildren<LayoutProps>): React.JSX.Element;

package/dist/apps/www/calendar/index.d.ts

@@ -0,0 +1,3 @@
+import React from "react";
+import { CalendarShellProps } from "../../shared/components/calendar/layout/CalendarShell.js";
+export default function CalendarPage(props: CalendarShellProps): React.JSX.Element;

package/dist/apps/www/contacts/[uid].d.ts

@@ -0,0 +1,13 @@
+import React from "react";
+import { ContactsShellProps } from "../../shared/components/contacts/layout/ContactsShell.js";
+/**
+ * Only reached on mobile (below the `md` breakpoint) — desktop's `apps/www/contacts/index.tsx` keeps its
+ * existing inline detail pane and never navigates here; see that file's `handleSelectRow`. There is no
+ * equivalent "new contact" route — an unsaved contact has no uid to route on, so creation stays an
+ * in-place mode-switch on every device (see `apps/www/contacts/index.tsx`'s `handleNew`).
+ */
+export default function ContactDetailPage(props: ContactsShellProps & {
+    params: {
+        uid: string;
+    };
+}): React.JSX.Element;

package/dist/apps/www/contacts/index.d.ts

@@ -0,0 +1,3 @@
+import React from "react";
+import { ContactsShellProps } from "../../shared/components/contacts/layout/ContactsShell.js";
+export default function ContactsPage(props: ContactsShellProps): React.JSX.Element;

package/dist/apps/www/index.d.ts

@@ -0,0 +1,3 @@
+import React from "react";
+import { MailShellProps } from "../shared/components/mail/layout/MailShell.js";
+export default function InboxPage(props: MailShellProps): React.JSX.Element;

package/dist/apps/www/messages/[uid].d.ts

@@ -0,0 +1,11 @@
+import React from "react";
+import { MailShellProps } from "../../shared/components/mail/layout/MailShell.js";
+/**
+ * Only reached on mobile (below the `md` breakpoint) — desktop's `apps/www/index.tsx` keeps its existing
+ * inline reading pane and never navigates here; see that file's `handleSelect`.
+ */
+export default function MessageDetailPage(props: MailShellProps & {
+    params: {
+        uid: string;
+    };
+}): React.JSX.Element;

package/dist/apps/www/settings/auto-reply/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsAutoReplyPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsAutoReplyPage(props: SettingsAutoReplyPageProps): React.JSX.Element;

package/dist/apps/www/settings/encryption/index.d.ts

@@ -0,0 +1,7 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+/** Mirrors restapi's `BaseKeyVaultRoute` `MAX_MASTER_KEY_WRAPS` - `addMasterKeyWrap()` refuses a vault
+ * already holding this many wraps (escrow included). */
+export declare const MAX_MASTER_KEY_WRAPS = 20;
+export type SettingsEncryptionPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsEncryptionPage(props: SettingsEncryptionPageProps): React.JSX.Element;

package/dist/apps/www/settings/filters/[uid].d.ts

@@ -0,0 +1,8 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type MailFilterDetailPageProps = Omit<SettingsShellProps, "active"> & {
+    params: {
+        uid: string;
+    };
+};
+export default function MailFilterDetailPage(props: MailFilterDetailPageProps): React.JSX.Element;

package/dist/apps/www/settings/filters/_mailFilterRuleConfig.d.ts

@@ -0,0 +1,11 @@
+import { Folder } from "@rapidmx/react-shared/mail/mailApi.js";
+import { MailFilterAction } from "@rapidmx/react-shared/mail/mailFilterRulesApi.js";
+import { ActionTypeDef, ConditionFieldDef } from "../../../shared/components/rules/RuleBuilder.js";
+export declare const MAIL_FILTER_CONDITION_FIELDS: ConditionFieldDef[];
+/**
+ * Builds the mail-filter action-type registry for `RuleBuilder`, closing over `folders` so the
+ * move/copy-to-folder actions can render a real folder picker. `RuleBuilder`'s own `render()` contract
+ * is synchronous, so this resolves `folders` once up front (the caller already loads them for the page
+ * itself) rather than each action row fetching independently.
+ */
+export declare function buildMailFilterActionTypes(folders: Folder[]): ActionTypeDef<MailFilterAction>[];

package/dist/apps/www/settings/filters/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsFiltersPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsFiltersPage(props: SettingsFiltersPageProps): React.JSX.Element;

package/dist/apps/www/settings/filters/new/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../../shared/components/settings/layout/SettingsShell.js";
+export type NewMailFilterPageProps = Omit<SettingsShellProps, "active">;
+export default function NewMailFilterPage(props: NewMailFilterPageProps): React.JSX.Element;

package/dist/apps/www/settings/focused-inbox/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsFocusedInboxPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsFocusedInboxPage(props: SettingsFocusedInboxPageProps): React.JSX.Element;

package/dist/apps/www/settings/labels/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsLabelsPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsLabelsPage(props: SettingsLabelsPageProps): React.JSX.Element;

package/dist/apps/www/settings/privacy/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsPrivacyPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsPrivacyPage(props: SettingsPrivacyPageProps): React.JSX.Element;

package/dist/apps/www/settings/read-receipts/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsReadReceiptsPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsReadReceiptsPage(props: SettingsReadReceiptsPageProps): React.JSX.Element;

package/dist/apps/www/settings/sharing/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsSharingPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsSharingPage(props: SettingsSharingPageProps): React.JSX.Element;

package/dist/apps/www/settings/signatures/[uid].d.ts

@@ -0,0 +1,8 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SignatureDetailPageProps = Omit<SettingsShellProps, "active"> & {
+    params: {
+        uid: string;
+    };
+};
+export default function SignatureDetailPage(props: SignatureDetailPageProps): React.JSX.Element;

package/dist/apps/www/settings/signatures/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../shared/components/settings/layout/SettingsShell.js";
+export type SettingsSignaturesPageProps = Omit<SettingsShellProps, "active">;
+export default function SettingsSignaturesPage(props: SettingsSignaturesPageProps): React.JSX.Element;

package/dist/apps/www/settings/signatures/new/index.d.ts

@@ -0,0 +1,4 @@
+import React from "react";
+import { SettingsShellProps } from "../../../../shared/components/settings/layout/SettingsShell.js";
+export type NewSignaturePageProps = Omit<SettingsShellProps, "active">;
+export default function NewSignaturePage(props: NewSignaturePageProps): React.JSX.Element;

package/dist/apps/www/settings/signatures/signatureDefaults.d.ts

@@ -0,0 +1,10 @@
+import { MailSignature } from "@rapidmx/react-shared/mail/mailSignaturesApi.js";
+/**
+ * `MailSignature.isDefaultForNewMessages`/`isDefaultForReplyForward` are each meant to hold on at most
+ * one signature per mailbox — enforced by convention, not a database constraint (see
+ * `mailSignaturesApi.ts`'s own doc comment, quoting restapi's). Before saving a signature with one of
+ * these flags newly turned on, this turns the flag off on whichever *other* signature in `signatures`
+ * currently has it — the "the composing client toggles the previous default off" responsibility that
+ * convention assigns to a caller like the New/Edit signature forms.
+ */
+export declare function clearPreviousDefaults(signatures: MailSignature[], savingUid: string | undefined, isDefaultForNewMessages: boolean, isDefaultForReplyForward: boolean): Promise<void>;

package/dist/apps/www/tasks/index.d.ts

@@ -0,0 +1,3 @@
+import React from "react";
+import { TasksShellProps } from "../../shared/components/tasks/layout/TasksShell.js";
+export default function TasksPage(props: TasksShellProps): React.JSX.Element;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rapidmx/web-client",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "RapidMX's webmail and admin console React UI for web-based clients.",
   "type": "module",
   "repository": "https://github.com/rapidmx/web-client",
@@ -17,7 +17,7 @@
     "./shared/styles/app.css": "./apps/shared/styles/app.css"
   },
   "scripts": {
-    "build": "rimraf dist && tsc -p tsconfig.json && shx mkdir -p dist/apps/shared/styles && shx cp apps/shared/styles/app.css dist/apps/shared/styles/app.css",
+    "build": "rimraf dist && tsc -p tsconfig.json && shx mkdir -p dist/apps/shared/styles && shx cp apps/shared/styles/app.css dist/apps/shared/styles/app.css && node scripts/checkDistReferences.mjs",
     "lint": "eslint ./apps ./test",
     "test": "vitest run --coverage",
     "test:watch": "vitest --watch --coverage",
@@ -39,7 +39,7 @@
     "@dnd-kit/core": "^6.3.1",
     "@journeyapps/wa-sqlite": "^2.0.4",
     "@peculiar/x509": "^2.1.0",
-    "@rapidmx/react-shared": "patch:@rapidmx/react-shared@npm%3A0.4.0#~/.yarn/patches/@rapidmx-react-shared-npm-0.4.0-e713beecf2.patch",
+    "@rapidmx/react-shared": "^0.6.0",
     "@tiptap/core": "^3.31.3",
     "@tiptap/extension-highlight": "^3.31.3",
     "@tiptap/extension-image": "^3.31.3",

package/apps/shared/components/booking/AvailabilityEditor.tsx

@@ -1,118 +0,0 @@
-///////////////////////////////////////////////////////////////////////////////
-// Copyright (C) 2026 Jean-Philippe Steinmetz
-// SPDX-License-Identifier: MPL-2.0
-///////////////////////////////////////////////////////////////////////////////
-import React, { useState } from "react";
-import { BookingAvailabilityWindow } from "@rapidmx/react-shared/booking/bookingApi.js";
-import Button from "@rapidmx/react-shared/components/buttons/Button.js";
-
-const DAY_LABELS = ["Sunday", "Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday"];
-const INPUT_CLASS =
-    "text-sm py-2 px-3 border border-border rounded-sm bg-surface text-text focus:outline-none focus:border-primary";
-
-function minutesToTimeValue(minutes: number): string {
-    const h = Math.floor(minutes / 60)
-        .toString()
-        .padStart(2, "0");
-    const m = (minutes % 60).toString().padStart(2, "0");
-    return `${h}:${m}`;
-}
-
-function timeValueToMinutes(value: string): number {
-    const [h, m] = value.split(":").map(Number);
-    return h * 60 + m;
-}
-
-export interface AvailabilityEditorProps {
-    value: BookingAvailabilityWindow[];
-    onChange: (value: BookingAvailabilityWindow[]) => void;
-}
-
-/**
- * Adds/removes `BookingType.availability` weekly windows — deliberately does not cover `dateOverrides`
- * (per-date blackouts/exceptions), a separate, less commonly needed escape hatch left for a future pass;
- * `BookingUtils.generateCandidateSlots()` already handles an empty `dateOverrides` array gracefully.
- */
-export default function AvailabilityEditor({ value, onChange }: AvailabilityEditorProps) {
-    const [dayOfWeek, setDayOfWeek] = useState(1);
-    const [startTime, setStartTime] = useState("09:00");
-    const [endTime, setEndTime] = useState("17:00");
-    const [error, setError] = useState<string | null>(null);
-
-    function handleAdd() {
-        const startMinute = timeValueToMinutes(startTime);
-        const endMinute = timeValueToMinutes(endTime);
-        if (startMinute >= endMinute) {
-            setError("Start time must be before end time.");
-            return;
-        }
-        setError(null);
-        onChange([...value, { dayOfWeek, startMinute, endMinute }]);
-    }
-
-    function handleRemove(index: number) {
-        onChange(value.filter((_, i) => i !== index));
-    }
-
-    return (
-        <div className="flex flex-col gap-3">
-            {error && <p className="text-sm text-danger">{error}</p>}
-            {value.length === 0 ? (
-                <p className="text-sm text-text-muted">No availability windows yet — add one below.</p>
-            ) : (
-                <ul className="flex flex-col gap-2">
-                    {value.map((window, index) => (
-                        <li
-                            key={index}
-                            className="flex items-center justify-between gap-3 py-2 px-3 border border-border rounded-sm text-sm"
-                        >
-                            <span>
-                                {DAY_LABELS[window.dayOfWeek]} {minutesToTimeValue(window.startMinute)}&ndash;
-                                {minutesToTimeValue(window.endMinute)}
-                            </span>
-                            <button
-                                type="button"
-                                onClick={() => handleRemove(index)}
-                                className="text-sm text-danger hover:underline"
-                            >
-                                Remove
-                            </button>
-                        </li>
-                    ))}
-                </ul>
-            )}
-            <div className="flex flex-wrap items-center gap-2">
-                <select
-                    aria-label="Day of week"
-                    value={dayOfWeek}
-                    onChange={(e) => setDayOfWeek(Number(e.target.value))}
-                    className={INPUT_CLASS}
-                >
-                    {DAY_LABELS.map((label, i) => (
-                        <option key={label} value={i}>
-                            {label}
-                        </option>
-                    ))}
-                </select>
-                <input
-                    type="time"
-                    aria-label="Start time"
-                    value={startTime}
-                    onChange={(e) => setStartTime(e.target.value)}
-                    className={INPUT_CLASS}
-                />
-                <span className="text-sm text-text-muted">to</span>
-                <input
-                    type="time"
-                    aria-label="End time"
-                    value={endTime}
-                    onChange={(e) => setEndTime(e.target.value)}
-                    className={INPUT_CLASS}
-                />
-                <Button type="button" variant="secondary" className="!w-auto" onClick={handleAdd}>
-                    Add window
-                </Button>
-            </div>
-        </div>
-    );
-}