Skip to content

perf(app): reduce startup work — remove macOS calendar + defer providers #6597

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mdmohsin7 wants to merge 17 commits into
base: main
Choose a base branch
from
Open

perf(app): reduce startup work — remove macOS calendar + defer providers #6597

Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
469c867
refactor(app): remove macOS-only CalendarProvider
mdmohsin7 Apr 13, 2026
ef1ae2e
refactor(app): remove macOS-only CalendarService
mdmohsin7 Apr 13, 2026
5cc9720
refactor(app): remove orphaned calendar settings page
mdmohsin7 Apr 13, 2026
5ef0ab8
refactor(app): remove unused calendar_meetings HTTP client
mdmohsin7 Apr 13, 2026
0bc25dc
refactor(app): remove unused CalendarMeetingContext schema
mdmohsin7 Apr 13, 2026
2869c09
refactor(app): remove calendar preference keys
mdmohsin7 Apr 13, 2026
c62c462
refactor(app): remove dead CalendarProvider reference from CapturePro…
mdmohsin7 Apr 13, 2026
37896d9
refactor(app): remove unused calendarTypeChanged mixpanel event
mdmohsin7 Apr 13, 2026
cea3de8
perf(app): scope DeveloperModeProvider and McpProvider to developer s…
mdmohsin7 Apr 13, 2026
a12be78
perf(app): scope AiAppGeneratorProvider to its page
mdmohsin7 Apr 13, 2026
dc15a5c
perf(app): scope VoiceRecorderProvider to ChatPage
mdmohsin7 Apr 13, 2026
5d2df66
perf(app): scope PaymentMethodProvider to PaymentsPage
mdmohsin7 Apr 13, 2026
f972d1f
refactor(app): use local PaymentMethodProvider for one-shot check in …
mdmohsin7 Apr 13, 2026
87dd272
perf(app): defer non-critical providers at startup
mdmohsin7 Apr 13, 2026
d034f00
fix(app): unscope McpProvider from DeveloperSettingsPage
mdmohsin7 Apr 14, 2026
d0a8a1a
fix(app): unscope PaymentMethodProvider from PaymentsPage
mdmohsin7 Apr 14, 2026
d715ffa
fix(app): restore McpProvider and PaymentMethodProvider at root as lazy
mdmohsin7 Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
133 changes: 0 additions & 133 deletions app/lib/backend/http/api/calendar_meetings.dart
View file
Open in desktop

This file was deleted.

