Skip to content

Conversation

@neooriginal
Copy link
Collaborator

Added proactive notification logic from the macos app to the web app. See below for technical details.

image image image image

This pull request introduces a new "Proactive Assistant" feature to the application, providing infrastructure for proactive notifications and monitoring using Gemini AI. The changes include environment variable setup, context management, UI components, and integration into the main layout and sidebar.

Proactive Assistant Feature Implementation:

  • Added a new context and provider (ProactiveContext, ProactiveProvider) to manage proactive assistant state, settings (including advice and focus assistants), and history, with localStorage persistence and legacy support.
  • Implemented a proactive monitoring widget (ProactiveMonitoringWidget) for user interaction, status display, error handling, and quick access to settings.
  • Integrated the proactive provider and widget into the main application layout, ensuring the feature is available throughout the app. [1] [2] [3]
  • Added a new "Proactive Assistant" entry to the sidebar menu with an appropriate icon for easy navigation. [1] [2]

Gemini API Integration:

  • Added a server action (analyzeScreenAction) to interact with the Gemini API for analyzing screen content, supporting image and prompt input, error handling, and extensibility for different models and response schemas.

Configuration and Environment:

  • Updated the example environment configuration file (env.local.example) to include required keys for Firebase, Gemini API, push notifications, and analytics, supporting the new proactive assistant and notification features.

gemini-code-assist[bot]

This comment was marked as outdated.

neooriginal and others added 3 commits January 28, 2026 19:51
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@neooriginal
Copy link
Collaborator Author

@gemini-code-assist re-review pls

gemini-code-assist[bot]

This comment was marked as outdated.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@neooriginal
Copy link
Collaborator Author

@gemini-code-assist re-review

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a significant new feature, the Proactive Assistant, by porting logic from the macOS app. The implementation is well-structured, with clear separation of concerns between server actions for Gemini API calls, context providers for state management, and React components for the UI. The use of a dedicated hook useProactiveNotifications to encapsulate the core logic is a great pattern.

I've identified two high-severity issues that should be addressed:

  1. A security vulnerability where the Gemini API key is exposed in the URL.
  2. A bug in the proactive settings widget UI that makes it non-functional.

Details and suggestions for fixes are in the review comments. Besides these points, the code is clean and the feature seems well-integrated into the existing application.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@neooriginal
Copy link
Collaborator Author

ready to be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant