Feat/obsidian plugin#9
Merged
Merged
Conversation
Implements Notion MCP integration core: - OAuth 2.0 discovery (RFC 9470 + 8414), dynamic client registration, PKCE token exchange, and refresh with rotation - NotionMCPAdapter connecting to mcp.notion.com/mcp with fallback to direct API on known serialization errors - Response parser translating MCP text responses into dicts matching NotionHistoryConnector output format - has_mcp_notion_connector() helper for connector gating
Fixes MODSetter#1248 handleAddTag had tags in its useCallback dependency array only so the closure-level duplicate check could read it, which forced the callback to re-create on every tag mutation and compared new additions against a potentially-stale closure value. Collapse the duplicate check into the functional setTags updater so the check always runs against the latest state, and drop tags from the dependency array - the callback is stable for the component's lifetime and downstream memoization won't get invalidated on every keystroke.
refactor: remove Azure OIDC token fetching steps from desktop release
…ag-functional fix(hitl-edit-panel): move duplicate-tag check into functional setTags (MODSetter#1248)
…and report panels
…esystem mode selection
…tkeys and update user settings dialog
[Improvement] MCP OAuth trust, 401 recovery, parallel discovery & connector UX
…ShortcutsContent component for better hotkey management
…ge with enhanced UI components
…virtual path handling in agent filesystem
…onents for consistency
… from Composer component
…t sidebar with local folder management features
…folder management and improve UI interactions
…resentation with text in DesktopShortcutsContent and login page
… LocalFilesystemBrowser for improved mount management
…stead of root paths, enhancing mount management and path normalization
…ddleware to handle Windows-style paths and mixed separators
…load refactor(anon-chat): dedupe upload via anonymousChatApiService
…ilesystem feat: introduce swappable filesystem on desktop & monaco editor to edit local files
…port panels - Updated styling for toggle buttons in DocumentsFilters for improved visual feedback. - Added toolbar actions prop to PdfViewer for customizable actions in the zoom toolbar. - Introduced download functionality in ReportPanel with a new Download icon. - Refactored ReportCard to remove word count and display version labels for better clarity. - Enhanced GenerateResume component to fetch and display version information for resumes.
…ng for improved UI consistency
- Changed plugin ID from "surfsense" to "surfsense-obsidian" for clarity. - Updated version from "0.1.1" to "0.1.0" in manifest and versions files. - Modified author URL to point to the official SurfSense website. - Adjusted GitHub release workflow to ensure compatibility with Obsidian and BRAT.
…ugin - Introduced an authentication block mechanism to prevent repeated invalid token submissions. - Updated notification logic to provide clearer feedback on API token status. - Refactored status visuals for better clarity and consistency in the user interface. - Improved connection handling in the sync engine to ensure robust error management.
…logic - Introduced a new `status-visuals.ts` file to centralize status icons and labels for consistency across the plugin. - Updated connection indicator logic in the settings tab to utilize the new centralized visuals. - Removed deprecated connection visual methods to streamline the codebase. - Enhanced error handling in the status bar to reflect the new status visuals structure.
…ugin - Changed initial status to "needs-setup" to prompt user configuration. - Added a refreshStatus method to SyncEngine for immediate status updates after settings changes. - Enhanced connection indicator logic in the settings tab to reflect the new status. - Introduced "needs-setup" status visual for improved user feedback on configuration requirements.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Motivation and Context
FIX #
Screenshots
API Changes
Change Type
Testing Performed
Checklist