25 changes: 0 additions & 25 deletions app/lib/backend/preferences.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -547,31 +547,6 @@ class SharedPreferencesUtil {

bool get calendarEnabled => getBool('calendarEnabled');

set calendarId(String value) => saveString('calendarId', value);

String get calendarId => getString('calendarId');

set calendarType(String value) => saveString('calendarType2', value); // auto, manual (only for now)

String get calendarType => getString('calendarType2', defaultValue: 'manual');

set calendarIntegrationEnabled(bool value) => saveBool('calendarIntegrationEnabled', value);

bool get calendarIntegrationEnabled => getBool('calendarIntegrationEnabled');

// Calendar UI Settings
set showEventsWithNoParticipants(bool value) => saveBool('showEventsWithNoParticipants', value);

bool get showEventsWithNoParticipants => getBool('showEventsWithNoParticipants');

set showMeetingsInMenuBar(bool value) => saveBool('showMeetingsInMenuBar', value);

bool get showMeetingsInMenuBar => getBool('showMeetingsInMenuBar');

set enabledCalendarIds(List<String> value) => saveStringList('enabledCalendarIds', value);

List<String> get enabledCalendarIds => getStringList('enabledCalendarIds');

//--------------------------------- Auth ------------------------------------//

String get authToken => getString('authToken');
Expand Down
90 changes: 0 additions & 90 deletions app/lib/backend/schema/calendar_meeting_context.dart
View file
Open in desktop

This file was deleted.

37 changes: 8 additions & 29 deletions app/lib/main.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,24 +36,19 @@ import 'package:omi/flavors.dart';
import 'package:omi/l10n/app_localizations.dart';
import 'package:omi/pages/apps/providers/add_app_provider.dart';
import 'package:omi/pages/conversation_detail/conversation_detail_provider.dart';
import 'package:omi/pages/payments/payment_method_provider.dart';
import 'package:omi/pages/settings/ai_app_generator_provider.dart';
import 'package:omi/providers/action_items_provider.dart';
import 'package:omi/providers/announcement_provider.dart';
import 'package:omi/providers/app_provider.dart';
import 'package:omi/providers/auth_provider.dart';
import 'package:omi/providers/calendar_provider.dart';
import 'package:omi/providers/capture_provider.dart';
import 'package:omi/providers/connectivity_provider.dart';
import 'package:omi/providers/conversation_provider.dart';
import 'package:omi/providers/developer_mode_provider.dart';
import 'package:omi/providers/device_provider.dart';
import 'package:omi/providers/folder_provider.dart';
import 'package:omi/providers/goals_provider.dart';
import 'package:omi/providers/home_provider.dart';
import 'package:omi/providers/integration_provider.dart';
import 'package:omi/providers/locale_provider.dart';
import 'package:omi/providers/mcp_provider.dart';
import 'package:omi/providers/memories_provider.dart';
import 'package:omi/providers/message_provider.dart';
import 'package:omi/providers/onboarding_provider.dart';
Expand All @@ -63,7 +58,6 @@ import 'package:omi/providers/sync_provider.dart';
import 'package:omi/providers/task_integration_provider.dart';
import 'package:omi/providers/usage_provider.dart';
import 'package:omi/providers/user_provider.dart';
import 'package:omi/providers/voice_recorder_provider.dart';
import 'package:omi/providers/phone_call_provider.dart';
import 'package:omi/services/auth_service.dart';
import 'package:omi/services/notifications.dart';
Expand Down Expand Up @@ -291,13 +285,8 @@ class _MyAppState extends State<MyApp> with WidgetsBindingObserver {
update: (BuildContext context, value, MessageProvider? previous) =>
(previous?..updateAppProvider(value)) ?? MessageProvider(),
),
ChangeNotifierProxyProvider4<
ConversationProvider,
MessageProvider,
PeopleProvider,
UsageProvider,
CaptureProvider
>(
ChangeNotifierProxyProvider4<ConversationProvider, MessageProvider, PeopleProvider, UsageProvider,
CaptureProvider>(
create: (context) => CaptureProvider(),
update: (BuildContext context, conversation, message, people, usage, CaptureProvider? previous) =>
(previous?..updateProviderInstances(conversation, message, people, usage)) ?? CaptureProvider(),
Expand All @@ -323,36 +312,26 @@ class _MyAppState extends State<MyApp> with WidgetsBindingObserver {
update: (BuildContext context, app, conversation, ConversationDetailProvider? previous) =>
(previous?..setProviders(app, conversation)) ?? ConversationDetailProvider(),
),
ChangeNotifierProvider(create: (context) => DeveloperModeProvider()..initialize()),
ChangeNotifierProvider(create: (context) => McpProvider()),
ChangeNotifierProxyProvider<AppProvider, AddAppProvider>(
create: (context) => AddAppProvider(),
update: (BuildContext context, value, AddAppProvider? previous) =>
(previous?..setAppProvider(value)) ?? AddAppProvider(),
),
ChangeNotifierProxyProvider<AppProvider, AiAppGeneratorProvider>(
create: (context) => AiAppGeneratorProvider(),
update: (BuildContext context, value, AiAppGeneratorProvider? previous) =>
(previous?..setAppProvider(value)) ?? AiAppGeneratorProvider(),
),
ChangeNotifierProvider(create: (context) => PaymentMethodProvider()),
ChangeNotifierProxyProvider<ConnectivityProvider, MemoriesProvider>(
create: (context) => MemoriesProvider(),
update: (context, connectivity, previous) =>
(previous?..setConnectivityProvider(connectivity)) ?? MemoriesProvider(),
),
ChangeNotifierProvider(create: (context) => UserProvider()),
ChangeNotifierProvider(create: (context) => ActionItemsProvider()),
ChangeNotifierProvider(create: (context) => GoalsProvider()..init()),
ChangeNotifierProvider(lazy: true, create: (context) => ActionItemsProvider()),
ChangeNotifierProvider(lazy: true, create: (context) => GoalsProvider()..init()),
ChangeNotifierProvider(create: (context) => SyncProvider()),
ChangeNotifierProvider(create: (context) => TaskIntegrationProvider()),
ChangeNotifierProvider(create: (context) => IntegrationProvider()),
ChangeNotifierProvider(create: (context) => CalendarProvider(), lazy: false),
ChangeNotifierProvider(create: (context) => FolderProvider()),
ChangeNotifierProvider(lazy: true, create: (context) => TaskIntegrationProvider()),
ChangeNotifierProvider(lazy: true, create: (context) => IntegrationProvider()),
ChangeNotifierProvider(lazy: true, create: (context) => FolderProvider()),
ChangeNotifierProvider(create: (context) => LocaleProvider()),
ChangeNotifierProvider(create: (context) => VoiceRecorderProvider()..checkPendingRecording()),
ChangeNotifierProvider(create: (context) => AnnouncementProvider()),
ChangeNotifierProvider(create: (context) => PhoneCallProvider()),
ChangeNotifierProvider(lazy: true, create: (context) => PhoneCallProvider()),
],
builder: (context, child) {
return WithForegroundTask(
Expand Down
4 changes: 2 additions & 2 deletions app/lib/pages/apps/add_app.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -744,8 +744,8 @@ class _AddAppPageState extends State<AddAppPage> {
if (appId != null) {
app = await context.read<AppProvider>().getAppFromId(appId);
}
var paymentProvider = context.read<PaymentMethodProvider>();
paymentProvider.getPaymentMethodsStatus();
var paymentProvider = PaymentMethodProvider();
await paymentProvider.getPaymentMethodsStatus();

if (app != null && mounted && context.mounted) {
if (app.isPaid && paymentProvider.activeMethod == null) {
Expand Down
Loading
Loading