ci: add llm reviewer

.github/workflows/code-review.yml

@@ -0,0 +1,34 @@
+name: Code Review
+
+permissions:
+  contents: read
+  pull-requests: write
+
+on:
+  pull_request:
+  pull_request_target:
+    types: [opened, synchronize, reopened]
+  pull_request_review_comment:
+    types: [created]
+
+concurrency:
+  group: ${{ github.repository }}-${{ github.event.number || github.head_ref ||
+    github.sha }}-${{ github.workflow }}-${{ github.event_name ==
+    'pull_request_review_comment' && 'pr_comment' || 'pr' }}
+  cancel-in-progress: ${{ github.event_name != 'pull_request_review_comment' }}
+
+jobs:
+  review:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: coderabbitai/ai-pr-reviewer@latest
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+        with:
+          debug: true
+          review_simple_changes: false
+          review_comment_lgtm: false
+          openai_base_url: "https://openrouter.ai/api/v1"
+          openai_light_model: "openai/gpt-5-nano"
+          openai_heavy_model: "meta-llama/llama-3-70b-instruct"

src/core/features/notes/controller/NotesController.test.ts

@@ -392,6 +392,23 @@ describe('data fetching', () => {
 
 		await db.close();
 	});
+
+	test('method getById returns all found notes and ignores missing ones', async () => {
+		const db = await openDatabase(dbFile);
+		const registry = new NotesController(db, FAKE_WORKSPACE_ID);
+
+		const notesId = await registry
+			.get({ limit: 10 })
+			.then((notes) => notes.map((note) => note.id));
+
+		const notExistingNote = getUUID();
+
+		await expect(
+			registry.getById([...notesId, notExistingNote]),
+		).resolves.toHaveLength(10);
+
+		await db.close();
+	});
 });
 
 describe('multi instances', () => {

src/features/App/Profile/index.tsx

@@ -1,10 +1,12 @@
 import React, { createContext, FC, useEffect, useMemo, useState } from 'react';
 import { isEqual } from 'lodash';
+import { Box } from '@chakra-ui/react';
 import { FileController } from '@core/features/files/FileController';
 import { StateFile } from '@core/features/files/StateFile';
 import { LexemesRegistry } from '@core/features/notes/controller/LexemesRegistry';
 import { WorkspacesController } from '@core/features/workspaces/WorkspacesController';
 import { StatusBarProvider } from '@features/MainScreen/StatusBar/StatusBarProvider';
+import { SplashScreen } from '@features/SplashScreen';
 import { GLOBAL_COMMANDS } from '@hooks/commands';
 import { useShortcutsBinding } from '@hooks/commands/shortcuts/useShortcutsBinding';
 import { useCommandCallback } from '@hooks/commands/useCommandCallback';
@@ -14,9 +16,8 @@ import {
 	createWorkspaceObject,
 	defaultVaultConfig,
 	ProfileConfigScheme,
+	selectIsActiveWorkspaceReady,
 	selectWorkspacesInfo,
-	WorkspaceConfigScheme,
-	WorkspaceData,
 	workspacesApi,
 } from '@state/redux/profiles/profiles';
 import { createContextGetterHook } from '@utils/react/createContextGetterHook';
@@ -77,20 +78,6 @@ export const Profile: FC<ProfileProps> = ({ profile: currentProfile, controls })
 		]).then(async ([workspaces, config, state]) => {
 			const [defaultWorkspace] = workspaces;
 
-			const workspaceConfigs = await Promise.all(
-				workspaces.map(async (workspace) => {
-					const config = await new StateFile(
-						new FileController(
-							`workspaces/${workspace.id}/config.json`,
-							controls.profile.files,
-						),
-						WorkspaceConfigScheme,
-					).get();
-
-					return [workspace.id, config] as const;
-				}),
-			).then((entries) => Object.fromEntries(entries));
-
 			if (!defaultWorkspace) return;
 
 			dispatch(
@@ -99,19 +86,10 @@ export const Profile: FC<ProfileProps> = ({ profile: currentProfile, controls })
 					profile: {
 						activeWorkspace: null,
 						workspaces: Object.fromEntries(
-							workspaces.map((workspace) => {
-								const workspaceObject = createWorkspaceObject(workspace);
-								return [
-									workspace.id,
-									{
-										...workspaceObject,
-										config: {
-											...workspaceObject.config,
-											...workspaceConfigs[workspace.id],
-										},
-									} satisfies WorkspaceData,
-								];
-							}),
+							workspaces.map((workspace) => [
+								workspace.id,
+								createWorkspaceObject(workspace),
+							]),
 						),
 						config: {
 							...defaultVaultConfig,
@@ -158,28 +136,39 @@ export const Profile: FC<ProfileProps> = ({ profile: currentProfile, controls })
 		enabled: controls.profile.profile.isEncrypted,
 	});
 
+	const isActiveWorkspaceReady = useAppSelector(
+		selectIsActiveWorkspaceReady({ profileId }),
+	);
+
 	return (
 		<ProfileControlsContext.Provider value={controls}>
+			{!isActiveWorkspaceReady && <SplashScreen />}
 			{workspaces.length > 0 && <ProfileServices />}
-			{workspaces.map((workspace) =>
-				workspace.touched ? (
-					<WorkspaceContext.Provider
-						key={workspace.id}
-						value={{ profileId: profileId, workspaceId: workspace.id }}
-					>
-						<StatusBarProvider>
-							<Workspace profile={currentProfile} />
-							<ProfileStatusBar />
-							{isDevMode && (
-								<ToggleSQLConsole
-									isVisible={isDBConsoleVisible}
-									onVisibilityChange={setIsDBConsoleVisible}
-								/>
-							)}
-						</StatusBarProvider>
-					</WorkspaceContext.Provider>
-				) : null,
-			)}
+			<Box
+				width="100%"
+				height="100vh"
+				display={isActiveWorkspaceReady ? 'flex' : 'none'}
+			>
+				{workspaces.map((workspace) =>
+					workspace.touched ? (
+						<WorkspaceContext.Provider
+							key={workspace.id}
+							value={{ profileId: profileId, workspaceId: workspace.id }}
+						>
+							<StatusBarProvider>
+								<Workspace profile={currentProfile} />
+								<ProfileStatusBar />
+								{isDevMode && (
+									<ToggleSQLConsole
+										isVisible={isDBConsoleVisible}
+										onVisibilityChange={setIsDBConsoleVisible}
+									/>
+								)}
+							</StatusBarProvider>
+						</WorkspaceContext.Provider>
+					) : null,
+				)}
+			</Box>
 			{isDevMode && (
 				<SQLConsole
 					isVisible={isDBConsoleVisible}

src/features/App/Profile/services/useVaultConfigSync.tsx

@@ -1,16 +1,8 @@
 import { useEffect } from 'react';
-import { isEqual } from 'lodash';
 import { FileController } from '@core/features/files/FileController';
 import { StateFile } from '@core/features/files/StateFile';
 import { useWatchSelector } from '@hooks/useWatchSelector';
-import { useAppSelector } from '@state/redux/hooks';
-import {
-	ProfileConfigScheme,
-	selectProfile,
-	selectWorkspace,
-	selectWorkspacesInfo,
-	WorkspaceConfigScheme,
-} from '@state/redux/profiles/profiles';
+import { ProfileConfigScheme, selectProfile } from '@state/redux/profiles/profiles';
 import { createAppSelector } from '@state/redux/utils';
 
 import { useProfileControls } from '..';
@@ -44,32 +36,4 @@ export const useVaultConfigSync = () => {
 			},
 		});
 	}, [files, profileId, watchSelector]);
-
-	const workspaces = useAppSelector(selectWorkspacesInfo({ profileId }), isEqual);
-	useEffect(() => {
-		const cleanups = workspaces
-			.filter((workspace) => workspace.touched)
-			.map((workspace) => {
-				const workspaceConfig = new StateFile(
-					new FileController(`workspaces/${workspace.id}/config.json`, files),
-					WorkspaceConfigScheme,
-				);
-
-				return watchSelector({
-					selector: createAppSelector(
-						selectWorkspace({ profileId, workspaceId: workspace.id }),
-						(workspace) => workspace?.config,
-					),
-					onChange(config) {
-						if (!config) return;
-
-						workspaceConfig.set(config).then(() => {
-							console.debug('Workspace config is saved');
-						});
-					},
-				});
-			});
-
-		return () => cleanups.forEach((cleanup) => cleanup());
-	}, [files, profileId, watchSelector, workspaces]);
 };

src/features/App/WorkspaceManager/ProfileLoginForm/index.tsx → src/features/App/ProfileLoginForm/index.tsx

@@ -2,11 +2,11 @@ import React, { FC, useCallback, useEffect, useId, useState } from 'react';
 import { Button, Input, useToast, VStack } from '@chakra-ui/react';
 import { TELEMETRY_EVENT_NAME } from '@core/features/telemetry';
 import { ProfileObject } from '@core/storage/ProfilesManager';
+import { OnPickProfile } from '@features/App/useVaultOpener';
 import { useTelemetryTracker } from '@features/telemetry';
 import { useFocusableRef } from '@hooks/useFocusableRef';
 
 import { ProfilesForm } from '../ProfilesForm';
-import { OnPickProfile } from '..';
 
 export type ProfileLoginFormProps = {
 	profile: ProfileObject;

src/features/App/VaultScreenManager.tsx

@@ -0,0 +1,131 @@
+import React, { useMemo, useState } from 'react';
+import { FaUser } from 'react-icons/fa6';
+import { Button, Divider, HStack, Text } from '@chakra-ui/react';
+import { NestedList } from '@components/NestedList';
+import { TELEMETRY_EVENT_NAME } from '@core/features/telemetry';
+import { telemetry } from '@electron/requests/telemetry/renderer';
+import { getRandomItem } from '@utils/collections/getRandomItem';
+
+import { ProfileCreator } from './ProfileCreator';
+import { ProfileLoginForm } from './ProfileLoginForm';
+import { ProfilesForm } from './ProfilesForm';
+import { ProfilesListApi } from './useProfilesList';
+import { OnPickProfile } from './useVaultOpener';
+
+export type VaultScreenManagerProps = {
+	profiles: ProfilesListApi;
+	currentProfile: string | null;
+	onChooseProfile: (id: string | null) => void;
+	onOpenProfile: OnPickProfile;
+	onLoginStart?: () => void;
+};
+
+export const defaultVaultNames = [
+	'Creative drafts',
+	'Second brain',
+	'Digital garden',
+	'Creative space',
+	'Mind space',
+	'Idea lab',
+];
+
+export const VaultScreenManager = ({
+	currentProfile,
+	profiles,
+	onOpenProfile,
+	onChooseProfile,
+	onLoginStart,
+}: VaultScreenManagerProps) => {
+	const currentVaultObject = useMemo(
+		() => profiles.profiles?.find((profile) => profile.id === currentProfile) ?? null,
+		[currentProfile, profiles.profiles],
+	);
+
+	const [screen, setScreen] = useState<'create' | 'choose'>('choose');
+	const hasNoVaults = profiles.profiles.length === 0;
+
+	if (currentVaultObject) {
+		return (
+			<ProfileLoginForm
+				profile={currentVaultObject}
+				onLogin={async (...arg) => {
+					if (onLoginStart) onLoginStart();
+					return await onOpenProfile(...arg);
+				}}
+				onPickAnotherProfile={() => onChooseProfile(null)}
+			/>
+		);
+	}
+
+	if (screen === 'create' || hasNoVaults) {
+		return (
+			<ProfileCreator
+				onCreateProfile={(profile) =>
+					profiles.createProfile(profile).then((newProfile) => {
+						onOpenProfile(newProfile, profile.password ?? undefined).then(
+							console.warn,
+						);
+					})
+				}
+				onCancel={hasNoVaults ? undefined : () => setScreen('choose')}
+				defaultProfileName={
+					hasNoVaults ? getRandomItem(defaultVaultNames) : undefined
+				}
+			/>
+		);
+	}
+
+	return (
+		<ProfilesForm
+			title="Choose the profile"
+			controls={
+				<Button
+					variant="accent"
+					size="lg"
+					w="100%"
+					onClick={() => setScreen('create')}
+				>
+					Create new profile
+				</Button>
+			}
+		>
+			<NestedList
+				divider={<Divider margin="0px !important" />}
+				sx={{
+					w: '100%',
+					borderRadius: '4px',
+					maxHeight: '230px',
+					overflow: 'auto',
+					border: '1px solid',
+				}}
+				items={profiles.profiles.map((profile) => ({
+					id: profile.id,
+					content: (
+						<HStack
+							as="button"
+							key={profile.id}
+							sx={{
+								padding: '.8rem 1rem',
+								w: '100%',
+								cursor: 'pointer',
+								gap: '.8rem',
+							}}
+							onClick={() => {
+								onChooseProfile(profile.id);
+
+								if (profile.encryption === null) {
+									onOpenProfile(profile);
+								}
+
+								telemetry.track(TELEMETRY_EVENT_NAME.PROFILE_SELECTED);
+							}}
+						>
+							<FaUser />
+							<Text>{profile.name}</Text>
+						</HStack>
+					),
+				}))}
+			/>
+		</ProfilesForm>
+	);
+};